mirror of
https://github.com/doctrine/orm.git
synced 2026-03-23 22:42:18 +01:00
DDC-267: CLI does only show last error message and ignores stacked exceptions #333
Reference in New Issue
Block a user
Delete Branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Originally created by @doctrinebot on GitHub (Jan 22, 2010).
Originally assigned to: @guilhermeblanco on GitHub.
Jira issue originally created by user shurakai:
If exceptions are stacked, only the last message is shown. The attached patch fixes this.
@doctrinebot commented on GitHub (Jan 22, 2010):
Comment created by romanb:
Apparently there is a --trace option that, if enabled, shows the full stack trace.
Is that not sufficient? The patch merely concatenates exception messages, even without a separator.
The stack trace should probably always be shown?
@doctrinebot commented on GitHub (Jan 22, 2010):
Comment created by shurakai:
Yes, I would disable --trace
The stacktrace is not sufficient because e.g. the ReflectionException generates a very useful message and is afterwards stacked by the MappingException to add more information, but the original message is lost...
Btw., yeah, maybe the output could be better ;)
@doctrinebot commented on GitHub (Jan 25, 2010):
Comment created by @guilhermeblanco:
The MappingException should be fixed then.
There's no reason to add a $e->getPrevious() lookup in CLIController.
Also... it's even a break in Exception class. There's no method prototype in PHP Exception class.
If MappingException is hiding something, point it to me and I fix it.
@doctrinebot commented on GitHub (Jan 25, 2010):
Comment created by @beberlei:
@Guilherme - That was because of the your latest commit on additional entity class information :-)
I guess the way to go (and i already initiazed first contact) is to patch the Reflection C API in PHP to show the class name instead.
@doctrinebot commented on GitHub (Apr 12, 2010):
Comment created by @guilhermeblanco:
That's a PHP issue, as Benjamin stated. Patch already submitted to PHP core team.
Cheers,
@doctrinebot commented on GitHub (Apr 12, 2010):
Issue was closed with resolution "Fixed"
@doctrinebot commented on GitHub (Dec 13, 2015):
Imported 1 attachments from Jira into https://gist.github.com/945b1db15562118749d6