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
F14082532: D12849.diff
Sat, Nov 23, 12:16 AM
Unknown Object (File)
Wed, Nov 20, 11:18 PM
Unknown Object (File)
Sat, Nov 16, 5:05 PM
Unknown Object (File)
Oct 22 2024, 6:33 PM
Unknown Object (File)
Oct 19 2024, 12:14 PM
Unknown Object (File)
Sep 30 2024, 11:59 PM
Unknown Object (File)
Sep 30 2024, 2:17 AM
Unknown Object (File)
Sep 26 2024, 12:11 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