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
Unknown Object (File)
Thu, Apr 25, 2:17 AM
Unknown Object (File)
Sun, Apr 21, 10:29 AM
Unknown Object (File)
Thu, Apr 18, 11:50 PM
Unknown Object (File)
Fri, Apr 5, 3:23 PM
Unknown Object (File)
Mar 26 2024, 6:19 AM
Unknown Object (File)
Mar 24 2024, 10:52 PM
Unknown Object (File)
Feb 14 2024, 8:31 PM
Unknown Object (File)
Feb 14 2024, 8:26 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
Branch
phameeditors
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 5995
Build 6015: [Placeholder Plan] Wait for 30 Seconds

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 30913

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