thorsten / phpMyFAQ

phpmyfaq/admin/ajax.section.php

Unused Use Statements -1 removed lines

@@ -17,7 +17,6 @@
 
 use phpMyFAQ\Filter;
 use phpMyFAQ\Permission\LargePermission;
-use phpMyFAQ\User;
 
 if (!defined('IS_VALID_PHPMYFAQ')) {
     $protocol = 'http';

Indentation +1 added lines, -1 removed lines

@@ -62,7 +62,7 @@
         foreach ($memberList as $single_member) {
             $group = $user->perm->getGroupData($single_member);
             $members[] = array('group_id' => $group['group_id'],
-                               'name' => $group['name'] );
+                                'name' => $group['name'] );
         }
         echo json_encode($members);
     }

Spacing +1 added lines, -1 removed lines

@@ -62,7 +62,7 @@
         foreach ($memberList as $single_member) {
             $group = $user->perm->getGroupData($single_member);
             $members[] = array('group_id' => $group['group_id'],
-                               'name' => $group['name'] );
+                               'name' => $group['name']);
         }
         echo json_encode($members);
     }

phpmyfaq/ask.php

Unused Use Statements +2 added lines, -2 removed lines

@@ -19,8 +19,8 @@
 
 use phpMyFAQ\Captcha;
 use phpMyFAQ\Filter;
-USE phpMyFAQ\Helper\CategoryHelper as HelperCategory;
-USE phpMyFAQ\Helper\CaptchaHelper;
+use phpMyFAQ\Helper\CategoryHelper as HelperCategory;
+use phpMyFAQ\Helper\CaptchaHelper;
 
 if (!defined('IS_VALID_PHPMYFAQ')) {
     $protocol = 'http';

phpmyfaq/faq.php

Unused Use Statements -1 removed lines

@@ -32,7 +32,6 @@
 use phpMyFAQ\Linkverifier;
 use phpMyFAQ\Rating;
 use phpMyFAQ\Relation;
-use phpMyFAQ\Search\Resultset as SearchResultSet;
 use phpMyFAQ\Strings;
 use phpMyFAQ\Tags;
 use phpMyFAQ\User\CurrentUser;

Spacing +8 added lines, -8 removed lines

@@ -44,7 +44,7 @@ 
     if (isset($_SERVER['HTTPS']) && strtoupper($_SERVER['HTTPS']) === 'ON') {
         $protocol = 'https';
     }
-    header('Location: ' . $protocol . '://' . $_SERVER['HTTP_HOST'] . dirname($_SERVER['SCRIPT_NAME']));
+    header('Location: '.$protocol.'://'.$_SERVER['HTTP_HOST'].dirname($_SERVER['SCRIPT_NAME']));
     exit();
 }
 
@@ -133,7 +133,7 @@ 
             if (strpos($_url, '&') === false) {
                 $_link = str_replace('&', '&', $_link);
             }
-            $oLink = new Link(Link::getSystemRelativeUri() . $_link, $faqConfig);
+            $oLink = new Link(Link::getSystemRelativeUri().$_link, $faqConfig);
             $oLink->itemTitle = $oLink->tooltip = $_title;
             $newFaqPath = $oLink->toString();
             $answer = str_replace($_url, $newFaqPath, $answer);
@@ -152,7 +152,7 @@ 
             $att->getFilename());
     }
     if (count($attList) > 0) {
-        $answer .= '<p>' . $PMF_LANG['msgAttachedFiles'] . ' ' . Strings::substr($outstr, 0, -2) . '</p>';
+        $answer .= '<p>'.$PMF_LANG['msgAttachedFiles'].' '.Strings::substr($outstr, 0, -2).'</p>';
     }
 }
 
@@ -189,7 +189,7 @@ 
         Link::getSystemRelativeUri('index.php'),
         $recordId,
         $lang,
-        $PMF_LANG['ad_entry_edit_1'] . ' ' . $PMF_LANG['ad_entry_edit_2']
+        $PMF_LANG['ad_entry_edit_1'].' '.$PMF_LANG['ad_entry_edit_2']
     );
 }
 
