Fixes T5235. Implement PhabricatorDestructableInterface on PhabricatorProject so that projects can be deleted with ./bin/remove destroy.
Details
Details
- Reviewers
epriestley - Group Reviewers
Blessed Reviewers - Maniphest Tasks
- T5235: Implement `PhabricatorDestructableInterface` on `PhabricatorProject`
- Commits
- Restricted Diffusion Commit
rPc2eff7c2163c: Allow projects to be deleted with `./bin/remove destroy`.
Created (and then destroyed) a test project. Verified that the corresponding objects (project, slugs and workboard columns) were removed from the database.
Diff Detail
Diff Detail
- Repository
- rP Phabricator
- Branch
- rm-projects
- Lint
Lint Passed - Unit
No Test Coverage - Build Status
Buildable 808 Build 808: [Placeholder Plan] Wait for 30 Seconds
Event Timeline
Comment Actions
Hmm... I don't think this is cleaning up all of the edges. I had also created a maniphest under the test project, which now displays as "Unknown Object" (whereas I was expecting the project to be removed from the maniphest competely).
Comment Actions
IIRC, Maniphest doesn't use edges to record projects right now. It should, it just predates the edge infrastructure.
Comment Actions
Closed by commit rPc2eff7c2163c (authored by @joshuaspence, committed by @epriestley).