Completed
Pull Request — develop (#518)
by Agel_Nash
06:14
created
install/actions/action_summary.php 2 patches
Indentation   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -248,9 +248,9 @@
 block discarded – undo
248 248
         echo '<span class="notok">' . $_lang['failed'] . '</span></b>' . $_lang['table_prefix_not_exist'] . '</p>';
249 249
         $errors++;
250 250
         echo '<p>' . $_lang['table_prefix_not_exist_note'] . '</p>';
251
-  } else {
251
+    } else {
252 252
         echo '<span class="ok">' . $_lang['ok'] . '</span></p>';
253
-  }
253
+    }
254 254
 }
255 255
 
256 256
 // check mysql version
Please login to merge, or discard this patch.
Braces   +35 added lines, -18 removed lines patch added patch discarded remove patch
@@ -1,19 +1,22 @@  discard block
 block discarded – undo
1 1
 <?php
2
-if( ! function_exists('f_owc')){
2
+if( ! function_exists('f_owc')) {
3 3
     /**
4 4
      * @param $path
5 5
      * @param $data
6 6
      * @param null|int $mode
7 7
      */
8
-    function f_owc($path, $data, $mode = null){
8
+    function f_owc($path, $data, $mode = null)
9
+    {
9 10
         try {
10 11
             // make an attempt to create the file
11 12
             $hnd = fopen($path, 'w');
12 13
             fwrite($hnd, $data);
13 14
             fclose($hnd);
14 15
 
15
-            if(null !== $mode) chmod($path, $mode);
16
-        }catch(Exception $e){
16
+            if(null !== $mode) {
17
+                chmod($path, $mode);
18
+            }
19
+        } catch(Exception $e) {
17 20
             // Nothing, this is NOT normal
18 21
             unset($e);
19 22
         }
@@ -21,7 +24,9 @@  discard block
 block discarded – undo
21 24
 }
22 25
 
23 26
 $installMode = isset($_POST['installmode']) ? (int)$_POST['installmode'] : 0;
24
-if( ! isset($_lang)) $_lang = array();
27
+if( ! isset($_lang)) {
28
+    $_lang = array();
29
+}
25 30
 
26 31
 echo '<div class="stepcontainer">
27 32
       <ul class="progressbar">
@@ -57,7 +62,7 @@  discard block
 block discarded – undo
57 62
 // check if iconv is available
58 63
 echo '<p>' . $_lang['checking_iconv'];
59 64
 $iconv = (int) function_exists('iconv');
60
-if ($iconv == '0'){
65
+if ($iconv == '0') {
61 66
     echo '<span class="notok">' . $_lang['failed'].'</span></p><p><strong>'.$_lang['checking_iconv_note'].'</strong></p>';
62 67
     $errors++;
63 68
 } else {
@@ -119,7 +124,7 @@  discard block
 block discarded – undo
119 124
 
120 125
 // File Browser directories exists?
121 126
 echo '<p>'.$_lang['checking_if_images_exist'];
122
-switch(true){
127
+switch(true) {
123 128
     case !file_exists("../assets/images"):
124 129
     case !file_exists("../assets/files"):
125 130
     case !file_exists("../assets/backup"):
@@ -134,7 +139,7 @@  discard block
 block discarded – undo
134 139
 
135 140
 // File Browser directories writable?
136 141
 echo '<p>'.$_lang['checking_if_images_writable'];
137
-switch(true){
142
+switch(true) {
138 143
     case !is_writable("../assets/images"):
139 144
     case !is_writable("../assets/files"):
140 145
     case !is_writable("../assets/backup"):
@@ -268,7 +273,7 @@  discard block
 block discarded – undo
268 273
 if ($conn) {
269 274
     echo '<p>'. $_lang['checking_mysql_strict_mode'];
270 275
     $mysqlmode = mysqli_query($conn, "SELECT @@global.sql_mode");
271
-    if (mysqli_num_rows($mysqlmode) > 0){
276
+    if (mysqli_num_rows($mysqlmode) > 0) {
272 277
         $modes = mysqli_fetch_array($mysqlmode, MYSQLI_NUM);
273 278
         //$modes = array("STRICT_TRANS_TABLES"); // for testing
274 279
         // print_r($modes);
@@ -305,9 +310,9 @@  discard block
 block discarded – undo
305 310
     echo '<p>';
306 311
     echo $_lang['setup_cannot_continue'] . ' ';
307 312
 
308
-    if($errors > 1){
313
+    if($errors > 1) {
309 314
         echo $errors . " " . $_lang['errors'] . $_lang['please_correct_errors'] . $_lang['and_try_again_plural'];
310
-    }else{
315
+    } else {
311 316
         echo $_lang['error'] . $_lang['please_correct_error'] . $_lang['and_try_again'];
312 317
     }
313 318
 
@@ -343,23 +348,35 @@  discard block
 block discarded – undo
343 348
     <input type="hidden" value="<?php echo $_POST['installdata'] ?>" name="installdata" />
344 349
 <?php
345 350
     $templates = isset ($_POST['template']) ? $_POST['template'] : array ();
346
-    foreach ($templates as $i => $template) echo '<input type="hidden" name="template[]" value="'.$template.'" />';
351
+    foreach ($templates as $i => $template) {
352
+        echo '<input type="hidden" name="template[]" value="'.$template.'" />';
353
+    }
347 354
 
348 355
     $tvs = isset ($_POST['tv']) ? $_POST['tv'] : array ();
349
-    foreach ($tvs as $i => $tv) echo '<input type="hidden" name="tv[]" value="'.$tv.'" />';
356
+    foreach ($tvs as $i => $tv) {
357
+        echo '<input type="hidden" name="tv[]" value="'.$tv.'" />';
358
+    }
350 359
 
351 360
     $chunks = isset ($_POST['chunk']) ? $_POST['chunk'] : array ();
352
-    foreach ($chunks as $i => $chunk) echo '<input type="hidden" name="chunk[]" value="'.$chunk.'" />';
361
+    foreach ($chunks as $i => $chunk) {
362
+        echo '<input type="hidden" name="chunk[]" value="'.$chunk.'" />';
363
+    }
353 364
 
354 365
     $snippets = isset ($_POST['snippet']) ? $_POST['snippet'] : array ();
355
-    foreach ($snippets as $i => $snippet) echo '<input type="hidden" name="snippet[]" value="'.$snippet.'" />';
366
+    foreach ($snippets as $i => $snippet) {
367
+        echo '<input type="hidden" name="snippet[]" value="'.$snippet.'" />';
368
+    }
356 369
 
357 370
     $plugins = isset ($_POST['plugin']) ? $_POST['plugin'] : array ();
358
-    foreach ($plugins as $i => $plugin) echo '<input type="hidden" name="plugin[]" value="'.$plugin.'" />';
371
+    foreach ($plugins as $i => $plugin) {
372
+        echo '<input type="hidden" name="plugin[]" value="'.$plugin.'" />';
373
+    }
359 374
 
360 375
     $modules = isset ($_POST['module']) ? $_POST['module'] : array ();
361
-    foreach ($modules as $i => $module) echo '<input type="hidden" name="module[]" value="'.$module.'" />';
362
-?>
376
+    foreach ($modules as $i => $module) {
377
+        echo '<input type="hidden" name="module[]" value="'.$module.'" />';
378
+    }
379
+    ?>
363 380
 </div>
364 381
 
365 382
 <h2><?php echo $_lang['agree_to_terms'];?></h2>
Please login to merge, or discard this patch.
install/actions/action_mode.php 1 patch
Braces   +12 added lines, -10 removed lines patch added patch discarded remove patch
@@ -1,20 +1,22 @@
 block discarded – undo
1 1
 <?php
2 2
 // Determine upgradeability
3 3
 $upgradeable = 0;
4
-if (is_file($base_path . MGR_DIR . '/includes/config.inc.php')) { // Include the file so we can test its validity
4
+if (is_file($base_path . MGR_DIR . '/includes/config.inc.php')) {
5
+// Include the file so we can test its validity
5 6
     include $base_path . MGR_DIR . '/includes/config.inc.php';
6 7
     // We need to have all connection settings - tho prefix may be empty so we have to ignore it
7 8
     if (isset($dbase)) {
8
-        if (!$conn = @mysqli_connect($database_server, $database_user, $database_password))
9
-            $upgradeable = isset($_POST['installmode']) && $_POST['installmode'] == 'new' ? 0 : 2;
10
-        elseif (!@mysqli_select_db($conn, trim($dbase, '`')))
11
-            $upgradeable = isset($_POST['installmode']) && $_POST['installmode'] == 'new' ? 0 : 2;
12
-        else
13
-            $upgradeable = 1;
9
+        if (!$conn = @mysqli_connect($database_server, $database_user, $database_password)) {
10
+                    $upgradeable = isset($_POST['installmode']) && $_POST['installmode'] == 'new' ? 0 : 2;
11
+        } elseif (!@mysqli_select_db($conn, trim($dbase, '`'))) {
12
+                    $upgradeable = isset($_POST['installmode']) && $_POST['installmode'] == 'new' ? 0 : 2;
13
+        } else {
14
+                    $upgradeable = 1;
15
+        }
16
+    } else {
17
+            $upgradeable = 2;
18
+    }
14 19
     }
15
-    else
16
-        $upgradeable = 2;
17
-}
18 20
 
19 21
 $ph['moduleName']       = $moduleName;
20 22
 $ph['displayNew']       = ($upgradeable!=0) ? 'display:none;' : '';
Please login to merge, or discard this patch.
install/actions/action_options.php 1 patch
Braces   +13 added lines, -7 removed lines patch added patch discarded remove patch
@@ -1,16 +1,18 @@  discard block
 block discarded – undo
1 1
 <?php
2 2
 $installMode = isset($_POST['installmode']) ? (int)$_POST['installmode'] : 0;
3 3
 
4
-switch($installMode){
4
+switch($installMode) {
5 5
     case 0:
6 6
     case 2:
7 7
         $database_collation = isset($_POST['database_collation']) ? $_POST['database_collation'] : 'utf8_general_ci';
8 8
         $database_charset = substr($database_collation, 0, strpos($database_collation, '_'));
9 9
         $_POST['database_connection_charset'] = $database_charset;
10
-        if(empty($_SESSION['databaseloginpassword']))
11
-            $_SESSION['databaseloginpassword'] = $_POST['databaseloginpassword'];
12
-        if(empty($_SESSION['databaseloginname']))
13
-            $_SESSION['databaseloginname'] = $_POST['databaseloginname'];
10
+        if(empty($_SESSION['databaseloginpassword'])) {
11
+                    $_SESSION['databaseloginpassword'] = $_POST['databaseloginpassword'];
12
+        }
13
+        if(empty($_SESSION['databaseloginname'])) {
14
+                    $_SESSION['databaseloginname'] = $_POST['databaseloginname'];
15
+        }
14 16
         break;
15 17
     case 1:
16 18
         include $base_path . MGR_DIR . '/includes/config.inc.php';
@@ -24,7 +26,9 @@  discard block
 block discarded – undo
24 26
                 }
25 27
             }
26 28
         }
27
-        if (empty ($database_collation)) $database_collation = 'utf8_general_ci';
29
+        if (empty ($database_collation)) {
30
+            $database_collation = 'utf8_general_ci';
31
+        }
28 32
 
29 33
         $database_charset = substr($database_collation, 0, strpos($database_collation, '_'));
30 34
         if (!isset ($database_connection_charset) || empty ($database_connection_charset)) {
@@ -33,7 +37,9 @@  discard block
 block discarded – undo
33 37
 
34 38
         if (!isset ($database_connection_method) || empty ($database_connection_method)) {
35 39
             $database_connection_method = 'SET CHARACTER SET';
36
-            if (function_exists('mysqli_set_charset')) mysqli_set_charset($conn, $database_connection_charset);
40
+            if (function_exists('mysqli_set_charset')) {
41
+                mysqli_set_charset($conn, $database_connection_charset);
42
+            }
37 43
         }
38 44
         if ($database_connection_method != 'SET NAMES' && $database_connection_charset != $database_charset) {
39 45
             $database_connection_method = 'SET NAMES';
Please login to merge, or discard this patch.
install/actions/action_connection.php 2 patches
Indentation   +7 added lines, -7 removed lines patch added patch discarded remove patch
@@ -15,13 +15,13 @@
 block discarded – undo
15 15
         include($base_path.MGR_DIR.'/includes/config.inc.php');
16 16
         // We need to have all connection settings - but prefix may be empty so we have to ignore it
17 17
         if ($dbase) {
18
-          $database_name = trim($dbase, '`');
19
-          if (!$conn = mysqli_connect($database_server, $database_user, $database_password))
20
-              $upgradeable = (isset($_POST['installmode']) && $_POST['installmode']=='new') ? 0 : 2;
21
-          elseif (! mysqli_select_db($conn, trim($dbase, '`')))
22
-              $upgradeable = (isset($_POST['installmode']) && $_POST['installmode']=='new') ? 0 : 2;
23
-          else
24
-              $upgradeable = 1;
18
+            $database_name = trim($dbase, '`');
19
+            if (!$conn = mysqli_connect($database_server, $database_user, $database_password))
20
+                $upgradeable = (isset($_POST['installmode']) && $_POST['installmode']=='new') ? 0 : 2;
21
+            elseif (! mysqli_select_db($conn, trim($dbase, '`')))
22
+                $upgradeable = (isset($_POST['installmode']) && $_POST['installmode']=='new') ? 0 : 2;
23
+            else
24
+                $upgradeable = 1;
25 25
         }
26 26
         else $upgradable= 2;
27 27
     }
Please login to merge, or discard this patch.
Braces   +12 added lines, -9 removed lines patch added patch discarded remove patch
@@ -9,21 +9,24 @@
 block discarded – undo
9 9
     $table_prefix= substr(md5(time()), rand(0, 27), rand(3, 5))."_";
10 10
 } else {
11 11
     $database_name = '';
12
-    if (!is_file($base_path.MGR_DIR.'/includes/config.inc.php')) $upgradeable = 0;
13
-    else {
12
+    if (!is_file($base_path.MGR_DIR.'/includes/config.inc.php')) {
13
+        $upgradeable = 0;
14
+    } else {
14 15
         // Include the file so we can test its validity
15 16
         include($base_path.MGR_DIR.'/includes/config.inc.php');
16 17
         // We need to have all connection settings - but prefix may be empty so we have to ignore it
17 18
         if ($dbase) {
18 19
           $database_name = trim($dbase, '`');
19
-          if (!$conn = mysqli_connect($database_server, $database_user, $database_password))
20
-              $upgradeable = (isset($_POST['installmode']) && $_POST['installmode']=='new') ? 0 : 2;
21
-          elseif (! mysqli_select_db($conn, trim($dbase, '`')))
22
-              $upgradeable = (isset($_POST['installmode']) && $_POST['installmode']=='new') ? 0 : 2;
23
-          else
24
-              $upgradeable = 1;
20
+          if (!$conn = mysqli_connect($database_server, $database_user, $database_password)) {
21
+                        $upgradeable = (isset($_POST['installmode']) && $_POST['installmode']=='new') ? 0 : 2;
22
+          } elseif (! mysqli_select_db($conn, trim($dbase, '`'))) {
23
+                        $upgradeable = (isset($_POST['installmode']) && $_POST['installmode']=='new') ? 0 : 2;
24
+          } else {
25
+                        $upgradeable = 1;
26
+          }
27
+        } else {
28
+            $upgradable= 2;
25 29
         }
26
-        else $upgradable= 2;
27 30
     }
28 31
 }
29 32
 
Please login to merge, or discard this patch.
manager/includes/veriword.php 1 patch
Braces   +16 added lines, -8 removed lines patch added patch discarded remove patch
@@ -43,7 +43,8 @@  discard block
 block discarded – undo
43 43
 ## see sample.php for test and usage
44 44
 ## sample URL: http://www.program-ruti.org/veriword/
45 45
 ####
46
-class VeriWord {
46
+class VeriWord
47
+{
47 48
 
48 49
     /* path to font directory*/
49 50
     var $dir_font   = "ttf/";
@@ -53,7 +54,8 @@  discard block
 block discarded – undo
53 54
     var $im_width   = 0;
54 55
     var $im_height  = 0;
55 56
 
56
-    function __construct($w=200, $h=80) {
57
+    function __construct($w=200, $h=80)
58
+    {
57 59
         /* create session to set word for verification */
58 60
         startCMSSession();
59 61
         $this->set_veriword();
@@ -62,21 +64,24 @@  discard block
 block discarded – undo
62 64
         $this->im_height        = $h;
63 65
     }
64 66
 
65
-    function set_veriword() {
67
+    function set_veriword()
68
+    {
66 69
         /* create session variable for verification,
67 70
            you may change the session variable name */
68 71
         $this->word             = $this->pick_word();
69 72
         $_SESSION['veriword']   = $this->word;
70 73
     }
71 74
 
72
-    function output_image() {
75
+    function output_image()
76
+    {
73 77
         /* output the image as jpeg */
74 78
         $this->draw_image();
75 79
         header("Content-type: image/jpeg");
76 80
         imagejpeg($this->im);
77 81
     }
78 82
 
79
-    function pick_word() {
83
+    function pick_word()
84
+    {
80 85
         global $modx;
81 86
         // set default words
82 87
         $words="MODX,Access,Better,BitCode,Chunk,Cache,Desc,Design,Excell,Enjoy,URLs,TechView,Gerald,Griff,Humphrey,Holiday,Intel,Integration,Joystick,Join(),Oscope,Genetic,Light,Likeness,Marit,Maaike,Niche,Netherlands,Ordinance,Oscillo,Parser,Phusion,Query,Question,Regalia,Righteous,Snippet,Sentinel,Template,Thespian,Unity,Enterprise,Verily,Veri,Website,WideWeb,Yap,Yellow,Zebra,Zygote";
@@ -87,7 +92,8 @@  discard block
 block discarded – undo
87 92
         return (string) $arr_words[array_rand($arr_words)].rand(10,999);
88 93
     }
89 94
 
90
-    function draw_text() {
95
+    function draw_text()
96
+    {
91 97
         $dir = dir($this->dir_font);
92 98
         $fontstmp = array();
93 99
         while (false !== ($file = $dir->read())) {
@@ -143,7 +149,8 @@  discard block
 block discarded – undo
143 149
     }
144 150
 
145 151
 
146
-    function draw_image() {
152
+    function draw_image()
153
+    {
147 154
 
148 155
         /* pick one background image randomly from image directory */
149 156
         $img_file       = $this->dir_noise."noise".rand(1,4).".jpg";
@@ -174,7 +181,8 @@  discard block
 block discarded – undo
174 181
         return $this->im;
175 182
     }
176 183
 
177
-    function destroy_image() {
184
+    function destroy_image()
185
+    {
178 186
 
179 187
         imagedestroy($this->im);
180 188
 
Please login to merge, or discard this patch.
manager/includes/user_settings.inc.php 1 patch
Indentation   +13 added lines, -13 removed lines patch added patch discarded remove patch
@@ -3,23 +3,23 @@
 block discarded – undo
3 3
 
4 4
 // START HACK
5 5
 if (isset ($modx)) {
6
-	$user_id = $modx->getLoginUserID();
6
+    $user_id = $modx->getLoginUserID();
7 7
 } else {
8
-	$user_id = $_SESSION['mgrInternalKey'];
8
+    $user_id = $_SESSION['mgrInternalKey'];
9 9
 }
10 10
 // END HACK
11 11
 
12 12
 if (!empty($user_id)) {
13
-	// Raymond: grab the user settings from the database.
14
-	$rs = $modx->db->select('setting_name, setting_value', $modx->getFullTableName('user_settings'), "user=".$modx->getLoginUserID());
13
+    // Raymond: grab the user settings from the database.
14
+    $rs = $modx->db->select('setting_name, setting_value', $modx->getFullTableName('user_settings'), "user=".$modx->getLoginUserID());
15 15
 
16
-	$which_browser_default = $which_browser;
17
-	while ($row = $modx->db->getRow($rs)) {
18
-		if($row['setting_name'] == 'which_browser' && $row['setting_value'] == 'default') $row['setting_value'] = $which_browser_default;
19
-		$settings[$row['setting_name']] = $row['setting_value'];
20
-		if (isset($modx->config)) {
21
-			$modx->config[$row['setting_name']] = $row['setting_value'];
22
-		}
23
-	}
24
-	extract($settings, EXTR_OVERWRITE);
16
+    $which_browser_default = $which_browser;
17
+    while ($row = $modx->db->getRow($rs)) {
18
+        if($row['setting_name'] == 'which_browser' && $row['setting_value'] == 'default') $row['setting_value'] = $which_browser_default;
19
+        $settings[$row['setting_name']] = $row['setting_value'];
20
+        if (isset($modx->config)) {
21
+            $modx->config[$row['setting_name']] = $row['setting_value'];
22
+        }
23
+    }
24
+    extract($settings, EXTR_OVERWRITE);
25 25
 }
Please login to merge, or discard this patch.
manager/includes/rss.inc.php 1 patch
Indentation   +32 added lines, -32 removed lines patch added patch discarded remove patch
@@ -1,5 +1,5 @@  discard block
 block discarded – undo
1 1
 <?php
2
- /*
2
+    /*
3 3
  *  MODX Manager Home Page Implmentation by pixelchutes (www.pixelchutes.com)
4 4
  *  Based on kudo's kRSS Module v1.0.72
5 5
  *
@@ -30,43 +30,43 @@  discard block
 block discarded – undo
30 30
 require_once(MODX_MANAGER_PATH.'media/rss/rss_fetch.inc');
31 31
 // Convert relative path into absolute url
32 32
 function rel2abs( $rel, $base ) {
33
-	// parse base URL  and convert to local variables: $scheme, $host,  $path
34
-	extract( parse_url( $base ) );
35
-	if ( strpos( $rel,"//" ) === 0 ) {
36
-		return $scheme . ':' . $rel;
37
-	}
38
-	// return if already absolute URL
39
-	if ( parse_url( $rel, PHP_URL_SCHEME ) != '' ) {
40
-		return $rel;
41
-	}
42
-	// queries and anchors
43
-	if ( $rel[0] == '#' || $rel[0] == '?' ) {
44
-		return $base . $rel;
45
-	}
46
-	// remove non-directory element from path
47
-	$path = preg_replace( '#/[^/]*$#', '', $path );
48
-	// destroy path if relative url points to root
49
-	if ( $rel[0] ==  '/' ) {
50
-		$path = '';
51
-	}
52
-	// dirty absolute URL
53
-	$abs = $host . $path . "/" . $rel;
54
-	// replace '//' or  '/./' or '/foo/../' with '/'
55
-	$abs = preg_replace( "/(\/\.?\/)/", "/", $abs );
56
-	$abs = preg_replace( "/\/(?!\.\.)[^\/]+\/\.\.\//", "/", $abs );
57
-	// absolute URL is ready!
58
-	return $scheme . '://' . $abs;
33
+    // parse base URL  and convert to local variables: $scheme, $host,  $path
34
+    extract( parse_url( $base ) );
35
+    if ( strpos( $rel,"//" ) === 0 ) {
36
+        return $scheme . ':' . $rel;
37
+    }
38
+    // return if already absolute URL
39
+    if ( parse_url( $rel, PHP_URL_SCHEME ) != '' ) {
40
+        return $rel;
41
+    }
42
+    // queries and anchors
43
+    if ( $rel[0] == '#' || $rel[0] == '?' ) {
44
+        return $base . $rel;
45
+    }
46
+    // remove non-directory element from path
47
+    $path = preg_replace( '#/[^/]*$#', '', $path );
48
+    // destroy path if relative url points to root
49
+    if ( $rel[0] ==  '/' ) {
50
+        $path = '';
51
+    }
52
+    // dirty absolute URL
53
+    $abs = $host . $path . "/" . $rel;
54
+    // replace '//' or  '/./' or '/foo/../' with '/'
55
+    $abs = preg_replace( "/(\/\.?\/)/", "/", $abs );
56
+    $abs = preg_replace( "/\/(?!\.\.)[^\/]+\/\.\.\//", "/", $abs );
57
+    // absolute URL is ready!
58
+    return $scheme . '://' . $abs;
59 59
 }
60 60
 $feedData = array();
61 61
 
62 62
 // create Feed
63 63
 foreach ($urls as $section=>$url) {
64
-	$output = '';
64
+    $output = '';
65 65
     $rss = @fetch_rss($url);
66 66
     if( !$rss ){
67
-    	$feedData[$section] = 'Failed to retrieve ' . $url;
68
-    	continue;
69
-	}
67
+        $feedData[$section] = 'Failed to retrieve ' . $url;
68
+        continue;
69
+    }
70 70
     $output .= '<ul>';
71 71
 
72 72
     $items = array_slice($rss->items, 0, $itemsNumber);
@@ -84,5 +84,5 @@  discard block
 block discarded – undo
84 84
     }
85 85
 
86 86
     $output .= '</ul>';
87
-	$feedData[$section] = $output;
87
+    $feedData[$section] = $output;
88 88
 }
Please login to merge, or discard this patch.
manager/includes/extenders/maketable.class.php 2 patches
Indentation   +494 added lines, -494 removed lines patch added patch discarded remove patch
@@ -9,366 +9,366 @@  discard block
 block discarded – undo
9 9
  * @author Jason Coward <[email protected]> (MODX)
10 10
  */
11 11
 class MakeTable {
12
-	var $actionField;
13
-	var $cellAction;
14
-	var $linkAction;
15
-	var $tableWidth;
16
-	var $tableClass;
17
-	var $tableID;
18
-	var $thClass;
19
-	var $rowHeaderClass;
20
-	var $columnHeaderClass;
21
-	var $rowRegularClass;
22
-	var $rowAlternateClass;
23
-	var $formName;
24
-	var $formAction;
25
-	var $formElementType;
26
-	var $formElementName;
27
-	var $rowAlternatingScheme;
28
-	var $excludeFields;
29
-	var $allOption;
30
-	var $pageNav;
31
-	var $columnWidths;
32
-	var $selectedValues;
33
-
34
-	function __construct() {
35
-		$this->fieldHeaders= array();
36
-		$this->excludeFields= array();
37
-		$this->actionField= '';
38
-		$this->cellAction= '';
39
-		$this->linkAction= '';
40
-		$this->tableWidth= '';
41
-		$this->tableClass= '';
42
-		$this->rowHeaderClass= '';
43
-		$this->columnHeaderClass= '';
44
-		$this->rowRegularClass= '';
45
-		$this->rowAlternateClass= 'alt';
46
-		$this->formName= 'tableForm';
47
-		$this->formAction= '[~[*id*]~]';
48
-		$this->formElementName= '';
49
-		$this->formElementType= '';
50
-		$this->rowAlternatingScheme= 'EVEN';
51
-		$this->allOption= 0;
52
-		$this->selectedValues= array();
53
-		$this->extra= '';
54
-	}
55
-
56
-	/**
57
-	 * Sets the default link href for all cells in the table.
58
-	 *
59
-	 * @param $value A URL to execute when table cells are clicked.
60
-	 */
61
-	function setCellAction($value) {
62
-		$this->cellAction= $this->prepareLink($value);
63
-	}
64
-
65
-	/**
66
-	 * Sets the default link href for the text presented in a cell.
67
-	 *
68
-	 * @param $value A URL to execute when text within table cells are clicked.
69
-	 */
70
-	function setLinkAction($value) {
71
-		$this->linkAction= $this->prepareLink($value);
72
-	}
73
-
74
-	/**
75
-	 * Sets the width attribute of the main HTML TABLE.
76
-	 *
77
-	 * @param $value A valid width attribute for the HTML TABLE tag
78
-	 */
79
-	function setTableWidth($value) {
80
-		$this->tableWidth= $value;
81
-	}
82
-
83
-	/**
84
-	 * Sets the class attribute of the main HTML TABLE.
85
-	 *
86
-	 * @param $value A class for the main HTML TABLE.
87
-	 */
88
-	function setTableClass($value) {
89
-		$this->tableClass= $value;
90
-	}
91
-
92
-	/**
93
-	 * Sets the id attribute of the main HTML TABLE.
94
-	 *
95
-	 * @param $value A class for the main HTML TABLE.
96
-	 */
97
-	function setTableID($value) {
98
-		$this->tableID= $value;
99
-	}
100
-
101
-	/**
102
-	 * Sets the class attribute of the table header row.
103
-	 *
104
-	 * @param $value A class for the table header row.
105
-	 */
106
-	function setRowHeaderClass($value) {
107
-		$this->rowHeaderClass= $value;
108
-	}
109
-
110
-		/**
111
-	 * Sets the class attribute of the table header row.
112
-	 *
113
-	 * @param $value A class for the table header row.
114
-	 */
115
-	function setThHeaderClass($value) {
116
-		$this->thClass= $value;
117
-	}
118
-
119
-	/**
120
-	 * Sets the class attribute of the column header row.
121
-	 *
122
-	 * @param $value A class for the column header row.
123
-	 */
124
-	function setColumnHeaderClass($value) {
125
-		$this->columnHeaderClass= $value;
126
-	}
127
-
128
-	/**
129
-	 * Sets the class attribute of regular table rows.
130
-	 *
131
-	 * @param $value A class for regular table rows.
132
-	 */
133
-	function setRowRegularClass($value) {
134
-		$this->rowRegularClass= $value;
135
-	}
136
-
137
-	/**
138
-	 * Sets the class attribute of alternate table rows.
139
-	 *
140
-	 * @param $value A class for alternate table rows.
141
-	 */
142
-	function setRowAlternateClass($value) {
143
-		$this->rowAlternateClass= $value;
144
-	}
145
-
146
-	/**
147
-	 * Sets the type of INPUT form element to be presented as the first column.
148
-	 *
149
-	 * @param $value Indicates the INPUT form element type attribute.
150
-	 */
151
-	function setFormElementType($value) {
152
-		$this->formElementType= $value;
153
-	}
154
-
155
-	/**
156
-	 * Sets the name of the INPUT form element to be presented as the first column.
157
-	 *
158
-	 * @param $value Indicates the INPUT form element name attribute.
159
-	 */
160
-	function setFormElementName($value) {
161
-		$this->formElementName= $value;
162
-	}
163
-
164
-	/**
165
-	 * Sets the name of the FORM to wrap the table in when a form element has
166
-	 * been indicated.
167
-	 *
168
-	 * @param $value Indicates the FORM name attribute.
169
-	 */
170
-	function setFormName($value) {
171
-		$this->formName= $value;
172
-	}
173
-
174
-	/**
175
-	 * Sets the action of the FORM element.
176
-	 *
177
-	 * @param $value Indicates the FORM action attribute.
178
-	 */
179
-	function setFormAction($value) {
180
-		$this->formAction= $value;
181
-	}
182
-
183
-	/**
184
-	 * Excludes fields from the table by array key.
185
-	 *
186
-	 * @param $value An Array of field keys to exclude from the table.
187
-	 */
188
-	function setExcludeFields($value) {
189
-		$this->excludeFields= $value;
190
-	}
191
-
192
-	/**
193
-	 * Sets the table to provide alternate row colors using ODD or EVEN rows
194
-	 *
195
-	 * @param $value 'ODD' or 'EVEN' to indicate the alternate row scheme.
196
-	 */
197
-	function setRowAlternatingScheme($value) {
198
-		$this->rowAlternatingScheme= $value;
199
-	}
200
-
201
-	/**
202
-	 * Sets the default field value to be used when appending query parameters
203
-	 * to link actions.
204
-	 *
205
-	 * @param $value The key of the field to add as a query string parameter.
206
-	 */
207
-	function setActionFieldName($value) {
208
-		$this->actionField= $value;
209
-	}
210
-
211
-	/**
212
-	 * Sets the width attribute of each column in the array.
213
-	 *
214
-	 * @param $value An Array of column widths in the order of the keys in the
215
-	 * 			source table array.
216
-	 */
217
-	function setColumnWidths($widthArray) {
218
-		$this->columnWidths= $widthArray;
219
-	}
220
-
221
-	/**
222
-	 * An optional array of values that can be preselected when using
223
-	 *
224
-	 * @param $value Indicates the INPUT form element type attribute.
225
-	 */
226
-	function setSelectedValues($valueArray) {
227
-		$this->selectedValues= $valueArray;
228
-	}
229
-
230
-	/**
231
-	 * Sets extra content to be presented following the table (but within
232
-	 * the form, if a form is being rendered with the table).
233
-	 *
234
-	 * @param $value A string of additional content.
235
-	 */
236
-	function setExtra($value) {
237
-		$this->extra= $value;
238
-	}
239
-
240
-	/**
241
-	 * Retrieves the width of a specific table column by index position.
242
-	 *
243
-	 * @param $columnPosition The index of the column to get the width for.
244
-	 */
245
-	function getColumnWidth($columnPosition) {
246
-		$currentWidth= '';
247
-		if (is_array($this->columnWidths)) {
248
-			$currentWidth= $this->columnWidths[$columnPosition] ? ' width="'.$this->columnWidths[$columnPosition].'" ' : '';
249
-		}
250
-		return $currentWidth;
251
-	}
252
-
253
-	/**
254
-	 * Determines what class the current row should have applied.
255
-	 *
256
-	 * @param $value The position of the current row being rendered.
257
-	 */
258
-	function determineRowClass($position) {
259
-		switch ($this->rowAlternatingScheme) {
260
-			case 'ODD' :
261
-				$modRemainder= 1;
262
-				break;
263
-			case 'EVEN' :
264
-				$modRemainder= 0;
265
-				break;
266
-		}
267
-		if ($position % 2 == $modRemainder) {
268
-			$currentClass= $this->rowRegularClass;
269
-		} else {
270
-			$currentClass= $this->rowAlternateClass;
271
-		}
272
-		return ' class="'.$currentClass.'"';
273
-	}
274
-
275
-	/**
276
-	 * Generates an onclick action applied to the current cell, to execute
277
-	 * any specified cell actions.
278
-	 *
279
-	 * @param $value Indicates the INPUT form element type attribute.
280
-	 */
281
-	function getCellAction($currentActionFieldValue) {
282
-		if ($this->cellAction) {
283
-			$cellAction= ' onClick="javascript:window.location=\''.$this->cellAction.$this->actionField.'='.urlencode($currentActionFieldValue).'\'" ';
284
-		}
285
-		return $cellAction;
286
-	}
287
-
288
-	/**
289
-	 * Generates the cell content, including any specified action fields values.
290
-	 *
291
-	 * @param $currentActionFieldValue The value to be applied to the link action.
292
-	 * @param $value The value of the cell.
293
-	 */
294
-	function createCellText($currentActionFieldValue, $value) {
295
-		$cell .= $value;
296
-		if ($this->linkAction) {
297
-			$cell= '<a href="'.$this->linkAction.$this->actionField.'='.urlencode($currentActionFieldValue).'">'.$cell.'</a>';
298
-		}
299
-		return $cell;
300
-	}
301
-
302
-	/**
303
-	 * Sets an option to generate a check all link when checkbox is indicated
304
-	 * as the table formElementType.
305
-	 */
306
-	function setAllOption() {
307
-		$this->allOption= 1;
308
-	}
309
-
310
-	/**
311
-	 * Function to prepare a link generated in the table cell/link actions.
312
-	 *
313
-	 * @param $value Indicates the INPUT form element type attribute.
314
-	 */
315
-	function prepareLink($link) {
316
-		if (strstr($link, '?')) {
317
-			$end= '&';
318
-		} else {
319
-			$end= '?';
320
-		}
321
-		return $link.$end;
322
-	}
323
-
324
-	/**
325
-	 * Generates the table content.
326
-	 *
327
-	 * @param $fieldsArray The associative array representing the table rows
328
-	 * and columns.
329
-	 * @param $fieldHeadersArray An optional array of values for providing
330
-	 * alternative field headers; this is an associative arrays of keys from
331
-	 * the $fieldsArray where the values represent the alt heading content
332
-	 * for each column.
333
-	 */
334
-	function create($fieldsArray, $fieldHeadersArray=array(),$linkpage="") {
335
-	global $_lang;
336
-		if (is_array($fieldsArray)) {
337
-			$i= 0;
338
-			foreach ($fieldsArray as $fieldName => $fieldValue) {
339
-				$table .= "\t<tr".$this->determineRowClass($i).">\n";
340
-				$currentActionFieldValue= $fieldValue[$this->actionField];
341
-				if (is_array($this->selectedValues)) {
342
-					$isChecked= array_search($currentActionFieldValue, $this->selectedValues)===false? 0 : 1;
343
-				} else {
344
-					$isChecked= false;
345
-				}
346
-				$table .= $this->addFormField($currentActionFieldValue, $isChecked);
347
-				$colPosition= 0;
348
-				foreach ($fieldValue as $key => $value) {
349
-					if (!in_array($key, $this->excludeFields)) {
350
-						$table .= "\t\t<td".$this->getCellAction($currentActionFieldValue).">";
351
-						$table .= $this->createCellText($currentActionFieldValue, $value);
352
-						$table .= "</td>\n";
353
-						if ($i == 0) {
354
-							if (empty ($header) && $this->formElementType) {
355
-								$header .= "\t\t<th style=\"width:32px\" ".($this->thClass ? 'class="'.$this->thClass.'"' : '').">". ($this->allOption ? '<a href="javascript:clickAll()">all</a>' : '')."</th>\n";
356
-							}
357
-							$headerText= array_key_exists($key, $fieldHeadersArray)? $fieldHeadersArray[$key]: $key;
358
-							$header .= "\t\t<th".$this->getColumnWidth($colPosition).($this->thClass ? ' class="'.$this->thClass.'" ' : '').">".$headerText."</th>\n";
359
-						}
360
-						$colPosition ++;
361
-					}
362
-				}
363
-				$i ++;
364
-				$table .= "\t</tr>\n";
365
-			}
366
-			$table= "\n".'<table'. ($this->tableWidth ? ' width="'.$this->tableWidth.'"' : ''). ($this->tableClass ? ' class="'.$this->tableClass.'"' : ''). ($this->tableID ? ' id="'.$this->tableID.'"' : '').">\n". ($header ? "\t<thead>\n\t<tr class=\"".$this->rowHeaderClass."\">\n".$header."\t</tr>\n\t</thead>\n" : '').$table."</table>\n";
367
-			if ($this->formElementType) {
368
-				$table= "\n".'<form id="'.$this->formName.'" name="'.$this->formName.'" action="'.$this->formAction.'" method="POST">'.$table;
369
-			}
370
-			if (strlen($this->pageNav) > 1) {//changed to display the pagination if exists.
371
-				/* commented this part because of cookie
12
+    var $actionField;
13
+    var $cellAction;
14
+    var $linkAction;
15
+    var $tableWidth;
16
+    var $tableClass;
17
+    var $tableID;
18
+    var $thClass;
19
+    var $rowHeaderClass;
20
+    var $columnHeaderClass;
21
+    var $rowRegularClass;
22
+    var $rowAlternateClass;
23
+    var $formName;
24
+    var $formAction;
25
+    var $formElementType;
26
+    var $formElementName;
27
+    var $rowAlternatingScheme;
28
+    var $excludeFields;
29
+    var $allOption;
30
+    var $pageNav;
31
+    var $columnWidths;
32
+    var $selectedValues;
33
+
34
+    function __construct() {
35
+        $this->fieldHeaders= array();
36
+        $this->excludeFields= array();
37
+        $this->actionField= '';
38
+        $this->cellAction= '';
39
+        $this->linkAction= '';
40
+        $this->tableWidth= '';
41
+        $this->tableClass= '';
42
+        $this->rowHeaderClass= '';
43
+        $this->columnHeaderClass= '';
44
+        $this->rowRegularClass= '';
45
+        $this->rowAlternateClass= 'alt';
46
+        $this->formName= 'tableForm';
47
+        $this->formAction= '[~[*id*]~]';
48
+        $this->formElementName= '';
49
+        $this->formElementType= '';
50
+        $this->rowAlternatingScheme= 'EVEN';
51
+        $this->allOption= 0;
52
+        $this->selectedValues= array();
53
+        $this->extra= '';
54
+    }
55
+
56
+    /**
57
+     * Sets the default link href for all cells in the table.
58
+     *
59
+     * @param $value A URL to execute when table cells are clicked.
60
+     */
61
+    function setCellAction($value) {
62
+        $this->cellAction= $this->prepareLink($value);
63
+    }
64
+
65
+    /**
66
+     * Sets the default link href for the text presented in a cell.
67
+     *
68
+     * @param $value A URL to execute when text within table cells are clicked.
69
+     */
70
+    function setLinkAction($value) {
71
+        $this->linkAction= $this->prepareLink($value);
72
+    }
73
+
74
+    /**
75
+     * Sets the width attribute of the main HTML TABLE.
76
+     *
77
+     * @param $value A valid width attribute for the HTML TABLE tag
78
+     */
79
+    function setTableWidth($value) {
80
+        $this->tableWidth= $value;
81
+    }
82
+
83
+    /**
84
+     * Sets the class attribute of the main HTML TABLE.
85
+     *
86
+     * @param $value A class for the main HTML TABLE.
87
+     */
88
+    function setTableClass($value) {
89
+        $this->tableClass= $value;
90
+    }
91
+
92
+    /**
93
+     * Sets the id attribute of the main HTML TABLE.
94
+     *
95
+     * @param $value A class for the main HTML TABLE.
96
+     */
97
+    function setTableID($value) {
98
+        $this->tableID= $value;
99
+    }
100
+
101
+    /**
102
+     * Sets the class attribute of the table header row.
103
+     *
104
+     * @param $value A class for the table header row.
105
+     */
106
+    function setRowHeaderClass($value) {
107
+        $this->rowHeaderClass= $value;
108
+    }
109
+
110
+        /**
111
+         * Sets the class attribute of the table header row.
112
+         *
113
+         * @param $value A class for the table header row.
114
+         */
115
+    function setThHeaderClass($value) {
116
+        $this->thClass= $value;
117
+    }
118
+
119
+    /**
120
+     * Sets the class attribute of the column header row.
121
+     *
122
+     * @param $value A class for the column header row.
123
+     */
124
+    function setColumnHeaderClass($value) {
125
+        $this->columnHeaderClass= $value;
126
+    }
127
+
128
+    /**
129
+     * Sets the class attribute of regular table rows.
130
+     *
131
+     * @param $value A class for regular table rows.
132
+     */
133
+    function setRowRegularClass($value) {
134
+        $this->rowRegularClass= $value;
135
+    }
136
+
137
+    /**
138
+     * Sets the class attribute of alternate table rows.
139
+     *
140
+     * @param $value A class for alternate table rows.
141
+     */
142
+    function setRowAlternateClass($value) {
143
+        $this->rowAlternateClass= $value;
144
+    }
145
+
146
+    /**
147
+     * Sets the type of INPUT form element to be presented as the first column.
148
+     *
149
+     * @param $value Indicates the INPUT form element type attribute.
150
+     */
151
+    function setFormElementType($value) {
152
+        $this->formElementType= $value;
153
+    }
154
+
155
+    /**
156
+     * Sets the name of the INPUT form element to be presented as the first column.
157
+     *
158
+     * @param $value Indicates the INPUT form element name attribute.
159
+     */
160
+    function setFormElementName($value) {
161
+        $this->formElementName= $value;
162
+    }
163
+
164
+    /**
165
+     * Sets the name of the FORM to wrap the table in when a form element has
166
+     * been indicated.
167
+     *
168
+     * @param $value Indicates the FORM name attribute.
169
+     */
170
+    function setFormName($value) {
171
+        $this->formName= $value;
172
+    }
173
+
174
+    /**
175
+     * Sets the action of the FORM element.
176
+     *
177
+     * @param $value Indicates the FORM action attribute.
178
+     */
179
+    function setFormAction($value) {
180
+        $this->formAction= $value;
181
+    }
182
+
183
+    /**
184
+     * Excludes fields from the table by array key.
185
+     *
186
+     * @param $value An Array of field keys to exclude from the table.
187
+     */
188
+    function setExcludeFields($value) {
189
+        $this->excludeFields= $value;
190
+    }
191
+
192
+    /**
193
+     * Sets the table to provide alternate row colors using ODD or EVEN rows
194
+     *
195
+     * @param $value 'ODD' or 'EVEN' to indicate the alternate row scheme.
196
+     */
197
+    function setRowAlternatingScheme($value) {
198
+        $this->rowAlternatingScheme= $value;
199
+    }
200
+
201
+    /**
202
+     * Sets the default field value to be used when appending query parameters
203
+     * to link actions.
204
+     *
205
+     * @param $value The key of the field to add as a query string parameter.
206
+     */
207
+    function setActionFieldName($value) {
208
+        $this->actionField= $value;
209
+    }
210
+
211
+    /**
212
+     * Sets the width attribute of each column in the array.
213
+     *
214
+     * @param $value An Array of column widths in the order of the keys in the
215
+     * 			source table array.
216
+     */
217
+    function setColumnWidths($widthArray) {
218
+        $this->columnWidths= $widthArray;
219
+    }
220
+
221
+    /**
222
+     * An optional array of values that can be preselected when using
223
+     *
224
+     * @param $value Indicates the INPUT form element type attribute.
225
+     */
226
+    function setSelectedValues($valueArray) {
227
+        $this->selectedValues= $valueArray;
228
+    }
229
+
230
+    /**
231
+     * Sets extra content to be presented following the table (but within
232
+     * the form, if a form is being rendered with the table).
233
+     *
234
+     * @param $value A string of additional content.
235
+     */
236
+    function setExtra($value) {
237
+        $this->extra= $value;
238
+    }
239
+
240
+    /**
241
+     * Retrieves the width of a specific table column by index position.
242
+     *
243
+     * @param $columnPosition The index of the column to get the width for.
244
+     */
245
+    function getColumnWidth($columnPosition) {
246
+        $currentWidth= '';
247
+        if (is_array($this->columnWidths)) {
248
+            $currentWidth= $this->columnWidths[$columnPosition] ? ' width="'.$this->columnWidths[$columnPosition].'" ' : '';
249
+        }
250
+        return $currentWidth;
251
+    }
252
+
253
+    /**
254
+     * Determines what class the current row should have applied.
255
+     *
256
+     * @param $value The position of the current row being rendered.
257
+     */
258
+    function determineRowClass($position) {
259
+        switch ($this->rowAlternatingScheme) {
260
+            case 'ODD' :
261
+                $modRemainder= 1;
262
+                break;
263
+            case 'EVEN' :
264
+                $modRemainder= 0;
265
+                break;
266
+        }
267
+        if ($position % 2 == $modRemainder) {
268
+            $currentClass= $this->rowRegularClass;
269
+        } else {
270
+            $currentClass= $this->rowAlternateClass;
271
+        }
272
+        return ' class="'.$currentClass.'"';
273
+    }
274
+
275
+    /**
276
+     * Generates an onclick action applied to the current cell, to execute
277
+     * any specified cell actions.
278
+     *
279
+     * @param $value Indicates the INPUT form element type attribute.
280
+     */
281
+    function getCellAction($currentActionFieldValue) {
282
+        if ($this->cellAction) {
283
+            $cellAction= ' onClick="javascript:window.location=\''.$this->cellAction.$this->actionField.'='.urlencode($currentActionFieldValue).'\'" ';
284
+        }
285
+        return $cellAction;
286
+    }
287
+
288
+    /**
289
+     * Generates the cell content, including any specified action fields values.
290
+     *
291
+     * @param $currentActionFieldValue The value to be applied to the link action.
292
+     * @param $value The value of the cell.
293
+     */
294
+    function createCellText($currentActionFieldValue, $value) {
295
+        $cell .= $value;
296
+        if ($this->linkAction) {
297
+            $cell= '<a href="'.$this->linkAction.$this->actionField.'='.urlencode($currentActionFieldValue).'">'.$cell.'</a>';
298
+        }
299
+        return $cell;
300
+    }
301
+
302
+    /**
303
+     * Sets an option to generate a check all link when checkbox is indicated
304
+     * as the table formElementType.
305
+     */
306
+    function setAllOption() {
307
+        $this->allOption= 1;
308
+    }
309
+
310
+    /**
311
+     * Function to prepare a link generated in the table cell/link actions.
312
+     *
313
+     * @param $value Indicates the INPUT form element type attribute.
314
+     */
315
+    function prepareLink($link) {
316
+        if (strstr($link, '?')) {
317
+            $end= '&';
318
+        } else {
319
+            $end= '?';
320
+        }
321
+        return $link.$end;
322
+    }
323
+
324
+    /**
325
+     * Generates the table content.
326
+     *
327
+     * @param $fieldsArray The associative array representing the table rows
328
+     * and columns.
329
+     * @param $fieldHeadersArray An optional array of values for providing
330
+     * alternative field headers; this is an associative arrays of keys from
331
+     * the $fieldsArray where the values represent the alt heading content
332
+     * for each column.
333
+     */
334
+    function create($fieldsArray, $fieldHeadersArray=array(),$linkpage="") {
335
+    global $_lang;
336
+        if (is_array($fieldsArray)) {
337
+            $i= 0;
338
+            foreach ($fieldsArray as $fieldName => $fieldValue) {
339
+                $table .= "\t<tr".$this->determineRowClass($i).">\n";
340
+                $currentActionFieldValue= $fieldValue[$this->actionField];
341
+                if (is_array($this->selectedValues)) {
342
+                    $isChecked= array_search($currentActionFieldValue, $this->selectedValues)===false? 0 : 1;
343
+                } else {
344
+                    $isChecked= false;
345
+                }
346
+                $table .= $this->addFormField($currentActionFieldValue, $isChecked);
347
+                $colPosition= 0;
348
+                foreach ($fieldValue as $key => $value) {
349
+                    if (!in_array($key, $this->excludeFields)) {
350
+                        $table .= "\t\t<td".$this->getCellAction($currentActionFieldValue).">";
351
+                        $table .= $this->createCellText($currentActionFieldValue, $value);
352
+                        $table .= "</td>\n";
353
+                        if ($i == 0) {
354
+                            if (empty ($header) && $this->formElementType) {
355
+                                $header .= "\t\t<th style=\"width:32px\" ".($this->thClass ? 'class="'.$this->thClass.'"' : '').">". ($this->allOption ? '<a href="javascript:clickAll()">all</a>' : '')."</th>\n";
356
+                            }
357
+                            $headerText= array_key_exists($key, $fieldHeadersArray)? $fieldHeadersArray[$key]: $key;
358
+                            $header .= "\t\t<th".$this->getColumnWidth($colPosition).($this->thClass ? ' class="'.$this->thClass.'" ' : '').">".$headerText."</th>\n";
359
+                        }
360
+                        $colPosition ++;
361
+                    }
362
+                }
363
+                $i ++;
364
+                $table .= "\t</tr>\n";
365
+            }
366
+            $table= "\n".'<table'. ($this->tableWidth ? ' width="'.$this->tableWidth.'"' : ''). ($this->tableClass ? ' class="'.$this->tableClass.'"' : ''). ($this->tableID ? ' id="'.$this->tableID.'"' : '').">\n". ($header ? "\t<thead>\n\t<tr class=\"".$this->rowHeaderClass."\">\n".$header."\t</tr>\n\t</thead>\n" : '').$table."</table>\n";
367
+            if ($this->formElementType) {
368
+                $table= "\n".'<form id="'.$this->formName.'" name="'.$this->formName.'" action="'.$this->formAction.'" method="POST">'.$table;
369
+            }
370
+            if (strlen($this->pageNav) > 1) {//changed to display the pagination if exists.
371
+                /* commented this part because of cookie
372 372
 				$table .= '<div id="max-display-records" ><select style="display:inline" onchange="javascript:updatePageSize(this[this.selectedIndex].value);">';
373 373
 				$pageSizes= array (10, 25, 50, 100, 250);
374 374
 				for ($i= 0; $i < count($pageSizes); $i ++) {
@@ -379,12 +379,12 @@  discard block
 block discarded – undo
379 379
 
380 380
 				$table .= '</select>'.$_lang["pagination_table_perpage"].'</div>';
381 381
 				*/
382
-				$table .= '<div id="pagination" class="paginate">'.$_lang["pagination_table_gotopage"].'<ul>'.$this->pageNav.'</ul></div>';
383
-				//$table .= '<script language="javascript">function updatePageSize(size){window.location = \''.$this->prepareLink($linkpage).'pageSize=\'+size;}</script>';
382
+                $table .= '<div id="pagination" class="paginate">'.$_lang["pagination_table_gotopage"].'<ul>'.$this->pageNav.'</ul></div>';
383
+                //$table .= '<script language="javascript">function updatePageSize(size){window.location = \''.$this->prepareLink($linkpage).'pageSize=\'+size;}</script>';
384 384
 
385
-			}
386
-			if ($this->allOption) {
387
-				$table .= '
385
+            }
386
+            if ($this->allOption) {
387
+                $table .= '
388 388
 <script language="javascript">
389 389
 	toggled = 0;
390 390
 	function clickAll() {
@@ -397,134 +397,134 @@  discard block
 block discarded – undo
397 397
 		toggled = (toggled?0:1);
398 398
 	}
399 399
 </script>';
400
-			}
401
-			if ($this->formElementType) {
402
-				if ($this->extra) {
403
-					$table.= "\n".$this->extra."\n";
404
-				}
405
-				$table.= "\n".'</form>'."\n";
406
-			}
407
-			return $table;
408
-		}
409
-	}
410
-
411
-	/**
412
-	 * Generates optional paging navigation controls for the table.
413
-	 *
414
-	 * @param $numRecords The number of records to show per page.
415
-	 * @param $qs An optional query string to be appended to the paging links
416
-	 */
417
-	function createPagingNavigation($numRecords, $qs='') {
418
-		global $_lang;
419
-		$currentPage= (is_numeric($_GET['page']) ? $_GET['page'] : 1);
420
-		$numPages= ceil($numRecords / MAX_DISPLAY_RECORDS_NUM);
421
-		if ($numPages > 1) {
422
-			$currentURL= empty($qs)? '': '?'.$qs;
423
-			if ($currentPage > 6) {
424
-				$nav .= $this->createPageLink($currentURL, 1, $_lang["pagination_table_first"]);
425
-			}
426
-			if ($currentPage != 1) {
427
-				$nav .= $this->createPageLink($currentURL, $currentPage -1, '&lt;&lt;');
428
-			}
429
-			$offset= -4 + ($currentPage < 5 ? (5 - $currentPage) : 0);
430
-			$i= 1;
431
-			while ($i < 10 && ($currentPage + $offset <= $numPages)) {
432
-				if ($currentPage == $currentPage + $offset)
433
-					$nav .= $this->createPageLink($currentURL, $currentPage + $offset, $currentPage + $offset, true);
434
-				else
435
-					$nav .= $this->createPageLink($currentURL, $currentPage + $offset, $currentPage + $offset);
436
-				$i ++;
437
-				$offset ++;
438
-			}
439
-			if ($currentPage < $numPages) {
440
-				$nav .= $this->createPageLink($currentURL, $currentPage +1, '&gt;&gt;');
441
-			}
442
-			if ($currentPage != $numPages) {
443
-				$nav .= $this->createPageLink($currentURL, $numPages, $_lang["pagination_table_last"]);
444
-			}
445
-		}
446
-		$this->pageNav= ' '.$nav;
447
-	}
448
-
449
-	/**
450
-	 * Creates an individual page link for the paging navigation.
451
-	 *
452
-	 * @param $link The link for the page, defaulted to the current document.
453
-	 * @param $pageNum The page number of the link.
454
-	 * @param $displayText The text of the link.
455
-	 * @param $currentPage Indicates if the link is to the current page.
456
-	 * @param $qs And optional query string to be appended to the link.
457
-	 */
458
-	function createPageLink($link='', $pageNum, $displayText, $currentPage=false, $qs='') {
459
-		global $modx;
460
-		$orderBy= !empty($_GET['orderby'])? '&orderby=' . $_GET['orderby']: '';
461
-		$orderDir= !empty($_GET['orderdir'])? '&orderdir=' . $_GET['orderdir']: '';
462
-		if (!empty($qs)) $qs= "?$qs";
463
-		$link= empty($link)? $modx->makeUrl($modx->documentIdentifier, $modx->documentObject['alias'], $qs . "page=$pageNum$orderBy$orderDir"): $this->prepareLink($link) . "page=$pageNum";
464
-		$nav .= '<li'.($currentPage? ' class="currentPage"': '').'><a'.($currentPage? ' class="currentPage"': '').' href="'.$link.'">'.$displayText.'</a></li>'."\n";
465
-		return $nav;
466
-	}
467
-
468
-	/**
469
-	 * Adds an INPUT form element column to the table.
470
-	 *
471
-	 * @param $value The value attribute of the element.
472
-	 * @param $isChecked Indicates if the checked attribute should apply to the
473
-	 * element.
474
-	 */
475
-	function addFormField($value, $isChecked) {
476
-		if ($this->formElementType) {
477
-			$checked= $isChecked? "checked ": "";
478
-			$field= "\t\t".'<td><input type="'.$this->formElementType.'" name="'. ($this->formElementName ? $this->formElementName : $value).'"  value="'.$value.'" '.$checked.'/></td>'."\n";
479
-		}
480
-		return $field;
481
-	}
482
-
483
-	/**
484
-	 * Generates the proper LIMIT clause for queries to retrieve paged results in
485
-	 * a MakeTable $fieldsArray.
486
-	 */
487
-	function handlePaging() {
488
-		$offset= (is_numeric($_GET['page']) && $_GET['page'] > 0) ? $_GET['page'] - 1 : 0;
489
-		$limitClause= ' LIMIT '. ($offset * MAX_DISPLAY_RECORDS_NUM).', '.MAX_DISPLAY_RECORDS_NUM;
490
-		return $limitClause;
491
-	}
492
-
493
-	/**
494
-	 * Generates the SORT BY clause for queries used to retrieve a MakeTable
495
-	 * $fieldsArray
496
-	 *
497
-	 * @param $natural_order If true, the results are returned in natural order.
498
-	 */
499
-	function handleSorting($natural_order=false) {
500
-		$orderByClause= '';
501
-		if (!$natural_order) {
502
-			$orderby= !empty($_GET['orderby'])? $_GET['orderby']: "id";
503
-			$orderdir= !empty($_GET['orderdir'])? $_GET['orderdir']: "DESC";
504
-			$orderbyClause= !empty($orderby)? ' ORDER BY ' . $orderby . ' ' . $orderdir . ' ': "";
505
-		}
506
-		return $orderbyClause;
507
-	}
508
-
509
-	/**
510
-	 * Generates a link to order by a specific $fieldsArray key; use to generate
511
-	 * sort by links in the MakeTable $fieldHeadingsArray values.
512
-	 *
513
-	 * @param $key The $fieldsArray key for the column to sort by.
514
-	 * @param $text The text for the link (e.g. table column header).
515
-	 * @param $qs An optional query string to append to the order by link.
516
-	 */
517
-	function prepareOrderByLink($key, $text, $qs='') {
518
-		global $modx;
519
-		if (!empty($_GET['orderdir'])) {
520
-			$orderDir= strtolower($_GET['orderdir'])=='desc'? '&orderdir=asc': '&orderdir=desc';
521
-		} else {
522
-			$orderDir= '&orderdir=asc';
523
-		}
524
-		if (!empty($qs)) {
525
-			if (!strrpos($qs, '&')==strlen($qs)-1) $qs.= '&';
526
-		}
527
-		return '<a href="[~'.$modx->documentIdentifier.'~]?'.$qs.'orderby='.$key.$orderDir.'">'.$text.'</a>';
528
-	}
400
+            }
401
+            if ($this->formElementType) {
402
+                if ($this->extra) {
403
+                    $table.= "\n".$this->extra."\n";
404
+                }
405
+                $table.= "\n".'</form>'."\n";
406
+            }
407
+            return $table;
408
+        }
409
+    }
410
+
411
+    /**
412
+     * Generates optional paging navigation controls for the table.
413
+     *
414
+     * @param $numRecords The number of records to show per page.
415
+     * @param $qs An optional query string to be appended to the paging links
416
+     */
417
+    function createPagingNavigation($numRecords, $qs='') {
418
+        global $_lang;
419
+        $currentPage= (is_numeric($_GET['page']) ? $_GET['page'] : 1);
420
+        $numPages= ceil($numRecords / MAX_DISPLAY_RECORDS_NUM);
421
+        if ($numPages > 1) {
422
+            $currentURL= empty($qs)? '': '?'.$qs;
423
+            if ($currentPage > 6) {
424
+                $nav .= $this->createPageLink($currentURL, 1, $_lang["pagination_table_first"]);
425
+            }
426
+            if ($currentPage != 1) {
427
+                $nav .= $this->createPageLink($currentURL, $currentPage -1, '&lt;&lt;');
428
+            }
429
+            $offset= -4 + ($currentPage < 5 ? (5 - $currentPage) : 0);
430
+            $i= 1;
431
+            while ($i < 10 && ($currentPage + $offset <= $numPages)) {
432
+                if ($currentPage == $currentPage + $offset)
433
+                    $nav .= $this->createPageLink($currentURL, $currentPage + $offset, $currentPage + $offset, true);
434
+                else
435
+                    $nav .= $this->createPageLink($currentURL, $currentPage + $offset, $currentPage + $offset);
436
+                $i ++;
437
+                $offset ++;
438
+            }
439
+            if ($currentPage < $numPages) {
440
+                $nav .= $this->createPageLink($currentURL, $currentPage +1, '&gt;&gt;');
441
+            }
442
+            if ($currentPage != $numPages) {
443
+                $nav .= $this->createPageLink($currentURL, $numPages, $_lang["pagination_table_last"]);
444
+            }
445
+        }
446
+        $this->pageNav= ' '.$nav;
447
+    }
448
+
449
+    /**
450
+     * Creates an individual page link for the paging navigation.
451
+     *
452
+     * @param $link The link for the page, defaulted to the current document.
453
+     * @param $pageNum The page number of the link.
454
+     * @param $displayText The text of the link.
455
+     * @param $currentPage Indicates if the link is to the current page.
456
+     * @param $qs And optional query string to be appended to the link.
457
+     */
458
+    function createPageLink($link='', $pageNum, $displayText, $currentPage=false, $qs='') {
459
+        global $modx;
460
+        $orderBy= !empty($_GET['orderby'])? '&orderby=' . $_GET['orderby']: '';
461
+        $orderDir= !empty($_GET['orderdir'])? '&orderdir=' . $_GET['orderdir']: '';
462
+        if (!empty($qs)) $qs= "?$qs";
463
+        $link= empty($link)? $modx->makeUrl($modx->documentIdentifier, $modx->documentObject['alias'], $qs . "page=$pageNum$orderBy$orderDir"): $this->prepareLink($link) . "page=$pageNum";
464
+        $nav .= '<li'.($currentPage? ' class="currentPage"': '').'><a'.($currentPage? ' class="currentPage"': '').' href="'.$link.'">'.$displayText.'</a></li>'."\n";
465
+        return $nav;
466
+    }
467
+
468
+    /**
469
+     * Adds an INPUT form element column to the table.
470
+     *
471
+     * @param $value The value attribute of the element.
472
+     * @param $isChecked Indicates if the checked attribute should apply to the
473
+     * element.
474
+     */
475
+    function addFormField($value, $isChecked) {
476
+        if ($this->formElementType) {
477
+            $checked= $isChecked? "checked ": "";
478
+            $field= "\t\t".'<td><input type="'.$this->formElementType.'" name="'. ($this->formElementName ? $this->formElementName : $value).'"  value="'.$value.'" '.$checked.'/></td>'."\n";
479
+        }
480
+        return $field;
481
+    }
482
+
483
+    /**
484
+     * Generates the proper LIMIT clause for queries to retrieve paged results in
485
+     * a MakeTable $fieldsArray.
486
+     */
487
+    function handlePaging() {
488
+        $offset= (is_numeric($_GET['page']) && $_GET['page'] > 0) ? $_GET['page'] - 1 : 0;
489
+        $limitClause= ' LIMIT '. ($offset * MAX_DISPLAY_RECORDS_NUM).', '.MAX_DISPLAY_RECORDS_NUM;
490
+        return $limitClause;
491
+    }
492
+
493
+    /**
494
+     * Generates the SORT BY clause for queries used to retrieve a MakeTable
495
+     * $fieldsArray
496
+     *
497
+     * @param $natural_order If true, the results are returned in natural order.
498
+     */
499
+    function handleSorting($natural_order=false) {
500
+        $orderByClause= '';
501
+        if (!$natural_order) {
502
+            $orderby= !empty($_GET['orderby'])? $_GET['orderby']: "id";
503
+            $orderdir= !empty($_GET['orderdir'])? $_GET['orderdir']: "DESC";
504
+            $orderbyClause= !empty($orderby)? ' ORDER BY ' . $orderby . ' ' . $orderdir . ' ': "";
505
+        }
506
+        return $orderbyClause;
507
+    }
508
+
509
+    /**
510
+     * Generates a link to order by a specific $fieldsArray key; use to generate
511
+     * sort by links in the MakeTable $fieldHeadingsArray values.
512
+     *
513
+     * @param $key The $fieldsArray key for the column to sort by.
514
+     * @param $text The text for the link (e.g. table column header).
515
+     * @param $qs An optional query string to append to the order by link.
516
+     */
517
+    function prepareOrderByLink($key, $text, $qs='') {
518
+        global $modx;
519
+        if (!empty($_GET['orderdir'])) {
520
+            $orderDir= strtolower($_GET['orderdir'])=='desc'? '&orderdir=asc': '&orderdir=desc';
521
+        } else {
522
+            $orderDir= '&orderdir=asc';
523
+        }
524
+        if (!empty($qs)) {
525
+            if (!strrpos($qs, '&')==strlen($qs)-1) $qs.= '&';
526
+        }
527
+        return '<a href="[~'.$modx->documentIdentifier.'~]?'.$qs.'orderby='.$key.$orderDir.'">'.$text.'</a>';
528
+    }
529 529
 
530 530
 }
Please login to merge, or discard this patch.
Braces   +83 added lines, -42 removed lines patch added patch discarded remove patch
@@ -8,7 +8,8 @@  discard block
 block discarded – undo
8 8
  *
9 9
  * @author Jason Coward <[email protected]> (MODX)
10 10
  */
11
-class MakeTable {
11
+class MakeTable
12
+{
12 13
 	var $actionField;
13 14
 	var $cellAction;
14 15
 	var $linkAction;
@@ -31,7 +32,8 @@  discard block
 block discarded – undo
31 32
 	var $columnWidths;
32 33
 	var $selectedValues;
33 34
 
34
-	function __construct() {
35
+	function __construct()
36
+	{
35 37
 		$this->fieldHeaders= array();
36 38
 		$this->excludeFields= array();
37 39
 		$this->actionField= '';
@@ -58,7 +60,8 @@  discard block
 block discarded – undo
58 60
 	 *
59 61
 	 * @param $value A URL to execute when table cells are clicked.
60 62
 	 */
61
-	function setCellAction($value) {
63
+	function setCellAction($value)
64
+	{
62 65
 		$this->cellAction= $this->prepareLink($value);
63 66
 	}
64 67
 
@@ -67,7 +70,8 @@  discard block
 block discarded – undo
67 70
 	 *
68 71
 	 * @param $value A URL to execute when text within table cells are clicked.
69 72
 	 */
70
-	function setLinkAction($value) {
73
+	function setLinkAction($value)
74
+	{
71 75
 		$this->linkAction= $this->prepareLink($value);
72 76
 	}
73 77
 
@@ -76,7 +80,8 @@  discard block
 block discarded – undo
76 80
 	 *
77 81
 	 * @param $value A valid width attribute for the HTML TABLE tag
78 82
 	 */
79
-	function setTableWidth($value) {
83
+	function setTableWidth($value)
84
+	{
80 85
 		$this->tableWidth= $value;
81 86
 	}
82 87
 
@@ -85,7 +90,8 @@  discard block
 block discarded – undo
85 90
 	 *
86 91
 	 * @param $value A class for the main HTML TABLE.
87 92
 	 */
88
-	function setTableClass($value) {
93
+	function setTableClass($value)
94
+	{
89 95
 		$this->tableClass= $value;
90 96
 	}
91 97
 
@@ -94,7 +100,8 @@  discard block
 block discarded – undo
94 100
 	 *
95 101
 	 * @param $value A class for the main HTML TABLE.
96 102
 	 */
97
-	function setTableID($value) {
103
+	function setTableID($value)
104
+	{
98 105
 		$this->tableID= $value;
99 106
 	}
100 107
 
@@ -103,7 +110,8 @@  discard block
 block discarded – undo
103 110
 	 *
104 111
 	 * @param $value A class for the table header row.
105 112
 	 */
106
-	function setRowHeaderClass($value) {
113
+	function setRowHeaderClass($value)
114
+	{
107 115
 		$this->rowHeaderClass= $value;
108 116
 	}
109 117
 
@@ -112,7 +120,8 @@  discard block
 block discarded – undo
112 120
 	 *
113 121
 	 * @param $value A class for the table header row.
114 122
 	 */
115
-	function setThHeaderClass($value) {
123
+	function setThHeaderClass($value)
124
+	{
116 125
 		$this->thClass= $value;
117 126
 	}
118 127
 
@@ -121,7 +130,8 @@  discard block
 block discarded – undo
121 130
 	 *
122 131
 	 * @param $value A class for the column header row.
123 132
 	 */
124
-	function setColumnHeaderClass($value) {
133
+	function setColumnHeaderClass($value)
134
+	{
125 135
 		$this->columnHeaderClass= $value;
126 136
 	}
127 137
 
@@ -130,7 +140,8 @@  discard block
 block discarded – undo
130 140
 	 *
131 141
 	 * @param $value A class for regular table rows.
132 142
 	 */
133
-	function setRowRegularClass($value) {
143
+	function setRowRegularClass($value)
144
+	{
134 145
 		$this->rowRegularClass= $value;
135 146
 	}
136 147
 
@@ -139,7 +150,8 @@  discard block
 block discarded – undo
139 150
 	 *
140 151
 	 * @param $value A class for alternate table rows.
141 152
 	 */
142
-	function setRowAlternateClass($value) {
153
+	function setRowAlternateClass($value)
154
+	{
143 155
 		$this->rowAlternateClass= $value;
144 156
 	}
145 157
 
@@ -148,7 +160,8 @@  discard block
 block discarded – undo
148 160
 	 *
149 161
 	 * @param $value Indicates the INPUT form element type attribute.
150 162
 	 */
151
-	function setFormElementType($value) {
163
+	function setFormElementType($value)
164
+	{
152 165
 		$this->formElementType= $value;
153 166
 	}
154 167
 
@@ -157,7 +170,8 @@  discard block
 block discarded – undo
157 170
 	 *
158 171
 	 * @param $value Indicates the INPUT form element name attribute.
159 172
 	 */
160
-	function setFormElementName($value) {
173
+	function setFormElementName($value)
174
+	{
161 175
 		$this->formElementName= $value;
162 176
 	}
163 177
 
@@ -167,7 +181,8 @@  discard block
 block discarded – undo
167 181
 	 *
168 182
 	 * @param $value Indicates the FORM name attribute.
169 183
 	 */
170
-	function setFormName($value) {
184
+	function setFormName($value)
185
+	{
171 186
 		$this->formName= $value;
172 187
 	}
173 188
 
@@ -176,7 +191,8 @@  discard block
 block discarded – undo
176 191
 	 *
177 192
 	 * @param $value Indicates the FORM action attribute.
178 193
 	 */
179
-	function setFormAction($value) {
194
+	function setFormAction($value)
195
+	{
180 196
 		$this->formAction= $value;
181 197
 	}
182 198
 
@@ -185,7 +201,8 @@  discard block
 block discarded – undo
185 201
 	 *
186 202
 	 * @param $value An Array of field keys to exclude from the table.
187 203
 	 */
188
-	function setExcludeFields($value) {
204
+	function setExcludeFields($value)
205
+	{
189 206
 		$this->excludeFields= $value;
190 207
 	}
191 208
 
@@ -194,7 +211,8 @@  discard block
 block discarded – undo
194 211
 	 *
195 212
 	 * @param $value 'ODD' or 'EVEN' to indicate the alternate row scheme.
196 213
 	 */
197
-	function setRowAlternatingScheme($value) {
214
+	function setRowAlternatingScheme($value)
215
+	{
198 216
 		$this->rowAlternatingScheme= $value;
199 217
 	}
200 218
 
@@ -204,7 +222,8 @@  discard block
 block discarded – undo
204 222
 	 *
205 223
 	 * @param $value The key of the field to add as a query string parameter.
206 224
 	 */
207
-	function setActionFieldName($value) {
225
+	function setActionFieldName($value)
226
+	{
208 227
 		$this->actionField= $value;
209 228
 	}
210 229
 
@@ -214,7 +233,8 @@  discard block
 block discarded – undo
214 233
 	 * @param $value An Array of column widths in the order of the keys in the
215 234
 	 * 			source table array.
216 235
 	 */
217
-	function setColumnWidths($widthArray) {
236
+	function setColumnWidths($widthArray)
237
+	{
218 238
 		$this->columnWidths= $widthArray;
219 239
 	}
220 240
 
@@ -223,7 +243,8 @@  discard block
 block discarded – undo
223 243
 	 *
224 244
 	 * @param $value Indicates the INPUT form element type attribute.
225 245
 	 */
226
-	function setSelectedValues($valueArray) {
246
+	function setSelectedValues($valueArray)
247
+	{
227 248
 		$this->selectedValues= $valueArray;
228 249
 	}
229 250
 
@@ -233,7 +254,8 @@  discard block
 block discarded – undo
233 254
 	 *
234 255
 	 * @param $value A string of additional content.
235 256
 	 */
236
-	function setExtra($value) {
257
+	function setExtra($value)
258
+	{
237 259
 		$this->extra= $value;
238 260
 	}
239 261
 
@@ -242,7 +264,8 @@  discard block
 block discarded – undo
242 264
 	 *
243 265
 	 * @param $columnPosition The index of the column to get the width for.
244 266
 	 */
245
-	function getColumnWidth($columnPosition) {
267
+	function getColumnWidth($columnPosition)
268
+	{
246 269
 		$currentWidth= '';
247 270
 		if (is_array($this->columnWidths)) {
248 271
 			$currentWidth= $this->columnWidths[$columnPosition] ? ' width="'.$this->columnWidths[$columnPosition].'" ' : '';
@@ -255,7 +278,8 @@  discard block
 block discarded – undo
255 278
 	 *
256 279
 	 * @param $value The position of the current row being rendered.
257 280
 	 */
258
-	function determineRowClass($position) {
281
+	function determineRowClass($position)
282
+	{
259 283
 		switch ($this->rowAlternatingScheme) {
260 284
 			case 'ODD' :
261 285
 				$modRemainder= 1;
@@ -278,7 +302,8 @@  discard block
 block discarded – undo
278 302
 	 *
279 303
 	 * @param $value Indicates the INPUT form element type attribute.
280 304
 	 */
281
-	function getCellAction($currentActionFieldValue) {
305
+	function getCellAction($currentActionFieldValue)
306
+	{
282 307
 		if ($this->cellAction) {
283 308
 			$cellAction= ' onClick="javascript:window.location=\''.$this->cellAction.$this->actionField.'='.urlencode($currentActionFieldValue).'\'" ';
284 309
 		}
@@ -291,7 +316,8 @@  discard block
 block discarded – undo
291 316
 	 * @param $currentActionFieldValue The value to be applied to the link action.
292 317
 	 * @param $value The value of the cell.
293 318
 	 */
294
-	function createCellText($currentActionFieldValue, $value) {
319
+	function createCellText($currentActionFieldValue, $value)
320
+	{
295 321
 		$cell .= $value;
296 322
 		if ($this->linkAction) {
297 323
 			$cell= '<a href="'.$this->linkAction.$this->actionField.'='.urlencode($currentActionFieldValue).'">'.$cell.'</a>';
@@ -303,7 +329,8 @@  discard block
 block discarded – undo
303 329
 	 * Sets an option to generate a check all link when checkbox is indicated
304 330
 	 * as the table formElementType.
305 331
 	 */
306
-	function setAllOption() {
332
+	function setAllOption()
333
+	{
307 334
 		$this->allOption= 1;
308 335
 	}
309 336
 
@@ -312,7 +339,8 @@  discard block
 block discarded – undo
312 339
 	 *
313 340
 	 * @param $value Indicates the INPUT form element type attribute.
314 341
 	 */
315
-	function prepareLink($link) {
342
+	function prepareLink($link)
343
+	{
316 344
 		if (strstr($link, '?')) {
317 345
 			$end= '&';
318 346
 		} else {
@@ -331,7 +359,8 @@  discard block
 block discarded – undo
331 359
 	 * the $fieldsArray where the values represent the alt heading content
332 360
 	 * for each column.
333 361
 	 */
334
-	function create($fieldsArray, $fieldHeadersArray=array(),$linkpage="") {
362
+	function create($fieldsArray, $fieldHeadersArray=array(),$linkpage="")
363
+	{
335 364
 	global $_lang;
336 365
 		if (is_array($fieldsArray)) {
337 366
 			$i= 0;
@@ -367,7 +396,8 @@  discard block
 block discarded – undo
367 396
 			if ($this->formElementType) {
368 397
 				$table= "\n".'<form id="'.$this->formName.'" name="'.$this->formName.'" action="'.$this->formAction.'" method="POST">'.$table;
369 398
 			}
370
-			if (strlen($this->pageNav) > 1) {//changed to display the pagination if exists.
399
+			if (strlen($this->pageNav) > 1) {
400
+//changed to display the pagination if exists.
371 401
 				/* commented this part because of cookie
372 402
 				$table .= '<div id="max-display-records" ><select style="display:inline" onchange="javascript:updatePageSize(this[this.selectedIndex].value);">';
373 403
 				$pageSizes= array (10, 25, 50, 100, 250);
@@ -414,7 +444,8 @@  discard block
 block discarded – undo
414 444
 	 * @param $numRecords The number of records to show per page.
415 445
 	 * @param $qs An optional query string to be appended to the paging links
416 446
 	 */
417
-	function createPagingNavigation($numRecords, $qs='') {
447
+	function createPagingNavigation($numRecords, $qs='')
448
+	{
418 449
 		global $_lang;
419 450
 		$currentPage= (is_numeric($_GET['page']) ? $_GET['page'] : 1);
420 451
 		$numPages= ceil($numRecords / MAX_DISPLAY_RECORDS_NUM);
@@ -429,10 +460,11 @@  discard block
 block discarded – undo
429 460
 			$offset= -4 + ($currentPage < 5 ? (5 - $currentPage) : 0);
430 461
 			$i= 1;
431 462
 			while ($i < 10 && ($currentPage + $offset <= $numPages)) {
432
-				if ($currentPage == $currentPage + $offset)
433
-					$nav .= $this->createPageLink($currentURL, $currentPage + $offset, $currentPage + $offset, true);
434
-				else
435
-					$nav .= $this->createPageLink($currentURL, $currentPage + $offset, $currentPage + $offset);
463
+				if ($currentPage == $currentPage + $offset) {
464
+									$nav .= $this->createPageLink($currentURL, $currentPage + $offset, $currentPage + $offset, true);
465
+				} else {
466
+									$nav .= $this->createPageLink($currentURL, $currentPage + $offset, $currentPage + $offset);
467
+				}
436 468
 				$i ++;
437 469
 				$offset ++;
438 470
 			}
@@ -455,11 +487,14 @@  discard block
 block discarded – undo
455 487
 	 * @param $currentPage Indicates if the link is to the current page.
456 488
 	 * @param $qs And optional query string to be appended to the link.
457 489
 	 */
458
-	function createPageLink($link='', $pageNum, $displayText, $currentPage=false, $qs='') {
490
+	function createPageLink($link='', $pageNum, $displayText, $currentPage=false, $qs='')
491
+	{
459 492
 		global $modx;
460 493
 		$orderBy= !empty($_GET['orderby'])? '&orderby=' . $_GET['orderby']: '';
461 494
 		$orderDir= !empty($_GET['orderdir'])? '&orderdir=' . $_GET['orderdir']: '';
462
-		if (!empty($qs)) $qs= "?$qs";
495
+		if (!empty($qs)) {
496
+		    $qs= "?$qs";
497
+		}
463 498
 		$link= empty($link)? $modx->makeUrl($modx->documentIdentifier, $modx->documentObject['alias'], $qs . "page=$pageNum$orderBy$orderDir"): $this->prepareLink($link) . "page=$pageNum";
464 499
 		$nav .= '<li'.($currentPage? ' class="currentPage"': '').'><a'.($currentPage? ' class="currentPage"': '').' href="'.$link.'">'.$displayText.'</a></li>'."\n";
465 500
 		return $nav;
@@ -472,7 +507,8 @@  discard block
 block discarded – undo
472 507
 	 * @param $isChecked Indicates if the checked attribute should apply to the
473 508
 	 * element.
474 509
 	 */
475
-	function addFormField($value, $isChecked) {
510
+	function addFormField($value, $isChecked)
511
+	{
476 512
 		if ($this->formElementType) {
477 513
 			$checked= $isChecked? "checked ": "";
478 514
 			$field= "\t\t".'<td><input type="'.$this->formElementType.'" name="'. ($this->formElementName ? $this->formElementName : $value).'"  value="'.$value.'" '.$checked.'/></td>'."\n";
@@ -484,7 +520,8 @@  discard block
 block discarded – undo
484 520
 	 * Generates the proper LIMIT clause for queries to retrieve paged results in
485 521
 	 * a MakeTable $fieldsArray.
486 522
 	 */
487
-	function handlePaging() {
523
+	function handlePaging()
524
+	{
488 525
 		$offset= (is_numeric($_GET['page']) && $_GET['page'] > 0) ? $_GET['page'] - 1 : 0;
489 526
 		$limitClause= ' LIMIT '. ($offset * MAX_DISPLAY_RECORDS_NUM).', '.MAX_DISPLAY_RECORDS_NUM;
490 527
 		return $limitClause;
@@ -496,7 +533,8 @@  discard block
 block discarded – undo
496 533
 	 *
497 534
 	 * @param $natural_order If true, the results are returned in natural order.
498 535
 	 */
499
-	function handleSorting($natural_order=false) {
536
+	function handleSorting($natural_order=false)
537
+	{
500 538
 		$orderByClause= '';
501 539
 		if (!$natural_order) {
502 540
 			$orderby= !empty($_GET['orderby'])? $_GET['orderby']: "id";
@@ -514,7 +552,8 @@  discard block
 block discarded – undo
514 552
 	 * @param $text The text for the link (e.g. table column header).
515 553
 	 * @param $qs An optional query string to append to the order by link.
516 554
 	 */
517
-	function prepareOrderByLink($key, $text, $qs='') {
555
+	function prepareOrderByLink($key, $text, $qs='')
556
+	{
518 557
 		global $modx;
519 558
 		if (!empty($_GET['orderdir'])) {
520 559
 			$orderDir= strtolower($_GET['orderdir'])=='desc'? '&orderdir=asc': '&orderdir=desc';
@@ -522,7 +561,9 @@  discard block
 block discarded – undo
522 561
 			$orderDir= '&orderdir=asc';
523 562
 		}
524 563
 		if (!empty($qs)) {
525
-			if (!strrpos($qs, '&')==strlen($qs)-1) $qs.= '&';
564
+			if (!strrpos($qs, '&')==strlen($qs)-1) {
565
+			    $qs.= '&';
566
+			}
526 567
 		}
527 568
 		return '<a href="[~'.$modx->documentIdentifier.'~]?'.$qs.'orderby='.$key.$orderDir.'">'.$text.'</a>';
528 569
 	}
Please login to merge, or discard this patch.
manager/includes/extenders/message.quit.inc.php 1 patch
Braces   +3 added lines, -1 removed lines patch added patch discarded remove patch
@@ -3,7 +3,9 @@
 block discarded – undo
3 3
  * Message Quit Template
4 4
  *
5 5
  */
6
-if(IN_MANAGER_MODE!="true" && IN_PARSER_MODE!="true") die("<b>INCLUDE ACCESS ERROR</b><br /><br />Direct access to this file prohibited.");
6
+if(IN_MANAGER_MODE!="true" && IN_PARSER_MODE!="true") {
7
+    die("<b>INCLUDE ACCESS ERROR</b><br /><br />Direct access to this file prohibited.");
8
+}
7 9
 
8 10
 $parsedMessageString = "
9 11
 <html><head><title>EVO Content Manager $version &raquo; $release_date</title>
Please login to merge, or discard this patch.