Rewrite the ThumbnailImage constructor to take parameter array
authorjarry1250 <jarry1250@gmail.com>
Wed, 29 Aug 2012 20:09:36 +0000 (21:09 +0100)
committerjarry1250 <jarry1250@gmail.com>
Wed, 29 Aug 2012 20:09:36 +0000 (21:09 +0100)
commit58157fd64bc9387ed812a3daaf5e828a6cce1552
treec292343320bfbeaa6309dc749b717ecab79dbcfd
parent52c61302a0586889e1899673f7b387fad2edd69e
Rewrite the ThumbnailImage constructor to take parameter array

This is required for a full resolution of bug #32987. Per Brion's
commentary on that bug, for the long-term preservation of
sanity we can't just keep adding arguments to the constructor
whenever a new feature is added.

Instead, we can have an array of optional parameters. At the moment,
this situation is a bit iffy for width and height: we should really
specify either a default height / width or error if one is not found.
I'm open to ideas on that, but it's not a big deal since nothing
uses the new syntax yet. When that is agreed, I'll convert all usage
in core and maybe some extensions too.

Change-Id: I116f71aeb90ef7c786f0874b150c7bcca527e106
includes/media/MediaTransformOutput.php