Task #33903

Make exceptions clearer

Added by Kay Strobach over 1 year ago. Updated 6 months ago.

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

Revision ead30e93
Added by Ingo Pfennigstorf 10 months ago

[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

Revision 34a32d30
Added by Ingo Pfennigstorf 6 months ago

[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

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

Also available in: Atom PDF