Last modified: 2011-10-25 22:20:25 UTC
Created attachment 9222 [details] Patch to have ParserCache check for __NOEDITSECTION__ After the fix for bug 31445, __NOEDITSECTION__ is now ignored when the page is loaded from the parser cache. The problem seems to be that ParserCache's get() is not taking into account the possibility of __NOEDITSECTION__ when calling ParserOutput's setEditSectionTokens(). An easy patch is attached, although it may not be the best way to do it.
I see. the parser, which is the entity knowing about __NOEDITSECTION__, is no longer run there, so it doesn't come to decide on it and the default (ie. without taking __NOEDITSECTION__ into account) is used. Still, I don't think that logic should be in ParserCache. Should be fixed in r100756.