Page MenuHomePhabricator

Move Phriction Title transaction to Modular Transactions
ClosedPublic

Authored by chad on May 16 2017, 5:54 PM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Nov 29, 8:08 AM
Unknown Object (File)
Wed, Nov 27, 1:18 AM
Unknown Object (File)
Mon, Nov 25, 3:25 PM
Unknown Object (File)
Sat, Nov 9, 1:40 PM
Unknown Object (File)
Oct 15 2024, 4:02 PM
Unknown Object (File)
Oct 14 2024, 12:21 AM
Unknown Object (File)
Oct 14 2024, 12:21 AM
Unknown Object (File)
Oct 1 2024, 7:17 PM
Subscribers

Details

Summary

Ref T12625. Moves TYPE_TITLE to modular transaction.

Test Plan

New Document, Edit Document, test validation, verify feed stories.

Diff Detail

Repository
rP Phabricator
Branch
phriction-title-xaction (branched from master)
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 17015
Build 22748: Run Core Tests
Build 22747: arc lint + arc unit

Event Timeline

src/applications/phriction/editor/PhrictionTransactionEditor.php
495–502

oops, I think I need to move this over too.

Feel free to just delete all the "get/setprocesscontentversionerror" junk if that makes things easier, and we can file a task like "make this work better". The way it works now isn't great and won't work with Conduit anyway.

Well, I guess it's kiiiiiind of a regression to not have a warning at all, I think we added it because someone was actually hitting it? I vaguely recall it ruining someone's life 100x the worst.

But the title field shouldn't really matter or need the check -- if you and someone else rename a document simultaneously it seems fine to just use "last save wins", at least for now.

then this is ready for review

This revision is now accepted and ready to land.May 16 2017, 6:06 PM
This revision was automatically updated to reflect the committed changes.