Page MenuHomePhabricator

Add a linter rule for curly brace array indexes
ClosedPublic

Authored by joshuaspence on Nov 30 2015, 11:26 AM.
Tags
None
Referenced Files
F13815344: D14603.diff
Thu, Sep 19, 9:34 AM
Unknown Object (File)
Fri, Sep 6, 4:55 PM
Unknown Object (File)
Mon, Sep 2, 12:07 PM
Unknown Object (File)
Aug 20 2024, 3:15 AM
Unknown Object (File)
Aug 14 2024, 1:47 AM
Unknown Object (File)
Aug 13 2024, 6:14 PM
Unknown Object (File)
Aug 12 2024, 10:51 AM
Unknown Object (File)
Aug 7 2024, 11:14 AM
Subscribers

Details

Summary

In PHP, both $x['key'] and $x{'key'} can be used to access an array (see http://stackoverflow.com/questions/8092248/php-curly-braces-in-array-notation), but the former should be preferred.

Test Plan

Added test cases.

Diff Detail

Repository
rARC Arcanist
Branch
master
Lint
Lint Passed
Unit
Test Failures
Build Status
Buildable 9134
Build 10779: Run Core Tests
Build 10778: arc lint + arc unit

Event Timeline

joshuaspence retitled this revision from to Add a linter rule for curly brace array indexes.
joshuaspence updated this object.
joshuaspence edited the test plan for this revision. (Show Details)
joshuaspence added a reviewer: epriestley.
epriestley edited edge metadata.
This revision is now accepted and ready to land.Nov 30 2015, 1:08 PM
joshuaspence edited edge metadata.

Improve handling of nested indexes

This revision was automatically updated to reflect the committed changes.