beco/RecoReco
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

target dropdown shows all columns; italicize current column (v1.9.1)

Browse Source
This commit is contained in:
Ruben Carlo Benante
2026-07-10 23:24:12 -03:00
parent fc1f38fb30
commit 9373d43e96
5 changed files with 42 additions and 8 deletions
Download Patch File Download Diff File Expand all files Collapse all files

10
Controller/RecurrenceController.php
View File

@@ -120,13 +120,13 @@ class RecurrenceController extends BaseController
return $this->response->redirect($this->helper->url->to('TaskViewController', 'show', array('task_id' => $task['id'])), true);
}
// Columns the copy can land in: every column except the template's own (the copy must move).
// Target columns: ALL of the project's columns. This used to exclude the card's own column (a
// duplicate copy "must move"), but move-mode legitimately rests the card IN the target, so the
// target must be any column -- otherwise a moved card's own target vanishes from the dropdown and
// a re-save would silently reset it. `target != trigger` is enforced in the modal instead.
private function targetColumns(array $task)
{
$columns = $this->columnModel->getList($task['project_id']);
unset($columns[$task['column_id']]);
return $columns;
return $this->columnModel->getList($task['project_id']);
}
// A stored/posted target, if still a valid non-current column; otherwise the first one (or 0).