Fixes for r72024:
authorTim Starling <tstarling@users.mediawiki.org>
Tue, 7 Sep 2010 10:38:19 +0000 (10:38 +0000)
committerTim Starling <tstarling@users.mediawiki.org>
Tue, 7 Sep 2010 10:38:19 +0000 (10:38 +0000)
commitdb643f03c27ce035441cff3085083c2189abcbfa
tree68dd4436b7ae06570be65924f3780bbbeeb25b0f
parent8d6d6539a78fe78b265e90056ae988981228eae9
Fixes for r72024:
* Renamed MediaHandler::verifyFileHook() to verifyUpload() since it isn't a hook and the fact that it operates on files is obvious.
* Separated some concerns by simply passing verifyUpload() function a file path instead of an UploadBase object and MIME type. This simplifies the implementation of subclasses, makes the function accessible to non-UploadBase callers, and avoids breaking the interface constantly due to UploadBase changes.
* Have verifyUpload() return a Status object instead of allowing the idiosyncratic and feature-poor error array convention from UploadBase to infect MediaHandler.

The required update to PagedTiffHandler will be in a subsequent commit.
includes/media/Generic.php
includes/upload/UploadBase.php