Details

    • Type: Bug
    • Status: Closed
    • Priority: Blocker
    • Resolution: Fixed
    • Affects Version/s: 2.0.2
    • Fix Version/s: 2.0.3
    • Component/s: Blog
    • Labels:
    • Affected Branches:
      MOODLE_20_STABLE
    • Fixed Branches:
      MOODLE_20_STABLE

      Description

      As reported by Jarrod Holder in http://moodle.org/mod/forum/discuss.php?d=169588

      Updated a play site from 1.9.10 to 2.0.2. Trying to run the Blog visibility update that's recommended and I get the following error:

      Notice: Undefined property: stdClass::$message in /mysite/mod/forum/lib.php on line 4113

      Followed by "Error writing to database"

        Gliffy Diagrams

          Attachments

            Issue Links

              Activity

              Hide
              andymccall Andy McCall added a comment -

              I am also having the same problem, I have recently upgraded from 1.9.11+ to 2.0.

              The upgrade was successful but when logging in to the admin page I had the following warning:

              "Your site is using an old blog visibility setting, the blog visibility upgrade is recommended."

              Running the upgrade produces the screenshot attached.

              Show
              andymccall Andy McCall added a comment - I am also having the same problem, I have recently upgraded from 1.9.11+ to 2.0. The upgrade was successful but when logging in to the admin page I had the following warning: "Your site is using an old blog visibility setting, the blog visibility upgrade is recommended." Running the upgrade produces the screenshot attached.
              Show
              nebgor Aparup Banerjee added a comment - a patch is up at https://github.com/nebgor/moodle/compare/mistress...MDL-26643
              Hide
              nebgor Aparup Banerjee added a comment -

              thanks for review rosie, PULL-422 created

              Show
              nebgor Aparup Banerjee added a comment - thanks for review rosie, PULL-422 created
              Hide
              jarrodholder Jarrod Holder added a comment -

              Updated bloglevelupgrade.php as listed and worked great. Thanks! One step closer to moving our site to 2.0

              Show
              jarrodholder Jarrod Holder added a comment - Updated bloglevelupgrade.php as listed and worked great. Thanks! One step closer to moving our site to 2.0
              Hide
              stronk7 Eloy Lafuente (stronk7) added a comment -

              This has been integrated for testing this week, thanks.

              Note: Just saw that, immediately below the changed line, there is one:

              $discussion->messageformat = $forum->introformat;

              Just guessing if that is really the best format to be used in the conversion, specially because forum_get_course_forum() doesn't define it, hence = 0 (FORMAT_MOODLE).

              Ciao

              Show
              stronk7 Eloy Lafuente (stronk7) added a comment - This has been integrated for testing this week, thanks. Note: Just saw that, immediately below the changed line, there is one: $discussion->messageformat = $forum->introformat; Just guessing if that is really the best format to be used in the conversion, specially because forum_get_course_forum() doesn't define it, hence = 0 (FORMAT_MOODLE). Ciao
              Hide
              dongsheng Dongsheng Cai added a comment -

              Closing Thanks

              Show
              dongsheng Dongsheng Cai added a comment - Closing Thanks
              Hide
              tsala Helen Foster added a comment -

              Reopening so that a 2.0.3 fix version can be set. Also, let's wait until the weekly package is available before closing. Apologies for the extra notification emails.

              Show
              tsala Helen Foster added a comment - Reopening so that a 2.0.3 fix version can be set. Also, let's wait until the weekly package is available before closing. Apologies for the extra notification emails.
              Hide
              tsala Helen Foster added a comment -

              Aparup, thanks for fixing.

              Show
              tsala Helen Foster added a comment - Aparup, thanks for fixing.
              Hide
              skodak Petr Skoda added a comment -

              the discussion format is not correct, I am going to file new pull in MDL-26876

              Show
              skodak Petr Skoda added a comment - the discussion format is not correct, I am going to file new pull in MDL-26876

                People

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

                  Dates

                  • Created:
                    Updated:
                    Resolved:
                    Fix Release Date:
                    5/May/11