Task #33903
Make exceptions clearer
| Status: | Resolved | Start date: | 2012-02-14 | |
|---|---|---|---|---|
| Priority: | Should have | Due date: | ||
| Assignee: | Ingo Pfennigstorf | % Done: | 100% |
|
| Category: | Extbase | |||
| Target version: | Extbase 6.0 | |||
| Has patch: | Yes | Tags: | ||
| Votes: | 0 |
Description
Hello guys,
using extbase is nice, but exceptions like #1295479650 can drive you nuts.
It would be nice, if these exceptions would offer an idea of where the error could be.
E.g. this exception was thrown because i wrote:
page.600 = USER
page.600 {
...
extensionName = simulatefe
}
The exception text is the following:
The default controller can not be determined.
In my oppinion the message of the exception should be:
The default controller can not be determined for "simulatefe".
It's a tiny change, but it can be very time saving. It's the same thing, if you write some TS with FLUIDTEMPLATES (many) and than you have a typo in one of the path, the exception doesn't tell you where it struggles ;) - please fix that ;)
Thanks
Associated revisions
[Task] Make Exception for Controllerdetermination more expressive
This makes exception 1316104317 more expressive and includes the
extension- and pluginname in the exception message.
Change-Id: Ia4787748487e32cd112327616992914bbf8bc736
Fixes: #33903
Releases: 6.0, 4.7, 4.6
Reviewed-on: http://review.typo3.org/12945
Reviewed-by: Tymoteusz Motylewski
Reviewed-by: Oliver Klee
Reviewed-by: Markus Günther
Tested-by: Markus Günther
Reviewed-by: Anja Leichsenring
Tested-by: Anja Leichsenring
[Task] Make Exception for Controllerdetermination more expressive
This makes exception 1316104317 more expressive and includes the
extension- and pluginname in the exception message.
Change-Id: Ia4787748487e32cd112327616992914bbf8bc736
Fixes: #33903
Releases: 6.0, 4.7, 4.6
Reviewed-on: http://review.typo3.org/16509
Reviewed-by: Anja Leichsenring
Tested-by: Anja Leichsenring
Reviewed-by: Wouter Wolters
Reviewed-by: Ingo Pfennigstorf
Tested-by: Ingo Pfennigstorf
History
Updated by Ingo Pfennigstorf 10 months ago
- Status changed from New to Accepted
- Priority changed from Must have to Should have
- Target version changed from 4.5.12 to 6.0.0
Updated by Gerrit Code Review 10 months ago
- Status changed from Accepted to Under Review
Patch set 1 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/12945
Updated by Ingo Pfennigstorf 10 months ago
- Has patch changed from No to Yes
Updated by Gerrit Code Review 10 months ago
Patch set 2 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/12945
Updated by Ingo Pfennigstorf 10 months ago
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
Applied in changeset ead30e93f75ed79f7955a3085e2b2ff0c398c3c8.
Updated by Marc Bastian Heinrichs 7 months ago
- Target version changed from 6.0.0 to Extbase 6.0
Updated by Gerrit Code Review 6 months ago
- Status changed from Resolved to Under Review
Patch set 1 for branch extbase_4-7 has been pushed to the review server.
It is available at http://review.typo3.org/16509
Updated by Ingo Pfennigstorf 6 months ago
- Assignee set to Ingo Pfennigstorf
Updated by Anja Leichsenring 6 months ago
- Status changed from Under Review to Resolved