Uploaded image for project: 'Moodle'
  1. Moodle
  2. MDL-33055 Formal White theme's standard theme layout is wrong in config.php
  3. MDL-33191

Magazine theme's base and standard $THEME->layouts are wrong in config.php

          Details

          • Type: Sub-task
          • Status: Closed
          • Priority: Minor
          • Resolution: Fixed
          • Affects Version/s: 2.1, 2.2, 2.3
          • Fix Version/s: 2.1.7, 2.2.4
          • Component/s: Themes
          • Labels:

            Description

            Like Formal White theme MDL-33055, under $THEME->LAYOUTS in config.php, the Standard theme layout is wrongly labeled 'general' whereas it should read 'standard'.
            Also discovered that 'base' layout was wrong too.

              Gliffy Diagrams

                Attachments

                  Issue Links

                  Testing discovered

                  Task - A task that needs to be completed. Tasks usually refer to work that must be done outside the product. MDL-33055 Formal White theme's standard theme layout is wrong in config.php

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

                    Activity

                    Ascending order - Click to sort in descending order
                    Hide
                    Permalink
                    lazydaisy Mary Evans added a comment - - edited

                    I have since found Magazine theme has the same problem as was discovered in Formal White

                    Show
                    lazydaisy Mary Evans added a comment - - edited I have since found Magazine theme has the same problem as was discovered in Formal White
                    Hide
                    Permalink
                    lazydaisy Mary Evans added a comment -

                    Fixing now...

                    Show
                    lazydaisy Mary Evans added a comment - Fixing now...
                    Hide
                    Permalink
                    samhemelryk Sam Hemelryk added a comment -

                    Thanks Mary, this has been integrated now

                    Show
                    samhemelryk Sam Hemelryk added a comment - Thanks Mary, this has been integrated now
                    Hide
                    Permalink
                    samhemelryk Sam Hemelryk added a comment -

                    Tested the magazine theme in all branches to ensure it still worked

                    Show
                    samhemelryk Sam Hemelryk added a comment - Tested the magazine theme in all branches to ensure it still worked
                    Hide
                    Permalink
                    poltawski Dan Poltawski added a comment -

                    Congratulations!

                    Your work has made into the latest Moodle release!

                    You are only authorised to celebrate after testing 15 Moodle 2.3 QA tests, thanks!

                    Show
                    poltawski Dan Poltawski added a comment - Congratulations! Your work has made into the latest Moodle release! You are only authorised to celebrate after testing 15 Moodle 2.3 QA tests, thanks!

                      People

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

                        Dates

                        • Created:
                          Updated:
                          Resolved:
                          Fix Release Date:
                          9/Jul/12