Page MenuHomePhabricator

Fix IfElseStructureSniff can't detect and fix multiple white spaces after else
Closed, ResolvedPublic

Description

IfElseStructureSniff can't detect and fix multiple white spaces after else, like following:

if ( $a === false ) {
    # code...
} else      {
    # code...
}

see https://gerrit.wikimedia.org/r/277227

Event Timeline

Restricted Application added a subscriber: Aklapper. · View Herald TranscriptMar 14 2016, 12:00 PM

Change 277227 had a related patch set uploaded (by Lethexie):
Fix IfElseStructureSniff can't detect and fix multiple white spaces after else

https://gerrit.wikimedia.org/r/277227

Lethexie renamed this task from Fix IfElseStructureSniff can't detect and fix multiple whitespace after else to Fix IfElseStructureSniff can't detect and fix multiple white spaces after else.Mar 14 2016, 12:34 PM
Lethexie updated the task description. (Show Details)

Change 277557 had a related patch set uploaded (by Billm):
Add ability for IfElseStructureSniff to detect multiple white spaces after else

https://gerrit.wikimedia.org/r/277557

Change 277227 merged by jenkins-bot:
Make IfElseStructureSniff can detect and fix multiple white spaces after else

https://gerrit.wikimedia.org/r/277227

Change 277557 abandoned by Umherirrender:
Add ability for IfElseStructureSniff to detect multiple white spaces after else

Reason:
Superseeded by Ie645bd1217614c828d146efac964f690ac48b6f1

https://gerrit.wikimedia.org/r/277557

Umherirrender closed this task as Resolved.Jun 29 2017, 6:33 PM