Page MenuHomePhabricator

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

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

Details

Summary

See https://discourse.phabricator-community.org/t/error-seems-to-be-related-with-da40f8074-and-php-7-3/2102/12. 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

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

Event Timeline

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