Skip to content

Commit

Permalink
Test deprecating ltiservice for phpunit runs
Browse files Browse the repository at this point in the history
Also removes from backup/restore, which should have been done
before deprecation anyway.
  • Loading branch information
snake committed Nov 20, 2024
1 parent 8872182 commit 5eacb7d
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 3 deletions.
1 change: 0 additions & 1 deletion mod/lti/backup/moodle2/backup_lti_stepslib.php
Original file line number Diff line number Diff line change
Expand Up @@ -213,7 +213,6 @@ protected function define_structure() {

// Add support for subplugin structures.
$this->add_subplugin_structure('ltisource', $lti, true);
$this->add_subplugin_structure('ltiservice', $lti, true);

// Return the root element (lti), wrapped into standard activity structure.
return $this->prepare_activity_structure($lti);
Expand Down
1 change: 0 additions & 1 deletion mod/lti/backup/moodle2/restore_lti_stepslib.php
Original file line number Diff line number Diff line change
Expand Up @@ -82,7 +82,6 @@ protected function define_structure() {

// Add support for subplugin structures.
$this->add_subplugin_structure('ltisource', $lti);
$this->add_subplugin_structure('ltiservice', $lti);

// Return the paths wrapped into standard activity structure.
return $this->prepare_activity_structure($paths);
Expand Down
4 changes: 3 additions & 1 deletion mod/lti/db/subplugins.json
Original file line number Diff line number Diff line change
@@ -1,6 +1,8 @@
{
"plugintypes": {
"ltisource": "mod\/lti\/source",
"ltisource": "mod\/lti\/source"
},
"deprecatedplugintypes": {
"ltiservice": "mod\/lti\/service"
}
}

0 comments on commit 5eacb7d

Please sign in to comment.