Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fixing scope issue and correcting date/time #4

Merged
merged 1 commit into from Nov 13, 2017
Merged

Conversation

@dcepler
Copy link
Contributor

@dcepler dcepler commented Oct 27, 2017

Issue running this on newer version of ColdFusion (2016 and probably 11) due to scoping issue which was fixed my explicitly using local.

Modified date/time fields to include date and time not just date. Also fixed the GMT fields so they are correctly calculated as well.

Issue running this on newer version of ColdFusion (2016 and probably 11) due to scoping issue.
* Fixed using `local` scope.

Modified date/time fields to include date and time not just date. Also fixed the GMT fields so they are correctly calculated as well.
@dcepler dcepler changed the title Fixing `local` scope issue and correcting date/time Fixing scope issue and correcting date/time Oct 27, 2017
@mplacona mplacona merged commit 45d22ba into mplacona:master Nov 13, 2017
@mplacona
Copy link
Owner

@mplacona mplacona commented Nov 13, 2017

I haven't looked at this project for many many years now, so glad it still kinda works.

Thanks for the PR!

@dcepler
Copy link
Contributor Author

@dcepler dcepler commented Nov 13, 2017

You're welcome for the PR.

Thanks for building it. Saved me quite a bit of time.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants
You can’t perform that action at this time.