Comment 3 for bug 1416890

Revision history for this message
Aaron Wells (u-aaronw) wrote :

Hi Gilles,

If you're interested in contributing your code back upstream, there are some instructions on the mahara.org wiki: https://wiki.mahara.org/index.php/Developer_Area/Contributing_Code

We use the gerrit code review system, which is based on git. If you can't get that working, I'm also willing to just take a look at a patch file or a github branch. :)

I suppose what we would want, is a new method, $THEME->get_image_url($imagename). This would be equivalent to a call to $THEME->get_url("images/{$imagename}.png"), except that it would check for multiple file extension types. And then we'd want to go through the existing code and find all the current places that are calling $THEME->get_url() for images and replace them with calls to $THEME->get_image_url().

This would multiply the number of filesystem checks to find an image file, so it indeed might be good to do this in conjunction with some cacheing to cut down on that.

Cheers,
Aaron