"Brion VIBBER" changed the status of MediaWiki.r102521 to "resolved" and commented it. URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/102521#c26797
Old Status: fixme > New Status: resolved Commit summary for MediaWiki.r102521: Some updates to EditPage and ExternalEdit: * Made ExternalEdit use a context * Updated DifferenceEngine to use ExternalEdit to send the diff stuff to reduce code duplication * Introduced ExternalEdit::useExternalEngine() to check whether to use the external edit or diff (except for the action, section and oldid which are still checked in Wiki.php) to remove code duplication; external diff can now also be controlled with externaledit or internaledit URL parameters * Use $wgContLang to get the name of the "Special" namespace instead of user's language * Modified the line breaks in the comment on the top of the control file so that the URL is on its own line * Updated extension to call EditPage::edit() instead of EditPage::submit(), the latter will just call the former * Updated extension to let core handle itself the ExternalEdit mode instead of doing this themself Brion VIBBER's comment: All seems to work in my testing with the followups. _______________________________________________ MediaWiki-CodeReview mailing list [email protected] https://lists.wikimedia.org/mailman/listinfo/mediawiki-codereview
