mambax7 / newbb5

class/Constants.php

Indentation +3 added lines, -3 removed lines

@@ -24,12 +24,12 @@
  */
 class Constants
 {
-    /**#@+
+	/**#@+
      * Constant definition
      */
 
-    const DISALLOW = 0;
+	const DISALLOW = 0;
 
-    /**#@-*/
+	/**#@-*/
 }
 

class/Read.php

Indentation +13 added lines, -13 removed lines

@@ -44,17 +44,17 @@
  */
 class Read extends \XoopsObject
 {
-    /**
-     * @internal param $type
-     */
-    public function __construct()
-    {
-        // parent::__construct("newbb_reads_" . $type);
-        parent::__construct();
-        $this->initVar('read_id', XOBJ_DTYPE_INT);
-        $this->initVar('uid', XOBJ_DTYPE_INT);
-        $this->initVar('read_item', XOBJ_DTYPE_INT);
-        $this->initVar('post_id', XOBJ_DTYPE_INT);
-        $this->initVar('read_time', XOBJ_DTYPE_INT);
-    }
+	/**
+	 * @internal param $type
+	 */
+	public function __construct()
+	{
+		// parent::__construct("newbb_reads_" . $type);
+		parent::__construct();
+		$this->initVar('read_id', XOBJ_DTYPE_INT);
+		$this->initVar('uid', XOBJ_DTYPE_INT);
+		$this->initVar('read_item', XOBJ_DTYPE_INT);
+		$this->initVar('post_id', XOBJ_DTYPE_INT);
+		$this->initVar('read_time', XOBJ_DTYPE_INT);
+	}
 }

class/Category.php

Indentation +13 added lines, -13 removed lines

@@ -13,17 +13,17 @@
 
 class Category extends \XoopsObject
 {
-    /**
-     *
-     */
-    public function __construct()
-    {
-        parent::__construct();
-        $this->initVar('cat_id', XOBJ_DTYPE_INT);
-        $this->initVar('cat_title', XOBJ_DTYPE_TXTBOX);
-        $this->initVar('cat_image', XOBJ_DTYPE_SOURCE, 'blank.gif');
-        $this->initVar('cat_description', XOBJ_DTYPE_TXTAREA);
-        $this->initVar('cat_order', XOBJ_DTYPE_INT, 99);
-        $this->initVar('cat_url', XOBJ_DTYPE_URL);
-    }
+	/**
+	 *
+	 */
+	public function __construct()
+	{
+		parent::__construct();
+		$this->initVar('cat_id', XOBJ_DTYPE_INT);
+		$this->initVar('cat_title', XOBJ_DTYPE_TXTBOX);
+		$this->initVar('cat_image', XOBJ_DTYPE_SOURCE, 'blank.gif');
+		$this->initVar('cat_description', XOBJ_DTYPE_TXTAREA);
+		$this->initVar('cat_order', XOBJ_DTYPE_INT, 99);
+		$this->initVar('cat_url', XOBJ_DTYPE_URL);
+	}
 }

class/Uploader.php

Indentation +70 added lines, -70 removed lines

