Cleaning up code to match coding guidelines

Marius Cramer

2013-11-14 7fe908c50c8dbc5cc05f571dbe11d66141caacd4

 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,35 +63,35 @@
            $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");
		

         // 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) {
@@ -102,21 +102,21 @@
            }
         }
      }
		
        // Get the record of the parent domain
        $parent_domain = $app->db->queryOneRecord("select * FROM web_domain WHERE domain_id = ".$app->functions->intval(@$this->dataRecord["parent_domain_id"]) . " AND ".$app->tform->getAuthSQL('r'));
        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 = ".$app->functions->intval($parent_domain["sys_groupid"]));
            //* True when the site is assigned to a client
      // Get the record of the parent domain
      $parent_domain = $app->db->queryOneRecord("select * FROM web_domain WHERE domain_id = ".$app->functions->intval(@$this->dataRecord["parent_domain_id"]) . " AND ".$app->tform->getAuthSQL('r'));
      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 = ".$app->functions->intval($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';
@@ -127,84 +127,84 @@
            //* True when the site is assigned to the admin
            $this->dataRecord["type"] = 'full';
         }
        }
        
        parent::onSubmit();
      }

      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 {

   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 {
         return parent::onInsertSave($sql);
      }
    }
    
   }

   function onAfterInsert() {
      global $app, $conf;
		
        $web = $app->db->queryOneRecord("SELECT * FROM web_domain WHERE domain_id = ".$app->functions->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 cron 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 = ".$app->functions->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 cron SET server_id = $server_id, sys_groupid = '$sys_groupid' WHERE id = ".$this->id;
      $app->db->query($sql);
   }
	

   function onAfterUpdate() {
      global $app, $conf;
		
		


   }
	

}

$page = new page_action;
$page->onLoad();

?>
?>