Moodle
  1. Moodle
  2. MDL-25636

PHP Notice: Undefined property: stdClass::$format in moodle/blocks/html/block_html.php on line 61

        Details

        • Type: Bug Bug
        • Status: Closed
        • Priority: Minor Minor
        • Resolution: Fixed
        • Affects Version/s: 2.0.4, 2.1.1, 2.2
        • Fix Version/s: 2.0.5, 2.1.2
        • Component/s: Blocks
        • Labels:
          None
        • Testing Instructions:
          Hide
          1. Backup world of water from Tomaz's school demo site
          2. Restore it on your local site
          3. Make sure you don't get any notices about undefined $format when you view the restored course.

          ps: recommended to test 2.1.x and 2.0.x in addition to master

          Show
          Backup world of water from Tomaz's school demo site Restore it on your local site Make sure you don't get any notices about undefined $format when you view the restored course. ps: recommended to test 2.1.x and 2.0.x in addition to master
        • Affected Branches:
          MOODLE_20_STABLE, MOODLE_21_STABLE, MOODLE_22_STABLE
        • Fixed Branches:
          MOODLE_20_STABLE, MOODLE_21_STABLE
        • Pull from Repository:
        • Pull Master Branch:
          wip-MDL-25636-master
        • Pull Master Diff URL:
        • Rank:
          15107

          Description

          Got the following error message while using a Moodle course from one of Tomaz's sites (I had restored a backup).

          PHP Notice: Undefined property: stdClass::$format in /var/git/moodle/blocks/html/block_html.php on line 61, referer: http://sam.moodle.local/git-moodle/backup/restore.php

          I've got to remember to check that the block settings were included in the backup and that it's not just a simple case of setting a default.

          Cheers
          Sam

            Activity

            Ascending order - Click to sort in descending order
            Hide
            Permalink
            Sam Hemelryk added a comment -

            Peer-reviewer if you are happy with this please put it up for integration.

            Integrator feel free to cherry-pick, squash, whatever you want.

            Cheers fella's

            Show
            Sam Hemelryk added a comment - Peer-reviewer if you are happy with this please put it up for integration. Integrator feel free to cherry-pick, squash, whatever you want. Cheers fella's
            Hide
            Permalink
            Andrew Davis added a comment -

            looks good to me.

            Show
            Andrew Davis added a comment - looks good to me.
            Hide
            Permalink
            Aparup Banerjee added a comment -

            Thanks Sam, this has been rather confidently integrated (and cherry-picked for 2.0.x and 2.1.x so testers please these branches too).

            Show
            Aparup Banerjee added a comment - Thanks Sam, this has been rather confidently integrated (and cherry-picked for 2.0.x and 2.1.x so testers please these branches too).
            Hide
            Permalink
            Rajesh Taneja added a comment -

            No notices with this patch.
            Thanks for fixing this Sam

            Show
            Rajesh Taneja added a comment - No notices with this patch. Thanks for fixing this Sam
            Hide
            Permalink
            Eloy Lafuente (stronk7) added a comment -

            Many thanks for the hard work, this has been sent upstream and is available in all the git and cvs repositories.

            Show
            Eloy Lafuente (stronk7) added a comment - Many thanks for the hard work, this has been sent upstream and is available in all the git and cvs repositories.

              People

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

                Dates

                • Created:
                  Updated:
                  Resolved: