It's really hard to read the diff because it looks like the line indenting was changed so it thinks the entire file is new :(
So as I understand it there's a check if the user is authenticated for that module. If the module doesn't exist in the DB then users that aren't SuperAdmins will fail that check and the code indicated in p= will never be loaded?
If I've understood it correctly then yes that sounds fine.
It's really hard to read the diff because it looks like the line indenting was changed so it thinks the entire file is new :(
So as I understand it there's a check if the user is authenticated for that module. If the module doesn't exist in the DB then users that aren't SuperAdmins will fail that check and the code indicated in p= will never be loaded?
If I've understood it correctly then yes that sounds fine.