search

k8thekat / GatekeeperV2

Update to Gatekeeper
GNU General Public License v3.0
27 stars 11 forks source link

No module named "AMP_Permissions" #78

Closed Acheronx98 closed 1 year ago

Acheronx98 commented 1 year ago

Doing some testing I found a bug having to deal with "what happens if Gatekeeper no longer exists?" in AMP so, tl;dr it breaks it, obviously, but it should re-create the role [giving the user Gatekeeper has assigned to it] and assign it all the perms it needs. That's not the case. It looks like a misspelled module is breaking it, I don't know, it's weird.

image

Changing line 244 in AMP.py fixes the issue Rename from 'AMP_Permissions' to 'amp_permissions' image image

k8thekat commented 1 year ago

I wonder if somehow during an update your file got renamed to a lowercase version.

My Linter screams at me trying to use the lowercase version. Ill give this some testing. Thanks.

k8thekat commented 1 year ago

Closing this issue as I believe it was an isolated issue.