Last modified: 2014-09-24 00:06:47 UTC
Created attachment 7327 [details] patch for ParserOutput.php To edit several pages and have them all be parsed the next time they are viewed, an extension can save the edits with $article->doEdit() (with some expensive operations disabled), then expire the affected pages from the parser cache using $title->invalidateCache() afterward. Unfortunately, this doesn't work if doEdit() and invalidateCache() are called in the same second. The attached patch fixes this problem by changing a < to <= in ParserOutput.php. However, this introduces a possible bug opposite to the existing one, in that calling invalidateCache() just BEFORE doEdit() can cause the page to be reparsed unnecessarily. It could also cause other problems, of course...
ok, you don't want to do it like that, because every time you edit a page, it parses during the submit operation, then redirects you to the page and parses it again.
*Bulk BZ Change: +Patch to open bugs with patches attached that are missing the keyword*
Lee, I'm adding the "reviewed" keyword here because it seems you have reviewed your own patch! :-) If you'd like to talk about how to change your approach and revise your patch, please come to #mediawiki on freenode and talk with other developers about the issue. Thanks for the patch!
