X-Git-Url: https://delta.frontaccounting.com/gitweb/?a=blobdiff_plain;f=admin%2Finst_upgrade.php;h=20a056557e80d07fc16c263376f77b612ca7898a;hb=07a0b37ca7360c5dbb058f5a0ba4c5752671dda7;hp=1048ed2f7926092aa1a4425a7e93d90a73dd7d54;hpb=e5536a0814ee5ce77c9cccfd4ec74a0ebd1973d0;p=fa-stable.git

diff --git a/admin/inst_upgrade.php b/admin/inst_upgrade.php
index 1048ed2f..20a05655 100644
--- a/admin/inst_upgrade.php
+++ b/admin/inst_upgrade.php
@@ -9,7 +9,7 @@
     MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  
     See the License here <http://www.gnu.org/licenses/gpl-3.0.html>.
 ***********************************************************************/
-$page_security = 20;
+$page_security = 'SA_SOFTWAREUPGRADE';
 $path_to_root="..";
 include_once($path_to_root . "/includes/session.inc");
 
@@ -29,7 +29,7 @@ include_once($path_to_root . "/includes/ui.inc");
 //
 function check_table($pref, $table, $field=null, $properties=null)
 {
-	$fields = db_query("SHOW COLUMNS FROM ".$pref.$table);
+	$fields = @db_query("SHOW COLUMNS FROM ".$pref.$table);
 	if (!$fields)
 		return 1;		// no such table or error
 
@@ -96,7 +96,8 @@ function upgrade_step($index, $conn)
 	$force = get_post('force_'.$index);
 	if ($force || get_post('install_'.$index)) 
 	{
-		if (!$inst->installed($pref) || $force) 
+		$state = $inst->installed($pref);
+		if (!$state || $force) 
 		{
 			if (!$inst->pre_check($pref)) return false;
 
@@ -104,7 +105,11 @@ function upgrade_step($index, $conn)
 				$ret &= db_import($path_to_root.'/sql/'.$sql, $conn, $force);
 
 			$ret &= $inst->install($pref, $force);
-		}
+		} else
+			if ($state!==true) {
+				display_error(_("Upgrade cannot be done because database has been already partially upgraded. Please downgrade database to clean previous version or try forced upgrade."));
+				$ret = false;
+			}
 	}
 	return $ret;
 }
@@ -134,13 +139,8 @@ if (get_post('Upgrade'))
 				." '".$conn['name']."'");
 			continue;
 		}
-	// create security backup		
-	 	if ($conn['tbpref'] != "")
-			$filename = $conn['dbname'] . "_" . $conn['tbpref'] . date("Ymd_Hi") . ".sql";
-		else
-			$filename = $conn['dbname'] . "_" . date("Ymd_Hi") . ".sql";
-
-		db_export($conn, $filename, 'no', 'Security backup before upgrade', $conn['tbpref']);
+	// create security backup	
+		db_backup($conn, 'no', 'Security backup before upgrade', $conn['tbpref']);
 	// apply all upgrade data
 		foreach ($installers as $i => $inst) 
 		{
@@ -173,6 +173,7 @@ $th = array(_("Version"), _("Description"), _("Sql file"), _("Install"),
 table_header($th);
 
 $k = 0; //row colour counter
+$partial = 0;
 foreach($installers as $i => $inst)
 {
 	alt_table_row_color($k);
@@ -183,14 +184,27 @@ foreach($installers as $i => $inst)
 // this is checked only for first (site admin) company, 
 // but in fact we should always upgrade all data sets after
 // source upgrade.
-	if ($inst->installed(TB_PREF))
+	$check = $inst->installed(TB_PREF);
+	if ($check === true)
 		label_cell(_("Installed"));
-	else
-		check_cells(null,'install_'.$i, 0);
+	else 
+		if (!$check)
+			check_cells(null,'install_'.$i, 0);
+		else {
+			label_cell("<span class=redfg>"
+				. sprintf(_("Partially installed (%s)"), $check) . "</span>");
+			$partial++;
+		}
+
 	check_cells(null,'force_'.$i, 0);
 	end_row();
 }
 end_table(1);
+if ($partial!=0)	{
+	display_note(_("Database upgrades marked as partially installed cannot be installed automatically.
+You have to clean database manually to enable them, or try to perform forced upgrade."));
+	br();
+}
 submit_center('Upgrade', _('Upgrade system'), true, _('Save database and perform upgrade'), 'process');
 end_form();