Uploaded image for project: 'Moodle'
  1. Moodle
  2. MDL-1983

Add error_reporting to config.php

    Details

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

      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.

        Gliffy Diagrams

          Attachments

            Activity

            imported Imported created issue -
            Hide
            dougiamas 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
            dougiamas 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.
            mblake Michael Blake made changes -
            Field Original Value New Value
            Assignee Martin Dougiamas [ martindougiamas ] Martin Dougiamas [ dougiamas ]
            dougiamas Martin Dougiamas made changes -
            Status In Progress [ 3 ] Open [ 1 ]
            Hide
            poltawski Dan Poltawski added a comment -

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

            Show
            poltawski Dan Poltawski added a comment - Pretty sure this is fixed with the current debug error reporting levels!
            poltawski Dan Poltawski made changes -
            Status Open [ 1 ] Closed [ 6 ]
            Resolution Fixed [ 1 ]
            Show
            imme-emosol 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 ?
            dougiamas Martin Dougiamas made changes -
            Workflow jira [ 12664 ] MDL Workflow [ 49600 ]
            dougiamas 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: