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

Notices from restore when there are no files

    Details

    • Testing Instructions:
      Hide

      1. Create a new Moodle installation.
      2. Go to Site administration -> Front page -> Front page restores.

      Show
      1. Create a new Moodle installation. 2. Go to Site administration -> Front page -> Front page restores.
    • Affected Branches:
      MOODLE_21_STABLE, MOODLE_23_STABLE
    • Fixed Branches:
      MOODLE_21_STABLE, MOODLE_22_STABLE, MOODLE_23_STABLE
    • Pull from Repository:

      Description

      Steps to reproduce:

      1. Make a new install of Moodle 2.1 dev (this may also affect 2.0) (May not need to be a completely fresh install, just one which has not had many files uploaded.)

      2. Go to Site administration -> Front page -> Front page restore

      Observe:

      Notice: Trying to get property of non-object in /fs1/www_root/tjh238/moodle_qe2/files/index.php on line 64

      Notice: Trying to get property of non-object in /fs1/www_root/tjh238/moodle_qe2/files/index.php on line 65

        Gliffy Diagrams

          Attachments

            Issue Links

              Activity

              Hide
              timhunt Tim Hunt added a comment -

              Actually, in this situation, I cannot see any way to get rid of the message "No files available". Why is there no filepicker?

              Show
              timhunt Tim Hunt added a comment - Actually, in this situation, I cannot see any way to get rid of the message "No files available". Why is there no filepicker?
              Hide
              tsala Helen Foster added a comment -

              Just noting that this issue also affects Moodle 2.2 - I ran into it on http://qa.moodle.net/. In addition to the notices, the navigation was messed up ('My home > Files' in the navigation bar) and the site header was missing.

              Show
              tsala Helen Foster added a comment - Just noting that this issue also affects Moodle 2.2 - I ran into it on http://qa.moodle.net/ . In addition to the notices, the navigation was messed up ('My home > Files' in the navigation bar) and the site header was missing.
              Hide
              cfulton Charles Fulton added a comment -

              We're seeing the same behavior at Lafayette under the 2.3 release. I think this is just a matter of pointing the site administration link to the same place as the Front page settings navigation block. This does nothing to resolve MDL-31500 but at least the behavior is consistent.

              Show
              cfulton Charles Fulton added a comment - We're seeing the same behavior at Lafayette under the 2.3 release. I think this is just a matter of pointing the site administration link to the same place as the Front page settings navigation block. This does nothing to resolve MDL-31500 but at least the behavior is consistent.
              Hide
              fred Frédéric Massart added a comment -

              Thanks for your patch Charles, this looks good to me! Feel free to push it for integration whenever your are ready.

              Show
              fred Frédéric Massart added a comment - Thanks for your patch Charles, this looks good to me! Feel free to push it for integration whenever your are ready.
              Hide
              fred Frédéric Massart added a comment -

              Submitting for integration as Charles does not have permissions to do so. This patch should be easily cherry-picked in 2.2 and 2.3

              Show
              fred Frédéric Massart added a comment - Submitting for integration as Charles does not have permissions to do so. This patch should be easily cherry-picked in 2.2 and 2.3
              Hide
              samhemelryk Sam Hemelryk added a comment -

              Thanks guys, this has been integrate now and cherry-picked to stable branches.

              Show
              samhemelryk Sam Hemelryk added a comment - Thanks guys, this has been integrate now and cherry-picked to stable branches.
              Hide
              phalacee Jason Fowler added a comment -

              No details about an expected outcome, but everything seems to be working

              Show
              phalacee Jason Fowler added a comment - No details about an expected outcome, but everything seems to be working
              Hide
              poltawski Dan Poltawski added a comment -

              Congratulations!

              You've made it into the weekly release!

              Thanks for your contribution - here are some random drummers to keep you inspired for the next week!
              http://www.youtube.com/watch?v=_QhpHUmVCmY

              Show
              poltawski Dan Poltawski added a comment - Congratulations! You've made it into the weekly release! Thanks for your contribution - here are some random drummers to keep you inspired for the next week! http://www.youtube.com/watch?v=_QhpHUmVCmY

                People

                • Votes:
                  8 Vote for this issue
                  Watchers:
                  4 Start watching this issue

                  Dates

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