Details

    • Type: Sub-task Sub-task
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 2.2.2
    • Fix Version/s: 2.3
    • Component/s: General
    • Labels:
    • Testing Instructions:
      Hide

      1/ enable strict mode in php.ini
      2/ force strict mode in main config.php
      3/ browse around a bit

      there are too many areas affected, detailed code review should be enough I guess

      Show
      1/ enable strict mode in php.ini 2/ force strict mode in main config.php 3/ browse around a bit there are too many areas affected, detailed code review should be enough I guess
    • Affected Branches:
      MOODLE_22_STABLE
    • Fixed Branches:
      MOODLE_23_STABLE
    • Pull from Repository:
    • Pull Master Branch:
      w13_MDL-32095_m23_strict2

      Description

      it is not enough to enable E_STRICT in config.php only

      areas to be fixed:

      • all filters
      • backup/restore (Eloy)
      • all themes
      • repositories

        Gliffy Diagrams

          Issue Links

            Activity

            Petr Skoda created issue -
            Petr Skoda made changes -
            Field Original Value New Value
            Link This issue has a clone MDL-32094 [ MDL-32094 ]
            Petr Skoda made changes -
            Pull Master Diff URL https://github.com/skodak/moodle/compare/master...w12_MDL-32094_m23_phpstrict
            Pull Master Branch w12_MDL-32094_m23_phpstrict
            Fix Version/s DEV backlog [ 10464 ]
            Fix Version/s 2.3 [ 10657 ]
            Pull from Repository git://github.com/skodak/moodle.git
            Michael de Raadt made changes -
            Labels triaged
            Rajesh Taneja made changes -
            Petr Skoda made changes -
            Description it is not enough to enable E_STRICT in config.php only it is not enough to enable E_STRICT in config.php only

            areas to be fixed:
            * all filters
            * backup/restore (Eloy)
            * all themes
            Petr Skoda made changes -
            Description it is not enough to enable E_STRICT in config.php only

            areas to be fixed:
            * all filters
            * backup/restore (Eloy)
            * all themes
            it is not enough to enable E_STRICT in config.php only

            areas to be fixed:
            * all filters
            * backup/restore (Eloy)
            * all themes
            * repositories
            Petr Skoda made changes -
            Status Open [ 1 ] Development in progress [ 3 ]
            Petr Skoda made changes -
            Pull Master Diff URL https://github.com/skodak/moodle/compare/master...w13_MDL-32095_m23_strict2
            Pull Master Branch w13_MDL-32095_m23_strict2
            Pull from Repository git://github.com/skodak/moodle.git
            Petr Skoda made changes -
            Status Development in progress [ 3 ] Waiting for integration review [ 10010 ]
            Fix Version/s 2.3 [ 10657 ]
            Fix Version/s DEV backlog [ 10464 ]
            Petr Skoda made changes -
            Link This issue blocks MDL-12730 [ MDL-12730 ]
            Eloy Lafuente (stronk7) made changes -
            Status Waiting for integration review [ 10010 ] Integration review in progress [ 10004 ]
            Integrator stronk7
            Eloy Lafuente (stronk7) made changes -
            Status Integration review in progress [ 10004 ] Waiting for integration review [ 10010 ]
            Eloy Lafuente (stronk7) made changes -
            Status Waiting for integration review [ 10010 ] Integration review in progress [ 10004 ]
            Currently in integration Yes [ 10041 ]
            Eloy Lafuente (stronk7) made changes -
            Status Integration review in progress [ 10004 ] Waiting for testing [ 10005 ]
            Eloy Lafuente (stronk7) made changes -
            Status Waiting for testing [ 10005 ] Testing in progress [ 10011 ]
            Tester stronk7
            Eloy Lafuente (stronk7) made changes -
            Status Testing in progress [ 10011 ] Tested [ 10006 ]
            Eloy Lafuente (stronk7) made changes -
            Status Tested [ 10006 ] Closed [ 6 ]
            Resolution Fixed [ 1 ]
            Currently in integration Yes [ 10041 ]
            Integration date 29/Mar/12
            Michael de Raadt made changes -
            Link This issue caused a regression MDL-32269 [ MDL-32269 ]

              People

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

                Dates

                • Created:
                  Updated:
                  Resolved: