mambax7 / instruction

class/utility.php

Doc Comments +5 added lines

@@ -39,6 +39,11 @@
     }
 
     // Редактор
+
+    /**
+     * @param string $caption
+     * @param string $name
+     */
     public static function getWysiwygForm($caption, $name, $value = '')
     {
         $editor                   = false;

Unused Use Statements -2 removed lines

@@ -1,7 +1,5 @@
 <?php
 
-use Xmf\Request;
-
 require_once __DIR__ . '/common/traitversionchecks.php';
 require_once __DIR__ . '/common/traitserverstats.php';
 require_once __DIR__ . '/common/traitfilesmgmt.php';

Indentation +53 added lines, -53 removed lines

@@ -13,63 +13,63 @@
  */
 class InstructionUtility
 {
-    use VersionChecks; //checkVerXoops, checkVerPhp Traits
+	use VersionChecks; //checkVerXoops, checkVerPhp Traits
 
-    use ServerStats; // getServerStats Trait
+	use ServerStats; // getServerStats Trait
 
-    use FilesManagement; // Files Management Trait
+	use FilesManagement; // Files Management Trait
 
-    // Права
-    public static function getItemIds($permtype = 'instruction_view')
-    {
-        //global $xoopsUser;
-        static $permissions = [];
-        // Если есть в статике
-        if (is_array($permissions) && array_key_exists($permtype, $permissions)) {
-            return $permissions[$permtype];
-        }
-        // Находим из базы
-        $moduleHandler          = xoops_getHandler('module');
-        $instrModule            = $moduleHandler->getByDirname('instruction');
-        $groups                 = is_object($GLOBALS['xoopsUser']) ? $GLOBALS['xoopsUser']->getGroups() : XOOPS_GROUP_ANONYMOUS;
-        $gpermHandler           = xoops_getHandler('groupperm');
-        $categories             = $gpermHandler->getItemIds($permtype, $groups, $instrModule->getVar('mid'));
-        $permissions[$permtype] = $categories;
-        return $categories;
-    }
+	// Права
+	public static function getItemIds($permtype = 'instruction_view')
+	{
+		//global $xoopsUser;
+		static $permissions = [];
+		// Если есть в статике
+		if (is_array($permissions) && array_key_exists($permtype, $permissions)) {
+			return $permissions[$permtype];
+		}
+		// Находим из базы
+		$moduleHandler          = xoops_getHandler('module');
+		$instrModule            = $moduleHandler->getByDirname('instruction');
+		$groups                 = is_object($GLOBALS['xoopsUser']) ? $GLOBALS['xoopsUser']->getGroups() : XOOPS_GROUP_ANONYMOUS;
+		$gpermHandler           = xoops_getHandler('groupperm');
+		$categories             = $gpermHandler->getItemIds($permtype, $groups, $instrModule->getVar('mid'));
+		$permissions[$permtype] = $categories;
+		return $categories;
+	}
 
-    // Редактор
-    public static function getWysiwygForm($caption, $name, $value = '')
-    {
-        $editor                   = false;
-        $editor_configs           = [];
-        $editor_configs['name']   = $name;
-        $editor_configs['value']  = $value;
-        $editor_configs['rows']   = 35;
-        $editor_configs['cols']   = 60;
-        $editor_configs['width']  = '100%';
-        $editor_configs['height'] = '350px';
-        $editor_configs['editor'] = strtolower(xoops_getModuleOption('form_options', 'instruction'));
+	// Редактор
+	public static function getWysiwygForm($caption, $name, $value = '')
+	{
+		$editor                   = false;
+		$editor_configs           = [];
+		$editor_configs['name']   = $name;
+		$editor_configs['value']  = $value;
+		$editor_configs['rows']   = 35;
+		$editor_configs['cols']   = 60;
+		$editor_configs['width']  = '100%';
+		$editor_configs['height'] = '350px';
+		$editor_configs['editor'] = strtolower(xoops_getModuleOption('form_options', 'instruction'));
 
-        $editor = new XoopsFormEditor($caption, $name, $editor_configs);
-        return $editor;
-    }
+		$editor = new XoopsFormEditor($caption, $name, $editor_configs);
+		return $editor;
+	}
 
-    // Получение значения переменной, переданной через GET или POST запрос
-    public static function cleanVars(&$global, $key, $default = '', $type = 'int')
-    {
-        switch ($type) {
-            case 'string':
-                $ret = isset($global[$key]) ? $global[$key] : $default;
-                break;
-            case 'int':
-            default:
-                $ret = isset($global[$key]) ? (int)$global[$key] : (int)$default;
-                break;
-        }
-        if (false === $ret) {
-            return $default;
-        }
-        return $ret;
-    }
+	// Получение значения переменной, переданной через GET или POST запрос
+	public static function cleanVars(&$global, $key, $default = '', $type = 'int')
+	{
+		switch ($type) {
+			case 'string':
+				$ret = isset($global[$key]) ? $global[$key] : $default;
+				break;
+			case 'int':
+			default:
+				$ret = isset($global[$key]) ? (int)$global[$key] : (int)$default;
+				break;
+		}
+		if (false === $ret) {
+			return $default;
+		}
+		return $ret;
+	}
 }

Spacing +1 added lines, -1 removed lines

@@ -64,7 +64,7 @@
                 break;
             case 'int':
             default:
-                $ret = isset($global[$key]) ? (int)$global[$key] : (int)$default;
+                $ret = isset($global[$key]) ? (int) $global[$key] : (int) $default;
                 break;
         }
         if (false === $ret) {

include/comment_functions.php

Indentation +4 added lines, -4 removed lines

@@ -5,12 +5,12 @@
 // Функция вызывается при добавлении комментария
 function instruction_com_update($pageid, $total_num)
 {
-    $db  = XoopsDatabaseFactory::getDatabaseConnection();
-    $sql = 'UPDATE ' . $db->prefix('instruction_page') . ' SET comments = ' . $total_num . ' WHERE pageid  = ' . $pageid;
-    $db->query($sql);
+	$db  = XoopsDatabaseFactory::getDatabaseConnection();
+	$sql = 'UPDATE ' . $db->prefix('instruction_page') . ' SET comments = ' . $total_num . ' WHERE pageid  = ' . $pageid;
+	$db->query($sql);
 }
 
 function instruction_com_approve(&$comment)
 {
-    // notification mail here
+	// notification mail here
 }

include/config.php

Spacing +2 added lines, -2 removed lines

@@ -22,7 +22,7 @@ 
 $moduleDirName = basename(dirname(__DIR__));
 $capsDirName   = strtoupper($moduleDirName);
 
-if (!defined($capsDirName . '_DIRNAME')) {
+if ( ! defined($capsDirName . '_DIRNAME')) {
     //if (!defined(constant($capsDirName . '_DIRNAME'))) {
     define($capsDirName . '_DIRNAME', $GLOBALS['xoopsModule']->dirname());
     define($capsDirName . '_PATH', XOOPS_ROOT_PATH . '/modules/' . constant($capsDirName . '_DIRNAME'));
@@ -35,7 +35,7 @@ 
 }
 
 //Configurator
-return (object)[
+return (object) [
     'name'          => strtoupper($moduleDirName) . ' Module Configurator',
     'paths'         => [
         'dirname'    => $moduleDirName,

Indentation +46 added lines, -46 removed lines

@@ -23,58 +23,58 @@
 $capsDirName   = strtoupper($moduleDirName);
 
 if (!defined($capsDirName . '_DIRNAME')) {
-    //if (!defined(constant($capsDirName . '_DIRNAME'))) {
-    define($capsDirName . '_DIRNAME', $GLOBALS['xoopsModule']->dirname());
-    define($capsDirName . '_PATH', XOOPS_ROOT_PATH . '/modules/' . constant($capsDirName . '_DIRNAME'));
-    define($capsDirName . '_URL', XOOPS_URL . '/modules/' . constant($capsDirName . '_DIRNAME'));
-    define($capsDirName . '_ADMIN', constant($capsDirName . '_URL') . '/admin/index.php');
-    define($capsDirName . '_ROOT_PATH', XOOPS_ROOT_PATH . '/modules/' . constant($capsDirName . '_DIRNAME'));
-    define($capsDirName . '_AUTHOR_LOGOIMG', constant($capsDirName . '_URL') . '/assets/images/logoModule.png');
-    define($capsDirName . '_UPLOAD_URL', XOOPS_UPLOAD_URL . '/' . $moduleDirName); // WITHOUT Trailing slash
-    define($capsDirName . '_UPLOAD_PATH', XOOPS_UPLOAD_PATH . '/' . $moduleDirName); // WITHOUT Trailing slash
+	//if (!defined(constant($capsDirName . '_DIRNAME'))) {
+	define($capsDirName . '_DIRNAME', $GLOBALS['xoopsModule']->dirname());
+	define($capsDirName . '_PATH', XOOPS_ROOT_PATH . '/modules/' . constant($capsDirName . '_DIRNAME'));
+	define($capsDirName . '_URL', XOOPS_URL . '/modules/' . constant($capsDirName . '_DIRNAME'));
+	define($capsDirName . '_ADMIN', constant($capsDirName . '_URL') . '/admin/index.php');
+	define($capsDirName . '_ROOT_PATH', XOOPS_ROOT_PATH . '/modules/' . constant($capsDirName . '_DIRNAME'));
+	define($capsDirName . '_AUTHOR_LOGOIMG', constant($capsDirName . '_URL') . '/assets/images/logoModule.png');
+	define($capsDirName . '_UPLOAD_URL', XOOPS_UPLOAD_URL . '/' . $moduleDirName); // WITHOUT Trailing slash
+	define($capsDirName . '_UPLOAD_PATH', XOOPS_UPLOAD_PATH . '/' . $moduleDirName); // WITHOUT Trailing slash
 }
 
 //Configurator
 return (object)[
-    'name'          => strtoupper($moduleDirName) . ' Module Configurator',
-    'paths'         => [
-        'dirname'    => $moduleDirName,
-        'admin'      => XOOPS_ROOT_PATH . '/modules/' . $moduleDirName . '/admin',
-        //        'path'       => XOOPS_ROOT_PATH . '/modules/' . $moduleDirName,
-        //        'url'        => XOOPS_URL . '/modules/' . $moduleDirName,
-        'uploadPath' => XOOPS_UPLOAD_PATH . '/' . $moduleDirName,
-        'uploadUrl'  => XOOPS_UPLOAD_URL . '/' . $moduleDirName,
-    ],
-    'uploadFolders' => [
-        constant($capsDirName . '_UPLOAD_PATH'),
-        //constant($capsDirName . '_UPLOAD_PATH') . '/category',
-        //constant($capsDirName . '_UPLOAD_PATH') . '/screenshots',
-        //XOOPS_UPLOAD_PATH . '/flags'
-    ],
-    'blankFiles'    => [
-        constant($capsDirName . '_UPLOAD_PATH'),
-        //constant($capsDirName . '_UPLOAD_PATH') . '/category',
-        //constant($capsDirName . '_UPLOAD_PATH') . '/screenshots',
-        //XOOPS_UPLOAD_PATH . '/flags'
-    ],
+	'name'          => strtoupper($moduleDirName) . ' Module Configurator',
+	'paths'         => [
+		'dirname'    => $moduleDirName,
+		'admin'      => XOOPS_ROOT_PATH . '/modules/' . $moduleDirName . '/admin',
+		//        'path'       => XOOPS_ROOT_PATH . '/modules/' . $moduleDirName,
+		//        'url'        => XOOPS_URL . '/modules/' . $moduleDirName,
+		'uploadPath' => XOOPS_UPLOAD_PATH . '/' . $moduleDirName,
+		'uploadUrl'  => XOOPS_UPLOAD_URL . '/' . $moduleDirName,
+	],
+	'uploadFolders' => [
+		constant($capsDirName . '_UPLOAD_PATH'),
+		//constant($capsDirName . '_UPLOAD_PATH') . '/category',
+		//constant($capsDirName . '_UPLOAD_PATH') . '/screenshots',
+		//XOOPS_UPLOAD_PATH . '/flags'
+	],
+	'blankFiles'    => [
+		constant($capsDirName . '_UPLOAD_PATH'),
+		//constant($capsDirName . '_UPLOAD_PATH') . '/category',
+		//constant($capsDirName . '_UPLOAD_PATH') . '/screenshots',
+		//XOOPS_UPLOAD_PATH . '/flags'
+	],
 
-    'templateFolders' => [
-        '/templates/',
-        '/templates/blocks/',
-        '/templates/admin/'
+	'templateFolders' => [
+		'/templates/',
+		'/templates/blocks/',
+		'/templates/admin/'
 
-    ],
-    'oldFiles'        => [
+	],
+	'oldFiles'        => [
 //        '/sql/mysql.sql',
-    ],
-    'oldFolders'      => [
-        '/images',
-        '/css',
-        '/js',
-        '/tree',
-        '/images/icons/16',
-        '/images/icons/32',
-    ],
-    'modCopyright'    => "<a href='https://xoops.org' title='XOOPS Project' target='_blank'>
+	],
+	'oldFolders'      => [
+		'/images',
+		'/css',
+		'/js',
+		'/tree',
+		'/images/icons/16',
+		'/images/icons/32',
+	],
+	'modCopyright'    => "<a href='https://xoops.org' title='XOOPS Project' target='_blank'>
                      <img src='" . $capsDirName . '_AUTHOR_LOGOIMG' . '\' alt=\'XOOPS Project\' /></a>',
 ];

include/pagetypes.inc.php

Indentation +2 added lines, -2 removed lines

@@ -1,6 +1,6 @@
 <?php
 // Типы страницы
 return [
-    0 => _AM_INSTR_PT_0,
-    1 => _AM_INSTR_PT_1
+	0 => _AM_INSTR_PT_0,
+	1 => _AM_INSTR_PT_1
 ];

include/common.php

Indentation +10 added lines, -10 removed lines

@@ -18,14 +18,14 @@ 
  */
 
 if (!defined('INSTRUCTION_MODULE_PATH')) {
-    define('INSTRUCTION_DIRNAME', basename(dirname(__DIR__)));
-    define('INSTRUCTION_URL', XOOPS_URL . '/modules/' . INSTRUCTION_DIRNAME);
-    define('INSTRUCTION_IMAGE_URL', INSTRUCTION_URL . '/assets/images/');
-    define('INSTRUCTION_ROOT_PATH', XOOPS_ROOT_PATH . '/modules/' . INSTRUCTION_DIRNAME);
-    define('INSTRUCTION_IMAGE_PATH', INSTRUCTION_ROOT_PATH . '/assets/images');
-    define('INSTRUCTION_ADMIN_URL', INSTRUCTION_URL . '/admin/');
-    define('INSTRUCTION_UPLOAD_URL', XOOPS_UPLOAD_URL . '/' . INSTRUCTION_DIRNAME);
-    define('INSTRUCTION_UPLOAD_PATH', XOOPS_UPLOAD_PATH . '/' . INSTRUCTION_DIRNAME);
+	define('INSTRUCTION_DIRNAME', basename(dirname(__DIR__)));
+	define('INSTRUCTION_URL', XOOPS_URL . '/modules/' . INSTRUCTION_DIRNAME);
+	define('INSTRUCTION_IMAGE_URL', INSTRUCTION_URL . '/assets/images/');
+	define('INSTRUCTION_ROOT_PATH', XOOPS_ROOT_PATH . '/modules/' . INSTRUCTION_DIRNAME);
+	define('INSTRUCTION_IMAGE_PATH', INSTRUCTION_ROOT_PATH . '/assets/images');
+	define('INSTRUCTION_ADMIN_URL', INSTRUCTION_URL . '/admin/');
+	define('INSTRUCTION_UPLOAD_URL', XOOPS_UPLOAD_URL . '/' . INSTRUCTION_DIRNAME);
+	define('INSTRUCTION_UPLOAD_PATH', XOOPS_UPLOAD_PATH . '/' . INSTRUCTION_DIRNAME);
 }
 xoops_loadLanguage('common', INSTRUCTION_DIRNAME);
 
@@ -46,8 +46,8 @@ 
 $instruction->loadLanguage('common');
 
 if (!isset($GLOBALS['xoopsTpl']) || !($GLOBALS['xoopsTpl'] instanceof XoopsTpl)) {
-    require_once $GLOBALS['xoops']->path('class/template.php');
-    $xoopsTpl = new XoopsTpl();
+	require_once $GLOBALS['xoops']->path('class/template.php');
+	$xoopsTpl = new XoopsTpl();
 }
 
 $moduleDirName = basename(dirname(__DIR__));

Spacing +2 added lines, -2 removed lines

@@ -17,7 +17,7 @@ 
  * @author       XOOPS Development Team
  */
 
-if (!defined('INSTRUCTION_MODULE_PATH')) {
+if ( ! defined('INSTRUCTION_MODULE_PATH')) {
     define('INSTRUCTION_DIRNAME', basename(dirname(__DIR__)));
     define('INSTRUCTION_URL', XOOPS_URL . '/modules/' . INSTRUCTION_DIRNAME);
     define('INSTRUCTION_IMAGE_URL', INSTRUCTION_URL . '/assets/images/');
@@ -45,7 +45,7 @@ 
 $instruction = Instruction::getInstance();
 $instruction->loadLanguage('common');
 
-if (!isset($GLOBALS['xoopsTpl']) || !($GLOBALS['xoopsTpl'] instanceof XoopsTpl)) {
+if ( ! isset($GLOBALS['xoopsTpl']) || ! ($GLOBALS['xoopsTpl'] instanceof XoopsTpl)) {
     require_once $GLOBALS['xoops']->path('class/template.php');
     $xoopsTpl = new XoopsTpl();
 }

include/plugin.tag.php

Indentation +53 added lines, -53 removed lines

@@ -3,66 +3,66 @@
 // Информация об теге
 function instruction_tag_iteminfo(&$items)
 {
-    if (empty($items) || !is_array($items)) {
-        return false;
-    }
+	if (empty($items) || !is_array($items)) {
+		return false;
+	}
 
-    $items_id = [];
-    foreach (array_keys($items) as $cat_id) {
-        foreach (array_keys($items[$cat_id]) as $item_id) {
-            $items_id[] = (int)$item_id;
-        }
-    }
+	$items_id = [];
+	foreach (array_keys($items) as $cat_id) {
+		foreach (array_keys($items[$cat_id]) as $item_id) {
+			$items_id[] = (int)$item_id;
+		}
+	}
 
-    $itemHandler = xoops_getModuleHandler('instruction', 'instruction');
-    $items_obj   = $itemHandler->getObjects(new Criteria('instrid', '(' . implode(', ', $items_id) . ')', 'IN'), true);
+	$itemHandler = xoops_getModuleHandler('instruction', 'instruction');
+	$items_obj   = $itemHandler->getObjects(new Criteria('instrid', '(' . implode(', ', $items_id) . ')', 'IN'), true);
 
-    foreach (array_keys($items) as $cat_id) {
-        foreach (array_keys($items[$cat_id]) as $item_id) {
-            if (isset($items_obj[$item_id])) {
-                $item_obj                 = $items_obj[$item_id];
-                $items[$cat_id][$item_id] = [
-                    'title'   => $item_obj->getVar('title'),
-                    'uid'     => $item_obj->getVar('uid'),
-                    'link'    => "instr.php?id={$item_id}",
-                    'time'    => $item_obj->getVar('datecreated'),
-                    'tags'    => '',
-                    'content' => '',
-                ];
-            }
-        }
-    }
-    unset($items_obj);
+	foreach (array_keys($items) as $cat_id) {
+		foreach (array_keys($items[$cat_id]) as $item_id) {
+			if (isset($items_obj[$item_id])) {
+				$item_obj                 = $items_obj[$item_id];
+				$items[$cat_id][$item_id] = [
+					'title'   => $item_obj->getVar('title'),
+					'uid'     => $item_obj->getVar('uid'),
+					'link'    => "instr.php?id={$item_id}",
+					'time'    => $item_obj->getVar('datecreated'),
+					'tags'    => '',
+					'content' => '',
+				];
+			}
+		}
+	}
+	unset($items_obj);
 }
 
 // Синхронизация тегов
 function instruction_tag_synchronization($mid)
 {
-    $itemHandler = xoops_getModuleHandler('instruction', 'instruction');
-    $linkHandler = xoops_getModuleHandler('link', 'tag');
+	$itemHandler = xoops_getModuleHandler('instruction', 'instruction');
+	$linkHandler = xoops_getModuleHandler('link', 'tag');
 
-    /* clear tag-item links */
-    if (version_compare(mysqli_get_server_info(), '4.1.0', 'ge')):
-        $sql = "    DELETE FROM {$linkHandler->table}"
-               . '    WHERE '
-               . "        tag_modid = {$mid}"
-               . '        AND '
-               . '        ( tag_itemid NOT IN '
-               . "            ( SELECT DISTINCT {$itemHandler->keyName} "
-               . "                FROM {$itemHandler->table} "
-               . "                WHERE {$itemHandler->table}.status > 0"
-               . '            ) '
-               . '        )'; else:
-        $sql = "    DELETE {$linkHandler->table} FROM {$linkHandler->table}"
-               . "    LEFT JOIN {$itemHandler->table} AS aa ON {$linkHandler->table}.tag_itemid = aa.{$itemHandler->keyName} "
-               . '    WHERE '
-               . "        tag_modid = {$mid}"
-               . '        AND '
-               . "        ( aa.{$itemHandler->keyName} IS NULL"
-               . '            OR aa.status < 1'
-               . '        )';
-    endif;
-    if (!$result = $linkHandler->db->queryF($sql)) {
-        //xoops_error($linkHandler->db->error());
-    }
+	/* clear tag-item links */
+	if (version_compare(mysqli_get_server_info(), '4.1.0', 'ge')):
+		$sql = "    DELETE FROM {$linkHandler->table}"
+			   . '    WHERE '
+			   . "        tag_modid = {$mid}"
+			   . '        AND '
+			   . '        ( tag_itemid NOT IN '
+			   . "            ( SELECT DISTINCT {$itemHandler->keyName} "
+			   . "                FROM {$itemHandler->table} "
+			   . "                WHERE {$itemHandler->table}.status > 0"
+			   . '            ) '
+			   . '        )'; else:
+		$sql = "    DELETE {$linkHandler->table} FROM {$linkHandler->table}"
+			   . "    LEFT JOIN {$itemHandler->table} AS aa ON {$linkHandler->table}.tag_itemid = aa.{$itemHandler->keyName} "
+			   . '    WHERE '
+			   . "        tag_modid = {$mid}"
+			   . '        AND '
+			   . "        ( aa.{$itemHandler->keyName} IS NULL"
+			   . '            OR aa.status < 1'
+			   . '        )';
+	endif;
+	if (!$result = $linkHandler->db->queryF($sql)) {
+		//xoops_error($linkHandler->db->error());
+	}
 }

Spacing +3 added lines, -3 removed lines

@@ -3,14 +3,14 @@ 
 // Информация об теге
 function instruction_tag_iteminfo(&$items)
 {
-    if (empty($items) || !is_array($items)) {
+    if (empty($items) || ! is_array($items)) {
         return false;
     }
 
     $items_id = [];
     foreach (array_keys($items) as $cat_id) {
         foreach (array_keys($items[$cat_id]) as $item_id) {
-            $items_id[] = (int)$item_id;
+            $items_id[] = (int) $item_id;
         }
     }
 
@@ -62,7 +62,7 @@ 
                . '            OR aa.status < 1'
                . '        )';
     endif;
-    if (!$result = $linkHandler->db->queryF($sql)) {
+    if ( ! $result = $linkHandler->db->queryF($sql)) {
         //xoops_error($linkHandler->db->error());
     }
 }

Braces +4 added lines, -2 removed lines

@@ -52,7 +52,8 @@ 
                . "                FROM {$itemHandler->table} "
                . "                WHERE {$itemHandler->table}.status > 0"
                . '            ) '
-               . '        )'; else:
+               . '        )'; else {

+               	:
         $sql = "    DELETE {$linkHandler->table} FROM {$linkHandler->table}"
                . "    LEFT JOIN {$itemHandler->table} AS aa ON {$linkHandler->table}.tag_itemid = aa.{$itemHandler->keyName} "
                . '    WHERE '
@@ -60,7 +61,8 @@ 
                . '        AND '
                . "        ( aa.{$itemHandler->keyName} IS NULL"
                . '            OR aa.status < 1'
-               . '        )';
+               . '        )';

+               }

     endif;
     if (!$result = $linkHandler->db->queryF($sql)) {
         //xoops_error($linkHandler->db->error());

include/search.inc.php

Spacing +1 added lines, -1 removed lines

@@ -9,7 +9,7 @@
 
     $sql = 'SELECT p.pageid, p.title, p.uid, p.datecreated, i.title FROM ' . $GLOBALS['xoopsDB']->prefix('instruction_page') . ' p, ' . $GLOBALS['xoopsDB']->prefix('instruction_instr') . ' i WHERE i.instrid = p.instrid AND i.status > 0 AND p.status > 0 AND p.type > 0';
     if (0 != $userid) {
-        $sql .= ' AND p.uid = ' . (int)$userid . ' ';
+        $sql .= ' AND p.uid = ' . (int) $userid . ' ';
         //return NULL;
     }
 

Indentation +38 added lines, -38 removed lines

@@ -4,45 +4,45 @@
 
 function instruction_search($queryarray, $andor, $limit, $offset, $userid)
 {
-    // Подключаем функции
-    include_once $GLOBALS['xoops']->path('modules/' . $moduleDirName . '/class/utility.php');
+	// Подключаем функции
+	include_once $GLOBALS['xoops']->path('modules/' . $moduleDirName . '/class/utility.php');
 
-    $sql = 'SELECT p.pageid, p.title, p.uid, p.datecreated, i.title FROM ' . $GLOBALS['xoopsDB']->prefix('instruction_page') . ' p, ' . $GLOBALS['xoopsDB']->prefix('instruction_instr') . ' i WHERE i.instrid = p.instrid AND i.status > 0 AND p.status > 0 AND p.type > 0';
-    if (0 != $userid) {
-        $sql .= ' AND p.uid = ' . (int)$userid . ' ';
-        //return NULL;
-    }
+	$sql = 'SELECT p.pageid, p.title, p.uid, p.datecreated, i.title FROM ' . $GLOBALS['xoopsDB']->prefix('instruction_page') . ' p, ' . $GLOBALS['xoopsDB']->prefix('instruction_instr') . ' i WHERE i.instrid = p.instrid AND i.status > 0 AND p.status > 0 AND p.type > 0';
+	if (0 != $userid) {
+		$sql .= ' AND p.uid = ' . (int)$userid . ' ';
+		//return NULL;
+	}
 
-    // Права на просмотр
-    $categories = InstructionUtility::getItemIds();
-    if (is_array($categories) && count($categories) > 0) {
-        $sql .= ' AND i.cid IN ( ' . implode(', ', $categories) . ' ) ';
-        // Если пользователь не имеет прав просмотра ни одной категории
-    } else {
-        return null;
-    }
+	// Права на просмотр
+	$categories = InstructionUtility::getItemIds();
+	if (is_array($categories) && count($categories) > 0) {
+		$sql .= ' AND i.cid IN ( ' . implode(', ', $categories) . ' ) ';
+		// Если пользователь не имеет прав просмотра ни одной категории
+	} else {
+		return null;
+	}
 
-    // Добавляем в условие ключевые слова поиска
-    if (is_array($queryarray) && $count = count($queryarray)) {
-        $sql .= " AND ( ( p.title LIKE '%$queryarray[0]%' OR p.hometext LIKE '%$queryarray[0]%' )";
-        for ($i = 1; $i < $count; $i++) {
-            $sql .= " $andor ";
-            $sql .= "( p.title LIKE '%$queryarray[$i]%' OR p.hometext LIKE '%$queryarray[$i]%' )";
-        }
-        $sql .= ' ) ';
-    }
-    //$sql .= "ORDER BY date DESC";
-    $result = $GLOBALS['xoopsDB']->query($sql, $limit, $offset);
-    $ret    = [];
-    $i      = 0;
-    // Перебираем все результаты
-    while (list($pageid, $ptitle, $puid, $pdatecreated, $ititle) = $GLOBALS['xoopsDB']->fetchRow($result)) {
-        $ret[$i]['image'] = 'assets/images/size2.gif';
-        $ret[$i]['link']  = 'page.php?id=' . $pageid;
-        $ret[$i]['title'] = $ititle . ': ' . $ptitle;
-        $ret[$i]['time']  = $pdatecreated;
-        $ret[$i]['uid']   = $puid;
-        $i++;
-    }
-    return $ret;
+	// Добавляем в условие ключевые слова поиска
+	if (is_array($queryarray) && $count = count($queryarray)) {
+		$sql .= " AND ( ( p.title LIKE '%$queryarray[0]%' OR p.hometext LIKE '%$queryarray[0]%' )";
+		for ($i = 1; $i < $count; $i++) {
+			$sql .= " $andor ";
+			$sql .= "( p.title LIKE '%$queryarray[$i]%' OR p.hometext LIKE '%$queryarray[$i]%' )";
+		}
+		$sql .= ' ) ';
+	}
+	//$sql .= "ORDER BY date DESC";
+	$result = $GLOBALS['xoopsDB']->query($sql, $limit, $offset);
+	$ret    = [];
+	$i      = 0;
+	// Перебираем все результаты
+	while (list($pageid, $ptitle, $puid, $pdatecreated, $ititle) = $GLOBALS['xoopsDB']->fetchRow($result)) {
+		$ret[$i]['image'] = 'assets/images/size2.gif';
+		$ret[$i]['link']  = 'page.php?id=' . $pageid;
+		$ret[$i]['title'] = $ititle . ': ' . $ptitle;
+		$ret[$i]['time']  = $pdatecreated;
+		$ret[$i]['uid']   = $puid;
+		$i++;
+	}
+	return $ret;
 }

page.php

Indentation +43 added lines, -43 removed lines

@@ -28,8 +28,8 @@ 
 $criteria->add(new Criteria('pageid ', $pageid));
 $criteria->add(new Criteria('status ', '0', '>'));
 if (0 == $inspageHandler->getCount($criteria)) {
-    redirect_header('index.php', 3, _MD_INSTRUCTION_PAGENOTEXIST);
-    exit();
+	redirect_header('index.php', 3, _MD_INSTRUCTION_PAGENOTEXIST);
+	exit();
 }
 //
 unset($criteria);
@@ -41,8 +41,8 @@ 
 
 // Если админ и ссылка на отключение кэша
 if (is_object($GLOBALS['xoopsUser']) && $GLOBALS['xoopsUser']->isAdmin() && $nocache) {
-    // Отключаем кэш
-    $GLOBALS['xoopsConfig']['module_cache'][$GLOBALS['xoopsModule']->getVar('mid')] = 0;
+	// Отключаем кэш
+	$GLOBALS['xoopsConfig']['module_cache'][$GLOBALS['xoopsModule']->getVar('mid')] = 0;
 }
 
 // Задание тайтла
@@ -59,8 +59,8 @@ 
 // Права на просмотр инструкции
 $categories = InstructionUtility::getItemIds();
 if (!in_array($objInsinstr->getVar('cid'), $categories)) {
-    redirect_header(XOOPS_URL . '/modules/' . $moduleDirName . '/', 3, _NOPERM);
-    exit();
+	redirect_header(XOOPS_URL . '/modules/' . $moduleDirName . '/', 3, _NOPERM);
+	exit();
 }
 
 // Массив данных о странице
@@ -77,9 +77,9 @@ 
 $footnote = $objInspage->getVar('footnote');
 // Если есть сноски
 if ($footnote) {
-    $pages['footnotes'] = explode('|', $objInspage->getVar('footnote'));
+	$pages['footnotes'] = explode('|', $objInspage->getVar('footnote'));
 } else {
-    $pages['footnotes'] = false;
+	$pages['footnotes'] = false;
 }
 // Мета-теги ключевых слов
 $pages['keywords'] = $objInspage->getVar('keywords');
@@ -88,39 +88,39 @@ 
 //
 // Если админ, рисуем админлинк
 if (is_object($GLOBALS['xoopsUser']) && $GLOBALS['xoopsUser']->isAdmin($GLOBALS['xoopsModule']->mid())) {
-    $pages['adminlink'] = '&nbsp;<a href="'
-                          . XOOPS_URL
-                          . '/modules/'
-                          . $moduleDirName
-                          . '/admin/instr.php?op=editpage&pageid='
-                          . $pages['pageid']
-                          . '"><img style="width:16px;" src="./assets/icons/edit_mini.png" alt='
-                          . _EDIT
-                          . ' title='
-                          . _EDIT
-                          . '></a>&nbsp;<a href="'
-                          . XOOPS_URL
-                          . '/modules/'
-                          . $moduleDirName
-                          . '/admin/instr.php?op=delpage&pageid='
-                          . $pages['pageid']
-                          . '"><img style="width:16px;" src="./assets/icons/delete_mini.png" alt='
-                          . _DELETE
-                          . ' title='
-                          . _DELETE
-                          . '></a>&nbsp;';
+	$pages['adminlink'] = '&nbsp;<a href="'
+						  . XOOPS_URL
+						  . '/modules/'
+						  . $moduleDirName
+						  . '/admin/instr.php?op=editpage&pageid='
+						  . $pages['pageid']
+						  . '"><img style="width:16px;" src="./assets/icons/edit_mini.png" alt='
+						  . _EDIT
+						  . ' title='
+						  . _EDIT
+						  . '></a>&nbsp;<a href="'
+						  . XOOPS_URL
+						  . '/modules/'
+						  . $moduleDirName
+						  . '/admin/instr.php?op=delpage&pageid='
+						  . $pages['pageid']
+						  . '"><img style="width:16px;" src="./assets/icons/delete_mini.png" alt='
+						  . _DELETE
+						  . ' title='
+						  . _DELETE
+						  . '></a>&nbsp;';
 } else {
-    $pages['adminlink'] = '&nbsp;';
-    // Если можно редактировать
-    if ($gpermHandler->checkRight($moduleDirName . '_edit', $objInsinstr->getVar('cid'), $groups, $GLOBALS['xoopsModule']->getVar('mid'))) {
-        $pages['adminlink'] .= '<a href="' . XOOPS_URL . '/modules/' . $moduleDirName . '/submit.php?op=editpage&pageid=' . $pages['pageid'] . '"><img style="width:16px;" src="./assets/icons/edit_mini.png" alt=' . _EDIT . ' title=' . _EDIT . '></a>';
-    }
-
-    $pages['adminlink'] .= '&nbsp;';
-    // Если нет админлика
-    if ('[&nbsp;&nbsp;]' == $pages['adminlink']) {
-        $pages['adminlink'] = '';
-    }
+	$pages['adminlink'] = '&nbsp;';
+	// Если можно редактировать
+	if ($gpermHandler->checkRight($moduleDirName . '_edit', $objInsinstr->getVar('cid'), $groups, $GLOBALS['xoopsModule']->getVar('mid'))) {
+		$pages['adminlink'] .= '<a href="' . XOOPS_URL . '/modules/' . $moduleDirName . '/submit.php?op=editpage&pageid=' . $pages['pageid'] . '"><img style="width:16px;" src="./assets/icons/edit_mini.png" alt=' . _EDIT . ' title=' . _EDIT . '></a>';
+	}
+
+	$pages['adminlink'] .= '&nbsp;';
+	// Если нет админлика
+	if ('[&nbsp;&nbsp;]' == $pages['adminlink']) {
+		$pages['adminlink'] = '';
+	}
 }
 // Выводим в шаблон
 $GLOBALS['xoopsTpl']->assign('insPage', $pages);
@@ -139,7 +139,7 @@ 
 $nav_parent_id = array_reverse($nav_parent_id);
 $navigation    = '<a href="' . XOOPS_URL . '/modules/' . $moduleDirName . '/">' . $GLOBALS['xoopsModule']->name() . '</a>&nbsp;:&nbsp;';
 foreach (array_keys($nav_parent_id) as $i) {
-    $navigation .= '<a href="' . XOOPS_URL . '/modules/' . $moduleDirName . '/index.php?cid=' . $nav_parent_id[$i]->getVar('cid') . '">' . $nav_parent_id[$i]->getVar('title') . '</a>&nbsp;:&nbsp;';
+	$navigation .= '<a href="' . XOOPS_URL . '/modules/' . $moduleDirName . '/index.php?cid=' . $nav_parent_id[$i]->getVar('cid') . '">' . $nav_parent_id[$i]->getVar('title') . '</a>&nbsp;:&nbsp;';
 }
 $navigation .= '<a href="' . XOOPS_URL . '/modules/' . $moduleDirName . '/index.php?cid=' . $objInscat->getVar('cid') . '">' . $objInscat->getVar('title') . '</a>&nbsp;:&nbsp;';
 $navigation .= '<a href="' . XOOPS_URL . '/modules/' . $moduleDirName . '/instr.php?id=' . $pages['instrid'] . '">' . $objInsinstr->getVar('title') . '</a>';
@@ -171,9 +171,9 @@ 
 
 // Рейтинг
 if (xoops_getModuleOption('userat', 'instruction')) {
-    $xoopsTpl->assign('insUserat', true);
+	$xoopsTpl->assign('insUserat', true);
 } else {
-    $xoopsTpl->assign('insUserat', false);
+	$xoopsTpl->assign('insUserat', false);
 }
 
 // Мета теги

Spacing +2 added lines, -2 removed lines

@@ -19,7 +19,7 @@ 
 
 // Получаем данные
 // ID страницы
-$pageid = isset($_GET['id']) ? (int)$_GET['id'] : 0;
+$pageid = isset($_GET['id']) ? (int) $_GET['id'] : 0;
 // Без кэша
 $nocache = InstructionUtility::cleanVars($_GET, 'nocache', 0, 'int');
 
@@ -58,7 +58,7 @@ 
 
 // Права на просмотр инструкции
 $categories = InstructionUtility::getItemIds();
-if (!in_array($objInsinstr->getVar('cid'), $categories)) {
+if ( ! in_array($objInsinstr->getVar('cid'), $categories)) {
     redirect_header(XOOPS_URL . '/modules/' . $moduleDirName . '/', 3, _NOPERM);
     exit();
 }

class/common/traitserverstats.php

Indentation +55 added lines, -55 removed lines

@@ -16,60 +16,60 @@
  */
 trait ServerStats
 {
-    /**
-     * serverStats()
-     *
-     * @return string
-     */
-    public static function getServerStats()
-    {
-        //mb    $wfdownloads = WfdownloadsWfdownloads::getInstance();
-        $moduleDirName      = basename(dirname(dirname(__DIR__)));
-        $moduleDirNameUpper = strtoupper($moduleDirName);
-        xoops_loadLanguage('common', $moduleDirName);
-        $html = '';
-        //        $sql   = 'SELECT metavalue';
-        //        $sql   .= ' FROM ' . $GLOBALS['xoopsDB']->prefix('wfdownloads_meta');
-        //        $sql   .= " WHERE metakey='version' LIMIT 1";
-        //        $query = $GLOBALS['xoopsDB']->query($sql);
-        //        list($meta) = $GLOBALS['xoopsDB']->fetchRow($query);
-        $html .= "<fieldset><legend style='font-weight: bold; color: #900;'>" . constant('CO_' . $moduleDirNameUpper . '_IMAGEINFO') . "</legend>\n";
-        $html .= "<div style='padding: 8px;'>\n";
-        //        $html .= '<div>' . constant('CO_' . $moduleDirNameUpper . '_METAVERSION') . $meta . "</div>\n";
-        //        $html .= "<br>\n";
-        //        $html .= "<br>\n";
-        $html .= '<div>' . constant('CO_' . $moduleDirNameUpper . '_SPHPINI') . "</div>\n";
-        $html .= "<ul>\n";
-        //
-        $gdlib = function_exists('gd_info') ? '<span style="color: green;">' . constant('CO_' . $moduleDirNameUpper . '_GDON') . '</span>' : '<span style="color: red;">' . constant('CO_' . $moduleDirNameUpper . '_GDOFF') . '</span>';
-        $html  .= '<li>' . constant('CO_' . $moduleDirNameUpper . '_GDLIBSTATUS') . $gdlib;
-        if (function_exists('gd_info')) {
-            if ($gdlib = true === gd_info()) {
-                $html .= '<li>' . constant('CO_' . $moduleDirNameUpper . '_GDLIBVERSION') . '<b>' . $gdlib['GD Version'] . '</b>';
-            }
-        }
-        //
-        //    $safemode = ini_get('safe_mode') ? constant('CO_' . $moduleDirNameUpper . '_ON') . constant('CO_' . $moduleDirNameUpper . '_SAFEMODEPROBLEMS : constant('CO_' . $moduleDirNameUpper . '_OFF');
-        //    $html .= '<li>' . constant('CO_' . $moduleDirNameUpper . '_SAFEMODESTATUS . $safemode;
-        //
-        //    $registerglobals = (!ini_get('register_globals')) ? "<span style=\"color: green;\">" . constant('CO_' . $moduleDirNameUpper . '_OFF') . '</span>' : "<span style=\"color: red;\">" . constant('CO_' . $moduleDirNameUpper . '_ON') . '</span>';
-        //    $html .= '<li>' . constant('CO_' . $moduleDirNameUpper . '_REGISTERGLOBALS . $registerglobals;
-        //
-        $downloads = ini_get('file_uploads') ? '<span style="color: green;">' . constant('CO_' . $moduleDirNameUpper . '_ON') . '</span>' : '<span style="color: red;">' . constant('CO_' . $moduleDirNameUpper . '_OFF') . '</span>';
-        $html      .= '<li>' . constant('CO_' . $moduleDirNameUpper . '_SERVERUPLOADSTATUS') . $downloads;
-        //
-        $html .= '<li>' . constant('CO_' . $moduleDirNameUpper . '_MAXUPLOADSIZE') . ' <b><span style="color: blue;">' . ini_get('upload_max_filesize') . "</span></b>\n";
-        $html .= '<li>' . constant('CO_' . $moduleDirNameUpper . '_MAXPOSTSIZE') . ' <b><span style="color: blue;">' . ini_get('post_max_size') . "</span></b>\n";
-        $html .= '<li>' . constant('CO_' . $moduleDirNameUpper . '_MEMORYLIMIT') . ' <b><span style="color: blue;">' . ini_get('memory_limit') . "</span></b>\n";
-        $html .= "</ul>\n";
-        $html .= "<ul>\n";
-        $html .= '<li>' . constant('CO_' . $moduleDirNameUpper . '_SERVERPATH') . ' <b>' . XOOPS_ROOT_PATH . "</b>\n";
-        $html .= "</ul>\n";
-        $html .= "<br>\n";
-        $html .= constant('CO_' . $moduleDirNameUpper . '_UPLOADPATHDSC') . "\n";
-        $html .= '</div>';
-        $html .= '</fieldset><br>';
+	/**
+	 * serverStats()
+	 *
+	 * @return string
+	 */
+	public static function getServerStats()
+	{
+		//mb    $wfdownloads = WfdownloadsWfdownloads::getInstance();
+		$moduleDirName      = basename(dirname(dirname(__DIR__)));
+		$moduleDirNameUpper = strtoupper($moduleDirName);
+		xoops_loadLanguage('common', $moduleDirName);
+		$html = '';
+		//        $sql   = 'SELECT metavalue';
+		//        $sql   .= ' FROM ' . $GLOBALS['xoopsDB']->prefix('wfdownloads_meta');
+		//        $sql   .= " WHERE metakey='version' LIMIT 1";
+		//        $query = $GLOBALS['xoopsDB']->query($sql);
+		//        list($meta) = $GLOBALS['xoopsDB']->fetchRow($query);
+		$html .= "<fieldset><legend style='font-weight: bold; color: #900;'>" . constant('CO_' . $moduleDirNameUpper . '_IMAGEINFO') . "</legend>\n";
+		$html .= "<div style='padding: 8px;'>\n";
+		//        $html .= '<div>' . constant('CO_' . $moduleDirNameUpper . '_METAVERSION') . $meta . "</div>\n";
+		//        $html .= "<br>\n";
+		//        $html .= "<br>\n";
+		$html .= '<div>' . constant('CO_' . $moduleDirNameUpper . '_SPHPINI') . "</div>\n";
+		$html .= "<ul>\n";
+		//
+		$gdlib = function_exists('gd_info') ? '<span style="color: green;">' . constant('CO_' . $moduleDirNameUpper . '_GDON') . '</span>' : '<span style="color: red;">' . constant('CO_' . $moduleDirNameUpper . '_GDOFF') . '</span>';
+		$html  .= '<li>' . constant('CO_' . $moduleDirNameUpper . '_GDLIBSTATUS') . $gdlib;
+		if (function_exists('gd_info')) {
+			if ($gdlib = true === gd_info()) {
+				$html .= '<li>' . constant('CO_' . $moduleDirNameUpper . '_GDLIBVERSION') . '<b>' . $gdlib['GD Version'] . '</b>';
+			}
+		}
+		//
+		//    $safemode = ini_get('safe_mode') ? constant('CO_' . $moduleDirNameUpper . '_ON') . constant('CO_' . $moduleDirNameUpper . '_SAFEMODEPROBLEMS : constant('CO_' . $moduleDirNameUpper . '_OFF');
+		//    $html .= '<li>' . constant('CO_' . $moduleDirNameUpper . '_SAFEMODESTATUS . $safemode;
+		//
+		//    $registerglobals = (!ini_get('register_globals')) ? "<span style=\"color: green;\">" . constant('CO_' . $moduleDirNameUpper . '_OFF') . '</span>' : "<span style=\"color: red;\">" . constant('CO_' . $moduleDirNameUpper . '_ON') . '</span>';
+		//    $html .= '<li>' . constant('CO_' . $moduleDirNameUpper . '_REGISTERGLOBALS . $registerglobals;
+		//
+		$downloads = ini_get('file_uploads') ? '<span style="color: green;">' . constant('CO_' . $moduleDirNameUpper . '_ON') . '</span>' : '<span style="color: red;">' . constant('CO_' . $moduleDirNameUpper . '_OFF') . '</span>';
+		$html      .= '<li>' . constant('CO_' . $moduleDirNameUpper . '_SERVERUPLOADSTATUS') . $downloads;
+		//
+		$html .= '<li>' . constant('CO_' . $moduleDirNameUpper . '_MAXUPLOADSIZE') . ' <b><span style="color: blue;">' . ini_get('upload_max_filesize') . "</span></b>\n";
+		$html .= '<li>' . constant('CO_' . $moduleDirNameUpper . '_MAXPOSTSIZE') . ' <b><span style="color: blue;">' . ini_get('post_max_size') . "</span></b>\n";
+		$html .= '<li>' . constant('CO_' . $moduleDirNameUpper . '_MEMORYLIMIT') . ' <b><span style="color: blue;">' . ini_get('memory_limit') . "</span></b>\n";
+		$html .= "</ul>\n";
+		$html .= "<ul>\n";
+		$html .= '<li>' . constant('CO_' . $moduleDirNameUpper . '_SERVERPATH') . ' <b>' . XOOPS_ROOT_PATH . "</b>\n";
+		$html .= "</ul>\n";
+		$html .= "<br>\n";
+		$html .= constant('CO_' . $moduleDirNameUpper . '_UPLOADPATHDSC') . "\n";
+		$html .= '</div>';
+		$html .= '</fieldset><br>';
 
-        return $html;
-    }
+		return $html;
+	}
 }

Spacing +2 added lines, -2 removed lines

@@ -42,7 +42,7 @@ 
         $html .= "<ul>\n";
         //
         $gdlib = function_exists('gd_info') ? '<span style="color: green;">' . constant('CO_' . $moduleDirNameUpper . '_GDON') . '</span>' : '<span style="color: red;">' . constant('CO_' . $moduleDirNameUpper . '_GDOFF') . '</span>';
-        $html  .= '<li>' . constant('CO_' . $moduleDirNameUpper . '_GDLIBSTATUS') . $gdlib;
+        $html .= '<li>' . constant('CO_' . $moduleDirNameUpper . '_GDLIBSTATUS') . $gdlib;
         if (function_exists('gd_info')) {
             if ($gdlib = true === gd_info()) {
                 $html .= '<li>' . constant('CO_' . $moduleDirNameUpper . '_GDLIBVERSION') . '<b>' . $gdlib['GD Version'] . '</b>';
@@ -56,7 +56,7 @@ 
         //    $html .= '<li>' . constant('CO_' . $moduleDirNameUpper . '_REGISTERGLOBALS . $registerglobals;
         //
         $downloads = ini_get('file_uploads') ? '<span style="color: green;">' . constant('CO_' . $moduleDirNameUpper . '_ON') . '</span>' : '<span style="color: red;">' . constant('CO_' . $moduleDirNameUpper . '_OFF') . '</span>';
-        $html      .= '<li>' . constant('CO_' . $moduleDirNameUpper . '_SERVERUPLOADSTATUS') . $downloads;
+        $html .= '<li>' . constant('CO_' . $moduleDirNameUpper . '_SERVERUPLOADSTATUS') . $downloads;
         //
         $html .= '<li>' . constant('CO_' . $moduleDirNameUpper . '_MAXUPLOADSIZE') . ' <b><span style="color: blue;">' . ini_get('upload_max_filesize') . "</span></b>\n";
         $html .= '<li>' . constant('CO_' . $moduleDirNameUpper . '_MAXPOSTSIZE') . ' <b><span style="color: blue;">' . ini_get('post_max_size') . "</span></b>\n";