Page MenuHomePhabricator

Fix two error strings in the diffusion.uri.edit Conduit method
ClosedPublic

Authored by epriestley on Nov 10 2016, 2:21 PM.
Tags
None
Referenced Files
F18836869: D16837.id40545.diff
Mon, Oct 27, 3:45 AM
F18826484: D16837.diff
Fri, Oct 24, 4:11 AM
F18822432: D16837.id40545.diff
Thu, Oct 23, 6:41 AM
F18816804: D16837.id40545.diff
Tue, Oct 21, 9:30 AM
F18812210: D16837.id40549.diff
Oct 20 2025, 7:45 AM
F18786910: D16837.diff
Oct 14 2025, 9:05 PM
F18737586: D16837.id40545.diff
Oct 1 2025, 11:31 AM
F18733432: D16837.id.diff
Sep 30 2025, 10:34 PM
Subscribers

Details

Summary

Fixes T11839. Both are missing a parameter and one is a copy/paste slop.

Test Plan

Screen Shot 2016-11-10 at 6.20.04 AM.png (258×1 px, 30 KB)

Screen Shot 2016-11-10 at 6.19.56 AM.png (215×1 px, 27 KB)

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley retitled this revision from to Fix two error strings in the diffusion.uri.edit Conduit method.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.
chad edited edge metadata.
This revision is now accepted and ready to land.Nov 10 2016, 3:17 PM
This revision was automatically updated to reflect the committed changes.
cxie added inline comments.
src/applications/diffusion/editor/DiffusionURIEditor.php
313

test

422

test