Fixed: FS#1111 - Delete an 'in use' template should be blocked

Author: tbrehm

interface/web/client/client_template_del.php

@@ -44,12 +44,21 @@
 
 //* Check permissions for module
 $app->auth->check_module_permissions('client');
-if(!$_SESSION["s"]["user"]["typ"] == 'admin') die('Client-Templates are only for Admins.');
+if(!$_SESSION["s"]["user"]["typ"] == 'admin') die('Client-Templates are for Admins only.');
 
 $app->uses('tpl,tform');
 $app->load('tform_actions');
 
 class page_action extends tform_actions {
+	function onBeforeDelete() {
+		global $app;
+		
+		$rec = $app->db->queryOneRecord("SELECT count(client_id) as number FROM client WHERE template_master = ".$this->id." OR template_additional like '%/".$this->id."/%'");
+		if($rec['number'] > 0) {
+			$app->error($app->tform->lng('template_del_aborted_txt'));
+		}
+		
+	}
 }
 
 $page = new page_action;

interface/web/client/lib/lang/en_client_template.lng

@@ -57,4 +57,5 @@ $wb["error_template_name_empty"] = 'Please enter a Template name';
 $wb["limit_web_quota_txt"] = 'Web Quota';
 $wb["limit_traffic_quota_txt"] = 'Traffic Quota';
 $wb["limit_trafficquota_error_notint"] = 'Traffic Quota must be a number.';
+$wb["template_del_aborted_txt"] = 'Delete aborted. There is still a client which has this template selected.';
 ?>