Remove old position-on-read board column code
ClosedPublic
Actions

Authored by epriestley on Feb 3 2016, 8:14 PM.

Tags

None

Referenced Files

	F18832795: D15177.id36636.diff
	Sat, Oct 25, 11:07 PM

	F18812858: D15177.id36645.diff
	Mon, Oct 20, 12:44 PM

	F18812647: D15177.id.diff
	Mon, Oct 20, 11:06 AM

	F18798837: D15177.id36636.diff
	Fri, Oct 17, 9:11 AM

	F18753067: D15177.id.diff
	Sat, Oct 4, 5:20 PM

	F18737891: D15177.diff
	Wed, Oct 1, 1:11 PM

	F18716961: D15177.diff
	Mon, Sep 29, 12:57 PM

	F18694285: D15177.id36645.diff
	Sep 27 2025, 3:22 AM

View All Files

Subscribers

None

Details

Reviewers

chad

Maniphest Tasks

T10010: Implement Subprojects

Commits

Restricted Diffusion Commit
rP9961de0e809a: Remove old position-on-read board column code

Summary

Ref T10010. This retires the old way of doing things inside ColumnPositionQuery. It is now obsolete and lives in BoardLayoutEngine instead.

Test Plan

Moved cards, created cards, swapped filters, orders, etc.
Some degree of unit testing coming in the next diff.

Diff Detail

Repository

rP Phabricator

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

epriestley updated this revision to Diff 36636.Feb 3 2016, 8:14 PM

epriestley retitled this revision from to Remove old position-on-read board column code.

epriestley updated this object.

epriestley edited the test plan for this revision. (Show Details)

epriestley added a reviewer: chad.

epriestley added a task: T10010: Implement Subprojects.

≧☉_☉≦
≧☉_☉≦
≧☉_☉≦
≧☉_☉≦
≧☉_☉≦

This revision is now accepted and ready to land.Feb 3 2016, 8:36 PM

Closed by commit rP9961de0e809a: Remove old position-on-read board column code (authored by epriestley, committed by epriestley). · Explain WhyFeb 3 2016, 11:07 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

src/

applications/

maniphest/

editor/

ManiphestEditEngine.php

3 lines

project/

query/

PhabricatorProjectColumnPositionQuery.php

255 lines

Diff 36645

View Options

src/applications/maniphest/editor/ManiphestEditEngine.php