ChrisChrome/sccp_manager
1
0
Fork 0
Code Issues Pull requests Actions Packages Projects Releases Wiki Activity

Modify installer db calls to remove IF EXISTS

Browse source 
IF EXISTS is not correct in MariaDb v5, used in all FreePbx Packages.
This commit is contained in:
stevenA 2022-01-16 16:04:41 +01:00
parent b1769e6378
commit 546de5add1
2 changed files with 4 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
install.php
View file

@ -433,7 +433,7 @@ function InstallDB_updateSchema($db_config)
}
// Processed all _Column names; now safe to delete them
$sqlDrop = array_reduce($fieldsArr, function($carry, $column) {
return "${carry} DROP COLUMN IF EXISTS ${column},";
return "${carry} DROP COLUMN ${column},";
});
$sqlDrop = rtrim($sqlDrop, ", ");
$stmt = $db->prepare("ALTER TABLE ${table} ${sqlDrop}");
@ -463,7 +463,7 @@ function InstallDB_updateSchema($db_config)
// occur as columns that are dropped should no longer be in the module.xml schema
// and so Doctrine will have already dropped them.
if (!empty($tab_modif[$fld_id]['drop'])) {
$sql_create .= "DROP COLUMN IF EXISTS {$row_fld}, ";
$sql_create .= "DROP COLUMN {$row_fld}, ";
unset($tab_modif[$fld_id]['drop']);
continue;
}

3
module.xml
View file

@ -1,7 +1,7 @@
<module>
<rawname>sccp_manager</rawname>
<name>SCCP Manager</name>
<version>14.4.0.1</version>
<version>14.4.0.2</version>
<type>setup</type>
<category>SCCP Connectivity</category>
<publisher>Steve Lad, Alex GP</publisher>
@ -38,6 +38,7 @@
* Version 14.3.0.30 * - Fix reversion in 29. Not returning to correct page when saving device.
* Version 14.3.0.31 * - Fix rewrite rules.
* Version 14.4.0.1 * - Modify installer to avoid data loss on existing 14.3 fields. Bump Minor version to reflect this.
* Version 14.4.0.2 * - Revert error in Installer db calls - IF EXISTS is not valid in MariaDb v5 (package DB)
</changelog>
<location>https://github.com/chan-sccp/sccp_manager</location>
<depends>