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

CalendarEvents.created fired on edit

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 3.4
    • Fix Version/s: 3.4
    • Component/s: Calendar
    • Labels:
    • Testing Instructions:
      Hide
      1. Navigate to the month view of the calendar
      2. Open the JS console
      3. Open first.js and search for all listeners for the CalendarEvents.created. and CalendarEvents.updated events - add a breakpoint within each handler
      4. Create a new event in the month view and hit save
        1. Confirm that the breakpoint stopped in the CalendarEvents.created listener
      5. Edit the event and hit save
        1. Confirm that the breakpoint stopped in the CalendarEvents.updated listener
      6. Repeat in the Day view
      Show
      Navigate to the month view of the calendar Open the JS console Open first.js and search for all listeners for the CalendarEvents.created . and CalendarEvents.updated events - add a breakpoint within each handler Create a new event in the month view and hit save Confirm that the breakpoint stopped in the CalendarEvents.created listener Edit the event and hit save Confirm that the breakpoint stopped in the CalendarEvents.updated listener Repeat in the Day view
    • Affected Branches:
      MOODLE_34_STABLE
    • Fixed Branches:
      MOODLE_34_STABLE
    • Pull Master Branch:
      MDL-60560-master

      Description

      We currently fire the wrong calendar event on edit.
      We currently fire the created event, instead of the updated event.
      There's no UI change at this time, but we should correct the distinction.

        Attachments

          Activity

            People

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

              Dates

              • Created:
                Updated:
                Resolved:
                Fix Release Date:
                13/Nov/17