Moodle
  1. Moodle
  2. MDL-31167 PHP strict META
  3. MDL-32717

While uninstallling a message processor I got a strict standards warning

    Details

    • Type: Sub-task Sub-task
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 2.3
    • Fix Version/s: 2.3
    • Component/s: Messages
    • Labels:
    • Testing Instructions:
      Hide

      Go to Site Administration > Plugins > Plugins Overview and click Uninstall next to the Jabber messaging processor.

      Go through the uninstallation process. Check that no strict standards warning are displayed.

      Go to Site Administration > Notifications to reinstall the Jabber processor.

      Show
      Go to Site Administration > Plugins > Plugins Overview and click Uninstall next to the Jabber messaging processor. Go through the uninstallation process. Check that no strict standards warning are displayed. Go to Site Administration > Notifications to reinstall the Jabber processor.
    • Affected Branches:
      MOODLE_23_STABLE
    • Fixed Branches:
      MOODLE_23_STABLE
    • Pull from Repository:
    • Pull Master Branch:
      MDL-32009-master-3
    • Rank:
      39671

      Description

      "Strict Standards: Creating default object from empty value in /home/andrew/Desktop/code/moodle/int/master/admin/message.php on line 73 "

      This appeared during the un-installation of a message processor. Site Administration > Plugins > Plugins Overview), scroll down to Messaging processors. Click uninstall. I uninstalled the Jabber processor. Not sure if that is significant.

        Issue Links

          Activity

          Hide
          Ruslan Kabalin added a comment -

          Well spotted Andrew!

          Show
          Ruslan Kabalin added a comment - Well spotted Andrew!
          Hide
          Ruslan Kabalin added a comment -

          Added commit to fix it. Since MDL-32009 is not on public master yet, I add fix to this branch. It can be cleanly cherry-picked to master once the MDL-32009 is released.

          Show
          Ruslan Kabalin added a comment - Added commit to fix it. Since MDL-32009 is not on public master yet, I add fix to this branch. It can be cleanly cherry-picked to master once the MDL-32009 is released.
          Hide
          Andrew Nicols added a comment -

          Looks good to me

          Show
          Andrew Nicols added a comment - Looks good to me
          Hide
          Dan Poltawski added a comment -

          Thanks, i've integrated this now

          Show
          Dan Poltawski added a comment - Thanks, i've integrated this now
          Hide
          Rossiani Wijaya added a comment -

          This is working fine.

          Test passed.

          Show
          Rossiani Wijaya added a comment - This is working fine. Test passed.
          Hide
          Eloy Lafuente (stronk7) added a comment -

          This is now part of Moodle and a few millions people around the globe will be using it soon. Isn't that awesome?

          Many, many thanks and don't forget http://youtu.be/4N7dPaP5Z8U

          Closing, ciao

          Show
          Eloy Lafuente (stronk7) added a comment - This is now part of Moodle and a few millions people around the globe will be using it soon. Isn't that awesome? Many, many thanks and don't forget http://youtu.be/4N7dPaP5Z8U Closing, ciao

            People

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

              Dates

              • Created:
                Updated:
                Resolved: