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

mod/assign/index.php page displays notice

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 2.5
    • Fix Version/s: 2.5
    • Component/s: Assignment
    • Labels:

      Description

      1. As an administrator create an assignment in a course.
      2. Visit <yoururl>/mod/assign/index.php?id=<courseid>
      3. You will get the notice:

      Notice: Trying to get property of non-object in /var/www/mstorage/im/moodle/mod/assign/locallib.php on line 358

      This is only an issue in master.

        Gliffy Diagrams

          Attachments

            Issue Links

              Activity

              Hide
              markn Mark Nelson added a comment -

              Hi Damyon, I added you as peer reviewer as you have the most knowledge regarding this module. I created a fix that works but am not sure if it is applicable to all scenarios. If you can verify it makes sense that would be great, cheers.

              Show
              markn Mark Nelson added a comment - Hi Damyon, I added you as peer reviewer as you have the most knowledge regarding this module. I created a fix that works but am not sure if it is applicable to all scenarios. If you can verify it makes sense that would be great, cheers.
              Hide
              damyon Damyon Wiese added a comment -

              Looks good to me - thanks Mark!

              Show
              damyon Damyon Wiese added a comment - Looks good to me - thanks Mark!
              Hide
              poltawski Dan Poltawski added a comment -

              Hehe, these index pages are the least visited pages in Moodle and often wrong. We should write some behat tests for them!

              Show
              poltawski Dan Poltawski added a comment - Hehe, these index pages are the least visited pages in Moodle and often wrong. We should write some behat tests for them!
              Hide
              poltawski Dan Poltawski added a comment -

              Integrated and tested during integration. Thanks Mark

              Show
              poltawski Dan Poltawski added a comment - Integrated and tested during integration. Thanks Mark
              Hide
              stronk7 Eloy Lafuente (stronk7) added a comment -

              Did you think this day was not going to arrive ever?

              Your patience has been rewarded, yay, sent upstream, thanks!

              Closing...ciao

              Show
              stronk7 Eloy Lafuente (stronk7) added a comment - Did you think this day was not going to arrive ever? Your patience has been rewarded, yay, sent upstream, thanks! Closing...ciao

                People

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

                  Dates

                  • Created:
                    Updated:
                    Resolved:
                    Fix Release Date:
                    14/May/13