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

Formal White report.php uses old style MAIN_CONTENT_TOKEN

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 2.3, 2.4
    • Fix Version/s: 2.3.2
    • Component/s: Themes
    • Labels:

      Description

      In checking Formal White theme in order to answer a query today, concerning the report layout, I discovered that theme/formal_white/layout/report.php still uses the old MAIN_CONTENT_TOKEN. This is a regression, as the previous version (Moodle 2.2.4) although that uses general.php for the report layout, nevertheless is uses the correct form of $OUTPUT->main_content().

        Gliffy Diagrams

          Activity

          Hide
          lazydaisy Mary Evans added a comment -

          Can you deal with this Danielle of do you want me to fix it?

          Show
          lazydaisy Mary Evans added a comment - Can you deal with this Danielle of do you want me to fix it?
          Hide
          daniss Daniele Cordella added a comment -

          Ciao Mary
          and thanks for this report.
          I made a correction following your suggestion. Can you check it please.
          It should really be safe.
          Let me know, please.
          Ciao.
          Happy Olympics games.

          Show
          daniss Daniele Cordella added a comment - Ciao Mary and thanks for this report. I made a correction following your suggestion. Can you check it please. It should really be safe. Let me know, please. Ciao. Happy Olympics games.
          Hide
          lazydaisy Mary Evans added a comment -

          Starting Peer Review

          Show
          lazydaisy Mary Evans added a comment - Starting Peer Review
          Hide
          lazydaisy Mary Evans added a comment -

          Looks good to me.
          I'll set it to start Integration Review.

          Show
          lazydaisy Mary Evans added a comment - Looks good to me. I'll set it to start Integration Review.
          Hide
          lazydaisy Mary Evans added a comment -

          Hi Daniele,

          I have just submitted this for Integration Review.

          But could you also fix Moodle 2.3 branch too? Otherwise the fix will not be in the next release.

          Show
          lazydaisy Mary Evans added a comment - Hi Daniele, I have just submitted this for Integration Review. But could you also fix Moodle 2.3 branch too? Otherwise the fix will not be in the next release.
          Hide
          daniss Daniele Cordella added a comment -

          Mary, I added the fix for m23 too.
          Ciao.

          Show
          daniss Daniele Cordella added a comment - Mary, I added the fix for m23 too. Ciao.
          Hide
          samhemelryk Sam Hemelryk added a comment -

          Thanks Daniele and Mary, this has been integrated now

          Show
          samhemelryk Sam Hemelryk added a comment - Thanks Daniele and Mary, this has been integrated now
          Hide
          andyjdavis Andrew Davis added a comment -

          This seems to be missing testing instructions. I can click around formal white but I'm not quite sure what I should be doing.

          Show
          andyjdavis Andrew Davis added a comment - This seems to be missing testing instructions. I can click around formal white but I'm not quite sure what I should be doing.
          Hide
          samhemelryk Sam Hemelryk added a comment -

          Aha safe to pass then thanks Andrew.
          Sorry I should have marked this as tested and passed when I integrated it.

          Accurate testing instructions would be.

          1. Change to the formal white theme
          2. Check any page and make sure the content area contains its normal stuff.
          Show
          samhemelryk Sam Hemelryk added a comment - Aha safe to pass then thanks Andrew. Sorry I should have marked this as tested and passed when I integrated it. Accurate testing instructions would be. Change to the formal white theme Check any page and make sure the content area contains its normal stuff.
          Hide
          andyjdavis Andrew Davis added a comment -

          Had a decent click around. All looks good. Passing.

          Show
          andyjdavis Andrew Davis added a comment - Had a decent click around. All looks good. Passing.
          Hide
          poltawski Dan Poltawski added a comment -

          asko, Дзякуй, ধন্যবাদ, Благодаря, Gràcies, 感谢, 謝謝, Hvala, Díky, Tak, Bedankt, Tänan, متشکریم, Salamat, Kiitokset, Merci, Grazas, Danke, Ευχαριστώ, આભાર, תודה, धन्यवाद, Köszönjük, Takk fyrir, Terima Kasih, Grazie, ありがとうございます, Рахмет, សូមអរគុណ, 감사합니다, gratiās, Pateicamies, Ačiū, Благодарам, Tēnā koa, Kia Ora Rawa Atu, आभारी आहोत, Талархал, Takk, Dziękuję, Obrigado, Mulţumesc, Engraziel, Спасибо, Fa'afetai, Хвала, Hvala, ස්තූතියි, Vďaka, Hvala, Mahadsanid, Thanks, Gracias, Tack, Salamat, நன்றி, నెనరులు, ขอบคุณค่ะ!

          Your work has made it into this weeks Moodle release! There are no gold medals available this week - but millions around the world will benefit. Thank you!

          Show
          poltawski Dan Poltawski added a comment - asko, Дзякуй, ধন্যবাদ, Благодаря, Gràcies, 感谢, 謝謝, Hvala, Díky, Tak, Bedankt, Tänan, متشکریم, Salamat, Kiitokset, Merci, Grazas, Danke, Ευχαριστώ, આભાર, תודה, धन्यवाद, Köszönjük, Takk fyrir, Terima Kasih, Grazie, ありがとうございます, Рахмет, សូមអរគុណ, 감사합니다, gratiās, Pateicamies, Ačiū, Благодарам, Tēnā koa, Kia Ora Rawa Atu, आभारी आहोत, Талархал, Takk, Dziękuję, Obrigado, Mulţumesc, Engraziel, Спасибо, Fa'afetai, Хвала, Hvala, ස්තූතියි, Vďaka, Hvala, Mahadsanid, Thanks, Gracias, Tack, Salamat, நன்றி, నెనరులు, ขอบคุณค่ะ! Your work has made it into this weeks Moodle release! There are no gold medals available this week - but millions around the world will benefit. Thank you!

            People

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

              Dates

              • Created:
                Updated:
                Resolved:
                Fix Release Date:
                10/Sep/12