"Jeroen De Dauw" posted a comment on MediaWiki.r111264. URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/111264#c30870
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 Jeroen De Dauw's comment: (unindent, reply to [[Special:Code/MediaWiki/111264#c30852]]) That's an interesting idea. I could live with having these methods non-static in a class representing a table. I'm not exactly sure how the split would work. Would you have a table class and a row class for each table? If so, the table classes would not hold a lot of stuff, they would just hold these methods plus something that associates their corresponding row class (unless you have some kind of table registration system, in which case you could have one single table class which knows what to use if you specify which table you are talking about and return an object of the right row class). I'd love to see more ideas on how this should be done. _______________________________________________ MediaWiki-CodeReview mailing list [email protected] https://lists.wikimedia.org/mailman/listinfo/mediawiki-codereview
