ext.popups.renderer.article.js
- Requests the "info" property inprop=watched, but never tests for "watched". Why request it?
- Doesn't seem to use any of the prop=info properties (contentmodel, pagelanguage, touched, lastrevid, and length). I think it could stop and drop that prop, it's a NOP :)
Brad Jorsch points out instead of requesting indexpageids=true in order to "Include an additional pageids section listing all returned page IDs", Popups could request formatversion=2 which returns "pages" in the query result as an array, simplifying the logic. Instead of
re.query.pages[ re.query.pageids[ 0 ] ].extract
just
re.query.pages[0].extract
etc.