Actions

User

Brettwilkins: Difference between revisions

From Mahara Wiki

No edit summary
 
Line 19: Line 19:


===Reviewing other's patches===
===Reviewing other's patches===
[https://reviews.mahara.org/#change,384 Mahara Gerrit, Changeset 384]
* [https://reviews.mahara.org/#change,384 Mahara Gerrit, Changeset 384]
* [https://reviews.mahara.org/#change,385 Mahara Gerrit, Changeset 385]

Latest revision as of 17:06, 24 Haziran 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.


Contributions

Overhaul of the externalvideo blocktype

Gitorious diff here: https://gitorious.org/mahara/mahara/commit/eb37d4db009426bfc0fceb37d20cef220933a4af

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

https://gitorious.org/mahara/mahara/commit/6c5ce464fd76bbffcbe7dce1ad155d1c47ecf4ed

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

Reviewing other's patches