6.0.0-alpha14
7/3/25

[#14699] Errors with new code to handle iCal export of categories/tags
Summary Errors with new code to handle iCal export of categories/tags
Queue Nag
Queue Version 4.2.17
Type Bug
State Resolved
Priority 2. Medium
Owners jan (at) horde (dot) org
Requester wahnes (at) uni-koeln (dot) de
Created 09/26/2017 (2837 days ago)
Due
Updated 10/20/2017 (2813 days ago)
Assigned
Resolved 09/26/2017 (2837 days ago)
Github Issue Link
Github Pull Request
Milestone 4.2.18
Patch No

History
10/20/2017 08:34:03 PM Git Commit Comment #7 Reply to this comment
Changes have been made in Git (FRAMEWORK_5_2):

commit 7162f4663299e2fe848f2c69a1f9e1e30f9b513f
Author: Jan Schneider <jan@horde.org>
Date:   Tue, 26 Sep 2017 15:35:54 +0200

[jan] Fix notice when re-exporting tasks without priority (Bug #14699).

  M docs/CHANGES
  M lib/Task.php
  M package.xml

https://github.com/horde/nag/commit/7162f4663299e2fe848f2c69a1f9e1e30f9b513f
10/20/2017 08:34:03 PM Git Commit Comment #6 Reply to this comment
Changes have been made in Git (FRAMEWORK_5_2):

commit 4dbbd34f898605fb79bf46cb10f21edf859f3790
Author: Jan Schneider <jan@horde.org>
Date:   Tue, 26 Sep 2017 15:35:18 +0200

[jan] Fix regression in exporting single tags to iCalendar CATEGORIES 
(Bug #14699).

  M docs/CHANGES
  M lib/Task.php
  M package.xml

https://github.com/horde/nag/commit/4dbbd34f898605fb79bf46cb10f21edf859f3790
09/26/2017 01:53:15 PM Git Commit Comment #5 Reply to this comment
Changes have been made in Git (master):

commit 0f3486d073f69c4036ead2fb22a7ba744cac3ad8
Author: Jan Schneider <jan@horde.org>
Date:   Tue Sep 26 15:35:54 2017 +0200

     [jan] Fix notice when re-exporting tasks without priority (Bug #14699).

  nag/docs/CHANGES | 9 +++++++++
  nag/lib/Task.php | 1 +
  2 files changed, 10 insertions(+)

http://github.com/horde/horde/commit/0f3486d073f69c4036ead2fb22a7ba744cac3ad8
09/26/2017 01:53:15 PM Git Commit Comment #4 Reply to this comment
Changes have been made in Git (master):

commit be530a444770eab6010ccac622472948c6504500
Author: Jan Schneider <jan@horde.org>
Date:   Tue Sep 26 15:35:18 2017 +0200

     [jan] Fix regression in exporting single tags to iCalendar 
CATEGORIES (Bug #14699).

  nag/lib/Task.php |  6 +-----
  nag/package.xml  | 16 +++++++++++++++-
  2 files changed, 16 insertions(+), 6 deletions(-)

http://github.com/horde/horde/commit/be530a444770eab6010ccac622472948c6504500
09/26/2017 01:37:17 PM Jan Schneider Assigned to Jan Schneider
State ⇒ Resolved
Milestone ⇒ 4.2.18
 
09/26/2017 01:36:14 PM Git Commit Comment #3 Reply to this comment
Changes have been made in Git (FRAMEWORK_5_2):

commit e740a718580ca392ae7d3a28da09e6b7e3219c75
Author: Jan Schneider <jan@horde.org>
Date:   Tue Sep 26 15:35:54 2017 +0200

     [jan] Fix notice when re-exporting tasks without priority (Bug #14699).

  nag/docs/CHANGES | 1 +
  nag/lib/Task.php | 1 +
  nag/package.xml  | 2 ++
  3 files changed, 4 insertions(+)

http://github.com/horde/horde/commit/e740a718580ca392ae7d3a28da09e6b7e3219c75
09/26/2017 01:36:13 PM Git Commit Comment #2 Reply to this comment
Changes have been made in Git (FRAMEWORK_5_2):

commit d97ca159549f2ab4342ce5f4b9cdaac1c6bfebb2
Author: Jan Schneider <jan@horde.org>
Date:   Tue Sep 26 15:35:18 2017 +0200

     [jan] Fix regression in exporting single tags to iCalendar 
CATEGORIES (Bug #14699).

  nag/docs/CHANGES | 2 ++
  nag/lib/Task.php | 6 +-----
  nag/package.xml  | 4 ++--
  3 files changed, 5 insertions(+), 7 deletions(-)

http://github.com/horde/horde/commit/d97ca159549f2ab4342ce5f4b9cdaac1c6bfebb2
09/26/2017 10:24:18 AM wahnes (at) uni-koeln (dot) de Comment #1
Priority ⇒ 2. Medium
Type ⇒ Bug
Summary ⇒ Errors with new code to handle iCal export of categories/tags
Queue ⇒ Nag
Milestone ⇒
Patch ⇒ No
State ⇒ Unconfirmed
Reply to this comment
There are two problems introduced with version 4.2.17.

The first is very similar to bug #14698 and concerns the code change at

https://github.com/horde/horde/commit/72560bfc1e01282cec6e9c8fdbdebfbe39a5809f#diff-c0ec275af3420bccfd7fe6f8de93f277

This code has got the right parathesis at the wrong spot as well and 
uses "$this->tags[0]" which is not correct. See #14698 for more details.

The second problem is a different one but has also been introducted 
with version 4.2.17. It leads to error messages in the log file like 
this:

HORDE [nag] PHP ERROR: Undefined offset: 0 [pid 11411 on line 1245 of 
"[...]/nag/lib/Task.php"]

It seems that the "$priorityMap" might be missing an entry at index 0, 
i.e. "0 => 0,", but I'm not sure that adding a line to that effect 
around line 1239 of Task.php would be a proper fix because I don't 
understand what those magic values do.

Saved Queries