They probably could turn that global class into "facade" - compact form of IoC container Laravel framework uses. Dmitriy
On Thu, Jan 22, 2015 at 12:51 AM, Brian Wolff <bawo...@gmail.com> wrote: > On Jan 21, 2015 1:40 PM, "Jeroen De Dauw" <jeroended...@gmail.com> wrote: > > > > Hey, > > > > Does the new syntax offer any advantage over the old one? > > > Assuming that we want to switch to non-static function calls eventually > > > (which I hope is the case), wouldn't it be friendlier towards extension > > > maintainers to only deprecate once we are there, instead of forcing > them to > > > update twice? > > > > > > > Good points and questions. While this deprecation is not as problematic > as > > simply ditching the current hook system altogether, it does indeed seem a > > bit of busy work. > > > > The Hooks class has this comment "Used to supersede $wgHooks, because > > globals are EVIL.", which is quite amusing if you consider all fields and > > methods are static. So it's a switch from a global var to a global field, > > thus adding a second global to get rid of the first one. I have this > > presentation on static code which has a screenshot of this comment and > > class in it :) > > > > Cheers > > > > -- > > Jeroen De Dauw - http://www.bn2vs.com > > Software craftsmanship advocate > > Evil software architect at Wikimedia Germany > > ~=[,,_,,]:3 > > Ill be honest i dont understand the point of deprecating that. As you say > the evil globalness is the same amount of evil regardless of the type of > global symbol. And really i dont think global hooks causes too many > problems. > > --bawolff > _______________________________________________ > Wikitech-l mailing list > Wikitech-l@lists.wikimedia.org > https://lists.wikimedia.org/mailman/listinfo/wikitech-l > _______________________________________________ Wikitech-l mailing list Wikitech-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikitech-l