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
F15529181: D16837.id.diff
Tue, Apr 22, 5:34 PM
F15476597: D16837.id40549.diff
Mon, Apr 7, 7:46 AM
F15475884: D16837.id40545.diff
Mon, Apr 7, 12:59 AM
F15473945: D16837.id40549.diff
Sun, Apr 6, 3:27 AM
F15469825: D16837.id.diff
Fri, Apr 4, 4:56 PM
F15466352: D16837.diff
Thu, Apr 3, 4:33 AM
F15410600: D16837.diff
Mar 19 2025, 7:42 AM
F15407290: D16837.id40545.diff
Mar 18 2025, 4:43 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