You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
If not, it will probably be more consistent if UsersAddSuperuserCommand reuses the role defined in UsersTable instead of hardcoding superuser. This way we can always read whatever the super-user-admin role name is by calling \CakeDC\Users\Model\Table\UsersTable::ROLE_ADMIN.
UPDATE: Code search says that ROLE_ADMIN is not being used anywhere inside this plugin.
UPDATE According to blame, ROLE_ADMIN originates in ebe0114. Searching ROLE_ though commit source shows only 2 matches where the two constants are introduced, doesn't show any usage.
The text was updated successfully, but these errors were encountered:
Is there a difference between
superuser
andadmin
?users/src/Model/Table/UsersTable.php
Lines 45 to 46 in 9f9f49b
users/src/Command/UsersAddSuperuserCommand.php
Lines 27 to 31 in 9f9f49b
If not, it will probably be more consistent if
UsersAddSuperuserCommand
reuses the role defined inUsersTable
instead of hardcodingsuperuser
. This way we can always read whatever the super-user-admin role name is by calling\CakeDC\Users\Model\Table\UsersTable::ROLE_ADMIN
.UPDATE: Code search says that
ROLE_ADMIN
is not being used anywhere inside this plugin.UPDATE According to blame,
ROLE_ADMIN
originates in ebe0114. SearchingROLE_
though commit source shows only 2 matches where the two constants are introduced, doesn't show any usage.The text was updated successfully, but these errors were encountered: