6.0.0-beta1
8/23/25

[#13837] nag add new task does not check show perms
Summary nag add new task does not check show perms
Queue Nag
Queue Version 4.2.3
Type Bug
State Resolved
Priority 1. Low
Owners jan (at) horde (dot) org
Requester michael.menge (at) zdv (dot) uni-tuebingen (dot) de
Created 02/02/2015 (3855 days ago)
Due
Updated 12/15/2015 (3539 days ago)
Assigned
Resolved 02/06/2015 (3851 days ago)
Github Issue Link
Github Pull Request
Milestone
Patch No

History
12/15/2015 04:31:00 AM Git Commit Comment #5 Reply to this comment
Changes have been made in Git (master):

commit 7c7903ae05bd252235c4130491290e04cfc8a1b4
Author: Michael J Rubinsky <mrubinsk@horde.org>
Date:   Mon Dec 14 23:26:28 2015 -0500

     Fix regression introduced with fix for Bug: 13837.

     The share list used when building the task form must match
     the share list used when determining the tasklist to save
     a task to. Otherwise, the form may see only 1 tasklist available
     (thus not including the tasklist_id), but the save controller may
     see more than 1 (thus expecting the tasklist_id).

  nag/app/controllers/SaveTask.php |   22 +++++++++++++++++++++-
  1 files changed, 21 insertions(+), 1 deletions(-)

http://github.com/horde/horde/commit/7c7903ae05bd252235c4130491290e04cfc8a1b4
12/15/2015 04:28:47 AM Git Commit Comment #4 Reply to this comment
Changes have been made in Git (FRAMEWORK_5_2):

commit 741bb60aef4f96f6e947dd6a8ad27648b86cf69b
Author: Michael J Rubinsky <mrubinsk@horde.org>
Date:   Mon Dec 14 23:26:28 2015 -0500

     Fix regression introduced with fix for Bug: 13837.

     The share list used when building the task form must match
     the share list used when determining the tasklist to save
     a task to. Otherwise, the form may see only 1 tasklist available
     (thus not including the tasklist_id), but the save controller may
     see more than 1 (thus expecting the tasklist_id).

  nag/app/controllers/SaveTask.php |   22 +++++++++++++++++++++-
  1 files changed, 21 insertions(+), 1 deletions(-)

http://github.com/horde/horde/commit/741bb60aef4f96f6e947dd6a8ad27648b86cf69b
02/06/2015 08:50:12 PM Git Commit Comment #3 Reply to this comment
Changes have been made in Git (FRAMEWORK_5_2):

commit e65944b81dcbab8ec6db8f83d713eb5a7d3aac16
Author: Jan Schneider <jan@horde.org>
Date:   Fri Feb 6 21:39:32 2015 +0100

     Check for SHOW permissions (Bug #13837).

  nag/lib/Application.php |    6 +++++-
  nag/lib/Form/Task.php   |   18 ++++++++++++------
  2 files changed, 17 insertions(+), 7 deletions(-)

http://github.com/horde/horde/commit/e65944b81dcbab8ec6db8f83d713eb5a7d3aac16
02/06/2015 08:45:52 PM Jan Schneider Assigned to Jan Schneider
State ⇒ Resolved
 
02/06/2015 08:44:58 PM Git Commit Comment #2 Reply to this comment
Changes have been made in Git (master):

commit c06481541ea86fb14ca790e5392e4f788b14bec0
Author: Jan Schneider <jan@horde.org>
Date:   Fri Feb 6 21:39:32 2015 +0100

     Check for SHOW permissions (Bug #13837).

  nag/lib/Application.php |    6 +++++-
  nag/lib/Form/Task.php   |   18 ++++++++++++------
  2 files changed, 17 insertions(+), 7 deletions(-)

http://github.com/horde/horde/commit/c06481541ea86fb14ca790e5392e4f788b14bec0
02/02/2015 11:01:54 AM michael (dot) menge (at) zdv (dot) uni-tuebingen (dot) de Comment #1
Priority ⇒ 1. Low
Type ⇒ Bug
Summary ⇒ nag add new task does not check show perms
Queue ⇒ Nag
Milestone ⇒
Patch ⇒ No
State ⇒ Unconfirmed
Reply to this comment
the task list drop down in the add new task dialog
shows task lists where i have read permission
(via prem_creator4 = true) but no show permission
i

Saved Queries