From 5ca959fa688255a8de61f89fe2751eb4d24a6912 Mon Sep 17 00:00:00 2001
From: Florian Schaal <florian@schaal-24.de>
Date: Tue, 22 Mar 2016 09:22:07 -0400
Subject: [PATCH] fixed typo

---
 interface/web/sites/cron_edit.php |  250 +++++++++++++++++++++++++-------------------------
 1 files changed, 125 insertions(+), 125 deletions(-)

diff --git a/interface/web/sites/cron_edit.php b/interface/web/sites/cron_edit.php
index a3244f0..a832649 100644
--- a/interface/web/sites/cron_edit.php
+++ b/interface/web/sites/cron_edit.php
@@ -39,8 +39,8 @@
 * End Form configuration
 ******************************************/
 
-require_once('../../lib/config.inc.php');
-require_once('../../lib/app.inc.php');
+require_once '../../lib/config.inc.php';
+require_once '../../lib/app.inc.php';
 
 //* Check permissions for module
 $app->auth->check_module_permissions('sites');
@@ -50,10 +50,10 @@
 $app->load('tform_actions');
 
 class page_action extends tform_actions {
-	
+
 	function onShowNew() {
 		global $app, $conf;
-		
+
 		// we will check only users, not admins
 		if($_SESSION["s"]["user"]["typ"] == 'user') {
 			if(!$app->tform->checkClientLimit('limit_cron')) {
@@ -63,158 +63,158 @@
 				$app->error('Reseller: '.$app->tform->wordbook["limit_cron_txt"]);
 			}
 		}
-		
+
 		parent::onShowNew();
 	}
-	
+
 	function onShowEnd() {
 		global $app, $conf;
-		
-        if($this->id > 0) {
-            //* we are editing a existing record
-            $app->tpl->setVar("edit_disabled", 1);
-            $app->tpl->setVar("parent_domain_id_value", $this->dataRecord["parent_domain_id"]);
-        } else {
-            $app->tpl->setVar("edit_disabled", 0);
-        }
-		
+
+		if($this->id > 0) {
+			//* we are editing a existing record
+			$app->tpl->setVar("edit_disabled", 1);
+			$app->tpl->setVar("parent_domain_id_value", $this->dataRecord["parent_domain_id"]);
+		} else {
+			$app->tpl->setVar("edit_disabled", 0);
+		}
+
 		parent::onShowEnd();
 	}
-	
+
 	function onSubmit() {
 		global $app, $conf;
-		
+
 		if($_SESSION["s"]["user"]["typ"] != 'admin') {
 			// Get the limits of the client
-			$client_group_id = $_SESSION["s"]["user"]["default_group"];
-			$client = $app->db->queryOneRecord("SELECT limit_cron, limit_cron_type FROM sys_group, client WHERE sys_group.client_id = client.client_id and sys_group.groupid = $client_group_id");
-		
+			$client_group_id = $app->functions->intval($_SESSION["s"]["user"]["default_group"]);
+			$client = $app->db->queryOneRecord("SELECT limit_cron, limit_cron_type FROM sys_group, client WHERE sys_group.client_id = client.client_id and sys_group.groupid = ?", $client_group_id);
+
 			// When the record is updated
 			if($this->id > 0) {
-			// When the record is inserted
+				// When the record is inserted
 			} else {
 				// Check if the user may add another cron job.
 				if($client["limit_cron"] >= 0) {
-					$tmp = $app->db->queryOneRecord("SELECT count(id) as number FROM cron WHERE sys_groupid = $client_group_id");
+					$tmp = $app->db->queryOneRecord("SELECT count(id) as number FROM cron WHERE sys_groupid = ?", $client_group_id);
 					if($tmp["number"] >= $client["limit_cron"]) {
 						$app->error($app->tform->wordbook["limit_cron_txt"]);
 					}
 				}
 			}
 		}
-		
-        // Get the record of the parent domain
-        $parent_domain = $app->db->queryOneRecord("select * FROM web_domain WHERE domain_id = ".intval(@$this->dataRecord["parent_domain_id"]));
-        
-        // Set fixed values
-        $this->dataRecord["server_id"] = $parent_domain["server_id"];
 
-        //* get type of command
-        $command = $this->dataRecord["command"];
-        if(preg_match("'^http(s)?:\/\/'i", $command)) {
-            $this->dataRecord["type"] = 'url';
-        } else {
-            $domain_owner = $app->db->queryOneRecord("SELECT limit_cron_type FROM sys_group, client WHERE sys_group.client_id = client.client_id and sys_group.groupid = ".intval($parent_domain["sys_groupid"]));
-            if($domain_owner["limit_cron_type"] == 'full') $this->dataRecord["type"] = 'full';
-            else $this->dataRecord["type"] = 'chrooted';
-        }
-        
-        parent::onSubmit();
+		// Get the record of the parent domain
+		$parent_domain = $app->db->queryOneRecord("select * FROM web_domain WHERE domain_id = ? AND ".$app->tform->getAuthSQL('r'), @$this->dataRecord["parent_domain_id"]);
+		if(!$parent_domain || $parent_domain['domain_id'] != @$this->dataRecord['parent_domain_id']) $app->tform->errorMessage .= $app->tform->lng("no_domain_perm");
+
+		// Set fixed values
+		$this->dataRecord["server_id"] = $parent_domain["server_id"];
+
+		//* get type of command
+		$command = $this->dataRecord["command"];
+		if(preg_match("'^http(s)?:\/\/'i", $command)) {
+			$this->dataRecord["type"] = 'url';
+		} else {
+			$domain_owner = $app->db->queryOneRecord("SELECT limit_cron_type FROM sys_group, client WHERE sys_group.client_id = client.client_id and sys_group.groupid = ?", $parent_domain["sys_groupid"]);
+			//* True when the site is assigned to a client
+			if(isset($domain_owner["limit_cron_type"])) {
+				if($domain_owner["limit_cron_type"] == 'full') {
+					$this->dataRecord["type"] = 'full';
+				} else {
+					$this->dataRecord["type"] = 'chrooted';
+				}
+			} else {
+				//* True when the site is assigned to the admin
+				$this->dataRecord["type"] = 'full';
+			}
+		}
+
+		parent::onSubmit();
 	}
-	
-    function onUpdateSave($sql) {
-        global $app;
-        
-        $has_error = false;
-        //* last chance to stop this, so check frequency limit!
-        if($_SESSION["s"]["user"]["typ"] != 'admin') {
-            // Get the limits of the client
-            $client_group_id = $_SESSION["s"]["user"]["default_group"];
-            $client = $app->db->queryOneRecord("SELECT limit_cron_frequency FROM sys_group, client WHERE sys_group.client_id = client.client_id and sys_group.groupid = $client_group_id");
-        
-            if($client["limit_cron_frequency"] > 1) {
-                if($app->tform->cron_min_freq < $client["limit_cron_frequency"]) {
-                    $app->error($app->tform->wordbook["limit_cron_frequency_txt"]);
-                    $has_error = true;
-                }
-            }
-        }
-        
-        if($has_error == true) {
-            parent::onError();
-            exit;
-        }
-        else parent::onUpdateSave($sql);
-    }
-    
-    function onInsertSave($sql) {
-        global $app;
-    
-        $has_error = false;
-        //* last chance to stop this, so check frequency limit!
-        if($_SESSION["s"]["user"]["typ"] != 'admin') {
-            // Get the limits of the client
-            $client_group_id = $_SESSION["s"]["user"]["default_group"];
-            $client = $app->db->queryOneRecord("SELECT limit_cron_frequency FROM sys_group, client WHERE sys_group.client_id = client.client_id and sys_group.groupid = $client_group_id");
-        
-            if($client["limit_cron_frequency"] > 1) {
-                if($app->tform->cron_min_freq < $client["limit_cron_frequency"]) {
-                    $app->error($app->tform->wordbook["limit_cron_frequency_txt"]);
-                    $has_error = true;
-                }
-            }
-        }
-        
-        if($has_error == true) {
-            parent::onError();
-            exit;
-        }
-        else parent::onInsertSave($sql);
-    }
-    
+
+	function onUpdateSave($sql) {
+		global $app;
+
+		$has_error = false;
+		//* last chance to stop this, so check frequency limit!
+		if($_SESSION["s"]["user"]["typ"] != 'admin') {
+			// Get the limits of the client
+			$client_group_id = $app->functions->intval($_SESSION["s"]["user"]["default_group"]);
+			$client = $app->db->queryOneRecord("SELECT limit_cron_frequency, limit_cron_type FROM sys_group, client WHERE sys_group.client_id = client.client_id and sys_group.groupid = ?", $client_group_id);
+
+			if($client["limit_cron_frequency"] > 1) {
+				if($app->tform->cron_min_freq < $client["limit_cron_frequency"]) {
+					$app->error($app->tform->wordbook["limit_cron_frequency_txt"]);
+					$has_error = true;
+				}
+			}
+			
+			if($client["limit_cron_type"] == 'url' && $this->dataRecord["type"] != 'url') {
+				$app->error($app->tform->wordbook["limit_cron_url_txt"]);
+				$has_error = true;
+			}
+		}
+
+		if($has_error == true) {
+			parent::onError();
+			exit;
+		}
+		else parent::onUpdateSave($sql);
+	}
+
+	function onInsertSave($sql) {
+		global $app;
+
+		$has_error = false;
+		//* last chance to stop this, so check frequency limit!
+		if($_SESSION["s"]["user"]["typ"] != 'admin') {
+			// Get the limits of the client
+			$client_group_id = $app->functions->intval($_SESSION["s"]["user"]["default_group"]);
+			$client = $app->db->queryOneRecord("SELECT limit_cron_frequency, limit_cron_type FROM sys_group, client WHERE sys_group.client_id = client.client_id and sys_group.groupid = ?", $client_group_id);
+
+			if($client["limit_cron_frequency"] > 1) {
+				if($app->tform->cron_min_freq < $client["limit_cron_frequency"]) {
+					$app->error($app->tform->wordbook["limit_cron_frequency_txt"]);
+					$has_error = true;
+				}
+			}
+			
+			if($client["limit_cron_type"] == 'url' && $this->dataRecord["type"] != 'url') {
+				$app->error($app->tform->wordbook["limit_cron_url_txt"]);
+				$has_error = true;
+			}
+		}
+
+		if($has_error == true) {
+			parent::onError();
+			exit;
+		} else {
+			return parent::onInsertSave($sql);
+		}
+	}
+
 	function onAfterInsert() {
 		global $app, $conf;
-		
-        $web = $app->db->queryOneRecord("SELECT * FROM web_domain WHERE domain_id = ".intval($this->dataRecord["parent_domain_id"]));
-        $server_id = $web["server_id"];
-        
-        // The cron shall be owned by the same group then the website
-        $sys_groupid = $web['sys_groupid'];
-        
-        $sql = "UPDATE shell_user SET server_id = $server_id, sys_groupid = '$sys_groupid' WHERE id = ".$this->id;
-        $app->db->query($sql);
+
+		$web = $app->db->queryOneRecord("SELECT * FROM web_domain WHERE domain_id = ?", $this->dataRecord["parent_domain_id"]);
+		$server_id = $web["server_id"];
+
+		// The cron shall be owned by the same group then the website
+		$sys_groupid = $app->functions->intval($web['sys_groupid']);
+
+		$sql = "UPDATE cron SET server_id = ?, sys_groupid = ? WHERE id = ?";
+		$app->db->query($sql, $server_id, $sys_groupid, $this->id);
 	}
-	
+
 	function onAfterUpdate() {
 		global $app, $conf;
-		
-		
+
+
 	}
-    
-    function getClientName() {
-        global $app, $conf;
-    
-        if($_SESSION["s"]["user"]["typ"] != 'admin') {
-            // Get the group-id of the user
-            $client_group_id = $_SESSION["s"]["user"]["default_group"];
-        } else {
-            // Get the group-id from the data itself
-            $web = $app->db->queryOneRecord("SELECT sys_groupid FROM web_domain WHERE domain_id = ".intval($this->dataRecord['parent_domain_id']));
-            $client_group_id = $web['sys_groupid'];
-        }
-        /* get the name of the client */
-        $tmp = $app->db->queryOneRecord("SELECT name FROM sys_group WHERE groupid = " . $client_group_id);
-        $clientName = $tmp['name'];
-        if ($clientName == "") $clientName = 'default';
-        $clientName = convertClientName($clientName);
-        
-        return $clientName;
-    
-    }
-	
+
 }
 
 $page = new page_action;
 $page->onLoad();
 
-?>
\ No newline at end of file
+?>

--
Gitblit v1.9.1