Page MenuHomePhabricator

Add a linter rule for alias functions
ClosedPublic

Authored by joshuaspence on Apr 15 2015, 1:28 PM.
Tags
None
Referenced Files
F18875771: D12422.id30995.diff
Nov 5 2025, 7:15 PM
F18875548: D12422.id30874.diff
Nov 5 2025, 6:16 PM
F18872204: D12422.diff
Nov 5 2025, 2:47 AM
F18837098: D12422.id30995.diff
Oct 27 2025, 4:24 AM
F18805337: D12422.id.diff
Oct 18 2025, 3:06 PM
F18780728: D12422.id29830.diff
Oct 12 2025, 7:16 PM
F18780727: D12422.id30874.diff
Oct 12 2025, 7:16 PM
F18772981: D12422.id30874.diff
Oct 9 2025, 6:10 AM
Subscribers

Details

Summary

Ref T7409. Adds ArcanistXHPASTLinter::LINT_ALIAS_FUNCTION for linting the use of alias funtions. Based on Generic_Sniffs_PHP_ForbiddenFunctionsSniff. See list of function aliases.

Test Plan

Added unit tests.

Diff Detail

Repository
rARC Arcanist
Branch
master
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 5322
Build 5340: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

joshuaspence retitled this revision from to Add a linter rule for alias functions.
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.Apr 15 2015, 1:29 PM
joshuaspence edited edge metadata.

Comprehensive list of aliases

This revision was automatically updated to reflect the committed changes.

Oh, I should've mentioned... I specifically excluded a bunch of aliases from the Ming extension because they seemed contradictory. For example, according to the documentation add() is an alias for swfmovie_add() and also swfsprite_add().