Difference between revisions of "Brettwilkins"

From Mahara Wiki

Line 19: Line 19:
===Reviewing other's patches===
===Reviewing other's patches===
[,384 Mahara Gerrit, Changeset 384]
* [,384 Mahara Gerrit, Changeset 384]
* [,385 Mahara Gerrit, Changeset 385]

Latest revision as of 16:06, 24 June 2011

Brett has been working at Catalyst IT Ltd since January 2010, taking up development primarily on the Moodle 1.9.x codebase. As of 2011, he is now taking on more Mahara work where possible.


Overhaul of the externalvideo blocktype

Gitorious diff here:

This patch splits out the logic for matching processing URLs on a per-provider basis. Also supported now are iframes, if the provider supports them. It is now trivial to add in page scraping for sites that require it to get a meaningful, embed-able URL from one which is not, and this can be implemented to occur once only, rather than each page load. See WikiEducator and Glogster for examples.

By breaking providers out into their own files and folders, I've made it much simpler to add new providers as necessary. (Documentation pending ;) ) - unfortunately, language strings still need to go into the one externalvideo language file.

coding standards test

This patch adds coding standards/guidelines checking to mahara and into the Makefile.

Reviewing other's patches