Paths

Table of Contentst

Differential D9973

Remove a stray `setActor()` on EdgeEditor
ClosedPublic
Actions

Authored by epriestley on Jul 18 2014, 12:26 AM.

Details

Reviewers

joshuaspence

Maniphest Tasks

T5245: Migrate Maniphest Projects to use edge infrastructure

Commits

Restricted Diffusion Commit
rPbd3c239d5aad: Remove a stray `setActor()` on EdgeEditor

Summary

These got removed recently but I missed one callsite.

Test Plan

Used git grep to double check all other callsites.

Diff Detail

Repository

rP Phabricator

Branch

edge1

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 1759
Build 1760: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

epriestley updated this revision to Diff 23964.Jul 18 2014, 12:26 AM

epriestley retitled this revision from to Remove a stray `setActor()` on EdgeEditor.

epriestley updated this object.

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

epriestley added a reviewer: joshuaspence.

Herald added a subscriber: epriestley. · View Herald TranscriptJul 18 2014, 12:26 AM

joshuaspence accepted this revision.Jul 18 2014, 12:26 AM

joshuaspence edited edge metadata.

This revision is now accepted and ready to land.Jul 18 2014, 12:26 AM

Harbormaster completed remote builds in B1759: Diff 23964.Jul 18 2014, 12:27 AM

Closed by commit rPbd3c239d5aad (authored by @epriestley).

joshuaspence added a task: T5245: Migrate Maniphest Projects to use edge infrastructure.Jul 19 2014, 2:52 AM

Revision Contents
Changeset List

Path

Size

src/

applications/

transactions/

editor/

PhabricatorApplicationTransactionEditor.php

3 lines

Diff 23964

View Options

src/applications/transactions/editor/PhabricatorApplicationTransactionEditor.php

Remove a stray `setActor()` on EdgeEditorClosedPublicActions