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.

Sign up for GitHub

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

Jump to bottom

Bakery errors show in HTML #751

Closed
abdullahseba opened this issue Jun 28, 2017 · 8 comments
Closed

Bakery errors show in HTML #751

abdullahseba opened this issue Jun 28, 2017 · 8 comments
Assignees
@lcharette
Labels
bakery Related to the Bakery feature user experience The one using UF

Comments

@abdullahseba
Copy link
Contributor

clipboard - june 28 2017 11_06 pm
@lcharette lcharette self-assigned this Jun 28, 2017
@Silic0nS0ldier
Copy link
Member

This has been around even before bakery. Any fatal unhandled error in any php based cli in uf does this.

@lcharette
Copy link
Member

I can't even replicate an error that throw that kind of error :/

@alexweissman
Copy link
Member

I'm pretty sure this is caused by the implementation of ShutdownHandler: https://github.com/userfrosting/UserFrosting/blob/master/app/sprinkles/core/src/Util/ShutdownHandler.php#L62-L69

@abdullahseba
Copy link
Contributor Author

@Silic0nS0ldier Maybe this is a windows only issue. Can you replicate?

@abdullahseba
Copy link
Contributor Author

@alexweissman I think so too from the error log.

@lcharette
Copy link
Member

Ok got it. Need a syntax error in migration.

Problem is we define the thing as CLI here, but we can't access that config elsewhere, like here.

lcharette added a commit that referenced this issue Jun 29, 2017
@lcharette
Fix Bakery errors show in HTML (#751)
8b151b8
@lcharette lcharette added bakery Related to the Bakery feature user experience The one using UF labels Jun 29, 2017
@Silic0nS0ldier
Copy link
Member

Just for the record, any error that causes the slim application error handling to kick in will result in this being output. Would be nice if there were a way to make it more CLI friendly.

@Silic0nS0ldier
Copy link
Member

Silic0nS0ldier commented Jun 29, 2017 •
edited
Loading

Output comes from here for reference.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@lcharette lcharette

Labels
bakery Related to the Bakery feature user experience The one using UF
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@lcharette @alexweissman @Silic0nS0ldier @abdullahseba