Skip to content
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

Contact summary (entity view) fails with missing "administer CiviCRM" permission #21

Open
jensschuppe opened this issue Jul 4, 2022 · 2 comments
Labels
bug Something isn't working needs funding There's no sufficient funding for this ticket

Comments

@jensschuppe
Copy link
Contributor

Since the extension tries to add a tab to the contact summary or entity pages (e.g. for ECK entities) and therefore executes API calls without explicit permissions (and thus falling back to administer CiviCRM), when those calls fail, the exception is not caught and the entire view fails with an Authorization fails Yellow Screen of Death".

The code that adds the tabs in resource_civicrm_tabset() should be wrapped with a try-catch to avoid this.

@jensschuppe jensschuppe added the bug Something isn't working label Jul 4, 2022
colemanw pushed a commit to colemanw/de.systopia.resource that referenced this issue Sep 5, 2024
…/guzzlehttp/guzzle-6.5.8

Bump guzzlehttp/guzzle from 6.5.6 to 6.5.8
@mclwill
Copy link

mclwill commented Sep 25, 2024

Any update on when this will be fixed. In my case I am looking for users with the CiviResource permission (but not 'administer CiviCRM') to be able to manage resource assignments

@jensschuppe jensschuppe added the needs funding There's no sufficient funding for this ticket label Sep 25, 2024
@jensschuppe
Copy link
Contributor Author

We're currently not planning to fix it, but a PR will always be welcome (and should be easy in this case).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working needs funding There's no sufficient funding for this ticket
Projects
None yet
Development

No branches or pull requests

2 participants