XOOPS / XoopsCore

htdocs/class/captcha/xoopscaptchamethod.php

Braces +5 added lines, -4 removed lines

@@ -75,12 +75,13 @@
      */
     public function loadConfig($name = '')
     {
-        if (!is_object($this->handler))
-            $this->config = array();
-        else
-            $this->config = empty($name)
+        if (!is_object($this->handler)) {
+                    $this->config = array();
+        } else {
+                    $this->config = empty($name)
                 ? $this->handler->config
                 : array_merge($this->handler->config, $this->handler->loadConfig($name));
+        }
     }
 
     /**

htdocs/class/captcha/recaptcha/recaptchalib.php

Indentation +7 added lines, -8 removed lines

@@ -105,7 +105,6 @@ 
  * @param string $pubkey A public key for reCAPTCHA
  * @param string $error The error given by reCAPTCHA (optional, default is null)
  * @param boolean $use_ssl Should the request be made over ssl? (optional, default is false)
-
  * @return string - The HTML to be embedded in the user's form.
  */
 function recaptcha_get_html($pubkey, $error = null, $use_ssl = false)
@@ -172,13 +171,13 @@ 
     }
 
     $response = _recaptcha_http_post(RECAPTCHA_VERIFY_SERVER,
-		"/recaptcha/api/verify",
-		array(
-			   'privatekey' => $privkey,
-			   'remoteip' => $remoteip,
-			   'challenge' => $challenge,
-			   'response' => $response
-		   ) + $extra_params);
+        "/recaptcha/api/verify",
+        array(
+                'privatekey' => $privkey,
+                'remoteip' => $remoteip,
+                'challenge' => $challenge,
+                'response' => $response
+            ) + $extra_params);
 
     $answers = explode("\n", $response [1]);
     $recaptcha_response = new ReCaptchaResponse();

Spacing +9 added lines, -9 removed lines

@@ -50,7 +50,7 @@ 
 {
     $req = "";
     foreach ($data as $key => $value) {
-        $req .= $key . '=' . urlencode(stripslashes($value)) . '&';
+        $req .= $key.'='.urlencode(stripslashes($value)).'&';
     }
 
     // Cut the last '&'
@@ -76,7 +76,7 @@ 
     $http_request = "POST $path HTTP/1.0\r\n";
     $http_request .= "Host: $host\r\n";
     $http_request .= "Content-Type: application/x-www-form-urlencoded;\r\n";
-    $http_request .= "Content-Length: " . strlen($req) . "\r\n";
+    $http_request .= "Content-Length: ".strlen($req)."\r\n";
     $http_request .= "User-Agent: reCAPTCHA/PHP\r\n";
     $http_request .= "\r\n";
     $http_request .= $req;
@@ -122,12 +122,12 @@ 
 
     $errorpart = "";
     if ($error) {
-        $errorpart = "&error=" . $error;
+        $errorpart = "&error=".$error;
     }
-    return '<script type="text/javascript" src="' . $server . '/challenge?k=' . $pubkey . $errorpart . '"></script>
+    return '<script type="text/javascript" src="'.$server.'/challenge?k='.$pubkey.$errorpart.'"></script>
 
     <noscript>
-        <iframe src="' . $server . '/noscript?k=' . $pubkey . $errorpart . '" height="300" width="500" frameborder="0"></iframe><br/>
+        <iframe src="' . $server.'/noscript?k='.$pubkey.$errorpart.'" height="300" width="500" frameborder="0"></iframe><br/>
         <textarea name="recaptcha_challenge_field" rows="3" cols="40"></textarea>
         <input type="hidden" name="recaptcha_response_field" value="manual_challenge"/>
     </noscript>';
@@ -203,7 +203,7 @@ 
  */
 function recaptcha_get_signup_url($domain = null, $appname = null)
 {
-    return "https://www.google.com/recaptcha/admin/create?" . _recaptcha_qsencode(
+    return "https://www.google.com/recaptcha/admin/create?"._recaptcha_qsencode(
         array('domains' => $domain, 'app' => $appname));
 }
 
@@ -256,13 +256,13 @@ 
 function recaptcha_mailhide_url($pubkey, $privkey, $email)
 {
     if ($pubkey == '' || $pubkey == null || $privkey == "" || $privkey == null) {
-        die ("To use reCAPTCHA Mailhide, you have to sign up for a public and private key, " . "you can do so at <a href='http://www.google.com/recaptcha/mailhide/apikey'>http://www.google.com/recaptcha/mailhide/apikey</a>");
+        die ("To use reCAPTCHA Mailhide, you have to sign up for a public and private key, "."you can do so at <a href='http://www.google.com/recaptcha/mailhide/apikey'>http://www.google.com/recaptcha/mailhide/apikey</a>");
     }
 
     $ky = pack('H*', $privkey);
     $cryptmail = _recaptcha_aes_encrypt($email, $ky);
 
-    return "http://www.google.com/recaptcha/mailhide/d?k=" . $pubkey . "&c=" . _recaptcha_mailhide_urlbase64($cryptmail);
+    return "http://www.google.com/recaptcha/mailhide/d?k=".$pubkey."&c="._recaptcha_mailhide_urlbase64($cryptmail);
 }
 
 /**
@@ -305,5 +305,5 @@ 
     $emailparts = _recaptcha_mailhide_email_parts($email);
     $url = recaptcha_mailhide_url($pubkey, $privkey, $email);
 
-    return htmlentities($emailparts[0]) . "<a href='" . htmlentities($url) . "' onclick=\"window.open('" . htmlentities($url) . "', '', 'toolbar=0,scrollbars=0,location=0,statusbar=0,menubar=0,resizable=0,width=500,height=300'); return false;\" title=\"Reveal this e-mail address\">...</a>@" . htmlentities($emailparts [1]);
+    return htmlentities($emailparts[0])."<a href='".htmlentities($url)."' onclick=\"window.open('".htmlentities($url)."', '', 'toolbar=0,scrollbars=0,location=0,statusbar=0,menubar=0,resizable=0,width=500,height=300'); return false;\" title=\"Reveal this e-mail address\">...</a>@".htmlentities($emailparts [1]);
 }

htdocs/class/captcha/image.php

Indentation +2 added lines, -2 removed lines

@@ -63,7 +63,7 @@ 
      */
     public function render()
     {
-		$xoops_url = \XoopsBaseConfig::get('url');
+        $xoops_url = \XoopsBaseConfig::get('url');
         $js = "<script type='text/javascript'>
                 function xoops_captcha_refresh(imgId)
                 {
@@ -88,7 +88,7 @@ 
      */
     public function loadImage()
     {
-		$xoops_url = \XoopsBaseConfig::get('url');
+        $xoops_url = \XoopsBaseConfig::get('url');
         return '<img id="' . ($this->config["name"]) . '" src="' . $xoops_url . '/class/captcha/image/scripts/image.php" onclick=\'this.src="' . $xoops_url . '/class/captcha/image/scripts/image.php?refresh="+Math.random()' . '\' style="cursor: pointer; vertical-align: middle;" alt="" />';
     }
 }

Spacing +13 added lines, -13 removed lines

@@ -40,15 +40,15 @@ 
             return false;
         } else {
             $required_functions = array(
-                'imagecreatetruecolor' ,
-                'imagecolorallocate' ,
-                'imagefilledrectangle' ,
-                'imagejpeg' ,
-                'imagedestroy' ,
+                'imagecreatetruecolor',
+                'imagecolorallocate',
+                'imagefilledrectangle',
+                'imagejpeg',
+                'imagedestroy',
                 'imageftbbox');
             foreach ($required_functions as $func) {
                 if (!function_exists($func)) {
-                    trigger_error('Function ' . $func . ' is not defined', E_USER_WARNING);
+                    trigger_error('Function '.$func.' is not defined', E_USER_WARNING);
                     return false;
                 }
             }
@@ -67,18 +67,18 @@ 
         $js = "<script type='text/javascript'>
                 function xoops_captcha_refresh(imgId)
                 {
-                    xoopsGetElementById(imgId).src = '" . $xoops_url . "/class/captcha/image/scripts/image.php?refresh='+Math.random();
+                    xoopsGetElementById(imgId).src = '" . $xoops_url."/class/captcha/image/scripts/image.php?refresh='+Math.random();
                 }
                 </script>";
         $image = $this->loadImage();
-        $image .= "<br /><a href=\"javascript: xoops_captcha_refresh('" . ($this->config['name']) . "')\">" . XoopsLocale::CLICK_TO_REFRESH_IMAGE_IF_NOT_CLEAR . "</a>";
-        $input = '<input type="text" name="' . $this->config['name'] . '" id="' . $this->config['name'] . '" size="' . $this->config['num_chars'] . '" maxlength="' . $this->config['num_chars'] . '" value="" required>';
+        $image .= "<br /><a href=\"javascript: xoops_captcha_refresh('".($this->config['name'])."')\">".XoopsLocale::CLICK_TO_REFRESH_IMAGE_IF_NOT_CLEAR."</a>";
+        $input = '<input type="text" name="'.$this->config['name'].'" id="'.$this->config['name'].'" size="'.$this->config['num_chars'].'" maxlength="'.$this->config['num_chars'].'" value="" required>';
         $rule = XoopsLocale::INPUT_LETTERS_IN_THE_IMAGE;
-        $rule .= '<br />' . (empty($this->config['casesensitive']) ? XoopsLocale::CODE_IS_CASE_INSENSITIVE : XoopsLocale::CODE_IS_CASE_SENSITIVE);
+        $rule .= '<br />'.(empty($this->config['casesensitive']) ? XoopsLocale::CODE_IS_CASE_INSENSITIVE : XoopsLocale::CODE_IS_CASE_SENSITIVE);
         if (!empty($this->config['maxattempts'])) {
-            $rule .= '<br />' . sprintf(XoopsLocale::F_MAXIMUM_ATTEMPTS, $this->config['maxattempts']);
+            $rule .= '<br />'.sprintf(XoopsLocale::F_MAXIMUM_ATTEMPTS, $this->config['maxattempts']);
         }
-        return $js . $image . '<br /><br />' . $input . '<br />' . $rule;
+        return $js.$image.'<br /><br />'.$input.'<br />'.$rule;
     }
 
     /**
@@ -89,6 +89,6 @@ 
     public function loadImage()
     {
 		$xoops_url = \XoopsBaseConfig::get('url');
-        return '<img id="' . ($this->config["name"]) . '" src="' . $xoops_url . '/class/captcha/image/scripts/image.php" onclick=\'this.src="' . $xoops_url . '/class/captcha/image/scripts/image.php?refresh="+Math.random()' . '\' style="cursor: pointer; vertical-align: middle;" alt="" />';
+        return '<img id="'.($this->config["name"]).'" src="'.$xoops_url.'/class/captcha/image/scripts/image.php" onclick=\'this.src="'.$xoops_url.'/class/captcha/image/scripts/image.php?refresh="+Math.random()'.'\' style="cursor: pointer; vertical-align: middle;" alt="" />';
     }
 }

htdocs/class/captcha/recaptcha.php

Indentation +1 added lines, -1 removed lines

@@ -66,7 +66,7 @@
         include_once __DIR__ . '/recaptcha/recaptchalib.php';
         if (!empty($_POST['recaptcha_response_field'])) {
             $resp = recaptcha_check_answer($this->config['private_key'], $_SERVER['REMOTE_ADDR'],
-				$_POST['recaptcha_challenge_field'], $_POST['recaptcha_response_field']);
+                $_POST['recaptcha_challenge_field'], $_POST['recaptcha_response_field']);
             if (!$resp->is_valid) {
                 $this->handler->message[] = $resp->error;
             } else {

Spacing +4 added lines, -4 removed lines

@@ -42,11 +42,11 @@ 
      */
     public function render()
     {
-        include_once __DIR__ . '/recaptcha/recaptchalib.php';
+        include_once __DIR__.'/recaptcha/recaptchalib.php';
         $form = "<script type=\"text/javascript\">
             var RecaptchaOptions = {
-            theme : '" . (empty($this->config['theme']) ? '' : $this->config['theme']) . "',
-            lang : '" . (empty($this->config['lang']) ? '' : $this->config['lang']) . "'
+            theme : '" . (empty($this->config['theme']) ? '' : $this->config['theme'])."',
+            lang : '" . (empty($this->config['lang']) ? '' : $this->config['lang'])."'
             };
             </script>";
         $public_key = empty($this->config['public_key']) ? '' : $this->config['public_key'];
@@ -63,7 +63,7 @@ 
     public function verify($sessionName = null)
     {
         $is_valid = false;
-        include_once __DIR__ . '/recaptcha/recaptchalib.php';
+        include_once __DIR__.'/recaptcha/recaptchalib.php';
         if (!empty($_POST['recaptcha_response_field'])) {
             $resp = recaptcha_check_answer($this->config['private_key'], $_SERVER['REMOTE_ADDR'],
 				$_POST['recaptcha_challenge_field'], $_POST['recaptcha_response_field']);

htdocs/class/captcha/text.php

Indentation +2 added lines, -2 removed lines

@@ -33,8 +33,8 @@
     public function render()
     {
         $form = $this->loadText() . '&nbsp;&nbsp; <input type="text" name="' . $this->config['name']
-			. '" id="' . $this->config['name'] . '" size="' . $this->config['num_chars']
-			. '" maxlength="' . $this->config['num_chars'] . '" value="" />';
+            . '" id="' . $this->config['name'] . '" size="' . $this->config['num_chars']
+            . '" maxlength="' . $this->config['num_chars'] . '" value="" />';
         $form .= '<br />' . XoopsLocale::INPUT_RESULT_FROM_EXPRESSION;
         if (!empty($this->config['maxattempts'])) {
             $form .= '<br />' . sprintf(XoopsLocale::F_MAXIMUM_ATTEMPTS, $this->config['maxattempts']);

Spacing +6 added lines, -6 removed lines

@@ -32,12 +32,12 @@ 
      */
     public function render()
     {
-        $form = $this->loadText() . '&nbsp;&nbsp; <input type="text" name="' . $this->config['name']
-			. '" id="' . $this->config['name'] . '" size="' . $this->config['num_chars']
-			. '" maxlength="' . $this->config['num_chars'] . '" value="" />';
-        $form .= '<br />' . XoopsLocale::INPUT_RESULT_FROM_EXPRESSION;
+        $form = $this->loadText().'&nbsp;&nbsp; <input type="text" name="'.$this->config['name']
+			. '" id="'.$this->config['name'].'" size="'.$this->config['num_chars']
+			. '" maxlength="'.$this->config['num_chars'].'" value="" />';
+        $form .= '<br />'.XoopsLocale::INPUT_RESULT_FROM_EXPRESSION;
         if (!empty($this->config['maxattempts'])) {
-            $form .= '<br />' . sprintf(XoopsLocale::F_MAXIMUM_ATTEMPTS, $this->config['maxattempts']);
+            $form .= '<br />'.sprintf(XoopsLocale::F_MAXIMUM_ATTEMPTS, $this->config['maxattempts']);
         }
         return $form;
     }
@@ -58,6 +58,6 @@ 
             $expression = "{$val_a} + {$val_b} = ?";
             $this->code = $val_a + $val_b;
         }
-        return '<span style="font-style: normal; font-weight: bold; font-size: 100%; font-color: #333; border: 1px solid #333; padding: 1px 5px;">' . $expression . '</span>';
+        return '<span style="font-style: normal; font-weight: bold; font-size: 100%; font-color: #333; border: 1px solid #333; padding: 1px 5px;">'.$expression.'</span>';
     }
 }

htdocs/class/captcha/image/scripts/imageclass.php

Switch Indentation +9 added lines, -9 removed lines

@@ -297,15 +297,15 @@
         if ($RandImage = $this->loadBackground()) {
             $ImageType = @getimagesize($RandImage);
             switch (@$ImageType[2]) {
-            case 1:
-                $BackgroundImage = imagecreatefromgif($RandImage);
-                break;
-            case 2:
-                $BackgroundImage = imagecreatefromjpeg($RandImage);
-                break;
-            case 3:
-                $BackgroundImage = imagecreatefrompng($RandImage);
-                break;
+                case 1:
+                    $BackgroundImage = imagecreatefromgif($RandImage);
+                    break;
+                case 2:
+                    $BackgroundImage = imagecreatefromjpeg($RandImage);
+                    break;
+                case 3:
+                    $BackgroundImage = imagecreatefrompng($RandImage);
+                    break;
             }
         }
         if (isset($BackgroundImage) && !empty($BackgroundImage)) {

Spacing +4 added lines, -4 removed lines

@@ -140,7 +140,7 @@ 
     {
         if ($this->invalid) {
             header("Content-type: image/gif");
-            readfile($this->xoops_root_path . "/images/subject/icon2.gif");
+            readfile($this->xoops_root_path."/images/subject/icon2.gif");
             return false;
         }
 
@@ -165,7 +165,7 @@ 
             return $items;
         }
         ;
-        $file_path = $this->xoops_root_path . "/class/captcha/image/{$name}";
+        $file_path = $this->xoops_root_path."/class/captcha/image/{$name}";
         $files = \Xoops\Core\Lists\File::getList($file_path);
         foreach ($files as $item) {
             if (empty($extension) || preg_match("/(\.{$extension})$/i", $item)) {
@@ -236,7 +236,7 @@ 
     public function loadFont()
     {
         $fonts = $this->getList("fonts", "ttf");
-        $this->font = $this->xoops_root_path . "/class/captcha/image/fonts/" . $fonts[array_rand($fonts)];
+        $this->font = $this->xoops_root_path."/class/captcha/image/fonts/".$fonts[array_rand($fonts)];
     }
 
     /**
@@ -282,7 +282,7 @@ 
     {
         $RandBackground = null;
         if ($backgrounds = $this->getList("backgrounds", "(gif|jpg|png)")) {
-            $RandBackground = $this->xoops_root_path . "/class/captcha/image/backgrounds/" . $backgrounds[array_rand($backgrounds)];
+            $RandBackground = $this->xoops_root_path."/class/captcha/image/backgrounds/".$backgrounds[array_rand($backgrounds)];
         }
         return $RandBackground;
     }

htdocs/class/captcha/image/scripts/image.php

Spacing +1 added lines, -1 removed lines

@@ -24,7 +24,7 @@
  * @since     2.6.0
  */
 
-require dirname(dirname(dirname(dirname(__DIR__)))) . '/mainfile.php';
+require dirname(dirname(dirname(dirname(__DIR__)))).'/mainfile.php';
 
 require_once 'imageclass.php';
 

htdocs/class/XoopsBaseConfig.php

Spacing +18 added lines, -18 removed lines

@@ -38,14 +38,14 @@ 
     final private function __construct($config)
     {
         if (!class_exists('XoopsLoad', false)) {
-            include __DIR__ . '/xoopsload.php';
+            include __DIR__.'/xoopsload.php';
         }
         if (is_string($config)) {
             $yamlString = file_get_contents($config);
             if ($yamlString === false) {
                 throw new \Exception('XoopsBaseConfig failed to load configuration.');
             }
-            $loaderPath = $this->extractLibPath($yamlString) . '/vendor/autoload.php';
+            $loaderPath = $this->extractLibPath($yamlString).'/vendor/autoload.php';
             if (file_exists($loaderPath)) {
                 include_once $loaderPath;
             }
@@ -231,11 +231,11 @@ 
             XOOPS_URL :
             ((isset($_SERVER['HTTPS']) && $_SERVER['HTTPS'] === "on") ? 'https://' : 'http://')
             . $_SERVER['SERVER_NAME']
-            . (($_SERVER['SERVER_PORT'] != '80') ? ':' . $_SERVER['SERVER_PORT'] : '');
+            . (($_SERVER['SERVER_PORT'] != '80') ? ':'.$_SERVER['SERVER_PORT'] : '');
 
-        $parts = parse_url($url . '/');
+        $parts = parse_url($url.'/');
         $host = isset($parts['host']) ? $parts['host'] : $_SERVER['SERVER_NAME'];
-        $host = ($host==='localhost') ? '' : $host;
+        $host = ($host === 'localhost') ? '' : $host;
         $urlpath = isset($parts['path']) ? $parts['path'] : '/';
 
         $libpath = self::defineDefault('XOOPS_PATH');
@@ -248,21 +248,21 @@ 
             'trust-path' => $libpath,
             'url' => $url,
             'prot' => self::defineDefault('XOOPS_PROT'),
-            'asset-path' => $path . '/assets',
-            'asset-url' => $url . '/assets',
-            'themes-path' => $path .'/themes',
-            'themes-url' => $url . '/themes',
-            'adminthemes-path' => $path . '/modules/system/themes',
-            'adminthemes-url' => $url . '/modules/system/themes',
-            'media-path' => $path . '/media',
-            'media-url' => $url . '/media',
-            'uploads-path' => $path . '/uploads',
-            'uploads-url' => $url . '/uploads',
+            'asset-path' => $path.'/assets',
+            'asset-url' => $url.'/assets',
+            'themes-path' => $path.'/themes',
+            'themes-url' => $url.'/themes',
+            'adminthemes-path' => $path.'/modules/system/themes',
+            'adminthemes-url' => $url.'/modules/system/themes',
+            'media-path' => $path.'/media',
+            'media-url' => $url.'/media',
+            'uploads-path' => $path.'/uploads',
+            'uploads-url' => $url.'/uploads',
             'cookie-domain' => $host,
             'cookie-path' => $urlpath,
-            'smarty-cache' => $varpath . '/caches/smarty_cache',
-            'smarty-compile' => $varpath . '/caches/smarty_compile',
-            'smarty-xoops-plugins' => $libpath . '/smarty/xoops_plugins',
+            'smarty-cache' => $varpath.'/caches/smarty_cache',
+            'smarty-compile' => $varpath.'/caches/smarty_compile',
+            'smarty-xoops-plugins' => $libpath.'/smarty/xoops_plugins',
             'db-type' => self::defineDefault('XOOPS_DB_TYPE'),
             'db-charset' => 'utf8',
             'db-prefix' => self::defineDefault('XOOPS_DB_PREFIX'),

htdocs/class/pagenav.php

Spacing +10 added lines, -10 removed lines

@@ -68,9 +68,9 @@ 
         $this->current = (int)($current_start);
         $this->extra = $extra_arg;
         if ($extra_arg != '' && (substr($extra_arg, - 5) !== '&' || substr($extra_arg, - 1) !== '&')) {
-            $this->extra = '&' . $extra_arg;
+            $this->extra = '&'.$extra_arg;
         }
-        $this->url = $_SERVER['PHP_SELF'] . '?' . trim($start_name) . '=';
+        $this->url = $_SERVER['PHP_SELF'].'?'.trim($start_name).'=';
     }
 
     /**
@@ -98,7 +98,7 @@ 
                 $prev = $this->current - $this->perpage;
                 if ($prev >= 0) {
                     $xoops->tpl()->assign('prev_text', $prev_text);
-                    $xoops->tpl()->assign('prev_url', $this->url . $prev . $this->extra);
+                    $xoops->tpl()->assign('prev_url', $this->url.$prev.$this->extra);
                 }
                 $last = 0;
                 $last_text = '';
@@ -120,10 +120,10 @@ 
                             $nav['active'] = 0;
                             $last = 1;
                             $last_text = $counter;
-                            $last_url = $this->url . (($counter - 1) * $this->perpage) . $this->extra;
+                            $last_url = $this->url.(($counter - 1) * $this->perpage).$this->extra;
                         } else {
                             $nav['text'] = $counter;
-                            $nav['url'] = $this->url . (($counter - 1) * $this->perpage) . $this->extra;
+                            $nav['url'] = $this->url.(($counter - 1) * $this->perpage).$this->extra;
                             $nav['active'] = 1;
                         }
                         if ($counter == 1 && $current_page > 1 + $offset) {
@@ -132,7 +132,7 @@ 
                             $nav['active'] = 0;
                             $first = 1;
                             $first_text = $counter;
-                            $first_url = $this->url . (($counter - 1) * $this->perpage) . $this->extra;
+                            $first_url = $this->url.(($counter - 1) * $this->perpage).$this->extra;
                         }
                     }
                     $xoops->tpl()->appendByRef('xo_nav', $nav);
@@ -149,7 +149,7 @@ 
                 $next = $this->current + $this->perpage;
                 if ($this->total > $next) {
                     $xoops->tpl()->assign('next_text', $next_text);
-                    $xoops->tpl()->assign('next_url', $this->url . $next . $this->extra);
+                    $xoops->tpl()->assign('next_url', $this->url.$next.$this->extra);
                 }
             }
         }
@@ -166,7 +166,7 @@ 
         }
         $xoops->tpl()->assign('size', $size);
         
-        $xoops->tpl()->assign('align', ' pagination-' . $align);
+        $xoops->tpl()->assign('align', ' pagination-'.$align);
         $xoops->tpl()->assign('pagination_nav', true);
         $ret = $xoops->tpl()->fetch('module:system/system_pagenav.tpl');
         $xoops->tpl()->assign('xo_nav');
@@ -193,7 +193,7 @@ 
             $current_page = (int)(floor(($this->current + $this->perpage) / $this->perpage));
             while ($counter <= $total_pages) {
                 $select['text'] = $counter;
-                $select['value'] = $this->url . (($counter - 1) * $this->perpage) . $this->extra;
+                $select['value'] = $this->url.(($counter - 1) * $this->perpage).$this->extra;
                 if ($counter == $current_page) {
                     $select['selected'] = 1;
                 } else {
@@ -207,7 +207,7 @@ 
         $xoops->tpl()->assign('onchange', "location=this.options[this.options.selectedIndex].value;");
         $xoops->tpl()->assign('pagination_select', true);
         $xoops->tpl()->assign('showbutton', $showbutton);
-        $xoops->tpl()->assign('align', ' pagination-' . $align);
+        $xoops->tpl()->assign('align', ' pagination-'.$align);
         $ret = $xoops->tpl()->fetch('module:system/system_pagenav.tpl');
         $xoops->tpl()->assign('xo_select');
         return $ret;