"^demon" posted a comment on MediaWiki.r111264. URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/111264#c30835
Commit summary for MediaWiki.r111264: adding DBDataObject class after having some people review it and posting on the list. docs can currently be found at https://www.mediawiki.org/wiki/User:Jeroen_De_Dauw/DBDataObject ^demon's comment: Not necessarily more sane, just lazier. Instantiating classes to call a method is a basic tenet of OOP. Static should be used for things that truly should be called from outside of the class on a non-instance of it (such as factory methods or utility functions). Making it static to save a line of code at the expense of making the code more complex isn't a win in my book. _______________________________________________ MediaWiki-CodeReview mailing list [email protected] https://lists.wikimedia.org/mailman/listinfo/mediawiki-codereview
