Skip to content

Fix bug #65003 Wrong date diff #973

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

Closed
wants to merge 3 commits into from
Closed

Fix bug #65003 Wrong date diff #973

wants to merge 3 commits into from

Conversation

pasindud
Copy link

@pasindud pasindud commented Jan 1, 2015

https://bugs.php.net/bug.php?id=65003

Fixed by only timelib_apply_localtime when zone is different, current date test passes. Hope this is correct.

pasindud and others added 2 commits January 1, 2015 08:19
timelib_apply_localtime when only zones are different
@pasindud pasindud changed the title Fix #65003 Wrong date diff Fix bug #65003 Wrong date diff Jan 1, 2015
@pasindud
Copy link
Author

pasindud commented Jan 2, 2015

Hi @derickr can check whether this is correct

@derickr
Copy link
Member

derickr commented Jan 2, 2015

I'm going to need to sit down when I'm back to my computer to have a look in a few days. Please ping me next week if i forget!

@smalyshev
Copy link
Contributor

@derickr ping? :)

@derickr
Copy link
Member

derickr commented Feb 1, 2015

I don't think it's correct, but haven't had the time yet... This logic is very sensitive, and, I have party tried fixing this in an non-php copy. Just bare with me.

@krakjoe
Copy link
Member

krakjoe commented Jan 3, 2017

@derickr can we know the status of this issue please ?

@krakjoe
Copy link
Member

krakjoe commented Feb 3, 2017

Having waited a month for feedback, and since changes to timelib need to land upstream, I'm closing this PR.

Please take this action as encouragement to open a PR on the upstream library, and when/if that change lands, to open a new test PR on php-src.

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

Successfully merging this pull request may close these issues.

4 participants