"Aaron Schulz" changed the status of MediaWiki.r105851 to "ok"
URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/105851

Old status:  new
> New status: ok

Commit summary for MediaWiki.r105851:

Reverted r92364 (per-namespace permissions).

This is the wrong configuration format for such a feature, and the wrong 
interface. We already have certain per-namespace permissions in the Title 
class, and we didn't need to add extra formal parameters to a whole lot of User 
methods in order to get them. The feature should be implemented wholly in 
Title, and the concept of user rights should remain relatively simple and easy 
to understand, and independent of its many applications, i.e. a user either has 
a right or doesn't. Rights are just a tool for developing access policies; the 
complexity should be in the caller.

The revert was mostly done by hand, since there were a lot of conflicts. I 
tried to preserve the gist of conflicting changes in r102187 and r102873. The 
test changes are not simple reverts, rather I just edited out the per-namespace 
tests. I reverted the followups r92589 and r104310.

_______________________________________________
MediaWiki-CodeReview mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-codereview

Reply via email to