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

Toggling notifications menu causes an exception in SQL Server

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 3.2.1, 3.2.4
    • Fix Version/s: 3.2.5, 3.3.2
    • Component/s: Messages, Other
    • Labels:
    • Testing Instructions:
      Hide
      1. Create a course
      2. Create two assignments and allow online text submissions for each
      3. Enrol a student
      4. Log in as the student
      5. Go to the student's notification preferences (notification popover in the nav bar > gear icon)
      6. Turn on the "web" notifications for assignments
      7. Go to the first assignment and add a submission
      8. Wait a small amount of time (10 seconds or so)
      9. Go to the second assignment and add a submission
      10. Confirm you receive two notifications in the popover that you've added a submission for each notification
      11. Confirm that the most recent notification (the one for the second assignment) is first in the list of notifications

      There is no need to test this on multiple databases as it is covered by unit tests.

      Also the specific database setup reported by in this issue has been tested by Matt (see in the comments) and confirmed fixed.

      Show
      Create a course Create two assignments and allow online text submissions for each Enrol a student Log in as the student Go to the student's notification preferences (notification popover in the nav bar > gear icon) Turn on the "web" notifications for assignments Go to the first assignment and add a submission Wait a small amount of time (10 seconds or so) Go to the second assignment and add a submission Confirm you receive two notifications in the popover that you've added a submission for each notification Confirm that the most recent notification (the one for the second assignment) is first in the list of notifications There is no need to test this on multiple databases as it is covered by unit tests. Also the specific database setup reported by in this issue has been tested by Matt (see in the comments) and confirmed fixed.
    • Affected Branches:
      MOODLE_32_STABLE
    • Fixed Branches:
      MOODLE_32_STABLE, MOODLE_33_STABLE
    • Pull Master Branch:
      MDL-58332-master

      Description

      using Microsoft SQL 2008r2, IIS 7 on Server 2008r2. Due to minimum requirements, needed to upgrade php 5.4.24 to php 5.6.30.
      upgraded to moodle 3.2.1, now when we click on the new "toggle notifications menu" button at the top of the screen, we get a dmlreadexception. This is for all users. I attempted to change my individual notifications settings, no effect.
      Enabled debug messages, appears to be issue with concat function? using sqlsrv (needed to upgrade version of microsoft sqlsrv php drivers to 3.2 since we upgraded to php 5.6.30)
      looks like Ahmad Z had same issue here: https://moodle.org/mod/forum/discuss.php?d=344050 and hasn't been resolved. We are having the exact same issue as Ahmad.
      Help!

        Attachments

        1. notifications_error.txt
          3 kB
        2. readexception2.txt
          2 kB
        3. readexception2all.txt
          2 kB
        4. screenshot-1.png
          screenshot-1.png
          24 kB

          Activity

            People

            • Votes:
              2 Vote for this issue
              Watchers:
              6 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:
                Fix Release Date:
                11/Sep/17