Quote Originally Posted by mc12345678 View Post
So now instead of an obvious display issue identifying that there is a problem with the image(s), an incorrectly loaded image will just not be resized and the store owner that doesn't/hasn't properly loaded the images to the site will be none-the-wiser until some other tool or resource identifies that the image(s) should be resized to optimize performance or there is some other effect on people selecting the site...

In other words, has anything really been solved? I bring this up also because other similar modifications have been discussed throughout this thread as well as in other threads, in fact if you look back at the history of the plugin errors have been uncommented before only to be again commented out like what has been "discouraging".

Generally though, the solution?! Follow the instructions...

Seems almost might be better to simply remove the plugin if it can not be used properly.

If, however, going to use code like above, I would recommend that the applicable image type constant be used to maximize compatibility and code understanding when comparing the image type against $image_type. Such as: IMAGETYPE_JPEG, IMAGETYPE_GIF, IMAGETYPE_PNG, etc. instead of just a number. Based on looking over a few other things it seems that if going to go to such extremes, then should also validate that the image is large enough to support evaluation with the applicable function so that won't throw another error that was just attempted to be prevented.
#1 a mostly blank page with NO error log generated is what the current code does. your "average joe" gets a blank page, posts on this forum, and the 1st question, most of us ask, "are there any debug logs?" no debug logs makes the task that much harder.
#2 i agree. follow the instructions. when you only have clients that follow instructions, life is so much easier.... please pass those clients my way....
#3 i disagree with regards to your remove the plugin comment.
#4 perhaps just remove the '@' from the above 3 referenced functions? the functionality remains the same and the system now generates debug logs.