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
F15497475: D12849.id.diff
Sun, Apr 13, 10:34 AM
F15491782: D12849.diff
Sat, Apr 12, 7:29 AM
F15476127: D12849.id30913.diff
Mon, Apr 7, 4:19 AM
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
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