search

johnitvn / yii2-rbac-plus

Database role base access control manager for yii2
49 stars 52 forks source link

Rule assignment to permission fails #9

Open Mcmester92 opened 9 years ago

Mcmester92 commented 9 years ago

Hello!

We started to use your RBAC plus extension, and at the beginning it worked fine, but after the updates the assignment fails. We tracked the error message, and it seems that the beforeSave() and afterSave() methods cause the crash, in yii2-rbac-plus/src/models/AuthItem.php line 98 and 123. I attached 2 screenshots, about the issue.

Thanks in advance!

screenshot_26 screenshot_27

johnitvn commented 9 years ago

I have update with this

Can you try to edit file vendor/johnitvn/yii2-rbac-plus/src/models/AuthItem.php add comment to line 98.

Tell me if it worked. I will release a hotfix version

Mcmester92 commented 9 years ago

Yes it is working. Thanks for your fast help, and we are looking forward for your hotfix :)

johnitvn commented 9 years ago

I have release v1.0.4

Mcmester92 commented 9 years ago

Thanks! :) 2015.09.07. 14:58 ezt írta ("John Martin" notifications@github.com):

I have release v1.0.4

— Reply to this email directly or view it on GitHub https://github.com/johnitvn/yii2-rbac-plus/issues/9#issuecomment-138291451 .

Mcmester92 commented 9 years ago

Hmm now that i checked it, I see that you only disabled beforeSave() method. I wrote in the issue, that afterSave() method kills the code too :S I commented out from the code, as you can see on the attachment, and now it finally works. When you asked me yesterday to try it, afterSave() was already commented out, and I forgot to mention that. It is my bad, sorry for that. screenshot_28

johnitvn commented 9 years ago

I have been update. I will take time for upgrade this extension soon, but not at the moment. I'm going to make it more stable

Mcmester92 commented 9 years ago

Thanks again! :)

2015-09-08 17:52 GMT+02:00 John Martin notifications@github.com:

I have been update. I will take time for upgrade this extension soon, but not at the moment. I'm going to make it more stable

— Reply to this email directly or view it on GitHub https://github.com/johnitvn/yii2-rbac-plus/issues/9#issuecomment-138606850 .

tihoho commented 6 years ago

I have been update. I will take time for upgrade this extension soon, but not at the moment. I'm going to make it more stable

wheeeeeen? ^)