vacuum wrote:

Note that if mode isn't defined the usual behaviour will be retained, also the defined behaviour for mode set to none, ordered, and unordered are retained.

Note also that there should be no impact on performance as long as there are no use of the templates. Due to the redirect through the mediawiki-namespace the evaluation of templates will be somwhat heavier than usual.

[[:no:Jeblad]]

vacuum wrote:

Allow use of templates, uses message cache through method call

A sligtly better approach..
[[:no:User:Jeblad]]

Attached:

Assign open DynamicPageList issues to new component maintainer.

sumanah wrote:

John, thanks for the patch. I'm cc'ing one of the extension's authors to request patch review.

Thank you for the patch. However, I don't think this is the right way of implementing this feature for several reasons:

• Shouldn't use the borderline private methods of message cache
• Questionable weather the template should be a specified in such an indirect way (Specify name, which resolves to a system message, which resolves back to a template, seems as if it should just directly take the template. I like mode=template, templatename=foo approach much better personally)
• Some coding convention issues (but those issues are more reflective of the extension in general then of this patch).