Page MenuHomePhabricator

Update Phriction Move Away transaction to modular transactions
ClosedPublic

Authored by chad on May 22 2017, 12:54 AM.

Details

Summary

Ref T12625

Test Plan

Move a document to a new location, verify the old and new document. Edit both. Grep for MOVE_AWAY

Diff Detail

Repository
rP Phabricator
Branch
move-away-phriction (branched from master)
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 17179
Build 22987: Run Core Tests
Build 22986: arc lint + arc unit

Event Timeline

epriestley added inline comments.
src/applications/phriction/xaction/PhrictionDocumentMoveAwayTransaction.php
35–37

(This is probably the default value?)

44

$old is unused

54

$old is unused.

This revision is now accepted and ready to land.May 22 2017, 10:18 PM
This revision was automatically updated to reflect the committed changes.