Suggestion #35420
Review use of try / catch in the Security Framework
| Status: | New | Start date: | 2012-03-30 | |
|---|---|---|---|---|
| Priority: | Should have | Due date: | ||
| Assigned To: | Andreas Förthner | % Done: | 0% |
|
| Category: | - | |||
| Target version: | - |
Description
Just to not forget about the comments in https://review.typo3.org/#/c/9558/
Karsten Dambekalns Mar 12
Patch Set 1:
If that is considered proper use of the security framework… would be nice to have something that doesn't use try/catch.
Andreas Förthner Mar 12
Patch Set 1: Looks good to me, but someone else must approve
Currently the security framework works like this. I agree that it is somehow strange to catch an exception here. On the other hand this makes it really explicit and really ensures that you are aware of coding the restricted case. For know I wouldn't change it, but we can discuss if we need some true/false method for access decisions.