evolution-cms / evolution

manager/includes/functions/actions/help.php

Spacing +7 added lines, -7 removed lines

@@ -1,5 +1,5 @@ 
 <?php
-if(!function_exists('createList')) {
+if (!function_exists('createList')) {
     /**
      * @param string $sectionHeader
      * @param array $linkArr
@@ -7,8 +7,8 @@ 
      */
     function createList($sectionHeader, $linkArr)
     {
-        $output = '<div class="sectionHeader">' . $sectionHeader . '</div><div class="sectionBody">' . "\n";
-        $output .= '<table width="500"  border="0" cellspacing="0" cellpadding="0">' . "\n";
+        $output = '<div class="sectionHeader">'.$sectionHeader.'</div><div class="sectionBody">'."\n";
+        $output .= '<table width="500"  border="0" cellspacing="0" cellpadding="0">'."\n";
         $links = '';
         foreach ($linkArr as $row) {
             if (!is_array($row['link'])) {
@@ -16,16 +16,16 @@ 
             }
             foreach ($row['link'] as $link) {
                 $links .= $links != '' ? '<br/>' : '';
-                $links .= '<a href="' . $link . '" target="_blank">' . $link . '</a>';
+                $links .= '<a href="'.$link.'" target="_blank">'.$link.'</a>';
             }
             $output .= '
 		<tr>
-			<td align="left"><strong>' . $row["title"] . '</strong></td>
-			<td align="left">' . $links . '</td>
+			<td align="left"><strong>' . $row["title"].'</strong></td>
+			<td align="left">' . $links.'</td>
 		</tr>';
             $links = '';
         }
-        $output .= '</table></div>' . "\n";
+        $output .= '</table></div>'."\n";
 
         return $output;
     }

manager/includes/functions/actions/mutate_plugin.php

Spacing +2 added lines, -2 removed lines

@@ -16,8 +16,8 @@
     {
         echo '<div class="row form-row">
                 <div class="col-sm-6 col-md-4 col-lg-3">' .
-                    implode('</div><div class="col-sm-6 col-md-4 col-lg-3">', $evtnames) .
-                '</div>' .
+                    implode('</div><div class="col-sm-6 col-md-4 col-lg-3">', $evtnames).
+                '</div>'.
         '</div>';
 
         $evtnames = array();

manager/includes/functions/actions/mutate_content.php

Spacing +1 added lines, -1 removed lines

@@ -1,6 +1,6 @@
 <?php
 
-if(!function_exists('getDefaultTemplate')) {
+if (!function_exists('getDefaultTemplate')) {
     /**
      * @return string
      */

manager/includes/functions/actions/settings.php

Spacing +12 added lines, -12 removed lines

@@ -1,5 +1,5 @@ 
 <?php
-if(!function_exists('get_lang_keys')) {
+if (!function_exists('get_lang_keys')) {
     /**
      * get_lang_keys
      *
@@ -8,7 +8,7 @@ 
      */
     function get_lang_keys($filename)
     {
-        $file = MODX_MANAGER_PATH . 'includes/lang' . DIRECTORY_SEPARATOR . $filename;
+        $file = MODX_MANAGER_PATH.'includes/lang'.DIRECTORY_SEPARATOR.$filename;
         if (is_file($file) && is_readable($file)) {
             include($file);
             $out = isset($_lang) ? array_keys($_lang) : array();
@@ -20,7 +20,7 @@ 
     }
 }
 
-if(!function_exists('get_langs_by_key')) {
+if (!function_exists('get_langs_by_key')) {
     /**
      * get_langs_by_key
      *
@@ -41,7 +41,7 @@ 
     }
 }
 
-if(!function_exists('get_lang_options')) {
+if (!function_exists('get_lang_options')) {
     /**
      * get_lang_options
      *
@@ -58,11 +58,11 @@ 
         if (!empty($key)) {
             $languages = get_langs_by_key($key);
             sort($languages);
-            $lang_options .= '<option value="">' . $_lang['language_title'] . '</option>';
+            $lang_options .= '<option value="">'.$_lang['language_title'].'</option>';
 
             foreach ($languages as $language_name) {
                 $uclanguage_name = ucwords(str_replace("_", " ", $language_name));
-                $lang_options .= '<option value="' . $language_name . '">' . $uclanguage_name . '</option>';
+                $lang_options .= '<option value="'.$language_name.'">'.$uclanguage_name.'</option>';
             }
 
             return $lang_options;
@@ -72,7 +72,7 @@ 
             foreach ($languages as $language_name) {
                 $uclanguage_name = ucwords(str_replace("_", " ", $language_name));
                 $sel = $language_name === $selected_lang ? ' selected="selected"' : '';
-                $lang_options .= '<option value="' . $language_name . '" ' . $sel . '>' . $uclanguage_name . '</option>';
+                $lang_options .= '<option value="'.$language_name.'" '.$sel.'>'.$uclanguage_name.'</option>';
             }
 
             return $lang_options;
@@ -80,7 +80,7 @@ 
     }
 }
 
-if(!function_exists('form_radio')) {
+if (!function_exists('form_radio')) {
     /**
      * @param string $name
      * @param string $value
@@ -99,7 +99,7 @@ 
             $disabled = '';
         }
         if ($add) {
-            $add = ' ' . $add;
+            $add = ' '.$add;
         }
 
         return sprintf('<input onchange="documentDirty=true;" type="radio" name="%s" value="%s" %s %s %s />', $name,
@@ -108,7 +108,7 @@ 
     }
 }
 
-if(!function_exists('wrap_label')) {
+if (!function_exists('wrap_label')) {
     /**
      * @param string $str
      * @param string $object
@@ -121,7 +121,7 @@ 
 
 }
 
-if(!function_exists('parseText')) {
+if (!function_exists('parseText')) {
     /**
      * @param string $tpl
      * @param array $ph
@@ -142,7 +142,7 @@ 
     }
 }
 
-if(!function_exists('showHide')) {
+if (!function_exists('showHide')) {
     /**
      * @param bool $cond
      * @return string

manager/includes/functions/actions/search.php

Spacing +5 added lines, -5 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-if(!function_exists('highlightingCoincidence')) {
+if (!function_exists('highlightingCoincidence')) {
     /**
      * @param string $text
      * @param string $search
@@ -8,19 +8,19 @@ 
      */
     function highlightingCoincidence($text, $search)
     {
-        $regexp = '!(' . str_replace(array(
+        $regexp = '!('.str_replace(array(
                 '(',
                 ')'
             ), array(
                 '\(',
                 '\)'
-            ), trim($search)) . ')!isu';
+            ), trim($search)).')!isu';
 
         return preg_replace($regexp, '<span class="text-danger">$1</span>', $text);
     }
 }
 
-if(!function_exists('addClassForItemList')) {
+if (!function_exists('addClassForItemList')) {
     /**
      * @param string $locked
      * @param string $disabled
@@ -40,7 +40,7 @@ 
             $class .= ' deleted';
         }
         if ($class) {
-            $class = ' class="' . trim($class) . '"';
+            $class = ' class="'.trim($class).'"';
         }
 
         return $class;

manager/includes/functions/actions/logging.php

Spacing +2 added lines, -2 removed lines

@@ -1,5 +1,5 @@ 
 <?php
-if(!function_exists('array_unique_multi')) {
+if (!function_exists('array_unique_multi')) {
     /**
      * @param array $array
      * @param string $checkKey
@@ -27,7 +27,7 @@ 
     }
 }
 
-if(!function_exists('record_sort')) {
+if (!function_exists('record_sort')) {
     /**
      * @param array $array
      * @param string $key

manager/includes/functions/actions/bkmanager.php

Spacing +6 added lines, -6 removed lines

@@ -1,5 +1,5 @@ 
 <?php
-if(!function_exists('import_sql')) {
+if (!function_exists('import_sql')) {
     /**
      * @param string $source
      * @param string $result_code
@@ -40,7 +40,7 @@ 
     }
 }
 
-if(!function_exists('dumpSql')) {
+if (!function_exists('dumpSql')) {
     /**
      * @param string $dumpstring
      * @return bool
@@ -66,7 +66,7 @@ 
     }
 }
 
-if(!function_exists('snapshot')) {
+if (!function_exists('snapshot')) {
     /**
      * @param string $dumpstring
      * @return bool
@@ -80,7 +80,7 @@ 
     }
 }
 
-if(!function_exists('getSettings')) {
+if (!function_exists('getSettings')) {
     /**
      * @return array
      */
@@ -107,7 +107,7 @@ 
     }
 }
 
-if(!function_exists('restoreSettings')) {
+if (!function_exists('restoreSettings')) {
     /**
      * @param array $settings
      */
@@ -122,7 +122,7 @@ 
     }
 }
 
-if(!function_exists('parsePlaceholder')) {
+if (!function_exists('parsePlaceholder')) {
     /**
      * @param string $tpl
      * @param array $ph

manager/includes/functions/actions/mutate_role.php

Spacing +1 added lines, -1 removed lines

@@ -1,6 +1,6 @@
 <?php
 
-if(!function_exists('render_form')) {
+if (!function_exists('render_form')) {
     /**
      * @param string $name
      * @param string $label

manager/includes/functions/processors.php

Spacing +33 added lines, -33 removed lines

@@ -36,7 +36,7 @@ 
     }
 }
 
-if(!function_exists('duplicateDocument')) {
+if (!function_exists('duplicateDocument')) {
     /**
      * @param int $docid
      * @param null|int $parent
@@ -90,12 +90,12 @@ 
             $count = $modx->db->getRecordCount($modx->db->select('pagetitle', $modx->getFullTableName('site_content'),
                 "pagetitle LIKE '{$pagetitle} Duplicate%'"));
             if ($count >= 1) {
-                $count = ' ' . ($count + 1);
+                $count = ' '.($count + 1);
             } else {
                 $count = '';
             }
 
-            $content['pagetitle'] = $_lang['duplicated_el_suffix'] . $count . ' ' . $content['pagetitle'];
+            $content['pagetitle'] = $_lang['duplicated_el_suffix'].$count.' '.$content['pagetitle'];
             $content['alias'] = null;
         } elseif ($modx->config['friendly_urls'] == 0 || $modx->config['allow_duplicate_alias'] == 0) {
             $content['alias'] = null;
@@ -153,7 +153,7 @@ 
     }
 }
 
-if(!function_exists('duplicateTVs')) {
+if (!function_exists('duplicateTVs')) {
     /**
      * Duplicate Document TVs
      *
@@ -166,8 +166,8 @@ 
 
         $tbltvc = $modx->getFullTableName('site_tmplvar_contentvalues');
 
-        $newid = (int)$newid;
-        $oldid = (int)$oldid;
+        $newid = (int) $newid;
+        $oldid = (int) $oldid;
 
         $modx->db->insert(
             array('contentid' => '', 'tmplvarid' => '', 'value' => ''), $tbltvc, // Insert into
@@ -176,7 +176,7 @@ 
     }
 }
 
-if(!function_exists('duplicateAccess')) {
+if (!function_exists('duplicateAccess')) {
     /**
      * Duplicate Document Access Permissions
      *
@@ -189,8 +189,8 @@ 
 
         $tbldg = $modx->getFullTableName('document_groups');
 
-        $newid = (int)$newid;
-        $oldid = (int)$oldid;
+        $newid = (int) $newid;
+        $oldid = (int) $oldid;
 
         $modx->db->insert(
             array('document' => '', 'document_group' => ''), $tbldg, // Insert into
@@ -199,7 +199,7 @@ 
     }
 }
 
-if(!function_exists('evalModule')) {
+if (!function_exists('evalModule')) {
     /**
      * evalModule
      *
@@ -235,17 +235,17 @@ 
             }
             if ($modx->config['error_reporting'] === '99' || 2 < $error_level) {
                 $modx->messageQuit('PHP Parse Error', '', true, $error_info['type'], $error_info['file'],
-                    $_SESSION['itemname'] . ' - Module', $error_info['message'], $error_info['line'], $msg);
+                    $_SESSION['itemname'].' - Module', $error_info['message'], $error_info['line'], $msg);
                 $modx->event->alert("An error occurred while loading. Please see the event log for more information<p>{$msg}</p>");
             }
         }
         unset($modx->event->params);
 
-        return $mod . $msg;
+        return $mod.$msg;
     }
 }
 
-if(!function_exists('allChildren')) {
+if (!function_exists('allChildren')) {
     /**
      * @param int $currDocID
      * @return array
@@ -265,7 +265,7 @@ 
     }
 }
 
-if(!function_exists('jsAlert')) {
+if (!function_exists('jsAlert')) {
     /**
      * show javascript alert
      *
@@ -275,14 +275,14 @@ 
     {
         $modx = evolutionCMS();
         if ($_POST['ajax'] != 1) {
-            echo "<script>window.setTimeout(\"alert('" . addslashes($modx->db->escape($msg)) . "')\",10);history.go(-1)</script>";
+            echo "<script>window.setTimeout(\"alert('".addslashes($modx->db->escape($msg))."')\",10);history.go(-1)</script>";
         } else {
-            echo $msg . "\n";
+            echo $msg."\n";
         }
     }
 }
 
-if(!function_exists('login')) {
+if (!function_exists('login')) {
     /**
      * @param string $username
      * @param string $givenPassword
@@ -297,7 +297,7 @@ 
     }
 }
 
-if(!function_exists('loginV1')) {
+if (!function_exists('loginV1')) {
     /**
      * @param int $internalKey
      * @param string $givenPassword
@@ -330,7 +330,7 @@ 
     }
 }
 
-if(!function_exists('loginMD5')) {
+if (!function_exists('loginMD5')) {
     /**
      * @param int $internalKey
      * @param string $givenPassword
@@ -351,7 +351,7 @@ 
     }
 }
 
-if(!function_exists('updateNewHash')) {
+if (!function_exists('updateNewHash')) {
     /**
      * @param string $username
      * @param string $password
@@ -366,7 +366,7 @@ 
     }
 }
 
-if(!function_exists('incrementFailedLoginCount')) {
+if (!function_exists('incrementFailedLoginCount')) {
     /**
      * @param int $internalKey
      * @param int $failedlogins
@@ -389,7 +389,7 @@ 
 
         if ($failedlogins < $failed_allowed) {
             //sleep to help prevent brute force attacks
-            $sleep = (int)$failedlogins / 2;
+            $sleep = (int) $failedlogins / 2;
             if ($sleep > 5) {
                 $sleep = 5;
             }
@@ -402,7 +402,7 @@ 
     }
 }
 
-if (! function_exists('rmdirRecursive')) {
+if (!function_exists('rmdirRecursive')) {
     /**
      * rmdirRecursive - detects symbollic links on unix
      *
@@ -426,7 +426,7 @@ 
     }
 }
 
-if(!function_exists('saveUserGroupAccessPermissons')) {
+if (!function_exists('saveUserGroupAccessPermissons')) {
     /**
      * saves module user group access
      */
@@ -457,7 +457,7 @@ 
     }
 }
 
-if(!function_exists('saveEventListeners')) {
+if (!function_exists('saveEventListeners')) {
 # Save Plugin Event Listeners
     function saveEventListeners($id, $sysevents, $mode)
     {
@@ -510,7 +510,7 @@ 
     }
 }
 
-if(!function_exists('getEventIdByName')) {
+if (!function_exists('getEventIdByName')) {
     /**
      * @param string $name
      * @return string|int
@@ -533,7 +533,7 @@ 
     }
 }
 
-if(!function_exists('saveTemplateAccess')) {
+if (!function_exists('saveTemplateAccess')) {
     /**
      * @param int $id
      */
@@ -572,7 +572,7 @@ 
     }
 }
 
-if(!function_exists('saveTemplateVarAccess')) {
+if (!function_exists('saveTemplateVarAccess')) {
     /**
      * @return void
      */
@@ -611,7 +611,7 @@ 
     }
 }
 
-if(!function_exists('saveDocumentAccessPermissons')) {
+if (!function_exists('saveDocumentAccessPermissons')) {
     function saveDocumentAccessPermissons()
     {
         global $id, $newid;
@@ -780,10 +780,10 @@ 
         }
 
         foreach ($defaults as $k) {
-            if (isset($settings['default_' . $k]) && $settings['default_' . $k] == '1') {
+            if (isset($settings['default_'.$k]) && $settings['default_'.$k] == '1') {
                 unset($settings[$k]);
             }
-            unset($settings['default_' . $k]);
+            unset($settings['default_'.$k]);
         }
 
         $modx->db->delete($tbl_user_settings, "user='{$id}'");
@@ -815,7 +815,7 @@ 
         global $id, $modx;
         $mode = $_POST['mode'];
         $modx->manager->saveFormValues($mode);
-        $modx->webAlertAndQuit($msg, "index.php?a={$mode}" . ($mode === $action ? "&id={$id}" : ''));
+        $modx->webAlertAndQuit($msg, "index.php?a={$mode}".($mode === $action ? "&id={$id}" : ''));
     }
 }
 
@@ -831,7 +831,7 @@ 
         global $deltime;
 
         $rs = $modx->db->select('id', $modx->getFullTableName('site_content'),
-            "parent='" . (int)$parent . "' AND deleted=1 AND deletedon='" . (int)$deltime . "'");
+            "parent='".(int) $parent."' AND deleted=1 AND deletedon='".(int) $deltime."'");
         // the document has children documents, we'll need to delete those too
         while ($row = $modx->db->getRow($rs)) {
             $children[] = $row['id'];

Braces +3 added lines, -1 removed lines

@@ -403,10 +403,12 @@
         $failedlogins += 1;
 
         $fields = array('failedlogincount' => $failedlogins);
-        if ($failedlogins >= $failed_allowed) //block user for too many fail attempts
+        if ($failedlogins >= $failed_allowed) {
+            //block user for too many fail attempts
         {
             $fields['blockeduntil'] = time() + ($blocked_minutes * 60);
         }
+        }
 
         $modx->db->update($fields, '[+prefix+]user_attributes', "internalKey='{$internalKey}'");