CTP4885 - Lifecycle plugin proj (with full commits) #301
CTP4885 - Lifecycle plugin proj (with full commits) #301ccaewan wants to merge 36 commits intolearnweb:mainfrom
Conversation
Updated to reduce risk of host timeout
Added error prevention for already deleted courses and commented out the delete_course function for testing.
| */ | ||
| public function process_course($processid, $instanceid, $course) { | ||
|
|
||
| if (!class_exists('\block_lifecycle\manager')) { |
There was a problem hiding this comment.
I don't know this class: block_lifecycle\manager ... what is this?
There was a problem hiding this comment.
Hi @bluetom apologies for the late response I have been on AL.
Block_lifecycle\manager is a reference to a scheduled task that archives courses (or makes read-only). I am not 100% sure how it came to be but I believe it was created within the organisation.
It successfully works with the CLC trigger as a step to archive active courses. However this code has issues when using the delete course step in succession
There was a problem hiding this comment.
So this step plugin works only within your organisation?
There was a problem hiding this comment.
I believe the repo holding the logic is public and can be found here:
[https://github.com/ucl-isd/moodle-block_lifecycle](url)
2 participants
🔀 Purpose of this PR:
📝 Description:
Please describe the purpose of this PR in a few sentences.
📋 Checklist
Please confirm the following (check all that apply):
phpunitand/orbehattests that cover my changes or additions.var_dump()orvar_exportor any other debugging statements (or commented out code) thatshould not appear on the productive branch.
db/upgrade.phpandupdated the
version.php..minfiles with thegrunt amdcommand.version.phpand theCHANGES.md.I ran all tests thoroughly checking for errors. I checked if bootstrap had any changes/deprecations that require
changes in the plugins UI.
🔍 Related Issues
🧾📸🌐 Additional Information (like screenshots, documentation, links, etc.)
Any other relevant information.