Hi! I've made significant cleanup and restructurization of my extension's code that I'd like to submit to SVN. Before trying to submit my extension, I'd like to ask two important questions related to Parser and Article cache.
1. I've implemented my own parser function. The description of function is located at the following page: http://www.mediawiki.org/wiki/Extension:QPoll#qpuserchoice_parser_function
The source code class qp_FunctionsHook is located in 'qp_user.php' file inside tgz archive: http://mediawiki.narod.ru/QPoll_0.6.0.tgz (if the one cares to take a quick look)
The function seems to work: it returns a string when #qpuserchoice function parameters are correct, or returns an array( 0=>'<span class="error">qpuserchoice: ' . wfMsgHTML( 'qp_func_' . $this->error_message, htmlspecialchars( $this->pollAddr ), htmlspecialchars( $this->question_id ), htmlspecialchars( $this->proposal_id ) ) . '</span>', 'isHTML'=>true ); in case of error.
I've recieved a bug report from extension user who complained that #iferror check on my function fails. Quick test shows that is true: * {{#expr: 1 + 1 }} *** displays correct result * {{#iferror: {{#expr: 1 + 1 }} | error | correct }} *** #iferror returns correct * {{#expr: 1 + X }} *** displays error message * {{#iferror: {{#expr: 1 + X }} | error | correct }} *** #iferror returns error * {{#qpuserchoice: #almaz1}} *** displays error message (not enough function parameters given) * {{#iferror: {{#qpuserchoice: #almaz1}} | error | correct }} *** #iferror returns correct - though there should be an error !!! * {{#qpuserchoice: #almaz1|3|2}} *** displays correct result (when the poll was previousely defined and submitted) * {{#iferror: {{#qpuserchoice: #almaz1|3|2}} | error | correct }} *** #iferror returns correct
(I know that ParserFunctions returns '<strong class="error">', instead of span - I've tried that with no change)
When checking for #iferror implementation (in 'extensions/ParserFunctions/ParserFunctions.php'), the method iferror( &$parser, $test = '', $then = '', $else = false ) recieves correct $test parameter values from "erroneous" #expr function: string (105) "<strong class="error">Ошибка выражения: неопознанное слово «x»</strong>" ("ru" locale wiki)
but an incorrect value from "erroneous" #qpuserchoice function: string(33) "UNIQ2c0ad9e32fb631-item-1--QINU"
Positive results are correct strings for both functions, so the problem is with error generation.
2. My extension generates dynamical content. Because of that, I use $parser->disableCache() in my tag parser hook. But, the dynamical content is being changed only in two cases:
a) The user edits the page. In such case, disableCache() is not required, because Article cache should be properly flushed by core.
b) The user submits the poll. After the processing of POST data I redirect with 302. In such case, which is the best way to conditionally flush the cache - will parser->disableCache() work after the POST and before the 302? To me it looks that it won't have an effect on the next GET after the redirect. Or, I'd better use Article::doPurge() (or, that is too slow?). Dmitriy