diff options
| author | Malf Furious <m@lfurio.us> | 2018-11-10 09:21:54 -0500 |
|---|---|---|
| committer | Malf Furious <m@lfurio.us> | 2018-11-10 09:21:54 -0500 |
| commit | be18c21941eb1309047e5cede3467cc538e71559 (patch) | |
| tree | 663bfdbd005399422299f25b48b8f35de95a5530 | |
| parent | 762208664a9173bc9507a66fbd0c8020e382db88 (diff) | |
| download | scrott-be18c21941eb1309047e5cede3467cc538e71559.tar.gz scrott-be18c21941eb1309047e5cede3467cc538e71559.zip | |
Fix bug in assignee selection
Previously, issues on pads owned by a group (rather than by a user)
would incorrectly present options for user assignment. It would show
the group as a member of the list, misleadingly with the user icon to
the left of it.
This patch uses the new agent::getContainedUsers() function to resolve
the pad's owner to an array of users accessible via the owner 'agent'.
Signed-off-by: Malf Furious <m@lfurio.us>
| -rw-r--r-- | app/view/issue.php | 8 |
1 files changed, 5 insertions, 3 deletions
diff --git a/app/view/issue.php b/app/view/issue.php index 6edf7cf..01a9f5f 100644 --- a/app/view/issue.php +++ b/app/view/issue.php @@ -138,9 +138,11 @@ require_once "class/issue.class.php"; <label>Select assignee</label> <select name="input[assignee]" class="form-control selectpicker"> - <option data-icon="glyphicon-user" value="<?=$i->getParent()->getParent()->getOwner()->guid?>"> - <?=$i->getParent()->getParent()->getOwner()->getDisplayName()?> - </option> + <?php foreach ($i->getParent()->getParent()->getOwner()->getContainedUsers() as $memb) { ?> + <option data-icon="glyphicon-user" value="<?=$memb->guid?>"> + <?=$memb->getDisplayName()?> + </option> + <?php } ?> <?php foreach ($i->getParent()->getParent()->getMembers() as $memb) { ?> <option data-icon="glyphicon-user" value="<?=$memb->guid?>"> |