@@ -19,80 +19,80 @@
  */
 class Uploader extends \XoopsMediaUploader
 {
-    /**
-     * No admin check for uploads
-     */
-    /**
-     * Constructor
-     *
-     * @param string           $uploadDir
-     * @param array|int|string $allowedMimeTypes
-     * @param int              $maxFileSize
-     * @param int              $maxWidth
-     * @param int              $maxHeight
-     */
-    public function __construct($uploadDir, $allowedMimeTypes = 0, $maxFileSize = 0, $maxWidth = 0, $maxHeight = 0)
-    {
-        //        $this->XoopsMediaUploader($uploadDir, $allowedMimeTypes, $maxFileSize, $maxWidth, $maxHeight);
+	/**
+	 * No admin check for uploads
+	 */
+	/**
+	 * Constructor
+	 *
+	 * @param string           $uploadDir
+	 * @param array|int|string $allowedMimeTypes
+	 * @param int              $maxFileSize
+	 * @param int              $maxWidth
+	 * @param int              $maxHeight
+	 */
+	public function __construct($uploadDir, $allowedMimeTypes = 0, $maxFileSize = 0, $maxWidth = 0, $maxHeight = 0)
+	{
+		//        $this->XoopsMediaUploader($uploadDir, $allowedMimeTypes, $maxFileSize, $maxWidth, $maxHeight);
 
-        if (!is_array($allowedMimeTypes)) {
-            if (empty($allowedMimeTypes) || '*' === $allowedMimeTypes) {
-                $allowedMimeTypes = [];
-            } else {
-                $allowedMimeTypes = array_filter(array_map('trim', explode('|', strtolower($allowedMimeTypes))));
-            }
-        }
-        $_allowedMimeTypes = [];
-        $extensionToMime   = require_once $GLOBALS['xoops']->path('include/mimetypes.inc.php');
-        foreach ($allowedMimeTypes as $type) {
-            if (isset($extensionToMime[$type])) {
-                $_allowedMimeTypes[] = $extensionToMime[$type];
-            } else {
-                $_allowedMimeTypes[] = $type;
-            }
-        }
-        parent::__construct($uploadDir, $_allowedMimeTypes, $maxFileSize, $maxWidth, $maxHeight);
-    }
+		if (!is_array($allowedMimeTypes)) {
+			if (empty($allowedMimeTypes) || '*' === $allowedMimeTypes) {
+				$allowedMimeTypes = [];
+			} else {
+				$allowedMimeTypes = array_filter(array_map('trim', explode('|', strtolower($allowedMimeTypes))));
+			}
+		}
+		$_allowedMimeTypes = [];
+		$extensionToMime   = require_once $GLOBALS['xoops']->path('include/mimetypes.inc.php');
+		foreach ($allowedMimeTypes as $type) {
+			if (isset($extensionToMime[$type])) {
+				$_allowedMimeTypes[] = $extensionToMime[$type];
+			} else {
+				$_allowedMimeTypes[] = $type;
+			}
+		}
+		parent::__construct($uploadDir, $_allowedMimeTypes, $maxFileSize, $maxWidth, $maxHeight);
+	}
 
-    /**
-     * Set the CheckMediaTypeByExt
-     * Deprecated
-     *
-     * @param bool|string $value
-     */
-    public function setCheckMediaTypeByExt($value = true)
-    {
-    }
+	/**
+	 * Set the CheckMediaTypeByExt
+	 * Deprecated
+	 *
+	 * @param bool|string $value
+	 */
+	public function setCheckMediaTypeByExt($value = true)
+	{
+	}
 
-    /**
-     * Set the imageSizeCheck
-     * Deprecated
-     *
-     * @param string $value
-     */
-    public function setImageSizeCheck($value)
-    {
-    }
+	/**
+	 * Set the imageSizeCheck
+	 * Deprecated
+	 *
+	 * @param string $value
+	 */
+	public function setImageSizeCheck($value)
+	{
+	}
 
-    /**
-     * Set the fileSizeCheck
-     * Deprecated
-     *
-     * @param string $value
-     */
-    public function setFileSizeCheck($value)
-    {
-    }
+	/**
+	 * Set the fileSizeCheck
+	 * Deprecated
+	 *
+	 * @param string $value
+	 */
+	public function setFileSizeCheck($value)
+	{
+	}
 
-    /**
-     * Get the file extension
-     *
-     * @return string
-     */
-    public function getExt()
-    {
-        $this->ext = strtolower(ltrim(strrchr($this->getMediaName(), '.'), '.'));
+	/**
+	 * Get the file extension
+	 *
+	 * @return string
+	 */
+	public function getExt()
+	{
+		$this->ext = strtolower(ltrim(strrchr($this->getMediaName(), '.'), '.'));
 
-        return $this->ext;
-    }
+		return $this->ext;
+	}
 }

class/Moderate.php

Indentation +14 added lines, -14 removed lines

@@ -24,18 +24,18 @@
  */
 class Moderate extends \XoopsObject
 {
-    /**
-     *
-     */
-    public function __construct()
-    {
-        parent::__construct();
-        $this->initVar('mod_id', XOBJ_DTYPE_INT);
-        $this->initVar('mod_start', XOBJ_DTYPE_INT);
-        $this->initVar('mod_end', XOBJ_DTYPE_INT);
-        $this->initVar('mod_desc', XOBJ_DTYPE_TXTBOX);
-        $this->initVar('uid', XOBJ_DTYPE_INT);
-        $this->initVar('ip', XOBJ_DTYPE_TXTBOX);
-        $this->initVar('forum_id', XOBJ_DTYPE_INT);
-    }
+	/**
+	 *
+	 */
+	public function __construct()
+	{
+		parent::__construct();
+		$this->initVar('mod_id', XOBJ_DTYPE_INT);
+		$this->initVar('mod_start', XOBJ_DTYPE_INT);
+		$this->initVar('mod_end', XOBJ_DTYPE_INT);
+		$this->initVar('mod_desc', XOBJ_DTYPE_TXTBOX);
+		$this->initVar('uid', XOBJ_DTYPE_INT);
+		$this->initVar('ip', XOBJ_DTYPE_TXTBOX);
+		$this->initVar('forum_id', XOBJ_DTYPE_INT);
+	}
 }

class/Rate.php

Indentation +13 added lines, -13 removed lines

@@ -39,17 +39,17 @@
  */
 class Rate extends \XoopsObject
 {
-    /**
-     *
-     */
-    public function __construct()
-    {
-        parent::__construct();
-        $this->initVar('ratingid', XOBJ_DTYPE_INT);
-        $this->initVar('topic_id', XOBJ_DTYPE_INT);
-        $this->initVar('ratinguser', XOBJ_DTYPE_INT);
-        $this->initVar('rating', XOBJ_DTYPE_INT);
-        $this->initVar('ratingtimestamp', XOBJ_DTYPE_INT);
-        $this->initVar('ratinghostname', XOBJ_DTYPE_TXTBOX);
-    }
+	/**
+	 *
+	 */
+	public function __construct()
+	{
+		parent::__construct();
+		$this->initVar('ratingid', XOBJ_DTYPE_INT);
+		$this->initVar('topic_id', XOBJ_DTYPE_INT);
+		$this->initVar('ratinguser', XOBJ_DTYPE_INT);
+		$this->initVar('rating', XOBJ_DTYPE_INT);
+		$this->initVar('ratingtimestamp', XOBJ_DTYPE_INT);
+		$this->initVar('ratinghostname', XOBJ_DTYPE_TXTBOX);
+	}
 }

class/UserHandler.php

Indentation +93 added lines, -93 removed lines

@@ -21,98 +21,98 @@
  */
 class UserHandler
 {
-    /** @var array */
-    public $users = [];
-
-    /** @var bool */
-    private $enableGroup;
-
-    /** @var bool */
-    private $enableOnline;
-
-    /** @var array */
-    private $userlist = [];
-
-    /**
-     * @param bool $enableGroup
-     * @param bool $enableOnline
-     */
-    public function __construct($enableGroup = true, $enableOnline = true)
-    {
-        $this->enableGroup  = $enableGroup;
-        $this->enableOnline = $enableOnline;
-    }
-
-    public function loadUserInfo()
-    {
-        /** @var Newbb\Helper $helper */
-        $helper = Newbb\Helper::getInstance();
-        $helper->loadLanguage('user');
+	/** @var array */
+	public $users = [];
+
+	/** @var bool */
+	private $enableGroup;
+
+	/** @var bool */
+	private $enableOnline;
+
+	/** @var array */
+	private $userlist = [];
+
+	/**
+	 * @param bool $enableGroup
+	 * @param bool $enableOnline
+	 */
+	public function __construct($enableGroup = true, $enableOnline = true)
+	{
+		$this->enableGroup  = $enableGroup;
+		$this->enableOnline = $enableOnline;
+	}
+
+	public function loadUserInfo()
+	{
+		/** @var Newbb\Helper $helper */
+		$helper = Newbb\Helper::getInstance();
+		$helper->loadLanguage('user');
 //        @require_once $GLOBALS['xoops']->path('modules/' . $GLOBALS['xoopsModule']->getVar('dirname', 'n') . '/language/' . $GLOBALS['xoopsConfig']['language'] . '/user.php');
-        if (class_exists('UserLanguage')) {
-            $handler = new Newbb\UserLanguage();
-        } else {
-            $handler = new User();
-        }
-        foreach (array_keys($this->users) as $uid) {
-            $this->userlist[$uid] = $handler->getInfo($this->users[$uid]);
-        }
-    }
-
-    public function loadUserOnline()
-    {
-        if (empty($this->users) || !$this->enableOnline) {
-            return;
-        }
-        require_once  dirname(__DIR__) . '/include/functions.render.php';
-        $image_online  = newbbDisplayImage('online', _MD_NEWBB_ONLINE);
-        $image_offline = newbbDisplayImage('offline', _MD_NEWBB_OFFLINE);
-
-        /** @var Newbb\OnlineHandler $onlineHandler */
-        $onlineHandler = Newbb\Helper::getInstance()->getHandler('Online');
-        $onlines       = $onlineHandler->checkStatus(array_keys($this->users));
-
-        foreach (array_keys($this->users) as $uid) {
-            $this->userlist[$uid]['status'] = empty($onlines[$uid]) ? $image_offline : $image_online;
-        }
-    }
-    // START irmtfan remove function - no deprecated is needed because just use in this file
-    //    function loadUserGroups()
-    //    {
-    //        return true;
-    //    }
-    // END irmtfan remove function - no deprecated is needed because just use in this file
-
-    public function loadUserDigest()
-    {
-        if (empty($this->users)) {
-            return;
-        }
-
-        $sql    = 'SELECT user_digests, uid FROM ' . $GLOBALS['xoopsDB']->prefix('newbb_user_stats') . ' WHERE uid IN( ' . implode(', ', array_keys($this->users)) . ')';
-        $result = $GLOBALS['xoopsDB']->query($sql);
-        while (false !== ($myrow = $GLOBALS['xoopsDB']->fetchArray($result))) {
-            $this->userlist[$myrow['uid']]['digests'] = (int)$myrow['user_digests'];
-        }
-    }
-    // START irmtfan remove function
-    //    function loadUserRank()
-    //    {
-    //          return true;
-    //    }
-    // END irmtfan remove function
-
-    /**
-     * @return array
-     */
-    public function getUsers()
-    {
-        $this->loadUserInfo();
-        $this->loadUserOnline();
-        // irmtfan removed $this->loadUserGroups();
-        // irmtfan removed $this->loadUserRank();
-        $this->loadUserDigest();
-
-        return $this->userlist;
-    }
+		if (class_exists('UserLanguage')) {
+			$handler = new Newbb\UserLanguage();
+		} else {
+			$handler = new User();
+		}
+		foreach (array_keys($this->users) as $uid) {
+			$this->userlist[$uid] = $handler->getInfo($this->users[$uid]);
+		}
+	}
+
+	public function loadUserOnline()
+	{
+		if (empty($this->users) || !$this->enableOnline) {
+			return;
+		}
+		require_once  dirname(__DIR__) . '/include/functions.render.php';
+		$image_online  = newbbDisplayImage('online', _MD_NEWBB_ONLINE);
+		$image_offline = newbbDisplayImage('offline', _MD_NEWBB_OFFLINE);
+
+		/** @var Newbb\OnlineHandler $onlineHandler */
+		$onlineHandler = Newbb\Helper::getInstance()->getHandler('Online');
+		$onlines       = $onlineHandler->checkStatus(array_keys($this->users));
+
+		foreach (array_keys($this->users) as $uid) {
+			$this->userlist[$uid]['status'] = empty($onlines[$uid]) ? $image_offline : $image_online;
+		}
+	}
+	// START irmtfan remove function - no deprecated is needed because just use in this file
+	//    function loadUserGroups()
+	//    {
+	//        return true;
+	//    }
+	// END irmtfan remove function - no deprecated is needed because just use in this file
+
+	public function loadUserDigest()
+	{
+		if (empty($this->users)) {
+			return;
+		}
+
+		$sql    = 'SELECT user_digests, uid FROM ' . $GLOBALS['xoopsDB']->prefix('newbb_user_stats') . ' WHERE uid IN( ' . implode(', ', array_keys($this->users)) . ')';
+		$result = $GLOBALS['xoopsDB']->query($sql);
+		while (false !== ($myrow = $GLOBALS['xoopsDB']->fetchArray($result))) {
+			$this->userlist[$myrow['uid']]['digests'] = (int)$myrow['user_digests'];
+		}
+	}
+	// START irmtfan remove function
+	//    function loadUserRank()
+	//    {
+	//          return true;
+	//    }
+	// END irmtfan remove function
+
+	/**
+	 * @return array
+	 */
+	public function getUsers()
+	{
+		$this->loadUserInfo();
+		$this->loadUserOnline();
+		// irmtfan removed $this->loadUserGroups();
+		// irmtfan removed $this->loadUserRank();
+		$this->loadUserDigest();
+
+		return $this->userlist;
+	}
 }

Spacing +3 added lines, -3 removed lines

@@ -64,7 +64,7 @@ 
         if (empty($this->users) || !$this->enableOnline) {
             return;
         }
-        require_once  dirname(__DIR__) . '/include/functions.render.php';
+        require_once  dirname(__DIR__).'/include/functions.render.php';
         $image_online  = newbbDisplayImage('online', _MD_NEWBB_ONLINE);
         $image_offline = newbbDisplayImage('offline', _MD_NEWBB_OFFLINE);
 
@@ -89,10 +89,10 @@ 
             return;
         }
 
-        $sql    = 'SELECT user_digests, uid FROM ' . $GLOBALS['xoopsDB']->prefix('newbb_user_stats') . ' WHERE uid IN( ' . implode(', ', array_keys($this->users)) . ')';
+        $sql    = 'SELECT user_digests, uid FROM '.$GLOBALS['xoopsDB']->prefix('newbb_user_stats').' WHERE uid IN( '.implode(', ', array_keys($this->users)).')';
         $result = $GLOBALS['xoopsDB']->query($sql);
         while (false !== ($myrow = $GLOBALS['xoopsDB']->fetchArray($result))) {
-            $this->userlist[$myrow['uid']]['digests'] = (int)$myrow['user_digests'];
+            $this->userlist[$myrow['uid']]['digests'] = (int) $myrow['user_digests'];
         }
     }
     // START irmtfan remove function

class/Forum.php

Indentation +35 added lines, -35 removed lines

@@ -21,41 +21,41 @@
 
 class Forum extends \XoopsObject
 {
-    /**
-     *
-     */
-    public function __construct()
-    {
-        parent::__construct();
-        $this->initVar('forum_id', XOBJ_DTYPE_INT);
-        $this->initVar('forum_name', XOBJ_DTYPE_TXTBOX);
-        $this->initVar('forum_desc', XOBJ_DTYPE_TXTAREA);
-        $this->initVar('forum_moderator', XOBJ_DTYPE_ARRAY, serialize([]));
-        $this->initVar('forum_topics', XOBJ_DTYPE_INT);
-        $this->initVar('forum_posts', XOBJ_DTYPE_INT);
-        $this->initVar('forum_last_post_id', XOBJ_DTYPE_INT);
-        $this->initVar('cat_id', XOBJ_DTYPE_INT);
-        $this->initVar('parent_forum', XOBJ_DTYPE_INT);
-        $this->initVar('hot_threshold', XOBJ_DTYPE_INT, 20);
-        $this->initVar('attach_maxkb', XOBJ_DTYPE_INT, 500);
-        $this->initVar('attach_ext', XOBJ_DTYPE_SOURCE, 'zip|jpg|gif|png');
-        $this->initVar('forum_order', XOBJ_DTYPE_INT, 99);
-        $this->initVar('dohtml', XOBJ_DTYPE_INT, 1);
-    }
+	/**
+	 *
+	 */
+	public function __construct()
+	{
+		parent::__construct();
+		$this->initVar('forum_id', XOBJ_DTYPE_INT);
+		$this->initVar('forum_name', XOBJ_DTYPE_TXTBOX);
+		$this->initVar('forum_desc', XOBJ_DTYPE_TXTAREA);
+		$this->initVar('forum_moderator', XOBJ_DTYPE_ARRAY, serialize([]));
+		$this->initVar('forum_topics', XOBJ_DTYPE_INT);
+		$this->initVar('forum_posts', XOBJ_DTYPE_INT);
+		$this->initVar('forum_last_post_id', XOBJ_DTYPE_INT);
+		$this->initVar('cat_id', XOBJ_DTYPE_INT);
+		$this->initVar('parent_forum', XOBJ_DTYPE_INT);
+		$this->initVar('hot_threshold', XOBJ_DTYPE_INT, 20);
+		$this->initVar('attach_maxkb', XOBJ_DTYPE_INT, 500);
+		$this->initVar('attach_ext', XOBJ_DTYPE_SOURCE, 'zip|jpg|gif|png');
+		$this->initVar('forum_order', XOBJ_DTYPE_INT, 99);
+		$this->initVar('dohtml', XOBJ_DTYPE_INT, 1);
+	}
 
-    /**
-     * @return string
-     */
-    public function dispForumModerators()
-    {
-        $ret = '';
-        if (!$valid_moderators = $this->getVar('forum_moderator')) {
-            return $ret;
-        }
-        require_once $GLOBALS['xoops']->path('modules/newbb/include/functions.user.php');
-        $moderators = newbbGetUnameFromIds($valid_moderators, !empty($GLOBALS['xoopsModuleConfig']['show_realname']), true);
-        $ret        = implode(', ', $moderators);
+	/**
+	 * @return string
+	 */
+	public function dispForumModerators()
+	{
+		$ret = '';
+		if (!$valid_moderators = $this->getVar('forum_moderator')) {
+			return $ret;
+		}
+		require_once $GLOBALS['xoops']->path('modules/newbb/include/functions.user.php');
+		$moderators = newbbGetUnameFromIds($valid_moderators, !empty($GLOBALS['xoopsModuleConfig']['show_realname']), true);
+		$ret        = implode(', ', $moderators);
 
-        return $ret;
-    }
+		return $ret;
+	}
 }

class/plugins/plugin.tag.php

Indentation +50 added lines, -50 removed lines

