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

Add index conversationid-userid to message_conversation_members table

    XMLWordPrintable

    Details

    • Type: Improvement
    • Status: Open
    • Priority: Minor
    • Resolution: Unresolved
    • Affects Version/s: 3.6
    • Fix Version/s: None
    • Component/s: Messages
    • Labels:
    • Testing Instructions:
      Hide
      Test 1
      1. Do a fresh install.
      2. View the database table 'message_conversation_members'.
      3. Confirm there is a unique index for conversationid-userid.
      Test 2
      1. Do git checkout 208950cff04f87e16eb008cd395c2222ec6934a7
      2. Install your site.
      3. Do git checkout master
      4. Perform the upgrade.
      5. Confirm there is a unique index for conversationid-userid.
      Show
      Test 1 Do a fresh install. View the database table 'message_conversation_members'. Confirm there is a unique index for conversationid-userid . Test 2 Do git checkout 208950cff04f87e16eb008cd395c2222ec6934a7 Install your site. Do git checkout master Perform the upgrade. Confirm there is a unique index for conversationid-userid .
    • Affected Branches:
      MOODLE_36_STABLE
    • Pull from Repository:
    • Pull Master Branch:
      MDL-63835_master

      Description

      At this moment in time a user can not be in the same conversation twice - if this does happen then it is a bug. Adding an index would prevent duplicate records and alert us to errors quicker.

        Attachments

          Issue Links

            Activity

              People

              Assignee:
              Unassigned Unassigned
              Reporter:
              markn Mark Nelson
              Participants:
              Component watchers:
              Amaia Anabitarte, Carlos Escobedo, Ferran Recio, Ilya Tregubov, Sara Arjona (@sarjona)
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

                Dates

                Created:
                Updated: