search

bdraco / addon_securityadvisor

Old Security Advisor for cPanel 11.40 (upstream repo) -- Use CPANELINC/addon_securityadvisor
Other
16 stars 56 forks source link

Implemented changes for Issue #12 #45

Closed demasu closed 11 years ago

demasu commented 11 years ago

This is a solution for issue #12. The info text could be changed, but I did some testing and think this is a good message for it. However feel free to change that if you think it should be different.

There is a small concern I have, in that should wheel users have a cPanel account at all? If I read the code correctly this gets all the users with a cPanel account and those are the ones we're searching through. This can be discussed in a side bar later, however, as it's not entirely relevant for this fix. screen shot 2013-06-11 at 12 25 53 pm

demasu commented 11 years ago

Looks like my commits for issue #28 got added in here as well.

bdraco commented 11 years ago

Code review inline.