Page MenuHomePhabricator
Differential D13801

Allow setting of Archive/Active on Paste
ClosedPublic
Actions

Authored by chad on Aug 5 2015, 8:08 PM.
Tags
None
Referenced Files
F19469187: D13801.id33330.diff
Sun, Jan 4, 5:50 PM
F19468721: D13801.id33330.diff
Sun, Jan 4, 9:11 AM
F19201670: D13801.id33330.diff
Tue, Dec 16, 4:22 PM
F19036854: D13801.diff
Nov 25 2025, 8:52 PM
F19014669: D13801.diff
Nov 22 2025, 8:54 PM
F18710542: D13801.id33330.diff
Sep 29 2025, 2:16 AM
F18590253: D13801.diff
Sep 12 2025, 5:37 AM
F18486517: D13801.id.diff
Sep 3 2025, 6:44 PM
View All Files
Subscribers
Korvin

Details

Reviewers
epriestley
Maniphest Tasks
T9076: Archiving pastes
Commits
Restricted Diffusion Commit
rP5eeda6040eca: Allow setting of Archive/Active on Paste
Summary

Ref T9076, adds basic plumbing for setting the state of a Paste.

Test Plan

Archive Paste, Activate Paste, New Paste

Diff Detail

Repository
rP Phabricator
Branch
paste-archive
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 7498
Build 8036: [Placeholder Plan] Wait for 30 Seconds
Build 8035: arc lint + arc unit

Event Timeline

chad updated this revision to Diff 33330.Aug 5 2015, 8:08 PM
chad retitled this revision from to Allow setting of Archive/Active on Paste.
chad updated this object.
chad edited the test plan for this revision. (Show Details)
chad added a reviewer: epriestley.
Herald added a subscriber: Korvin. · View Herald TranscriptAug 5 2015, 8:08 PM
chad updated this object.Aug 5 2015, 8:10 PM
chad added a task: T9076: Archiving pastes.
epriestley accepted this revision.Aug 5 2015, 8:38 PM
epriestley edited edge metadata.

I think most applications make this a separate action, but whatever.

This revision is now accepted and ready to land.Aug 5 2015, 8:38 PM
chad added a comment.Aug 5 2015, 8:42 PM

whats that mean?

chad added a comment.Aug 5 2015, 8:42 PM

oh, like an "Action" action

epriestley added a comment.Aug 5 2015, 8:44 PM

Yeah, like you click "Archive" instead of "Edit Thing > Status: Archived".

Closed by commit rP5eeda6040eca: Allow setting of Archive/Active on Paste (authored by chad, committed by chad). · Explain WhyAug 5 2015, 8:44 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
resources/
sql/
autopatches/
2 lines
2 lines
src/
applications/
paste/
controller/
12 lines
12 lines
editor/
8 lines
storage/
17 lines
21 lines

Diff 33330

View Options

resources/sql/autopatches/20150805.paste.status.1.sql

Loading...
View Options

resources/sql/autopatches/20150805.paste.status.2.sql

Loading...
View Options

src/applications/paste/controller/PhabricatorPasteEditController.php

Loading...
View Options

src/applications/paste/controller/PhabricatorPasteViewController.php

Loading...
View Options

src/applications/paste/editor/PhabricatorPasteEditor.php

Loading...
View Options

src/applications/paste/storage/PhabricatorPaste.php

Loading...
View Options

src/applications/paste/storage/PhabricatorPasteTransaction.php

Loading...
Phabricator · Built by Phacility