Uploaded image for project: 'Moodle app'
  1. Moodle app
  2. MOBILE-1983

Calls to core_message_mark_message_read are not backwards compatible

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 3.2
    • Fix Version/s: 3.2.1
    • Component/s: Messages add-on
    • Labels:
    • Testing Instructions:
      Hide
      1. In a Moodle 2.9, login in web as a user and send a message to another user.
      2. In another browser, login as the user who received the message, go to Messages and check that you have an unread message (don't read the message).
      3. In the app, login as the user who received the message. Please notice you won't see a badge in the side menu with the unread messages because Moodle 2.9 doesn't allow it yet.
      4. Still in the app, go to Messages and open the conversation with the unread message so it's marked as read.
      5. In Moodle web, refresh the list of unread conversations and check that the conversation isn't marked as read now.
      Show
      In a Moodle 2.9, login in web as a user and send a message to another user. In another browser, login as the user who received the message, go to Messages and check that you have an unread message (don't read the message). In the app, login as the user who received the message. Please notice you won't see a badge in the side menu with the unread messages because Moodle 2.9 doesn't allow it yet. Still in the app, go to Messages and open the conversation with the unread message so it's marked as read. In Moodle web, refresh the list of unread conversations and check that the conversation isn't marked as read now.
    • Affected Branches:
      MOODLE_32_STABLE
    • Fixed Branches:
      MOODLE_32_STABLE
    • Sprint:
      Moodle Mobile 3.2.1

      Description

      We must include the timeread value always because in older Moodle versions was mandatory, we can include it with a default to 0 or maybe is better to set it to the current time

        Attachments

          Activity

            People

            Assignee:
            dpalou Dani Palou
            Reporter:
            jleyva Juan Leyva
            Peer reviewer:
            Pau Ferrer
            Integrator:
            Juan Leyva
            Tester:
            Pau Ferrer
            Participants:
            Component watchers:
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved:
              Fix Release Date:
              28/Feb/17