Elgg/community_plugins — pull request #72 by juho-jaakkola (2014-08-20 09:15:43+00:00)
Title: Enhances thumbail handling

I did the mistake of taking a look at the thumbnail code and simply couldn't help doing at least some refactoring to make it a bit less terrible:
- Removes duplicate image handling code
- Serves the images without loading the engine
- Makes the code more readable

What do you think of the approach?

BTW, we allow users to define ```access_id``` for the plugins, so should the images also respect it? I find it tempting to serve the images without checking the read access in ```image.php```.