Allows Blogs and Posts to be destroyed. Fixes T9756
Details
Details
- Reviewers
epriestley - Maniphest Tasks
- T9756: Allow Blogs and Posts to be archived
- Commits
- Restricted Diffusion Commit
rPe8a39ca3e5f0: Implement PhabricatorDestructibleInterface in Phame
Test bin/remove destroy POST and bin/remove destroy BLOG to great success.
Diff Detail
Diff Detail
- Repository
- rP Phabricator
- Branch
- phame-destruct
- Lint
Lint Passed - Unit
Tests Passed - Build Status
Buildable 9099 Build 10720: Run Core Tests Build 10719: arc lint + arc unit
Event Timeline
Comment Actions
Build failure looks like a missing arc liberate. Couple o' inlines.
| src/applications/phame/storage/PhameBlog.php | ||
|---|---|---|
| 331–332 | You don't need this, $this is the same as $blog. | |
| src/applications/phame/storage/PhamePost.php | ||
| 262–263 | You can get rid of this, it's unused and $this->delete() always does the same thing anyway. | |