User:MarkAHershberger/Weekly reports/2010-12-20/log/2010-12-15

I was doing code review and came across your modifications to core for DataTransclusion (r67819, r67820, r67821). I tried the tests in your extension, but they failed to work. Do they work for you still? It would be good to have the tests working, and the extension documented. I found the documentation on mediawiki.org, but it seemed incomplete to me. I'm willing to help you get the tests working and/or update the documentation -- Since I maintain the code on the test server, so getting the tests to run regularly against trunk shouldn't be too much of a problem. Where can I start?
 * Code Review
 * 67819, 67820, 67821 — found and commented on superfluous assignment, tried and failed to get the tests working.... it is important to get the test runner working for extensions. Sent email to the developer, Duesentrieb offering to help.
 * 69875, 69949 — replace '\0' with "\0" — makes total sense to me!
 * 68342