[Tickets #13001] horde-set-perms "Are you sure"-Check fails

[noreply at bugs.horde.org]

DO NOT REPLY TO THIS MESSAGE. THIS EMAIL ADDRESS IS NOT MONITORED.

Ticket URL: http://bugs.horde.org/ticket/13001
------------------------------------------------------------------------------
  Ticket             | 13001
  Created By         | Olbrichj at students.uni-marburg.de
  Summary            | horde-set-perms "Are you sure"-Check fails
  Queue              | Horde Base
  Version            | Git master
  Type               | Bug
  State              | Unconfirmed
  Priority           | 1. Low
  Milestone          |
  Patch              | 1
  Owners             |
------------------------------------------------------------------------------


Olbrichj at students.uni-marburg.de (2014-02-28 08:47) wrote:

Line 53 and the following are:

if (!$cli->prompt($cli->red('Are you sure you want to alter  
permissions?'), array('y' => 'Yes', 'n' => 'No'), 'n') == 'y') {
     exit(0);
}

and the body of the if never gets executed because ! $cli->prompt...  
is never 'y'. The attached patch adds the missing parentheses.



Olbrichj at students.uni-marburg.de (2014-02-28 08:47) uploaded: set-perms-patch

http://bugs.horde.org/h/services/download/?app=whups&actionID=download_file&file=set-perms-patch&ticket=13001&fn=%2Fset-perms-patch