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

CANVAS theme uses wrong layout for default in $THEME->layouts = array(...) in config.php

XMLWordPrintable

    • Icon: Bug Bug
    • Resolution: Deferred
    • Icon: Minor Minor
    • None
    • 2.0.8, 2.1, 2.2, 2.3
    • Themes
    • MOODLE_20_STABLE, MOODLE_21_STABLE, MOODLE_22_STABLE, MOODLE_23_STABLE

      In CANVAS theme, and all other themes using Canvas as a PARENT, the layout for 'default' is wrong.

      If you look at BASE theme it gives the 'default' layout as:

      $THEME->layouts = array(
          // Most backwards compatible layout without the blocks - 
             this is the layout used by default
          'base' => array(
              'file' => 'default.php',
              'regions' => array(),
          ),

      However, if you compare this with CANVAS theme you can see the difference:

      $THEME->layouts = array(
          'base' => array(
              'file' => 'general.php',
              'regions' => array('side-pre', 'side-post'),
              'defaultregion' => 'side-post',
          ),

      In BASE there are NO regions or default regions declared, whereas in CANVAS there are.

      It is hard to understand why this has come about, as the default layout in BASE states clearly:

      Most backwards compatible layout without the blocks -
      this is the layout used by default

      So it looks very much like this has been an oversight from the very beginning of Moodle 2.0 themes!

      I am just wondering if it is this change of layout in Canvas that has been bugging Moodle all this time without detection, causing problems where problems should not arise? Making pages works differently for no apparent reason? One problem comes to mind when all blocks are docked, so that by default in BASE it would be no blocks, yet in Canvas, Moodle would still be looking for block regions, and may even be giving out wrong variables and confusing Moodle? Scary!

            lazydaisy Mary Evans
            lazydaisy Mary Evans
            Votes:
            1 Vote for this issue
            Watchers:
            5 Start watching this issue

              Created:
              Updated:
              Resolved:

                Error rendering 'clockify-timesheets-time-tracking-reports:timer-sidebar'. Please contact your Jira administrators.