6.0.0-beta1
9/2/25

[#12653] Missing timezone in setTimezone call
Summary Missing timezone in setTimezone call
Queue Kronolith
Queue Version Git master
Type Bug
State Resolved
Priority 2. Medium
Owners mrubinsk (at) horde (dot) org
Requester ctimoteo (at) sapo (dot) pt
Created 09/03/2013 (4382 days ago)
Due
Updated 12/17/2013 (4277 days ago)
Assigned
Resolved 09/04/2013 (4381 days ago)
Github Issue Link
Github Pull Request
Milestone
Patch Yes

History
12/17/2013 11:35:32 AM Git Commit Comment #4 Reply to this comment
Changes have been made in Git (master):

commit b788f40a75e486dce2812dcfcc436eb86cfb9744
Author: Michael J Rubinsky <mrubinsk@horde.org>
Date:   Wed Sep 4 11:13:55 2013 -0400

     Bug: 12653 Always set $tz_local

  kronolith/lib/Event/Sql.php |    2 +-
  1 files changed, 1 insertions(+), 1 deletions(-)

http://git.horde.org/horde-git/-/commit/b788f40a75e486dce2812dcfcc436eb86cfb9744
09/04/2013 03:15:34 PM Michael Rubinsky Comment #3
Assigned to Michael Rubinsky
State ⇒ Resolved
Reply to this comment
Fixed, in a slightly simpler fashion.

Thanks for the report.
09/04/2013 03:14:53 PM Git Commit Comment #2 Reply to this comment
Changes have been made in Git (master):

commit b9a0adbe6cfca245387a4f20f9bbb0788c312a7f
Author: Michael J Rubinsky <mrubinsk@horde.org>
Date:   Wed Sep 4 11:13:55 2013 -0400

     Bug: 12653 Always set $tz_local

  kronolith/lib/Event/Sql.php |    2 +-
  1 files changed, 1 insertions(+), 1 deletions(-)

http://git.horde.org/horde-git/-/commit/b9a0adbe6cfca245387a4f20f9bbb0788c312a7f
09/03/2013 01:58:37 PM ctimoteo (at) sapo (dot) pt Comment #1
Priority ⇒ 2. Medium
New Attachment: 0001-Check-valid-timezone.patch Download
Patch ⇒ Yes
Milestone ⇒
Queue ⇒ Kronolith
Summary ⇒ Missing timezone in setTimezone call
Type ⇒ Bug
State ⇒ Unconfirmed
Reply to this comment
After last update i tried to export some calendar events,

i get one fatal error in class Horde_Date, function setTimezone 
(framework/Date/lib/Horde/Date.php, line 532)

after debug, i notice that in commit:

https://github.com/horde/horde/commit/fca899cecfd8c8cf8a549d6fdd183e1068307808

the commit added the call to function setTimezone but in my case the 
variable tz_local was not set, (kronolith/lib/Event/Sql.php, line 152)

i send a patch to check variable and get default system timezone if not set.

--
Carlos Timóteo

Saved Queries