| Summary | horde-set-perms "Are you sure"-Check fails |
| Queue | Horde Base |
| Queue Version | Git master |
| Type | Bug |
| State | Resolved |
| Priority | 1. Low |
| Owners | jan (at) horde (dot) org |
| Requester | Olbrichj (at) students (dot) uni-marburg (dot) de |
| Created | 02/28/2014 (4122 days ago) |
| Due | |
| Updated | 03/03/2014 (4119 days ago) |
| Assigned | |
| Resolved | 03/03/2014 (4119 days ago) |
| Github Issue Link | |
| Github Pull Request | |
| Milestone | |
| Patch | Yes |
commit da63e040a416ee880ff0dff49553358134453830
Author: Jan Schneider <jan@horde.org>
Date: Mon Mar 3 10:31:03 2014 +0100
Fix logic (
Bug #13001).horde/bin/horde-set-perms | 2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
http://git.horde.org/horde-git/-/commit/da63e040a416ee880ff0dff49553358134453830
Assigned to Jan Schneider
State ⇒ Resolved
bug #12994commit 21297155d08207ebb2aa49f5e80a9790a378aca2
Author: Jan Schneider <jan@horde.org>
Date: Mon Mar 3 10:31:03 2014 +0100
Fix logic (
Bug #13001).horde/bin/horde-set-perms | 2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
http://git.horde.org/horde-git/-/commit/21297155d08207ebb2aa49f5e80a9790a378aca2
Priority ⇒ 1. Low
New Attachment: set-perms-patch
Patch ⇒ Yes
Milestone ⇒
Queue ⇒ Horde Base
Summary ⇒ horde-set-perms "Are you sure"-Check fails
Type ⇒ Bug
State ⇒ Unconfirmed
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.