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

CACHE_DISABLE_ALL should not disable cache_data_sources

    XMLWordPrintable

    Details

    • Testing Instructions:
      Hide

      Test 1

      1. Open your config.php in your preferred editor
      2. At the top of the file, just beneath the PHP open tag (<?php) add: 

        define('CACHE_DISABLE_ALL', true);

      3. Log in as editing teacher or above
      4. Navigate to a course
      5. From the cog, choose More
      6. Navigate to Question bank -> Questions
      7. Create a new question (any question will do)
      8. Locate the newly created question in the question bank
      9. From the Edit menu, choose "Preview"
        1. Confirm that a new window opened
        2. Confirm that no errors were shown on the page

      Test 2

      1. Download the attached testcache.php and save it in the Root of you Moodle install where you run PHPunit.
      2. Run it in the CLI: php testcache.php nocache.
        1. Confirm that there were no assertion failures
      Show
      Test 1 Open your config.php in your preferred editor At the top of the file, just beneath the PHP open tag ( <?php ) add: define('CACHE_DISABLE_ALL', true); Log in as editing teacher or above Navigate to a course From the cog, choose More Navigate to Question bank -> Questions Create a new question (any question will do) Locate the newly created question in the question bank From the Edit menu, choose "Preview" Confirm that a new window opened Confirm that no errors were shown on the page Test 2 Download the attached testcache.php and save it in the Root of you Moodle install where you run PHPunit. Run it in the CLI: php testcache.php nocache . Confirm that there were no assertion failures
    • Affected Branches:
      MOODLE_26_STABLE, MOODLE_310_STABLE, MOODLE_35_STABLE, MOODLE_37_STABLE, MOODLE_38_STABLE, MOODLE_39_STABLE, MOODLE_400_STABLE
    • Fixed Branches:
      MOODLE_38_STABLE, MOODLE_39_STABLE
    • Pull from Repository:
    • Pull 3.9 Branch:
      MDL-42012-39-alt
    • Pull 3.9 Diff URL:
    • Pull 3.10 Branch:
      MDL-42012-310-alt
    • Pull 3.10 Diff URL:
    • Pull Master Branch:
      MDL-42012-master-alt
    • Pull Master Diff URL:

      Description

      There were two behat test failures when when I set
      define('CACHE_DISABLE_ALL', true); in config.php, both of them seem to be in the same line.

      line 67 of /question/preview.php: call to question_require_capability_on()

      I attach the full log.

      If you re-run behat tests, please cherry-pick MDL-41942 because without it there will be much more failures

        Attachments

        1. behat_nocaches.log
          6 kB
        2. example_isolated_test.php
          3 kB
        3. MDL-42012-master-alt.mdk.patch
          7 kB
        4. Screenshot_1.jpg
          Screenshot_1.jpg
          31 kB
        5. Screenshot_2.jpg
          Screenshot_2.jpg
          66 kB
        6. testcache.php
          4 kB

          Issue Links

          has been marked as being related by

          Bug - A problem which impairs or prevents Moodle from functioning correctly. MDL-57936 Coding error detected, it must be fixed by a programmer: The data source class does not exist.

          • Major - Major loss of function, incorrect output
          • Open

          Bug - A problem which impairs or prevents Moodle from functioning correctly. MDL-43356 CACHE_DISABLE_ALL is not suitable for upgrade or installation

          • Minor - Minor loss of function where workaround is possible
          • Open

          Bug - A problem which impairs or prevents Moodle from functioning correctly. MDL-49979 $usecache inverted in get_all_versions_hash

          • Trivial - Cosmetic problem like misspelt words or misaligned text
          • Closed
          Testing discovered

          Bug - A problem which impairs or prevents Moodle from functioning correctly. MDL-41942 Courses in categories invisible

          • Blocker - Blocks development and/or testing, prevents Moodle from running (applicable to bugs only)
          • Closed

            Activity

              People

              Assignee:
              dobedobedoh Andrew Lyons
              Reporter:
              marina Marina Glancy
              Peer reviewer:
              Tim Hunt Tim Hunt
              Integrator:
              Eloy Lafuente (stronk7) Eloy Lafuente (stronk7)
              Tester:
              Janelle Barcega Janelle Barcega
              Participants:
              Component watchers:
              Andrew Lyons, Dongsheng Cai, Huong Nguyen, Jun Pataleta, Michael Hawkins, Shamim Rezaie, Simey Lameze, Matteo Scaramuccia, Amaia Anabitarte, Carlos Escobedo, Ferran Recio, Ilya Tregubov, Sara Arjona (@sarjona), Tim Hunt, Andrew Lyons, Dongsheng Cai, Huong Nguyen, Jun Pataleta, Michael Hawkins, Shamim Rezaie, Simey Lameze
              Votes:
              4 Vote for this issue
              Watchers:
              12 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:
                Fix Release Date:
                9/Nov/20

                  Time Tracking

                  Estimated:
                  Original Estimate - Not Specified
                  Not Specified
                  Remaining:
                  Remaining Estimate - 0 minutes
                  0m
                  Logged:
                  Time Spent - 4 hours, 30 minutes
                  4h 30m