Directly return True for superuser permission list check

[Rohan Jain]

For super users the has_perms method in auth.User should directly return True, instead of iterating through the perm_list.

[Julien Phalip]

Thank you for the suggestion, but the reason Django provides the has_perm() hook is exactly so that you can customize permissions, and one can absolutely imagine having for example a row-level permission disallowing some or even all superusers.

[Rohan Jain]

The original has_perms too will return True for
superusers as the function is finally calling self.has_perm
which would never return False for superusers.

Hence, in case of overridden backend.has_perm, shouldn't
user.has_perm too let the backend handle superuser instead of
returning True?

[Julien Phalip]

My point was that you can override the has_perm() hook exactly to change the default behaviour, for example to disallow permission for some or even all superusers. If you shortcut this logic from inside has_perms() then the has_perm() hook would become pointless and the system would lose in flexibility.