caorongjin/liferay-portal — pull request #54 by danielreuther (2012-05-11 10:05:41+00:00)
Title: LPS-26497, LPS-26662

Hi Alex,

@sergiogonzalez asked me to send the pull directly to you after I've had another look at it. And he was in fact totally right, we have to rollback LPS-25970 first since ImageImpl relies on the file extension (yeah, I should've checked that before).

Anyway, the point of those fixes is to get rid of the extension checking during the upgrade process (LPS-26497) and preview creation (LPS-26662) because it has a lot of draw backs and is error prone. This branch includes both fixes since the former wouldn't work without the latter. @matethurzo had his doubts about whether DLPreviewableProcessor#addFileToStore is used somewhere else and if the change might therefore cause problems—so that might be something to consider when reviewing this.

Thanks already,
Daniel