Page MenuHomePhorge
Files F2694421

D25550.1734783276.diff
No OneTemporary
Actions

D25550.1734783276.diff
View Options

diff --git a/src/applications/maniphest/editor/ManiphestTransactionEditor.php b/src/applications/maniphest/editor/ManiphestTransactionEditor.php
--- a/src/applications/maniphest/editor/ManiphestTransactionEditor.php
+++ b/src/applications/maniphest/editor/ManiphestTransactionEditor.php
@@ -327,11 +327,11 @@
$is_unassigned = ($object->getOwnerPHID() === null);
- $any_assign = false;
+ $any_xassign = null;
foreach ($xactions as $xaction) {
if ($xaction->getTransactionType() ==
ManiphestTaskOwnerTransaction::TRANSACTIONTYPE) {
- $any_assign = true;
+ $any_xassign = $xaction;
break;
}
}
@@ -355,15 +355,24 @@
// If the task is not assigned, not being assigned, currently open, and
// being closed, try to assign the actor as the owner.
- if ($is_unassigned && !$any_assign && $is_open && $is_closing) {
- $is_claim = ManiphestTaskStatus::isClaimStatus($new_status);
-
- // Don't assign the actor if they aren't a real user.
- // Don't claim the task if the status is configured to not claim.
- if ($actor_phid && $is_claim) {
- $results[] = id(new ManiphestTransaction())
- ->setTransactionType(ManiphestTaskOwnerTransaction::TRANSACTIONTYPE)
- ->setNewValue($actor_phid);
+ // Don't assign the actor if they aren't a real user.
+ if ($is_unassigned && $is_open && $is_closing && $actor_phid) {
+ $is_autoclaim = ManiphestTaskStatus::isClaimStatus($new_status);
+ if ($is_autoclaim) {
+ if ($any_xassign === null) {
+ $results[] = id(new ManiphestTransaction())
+ ->setTransactionType(ManiphestTaskOwnerTransaction::TRANSACTIONTYPE)
+ ->setNewValue($actor_phid);
+ } else if ($any_xassign->getNewValue() === null) {
+ // We have an explicit "Assign / Claim" = nothing in the frontend.
+ // The user is trying to "undo" the above automatic auto-claim.
+ // When saving, this would cause the "no effect" warning.
+ // So we suppress that confusing warning.
+ // https://we.phorge.it/T15164
+ if ($any_xassign->getNewValue() === null) {
+ $any_xassign->setIgnoreOnNoEffect(true);
+ }
+ }
}
}

File Metadata

Mime Type
text/plain
Expires
Sat, Dec 21, 12:14 (18 h, 9 m)
Storage Engine
blob
Storage Format
Raw Data
Storage Handle
1024624
Default Alt Text
D25550.1734783276.diff (2 KB)

Event Timeline

Content licensed under Creative Commons Attribution-ShareAlike 4.0 (CC-BY-SA) unless otherwise noted; code licensed under Apache 2.0 or other open source licenses. · CC BY-SA 4.0 · Apache 2.0