Page MenuHomePhabricator
Differential D13267

Remove backticks from SQL statements
ClosedPublic
Actions

Authored by joshuaspence on Jun 13 2015, 4:06 AM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Feb 20, 9:51 AM
Unknown Object (File)
Sat, Feb 8, 10:00 AM
Unknown Object (File)
Sat, Feb 8, 10:00 AM
Unknown Object (File)
Sat, Feb 8, 9:59 AM
Unknown Object (File)
Sat, Feb 8, 9:59 AM
Unknown Object (File)
Fri, Jan 24, 6:12 PM
Unknown Object (File)
Fri, Jan 24, 6:11 PM
Unknown Object (File)
Fri, Jan 24, 6:17 AM
View All Files
Subscribers
epriestley

Details

Reviewers
epriestley
eadler
Group Reviewers
Blessed Reviewers
Commits
Restricted Diffusion Commit
rP070da8286407: Remove backticks from SQL statements
Summary

Remove backticks from SQL statements for consistency. In most places, we don't use backticks around table/field names, so at least be consistent about this.

Test Plan

Learned what backticks are used for in MySQL.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

joshuaspence updated this revision to Diff 32064.Jun 13 2015, 4:06 AM
joshuaspence retitled this revision from to Remove backticks from SQL statements.
joshuaspence updated this object.
joshuaspence edited the test plan for this revision. (Show Details)
Herald added a reviewer: Blessed Reviewers. · View Herald TranscriptJun 13 2015, 4:06 AM
Herald added a subscriber: epriestley. · View Herald Transcript
epriestley accepted this revision.Jun 13 2015, 3:22 PM
epriestley added a reviewer: epriestley.
This revision is now accepted and ready to land.Jun 13 2015, 3:22 PM
eadler accepted this revision.Jun 14 2015, 6:23 AM
eadler added a reviewer: eadler.
joshuaspence updated this object.Jun 14 2015, 9:37 PM
joshuaspence edited the test plan for this revision. (Show Details)
joshuaspence edited edge metadata.
Closed by commit rP070da8286407: Remove backticks from SQL statements (authored by joshuaspence, committed by joshuaspence). · Explain WhyJun 14 2015, 9:38 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
src/
applications/
herald/
garbagecollector/
2 lines
ponder/
editor/
4 lines
xhprof/
controller/
6 lines

Diff 32119

View Options

src/applications/herald/garbagecollector/HeraldTranscriptGarbageCollector.php

Loading...
View Options

src/applications/ponder/editor/PonderVoteEditor.php

Loading...
View Options

src/applications/xhprof/controller/PhabricatorXHProfSampleListController.php

Loading...
Phabricator · Built by Phacility