Page MenuHomePhabricator

Correct two straggling "%Q" + "implode(...)" callsites in Revision updates

Authored by epriestley on Dec 5 2018, 2:14 PM.



See When creating or updating revisions, we do some manual query construction to update the affected path table.

Update these queries to modern qsprintf().

Test Plan

Created and updated revisions affecting paths, no more logs in the webserver log.

Diff Detail

rP Phabricator
Automatic diff as part of commit; lint not applicable.
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

epriestley created this revision.Dec 5 2018, 2:14 PM
epriestley requested review of this revision.Dec 5 2018, 2:16 PM
amckinley accepted this revision.Dec 7 2018, 10:25 PM
This revision is now accepted and ready to land.Dec 7 2018, 10:25 PM
This revision was automatically updated to reflect the committed changes.