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

          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