https://bugzilla.wikimedia.org/show_bug.cgi?id=29246
Roan Kattouw <[email protected]> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |RESOLVED Resolution| |FIXED --- Comment #3 from Roan Kattouw <[email protected]> 2011-08-26 16:27:03 UTC --- (In reply to comment #2) > (In reply to comment #1) > > I guess the best way to fix this would be to refactor internalAttemptSave() > > to > > return a Status object instead of one of 30+ error codes. > > > > Please! For back-compat, you can set the error code in $status->value on > failure. Thanks for that tip. I took it a little bit further and just made all flow decisions based on $status->value, because EditPage is such a mess. Fixed in r95562. -- Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug. _______________________________________________ Wikibugs-l mailing list [email protected] https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
