SqlProvisioner bug fixes (CFM-74)

app/availableplugins/SqlConnector/src/Model/Table/SqlProvisionersTable.php

@@ -332,8 +332,11 @@ public function applySchema($id) {
    */
 
   public function localAfterSave(\Cake\Event\EventInterface $event, \Cake\Datasource\EntityInterface $entity, \ArrayObject $options): bool {
-    // We may not have a Server configuration yet on first save
-    if(!empty($spcfg->server_id)) {
+    // We may not have a Server configuration yet on first save.
+
+    // Also, When the SQL Provisioner is deleted, neither the
+    // database schema nor reference data is touched (PAR-SqlProvisioner-4).
+    if(!empty($entity->server_id) && !$entity->deleted) {
       // Apply the database schema (PAR-SqlProvisioner-1)
       $this->llog('rule', "PAR-SqlProvisioner-1 Applying database schema for SqlProvisioner " . $entity->id);
       $this->applySchema($entity->id);

app/plugins/CoreServer/src/Model/Table/SqlServersTable.php

@@ -133,6 +133,9 @@ public function connect(int $serverId, string $name): bool {
       'timezone'          => 'UTC'
     ];
 
+    // We need to drop the existing configuration before we can reconfigure it
+    ConnectionManager::drop('targetdb');
+
     ConnectionManager::setConfig('targetdb', $dbconfig);
 
     return true;

app/src/Lib/Util/SchemaManager.php

@@ -93,6 +93,9 @@ public function __construct(?ConsoleIo $io=null, string $connection='default') {
       $cfargs['ssl_ca'] = $cfg['ssl_ca'];
     }
 
+    if($this->io) $this->io->out("Connecting to database " . $cfg['database'] . " as " 
+                                  . $cfg['username'] . "@" . $cfg['host']);
+
     $this->conn = DriverManager::getConnection($cfargs, $config);
     $this->driver = $cfg['driver'];
   }