Actions

User

Difference between revisions of "Brettwilkins"

From Mahara Wiki

(Created page with "==Contributions== ===Overhaul of the externalvideo blocktype=== Gitorious diff here: https://gitorious.org/mahara/mahara/commit/eb37d4db009426bfc0fceb37d20cef220933a4af This pat…")
 
Line 1: Line 1:
 +
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==
 
==Contributions==
 
===Overhaul of the externalvideo blocktype===
 
===Overhaul of the externalvideo blocktype===
Line 13: Line 17:
  
 
This patch adds coding standards/guidelines checking to mahara and into the Makefile.
 
This patch adds coding standards/guidelines checking to mahara and into the Makefile.
 +
 +
===Reviewing other's patches===
 +
[https://reviews.mahara.org/#change,384 Mahara Gerrit, Changeset 384]

Revision as of 15:10, 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.


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

Mahara Gerrit, Changeset 384