User "Reedy" posted a comment on MediaWiki.r85288. Full URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/85288#c17433 Commit summary:
Implement magic accessors for RequestContext variables: you can now just call $context->request->stuff(), and that is internally mapped to the get accessor. Rename the private variables to the old $mName syntax: I know that that's "discouraged in new code", but in this case it stops over-clever IDEs highlighting the magic accesses as potential visibility errors and sticking big error tags on them. Comment: Just seemingly having 2 references to possibly multiple objects of the same things seems a bit daft/wrong :) Which is more confusing. I was looking at index.php and noticed both seemed to be used at a low level above it (in the RequestContext and the MediaWiki classes) If you want me to find it specifically, let me know, but it'd be nice to tidy it up _______________________________________________ MediaWiki-CodeReview mailing list [email protected] https://lists.wikimedia.org/mailman/listinfo/mediawiki-codereview
