Moodle
  1. Moodle
  2. MDL-1983

Add error_reporting to config.php

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Trivial Trivial
    • Resolution: Fixed
    • Affects Version/s: 1.5
    • Fix Version/s: None
    • Component/s: General
    • Labels:
      None
    • Environment:
      All
    • Affected Branches:
      MOODLE_15_STABLE
    • Rank:
      14018

      Description

      Moodle does not currently work properly with error_reporting(E_STRICT). Does it all with E_ALL?

      Knowing what error level an app is being programmed for is always good to know IMHO.

      I suggest you put error_reporting(E_ALL) into config.php (or some other central file) and ask developers to set this to E_STRICT when developing.

        Activity

        Imported created issue -
        Hide
        Martin Dougiamas added a comment -

        From Hannes Gassert (hannes at mediagonal.ch) Thursday, 23 September 2004, 04:46 AM:

        Moodle does not work with error_reporting(E_STRICT), so it would be good to know what error level is used by default. I suggest putting E_ALL into config.php and ask developers to set this to E_STRICT when developing?

        Defining a default error level would be a good idea in any case, I guess?

        From Hannes Gassert (hannes at mediagonal.ch) Thursday, 23 September 2004, 04:46 AM:

        Jesus, why did I write this twice?

        From Martin Dougiamas (martin at moodle.com) Thursday, 23 September 2004, 12:21 PM:

        Have you seen the debug variable on admin >> config >> variables? it wswitches between 7 and 15 ...

        From Hannes Gassert (hannes at mediagonal.ch) Thursday, 23 September 2004, 08:56 PM:

        I did not check that variable, all I saw is that on my box (that has E_STRICT as default error level) a fresh copy from CVS HEAD shows some ugly warnings, about the usage of StdClass for example.

        Show
        Martin Dougiamas added a comment - From Hannes Gassert (hannes at mediagonal.ch) Thursday, 23 September 2004, 04:46 AM: Moodle does not work with error_reporting(E_STRICT), so it would be good to know what error level is used by default. I suggest putting E_ALL into config.php and ask developers to set this to E_STRICT when developing? Defining a default error level would be a good idea in any case, I guess? From Hannes Gassert (hannes at mediagonal.ch) Thursday, 23 September 2004, 04:46 AM: Jesus, why did I write this twice? From Martin Dougiamas (martin at moodle.com) Thursday, 23 September 2004, 12:21 PM: Have you seen the debug variable on admin >> config >> variables? it wswitches between 7 and 15 ... From Hannes Gassert (hannes at mediagonal.ch) Thursday, 23 September 2004, 08:56 PM: I did not check that variable, all I saw is that on my box (that has E_STRICT as default error level) a fresh copy from CVS HEAD shows some ugly warnings, about the usage of StdClass for example.
        Michael Blake made changes -
        Field Original Value New Value
        Assignee Martin Dougiamas [ martindougiamas ] Martin Dougiamas [ dougiamas ]
        Martin Dougiamas made changes -
        Status In Progress [ 3 ] Open [ 1 ]
        Hide
        Dan Poltawski added a comment -

        Pretty sure this is fixed with the current debug error reporting levels!

        Show
        Dan Poltawski added a comment - Pretty sure this is fixed with the current debug error reporting levels!
        Dan Poltawski made changes -
        Status Open [ 1 ] Closed [ 6 ]
        Resolution Fixed [ 1 ]
        Show
        W.H. Fris added a comment - Related to: http://moodle.org/mod/forum/discuss.php?d=133870#p617572 http://tracker.moodle.org/browse/MDL-1983?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel http://tracker.moodle.org/browse/MDL-15267 ?
        Martin Dougiamas made changes -
        Workflow jira [ 12664 ] MDL Workflow [ 49600 ]
        Martin Dougiamas made changes -
        Workflow MDL Workflow [ 49600 ] MDL Full Workflow [ 78679 ]

          People

          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: