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

Assigned To

None

Authored By

	Lethexie
	Mar 14 2016, 12:00 PM

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

Details

	Subject	Repo	Branch	Lines +/-
	Add ability for IfElseStructureSniff to detect multiple white spaces after else	mediawiki/tools/codesniffer	master	+23 -1
	Make IfElseStructureSniff can detect and fix multiple white spaces after else	mediawiki/tools/codesniffer	master	+27 -0

Customize query in gerrit

Event Timeline

Lethexie created this task.Mar 14 2016, 12:00 PM

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

gerritbot added a project: Patch-For-Review.Mar 14 2016, 12:30 PM

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

Fix IfElseStructureSniff can't detect and fix multiple white spaces after elseClosed, ResolvedPublicActions

Description

Details

Event Timeline

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