Page MenuHomePhabricator

FileSearchIndexer.php code review
Closed, InvalidPublic

Description

Author: alno

Description:
typos and variable name corrections

Bugs found while reviewing MediaWiki extensions code in Eclipse 3.3


Version: unspecified
Severity: trivial

Attached:

Details

Reference
bz12269

Event Timeline

bzimport raised the priority of this task from to Medium.Nov 21 2014, 9:59 PM
bzimport set Reference to bz12269.
bzimport added a subscriber: Unknown Object (MLST).

The code appears quite correct as it is; patch would break it.

getExtractor() instantiates the appropriate class for a given MIME type and returns the object, or null if no class is registered for that type.

Patch would return the class name instead of a new instance of the class, causing the code to fail.