Tickets :: [#14507] again regression with due dates

6.0.0-RC7

6/21/26

History
Attachments
Comment
Watch
Download

Summary	again regression with due dates
Queue	Nag
Queue Version	4.2.12
Type	Bug
State	Resolved
Priority	3. High
Owners	jan (at) horde (dot) org
Requester	Klaus.Steinberger (at) physik (dot) uni-muenchen (dot) de
Created	11/9/16 (3511 days ago)
Due
Updated	10/20/17 (3166 days ago)
Assigned	11/9/16 (3511 days ago)
Resolved	11/9/16 (3511 days ago)
Github Issue Link
Github Pull Request
Milestone	4.2.13
Patch	No

578	Git Commit	Comment #7	Reply to this comment
Changes have been made in Git (FRAMEWORK_5_2): commit 23732624b10a96fbb6af94490c67503d0a7869f3 Author: Jan Schneider <jan@horde.org> Date: Wed, 09 Nov 2016 14:15:54 +0100 [jan] Fix sorting by due date with PHP < 5.5 (~~Bug #14507~~). Fixes regression introduced with fix for ~~bug 14504~~. M docs/CHANGES M lib/Nag.php M package.xml https://github.com/horde/nag/commit/23732624b10a96fbb6af94490c67503d0a7869f3

424	Git Commit	Comment #6	Reply to this comment
Changes have been made in Git (master): commit 6f99b447977f429e6aee16d5fae7142dc50e30ea Author: Jan Schneider <jan@horde.org> Date: Wed Nov 9 14:15:44 2016 +0100 [jan] Fix sorting by due date with PHP < 5.5 (~~Bug #14507~~). Fixes regression introduced with fix for ~~bug 14504~~. nag/docs/CHANGES \| 8 ++++++++ nag/lib/Nag.php \| 22 +++++++++++----------- nag/package.xml \| 17 +++++++++++++++-- 3 files changed, 34 insertions(+), 13 deletions(-) http://github.com/horde/horde/commit/6f99b447977f429e6aee16d5fae7142dc50e30ea

443	Jan Schneider	State ⇒ Resolved Milestone ⇒ 4.2.13

552	Klaus (dot) Steinberger (at) physik (dot) uni-muenchen (dot) de	Comment #5	Reply to this comment
Fine! That works.

501	Jan Schneider	Comment #4 Assigned to Jan Schneider State ⇒ Feedback	Reply to this comment
Try this.

391	Git Commit	Comment #3	Reply to this comment
Changes have been made in Git (FRAMEWORK_5_2): commit 00ec780f20c6e3cd8a895c22993bc7fa16a6cfd7 Author: Jan Schneider <jan@horde.org> Date: Wed Nov 9 14:15:44 2016 +0100 [jan] Fix sorting by due date with PHP < 5.5 (~~Bug #14507~~). Fixes regression introduced with fix for ~~bug 14504~~. nag/docs/CHANGES \| 1 + nag/lib/Nag.php \| 22 +++++++++++----------- nag/package.xml \| 4 ++-- 3 files changed, 14 insertions(+), 13 deletions(-) http://github.com/horde/horde/commit/00ec780f20c6e3cd8a895c22993bc7fa16a6cfd7

3211	Klaus (dot) Steinberger (at) physik (dot) uni-muenchen (dot) de	Comment #2 New Attachment: nag.patch	Reply to this comment
I think this patch solves it

3311	Klaus (dot) Steinberger (at) physik (dot) uni-muenchen (dot) de	Comment #1 Priority ⇒ 3. High State ⇒ Unconfirmed Patch ⇒ No Milestone ⇒ Queue ⇒ Nag Summary ⇒ again regression with due dates Type ⇒ Bug	Reply to this comment
nag 4.2.12 throws an fatal error in line 1538 nag/lib/Nag.php: PHP Fatal error: Can't use method return value in write context in /var/www/html/horde/nag/lib/Nag.php on line 1538, referer: https://testwebmail.physik.uni-muenchen.de/horde/login.php See also: https://www.davidtan.org/php-solution-for-cant-use-method-return-value-in-write-context/