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

Add course id to message eventdata

    XMLWordPrintable

Details

    Description

      The new events2 api now includes the course id which is awesome. Could it also be added to the message api please? It is a piece of info which could be useful for all sorts of plugins such as CONTRIB-5295

      I have a core code patch here: https://github.com/amandadoughty/moodle-core_culactivity_stream for v2.6.4

      I would be happy to submit this as a proper fix for current branches if the idea is accepted?

      Attachments

        Issue Links

          has a non-specific relationship to

          Improvement - An enhancement to an existing Moodle feature. MDL-26217 Course submission notifications should include a context url

          • Minor - Minor loss of function where workaround is possible
          • Closed
          has been marked as being related by

          Bug - A problem which impairs or prevents Moodle from functioning correctly. MDL-48912 Essay page title disclosed to student in email notification

          • Minor - Minor loss of function where workaround is possible
          • Closed

          Task - A task that needs to be completed. Tasks usually refer to work that must be done outside the product. CONTRIB-5295 Request for Activity Feed plugin review

          • Minor - Minor loss of function where workaround is possible
          • Closed
          has to be done before

          Task - A task that needs to be completed. Tasks usually refer to work that must be done outside the product. MDL-55449 Remove support for message objects as stdClass

          • Minor - Minor loss of function where workaround is possible
          • Closed
          will help resolve

          Improvement - An enhancement to an existing Moodle feature. MDL-49168 Refactor all instances of messages object to use \core\message\message

          • Minor - Minor loss of function where workaround is possible
          • Closed

          Activity

            People

              emerrill Eric Merrill
              amanda.doughty Amanda Doughty
              Simey Lameze Simey Lameze
              Eloy Lafuente (stronk7) Eloy Lafuente (stronk7)
              CiBoT CiBoT
              Adrian Greeve, Ilya Tregubov, Kevin Percy, Mathew May, Mihail Geshoski, Shamim Rezaie, Amaia Anabitarte, Carlos Escobedo, Laurent David, Sabina Abellan, Sara Arjona (@sarjona), Adrian Greeve, Ilya Tregubov, Kevin Percy, Mathew May, Mihail Geshoski, Shamim Rezaie, Ilya Tregubov, Kevin Percy, Mathew May, Mihail Geshoski, Shamim Rezaie, David Woloszyn, Huong Nguyen, Jake Dallimore, Meirza, Michael Hawkins, Raquel Ortega, Safat Shahin, Stevani Andolo, David Woloszyn, Huong Nguyen, Jake Dallimore, Meirza, Michael Hawkins, Raquel Ortega, Safat Shahin, Stevani Andolo, Tim Hunt, Ilya Tregubov, Kevin Percy, Mathew May, Mihail Geshoski, Shamim Rezaie
              Votes:
              10 Vote for this issue
              Watchers:
              17 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:
                5/Dec/16