User "Aaron Schulz" posted a comment on MediaWiki.r91284.

Full URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/91284#c20691
Commit summary:

* Changed action=revert to use a subclass of Action
* Added WikiPage::getActionOverrides() to be able to execute different actions 
depending on the namespace (obviously needed for action=revert). This is only 
used when the value of $wgActions for the corresponding action is true; so 
extension can still override this.
* Added Action::getDescription() to ease the change of the page header and the 
<title> element

Comment:

It bothers me that action stuff (with is UI level) is in the WikiPage classes, 
which are supposed to just be DAOs (with a bit of business logic for now).

_______________________________________________
MediaWiki-CodeReview mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-codereview

Reply via email to