User "Nikerabbit" changed the status of MediaWiki.r95562. Old Status: ok New Status: fixme
User "Nikerabbit" also posted a comment on MediaWiki.r95562. Full URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/95562#c22698 Commit summary: (bug 29246) Edit API occasionally throws "unknown error 231". Per comment #1 on the bug and a comment in ApiEditPage, I refactored internalAttemptSave() to return a status object. However, there are so many idiosyncracies wrt how EditPage handles various errors that I decided to put the AS_* error code in $status->value (thanks Chad for that tip) and use that to decide what to do (not part of Chad's tip). The resulting code is still a mess but at least Status objects from doEdit() are propagated now, and it'll be a little bit easier to migrate internalAttemptSave() to a proper Status-based architecture in the future. Or maybe we should just throw away EditPage and start with a blank screen, that sounds appealing to me :) Comment: This seems to miss one case, which results in warnings like: <pre> [17-Sep-2011 11:34:40] PHP Notice: Trying to get property of non-object in /www/w/includes/EditPage.php on line 2893 [17-Sep-2011 11:34:40] PHP Notice: Trying to get property of non-object in /www/w/includes/EditPage.php on line 2897 </pre> In trunk the offending line is 1135. _______________________________________________ MediaWiki-CodeReview mailing list [email protected] https://lists.wikimedia.org/mailman/listinfo/mediawiki-codereview
