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.

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

Request: Integrate Report All Errors as a core feature #2460

Closed
scottcwilson opened this issue Jun 17, 2019 · 3 comments · Fixed by #2493
Closed

Request: Integrate Report All Errors as a core feature #2460

scottcwilson opened this issue Jun 17, 2019 · 3 comments · Fixed by #2493

Comments

@scottcwilson
Copy link
Sponsor Contributor

This is tremendously useful for developers. Alternately, instead of integrating the entire module, perhaps it could be integrated without configuration, and triggered when STRICT_ERROR_REPORTING was set.

@lat9
Copy link
Contributor

lat9 commented Jun 17, 2019

If integrated, my preference would be to keep the configuration settings, as the IgnoreDups and Backtrace on Notice settings can be quite useful.

Debugging, for example, anything to do with the account_history_info page results in a ton of duplicated-constant notices since the language files for account and account_history are loaded as well.

@scottcwilson
Copy link
Sponsor Contributor Author

Good point - I guess I was thinking that IgnoreDups would be the default setting because there are many of them.

@torvista
Copy link
Member

there are many of them.

...not in the Admin anymore, apart from payment modules (paypal)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants