Do not process sub templates if no main template (master template) is present!

Author: mcramer

interface/lib/classes/client_templates.inc.php

@@ -9,11 +9,9 @@
 
 class client_templates {
 
-	function apply_client_templates($clientId, $limits = array()) {
+	function apply_client_templates($clientId) {
         global $app;
 
-        if(!is_array($limits)) $limits = array();
-        
         /*
          * Get the master-template for the client
          */
@@ -28,6 +26,11 @@ function apply_client_templates($clientId, $limits = array()) {
         if ($masterTemplateId > 0){
             $sql = "SELECT * FROM client_template WHERE template_id = " . intval($masterTemplateId);
             $limits = $app->db->queryOneRecord($sql);
+        } else {
+            // if there is no master template it makes NO SENSE adding sub templates.
+            // adding subtemplates are stored in client limits, so they would add up
+            // on every save action for the client -> too high limits!
+            return;
         }
 
         /*

interface/lib/plugins/clients_template_plugin.inc.php

@@ -29,6 +29,6 @@ function apply_client_templates($event_name, $page_form) {
         global $app;
 
         $app->uses('client_templates');
-        $app->client_templates->apply_client_templates($page_form->id, $page_form->dataRecord);
+        $app->client_templates->apply_client_templates($page_form->id);
     }
 }