XOOPS / XoopsCore

xoops_lib/Xoops/Locale/AbstractLocale.php

Spacing +1 added lines, -1 removed lines

@@ -171,7 +171,7 @@
     public static function substr($str, $start, $length, $ellipsis = '…')
     {
         $str2 = mb_strcut($str, $start, $length - strlen($ellipsis));
-        return $str2 . (mb_strlen($str)-$start != mb_strlen($str2) ? $ellipsis : '');
+        return $str2.(mb_strlen($str) - $start != mb_strlen($str2) ? $ellipsis : '');
     }
 
     /**

xoops_lib/Xoops/Html/Img.php

Spacing +1 added lines, -1 removed lines

@@ -41,7 +41,7 @@
      */
     public function render()
     {
-        $tag = '<img ' . $this->renderAttributeString() . ' />';
+        $tag = '<img '.$this->renderAttributeString().' />';
 
         return $tag;
     }

xoops_lib/Xoops/Html/Menu/Render/BreadCrumb.php

Spacing +2 added lines, -2 removed lines

@@ -60,13 +60,13 @@
                 $anchorEnd = '';
                 $liClass = ' class="active"';
                 if ($item->has('link')) {
-                    $anchorStart = '<a href="' . $this->xoops->url($item->get('link')) . '">';
+                    $anchorStart = '<a href="'.$this->xoops->url($item->get('link')).'">';
                     $anchorEnd = '</a>';
                     $liClass = '';
                 }
                 $caption = $item->get('caption', '');
                 $icon = $item->has('icon') ?
-                    '<span class="' . $item->get('icon') . '" aria-hidden="true"></span> ' : '';
+                    '<span class="'.$item->get('icon').'" aria-hidden="true"></span> ' : '';
                 $renderedItems .= "<li{$liClass}>{$anchorStart}{$icon}{$caption}{$anchorEnd}</li>";
                 break;
             case Item::TYPE_LIST:

xoops_lib/Xoops/Html/Menu/Render/DropDownButton.php

Spacing +5 added lines, -5 removed lines

@@ -38,10 +38,10 @@ 
         $dropdown = $menu->get('dropdown', 'dropdown');
         $renderedMenu = "<div class=\"{$dropdown}\">\n";
         $class = $menu->get('class', 'btn btn-default dropdown-toggle');
-        $id = ($menu->has('id')) ? ' id="' . $menu->get('id') . '"' : '';
-        $labeledId = ($menu->has('id')) ? ' aria-labelledby="' . $menu->get('id') . '"' : '';
+        $id = ($menu->has('id')) ? ' id="'.$menu->get('id').'"' : '';
+        $labeledId = ($menu->has('id')) ? ' aria-labelledby="'.$menu->get('id').'"' : '';
         $caption = $menu->get('caption', '');
-        $icon = $menu->has('icon') ? '<span class="' . $menu->get('icon') . '" aria-hidden="true"></span> ' : '';
+        $icon = $menu->has('icon') ? '<span class="'.$menu->get('icon').'" aria-hidden="true"></span> ' : '';
         $renderedMenu .= <<<EOT
 <button class="{$class}" type="button"{$id} data-toggle="dropdown" aria-haspopup="true" aria-expanded="true">
 {$icon}{$caption} <span class="caret"></span>
@@ -74,13 +74,13 @@ 
                 $anchorEnd = '';
                 $liClass = ' class="active"';
                 if ($item->has('link')) {
-                    $anchorStart = '<a href="' . $this->xoops->url($item->get('link')) . '">';
+                    $anchorStart = '<a href="'.$this->xoops->url($item->get('link')).'">';
                     $anchorEnd = '</a>';
                     $liClass = '';
                 }
                 $caption = $item->get('caption', '');
                 $icon = $item->has('icon') ?
-                    '<span class="' . $item->get('icon') . '" aria-hidden="true"></span> ' : '';
+                    '<span class="'.$item->get('icon').'" aria-hidden="true"></span> ' : '';
                 $renderedItems .= "<li{$liClass}>{$anchorStart}{$icon}{$caption}{$anchorEnd}</li>";
                 break;
             case Item::TYPE_LIST:

xoops_lib/Xoops/Auth/Provisioning.php

Spacing +2 added lines, -2 removed lines

@@ -197,7 +197,7 @@ 
 
             return $newuser;
         } else {
-            $xoops->redirect(\XoopsBaseConfig::get('url') . '/user.php', 5, $newuser->getHtmlErrors());
+            $xoops->redirect(\XoopsBaseConfig::get('url').'/user.php', 5, $newuser->getHtmlErrors());
         }
 
         return $ret;
@@ -225,7 +225,7 @@ 
         if ($member_handler->insertUser($xoopsUser)) {
             return $xoopsUser;
         } else {
-            $xoops->redirect(\XoopsBaseConfig::get('url') . '/user.php', 5, $xoopsUser->getHtmlErrors());
+            $xoops->redirect(\XoopsBaseConfig::get('url').'/user.php', 5, $xoopsUser->getHtmlErrors());
         }
 
         return $ret;

xoops_lib/Xoops/Auth/AuthAbstract.php

Spacing +2 added lines, -2 removed lines

@@ -94,10 +94,10 @@
         if ($xoops->getConfig('debug_mode') == 1 || $xoops->getConfig('debug_mode') == 2) {
             if (!empty($this->errors)) {
                 foreach ($this->errors as $errstr) {
-                    $ret .= $errstr . '<br/>';
+                    $ret .= $errstr.'<br/>';
                 }
             } else {
-                $ret .= \XoopsLocale::NONE . '<br />';
+                $ret .= \XoopsLocale::NONE.'<br />';
             }
             $ret .= sprintf(\XoopsLocale::F_USING_AUTHENTICATION_METHOD, $this->auth_method);
         } else {

xoops_lib/Xoops/Auth/Factory.php

Spacing +2 added lines, -2 removed lines

@@ -35,7 +35,7 @@ 
      *
      * @return AuthAbstract|bool Reference to the only instance of authentication class
      */
-    public static function getAuthConnection($uname, $_force=false)
+    public static function getAuthConnection($uname, $_force = false)
     {
         $xoops = \Xoops::getInstance();
         static $auth_instance;
@@ -52,7 +52,7 @@ 
                 $xoops_auth_method = 'xoops';
             }
 
-            $class = '\Xoops\Auth\\' . ucfirst($xoops_auth_method);
+            $class = '\Xoops\Auth\\'.ucfirst($xoops_auth_method);
             if (!class_exists($class)) {
                 trigger_error(\XoopsLocale::EF_CLASS_NOT_FOUND, E_USER_ERROR);
                 return false;

xoops_lib/Xoops/Auth/Ads.php

Spacing +2 added lines, -2 removed lines

@@ -85,7 +85,7 @@ 
                     return false;
                 }
             } else {
-                $this->setErrors(ldap_errno($this->ds), ldap_err2str(ldap_errno($this->ds)) . '(' . $userUPN . ')');
+                $this->setErrors(ldap_errno($this->ds), ldap_err2str(ldap_errno($this->ds)).'('.$userUPN.')');
             }
         } else {
             $this->setErrors(0, \XoopsLocale::E_CANNOT_CONNECT_TO_SERVER);
@@ -106,7 +106,7 @@ 
      */
     public function getUPN($uname)
     {
-        $userDN = $uname . '@' . $this->ldap_domain_name;
+        $userDN = $uname.'@'.$this->ldap_domain_name;
         return $userDN;
     }
 }

xoops_lib/Xoops/Auth/Ldap.php

Spacing +5 added lines, -5 removed lines

@@ -162,7 +162,7 @@ 
                 // We load the Xoops User database
                 return $this->loadXoopsUser($userDN, $uname, $pwd);
             } else {
-                $this->setErrors(ldap_errno($this->ds), ldap_err2str(ldap_errno($this->ds)) . '(' . $userDN . ')');
+                $this->setErrors(ldap_errno($this->ds), ldap_err2str(ldap_errno($this->ds)).'('.$userDN.')');
             }
         } else {
             $this->setErrors(0, \XoopsLocale::E_CANNOT_CONNECT_TO_SERVER);
@@ -187,7 +187,7 @@ 
             if (!ldap_bind($this->ds, $this->ldap_manager_dn, stripslashes($this->ldap_manager_pass))) {
                 $this->setErrors(
                     ldap_errno($this->ds),
-                    ldap_err2str(ldap_errno($this->ds)) . '(' . $this->ldap_manager_dn . ')'
+                    ldap_err2str(ldap_errno($this->ds)).'('.$this->ldap_manager_dn.')'
                 );
 
                 return false;
@@ -206,7 +206,7 @@ 
                 ));
             }
         } else {
-            $userDN = $this->ldap_loginldap_attr . '=' . $uname . ',' . $this->ldap_base_dn;
+            $userDN = $this->ldap_loginldap_attr.'='.$uname.','.$this->ldap_base_dn;
         }
 
         return $userDN;
@@ -224,7 +224,7 @@ 
         if ($this->ldap_filter_person != '') {
             $filter = str_replace('@@loginname@@', $uname, $this->ldap_filter_person);
         } else {
-            $filter = $this->ldap_loginldap_attr . '=' . $uname;
+            $filter = $this->ldap_loginldap_attr.'='.$uname;
         }
 
         return $filter;
@@ -248,7 +248,7 @@ 
         if ($entries['count'] > 0) {
             $xoopsUser = $provisHandler->sync($entries[0], $uname, $pwd);
         } else {
-            $this->setErrors(0, sprintf('loadXoopsUser - ' . \XoopsLocale::EF_ENTRY_NOT_READ, $userdn));
+            $this->setErrors(0, sprintf('loadXoopsUser - '.\XoopsLocale::EF_ENTRY_NOT_READ, $userdn));
         }
 
         return $xoopsUser;