@@ -213,7 +213,7 @@ 
         '%',
         '%%',
         Link::getSystemRelativeUri('index.php')
-    ) . 'index.php?%saction=translate&amp;cat=%s&amp;id=%d&amp;srclang=%s',
+    ).'index.php?%saction=translate&amp;cat=%s&amp;id=%d&amp;srclang=%s',
     $sids,
     $currentCategory,
     $recordId,
@@ -259,7 +259,7 @@ 
         'writeContent',
         'tagsAvailable',
         [
-            'renderTags' => $PMF_LANG['msg_tags'] . ': ' . $faqTagging->getAllLinkTagsById($recordId),
+            'renderTags' => $PMF_LANG['msg_tags'].': '.$faqTagging->getAllLinkTagsById($recordId),
         ]
     );
 }
@@ -303,7 +303,7 @@ 
         'baseHref' => $faqSystem->getSystemUri($faqConfig),
         'writeRubrik' => $categoryName,
         'solution_id' => $faq->faqRecord['solution_id'],
-        'solution_id_link' => Link::getSystemRelativeUri() . '?solution_id=' . $faq->faqRecord['solution_id'],
+        'solution_id_link' => Link::getSystemRelativeUri().'?solution_id='.$faq->faqRecord['solution_id'],
         'writeThema' => $question,
         'writeContent' => $answer,
         'writeDateMsg' => $date->format($faq->faqRecord['date']),
@@ -322,7 +322,7 @@ 
                 '%',
                 '%%',
                 Link::getSystemRelativeUri('index.php')
-            ) . 'index.php?%saction=savevoting',
+            ).'index.php?%saction=savevoting',
             $sids
         ),
         'saveVotingID' => $recordId,

phpmyfaq/src/phpMyFAQ/Attachment/Factory.php

Doc Comments +1 added lines, -1 removed lines

@@ -105,7 +105,7 @@
      *
      * @throws Exception
      *
-     * @return array
+     * @return integer|null
      */
     public static function fetchByRecordId(Configuration $config, $recordId): array
     {

phpmyfaq/src/phpMyFAQ/Category/CategoryNews.php

Doc Comments -1 removed lines

@@ -95,7 +95,6 @@
      * Removes the news $newsId from all categories.
      * Returns true on success, otherwise false.
      *
-     * @param int $categoryId
      * @return bool
      */
     public function removeNewsFromAllCategories($newsId)

phpmyfaq/src/phpMyFAQ/Category/Entity.php

Doc Comments +1 added lines, -2 removed lines

@@ -236,7 +236,7 @@ 
     }
 
     /**
-     * @return string
+     * @return integer
      */
     public function getUserId()
     {
@@ -264,7 +264,6 @@ 
     }
 
     /**
-     * @param $showHome $active
      *
      * @return Entity
      */

Spacing +3 added lines, -3 removed lines

@@ -160,7 +160,7 @@ 
      */
     public function getId()
     {
-        return (int) $this->id;
+        return (int)$this->id;
     }
 
     /**
@@ -220,7 +220,7 @@ 
      */
     public function getParentId()
     {
-        return (int) $this->parentId;
+        return (int)$this->parentId;
     }
 
     /**
@@ -240,7 +240,7 @@ 
      */
     public function getUserId()
     {
-        return (int) $this->userId;
+        return (int)$this->userId;
     }
 
     /**

phpmyfaq/src/phpMyFAQ/Db/Pgsql.php

Doc Comments +5 added lines, -4 removed lines

@@ -66,7 +66,7 @@ 
      * @param string $password Password
      * @param string $database Database name
      *
-     * @return bool true, if connected, otherwise false
+     * @return null|boolean true, if connected, otherwise false
      */
     public function connect($host, $user, $password, $database = '')
     {
@@ -133,7 +133,7 @@ 
     /**
      * Fetch a result row as an object.
      *
-     * @param mixed $result
+     * @param resource $result
      *
      * @return mixed
      */
@@ -192,7 +192,7 @@ 
     /**
      * Logs the queries.
      *
-     * @return int
+     * @return string
      */
     public function log()
     {
@@ -203,6 +203,7 @@ 
      * Returns just one row.
      *
      * @param  string
+     * @param string $query
      *
      * @return string
      */
@@ -294,7 +295,7 @@ 
      *
      * @param string $prefix Table prefix
      *
-     * @return array
+     * @return string[]
      */
     public function getTableNames($prefix = '')
     {

phpmyfaq/src/phpMyFAQ/Db/Sqlite3.php

Doc Comments +3 added lines, -3 removed lines

@@ -69,7 +69,7 @@ 
      * @param string
      * @param string
      *
-     * @return bool
+     * @return null|boolean
      */
     public function connect($host, $user, $passwd, $db = '')
     {
@@ -214,7 +214,7 @@ 
     /**
      * Logs the queries.
      *
-     * @return int
+     * @return string
      */
     public function log()
     {
@@ -248,7 +248,7 @@ 
      *
      * @param string $prefix Table prefix
      *
-     * @return array
+     * @return string[]
      */
     public function getTableNames($prefix = '')
     {

Spacing +3 added lines, -3 removed lines

@@ -134,7 +134,7 @@ 
         $return = $result->fetchArray(SQLITE3_ASSOC);
 
         return $return
-            ? (object) $return
+            ? (object)$return
             : null;
     }
 
@@ -184,7 +184,7 @@ 
 
         $result->fetchedByPMF = true;
         while ($row = $result->fetchArray(SQLITE3_ASSOC)) {
-            $ret[] = (object) $row;
+            $ret[] = (object)$row;
         }
 
         return $ret;
@@ -297,7 +297,7 @@ 
      */
     public function nextId($table, $id)
     {
-        $result = (int) $this->conn->querySingle(
+        $result = (int)$this->conn->querySingle(
             sprintf(
                 'SELECT max(%s) AS current_id FROM %s',
                 $id,

phpmyfaq/src/phpMyFAQ/Faq.php

Doc Comments +7 added lines, -7 removed lines

@@ -1279,7 +1279,7 @@ 
      * @param int $recordId Record id
      * @param string $language Language
      *
-     * @return int
+     * @return boolean
      */
     public function addCategoryRelations(Array $categories, $recordId, $language)
     {
@@ -2575,11 +2575,11 @@ 
     /**
      * Build the SQL query for retrieving faq records according to the constraints provided.
      *
-     * @param $queryType
-     * @param $categoryId
-     * @param $bDownwards
-     * @param $lang
-     * @param $date
+     * @param string $queryType
+     * @param integer $categoryId
+     * @param boolean $bDownwards
+     * @param string $lang
+     * @param string $date
      * @param $faqId
      *
      * @return string
@@ -2703,7 +2703,7 @@ 
      * Build a logic sequence, for a WHERE statement, of those category IDs
      * children of the provided category ID, if any.
      *
-     * @param   $nCatid
+     * @param   integer $nCatid
      * @param   $logicOp
      * @param   $oCat
      *

Spacing +62 added lines, -62 removed lines

@@ -408,20 +408,20 @@ 
 
         $result = $this->config->getDb()->query($query);
         $num = $this->config->getDb()->numRows($result);
-        $pages = (int)ceil($num / $numPerPage);
+        $pages = (int)ceil($num/$numPerPage);
 
         if ($page == 1) {
             $first = 0;
         } else {
-            $first = $page * $numPerPage - $numPerPage;
+            $first = $page*$numPerPage - $numPerPage;
         }
 
         if ($num > 0) {
             if ($pages > 1) {
                 $output .= sprintf('<p><strong>%s %s %s</strong></p>',
-                    $this->translation['msgPage'] . $page,
+                    $this->translation['msgPage'].$page,
                     $this->translation['msgVoteFrom'],
-                    $pages . $this->translation['msgPages']);
+                    $pages.$this->translation['msgPages']);
             }
             $output .= '<ul class="phpmyfaq_ul">';
 
@@ -594,20 +594,20 @@ 
         $result = $this->config->getDb()->query($query);
 
         $num = $this->config->getDb()->numRows($result);
-        $pages = ceil($num / $this->config->get('records.numberOfRecordsPerPage'));
+        $pages = ceil($num/$this->config->get('records.numberOfRecordsPerPage'));
 
         if ($page == 1) {
             $first = 0;
         } else {
-            $first = ($page * $this->config->get('records.numberOfRecordsPerPage')) - $this->config->get('records.numberOfRecordsPerPage');
+            $first = ($page*$this->config->get('records.numberOfRecordsPerPage')) - $this->config->get('records.numberOfRecordsPerPage');
         }
 
         if ($num > 0) {
             if ($pages > 1) {
                 $output .= sprintf('<p><strong>%s %s %s</strong></p>',
-                    $this->translation['msgPage'] . $page,
+                    $this->translation['msgPage'].$page,
                     $this->translation['msgVoteFrom'],
-                    $pages . $this->translation['msgPages']);
+                    $pages.$this->translation['msgPages']);
             }
             $output .= '<ul class="phpmyfaq_ul">';
             $counter = 0;
@@ -654,7 +654,7 @@ 
 
                 $lastFaqId = $row->id;
             }
-            $output .= '</ul><span id="totFaqRecords" style="display: none;">' . $num . '</span>';
+            $output .= '</ul><span id="totFaqRecords" style="display: none;">'.$num.'</span>';
         } else {
             return false;
         }
@@ -667,21 +667,21 @@ 
             $vor = $page - 1;
             $next = $page + 1;
             if ($vor != 0) {
-                $url = $sids . '&amp;action=search&amp;tagging_id=' . $taggingId . '&amp;seite=' . $vor;
-                $oLink = new Link(Link::getSystemRelativeUri() . '?' . $url, $this->config);
+                $url = $sids.'&amp;action=search&amp;tagging_id='.$taggingId.'&amp;seite='.$vor;
+                $oLink = new Link(Link::getSystemRelativeUri().'?'.$url, $this->config);
                 $oLink->itemTitle = 'tag';
                 $oLink->text = $this->translation['msgPrevious'];
                 $oLink->tooltip = $this->translation['msgPrevious'];
-                $output .= '[ ' . $oLink->toHtmlAnchor() . ' ]';
+                $output .= '[ '.$oLink->toHtmlAnchor().' ]';
             }
             $output .= ' ';
             if ($next <= $pages) {
-                $url = $sids . '&amp;action=search&amp;tagging_id=' . $taggingId . '&amp;seite=' . $next;
-                $oLink = new Link(Link::getSystemRelativeUri() . '?' . $url, $this->config);
+                $url = $sids.'&amp;action=search&amp;tagging_id='.$taggingId.'&amp;seite='.$next;
+                $oLink = new Link(Link::getSystemRelativeUri().'?'.$url, $this->config);
                 $oLink->itemTitle = 'tag';
                 $oLink->text = $this->translation['msgNext'];
                 $oLink->tooltip = $this->translation['msgNext'];
-                $output .= '[ ' . $oLink->toHtmlAnchor() . ' ]';
+                $output .= '[ '.$oLink->toHtmlAnchor().' ]';
             }
             $output .= '</strong></p>';
         }
@@ -808,7 +808,7 @@ 
             Db::getTablePrefix(),
             Db::getTablePrefix(),
             $faqId,
-            isset($faqRevisionId) ? 'AND revision_id = ' . $faqRevisionId : '',
+            isset($faqRevisionId) ? 'AND revision_id = '.$faqRevisionId : '',
             $faqLanguage,
             ($isAdmin) ? 'AND 1=1' : $this->queryPermission($this->groupSupport)
         );
@@ -901,7 +901,7 @@ 
                 'record_title' => $row->question,
                 'record_preview' => $faqHelper->renderAnswerPreview($row->answer, 25),
                 'record_link' => $oLink->toString(),
-                'record_updated' => Date::createIsoDate($row->updated) . ':00',
+                'record_updated' => Date::createIsoDate($row->updated).':00',
                 'visits' => (int)$visits,
                 'record_created' => $row->created
             ];
@@ -921,7 +921,7 @@ 
     public function addRecord(Array $data, $newRecord = true)
     {
         if ($newRecord) {
-            $recordId = $this->config->getDb()->nextId(Db::getTablePrefix() . 'faqdata', 'id');
+            $recordId = $this->config->getDb()->nextId(Db::getTablePrefix().'faqdata', 'id');
         } else {
             $recordId = $data['id'];
         }
@@ -1464,12 +1464,12 @@ 
                     $where .= ' IN (';
                     $separator = '';
                     foreach ($data as $value) {
-                        $where .= $separator . "'" . $this->config->getDb()->escape($value) . "'";
+                        $where .= $separator."'".$this->config->getDb()->escape($value)."'";
                         $separator = ', ';
                     }
                     $where .= ')';
                 } else {
-                    $where .= " = '" . $this->config->getDb()->escape($data) . "'";
+                    $where .= " = '".$this->config->getDb()->escape($data)."'";
                 }
                 if ($num > 0) {
                     $where .= ' AND ';
@@ -1818,7 +1818,7 @@ 
             AND
                 date_end >= '%s'",
             Db::getTablePrefix(),
-            null == $language ? '' : "AND lang = '" . $language . "'",
+            null == $language ? '' : "AND lang = '".$language."'",
             $now,
             $now
         );
@@ -1905,25 +1905,25 @@ 
                 fdg.group_id AS group_id,
                 fdu.user_id AS user_id
             FROM
-                ' . Db::getTablePrefix() . 'faqvisits fv,
-                ' . Db::getTablePrefix() . 'faqdata fd
+                ' . Db::getTablePrefix().'faqvisits fv,
+                ' . Db::getTablePrefix().'faqdata fd
             LEFT JOIN
-                ' . Db::getTablePrefix() . 'faqcategoryrelations fcr
+                ' . Db::getTablePrefix().'faqcategoryrelations fcr
             ON
                 fd.id = fcr.record_id
             AND
                 fd.lang = fcr.record_lang
             LEFT JOIN
-                ' . Db::getTablePrefix() . 'faqdata_group AS fdg
+                ' . Db::getTablePrefix().'faqdata_group AS fdg
             ON
                 fd.id = fdg.record_id
             LEFT JOIN
-                ' . Db::getTablePrefix() . 'faqdata_user AS fdu
+                ' . Db::getTablePrefix().'faqdata_user AS fdu
             ON
                 fd.id = fdu.record_id
             WHERE
-                    fd.date_start <= \'' . $now . '\'
-                AND fd.date_end   >= \'' . $now . '\'
+                    fd.date_start <= \'' . $now.'\'
+                AND fd.date_end   >= \'' . $now.'\'
                 AND fd.id = fv.id
                 AND fd.lang = fv.lang
                 AND fd.active = \'yes\'';
@@ -1931,15 +1931,15 @@ 
         if (isset($categoryId) && is_numeric($categoryId) && ($categoryId != 0)) {
             $query .= '
             AND
-                fcr.category_id = \'' . $categoryId . '\'';
+                fcr.category_id = \'' . $categoryId.'\'';
         }
         if (isset($language) && Language::isASupportedLanguage($language)) {
             $query .= '
             AND
-                fd.lang = \'' . $language . '\'';
+                fd.lang = \'' . $language.'\'';
         }
         $query .= '
-                ' . $this->queryPermission($this->groupSupport) . '
+                ' . $this->queryPermission($this->groupSupport).'
 
             GROUP BY
                 fd.id,fd.lang,fcr.category_id,fv.visits,fv.last_visit,fdg.group_id,fdu.user_id
@@ -2020,40 +2020,40 @@ 
                 (fv.vote/fv.usr) AS avg,
                 fv.usr AS user
             FROM
-                ' . Db::getTablePrefix() . 'faqvoting fv,
-                ' . Db::getTablePrefix() . 'faqdata fd
+                ' . Db::getTablePrefix().'faqvoting fv,
+                ' . Db::getTablePrefix().'faqdata fd
             LEFT JOIN
-                ' . Db::getTablePrefix() . 'faqcategoryrelations fcr
+                ' . Db::getTablePrefix().'faqcategoryrelations fcr
             ON
                 fd.id = fcr.record_id
             AND
                 fd.lang = fcr.record_lang
             LEFT JOIN
-                ' . Db::getTablePrefix() . 'faqdata_group AS fdg
+                ' . Db::getTablePrefix().'faqdata_group AS fdg
             ON
                 fd.id = fdg.record_id
             LEFT JOIN
-                ' . Db::getTablePrefix() . 'faqdata_user AS fdu
+                ' . Db::getTablePrefix().'faqdata_user AS fdu
             ON
                 fd.id = fdu.record_id
             WHERE
-                    fd.date_start <= \'' . $now . '\'
-                AND fd.date_end   >= \'' . $now . '\'
+                    fd.date_start <= \'' . $now.'\'
+                AND fd.date_end   >= \'' . $now.'\'
                 AND fd.id = fv.artikel
                 AND fd.active = \'yes\'';
 
         if (isset($categoryId) && is_numeric($categoryId) && ($categoryId != 0)) {
             $query .= '
             AND
-                fcr.category_id = \'' . $categoryId . '\'';
+                fcr.category_id = \'' . $categoryId.'\'';
         }
         if (isset($language) && Language::isASupportedLanguage($language)) {
             $query .= '
             AND
-                fd.lang = \'' . $language . '\'';
+                fd.lang = \'' . $language.'\'';
         }
         $query .= '
-                ' . $this->queryPermission($this->groupSupport) . '
+                ' . $this->queryPermission($this->groupSupport).'
             ORDER BY
                 avg DESC';
 
@@ -2142,25 +2142,25 @@ 
                 fdg.group_id AS group_id,
                 fdu.user_id AS user_id
             FROM
-                ' . Db::getTablePrefix() . 'faqvisits fv,
-                ' . Db::getTablePrefix() . 'faqdata fd
+                ' . Db::getTablePrefix().'faqvisits fv,
+                ' . Db::getTablePrefix().'faqdata fd
             LEFT JOIN
-                ' . Db::getTablePrefix() . 'faqcategoryrelations fcr
+                ' . Db::getTablePrefix().'faqcategoryrelations fcr
             ON
                 fd.id = fcr.record_id
             AND
                 fd.lang = fcr.record_lang
             LEFT JOIN
-                ' . Db::getTablePrefix() . 'faqdata_group AS fdg
+                ' . Db::getTablePrefix().'faqdata_group AS fdg
             ON
                 fd.id = fdg.record_id
             LEFT JOIN
-                ' . Db::getTablePrefix() . 'faqdata_user AS fdu
+                ' . Db::getTablePrefix().'faqdata_user AS fdu
             ON
                 fd.id = fdu.record_id
             WHERE
-                    fd.date_start <= \'' . $now . '\'
-                AND fd.date_end   >= \'' . $now . '\'
+                    fd.date_start <= \'' . $now.'\'
+                AND fd.date_end   >= \'' . $now.'\'
                 AND fd.id = fv.id
                 AND fd.lang = fv.lang
                 AND fd.active = \'yes\'';
@@ -2168,10 +2168,10 @@ 
         if (isset($language) && Language::isASupportedLanguage($language)) {
             $query .= '
             AND
-                fd.lang = \'' . $language . '\'';
+                fd.lang = \'' . $language.'\'';
         }
         $query .= '
-                ' . $this->queryPermission($this->groupSupport) . '
+                ' . $this->queryPermission($this->groupSupport).'
             GROUP BY
                 fd.id,fd.lang,fcr.category_id,fv.visits,fdg.group_id,fdu.user_id
             ORDER BY
@@ -2329,7 +2329,7 @@ 
                 VALUES
             (%d, '%s', '%s', '%s', %d, '%s', '%s', '%s', %d)",
             Db::getTablePrefix(),
-            $this->config->getDb()->nextId(Db::getTablePrefix() . 'faqquestions', 'id'),
+            $this->config->getDb()->nextId(Db::getTablePrefix().'faqquestions', 'id'),
             $this->config->getLanguage()->getLanguage(),
             $this->config->getDb()->escape($questionData['username']),
             $this->config->getDb()->escape($questionData['email']),
@@ -2476,7 +2476,7 @@ 
                 VALUES
             (%d, %d, '%s', %d, %d, %d, '%s')",
             Db::getTablePrefix(),
-            $this->config->getDb()->nextId(Db::getTablePrefix() . 'faqchanges', 'id'),
+            $this->config->getDb()->nextId(Db::getTablePrefix().'faqchanges', 'id'),
             $id,
             $lang,
             $revision_id,
@@ -2627,7 +2627,7 @@ 
         // faqvisits data selection
         if (!empty($faqId)) {
             // Select ONLY the faq with the provided $faqid
-            $query .= "fd.id = '" . $faqId . "' AND ";
+            $query .= "fd.id = '".$faqId."' AND ";
         }
         $query .= 'fd.id = fv.id
             AND
@@ -2637,7 +2637,7 @@ 
             if ($needAndOp) {
                 $query .= ' AND';
             }
-            $query .= ' (fcr.category_id = ' . $categoryId;
+            $query .= ' (fcr.category_id = '.$categoryId;
             if ($bDownwards) {
                 $query .= $this->getCatidWhereSequence($categoryId, 'OR');
             }
@@ -2648,14 +2648,14 @@ 
             if ($needAndOp) {
                 $query .= ' AND';
             }
-            $query .= " fd.updated LIKE '" . $date . "'";
+            $query .= " fd.updated LIKE '".$date."'";
             $needAndOp = true;
         }
         if ((!empty($lang)) && Utils::isLanguage($lang)) {
             if ($needAndOp) {
                 $query .= ' AND';
             }
-            $query .= " fd.lang = '" . $lang . "'";
+            $query .= " fd.lang = '".$lang."'";
             $needAndOp = true;
         }
         switch ($queryType) {
@@ -2663,7 +2663,7 @@ 
                 if ($needAndOp) {
                     $query .= ' AND';
                 }
-                $query .= " fd.active = '" . FAQ_SQL_ACTIVE_NO . "'";
+                $query .= " fd.active = '".FAQ_SQL_ACTIVE_NO."'";
                 break;
             case FAQ_QUERY_TYPE_EXPORT_PDF:
             case FAQ_QUERY_TYPE_EXPORT_XHTML:
@@ -2671,13 +2671,13 @@ 
                 if ($needAndOp) {
                     $query .= ' AND';
                 }
-                $query .= " fd.active = '" . FAQ_SQL_ACTIVE_YES . "'";
+                $query .= " fd.active = '".FAQ_SQL_ACTIVE_YES."'";
                 break;
             default:
                 if ($needAndOp) {
                     $query .= ' AND';
                 }
-                $query .= " fd.active = '" . FAQ_SQL_ACTIVE_YES . "'";
+                $query .= " fd.active = '".FAQ_SQL_ACTIVE_YES."'";
                 break;
         }
         // Sort criteria
@@ -2719,7 +2719,7 @@ 
         $aChildren = array_values($oCat->getChildren($nCatid));
 
         foreach ($aChildren as $catid) {
-            $sqlWhereFilter .= ' ' . $logicOp . ' fcr.category_id = ' . $catid;
+            $sqlWhereFilter .= ' '.$logicOp.' fcr.category_id = '.$catid;
             $sqlWhereFilter .= $this->getCatidWhereSequence($catid, 'OR', $oCat);
         }
 
@@ -2922,7 +2922,7 @@ 
                 $oLink->itemTitle = $row->thema;
                 $oLink->text = $title;
                 $oLink->tooltip = $title;
-                $listItem = '<li>' . $oLink->toHtmlAnchor() . '</li>';
+                $listItem = '<li>'.$oLink->toHtmlAnchor().'</li>';
 
                 $output .= $listItem;
             }
@@ -3029,7 +3029,7 @@ 
             );
         }
 
-        return $output . $extraout;
+        return $output.$extraout;
     }
 
     /**