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
F19430175: D12849.diff
Sun, Dec 28, 12:13 PM
F19085895: D12849.id30913.diff
Dec 2 2025, 7:56 PM
F18873676: D12849.id.diff
Nov 5 2025, 10:54 AM
F18867926: D12849.diff
Nov 4 2025, 3:36 AM
F18854560: D12849.id30915.diff
Nov 1 2025, 12:22 AM
F18854559: D12849.id30913.diff
Nov 1 2025, 12:22 AM
F18854558: D12849.id.diff
Nov 1 2025, 12:22 AM
F18817317: D12849.id30915.diff
Oct 21 2025, 2:29 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