Moodle
  1. Moodle
  2. MDL-33118

Change MAIN_CONTENT_TOKEN to new format in theme/mymobile/layout/embedded.php

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 2.2
    • Fix Version/s: 2.2.4
    • Component/s: Themes
    • Labels:
    • Testing Instructions:
      Hide

      With 'MyMobile' selected as the prefered theme for "Mobile" device, TEST on an appropriate mobile phone (iPhone, Android etc) to access any embedded file such as a PDF.

      Show
      With 'MyMobile' selected as the prefered theme for "Mobile" device, TEST on an appropriate mobile phone (iPhone, Android etc) to access any embedded file such as a PDF.
    • Affected Branches:
      MOODLE_22_STABLE
    • Fixed Branches:
      MOODLE_22_STABLE
    • Pull Master Branch:
    • Rank:
      40381

      Description

      Discovered while testing, it appears that theme/mymobile/layout/embedded.php contains

       <?php echo core_renderer::MAIN_CONTENT_TOKEN ?>
      

      instead of

       <?php echo $OUTPUT->main_content() ?>
      

      as can be seen here...
      https://github.com/lazydaisy/moodle/blob/master/theme/mymobile/layout/embedded.php

        Issue Links

          Activity

          Hide
          Sam Hemelryk added a comment -

          Good spotting and thanks for fixing Mary, this has been integrated now.

          Show
          Sam Hemelryk added a comment - Good spotting and thanks for fixing Mary, this has been integrated now.
          Hide
          Sam Hemelryk added a comment -

          Tested during integration review

          Show
          Sam Hemelryk added a comment - Tested during integration review
          Hide
          Eloy Lafuente (stronk7) added a comment -

          This issue has been integrated upstream and is now available both via git and cvs (and in some hours, via mirrors and downloads).

          Thanks!

          Show
          Eloy Lafuente (stronk7) added a comment - This issue has been integrated upstream and is now available both via git and cvs (and in some hours, via mirrors and downloads). Thanks!

            People

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

              Dates

              • Created:
                Updated:
                Resolved: