drush_choice_multiple seems broken when $defaults is FALSE and $max is not set. The problem code seems to be here:
// If the user selected too many options, drop the oldest selection.
if (count($selections) > $max) {
array_pop($selections);
}
Should be:
if (isset($max) && count($selections) > $max) {
array_pop($selections);
}
Otherwise, every selection the user makes then gets unset again in this code.
| Comment | File | Size | Author |
|---|---|---|---|
| #1 | drush-1154132-drush_choice_multiple-1.patch | 842 bytes | q0rban |
Comments
Comment #1
q0rban commentedComment #2
q0rban commentedComment #3
kostajh commentedThis works great, thanks for the patch!
Comment #4
moshe weitzman commentedCommitted to master and 4.x. Sorry this sat for so long. I just happenned upon it.