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
    • Rank:
      38799

      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

        Issue Links

          Activity

          Petr Škoda created issue -
          Petr Škoda made changes -
          Field Original Value New Value
          Link This issue has a clone MDL-32094 [ MDL-32094 ]
          Petr Škoda 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 Škoda 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 Škoda 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 Škoda made changes -
          Status Open [ 1 ] Development in progress [ 3 ]
          Petr Škoda 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 Škoda 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 Škoda 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: