mambax7 / instruction

class/utility.php

Unused Use Statements -1 removed lines

@@ -1,6 +1,5 @@
 <?php
 
-use Xmf\Request;
 use Xoopsmodules\instruction\common;
 
 require_once __DIR__ . '/common/traitversionchecks.php';

Indentation +53 added lines, -53 removed lines

@@ -14,63 +14,63 @@
  */
 class InstructionUtility
 {
-    use common\VersionChecks; //checkVerXoops, checkVerPhp Traits
+	use common\VersionChecks; //checkVerXoops, checkVerPhp Traits
 
-    use common\ServerStats; // getServerStats Trait
+	use common\ServerStats; // getServerStats Trait
 
-    use common\FilesManagement; // Files Management Trait
+	use common\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;
+	}
 }

index.php

Indentation +54 added lines, -54 removed lines

@@ -55,11 +55,11 @@ 
 
 
 if (InstructionUtility::checkVerXoops($module1, '2.5.9')) {
-    $cat_select = $mytree->makeSelectElement('cid', 'title', '--', $cid, true, 0, "onChange='javascript: document.insformselcat.submit()'", '');
-    $GLOBALS['xoopsTpl']->assign('insFormSelCat', $cat_select->render());
+	$cat_select = $mytree->makeSelectElement('cid', 'title', '--', $cid, true, 0, "onChange='javascript: document.insformselcat.submit()'", '');
+	$GLOBALS['xoopsTpl']->assign('insFormSelCat', $cat_select->render());
 } else {
-    $cat_select = $mytree->makeSelBox('cid', 'title', '--', $cid, true, 0, "onChange='javascript: document.insformselcat.submit()'");
-    $GLOBALS['xoopsTpl']->assign('insFormSelCat', $cat_select);
+	$cat_select = $mytree->makeSelBox('cid', 'title', '--', $cid, true, 0, "onChange='javascript: document.insformselcat.submit()'");
+	$GLOBALS['xoopsTpl']->assign('insFormSelCat', $cat_select);
 }
 
 // Находим список всех инструкций
@@ -69,14 +69,14 @@ 
 $criteria->add(new Criteria('status', '0', '>'));
 // Если есть категория
 if ($cid) {
-    // Если нельзя просматривать эту категорию
-    if (!in_array($cid, $categories)) {
-        redirect_header('index.php', 3, _MD_INSTRUCTION_NOPERM_CAT);
-    }
-    $criteria->add(new Criteria('cid', $cid, '='));
-    // Иначе находим список всех
+	// Если нельзя просматривать эту категорию
+	if (!in_array($cid, $categories)) {
+		redirect_header('index.php', 3, _MD_INSTRUCTION_NOPERM_CAT);
+	}
+	$criteria->add(new Criteria('cid', $cid, '='));
+	// Иначе находим список всех
 } else {
-    $criteria->add(new Criteria('cid', '( ' . implode(', ', $categories) . ' )', 'IN'));
+	$criteria->add(new Criteria('cid', '( ' . implode(', ', $categories) . ' )', 'IN'));
 }
 
 // Число инструкций, удовлетворяющих данному условию
@@ -93,10 +93,10 @@ 
 $instr_arr = $insinstrHandler->getall($criteria);
 // Если записей больше чем $limit, то выводим пагинатор
 if ($numrows > $limit) {
-    $pagenav = new XoopsPageNav($numrows, $limit, $start, 'start', 'cid=' . $cid);
-    $pagenav = $pagenav->renderNav(4);
+	$pagenav = new XoopsPageNav($numrows, $limit, $start, 'start', 'cid=' . $cid);
+	$pagenav = $pagenav->renderNav(4);
 } else {
-    $pagenav = '';
+	$pagenav = '';
 }
 // Выводим пагинатор в шаблон
 $GLOBALS['xoopsTpl']->assign('insPagenav', $pagenav);
@@ -107,52 +107,52 @@ 
 
 // Если есть записи
 if ($numrows > 0) {
-    $class = 'odd';
-    foreach (array_keys($instr_arr) as $i) {
-
-        //
-        $class = ('even' == $class) ? 'odd' : 'even';
-        // ID
-        $insinstr_instrid = $instr_arr[$i]->getVar('instrid');
-        // Название
-        $insinstr_title = $instr_arr[$i]->getVar('title');
-        // Статус
-        $insinstr_status = $instr_arr[$i]->getVar('status');
-        // Количество страниц
-        $insinstr_pages = $instr_arr[$i]->getVar('pages');
-        // Категория
-        $insinstr_cid = $instr_arr[$i]->getVar('cid');
-        $insinstr_cat = $inscatHandler->get($insinstr_cid);
-        // Права на добавление
-        $perm_submit = in_array($insinstr_cid, $cat_submit) ? true : false;
-        // Права на редактирование
-        $perm_edit = in_array($insinstr_cid, $cat_edit) ? true : false;
-        //Мета-теги ключевых слов
-        $insinstr_metakeywords = $instr_arr[$i]->getVar('metakeywords');
-        // Если есть - добавляем в мета-теги страницы
-        if ($insinstr_metakeywords) {
-            $index_metakeywords[] = $insinstr_metakeywords;
-        }
-        // Мета-теги описания
-        $insinstr_metadescript = $instr_arr[$i]->getVar('metadescription');
-        // Если есть - добавляем в мета-теги страницы
-        if ($insinstr_metadescript) {
-            $index_metadescript[] = $insinstr_metadescript;
-        }
-
-        // Выводим в шаблон
-        $GLOBALS['xoopsTpl']->append('insListInstr', ['instrid' => $insinstr_instrid, 'title' => $insinstr_title, 'status' => $insinstr_status, 'pages' => $insinstr_pages, 'ctitle' => $insinstr_cat->getVar('title'), 'cid' => $insinstr_cid, 'permsubmit' => $perm_submit, 'permedit' => $perm_edit, 'class' => $class]);
-    }
-
-    // Языковые константы
+	$class = 'odd';
+	foreach (array_keys($instr_arr) as $i) {
+
+		//
+		$class = ('even' == $class) ? 'odd' : 'even';
+		// ID
+		$insinstr_instrid = $instr_arr[$i]->getVar('instrid');
+		// Название
+		$insinstr_title = $instr_arr[$i]->getVar('title');
+		// Статус
+		$insinstr_status = $instr_arr[$i]->getVar('status');
+		// Количество страниц
+		$insinstr_pages = $instr_arr[$i]->getVar('pages');
+		// Категория
+		$insinstr_cid = $instr_arr[$i]->getVar('cid');
+		$insinstr_cat = $inscatHandler->get($insinstr_cid);
+		// Права на добавление
+		$perm_submit = in_array($insinstr_cid, $cat_submit) ? true : false;
+		// Права на редактирование
+		$perm_edit = in_array($insinstr_cid, $cat_edit) ? true : false;
+		//Мета-теги ключевых слов
+		$insinstr_metakeywords = $instr_arr[$i]->getVar('metakeywords');
+		// Если есть - добавляем в мета-теги страницы
+		if ($insinstr_metakeywords) {
+			$index_metakeywords[] = $insinstr_metakeywords;
+		}
+		// Мета-теги описания
+		$insinstr_metadescript = $instr_arr[$i]->getVar('metadescription');
+		// Если есть - добавляем в мета-теги страницы
+		if ($insinstr_metadescript) {
+			$index_metadescript[] = $insinstr_metadescript;
+		}
+
+		// Выводим в шаблон
+		$GLOBALS['xoopsTpl']->append('insListInstr', ['instrid' => $insinstr_instrid, 'title' => $insinstr_title, 'status' => $insinstr_status, 'pages' => $insinstr_pages, 'ctitle' => $insinstr_cat->getVar('title'), 'cid' => $insinstr_cid, 'permsubmit' => $perm_submit, 'permedit' => $perm_edit, 'class' => $class]);
+	}
+
+	// Языковые константы
 }
 
 // Если есть мета-теги
 if (count($index_metakeywords)) {
-    $xoTheme->addMeta('meta', 'keywords', implode(', ', $index_metakeywords));
+	$xoTheme->addMeta('meta', 'keywords', implode(', ', $index_metakeywords));
 }
 if (count($index_metadescript)) {
-    $xoTheme->addMeta('meta', 'description', implode(', ', $index_metadescript));
+	$xoTheme->addMeta('meta', 'description', implode(', ', $index_metadescript));
 }
 
 // Подвал

class/common/traitversionchecks.php

Indentation +69 added lines, -69 removed lines

@@ -16,77 +16,77 @@
  */
 trait VersionChecks
 {
-    /**
-     *
-     * Verifies XOOPS version meets minimum requirements for this module
-     * @static
-     * @param \XoopsModule|null $module
-     *
-     * @param null|string $requiredVer
-     * @return bool true if meets requirements, false if not
-     */
-    public static function checkVerXoops(\XoopsModule $module = null, $requiredVer = null)
-    {
-        $moduleDirName = basename(dirname(dirname(__DIR__)));
-        if (null === $module) {
-            $module = XoopsModule::getByDirname($moduleDirName);
-        }
-        xoops_loadLanguage('admin', $moduleDirName);
-        //check for minimum XOOPS version
-        $currentVer = substr(XOOPS_VERSION, 6); // get the numeric part of string
-        $currArray  = explode('.', $currentVer);
-        if (null === $requiredVer) {
-            $requiredVer = '' . $module->getInfo('min_xoops'); //making sure it's a string
-        }
-        $reqArray = explode('.', $requiredVer);
-        $success  = true;
-        foreach ($reqArray as $k => $v) {
-            if (isset($currArray[$k])) {
-                if ($currArray[$k] > $v) {
-                    break;
-                } elseif ($currArray[$k] == $v) {
-                    continue;
-                } else {
-                    $success = false;
-                    break;
-                }
-            } else {
-                if ((int)$v > 0) { // handles versions like x.x.x.0_RC2
-                    $success = false;
-                    break;
-                }
-            }
-        }
+	/**
+	 *
+	 * Verifies XOOPS version meets minimum requirements for this module
+	 * @static
+	 * @param \XoopsModule|null $module
+	 *
+	 * @param null|string $requiredVer
+	 * @return bool true if meets requirements, false if not
+	 */
+	public static function checkVerXoops(\XoopsModule $module = null, $requiredVer = null)
+	{
+		$moduleDirName = basename(dirname(dirname(__DIR__)));
+		if (null === $module) {
+			$module = XoopsModule::getByDirname($moduleDirName);
+		}
+		xoops_loadLanguage('admin', $moduleDirName);
+		//check for minimum XOOPS version
+		$currentVer = substr(XOOPS_VERSION, 6); // get the numeric part of string
+		$currArray  = explode('.', $currentVer);
+		if (null === $requiredVer) {
+			$requiredVer = '' . $module->getInfo('min_xoops'); //making sure it's a string
+		}
+		$reqArray = explode('.', $requiredVer);
+		$success  = true;
+		foreach ($reqArray as $k => $v) {
+			if (isset($currArray[$k])) {
+				if ($currArray[$k] > $v) {
+					break;
+				} elseif ($currArray[$k] == $v) {
+					continue;
+				} else {
+					$success = false;
+					break;
+				}
+			} else {
+				if ((int)$v > 0) { // handles versions like x.x.x.0_RC2
+					$success = false;
+					break;
+				}
+			}
+		}
 
-        if (false === $success) {
-            $module->setErrors(sprintf(_AM_WFL_ERROR_BAD_XOOPS, $requiredVer, $currentVer));
-        }
+		if (false === $success) {
+			$module->setErrors(sprintf(_AM_WFL_ERROR_BAD_XOOPS, $requiredVer, $currentVer));
+		}
 
-        return $success;
-    }
+		return $success;
+	}
 
-    /**
-     *
-     * Verifies PHP version meets minimum requirements for this module
-     * @static
-     * @param \XoopsModule $module
-     *
-     * @return bool true if meets requirements, false if not
-     */
-    public static function checkVerPhp(\XoopsModule $module)
-    {
-        xoops_loadLanguage('admin', $module->dirname());
-        // check for minimum PHP version
-        $success = true;
-        $verNum  = PHP_VERSION;
-        $reqVer  = $module->getInfo('min_php');
-        if (false !== $reqVer && '' !== $reqVer) {
-            if (version_compare($verNum, $reqVer, '<')) {
-                $module->setErrors(sprintf(_AM_WFL_ERROR_BAD_PHP, $reqVer, $verNum));
-                $success = false;
-            }
-        }
+	/**
+	 *
+	 * Verifies PHP version meets minimum requirements for this module
+	 * @static
+	 * @param \XoopsModule $module
+	 *
+	 * @return bool true if meets requirements, false if not
+	 */
+	public static function checkVerPhp(\XoopsModule $module)
+	{
+		xoops_loadLanguage('admin', $module->dirname());
+		// check for minimum PHP version
+		$success = true;
+		$verNum  = PHP_VERSION;
+		$reqVer  = $module->getInfo('min_php');
+		if (false !== $reqVer && '' !== $reqVer) {
+			if (version_compare($verNum, $reqVer, '<')) {
+				$module->setErrors(sprintf(_AM_WFL_ERROR_BAD_PHP, $reqVer, $verNum));
+				$success = false;
+			}
+		}
 
-        return $success;
-    }
+		return $success;
+	}
 }

class/common/breadcrumb.php

Indentation +39 added lines, -39 removed lines

@@ -32,48 +32,48 @@
  */
 class Breadcrumb
 {
-    public $dirname;
-    private $bread = [];
+	public $dirname;
+	private $bread = [];
 
-    /**
-     *
-     */
-    public function __construct()
-    {
-        $this->dirname = basename(dirname(__DIR__));
-    }
+	/**
+	 *
+	 */
+	public function __construct()
+	{
+		$this->dirname = basename(dirname(__DIR__));
+	}
 
-    /**
-     * Add link to breadcrumb
-     *
-     * @param string $title
-     * @param string $link
-     */
-    public function addLink($title = '', $link = '')
-    {
-        $this->bread[] = [
-            'link'  => $link,
-            'title' => $title
-        ];
-    }
+	/**
+	 * Add link to breadcrumb
+	 *
+	 * @param string $title
+	 * @param string $link
+	 */
+	public function addLink($title = '', $link = '')
+	{
+		$this->bread[] = [
+			'link'  => $link,
+			'title' => $title
+		];
+	}
 
-    /**
-     * Render Pedigree BreadCrumb
-     *
-     */
-    public function render()
-    {
-        if (!isset($GLOBALS['xoTheme']) || !is_object($GLOBALS['xoTheme'])) {
-            require_once $GLOBALS['xoops']->path('class/theme.php');
-            $GLOBALS['xoTheme'] = new xos_opal_Theme();
-        }
+	/**
+	 * Render Pedigree BreadCrumb
+	 *
+	 */
+	public function render()
+	{
+		if (!isset($GLOBALS['xoTheme']) || !is_object($GLOBALS['xoTheme'])) {
+			require_once $GLOBALS['xoops']->path('class/theme.php');
+			$GLOBALS['xoTheme'] = new xos_opal_Theme();
+		}
 
-        require_once $GLOBALS['xoops']->path('class/template.php');
-        $breadcrumbTpl = new XoopsTpl();
-        $breadcrumbTpl->assign('breadcrumb', $this->bread);
-        $html = $breadcrumbTpl->fetch('db:' . $this->dirname . '_common_breadcrumb.tpl');
-        unset($breadcrumbTpl);
+		require_once $GLOBALS['xoops']->path('class/template.php');
+		$breadcrumbTpl = new XoopsTpl();
+		$breadcrumbTpl->assign('breadcrumb', $this->bread);
+		$html = $breadcrumbTpl->fetch('db:' . $this->dirname . '_common_breadcrumb.tpl');
+		unset($breadcrumbTpl);
 
-        return $html;
-    }
+		return $html;
+	}
 }