revi-archive/phorge-arcanist
1
0
Fork 0
mirror of https://we.phorge.it/source/arcanist.git synced 2024-11-25 16:22:42 +01:00
Code Issues Releases Wiki Activity

Fix linter rule after XHPAST change

Browse source 
Summary: Depends on D13959.

Test Plan: Ran unit tests.

Reviewers: #blessed_reviewers, epriestley

Reviewed By: #blessed_reviewers, epriestley

Subscribers: Korvin

Differential Revision: https://secure.phabricator.com/D13961
This commit is contained in:
Joshua Spence 2015-08-27 22:08:49 +10:00 committed by Joshua Spence
parent c22dfe61ed
commit 18e32d6ec7
1 changed files with 1 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
src/lint/linter/xhpast/rules/ArcanistPHPCompatibilityXHPASTLinterRule.php
View file

@ -327,7 +327,7 @@ final class ArcanistPHPCompatibilityXHPASTLinterRule
$ternaries = $root->selectDescendantsOfType('n_TERNARY_EXPRESSION'); $ternaries = $root->selectDescendantsOfType('n_TERNARY_EXPRESSION');
foreach ($ternaries as $ternary) { foreach ($ternaries as $ternary) {
$yes = $ternary->getChildByIndex(1); $yes = $ternary->getChildByIndex(2);
if ($yes->getTypeName() === 'n_EMPTY') { if ($yes->getTypeName() === 'n_EMPTY') {
$this->raiseLintAtNode( $this->raiseLintAtNode(
$ternary, $ternary,