Loading...

XML

Word

Printable

Affected Branches:
MOODLE_26_STABLE, MOODLE_32_STABLE
Fixed Branches:
MOODLE_32_STABLE
Epic Link:
Action logging
Pull from Repository:
https://github.com/sbourget/moodle.git
Pull Main Branch:
~~MDL-44369~~_m32v3
Pull Main Diff URL:
https://github.com/moodle/moodle/compare/master...sbourget:MDL-44369_m32v3
Difficulty:
Moderate
Testing Instructions:
Hide

Automated Testing:

Run phpunit tests for calendar (vendor/bin/phpunit core_calendar_events_testcase calendar/tests/events_test.php)

Manual testing:

Add a calendar subscription

Update a calendar subscription

Delete a calendar subscription

(Perform these for site, course, and user events) Make sure each are logged appear in the logs.

NOTE: Deleting calendar subscriptions will not create log entries for removing the individual calendar events. That should be addressed by ~~MDL-32351~~.
Show
Automated Testing: Run phpunit tests for calendar (vendor/bin/phpunit core_calendar_events_testcase calendar/tests/events_test.php) Manual testing: Add a calendar subscription Update a calendar subscription Delete a calendar subscription (Perform these for site, course, and user events) Make sure each are logged appear in the logs. NOTE: Deleting calendar subscriptions will not create log entries for removing the individual calendar events. That should be addressed by MDL-32351 .

Calendar needs a lot of events. Most calendar actions do not trigger events. Some clear ones are :-

A detailed review might bring up other cases where an event seems required.

Also fix the bug: when subscription is deleted there are no events for deleting the individual calendar events (~~MDL-32351~~)

Participants:: Adrian Greeve, Ankit Agarwal, CiBoT, Dan Poltawski, Eloy Lafuente (stronk7), Rajeev S, Shubhangee Verma, Simey Lameze, Stephen Bourget