From: Janusz Dobrowolski <janusz@frontaccouting.eu>
Date: Fri, 30 Nov 2012 13:17:58 +0000 (+0100)
Subject: Fixed error in attachments edition resulting in unexpected attachment file deletion... 
X-Git-Url: https://delta.frontaccounting.com/gitweb/?a=commitdiff_plain;h=61d3c6b87de1847245227b4b41d4fe48893a3f4a;p=textcart.git

Fixed error in attachments edition resulting in unexpected attachment file deletion, fixed attachments list update after change.
---

diff --git a/admin/attachments.php b/admin/attachments.php
index b75c58b..b34cbe3 100644
--- a/admin/attachments.php
+++ b/admin/attachments.php
@@ -98,22 +98,24 @@ if ($Mode == 'ADD_ITEM' || $Mode == 'UPDATE_ITEM')
 			fwrite($fp, $index_file);
 			fclose($fp);
 		}
+
+		$filename = basename($_FILES['filename']['name']);
+		$filesize = $_FILES['filename']['size'];
+		$filetype = $_FILES['filename']['type'];
+
 		// file name compatible with POSIX
 		// protect against directory traversal
 		if ($Mode == 'UPDATE_ITEM')
 		{
 			$unique_name = preg_replace('/[^a-zA-Z0-9.\-_]/', '', $_POST['unique_name']);
-			if ($Mode == 'UPDATE_ITEM' && file_exists($dir."/".$unique_name))
+			if ($filename && file_exists($dir."/".$unique_name))
 				unlink($dir."/".$unique_name);
 		}
 		else
 			$unique_name = uniqid('');
-		move_uploaded_file($tmpname, $dir."/".$unique_name);
 
 		//save the file
-		$filename = basename($_FILES['filename']['name']);
-		$filesize = $_FILES['filename']['size'];
-		$filetype = $_FILES['filename']['type'];
+		move_uploaded_file($tmpname, $dir."/".$unique_name);
 
 		if ($Mode == 'ADD_ITEM')
 		{
@@ -128,6 +130,8 @@ if ($Mode == 'ADD_ITEM' || $Mode == 'UPDATE_ITEM')
 			display_notification(_("Attachment has been updated.")); 
 		}
 	}
+	refresh_pager('trans_table');
+	$Ajax->activate('_page_body');
 	$Mode = 'RESET';
 }