[Wikitech-l] Patch for gallery tag, review questions.

I have created a patch for the gallery tag and have been given the following review. https://gerrit.wikimedia.org/r/4609 * JavaScript injection: you can inject javascript: URIs which execute code when clicked * plain links ("link=Firefox") are taken as relative URLs which will randomly work or not work depending on where they're viewed from * need parser test cases to demo it working So my questions are: What would be the recommended way of stripping away javascript from uris? Are there any shared functions which do exactly this? And how would i solve the plain links problem? do a regex check for an absolute uri? e.g http://example.org/foo/bar? And what is "parser test cases", phpunit tests? or some other form of testing? Thank you! Kim Eik.