Page MenuHomePhabricator
Differential D12849

Phame - move over blog create + edit to transactions + editor
ClosedPublic
Actions

Authored by btrahan on May 14 2015, 11:44 PM.
Tags
None
Referenced Files
F15458834: D12849.diff
Mon, Mar 31, 7:49 AM
F15448570: D12849.id.diff
Fri, Mar 28, 5:40 AM
F15442809: D12849.id30915.diff
Thu, Mar 27, 12:36 AM
F15434949: D12849.id30915.diff
Tue, Mar 25, 6:10 AM
F15431912: D12849.id30915.diff
Mon, Mar 24, 3:21 PM
F15427049: D12849.diff
Sun, Mar 23, 12:20 PM
F15425414: D12849.id.diff
Sun, Mar 23, 4:37 AM
F15419946: D12849.id30915.diff
Fri, Mar 21, 10:00 AM
View All Files
Subscribers
epriestley
Korvin
Tokens
"Piece of Eight" token, awarded by epriestley.

Details

Reviewers
epriestley
Maniphest Tasks
T7626: Attaching files to a Phame post doesn't mark them as public
Commits
Restricted Diffusion Commit
rPf16dda288d0c: Phame - move over blog create + edit to transactions + editor
Summary

Ref T7626. Modernizes the code a bit here so we can eventually make progress on T7626 and other stuff.

Test Plan

made a blog, edited a blog, made errors - stuff looked good

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

btrahan updated this revision to Diff 30913.May 14 2015, 11:44 PM
btrahan retitled this revision from to Phame - move over blog create + edit to transactions + editor.
btrahan updated this object.
btrahan edited the test plan for this revision. (Show Details)
btrahan added a reviewer: epriestley.
btrahan added a task: T7626: Attaching files to a Phame post doesn't mark them as public.
Herald added subscribers: epriestley, Korvin. · View Herald TranscriptMay 14 2015, 11:44 PM
epriestley accepted this revision.May 14 2015, 11:52 PM
epriestley edited edge metadata.
This revision is now accepted and ready to land.May 14 2015, 11:52 PM
epriestley awarded a token.May 14 2015, 11:52 PM
Closed by commit rPf16dda288d0c: Phame - move over blog create + edit to transactions + editor (authored by btrahan, committed by btrahan). · Explain WhyMay 15 2015, 12:05 AM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Diff 30915

View Options

resources/sql/autopatches/20150514.phame.blog.xaction.sql

Loading...
View Options

src/__phutil_library_map__.php

Loading...
View Options

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

Loading...
View Options

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

Loading...
View Options

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

Loading...
View Options

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

Loading...
View Options

src/applications/phame/controller/blog/PhameBlogLiveController.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/storage/PhameBlog.php

Loading...
View Options

src/applications/phame/storage/PhameBlogTransaction.php

Loading...
Phabricator · Built by Phacility