-
-
Notifications
You must be signed in to change notification settings - Fork 633
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Include extra args in AccessDenied exceptions #173
Include extra args in AccessDenied exceptions #173
Conversation
What's the status of this PR? |
|
||
1.10.0 (August 28, 2014) | ||
|
||
* Complete cancancan#115 - Specify authorization action for parent resources. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please separate the two features in two different PR.
@@ -376,6 +376,7 @@ class Container < Hash; end | |||
expect(e.message).to eq("Access denied!") | |||
expect(e.action).to eq(:read) | |||
expect(e.subject).to eq(:foo) | |||
expect(e.extra_args.first).to eq(1) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
please add some more tests: check what is the content of extra_args
in different cases (also when there is no extra_args)
I'd be happy to include these features but I need two different PR. Thanks for your work |
It's been 2 years since I submitted this PR. Unfortunately we've moved on from cancan and I don't really have time to invest in splitting this into separate pieces. |
Completes #172