Page MenuHomePhabricator
Files F15394763

D8622.id20456.diff
No OneTemporary
Actions

D8622.id20456.diff
View Options

diff --git a/resources/celerity/map.php b/resources/celerity/map.php
--- a/resources/celerity/map.php
+++ b/resources/celerity/map.php
@@ -404,7 +404,7 @@
'rsrc/js/application/policy/behavior-policy-control.js' => 'c01153ea',
'rsrc/js/application/policy/behavior-policy-rule-editor.js' => '263aeb8c',
'rsrc/js/application/ponder/behavior-votebox.js' => '327dbe61',
- 'rsrc/js/application/projects/behavior-project-boards.js' => '04c59172',
+ 'rsrc/js/application/projects/behavior-project-boards.js' => 'd8e135db',
'rsrc/js/application/projects/behavior-project-create.js' => '065227cc',
'rsrc/js/application/releeph/releeph-preview-branch.js' => '9eb2cedb',
'rsrc/js/application/releeph/releeph-request-state-change.js' => 'fe7fc914',
@@ -614,7 +614,7 @@
'javelin-behavior-policy-control' => 'c01153ea',
'javelin-behavior-policy-rule-editor' => '263aeb8c',
'javelin-behavior-ponder-votebox' => '327dbe61',
- 'javelin-behavior-project-boards' => '04c59172',
+ 'javelin-behavior-project-boards' => 'd8e135db',
'javelin-behavior-project-create' => '065227cc',
'javelin-behavior-refresh-csrf' => 'c4b31646',
'javelin-behavior-releeph-preview-branch' => '9eb2cedb',
@@ -843,15 +843,6 @@
3 => 'javelin-vector',
4 => 'javelin-install',
),
- '04c59172' =>
- array(
- 0 => 'javelin-behavior',
- 1 => 'javelin-dom',
- 2 => 'javelin-util',
- 3 => 'javelin-stratcom',
- 4 => 'javelin-workflow',
- 5 => 'phabricator-draggable-list',
- ),
'065227cc' =>
array(
0 => 'javelin-behavior',
@@ -1752,6 +1743,15 @@
3 => 'javelin-dom',
4 => 'phabricator-keyboard-shortcut',
),
+ 'd8e135db' =>
+ array(
+ 0 => 'javelin-behavior',
+ 1 => 'javelin-dom',
+ 2 => 'javelin-util',
+ 3 => 'javelin-stratcom',
+ 4 => 'javelin-workflow',
+ 5 => 'phabricator-draggable-list',
+ ),
'd8ef8659' =>
array(
0 => 'javelin-behavior',
diff --git a/src/applications/project/controller/PhabricatorProjectMoveController.php b/src/applications/project/controller/PhabricatorProjectMoveController.php
--- a/src/applications/project/controller/PhabricatorProjectMoveController.php
+++ b/src/applications/project/controller/PhabricatorProjectMoveController.php
@@ -16,6 +16,7 @@
$column_phid = $request->getStr('columnPHID');
$object_phid = $request->getStr('objectPHID');
$after_phid = $request->getStr('afterPHID');
+ $before_phid = $request->getStr('beforePHID');
$project = id(new PhabricatorProjectQuery())
->setViewer($viewer)
@@ -78,23 +79,51 @@
'columnPHIDs' => $edge_phids,
'projectPHID' => $column->getProjectPHID()));
+ $task_phids = array();
if ($after_phid) {
- $after_task = id(new ManiphestTaskQuery())
+ $task_phids[] = $after_phid;
+ }
+ if ($before_phid) {
+ $task_phids[] = $before_phid;
+ }
+ if ($task_phids) {
+ $tasks = id(new ManiphestTaskQuery())
->setViewer($viewer)
- ->withPHIDs(array($after_phid))
+ ->withPHIDs($task_phids)
->requireCapabilities(array(PhabricatorPolicyCapability::CAN_EDIT))
- ->executeOne();
- if (!$after_task) {
+ ->execute();
+ if (!$tasks) {
return new Aphront404Response();
}
- $after_pri = $after_task->getPriority();
- $after_sub = $after_task->getSubpriority();
-
- $xactions[] = id(new ManiphestTransaction())
- ->setTransactionType(ManiphestTransaction::TYPE_SUBPRIORITY)
- ->setNewValue(array(
- 'newPriority' => $after_pri,
- 'newSubpriorityBase' => $after_sub));
+ $tasks = mpull($tasks, null, 'getPHID');
+
+ $after_task = idx($tasks, $after_phid);
+ $before_task = idx($tasks, $before_phid);
+
+ if ($after_task &&
+ $after_task->getPriority() < $object->getPriority()) {
+
+ $after_pri = $after_task->getPriority();
+ $after_sub = $after_task->getSubpriority();
+
+ $xactions[] = id(new ManiphestTransaction())
+ ->setTransactionType(ManiphestTransaction::TYPE_SUBPRIORITY)
+ ->setNewValue(array(
+ 'newPriority' => $after_pri,
+ 'newSubpriorityBase' => $after_sub));
+
+ } else if ($before_task &&
+ $before_task->getPriority() > $object->getPriority()) {
+
+ $before_pri = $before_task->getPriority();
+ $before_sub = $before_task->getSubpriority();
+
+ $xactions[] = id(new ManiphestTransaction())
+ ->setTransactionType(ManiphestTransaction::TYPE_SUBPRIORITY)
+ ->setNewValue(array(
+ 'newPriority' => $before_pri,
+ 'newSubpriorityBase' => $before_sub));
+ }
}
$editor = id(new ManiphestTransactionEditor())
diff --git a/webroot/rsrc/js/application/projects/behavior-project-boards.js b/webroot/rsrc/js/application/projects/behavior-project-boards.js
--- a/webroot/rsrc/js/application/projects/behavior-project-boards.js
+++ b/webroot/rsrc/js/application/projects/behavior-project-boards.js
@@ -28,13 +28,40 @@
list.lock();
JX.DOM.alterClass(item, 'drag-sending', true);
+ var item_phid = JX.Stratcom.getData(item).objectPHID;
var data = {
- objectPHID: JX.Stratcom.getData(item).objectPHID,
+ objectPHID: item_phid,
columnPHID: JX.Stratcom.getData(list.getRootNode()).columnPHID
};
+ var after_phid = null;
+ var items = finditems(list.getRootNode());
if (after) {
- data.afterPHID = JX.Stratcom.getData(after).objectPHID;
+ after_phid = JX.Stratcom.getData(after).objectPHID;
+ data.afterPHID = after_phid;
+ }
+ var ii;
+ var ii_item;
+ var ii_item_phid;
+ var ii_prev_item_phid = null;
+ var before_phid = null;
+ for (ii = 0; ii < items.length; ii++) {
+ ii_item = items[ii];
+ ii_item_phid = JX.Stratcom.getData(ii_item).objectPHID;
+ if (ii_item_phid == item_phid) {
+ // skip the item we just dropped
+ continue;
+ }
+ // note this handles when there is no after phid - we are at the top of
+ // the list - quite nicely
+ if (ii_prev_item_phid == after_phid) {
+ before_phid = ii_item_phid;
+ break;
+ }
+ ii_prev_item_phid = ii_item_phid;
+ }
+ if (before_phid) {
+ data.beforePHID = before_phid;
}
var workflow = new JX.Workflow(config.moveURI, data)

File Metadata

Mime Type
text/plain
Expires
Mon, Mar 17, 12:58 AM (1 w, 19 h ago)
Storage Engine
blob
Storage Format
Encrypted (AES-256-CBC)
Storage Handle
7707902
Default Alt Text
D8622.id20456.diff (6 KB)

Event Timeline

Phabricator · Built by Phacility