Change call to withFilterPHIDs after method rename

Authored by JeanFred on Nov 10 2017, 2:04 PM.


Maniphest Tasks
T177051: "Call to undefined method" error due to using setFilterPHIDs instead of withFilterPHIDs
Group Reviewers
Restricted Owners Package(Owns No Changed Paths)
rPHSP3148c9ccb92a: Change call to withFilterPHIDs after method rename
Patch without arc
git checkout -b D876 && curl -L | git apply

Clicking on a sprint name on /project/sprint/ results in:

Call to undefined method PhabricatorFeedQuery::setFilterPHIDs()
┻━┻ ︵ ¯\_(ツ)_/¯ ︵ ┻━┻

This is because the method setFilterPHIDs was renamed to withFilterPHIDs
in rPHAB2d79229.

Ref T177051

Test Plan

Check syntax for possible typos. Deploy and see if it works.

Diff Detail

rPHSP phabricator-Sprint
Automatic diff as part of commit; lint not applicable.
Automatic diff as part of commit; unit tests not applicable.
JeanFred created this revision.Nov 10 2017, 2:04 PM
Restricted Application added a reviewer: Restricted Owners Package.Nov 10 2017, 2:04 PM
Paladox accepted this revision.Nov 10 2017, 2:39 PM
This revision is now accepted and ready to land.Nov 10 2017, 2:39 PM

Thanks for approving! I’m afraid I cannot land this though.

$ arc land
Landing current branch 'T177051'.
 TARGET  Landing onto "master", the default target under git.
 REMOTE  Using remote "origin", the default remote under git.
 FETCH  Fetching origin/master...
This commit will be landed:

      - b4f4e99 Change call to withFilterPHIDs after method rename

Landing revision 'D876: Change call to withFilterPHIDs after method rename'...
 BUILDS PASSED  Harbormaster builds for the active diff completed successfully.
 PUSHING  Pushing changes to "origin/master".

Username for '': JeanFred
Password for '': 
fatal: unable to access '': The requested URL returned error: 403
Usage Exception: Push failed! Fix the error and run "arc land" again.

Yep, I believe @mmodell can only land this.

This revision was automatically updated to reflect the committed changes.