Page MenuHomePhabricator

Move Phriction Title transaction to Modular Transactions
ClosedPublic

Authored by chad on May 16 2017, 5:54 PM.
Tags
None
Referenced Files
F18816434: D17912.id43087.diff
Tue, Oct 21, 6:50 AM
F18812608: D17912.id43090.diff
Mon, Oct 20, 10:45 AM
F18787294: D17912.id.diff
Tue, Oct 14, 11:03 PM
F18769721: D17912.id.diff
Wed, Oct 8, 8:51 AM
F18763706: D17912.id43090.diff
Tue, Oct 7, 1:43 AM
F18755413: D17912.id.diff
Sun, Oct 5, 5:26 AM
F18749339: D17912.diff
Sat, Oct 4, 3:25 AM
F18748254: D17912.id43090.diff
Fri, Oct 3, 8:09 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
Lint
Lint Not Applicable
Unit
Tests Not Applicable

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.