@@ -32,43 +32,43 @@ 
  */
 function newbb_tag_iteminfo(&$items)
 {
-    if (0 === count($items) || !is_array($items)) {
-        return false;
-    }
+	if (0 === count($items) || !is_array($items)) {
+		return false;
+	}
 
-    $items_id = [];
-    foreach (array_keys($items) as $cat_id) {
-        // Some handling here to build the link upon catid
-        // catid is not used in newbb, so just skip it
-        foreach (array_keys($items[$cat_id]) as $item_id) {
-            // In newbb, the item_id is "topic_id"
-            $items_id[] = (int)$item_id;
-        }
-    }
-    /** @var TopicHandler $itemHandler */
-    $itemHandler = Newbb\Helper::getInstance()->getHandler('Topic');
-    /** @var \XoopsObject $itemsObject */
-    $itemsObject = $itemHandler->getObjects(new \Criteria('topic_id', '(' . implode(', ', $items_id) . ')', 'IN'), true);
+	$items_id = [];
+	foreach (array_keys($items) as $cat_id) {
+		// Some handling here to build the link upon catid
+		// catid is not used in newbb, so just skip it
+		foreach (array_keys($items[$cat_id]) as $item_id) {
+			// In newbb, the item_id is "topic_id"
+			$items_id[] = (int)$item_id;
+		}
+	}
+	/** @var TopicHandler $itemHandler */
+	$itemHandler = Newbb\Helper::getInstance()->getHandler('Topic');
+	/** @var \XoopsObject $itemsObject */
+	$itemsObject = $itemHandler->getObjects(new \Criteria('topic_id', '(' . implode(', ', $items_id) . ')', 'IN'), true);
 
-    foreach (array_keys($items) as $cat_id) {
-        foreach (array_keys($items[$cat_id]) as $item_id) {
-            /** @var \XoopsObject $itemObject */
-            if (!$itemObject = $itemsObject[$item_id]) {
-                continue;
-            }
-            $items[$cat_id][$item_id] = [
-                'title'   => $itemObject->getVar('topic_title'),
-                'uid'     => $itemObject->getVar('topic_poster'),
-                'link'    => "viewtopic.php?topic_id={$item_id}",
-                'time'    => $itemObject->getVar('topic_time'),
-                'tags'    => tag_parse_tag($itemObject->getVar('topic_tags', 'n')),
-                'content' => ''
-            ];
-        }
-    }
-    unset($itemsObject);
+	foreach (array_keys($items) as $cat_id) {
+		foreach (array_keys($items[$cat_id]) as $item_id) {
+			/** @var \XoopsObject $itemObject */
+			if (!$itemObject = $itemsObject[$item_id]) {
+				continue;
+			}
+			$items[$cat_id][$item_id] = [
+				'title'   => $itemObject->getVar('topic_title'),
+				'uid'     => $itemObject->getVar('topic_poster'),
+				'link'    => "viewtopic.php?topic_id={$item_id}",
+				'time'    => $itemObject->getVar('topic_time'),
+				'tags'    => tag_parse_tag($itemObject->getVar('topic_tags', 'n')),
+				'content' => ''
+			];
+		}
+	}
+	unset($itemsObject);
 
-    return true;
+	return true;
 }
 
 /**
@@ -78,21 +78,21 @@ 
  */
 function newbb_tag_synchronization($mid)
 {
-    /** @var TopicHandler $itemHandler */
-    $itemHandler = Newbb\Helper::getInstance()->getHandler('Topic');
-    /** @var \XoopsPersistableObjectHandler $linkHandler */
-    $linkHandler = Tag\Helper::getInstance()->getHandler('Link');
+	/** @var TopicHandler $itemHandler */
+	$itemHandler = Newbb\Helper::getInstance()->getHandler('Topic');
+	/** @var \XoopsPersistableObjectHandler $linkHandler */
+	$linkHandler = Tag\Helper::getInstance()->getHandler('Link');
 
-    /* clear tag-item links */
-    $sql = "    DELETE FROM {$linkHandler->table}"
-           . '    WHERE '
-           . "        tag_modid = {$mid}"
-           . '        AND '
-           . '        ( tag_itemid NOT IN '
-           . "            ( SELECT DISTINCT {$itemHandler->keyName} "
-           . "                FROM {$itemHandler->table} "
-           . "                WHERE {$itemHandler->table}.approved > 0"
-           . '            ) '
-           . '        )';
-    $linkHandler->db->queryF($sql);
+	/* clear tag-item links */
+	$sql = "    DELETE FROM {$linkHandler->table}"
+		   . '    WHERE '
+		   . "        tag_modid = {$mid}"
+		   . '        AND '
+		   . '        ( tag_itemid NOT IN '
+		   . "            ( SELECT DISTINCT {$itemHandler->keyName} "
+		   . "                FROM {$itemHandler->table} "
+		   . "                WHERE {$itemHandler->table}.approved > 0"
+		   . '            ) '
+		   . '        )';
+	$linkHandler->db->queryF($sql);
 }

Spacing +2 added lines, -2 removed lines

@@ -42,13 +42,13 @@
         // catid is not used in newbb, so just skip it
         foreach (array_keys($items[$cat_id]) as $item_id) {
             // In newbb, the item_id is "topic_id"
-            $items_id[] = (int)$item_id;
+            $items_id[] = (int) $item_id;
         }
     }
     /** @var TopicHandler $itemHandler */
     $itemHandler = Newbb\Helper::getInstance()->getHandler('Topic');
     /** @var \XoopsObject $itemsObject */
-    $itemsObject = $itemHandler->getObjects(new \Criteria('topic_id', '(' . implode(', ', $items_id) . ')', 'IN'), true);
+    $itemsObject = $itemHandler->getObjects(new \Criteria('topic_id', '('.implode(', ', $items_id).')', 'IN'), true);
 
     foreach (array_keys($items) as $cat_id) {
         foreach (array_keys($items[$cat_id]) as $item_id) {