| @@ -1,22 +1,22 @@ discard block | ||
| 1 | 1 | <?php | 
| 2 | -if( ! defined('IN_MANAGER_MODE') || IN_MANAGER_MODE !== true) {
 | |
| 2 | +if( ! defined('IN_MANAGER_MODE') || IN_MANAGER_MODE !== true) { | |
| 3 | 3 |      die("<b>INCLUDE_ORDERING_ERROR</b><br /><br />Please use the EVO Content Manager instead of accessing this file directly.");
 | 
| 4 | 4 | } | 
| 5 | -if (!$modx->hasPermission('logs')) {
 | |
| 5 | +if (!$modx->hasPermission('logs')) { | |
| 6 | 6 | $modx->webAlertAndQuit($_lang["error_no_privileges"]); | 
| 7 | 7 | } | 
| 8 | 8 | |
| 9 | -function array_unique_multi($array, $checkKey) | |
| 10 | -{
 | |
| 9 | +function array_unique_multi($array, $checkKey) | |
| 10 | +{ | |
| 11 | 11 | // Use the builtin if we're not a multi-dimensional array | 
| 12 | -    if (!is_array(current($array)) || empty($checkKey)) {
 | |
| 12 | +    if (!is_array(current($array)) || empty($checkKey)) { | |
| 13 | 13 | return array_unique($array); | 
| 14 | 14 | } | 
| 15 | 15 | |
| 16 | 16 | $ret = array(); | 
| 17 | 17 | $checkValues = array(); // contains the unique key Values | 
| 18 | -    foreach ($array as $key => $current) {
 | |
| 19 | -        if (in_array($current[$checkKey], $checkValues)) {
 | |
| 18 | +    foreach ($array as $key => $current) { | |
| 19 | +        if (in_array($current[$checkKey], $checkValues)) { | |
| 20 | 20 | continue; | 
| 21 | 21 | } // duplicate | 
| 22 | 22 | |
| @@ -26,17 +26,17 @@ discard block | ||
| 26 | 26 | return $ret; | 
| 27 | 27 | } | 
| 28 | 28 | |
| 29 | -function record_sort($array, $key) | |
| 30 | -{
 | |
| 29 | +function record_sort($array, $key) | |
| 30 | +{ | |
| 31 | 31 | $hash = array(); | 
| 32 | -    foreach ($array as $k => $v) {
 | |
| 32 | +    foreach ($array as $k => $v) { | |
| 33 | 33 | $hash[$k] = $v[$key]; | 
| 34 | 34 | } | 
| 35 | 35 | |
| 36 | 36 | natsort($hash); | 
| 37 | 37 | |
| 38 | 38 | $records = array(); | 
| 39 | -    foreach ($hash as $k => $row) {
 | |
| 39 | +    foreach ($hash as $k => $row) { | |
| 40 | 40 | $records[$k] = $array[$k]; | 
| 41 | 41 | } | 
| 42 | 42 | |
| @@ -63,7 +63,7 @@ discard block | ||
| 63 | 63 | <?php | 
| 64 | 64 | // get all users currently in the log | 
| 65 | 65 | $logs_user = record_sort(array_unique_multi($logs, 'internalKey'), 'username'); | 
| 66 | -                            foreach ($logs_user as $row) {
 | |
| 66 | +                            foreach ($logs_user as $row) { | |
| 67 | 67 | $selectedtext = $row['internalKey'] == $_REQUEST['searchuser'] ? ' selected="selected"' : ''; | 
| 68 | 68 | echo "\t\t" . '<option value="' . $row['internalKey'] . '"' . $selectedtext . '>' . $row['username'] . "</option>\n"; | 
| 69 | 69 | } | 
| @@ -80,9 +80,9 @@ discard block | ||
| 80 | 80 | // get all available actions in the log | 
| 81 | 81 | include_once "actionlist.inc.php"; | 
| 82 | 82 | $logs_actions = record_sort(array_unique_multi($logs, 'action'), 'action'); | 
| 83 | -                            foreach ($logs_actions as $row) {
 | |
| 83 | +                            foreach ($logs_actions as $row) { | |
| 84 | 84 | $action = getAction($row['action']); | 
| 85 | -                                if ($action == 'Idle') {
 | |
| 85 | +                                if ($action == 'Idle') { | |
| 86 | 86 | continue; | 
| 87 | 87 | } | 
| 88 | 88 | $selectedtext = $row['action'] == $_REQUEST['action'] ? ' selected="selected"' : ''; | 
| @@ -100,7 +100,7 @@ discard block | ||
| 100 | 100 | <?php | 
| 101 | 101 | // get all itemid currently in logging | 
| 102 | 102 | $logs_items = record_sort(array_unique_multi($logs, 'itemid'), 'itemid'); | 
| 103 | -                            foreach ($logs_items as $row) {
 | |
| 103 | +                            foreach ($logs_items as $row) { | |
| 104 | 104 | $selectedtext = $row['itemid'] == $_REQUEST['itemid'] ? ' selected="selected"' : ''; | 
| 105 | 105 | echo "\t\t" . '<option value="' . $row['itemid'] . '"' . $selectedtext . '>' . $row['itemid'] . "</option>\n"; | 
| 106 | 106 | } | 
| @@ -116,7 +116,7 @@ discard block | ||
| 116 | 116 | <?php | 
| 117 | 117 | // get all itemname currently in logging | 
| 118 | 118 | $logs_names = record_sort(array_unique_multi($logs, 'itemname'), 'itemname'); | 
| 119 | -                            foreach ($logs_names as $row) {
 | |
| 119 | +                            foreach ($logs_names as $row) { | |
| 120 | 120 | $selectedtext = $row['itemname'] == $_REQUEST['itemname'] ? ' selected="selected"' : ''; | 
| 121 | 121 | echo "\t\t" . '<option value="' . $row['itemname'] . '"' . $selectedtext . '>' . $row['itemname'] . "</option>\n"; | 
| 122 | 122 | } | 
| @@ -172,36 +172,36 @@ discard block | ||
| 172 | 172 | <div class="container container-body"> | 
| 173 | 173 | |
| 174 | 174 | <?php | 
| 175 | -if (isset($_REQUEST['log_submit'])) {
 | |
| 175 | +if (isset($_REQUEST['log_submit'])) { | |
| 176 | 176 | // get the selections the user made. | 
| 177 | 177 | $sqladd = array(); | 
| 178 | -    if ($_REQUEST['searchuser'] != 0) {
 | |
| 178 | +    if ($_REQUEST['searchuser'] != 0) { | |
| 179 | 179 | $sqladd[] = "internalKey='" . (int)$_REQUEST['searchuser'] . "'"; | 
| 180 | 180 | } | 
| 181 | -    if ($_REQUEST['action'] != 0) {
 | |
| 181 | +    if ($_REQUEST['action'] != 0) { | |
| 182 | 182 | $sqladd[] = "action=" . (int)$_REQUEST['action']; | 
| 183 | 183 | } | 
| 184 | -    if ($_REQUEST['itemid'] != 0 || $_REQUEST['itemid'] == "-") {
 | |
| 184 | +    if ($_REQUEST['itemid'] != 0 || $_REQUEST['itemid'] == "-") { | |
| 185 | 185 | $sqladd[] = "itemid='" . $_REQUEST['itemid'] . "'"; | 
| 186 | 186 | } | 
| 187 | -    if ($_REQUEST['itemname'] != '0') {
 | |
| 187 | +    if ($_REQUEST['itemname'] != '0') { | |
| 188 | 188 | $sqladd[] = "itemname='" . $modx->db->escape($_REQUEST['itemname']) . "'"; | 
| 189 | 189 | } | 
| 190 | -    if ($_REQUEST['message'] != "") {
 | |
| 190 | +    if ($_REQUEST['message'] != "") { | |
| 191 | 191 | $sqladd[] = "message LIKE '%" . $modx->db->escape($_REQUEST['message']) . "%'"; | 
| 192 | 192 | } | 
| 193 | 193 | // date stuff | 
| 194 | -    if ($_REQUEST['datefrom'] != "") {
 | |
| 194 | +    if ($_REQUEST['datefrom'] != "") { | |
| 195 | 195 | $sqladd[] = "timestamp>" . $modx->toTimeStamp($_REQUEST['datefrom']); | 
| 196 | 196 | } | 
| 197 | -    if ($_REQUEST['dateto'] != "") {
 | |
| 197 | +    if ($_REQUEST['dateto'] != "") { | |
| 198 | 198 | $sqladd[] = "timestamp<" . $modx->toTimeStamp($_REQUEST['dateto']); | 
| 199 | 199 | } | 
| 200 | 200 | |
| 201 | 201 | // If current position is not set, set it to zero | 
| 202 | -    if (!isset($_REQUEST['int_cur_position']) || $_REQUEST['int_cur_position'] == 0) {
 | |
| 202 | +    if (!isset($_REQUEST['int_cur_position']) || $_REQUEST['int_cur_position'] == 0) { | |
| 203 | 203 | $int_cur_position = 0; | 
| 204 | -    } else {
 | |
| 204 | +    } else { | |
| 205 | 205 | $int_cur_position = $_REQUEST['int_cur_position']; | 
| 206 | 206 | } | 
| 207 | 207 | |
| @@ -215,9 +215,9 @@ discard block | ||
| 215 | 215 | |
| 216 | 216 |      $rs = $modx->db->select('*', $modx->getFullTableName('manager_log'), (!empty($sqladd) ? implode(' AND ', $sqladd) : ''), 'timestamp DESC, id DESC', "{$int_cur_position}, {$int_num_result}");
 | 
| 217 | 217 | |
| 218 | -if ($limit < 1) {
 | |
| 218 | +if ($limit < 1) { | |
| 219 | 219 | echo '<p>' . $_lang["mgrlog_emptysrch"] . '</p>'; | 
| 220 | -} else {
 | |
| 220 | +} else { | |
| 221 | 221 | echo '<p>' . $_lang["mgrlog_sortinst"] . '</p>'; | 
| 222 | 222 | |
| 223 | 223 | include_once "paginate.inc.php"; | 
| @@ -236,14 +236,14 @@ discard block | ||
| 236 | 236 | $paging = $array_paging['first_link'] . $_lang["paging_first"] . (isset($array_paging['first_link']) ? "</a> " : " "); | 
| 237 | 237 | $paging .= $array_paging['previous_link'] . $_lang["paging_prev"] . (isset($array_paging['previous_link']) ? "</a> " : " "); | 
| 238 | 238 | $pagesfound = sizeof($array_row_paging); | 
| 239 | -    if ($pagesfound > 6) {
 | |
| 239 | +    if ($pagesfound > 6) { | |
| 240 | 240 | $paging .= $array_row_paging[$current_row - 2]; // ." "; | 
| 241 | 241 | $paging .= $array_row_paging[$current_row - 1]; // ." "; | 
| 242 | 242 | $paging .= $array_row_paging[$current_row]; // ." "; | 
| 243 | 243 | $paging .= $array_row_paging[$current_row + 1]; // ." "; | 
| 244 | 244 | $paging .= $array_row_paging[$current_row + 2]; // ." "; | 
| 245 | -    } else {
 | |
| 246 | -        for ($i = 0; $i < $pagesfound; $i++) {
 | |
| 245 | +    } else { | |
| 246 | +        for ($i = 0; $i < $pagesfound; $i++) { | |
| 247 | 247 | $paging .= $array_row_paging[$i] . " "; | 
| 248 | 248 | } | 
| 249 | 249 | } | 
| @@ -278,12 +278,12 @@ discard block | ||
| 278 | 278 | // grab the entire log file... | 
| 279 | 279 | $logentries = array(); | 
| 280 | 280 | $i = 0; | 
| 281 | -                while ($logentry = $modx->db->getRow($rs)) {
 | |
| 282 | -                    if (!preg_match("/^[0-9]+$/", $logentry['itemid'])) {
 | |
| 281 | +                while ($logentry = $modx->db->getRow($rs)) { | |
| 282 | +                    if (!preg_match("/^[0-9]+$/", $logentry['itemid'])) { | |
| 283 | 283 | $item = '<div style="text-align:center;">-</div>'; | 
| 284 | -                    } elseif ($logentry['action'] == 3 || $logentry['action'] == 27 || $logentry['action'] == 5) {
 | |
| 284 | +                    } elseif ($logentry['action'] == 3 || $logentry['action'] == 27 || $logentry['action'] == 5) { | |
| 285 | 285 | $item = '<a href="index.php?a=3&id=' . $logentry['itemid'] . '">' . $logentry['itemname'] . '</a>'; | 
| 286 | -                    } else {
 | |
| 286 | +                    } else { | |
| 287 | 287 | $item = $logentry['itemname']; | 
| 288 | 288 | } | 
| 289 | 289 | //index.php?a=13&searchuser=' . $logentry['internalKey'] . '&action=' . $logentry['action'] . '&itemname=' . $logentry['itemname'] . '&log_submit=true' | 
| @@ -318,6 +318,6 @@ discard block | ||
| 318 | 318 | // @see index.php @ 915 | 
| 319 | 319 | global $action; | 
| 320 | 320 | $action = 1; | 
| 321 | -} else {
 | |
| 321 | +} else { | |
| 322 | 322 | echo $_lang["mgrlog_noquery"]; | 
| 323 | 323 | } | 
| @@ -4,18 +4,18 @@ | ||
| 4 | 4 | } | 
| 5 | 5 | |
| 6 | 6 |  switch ($modx->manager->action) { | 
| 7 | - case 78: | |
| 8 | -        if (!$modx->hasPermission('edit_chunk')) { | |
| 9 | - $modx->webAlertAndQuit($_lang["error_no_privileges"]); | |
| 10 | - } | |
| 11 | - break; | |
| 12 | - case 77: | |
| 13 | -        if (!$modx->hasPermission('new_chunk')) { | |
| 7 | + case 78: | |
| 8 | +            if (!$modx->hasPermission('edit_chunk')) { | |
| 9 | + $modx->webAlertAndQuit($_lang["error_no_privileges"]); | |
| 10 | + } | |
| 11 | + break; | |
| 12 | + case 77: | |
| 13 | +            if (!$modx->hasPermission('new_chunk')) { | |
| 14 | + $modx->webAlertAndQuit($_lang["error_no_privileges"]); | |
| 15 | + } | |
| 16 | + break; | |
| 17 | + default: | |
| 14 | 18 | $modx->webAlertAndQuit($_lang["error_no_privileges"]); | 
| 15 | - } | |
| 16 | - break; | |
| 17 | - default: | |
| 18 | - $modx->webAlertAndQuit($_lang["error_no_privileges"]); | |
| 19 | 19 | } | 
| 20 | 20 | |
| 21 | 21 | $id = isset($_REQUEST['id']) ? (int)$_REQUEST['id'] : 0; | 
| @@ -4,18 +4,18 @@ | ||
| 4 | 4 | } | 
| 5 | 5 | |
| 6 | 6 |  switch($modx->manager->action) {
 | 
| 7 | - case 12: | |
| 8 | -		if(!$modx->hasPermission('edit_user')) {
 | |
| 9 | - $modx->webAlertAndQuit($_lang["error_no_privileges"]); | |
| 10 | - } | |
| 11 | - break; | |
| 12 | - case 11: | |
| 13 | -		if(!$modx->hasPermission('new_user')) {
 | |
| 14 | - $modx->webAlertAndQuit($_lang["error_no_privileges"]); | |
| 15 | - } | |
| 16 | - break; | |
| 17 | - default: | |
| 18 | - $modx->webAlertAndQuit($_lang["error_no_privileges"]); | |
| 7 | + case 12: | |
| 8 | +		    if(!$modx->hasPermission('edit_user')) {
 | |
| 9 | + $modx->webAlertAndQuit($_lang["error_no_privileges"]); | |
| 10 | + } | |
| 11 | + break; | |
| 12 | + case 11: | |
| 13 | +		    if(!$modx->hasPermission('new_user')) {
 | |
| 14 | + $modx->webAlertAndQuit($_lang["error_no_privileges"]); | |
| 15 | + } | |
| 16 | + break; | |
| 17 | + default: | |
| 18 | + $modx->webAlertAndQuit($_lang["error_no_privileges"]); | |
| 19 | 19 | } | 
| 20 | 20 | |
| 21 | 21 | $user = isset($_REQUEST['id']) ? (int)$_REQUEST['id'] : 0; | 
| @@ -4,18 +4,18 @@ | ||
| 4 | 4 | } | 
| 5 | 5 | |
| 6 | 6 |  switch($modx->manager->action) {
 | 
| 7 | - case 88: | |
| 8 | -		if(!$modx->hasPermission('edit_web_user')) {
 | |
| 9 | - $modx->webAlertAndQuit($_lang["error_no_privileges"]); | |
| 10 | - } | |
| 11 | - break; | |
| 12 | - case 87: | |
| 13 | -		if(!$modx->hasPermission('new_web_user')) {
 | |
| 14 | - $modx->webAlertAndQuit($_lang["error_no_privileges"]); | |
| 15 | - } | |
| 16 | - break; | |
| 17 | - default: | |
| 18 | - $modx->webAlertAndQuit($_lang["error_no_privileges"]); | |
| 7 | + case 88: | |
| 8 | +		    if(!$modx->hasPermission('edit_web_user')) {
 | |
| 9 | + $modx->webAlertAndQuit($_lang["error_no_privileges"]); | |
| 10 | + } | |
| 11 | + break; | |
| 12 | + case 87: | |
| 13 | +		    if(!$modx->hasPermission('new_web_user')) {
 | |
| 14 | + $modx->webAlertAndQuit($_lang["error_no_privileges"]); | |
| 15 | + } | |
| 16 | + break; | |
| 17 | + default: | |
| 18 | + $modx->webAlertAndQuit($_lang["error_no_privileges"]); | |
| 19 | 19 | } | 
| 20 | 20 | |
| 21 | 21 | $user = isset($_REQUEST['id']) ? (int)$_REQUEST['id'] : 0; | 
| @@ -4,18 +4,18 @@ | ||
| 4 | 4 | } | 
| 5 | 5 | |
| 6 | 6 |  switch((int) $modx->manager->action) { | 
| 7 | - case 35: | |
| 8 | -		if(!$modx->hasPermission('edit_role')) { | |
| 9 | - $modx->webAlertAndQuit($_lang["error_no_privileges"]); | |
| 10 | - } | |
| 11 | - break; | |
| 12 | - case 38: | |
| 13 | -		if(!$modx->hasPermission('new_role')) { | |
| 14 | - $modx->webAlertAndQuit($_lang["error_no_privileges"]); | |
| 15 | - } | |
| 16 | - break; | |
| 17 | - default: | |
| 18 | - $modx->webAlertAndQuit($_lang["error_no_privileges"]); | |
| 7 | + case 35: | |
| 8 | +		    if(!$modx->hasPermission('edit_role')) { | |
| 9 | + $modx->webAlertAndQuit($_lang["error_no_privileges"]); | |
| 10 | + } | |
| 11 | + break; | |
| 12 | + case 38: | |
| 13 | +		    if(!$modx->hasPermission('new_role')) { | |
| 14 | + $modx->webAlertAndQuit($_lang["error_no_privileges"]); | |
| 15 | + } | |
| 16 | + break; | |
| 17 | + default: | |
| 18 | + $modx->webAlertAndQuit($_lang["error_no_privileges"]); | |
| 19 | 19 | } | 
| 20 | 20 | |
| 21 | 21 | $role = isset($_REQUEST['id']) ? (int)$_REQUEST['id'] : 0; | 
| @@ -4,18 +4,18 @@ | ||
| 4 | 4 | } | 
| 5 | 5 | |
| 6 | 6 |  switch($modx->manager->action) { | 
| 7 | - case 16: | |
| 8 | -		if(!$modx->hasPermission('edit_template')) { | |
| 9 | - $modx->webAlertAndQuit($_lang["error_no_privileges"]); | |
| 10 | - } | |
| 11 | - break; | |
| 12 | - case 19: | |
| 13 | -		if(!$modx->hasPermission('new_template')) { | |
| 14 | - $modx->webAlertAndQuit($_lang["error_no_privileges"]); | |
| 15 | - } | |
| 16 | - break; | |
| 17 | - default: | |
| 18 | - $modx->webAlertAndQuit($_lang["error_no_privileges"]); | |
| 7 | + case 16: | |
| 8 | +		    if(!$modx->hasPermission('edit_template')) { | |
| 9 | + $modx->webAlertAndQuit($_lang["error_no_privileges"]); | |
| 10 | + } | |
| 11 | + break; | |
| 12 | + case 19: | |
| 13 | +		    if(!$modx->hasPermission('new_template')) { | |
| 14 | + $modx->webAlertAndQuit($_lang["error_no_privileges"]); | |
| 15 | + } | |
| 16 | + break; | |
| 17 | + default: | |
| 18 | + $modx->webAlertAndQuit($_lang["error_no_privileges"]); | |
| 19 | 19 | } | 
| 20 | 20 | |
| 21 | 21 | $id = isset($_REQUEST['id']) ? (int)$_REQUEST['id'] : 0; | 
| @@ -3,18 +3,18 @@ | ||
| 3 | 3 |  	die("<b>INCLUDE_ORDERING_ERROR</b><br /><br />Please use the EVO Content Manager instead of accessing this file directly.");
 | 
| 4 | 4 | } | 
| 5 | 5 |  switch($modx->manager->action) {
 | 
| 6 | - case 107: | |
| 7 | -		if(!$modx->hasPermission('new_module')) {
 | |
| 8 | - $modx->webAlertAndQuit($_lang["error_no_privileges"]); | |
| 9 | - } | |
| 10 | - break; | |
| 11 | - case 108: | |
| 12 | -		if(!$modx->hasPermission('edit_module')) {
 | |
| 13 | - $modx->webAlertAndQuit($_lang["error_no_privileges"]); | |
| 14 | - } | |
| 15 | - break; | |
| 16 | - default: | |
| 17 | - $modx->webAlertAndQuit($_lang["error_no_privileges"]); | |
| 6 | + case 107: | |
| 7 | +		    if(!$modx->hasPermission('new_module')) {
 | |
| 8 | + $modx->webAlertAndQuit($_lang["error_no_privileges"]); | |
| 9 | + } | |
| 10 | + break; | |
| 11 | + case 108: | |
| 12 | +		    if(!$modx->hasPermission('edit_module')) {
 | |
| 13 | + $modx->webAlertAndQuit($_lang["error_no_privileges"]); | |
| 14 | + } | |
| 15 | + break; | |
| 16 | + default: | |
| 17 | + $modx->webAlertAndQuit($_lang["error_no_privileges"]); | |
| 18 | 18 | } | 
| 19 | 19 | $id = isset($_REQUEST['id']) ? (int)$_REQUEST['id'] : 0; | 
| 20 | 20 | // Get table names (alphabetical) | 
| @@ -18,8 +18,7 @@ discard block | ||
| 18 | 18 | $this->db = & $modx->db; | 
| 19 | 19 |          $this->db_tbl['categories'] = $modx->getFullTableName('categories'); | 
| 20 | 20 | |
| 21 | - foreach( $this->elements as $element ) | |
| 22 | -        { | |
| 21 | +        foreach( $this->elements as $element ) { | |
| 23 | 22 |              $this->db_tbl[$element] = $modx->getFullTableName('site_' . $element ); | 
| 24 | 23 | } | 
| 25 | 24 | } | 
| @@ -69,8 +68,7 @@ discard block | ||
| 69 | 68 | |
| 70 | 69 | public function getAssignedElements( $category_id, $element ) | 
| 71 | 70 |      { | 
| 72 | - if( in_array( $element, $this->elements, true ) ) | |
| 73 | -        { | |
| 71 | +        if( in_array( $element, $this->elements, true ) ) { | |
| 74 | 72 | $elements = $this->db->makeArray( | 
| 75 | 73 | $this->db->select( | 
| 76 | 74 | '*', | 
| @@ -80,11 +78,9 @@ discard block | ||
| 80 | 78 | ); | 
| 81 | 79 | |
| 82 | 80 | // correct the name of templates | 
| 83 | - if( $element === 'templates' ) | |
| 84 | -            { | |
| 81 | +            if( $element === 'templates' ) { | |
| 85 | 82 | $_elements_count = count($elements); | 
| 86 | - for( $i=0; $i < $_elements_count; $i++ ) | |
| 87 | -                { | |
| 83 | +                for( $i=0; $i < $_elements_count; $i++ ) { | |
| 88 | 84 | $elements[$i]['name'] = $elements[$i]['templatename']; | 
| 89 | 85 | } | 
| 90 | 86 | } | 
| @@ -96,8 +92,7 @@ discard block | ||
| 96 | 92 | public function getAllAssignedElements( $category_id ) | 
| 97 | 93 |      { | 
| 98 | 94 | $elements = array(); | 
| 99 | - foreach( $this->elements as $element ) | |
| 100 | -        { | |
| 95 | +        foreach( $this->elements as $element ) { | |
| 101 | 96 | $elements[$element] = $this->getAssignedElements( $category_id, $element ); | 
| 102 | 97 | } | 
| 103 | 98 | return $elements; | 
| @@ -106,8 +101,7 @@ discard block | ||
| 106 | 101 | public function deleteCategory( $category_id ) | 
| 107 | 102 |      { | 
| 108 | 103 |          $_update = array('category' => 0); | 
| 109 | - foreach( $this->elements as $element ) | |
| 110 | -        { | |
| 104 | +        foreach( $this->elements as $element ) { | |
| 111 | 105 | $this->db->update( | 
| 112 | 106 | $_update, | 
| 113 | 107 | $this->db_tbl[$element], | 
| @@ -126,8 +120,7 @@ discard block | ||
| 126 | 120 | public function updateCategory( $category_id, $data = array() ) | 
| 127 | 121 |      { | 
| 128 | 122 | if( empty( $data ) | 
| 129 | - || empty( $category_id ) ) | |
| 130 | -        { | |
| 123 | +            || empty( $category_id ) ) { | |
| 131 | 124 | return false; | 
| 132 | 125 | } | 
| 133 | 126 | |
| @@ -142,8 +135,7 @@ discard block | ||
| 142 | 135 | "`id` = '" . (int)$category_id . "'" | 
| 143 | 136 | ); | 
| 144 | 137 | |
| 145 | - if( $this->db->getAffectedRows() === 1 ) | |
| 146 | -        { | |
| 138 | +        if( $this->db->getAffectedRows() === 1 ) { | |
| 147 | 139 | return true; | 
| 148 | 140 | } | 
| 149 | 141 | |
| @@ -152,8 +144,7 @@ discard block | ||
| 152 | 144 | |
| 153 | 145 | public function addCategory( $category_name, $category_rank ) | 
| 154 | 146 |      { | 
| 155 | - if( $this->isCategoryExists( $category_name ) ) | |
| 156 | -        { | |
| 147 | +        if( $this->isCategoryExists( $category_name ) ) { | |
| 157 | 148 | return false; | 
| 158 | 149 | } | 
| 159 | 150 | |
| @@ -167,8 +158,7 @@ discard block | ||
| 167 | 158 | $this->db_tbl['categories'] | 
| 168 | 159 | ); | 
| 169 | 160 | |
| 170 | - if( $this->db->getAffectedRows() === 1 ) | |
| 171 | -        { | |
| 161 | +        if( $this->db->getAffectedRows() === 1 ) { | |
| 172 | 162 | return $this->db->getInsertId(); | 
| 173 | 163 | } | 
| 174 | 164 | |
| @@ -187,8 +177,7 @@ discard block | ||
| 187 | 177 | ) | 
| 188 | 178 | ); | 
| 189 | 179 | |
| 190 | - if( $this->db->getAffectedRows() === 1 ) | |
| 191 | -        { | |
| 180 | +        if( $this->db->getAffectedRows() === 1 ) { | |
| 192 | 181 | return $category_id; | 
| 193 | 182 | } | 
| 194 | 183 | return false; | 
| @@ -4,18 +4,18 @@ | ||
| 4 | 4 | } | 
| 5 | 5 | |
| 6 | 6 |  switch ($modx->manager->action) { | 
| 7 | - case 102: | |
| 8 | -        if (!$modx->hasPermission('edit_plugin')) { | |
| 9 | - $modx->webAlertAndQuit($_lang["error_no_privileges"]); | |
| 10 | - } | |
| 11 | - break; | |
| 12 | - case 101: | |
| 13 | -        if (!$modx->hasPermission('new_plugin')) { | |
| 7 | + case 102: | |
| 8 | +            if (!$modx->hasPermission('edit_plugin')) { | |
| 9 | + $modx->webAlertAndQuit($_lang["error_no_privileges"]); | |
| 10 | + } | |
| 11 | + break; | |
| 12 | + case 101: | |
| 13 | +            if (!$modx->hasPermission('new_plugin')) { | |
| 14 | + $modx->webAlertAndQuit($_lang["error_no_privileges"]); | |
| 15 | + } | |
| 16 | + break; | |
| 17 | + default: | |
| 14 | 18 | $modx->webAlertAndQuit($_lang["error_no_privileges"]); | 
| 15 | - } | |
| 16 | - break; | |
| 17 | - default: | |
| 18 | - $modx->webAlertAndQuit($_lang["error_no_privileges"]); | |
| 19 | 19 | } | 
| 20 | 20 | |
| 21 | 21 | $id = isset($_REQUEST['id']) ? (int)$_REQUEST['id'] : 0; |