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

Add relevant record snapshot to existing events

    XMLWordPrintable

    Details

    • Type: Task
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 3.1
    • Fix Version/s: None
    • Component/s: Competencies
    • Labels:
    • Story Points:
      1
    • Sprint:
      CBE Sprint 6

      Description

      I failed to notice and explain what record snapshots were and therefore most of the events in tool_lp do not include any snapshot. This issue is about checking the existing events and where they are triggered from and add relevant snapshots where they are missing.

      I think this means:

      1. Adding self to the create_from*()_ methods:

        $event->add_record_snapshot(self::TABLE, $this->to_record());
        

      2. Add extra snapshots where they are triggered
        • For instance when a user competency is updated, adding the snapshot of the competency if we have it, etc...

        Attachments

          Activity

            People

            Assignee:
            sergeg55 Serge Gauthier
            Reporter:
            fred Frédéric Massart
            Peer reviewer:
            Frédéric Massart
            Participants:
            Component watchers:
            Damyon Wiese, Jean-Philippe Gaudreau, Steve Massicotte, Serge Gauthier, Issam Taboubi, Adrian Greeve, Jake Dallimore, Mathew May, Mihail Geshoski, Peter Dias
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved: