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

Always enabled display errors in developer mode #114

Closed
wants to merge 1 commit into from

Conversation

amenk
Copy link
Contributor

@amenk amenk commented Nov 5, 2012

No description provided.

@magento-team
Copy link
Contributor

Hi @amenk

We can't move ini_set('display_errors',1) to developer mode section.
There are a few reasons:

  • It should be before all includes like ( require_once BP . '/lib/Magento/Autoload.php'; )
  • Display errors should not depend on developer mode (Developer mode will catch errors regardless display errors enabled)

Thanks!

vpelipenko pushed a commit to vpelipenko/magento2 that referenced this pull request Mar 5, 2015
[Firedrakes] MTF Fixtures and Repositories configuration. MTF Dynamic Generation
magento-engcom-team pushed a commit that referenced this pull request Apr 13, 2018
magento-engcom-team added a commit that referenced this pull request Apr 28, 2018
 - Merge Pull Request magento-engcom-team/magento2ce#114 from magento-engcom-team/magento2ce:forwardport-2.3-develop-magento-magento2-11539
 - Merged commits:
   1. da5692c
   2. 9775cd8
   3. e10a3ff
magento-engcom-team added a commit that referenced this pull request Jul 12, 2018
 - Merge Pull Request magento/graphql-ce#114 from magento/graphql-ce:grqphql-mutations-74
 - Merged commits:
   1. db3f2c7
   2. eb2a5a3
magento-engcom-team pushed a commit to okorshenko/magento2 that referenced this pull request May 31, 2019
magento-devops-reposync-svc pushed a commit that referenced this pull request May 19, 2023
* LYNX-149: Cache identity for attributesMetadata query
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Issue: Ready for Work Gate 4. Acknowledged. Issue is added to backlog and ready for development
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants