Page MenuHomePhabricator
Differential D14465

Allow Phame Blogs to be archived instead of deleted
ClosedPublic
Actions

Authored by chad on Nov 12 2015, 12:43 AM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Apr 9, 12:11 PM
Unknown Object (File)
Fri, Apr 5, 7:59 AM
Unknown Object (File)
Mon, Mar 25, 8:35 AM
Unknown Object (File)
Mar 10 2024, 1:52 AM
Unknown Object (File)
Mar 6 2024, 5:13 AM
Unknown Object (File)
Mar 6 2024, 5:13 AM
Unknown Object (File)
Mar 6 2024, 5:13 AM
Unknown Object (File)
Mar 6 2024, 5:13 AM
View All Files
Subscribers
joshuaspence
Korvin

Details

Reviewers
epriestley
Maniphest Tasks
T9756: Allow Blogs and Posts to be archived
Commits
Restricted Diffusion Commit
rP62e129d7a6d1: Allow Phame Blogs to be archived instead of deleted
Summary

Removes "delete" and uses "archive/activate" instead for Phame Blogs. Ref T9756

Test Plan

Archive a blog, see in search, activate blog, see in other search.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

chad updated this revision to Diff 34985.Nov 12 2015, 12:43 AM
chad retitled this revision from to Allow Phame Blogs to be archived instead of deleted.
chad updated this object.
chad edited the test plan for this revision. (Show Details)
chad added a reviewer: epriestley.
chad added a task: T9756: Allow Blogs and Posts to be archived.
Herald added a subscriber: Korvin. · View Herald TranscriptNov 12 2015, 12:43 AM
joshuaspence added a subscriber: joshuaspence.Nov 13 2015, 7:30 AM
epriestley accepted this revision.Nov 15 2015, 4:48 PM
epriestley edited edge metadata.

Consider withStatuses(array $statuses) instead of withStatus($status). The former is a bit more flexible if we introduce other types of statuses later.

This revision is now accepted and ready to land.Nov 15 2015, 4:48 PM
chad updated this revision to Diff 35149.Nov 21 2015, 4:53 PM
chad edited edge metadata.
  • Use withStatuses
Closed by commit rP62e129d7a6d1: Allow Phame Blogs to be archived instead of deleted (authored by chad, committed by chad). · Explain WhyNov 21 2015, 4:54 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Diff 35150

View Options

resources/sql/autopatches/20151111.phame.blog.archive.1.sql

Loading...
View Options

resources/sql/autopatches/20151111.phame.blog.archive.2.sql

Loading...
View Options

src/__phutil_library_map__.php

Loading...
View Options

src/applications/phame/application/PhabricatorPhameApplication.php

Loading...
View Options

src/applications/phame/controller/blog/PhameBlogArchiveController.php

Loading...
View Options

src/applications/phame/controller/blog/PhameBlogDeleteController.php

Loading...
View Options

src/applications/phame/controller/blog/PhameBlogViewController.php

Loading...
View Options

src/applications/phame/editor/PhameBlogEditor.php

Loading...
View Options

src/applications/phame/query/PhameBlogQuery.php

Loading...
View Options

src/applications/phame/query/PhameBlogSearchEngine.php

Loading...
View Options

src/applications/phame/storage/PhameBlog.php

Loading...
View Options

src/applications/phame/storage/PhameBlogTransaction.php

Loading...
Phabricator · Built by Phacility