Details

    • Type: Sub-task
    • Status: Closed
    • Priority: 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

          Attachments

            Issue Links

              Activity

              skodak Petr Skoda created issue -
              skodak Petr Skoda made changes -
              Field Original Value New Value
              Link This issue has a clone MDL-32094 [ MDL-32094 ]
              skodak 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
              salvetore Michael de Raadt made changes -
              Labels triaged
              rajeshtaneja Rajesh Taneja made changes -
              skodak 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
              skodak 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
              skodak Petr Skoda made changes -
              Status Open [ 1 ] Development in progress [ 3 ]
              skodak 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
              skodak 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 ]
              skodak Petr Skoda made changes -
              Link This issue blocks MDL-12730 [ MDL-12730 ]
              stronk7 Eloy Lafuente (stronk7) made changes -
              Status Waiting for integration review [ 10010 ] Integration review in progress [ 10004 ]
              Integrator stronk7
              stronk7 Eloy Lafuente (stronk7) made changes -
              Status Integration review in progress [ 10004 ] Waiting for integration review [ 10010 ]
              stronk7 Eloy Lafuente (stronk7) made changes -
              Status Waiting for integration review [ 10010 ] Integration review in progress [ 10004 ]
              Currently in integration Yes [ 10041 ]
              stronk7 Eloy Lafuente (stronk7) made changes -
              Status Integration review in progress [ 10004 ] Waiting for testing [ 10005 ]
              stronk7 Eloy Lafuente (stronk7) made changes -
              Status Waiting for testing [ 10005 ] Testing in progress [ 10011 ]
              Tester stronk7
              stronk7 Eloy Lafuente (stronk7) made changes -
              Status Testing in progress [ 10011 ] Tested [ 10006 ]
              stronk7 Eloy Lafuente (stronk7) made changes -
              Status Tested [ 10006 ] Closed [ 6 ]
              Resolution Fixed [ 1 ]
              Currently in integration Yes [ 10041 ]
              Integration date 29/Mar/12
              salvetore 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:
                    Fix Release Date:
                    25/Jun/12