Phriction - fix bug with delete
ClosedPublic
Actions

Authored by btrahan on Nov 7 2014, 4:59 PM.

Details

Reviewers

epriestley
chad

Maniphest Tasks

T6489: Unable to delete Phriction document completely

Commits

Restricted Diffusion Commit
rP677e161865a4: Phriction - fix bug with delete

Summary

Fixes T6489. In an earlier diff I forgot to update the document status itself, thinking that the content update percolated up magically. Restore this functionality so the wiki works better.

Test Plan

deleted a document and observed that i did not get the option to delete it again and it disappeared from document index

Diff Detail

Repository

rP Phabricator

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

btrahan updated this revision to Diff 25932.Nov 7 2014, 4:59 PM

btrahan added a task: T6489: Unable to delete Phriction document completely.

btrahan retitled this revision from to Phriction - fix bug with delete.

btrahan updated this object.

btrahan edited the test plan for this revision. (Show Details)

btrahan added reviewers: epriestley, chad.

Herald added subscribers: epriestley, Korvin. · View Herald TranscriptNov 7 2014, 4:59 PM

Harbormaster completed remote builds in B2980: Diff 25932.Nov 7 2014, 4:59 PM

chad accepted this revision.Nov 7 2014, 5:05 PM

chad edited edge metadata.

This revision is now accepted and ready to land.Nov 7 2014, 5:05 PM

Closed by commit rP677e161865a4: Phriction - fix bug with delete (authored by btrahan, committed by btrahan). · Explain WhyNov 7 2014, 5:08 PM

This revision was automatically updated to reflect the committed changes.

epriestley awarded a token.Nov 7 2014, 5:45 PM

Revision Contents
Changeset List

Path

Size

src/

applications/

phriction/

editor/

PhrictionTransactionEditor.php

3 lines

Diff 25933

View Options

src/applications/phriction/editor/PhrictionTransactionEditor.php

Phriction - fix bug with deleteClosedPublicActions