Fix bug where admin is able to delete reseller_admin

gholt / swauth

This is the historical location of Swauth; active development is now at https://github.com/openstack/swauth

45 stars 30 forks source link

Fix bug where admin is able to delete reseller_admin #67

Closed thiagodasilva closed 10 years ago

thiagodasilva commented 10 years ago

Changed the code to only allow the super_admin to delete a reseller_admin. This follows the same logic of user creation, where only the super_admin can create a reseller_admin.

Also, took the opportunity to refactor some code and implemented get_user_detail method to remove duplicated code

Signed-off-by: Thiago da Silva thiago@redhat.com

gholt commented 10 years ago

Thanks! I'll try to have a look at this within a few days.

gholt commented 10 years ago

This all looks good to me. Thank you! I had one comment on an inline comment I'm pretty sure was just simple mistake I can fix up, but I figured I'd double check I understood correctly. Let me know and thanks again.