Tickets :: [#13757] nextFreebusy/prevFreebusy not working if french date

6.0.0-beta1

9/2/25

Summary	nextFreebusy/prevFreebusy not working if french date
Queue	Kronolith
Queue Version	4.2.3
Type	Bug
State	Resolved
Priority	2. Medium
Owners	jan (at) horde (dot) org
Requester	pascal.rigaux (at) univ-paris1 (dot) fr
Created	12/15/2014 (3914 days ago)
Due
Updated	12/19/2014 (3910 days ago)
Assigned
Resolved	12/19/2014 (3910 days ago)
Github Issue Link
Github Pull Request
Milestone
Patch	Yes

12/19/2014 10:13:10 AM	Git Commit	Comment #6	Reply to this comment
Changes have been made in Git (FRAMEWORK_5_2): commit 3efc5e4038a10501a627ff55f505f6ef58786b5a Author: Jan Schneider <jan@horde.org> Date: Fri Dec 19 10:58:33 2014 +0100 Simplify, coding style (~~Bug #13757~~). kronolith/js/kronolith.js \| 12 +++++------- 1 files changed, 5 insertions(+), 7 deletions(-) http://github.com/horde/horde/commit/3efc5e4038a10501a627ff55f505f6ef58786b5a

12/19/2014 10:13:04 AM	Git Commit	Comment #5	Reply to this comment
Changes have been made in Git (FRAMEWORK_5_2): commit bd8ef95ff2a190e2e6638a9863add7f37dbe8677 Author: Pascal Rigaux <pascal-rigaux@univ-paris1.fr> Date: Mon Dec 15 09:49:31 2014 +0100 fix nextFreebusy/prevFreebusy Signed-off-by: Jan Schneider <jan@horde.org> ~~Bug: 13757~~ kronolith/js/kronolith.js \| 41 ++++++++++++++++++++++------------------- 1 files changed, 22 insertions(+), 19 deletions(-) http://github.com/horde/horde/commit/bd8ef95ff2a190e2e6638a9863add7f37dbe8677

12/19/2014 09:59:33 AM	Jan Schneider	Assigned to Jan Schneider State ⇒ Resolved

12/19/2014 09:58:49 AM	Git Commit	Comment #4	Reply to this comment
Changes have been made in Git (master): commit edfca503014abc815c843dfd56d957e24471dd99 Author: Jan Schneider <jan@horde.org> Date: Fri Dec 19 10:58:33 2014 +0100 Simplify, coding style (~~Bug #13757~~). kronolith/js/kronolith.js \| 12 +++++------- 1 files changed, 5 insertions(+), 7 deletions(-) http://github.com/horde/horde/commit/edfca503014abc815c843dfd56d957e24471dd99

12/19/2014 09:58:42 AM	Git Commit	Comment #3	Reply to this comment
Changes have been made in Git (master): commit 3bf1837e2cf4baf7d79b9163b651e9c2b3929a2f Author: Pascal Rigaux <pascal-rigaux@univ-paris1.fr> Date: Mon Dec 15 09:49:31 2014 +0100 fix nextFreebusy/prevFreebusy Signed-off-by: Jan Schneider <jan@horde.org> ~~Bug: 13757~~ kronolith/js/kronolith.js \| 41 ++++++++++++++++++++++------------------- 1 files changed, 22 insertions(+), 19 deletions(-) http://github.com/horde/horde/commit/3bf1837e2cf4baf7d79b9163b651e9c2b3929a2f

12/15/2014 03:30:50 PM	pascal (dot) rigaux (at) univ-paris1 (dot) fr	Comment #2 New Attachment: 0001-fix-nextFreebusy-prevFreebusy.patch	Reply to this comment
Add the fix.

12/15/2014 03:29:15 PM	pascal (dot) rigaux (at) univ-paris1 (dot) fr	Comment #1 Priority ⇒ 2. Medium Type ⇒ Bug Summary ⇒ nextFreebusy/prevFreebusy not working if french date Queue ⇒ Kronolith Milestone ⇒ Patch ⇒ Yes State ⇒ Unconfirmed	Reply to this comment
In dynamic interface, nextFreebusy/prevFreeBusy are doing Date.parse($('kronolithFBDate').innerHTML) which can't work if Kronolith.conf.date_format is not default js date. "insertFreeBusy" is doing the right job : removing the day of week + Date.parseExact with Kronolith.conf.date_format The attached patch is quite straight-forward : it creates a function to do the job and uses it (the date passed around is no more the unparsed string, but a Date js object) (patch tested on 4.2.3 but applies cleanly on git HEAD)