Commit de2d7fc4 authored by Cecilia Vela Gurovic's avatar Cecilia Vela Gurovic Committed by Robert Lyon

Bug 1850786: remove plugin ES triggers

we need to remove the plugin ES triggers in lib/db/upgrade.php
when we remove the core triggers,
so they are not present on the DB when the plugins get upgraded

This is the same as calling drop_elasticsearch_triggers() but without
the searchplugin check - in case someone manually changed the search
type in db

behatnotneeded

Change-Id: I47df9cacfa69dd6f7532f570fe17b6a66e5d0fea
parent 29d32897
......@@ -1542,5 +1542,15 @@ function xmldb_core_upgrade($oldversion=0) {
}
}
if ($oldversion < 2019111100) {
log_debug('Drop the elasticsearch triggers for the plugins');
require_once(get_config('docroot') . 'search/elasticsearch/lib.php');
$enabledtypes = explode(',', get_config_plugin('search', 'elasticsearch', 'types'));
foreach ($enabledtypes as $type) {
ElasticsearchIndexing::drop_triggers($type);
}
ElasticsearchIndexing::drop_trigger_functions();
}
return $status;
}
......@@ -16,7 +16,7 @@ $config = new stdClass();
// See https://wiki.mahara.org/wiki/Developer_Area/Version_Numbering_Policy
// For upgrades on stable branches, increment the version by one. On master, use the date.
$config->version = 2019102000;
$config->version = 2019111100;
$config->series = '20.04';
$config->release = '20.04dev';
$config->minupgradefrom = 2017031605;
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment