Page MenuHomePhabricator

Fix Phriction status not set property on new document
ClosedPublic

Authored by chad on May 19 2017, 4:25 PM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Mar 22, 10:10 AM
Unknown Object (File)
Jan 23 2024, 11:01 AM
Unknown Object (File)
Dec 27 2023, 11:07 AM
Unknown Object (File)
Dec 27 2023, 11:07 AM
Unknown Object (File)
Dec 27 2023, 11:07 AM
Unknown Object (File)
Dec 24 2023, 9:23 AM
Unknown Object (File)
Dec 23 2023, 5:19 PM
Unknown Object (File)
Dec 21 2023, 6:45 PM
Subscribers

Details

Summary

I deleted too many lines of code here and TYPE_MOVE was always being applied when CONTENT was set. This should fix on next document save, but should I write some migration tool anyways?

Test Plan

Create a new document, see document with correct status.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

src/applications/phriction/xaction/PhrictionDocumentTitleTransaction.php
17–19

Seems to work in either place, but this was missed during conversion.

This revision is now accepted and ready to land.May 19 2017, 5:56 PM
This revision was automatically updated to reflect the committed changes.