Page MenuHomePhabricator
Differential D21453

Fix a false negative in lint for "xsprintf()"-family functions
ClosedPublic
Actions

Authored by epriestley on Sep 5 2020, 12:07 AM.
Tags
None
Referenced Files
F15535620: D21453.id51066.diff
Thu, Apr 24, 6:00 AM
F15518955: D21453.id51058.diff
Sat, Apr 19, 6:27 PM
F15518584: D21453.id51066.diff
Sat, Apr 19, 2:50 PM
F15510076: D21453.id.diff
Wed, Apr 16, 6:35 PM
F15506081: D21453.diff
Tue, Apr 15, 8:31 AM
F15492160: D21453.id51066.diff
Sat, Apr 12, 11:46 AM
F15485896: D21453.id51058.diff
Thu, Apr 10, 3:23 AM
F15472075: D21453.diff
Sat, Apr 5, 10:49 AM
View All Files
Subscribers
None

Details

Reviewers
None
Maniphest Tasks
T13577: HTTPSFuture has a missing parameter in an exception message when overwriting a download path
Commits
rARC73847a4b194b: Fix a false negative in lint for "xsprintf()"-family functions
Summary

Ref T13577. This lint rule correctly detects the error in pht('x %s y') but the narrow test for n_STRING_SCALAR prevents it from detecting the error in pht('x %s y'.'z').

Make the test broader.

Test Plan
  • Ran arc lint on HTTPSFuture.php, got a detection of the issue in T13577.
  • Added a failing test and made it pass.

Diff Detail

Repository
rARC Arcanist
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley created this revision.Sep 5 2020, 12:07 AM
Harbormaster completed remote builds in B24873: Diff 51058.Sep 5 2020, 12:07 AM
epriestley requested review of this revision.Sep 5 2020, 12:07 AM
epriestley mentioned this in T13577: HTTPSFuture has a missing parameter in an exception message when overwriting a download path.Sep 5 2020, 12:09 AM
epriestley mentioned this in D21454: Fix a missing "pht()" parameter in HTTPSFuture.Sep 5 2020, 12:09 AM
epriestley mentioned this in D21456: Fix additional "xsprintf()"-family static parameter errors.Sep 5 2020, 12:32 AM
epriestley mentioned this in D21457: Fix additional "xprintf()"-class static parameter lint errors.Sep 8 2020, 3:55 PM
This revision was not accepted when it landed; it landed in state Needs Review.Sep 8 2020, 6:45 PM
Closed by commit rARC73847a4b194b: Fix a false negative in lint for "xsprintf()"-family functions (authored by epriestley). · Explain Why
This revision was automatically updated to reflect the committed changes.
epriestley mentioned this in rP6e1b5da1121c: Fix additional "xprintf()"-class static parameter lint errors.
epriestley mentioned this in rARC1c327208d7ba: Fix a missing "pht()" parameter in HTTPSFuture.
epriestley added a commit: rARC73847a4b194b: Fix a false negative in lint for "xsprintf()"-family functions.
epriestley mentioned this in rARC7112ee3d5960: Fix additional "xsprintf()"-family static parameter errors.

Revision Contents
Changeset List

PathSize
src/
lint/
linter/
xhpast/
rules/
7 lines
__tests__/
formatted-string/
5 lines

Diff 51066

View Options

src/lint/linter/xhpast/rules/ArcanistFormattedStringXHPASTLinterRule.php

Loading...
View Options

src/lint/linter/xhpast/rules/__tests__/formatted-string/formatted-string.lint-test

Loading...
Phabricator · Built by Phacility