"Jeroen De Dauw" posted a comment on MediaWiki.r106883. URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/106883#c28213
Commit summary for MediaWiki.r106883: partial revert of r106872 after discussion with Brion on #mediawiki irc Jeroen De Dauw's comment: Having the deprecation notice system is a means for extension developers to find out if they are using any deprecated code that they will need to migrate away from or risk sudden breakage two or three releases of MW later. > the function should not be deprecated Sure it should. Not putting in the notice because it might cause warnings for people that do not care about them breaks the whole system, as it does not allow people who care about them to get them. Note the various settings that in practice by default hide most warnings, and need to be explicitly changed to do show them. If there is a lazy extension developer that does not want to care at all, they'll leave dev notices off, not turn them off and put everything in the whitelist, that is just absurd. > extension developer need help to migrate its code On the other hand, the whitelist allows developers to ignore deprecation of functions they cannot currently migrate, which in my experience mainly occurs when having wide compat restraints, while still getting everything else. So this argument is as invalid as the other one. _______________________________________________ MediaWiki-CodeReview mailing list [email protected] https://lists.wikimedia.org/mailman/listinfo/mediawiki-codereview
