evolution-cms / evolution

manager/processors/save_template.processor.php

Spacing +7 added lines, -7 removed lines

@@ -1,27 +1,27 @@ 
 <?php
-if( ! defined('IN_MANAGER_MODE') || IN_MANAGER_MODE !== true) {
+if (!defined('IN_MANAGER_MODE') || IN_MANAGER_MODE !== true) {
     die("<b>INCLUDE_ORDERING_ERROR</b><br /><br />Please use the EVO Content Manager instead of accessing this file directly.");
 }
 if (!$modx->hasPermission('save_template')) {
     $modx->webAlertAndQuit($_lang["error_no_privileges"]);
 }
 
-$id = (int)$_POST['id'];
+$id = (int) $_POST['id'];
 $template = $modx->getDatabase()->escape($_POST['post']);
 $templatename = $modx->getDatabase()->escape(trim($_POST['templatename']));
 $description = $modx->getDatabase()->escape($_POST['description']);
 $locked = isset($_POST['locked']) && $_POST['locked'] == 'on' ? 1 : 0;
-$selectable = $id == $modx->config['default_template'] ? 1 :    // Force selectable
+$selectable = $id == $modx->config['default_template'] ? 1 : // Force selectable
     isset($_POST['selectable']) && $_POST['selectable'] == 'on' ? 1 : 0;
 $currentdate = time() + $modx->config['server_offset_time'];
 
 //Kyle Jaebker - added category support
 if (empty($_POST['newcategory']) && $_POST['categoryid'] > 0) {
-    $categoryid = (int)$_POST['categoryid'];
+    $categoryid = (int) $_POST['categoryid'];
 } elseif (empty($_POST['newcategory']) && $_POST['categoryid'] <= 0) {
     $categoryid = 0;
 } else {
-    include_once(MODX_MANAGER_PATH . 'includes/categories.inc.php');
+    include_once(MODX_MANAGER_PATH.'includes/categories.inc.php');
     $categoryid = checkCategory($_POST['newcategory']);
     if (!$categoryid) {
         $categoryid = newCategory($_POST['newcategory']);
@@ -78,7 +78,7 @@ 
         // finished emptying cache - redirect
         if ($_POST['stay'] != '') {
             $a = ($_POST['stay'] == '2') ? "16&id=$newid" : "19";
-            $header = "Location: index.php?a=" . $a . "&r=2&stay=" . $_POST['stay'];
+            $header = "Location: index.php?a=".$a."&r=2&stay=".$_POST['stay'];
             header($header);
         } else {
             $header = "Location: index.php?a=76&r=2";
@@ -130,7 +130,7 @@ 
         // finished emptying cache - redirect
         if ($_POST['stay'] != '') {
             $a = ($_POST['stay'] == '2') ? "16&id=$id" : "19";
-            $header = "Location: index.php?a=" . $a . "&r=2&stay=" . $_POST['stay'];
+            $header = "Location: index.php?a=".$a."&r=2&stay=".$_POST['stay'];
             header($header);
         } else {
             $modx->unlockElement(1, $id);

manager/processors/duplicate_tmplvars.processor.php

Spacing +7 added lines, -7 removed lines

@@ -1,21 +1,21 @@ 
 <?php
-if( ! defined('IN_MANAGER_MODE') || IN_MANAGER_MODE !== true) {
+if (!defined('IN_MANAGER_MODE') || IN_MANAGER_MODE !== true) {
     die("<b>INCLUDE_ORDERING_ERROR</b><br /><br />Please use the EVO Content Manager instead of accessing this file directly.");
 }
-if(!$modx->hasPermission('edit_template')) {
+if (!$modx->hasPermission('edit_template')) {
 	$modx->webAlertAndQuit($_lang["error_no_privileges"]);
 }
 
-$id = isset($_GET['id'])? (int)$_GET['id'] : 0;
-if($id==0) {
+$id = isset($_GET['id']) ? (int) $_GET['id'] : 0;
+if ($id == 0) {
 	$modx->webAlertAndQuit($_lang["error_no_id"]);
 }
 
 // count duplicates
-$tmplvar = EvolutionCMS\Models\SiteTmplvar::with(['tmplvarAccess','tmplvarTemplate'])->findOrFail($id);
+$tmplvar = EvolutionCMS\Models\SiteTmplvar::with(['tmplvarAccess', 'tmplvarTemplate'])->findOrFail($id);
 $name = $tmplvar->name;
 $count = EvolutionCMS\Models\SiteTmplvar::where('name', 'like', $name.' '.$_lang['duplicated_el_suffix'].'%')->count();
-if($count>=1) $count = ' '.($count+1);
+if ($count >= 1) $count = ' '.($count + 1);
 else $count = '';
 
 
@@ -38,5 +38,5 @@ 
 $_SESSION['itemname'] = $newTmplvar->name;
 
 // finish duplicating - redirect to new variable
-$header="Location: index.php?r=2&a=301&id=".$newTmplvar->getKey();
+$header = "Location: index.php?r=2&a=301&id=".$newTmplvar->getKey();
 header($header);

manager/processors/delete_web_user.processor.php

Spacing +5 added lines, -5 removed lines

@@ -1,13 +1,13 @@ 
 <?php
-if( ! defined('IN_MANAGER_MODE') || IN_MANAGER_MODE !== true) {
+if (!defined('IN_MANAGER_MODE') || IN_MANAGER_MODE !== true) {
     die("<b>INCLUDE_ORDERING_ERROR</b><br /><br />Please use the EVO Content Manager instead of accessing this file directly.");
 }
-if(!$modx->hasPermission('delete_web_user')) {
+if (!$modx->hasPermission('delete_web_user')) {
     $modx->webAlertAndQuit($_lang["error_no_privileges"]);
 }
 
-$id = isset($_GET['id'])? (int)$_GET['id'] : 0;
-if($id==0) {
+$id = isset($_GET['id']) ? (int) $_GET['id'] : 0;
+if ($id == 0) {
     $modx->webAlertAndQuit($_lang["error_no_id"]);
 }
 
@@ -39,5 +39,5 @@ 
         "id"	=> $id
     ));
 
-$header="Location: index.php?a=99";
+$header = "Location: index.php?a=99";
 header($header);

manager/processors/duplicate_htmlsnippet.processor.php

Spacing +6 added lines, -6 removed lines

@@ -1,13 +1,13 @@ 
 <?php
-if( ! defined('IN_MANAGER_MODE') || IN_MANAGER_MODE !== true) {
+if (!defined('IN_MANAGER_MODE') || IN_MANAGER_MODE !== true) {
     die("<b>INCLUDE_ORDERING_ERROR</b><br /><br />Please use the EVO Content Manager instead of accessing this file directly.");
 }
-if(!$modx->hasPermission('new_chunk')) {
+if (!$modx->hasPermission('new_chunk')) {
 	$modx->webAlertAndQuit($_lang["error_no_privileges"]);
 }
 
-$id = isset($_GET['id'])? (int)$_GET['id'] : 0;
-if($id==0) {
+$id = isset($_GET['id']) ? (int) $_GET['id'] : 0;
+if ($id == 0) {
 	$modx->webAlertAndQuit($_lang["error_no_id"]);
 }
 
@@ -15,7 +15,7 @@ 
 $htmlsnippet = EvolutionCMS\Models\SiteHtmlsnippet::findOrFail($id);
 $name = $htmlsnippet->name;
 $count = EvolutionCMS\Models\SiteHtmlsnippet::where('name', 'like', $name.' '.$_lang['duplicated_el_suffix'].'%')->count();
-if($count>=1) $count = ' '.($count+1);
+if ($count >= 1) $count = ' '.($count + 1);
 else $count = '';
 
 // duplicate htmlsnippet
@@ -26,5 +26,5 @@ 
 $_SESSION['itemname'] = $newHtmlsnippet->name;
 
 // finish duplicating - redirect to new chunk
-$header="Location: index.php?r=2&a=78&id=".$newHtmlsnippet->getKey();
+$header = "Location: index.php?r=2&a=78&id=".$newHtmlsnippet->getKey();
 header($header);

manager/processors/save_htmlsnippet.processor.php

Spacing +10 added lines, -10 removed lines

@@ -1,12 +1,12 @@ 
 <?php
-if( ! defined('IN_MANAGER_MODE') || IN_MANAGER_MODE !== true) {
+if (!defined('IN_MANAGER_MODE') || IN_MANAGER_MODE !== true) {
     die("<b>INCLUDE_ORDERING_ERROR</b><br /><br />Please use the EVO Content Manager instead of accessing this file directly.");
 }
 if (!$modx->hasPermission('save_chunk')) {
     $modx->webAlertAndQuit($_lang["error_no_privileges"]);
 }
 
-$id = (int)$_POST['id'];
+$id = (int) $_POST['id'];
 $snippet = $_POST['post'];
 $name = trim($_POST['name']);
 $description = $_POST['description'];
@@ -16,11 +16,11 @@ 
 
 //Kyle Jaebker - added category support
 if (empty($_POST['newcategory']) && $_POST['categoryid'] > 0) {
-    $category = (int)$_POST['categoryid'];
+    $category = (int) $_POST['categoryid'];
 } elseif (empty($_POST['newcategory']) && $_POST['categoryid'] <= 0) {
     $category = 0;
 } else {
-    include_once(MODX_MANAGER_PATH . 'includes/categories.inc.php');
+    include_once(MODX_MANAGER_PATH.'includes/categories.inc.php');
     $category = checkCategory($_POST['newcategory']);
     if (!$category) {
         $category = newCategory($_POST['newcategory']);
@@ -44,13 +44,13 @@ 
         ));
 
         // disallow duplicate names for new chunks
-        if (EvolutionCMS\Models\SiteHtmlsnippet::where('name','=',$name)->first()) {
+        if (EvolutionCMS\Models\SiteHtmlsnippet::where('name', '=', $name)->first()) {
             $modx->getManagerApi()->saveFormValues(77);
             $modx->webAlertAndQuit(sprintf($_lang['duplicate_name_found_general'], $_lang['chunk'], $name), "index.php?a=77");
         }
 
         //do stuff to save the new doc
-        $id = EvolutionCMS\Models\SiteHtmlsnippet::create(compact('name', 'description','snippet','locked','category','editor_type','editor_name','disabled','createdon','editedon'))->getKey();
+        $id = EvolutionCMS\Models\SiteHtmlsnippet::create(compact('name', 'description', 'snippet', 'locked', 'category', 'editor_type', 'editor_name', 'disabled', 'createdon', 'editedon'))->getKey();
 
         // invoke OnChunkFormSave event
         $modx->invokeEvent("OnChunkFormSave", array(
@@ -67,7 +67,7 @@ 
         // finished emptying cache - redirect
         if ($_POST['stay'] != '') {
             $a = ($_POST['stay'] == '2') ? "78&id=$id" : "77";
-            $header = "Location: index.php?a=" . $a . "&r=2&stay=" . $_POST['stay'];
+            $header = "Location: index.php?a=".$a."&r=2&stay=".$_POST['stay'];
             header($header);
         } else {
             $header = "Location: index.php?a=76&r=2";
@@ -82,7 +82,7 @@ 
         ));
 
         // disallow duplicate names for chunks
-        if (EvolutionCMS\Models\SiteHtmlsnippet::where('id','!=',$id)->where('name','=',$name)->first()) {
+        if (EvolutionCMS\Models\SiteHtmlsnippet::where('id', '!=', $id)->where('name', '=', $name)->first()) {
             $modx->getManagerApi()->saveFormValues(78);
             $modx->webAlertAndQuit(sprintf($_lang['duplicate_name_found_general'], $_lang['chunk'], $name), "index.php?a=78&id={$id}");
         }
@@ -90,7 +90,7 @@ 
         //do stuff to save the edited doc
         $chunk = EvolutionCMS\Models\SiteHtmlsnippet::find($id);
 
-        $chunk->update(compact('name', 'description','snippet','locked','category','editor_type','editor_name','disabled','editedon'));
+        $chunk->update(compact('name', 'description', 'snippet', 'locked', 'category', 'editor_type', 'editor_name', 'disabled', 'editedon'));
 
         // invoke OnChunkFormSave event
         $modx->invokeEvent("OnChunkFormSave", array(
@@ -107,7 +107,7 @@ 
         // finished emptying cache - redirect
         if ($_POST['stay'] != '') {
             $a = ($_POST['stay'] == '2') ? "78&id=$id" : "77";
-            $header = "Location: index.php?a=" . $a . "&r=2&stay=" . $_POST['stay'];
+            $header = "Location: index.php?a=".$a."&r=2&stay=".$_POST['stay'];
             header($header);
         } else {
             $modx->unlockElement(3, $id);

manager/processors/save_user.processor.php

Spacing +18 added lines, -18 removed lines

@@ -8,7 +8,7 @@ 
 

 $input = $_POST;

 

-$id = (int)$input['id'];

+$id = (int) $input['id'];

 $oldusername = $input['oldusername'];

 $newusername = !empty ($input['newusername']) ? trim($input['newusername']) : "New User";

 $fullname = $input['fullname'];

@@ -96,9 +96,9 @@ 
 		$field = array();

 		$field['password'] = $modx->getPasswordHash()->HashPassword($newpassword);

 		$field['username'] = $newusername;

-		$managerUser= EvolutionCMS\Models\ManagerUser::create($field);

+		$managerUser = EvolutionCMS\Models\ManagerUser::create($field);

 		$internalKey = $managerUser->getKey();

-		$field = compact( 'fullname', 'role', 'email', 'phone', 'mobilephone', 'fax', 'zip', 'street', 'city', 'state', 'country', 'gender', 'dob', 'photo', 'comment', 'blocked', 'blockeduntil', 'blockedafter');

+		$field = compact('fullname', 'role', 'email', 'phone', 'mobilephone', 'fax', 'zip', 'street', 'city', 'state', 'country', 'gender', 'dob', 'photo', 'comment', 'blocked', 'blockeduntil', 'blockedafter');

 		$managerUser->attributes()->create($field);

 

         $field = compact('internalKey', 'fullname', 'role', 'email', 'phone', 'mobilephone', 'fax', 'zip', 'street',

@@ -132,11 +132,11 @@ 
 		/*******************************************************************************/

 		// put the user in the user_groups he/ she should be in

 		// first, check that up_perms are switched on!

-		if($modx->getConfig('use_udperms') == 1) {

-			if(!empty($user_groups)) {

-				for($i = 0; $i < count($user_groups); $i++) {

+		if ($modx->getConfig('use_udperms') == 1) {

+			if (!empty($user_groups)) {

+				for ($i = 0; $i < count($user_groups); $i++) {

 					$field = array();

-					$field['user_group'] = (int)$user_groups[$i];

+					$field['user_group'] = (int) $user_groups[$i];

 					$field['member'] = $id;

 					$managerUser->memberGroups()->create($field);

 				}

@@ -148,7 +148,7 @@ 
             sendMailMessageForUser($email, $newusername, $newpassword, $fullname, $signupemail_message, MODX_MANAGER_URL);

             if ($input['stay'] != '') {

 				$a = ($input['stay'] == '2') ? "12&id={$internalKey}" : "11";

-				$header = "Location: index.php?a={$a}&r=2&stay=" . $input['stay'];

+				$header = "Location: index.php?a={$a}&r=2&stay=".$input['stay'];

 				header($header);

 			} else {

 				$header = "Location: index.php?a=75&r=2";

@@ -157,12 +157,12 @@ 
 		} else {

             if ($input['stay'] != '') {

 				$a = ($input['stay'] == '2') ? "12&id={$internalKey}" : "11";

-				$stayUrl = "index.php?a={$a}&r=2&stay=" . $input['stay'];

+				$stayUrl = "index.php?a={$a}&r=2&stay=".$input['stay'];

 			} else {

 				$stayUrl = "index.php?a=75&r=2";

 			}

 

-			include_once MODX_MANAGER_PATH . "includes/header.inc.php";

+			include_once MODX_MANAGER_PATH."includes/header.inc.php";

 			?>

 

 			<h1><?php echo $_lang['user_title']; ?></h1>

@@ -185,7 +185,7 @@ 
 			</div>

 			<?php

 

-			include_once MODX_MANAGER_PATH . "includes/footer.inc.php";

+			include_once MODX_MANAGER_PATH."includes/footer.inc.php";

 		}

 		break;

 	case '12' : // edit user

@@ -275,13 +275,13 @@ 
 		/*******************************************************************************/

 		// put the user in the user_groups he/ she should be in

 		// first, check that up_perms are switched on!

-		if($modx->getConfig('use_udperms') == 1) {

+		if ($modx->getConfig('use_udperms') == 1) {

 			// as this is an existing user, delete his/ her entries in the groups before saving the new groups

 			$managerUser->memberGroups()->delete();

             if (!empty($user_groups)) {

                 for ($i = 0; $i < count($user_groups); $i++) {

 					$field = array();

-					$field['user_group'] = (int)$user_groups[$i];

+					$field['user_group'] = (int) $user_groups[$i];

 					$field['member'] = $id;

 					$managerUser->memberGroups()->create($field);

 				}

@@ -295,12 +295,12 @@ 
         if ($genpassword == 1 && $passwordnotifymethod == 's') {

             if ($input['stay'] != '') {

 				$a = ($input['stay'] == '2') ? "12&id={$id}" : "11";

-				$stayUrl = "index.php?a={$a}&r=2&stay=" . $input['stay'];

+				$stayUrl = "index.php?a={$a}&r=2&stay=".$input['stay'];

 			} else {

 				$stayUrl = "index.php?a=75&r=2";

 			}

 

-			include_once MODX_MANAGER_PATH . "includes/header.inc.php";

+			include_once MODX_MANAGER_PATH."includes/header.inc.php";

 			?>

 

 			<h1><?php echo $_lang['user_title']; ?></h1>

@@ -318,17 +318,17 @@ 
                 <div class="tab-page">

                     <div class="container container-body" id="disp">

                         <p><?php echo sprintf($_lang["password_msg"], $modx->getPhpCompat()->htmlspecialchars($newusername),

-                                    $modx->getPhpCompat()->htmlspecialchars($newpassword)) . (($id == $modx->getLoginUserID()) ? ' ' . $_lang['user_changeddata'] : ''); ?></p>

+                                    $modx->getPhpCompat()->htmlspecialchars($newpassword)).(($id == $modx->getLoginUserID()) ? ' '.$_lang['user_changeddata'] : ''); ?></p>

 					</div>

 				</div>

 			</div>

 			<?php

 

-			include_once MODX_MANAGER_PATH . "includes/footer.inc.php";

+			include_once MODX_MANAGER_PATH."includes/footer.inc.php";

 		} else {

             if ($input['stay'] != '') {

 				$a = ($input['stay'] == '2') ? "12&id={$id}" : "11";

-				$header = "Location: index.php?a={$a}&r=2&stay=" . $input['stay'];

+				$header = "Location: index.php?a={$a}&r=2&stay=".$input['stay'];

 				header($header);

 			} else {

 				$header = "Location: index.php?a=75&r=2";

manager/processors/execute_module.processor.php

Spacing +10 added lines, -10 removed lines

@@ -1,18 +1,18 @@ 
 <?php
-if( ! defined('IN_MANAGER_MODE') || IN_MANAGER_MODE !== true) {
+if (!defined('IN_MANAGER_MODE') || IN_MANAGER_MODE !== true) {
     die("<b>INCLUDE_ORDERING_ERROR</b><br /><br />Please use the EVO Content Manager instead of accessing this file directly.");
 }
-if(!$modx->hasPermission('exec_module')) {
+if (!$modx->hasPermission('exec_module')) {
 	$modx->webAlertAndQuit($_lang["error_no_privileges"]);
 }
 
-$id = isset($_GET['id'])? (int)$_GET['id'] : 0;
-if($id==0) {
+$id = isset($_GET['id']) ? (int) $_GET['id'] : 0;
+if ($id == 0) {
 	$modx->webAlertAndQuit($_lang["error_no_id"]);
 }
 
 // check if user has access permission, except admins
-if($_SESSION['mgrRole']!=1){
+if ($_SESSION['mgrRole'] != 1) {
 	$rs = $modx->getDatabase()->select(
 		'sma.usergroup,mg.member',
 		$modx->getDatabase()->getFullTableName("site_module_access")." sma
@@ -24,18 +24,18 @@ 
 	$permissionAccessInt = -1;
 
 	while ($row = $modx->getDatabase()->getRow($rs)) {
-		if($row["usergroup"] && $row["member"]) {
+		if ($row["usergroup"] && $row["member"]) {
 			//if there are permissions and this member has permission, ofcourse
 			//this is granted
 			$permissionAccessInt = 1;
-		} elseif ($permissionAccessInt==-1) {
+		} elseif ($permissionAccessInt == -1) {
 			//if there are permissions but this member has no permission and the
 			//variable was still in init state we set permission to 0; no permissions
 			$permissionAccessInt = 0;
 		}
 	}
 
-	if($permissionAccessInt==0) {
+	if ($permissionAccessInt == 0) {
 		$modx->webAlertAndQuit("You do not sufficient privileges to execute this module.", "index.php?a=106");
 	}
 }
@@ -43,10 +43,10 @@ 
 // get module data
 $rs = $modx->getDatabase()->select('*', $modx->getDatabase()->getFullTableName("site_modules"), "id='{$id}'");
 $content = $modx->getDatabase()->getRow($rs);
-if(!$content) {
+if (!$content) {
 	$modx->webAlertAndQuit("No record found for id {$id}.", "index.php?a=106");
 }
-if($content['disabled']) {
+if ($content['disabled']) {
 	$modx->webAlertAndQuit("This module is disabled and cannot be executed.", "index.php?a=106");
 }
 

manager/processors/save_settings.processor.php

Spacing +39 added lines, -39 removed lines

@@ -1,46 +1,46 @@ 
 <?php

-if( ! defined('IN_MANAGER_MODE') || IN_MANAGER_MODE !== true) {

+if (!defined('IN_MANAGER_MODE') || IN_MANAGER_MODE !== true) {

     die("<b>INCLUDE_ORDERING_ERROR</b><br /><br />Please use the EVO Content Manager instead of accessing this file directly.");

 }

-if(!$modx->hasPermission('settings')) {

+if (!$modx->hasPermission('settings')) {

 	$modx->webAlertAndQuit($_lang["error_no_privileges"]);

 }

 $data = $_POST;

 // lose the POST now, gets rid of quirky issue with Safari 3 - see FS#972

 unset($_POST);

 

-if($data['friendly_urls']==='1' && strpos($_SERVER['SERVER_SOFTWARE'],'IIS')===false)

+if ($data['friendly_urls'] === '1' && strpos($_SERVER['SERVER_SOFTWARE'], 'IIS') === false)

 {

-	$htaccess        = MODX_BASE_PATH . '.htaccess';

-	$sample_htaccess = MODX_BASE_PATH . 'ht.access';

-	$dir = '/' . trim(MODX_BASE_URL,'/');

-	if(is_file($htaccess))

+	$htaccess        = MODX_BASE_PATH.'.htaccess';

+	$sample_htaccess = MODX_BASE_PATH.'ht.access';

+	$dir = '/'.trim(MODX_BASE_URL, '/');

+	if (is_file($htaccess))

 	{

 		$_ = file_get_contents($htaccess);

-		if(strpos($_,'RewriteBase')===false)

+		if (strpos($_, 'RewriteBase') === false)

 		{

 			$warnings[] = $_lang["settings_friendlyurls_alert2"];

 		}

-		elseif(is_writable($htaccess))

+		elseif (is_writable($htaccess))

 		{

-			$_ = preg_replace('@RewriteBase.+@',"RewriteBase {$dir}", $_);

-			if(!@file_put_contents($htaccess,$_))

+			$_ = preg_replace('@RewriteBase.+@', "RewriteBase {$dir}", $_);

+			if (!@file_put_contents($htaccess, $_))

 			{

 				$warnings[] = $_lang["settings_friendlyurls_alert2"];

 			}

 		}

 	}

-	elseif(is_file($sample_htaccess))

+	elseif (is_file($sample_htaccess))

 	{

-		if(!@rename($sample_htaccess,$htaccess))

+		if (!@rename($sample_htaccess, $htaccess))

         {

         	$warnings[] = $_lang["settings_friendlyurls_alert"];

 		}

-		elseif(MODX_BASE_URL!=='/')

+		elseif (MODX_BASE_URL !== '/')

 		{

 			$_ = file_get_contents($htaccess);

-			$_ = preg_replace('@RewriteBase.+@',"RewriteBase {$dir}", $_);

-			if(!@file_put_contents($htaccess,$_))

+			$_ = preg_replace('@RewriteBase.+@', "RewriteBase {$dir}", $_);

+			if (!@file_put_contents($htaccess, $_))

 			{

 				$warnings[] = $_lang["settings_friendlyurls_alert2"];

 			}

@@ -48,17 +48,17 @@ 
 	}

 }

 

-if (file_exists(MODX_MANAGER_PATH . 'media/style/' . $modx->config['manager_theme'] . '/css/styles.min.css')) {

-    unlink(MODX_MANAGER_PATH . 'media/style/' . $modx->config['manager_theme'] . '/css/styles.min.css');

+if (file_exists(MODX_MANAGER_PATH.'media/style/'.$modx->config['manager_theme'].'/css/styles.min.css')) {

+    unlink(MODX_MANAGER_PATH.'media/style/'.$modx->config['manager_theme'].'/css/styles.min.css');

 }

 

-$data['filemanager_path'] = str_replace('[(base_path)]',MODX_BASE_PATH,$data['filemanager_path']);

-$data['rb_base_dir']      = str_replace('[(base_path)]',MODX_BASE_PATH,$data['rb_base_dir']);

+$data['filemanager_path'] = str_replace('[(base_path)]', MODX_BASE_PATH, $data['filemanager_path']);

+$data['rb_base_dir']      = str_replace('[(base_path)]', MODX_BASE_PATH, $data['rb_base_dir']);

 

 if (isset($data) && count($data) > 0) {

-	if(isset($data['manager_language'])) {

-		$lang_path = MODX_MANAGER_PATH . 'includes/lang/' . $data['manager_language'] . '.inc.php';

-		if(is_file($lang_path)) {

+	if (isset($data['manager_language'])) {

+		$lang_path = MODX_MANAGER_PATH.'includes/lang/'.$data['manager_language'].'.inc.php';

+		if (is_file($lang_path)) {

 			include $lang_path;

             global $modx_lang_attribute;

             $data['lang_code'] = !$modx_lang_attribute ? 'en' : $modx_lang_attribute;

@@ -66,12 +66,12 @@ 
 	}

 	$savethese = array();

 	$data['sys_files_checksum'] = $modx->getManagerApi()->getSystemChecksum($data['check_files_onlogin']);

-	$data['mail_check_timeperiod'] = (int)$data['mail_check_timeperiod'] < 60 ? 60 : $data['mail_check_timeperiod']; // updateMail() in mainMenu no faster than every minute

+	$data['mail_check_timeperiod'] = (int) $data['mail_check_timeperiod'] < 60 ? 60 : $data['mail_check_timeperiod']; // updateMail() in mainMenu no faster than every minute

 	foreach ($data as $k => $v) {

 		switch ($k) {

             case 'settings_version':{

-                if($modx->getVersionData('version')!=$data['settings_version']){

-                    $modx->logEvent(17,2,'<pre>'.var_export($data['settings_version'],true).'</pre>','fake settings_version');

+                if ($modx->getVersionData('version') != $data['settings_version']) {

+                    $modx->logEvent(17, 2, '<pre>'.var_export($data['settings_version'], true).'</pre>', 'fake settings_version');

                     $v = $modx->getVersionData('version');

                 }

                 break;

@@ -92,28 +92,28 @@ 
 			case 'rb_base_url':

 			case 'filemanager_path':

 				$v = trim($v);

-				$v = rtrim($v,'/') . '/';

+				$v = rtrim($v, '/').'/';

 				break;

             case 'manager_language':

-                $langDir = realpath(MODX_MANAGER_PATH . 'includes/lang');

-                $langFile = realpath(MODX_MANAGER_PATH . 'includes/lang/' . $v . '.inc.php');

+                $langDir = realpath(MODX_MANAGER_PATH.'includes/lang');

+                $langFile = realpath(MODX_MANAGER_PATH.'includes/lang/'.$v.'.inc.php');

                 $langFileDir = dirname($langFile);

-                if($langDir !== $langFileDir || !file_exists($langFile)) {

+                if ($langDir !== $langFileDir || !file_exists($langFile)) {

                     $v = 'english';

                 }

 				break;

 			case 'smtppw':

 				if ($v !== '********************' && $v !== '') {

 					$v = trim($v);

-					$v = base64_encode($v) . substr(str_shuffle('abcdefghjkmnpqrstuvxyzABCDEFGHJKLMNPQRSTUVWXYZ23456789'), 0, 7);

-					$v = str_replace('=','%',$v);

+					$v = base64_encode($v).substr(str_shuffle('abcdefghjkmnpqrstuvxyzABCDEFGHJKLMNPQRSTUVWXYZ23456789'), 0, 7);

+					$v = str_replace('=', '%', $v);

 				} elseif ($v === '********************') {

 					$k = '';

 				}

 				break;

 			case 'session_timeout':

 				$mail_check_timeperiod = $data['mail_check_timeperiod'];

-				$v = (int)$v < ($data['mail_check_timeperiod']/60+1) ? ($data['mail_check_timeperiod']/60+1) : $v; // updateMail() in mainMenu pings as per mail_check_timeperiod, so +1min is minimum

+				$v = (int) $v < ($data['mail_check_timeperiod'] / 60 + 1) ? ($data['mail_check_timeperiod'] / 60 + 1) : $v; // updateMail() in mainMenu pings as per mail_check_timeperiod, so +1min is minimum

 				break;

 			default:

 			break;

@@ -122,7 +122,7 @@ 
 

 		$modx->config[$k] = $v;

 

-		if(!empty($k)) $savethese[] = '(\''.$modx->getDatabase()->escape($k).'\', \''.$modx->getDatabase()->escape($v).'\')';

+		if (!empty($k)) $savethese[] = '(\''.$modx->getDatabase()->escape($k).'\', \''.$modx->getDatabase()->escape($v).'\')';

 	}

 

 	// Run a single query to save all the values

@@ -132,16 +132,16 @@ 
 

 	// Reset Template Pages

 	if (isset($data['reset_template'])) {

-		$newtemplate = (int)$data['default_template'];

-		$oldtemplate = (int)$data['old_template'];

+		$newtemplate = (int) $data['default_template'];

+		$oldtemplate = (int) $data['old_template'];

 		$tbl = $modx->getDatabase()->getFullTableName('site_content');

 		$reset = $data['reset_template'];

-		if($reset==1) $modx->getDatabase()->update(array('template' => $newtemplate), $tbl, "type='document'");

-		else if($reset==2) $modx->getDatabase()->update(array('template' => $newtemplate), $tbl, "template='{$oldtemplate}'");

+		if ($reset == 1) $modx->getDatabase()->update(array('template' => $newtemplate), $tbl, "type='document'");

+		else if ($reset == 2) $modx->getDatabase()->update(array('template' => $newtemplate), $tbl, "template='{$oldtemplate}'");

 	}

 

 	// empty cache

 	$modx->clearCache('full');

 }

-$header="Location: index.php?a=7&r=10";

+$header = "Location: index.php?a=7&r=10";

 header($header);

manager/processors/save_tmplvars.processor.php

Spacing +10 added lines, -10 removed lines

@@ -1,12 +1,12 @@ 
 <?php
-if( ! defined('IN_MANAGER_MODE') || IN_MANAGER_MODE !== true) {
+if (!defined('IN_MANAGER_MODE') || IN_MANAGER_MODE !== true) {
     die("<b>INCLUDE_ORDERING_ERROR</b><br /><br />Please use the EVO Content Manager instead of accessing this file directly.");
 }
 if (!$modx->hasPermission('save_template')) {
     $modx->webAlertAndQuit($_lang["error_no_privileges"]);
 }
 
-$id = (int)$_POST['id'];
+$id = (int) $_POST['id'];
 $name = trim($_POST['name']);
 $description = $_POST['description'];
 $caption = $_POST['caption'];
@@ -17,17 +17,17 @@ 
 $display = $_POST['display'];
 $params = $_POST['params'];
 $locked = isset($_POST['locked']) && $_POST['locked'] == 'on' ? 1 : 0;
-$origin = isset($_REQUEST['or']) ? (int)$_REQUEST['or'] : 76;
-$originId = isset($_REQUEST['oid']) ? (int)$_REQUEST['oid'] : null;
+$origin = isset($_REQUEST['or']) ? (int) $_REQUEST['or'] : 76;
+$originId = isset($_REQUEST['oid']) ? (int) $_REQUEST['oid'] : null;
 $currentdate = time() + $modx->config['server_offset_time'];
 
 //Kyle Jaebker - added category support
 if (empty($_POST['newcategory']) && $_POST['categoryid'] > 0) {
-    $categoryid = (int)$_POST['categoryid'];
+    $categoryid = (int) $_POST['categoryid'];
 } elseif (empty($_POST['newcategory']) && $_POST['categoryid'] <= 0) {
     $categoryid = 0;
 } else {
-    include_once(MODX_MANAGER_PATH . 'includes/categories.inc.php');
+    include_once(MODX_MANAGER_PATH.'includes/categories.inc.php');
     $categoryid = checkCategory($_POST['newcategory']);
     if (!$categoryid) {
         $categoryid = newCategory($_POST['newcategory']);
@@ -74,7 +74,7 @@ 
             'createdon' => $currentdate,
             'editedon' => $currentdate
         );
-        $tmplVar= EvolutionCMS\Models\SiteTmplvar::create($field);
+        $tmplVar = EvolutionCMS\Models\SiteTmplvar::create($field);
         $newid = $tmplVar->getKey();
 
         // save access permissions
@@ -96,7 +96,7 @@ 
         // finished emptying cache - redirect
         if ($_POST['stay'] != '') {
             $a = ($_POST['stay'] == '2') ? "301&id=$newid" : "300";
-            $header = "Location: index.php?a=" . $a . "&r=2&stay=" . $_POST['stay'];
+            $header = "Location: index.php?a=".$a."&r=2&stay=".$_POST['stay'];
             header($header);
         } else {
             $header = "Location: index.php?a=76&r=2";
@@ -158,11 +158,11 @@ 
         // finished emptying cache - redirect
         if ($_POST['stay'] != '') {
             $a = ($_POST['stay'] == '2') ? "301&id=$id" : "300";
-            $header = "Location: index.php?a=" . $a . "&r=2&stay=" . $_POST['stay'] . "&or=" . $origin . "&oid=" . $originId;
+            $header = "Location: index.php?a=".$a."&r=2&stay=".$_POST['stay']."&or=".$origin."&oid=".$originId;
             header($header);
         } else {
             $modx->unlockElement(2, $id);
-            $header = "Location: index.php?a=" . $origin . "&r=2" . (empty($originId) ? '' : '&id=' . $originId);
+            $header = "Location: index.php?a=".$origin."&r=2".(empty($originId) ? '' : '&id='.$originId);
             header($header);
         }