mambax7 / instruction

class/common/traitfilesmgmt.php

Indentation +222 added lines, -222 removed lines

@@ -16,226 +16,226 @@
  */
 trait FilesManagement
 {
-    /**
-     * Function responsible for checking if a directory exists, we can also write in and create an index.html file
-     *
-     * @param string $folder The full path of the directory to check
-     *
-     * @return void
-     */
-    public static function createFolder($folder)
-    {
-        try {
-            if (!file_exists($folder)) {
-                if (!mkdir($folder) && !is_dir($folder)) {
-                    throw new \RuntimeException(sprintf('Unable to create the %s directory', $folder));
-                }
-
-                file_put_contents($folder . '/index.html', '<script>history.go(-1);</script>');
-            }
-        } catch (Exception $e) {
-            echo 'Caught exception: ', $e->getMessage(), "\n", '<br>';
-        }
-    }
-
-    /**
-     * @param $file
-     * @param $folder
-     * @return bool
-     */
-    public static function copyFile($file, $folder)
-    {
-        return copy($file, $folder);
-    }
-
-    /**
-     * @param $src
-     * @param $dst
-     */
-    public static function recurseCopy($src, $dst)
-    {
-        $dir = opendir($src);
-        @mkdir($dst);
-        while (false !== ($file = readdir($dir))) {
-            if (('.' !== $file) && ('..' !== $file)) {
-                if (is_dir($src . '/' . $file)) {
-                    self::recurseCopy($src . '/' . $file, $dst . '/' . $file);
-                } else {
-                    copy($src . '/' . $file, $dst . '/' . $file);
-                }
-            }
-        }
-        closedir($dir);
-    }
-
-    /**
-     *
-     * Remove files and (sub)directories
-     *
-     * @param string $src source directory to delete
-     *
-     * @uses \Xmf\Module\Helper::getHelper()
-     * @uses \Xmf\Module\Helper::isUserAdmin()
-     *
-     * @return bool true on success
-     */
-    public static function deleteDirectory($src)
-    {
-        // Only continue if user is a 'global' Admin
-        if (!($GLOBALS['xoopsUser'] instanceof XoopsUser) || !$GLOBALS['xoopsUser']->isAdmin()) {
-            return false;
-        }
-
-        $success = true;
-        // remove old files
-        $dirInfo = new SplFileInfo($src);
-        // validate is a directory
-        if ($dirInfo->isDir()) {
-            $fileList = array_diff(scandir($src, SCANDIR_SORT_NONE), ['..', '.']);
-            foreach ($fileList as $k => $v) {
-                $fileInfo = new SplFileInfo("{$src}/{$v}");
-                if ($fileInfo->isDir()) {
-                    // recursively handle subdirectories
-                    if (!$success = self::deleteDirectory($fileInfo->getRealPath())) {
-                        break;
-                    }
-                } else {
-                    // delete the file
-                    if (!($success = unlink($fileInfo->getRealPath()))) {
-                        break;
-                    }
-                }
-            }
-            // now delete this (sub)directory if all the files are gone
-            if ($success) {
-                $success = rmdir($dirInfo->getRealPath());
-            }
-        } else {
-            // input is not a valid directory
-            $success = false;
-        }
-        return $success;
-    }
-
-    /**
-     *
-     * Recursively remove directory
-     *
-     * @todo currently won't remove directories with hidden files, should it?
-     *
-     * @param string $src directory to remove (delete)
-     *
-     * @return bool true on success
-     */
-    public static function rrmdir($src)
-    {
-        // Only continue if user is a 'global' Admin
-        if (!($GLOBALS['xoopsUser'] instanceof XoopsUser) || !$GLOBALS['xoopsUser']->isAdmin()) {
-            return false;
-        }
-
-        // If source is not a directory stop processing
-        if (!is_dir($src)) {
-            return false;
-        }
-
-        $success = true;
-
-        // Open the source directory to read in files
-        $iterator = new DirectoryIterator($src);
-        foreach ($iterator as $fObj) {
-            if ($fObj->isFile()) {
-                $filename = $fObj->getPathname();
-                $fObj     = null; // clear this iterator object to close the file
-                if (!unlink($filename)) {
-                    return false; // couldn't delete the file
-                }
-            } elseif (!$fObj->isDot() && $fObj->isDir()) {
-                // Try recursively on directory
-                self::rrmdir($fObj->getPathname());
-            }
-        }
-        $iterator = null;   // clear iterator Obj to close file/directory
-        return rmdir($src); // remove the directory & return results
-    }
-
-    /**
-     * Recursively move files from one directory to another
-     *
-     * @param string $src  - Source of files being moved
-     * @param string $dest - Destination of files being moved
-     *
-     * @return bool true on success
-     */
-    public static function rmove($src, $dest)
-    {
-        // Only continue if user is a 'global' Admin
-        if (!($GLOBALS['xoopsUser'] instanceof XoopsUser) || !$GLOBALS['xoopsUser']->isAdmin()) {
-            return false;
-        }
-
-        // If source is not a directory stop processing
-        if (!is_dir($src)) {
-            return false;
-        }
-
-        // If the destination directory does not exist and could not be created stop processing
-        if (!is_dir($dest) && !mkdir($dest, 0755)) {
-            return false;
-        }
-
-        // Open the source directory to read in files
-        $iterator = new DirectoryIterator($src);
-        foreach ($iterator as $fObj) {
-            if ($fObj->isFile()) {
-                rename($fObj->getPathname(), "{$dest}/" . $fObj->getFilename());
-            } elseif (!$fObj->isDot() && $fObj->isDir()) {
-                // Try recursively on directory
-                self::rmove($fObj->getPathname(), "{$dest}/" . $fObj->getFilename());
-                //                rmdir($fObj->getPath()); // now delete the directory
-            }
-        }
-        $iterator = null;   // clear iterator Obj to close file/directory
-        return rmdir($src); // remove the directory & return results
-    }
-
-    /**
-     * Recursively copy directories and files from one directory to another
-     *
-     * @param string $src  - Source of files being moved
-     * @param string $dest - Destination of files being moved
-     *
-     * @uses \Xmf\Module\Helper::getHelper()
-     * @uses \Xmf\Module\Helper::isUserAdmin()
-     *
-     * @return bool true on success
-     */
-    public static function rcopy($src, $dest)
-    {
-        // Only continue if user is a 'global' Admin
-        if (!($GLOBALS['xoopsUser'] instanceof XoopsUser) || !$GLOBALS['xoopsUser']->isAdmin()) {
-            return false;
-        }
-
-        // If source is not a directory stop processing
-        if (!is_dir($src)) {
-            return false;
-        }
-
-        // If the destination directory does not exist and could not be created stop processing
-        if (!is_dir($dest) && !mkdir($dest, 0755)) {
-            return false;
-        }
-
-        // Open the source directory to read in files
-        $iterator = new DirectoryIterator($src);
-        foreach ($iterator as $fObj) {
-            if ($fObj->isFile()) {
-                copy($fObj->getPathname(), "{$dest}/" . $fObj->getFilename());
-            } elseif (!$fObj->isDot() && $fObj->isDir()) {
-                self::rcopy($fObj->getPathname(), "{$dest}/" . $fObj->getFilename());
-            }
-        }
-        return true;
-    }
+	/**
+	 * Function responsible for checking if a directory exists, we can also write in and create an index.html file
+	 *
+	 * @param string $folder The full path of the directory to check
+	 *
+	 * @return void
+	 */
+	public static function createFolder($folder)
+	{
+		try {
+			if (!file_exists($folder)) {
+				if (!mkdir($folder) && !is_dir($folder)) {
+					throw new \RuntimeException(sprintf('Unable to create the %s directory', $folder));
+				}
+
+				file_put_contents($folder . '/index.html', '<script>history.go(-1);</script>');
+			}
+		} catch (Exception $e) {
+			echo 'Caught exception: ', $e->getMessage(), "\n", '<br>';
+		}
+	}
+
+	/**
+	 * @param $file
+	 * @param $folder
+	 * @return bool
+	 */
+	public static function copyFile($file, $folder)
+	{
+		return copy($file, $folder);
+	}
+
+	/**
+	 * @param $src
+	 * @param $dst
+	 */
+	public static function recurseCopy($src, $dst)
+	{
+		$dir = opendir($src);
+		@mkdir($dst);
+		while (false !== ($file = readdir($dir))) {
+			if (('.' !== $file) && ('..' !== $file)) {
+				if (is_dir($src . '/' . $file)) {
+					self::recurseCopy($src . '/' . $file, $dst . '/' . $file);
+				} else {
+					copy($src . '/' . $file, $dst . '/' . $file);
+				}
+			}
+		}
+		closedir($dir);
+	}
+
+	/**
+	 *
+	 * Remove files and (sub)directories
+	 *
+	 * @param string $src source directory to delete
+	 *
+	 * @uses \Xmf\Module\Helper::getHelper()
+	 * @uses \Xmf\Module\Helper::isUserAdmin()
+	 *
+	 * @return bool true on success
+	 */
+	public static function deleteDirectory($src)
+	{
+		// Only continue if user is a 'global' Admin
+		if (!($GLOBALS['xoopsUser'] instanceof XoopsUser) || !$GLOBALS['xoopsUser']->isAdmin()) {
+			return false;
+		}
+
+		$success = true;
+		// remove old files
+		$dirInfo = new SplFileInfo($src);
+		// validate is a directory
+		if ($dirInfo->isDir()) {
+			$fileList = array_diff(scandir($src, SCANDIR_SORT_NONE), ['..', '.']);
+			foreach ($fileList as $k => $v) {
+				$fileInfo = new SplFileInfo("{$src}/{$v}");
+				if ($fileInfo->isDir()) {
+					// recursively handle subdirectories
+					if (!$success = self::deleteDirectory($fileInfo->getRealPath())) {
+						break;
+					}
+				} else {
+					// delete the file
+					if (!($success = unlink($fileInfo->getRealPath()))) {
+						break;
+					}
+				}
+			}
+			// now delete this (sub)directory if all the files are gone
+			if ($success) {
+				$success = rmdir($dirInfo->getRealPath());
+			}
+		} else {
+			// input is not a valid directory
+			$success = false;
+		}
+		return $success;
+	}
+
+	/**
+	 *
+	 * Recursively remove directory
+	 *
+	 * @todo currently won't remove directories with hidden files, should it?
+	 *
+	 * @param string $src directory to remove (delete)
+	 *
+	 * @return bool true on success
+	 */
+	public static function rrmdir($src)
+	{
+		// Only continue if user is a 'global' Admin
+		if (!($GLOBALS['xoopsUser'] instanceof XoopsUser) || !$GLOBALS['xoopsUser']->isAdmin()) {
+			return false;
+		}
+
+		// If source is not a directory stop processing
+		if (!is_dir($src)) {
+			return false;
+		}
+
+		$success = true;
+
+		// Open the source directory to read in files
+		$iterator = new DirectoryIterator($src);
+		foreach ($iterator as $fObj) {
+			if ($fObj->isFile()) {
+				$filename = $fObj->getPathname();
+				$fObj     = null; // clear this iterator object to close the file
+				if (!unlink($filename)) {
+					return false; // couldn't delete the file
+				}
+			} elseif (!$fObj->isDot() && $fObj->isDir()) {
+				// Try recursively on directory
+				self::rrmdir($fObj->getPathname());
+			}
+		}
+		$iterator = null;   // clear iterator Obj to close file/directory
+		return rmdir($src); // remove the directory & return results
+	}
+
+	/**
+	 * Recursively move files from one directory to another
+	 *
+	 * @param string $src  - Source of files being moved
+	 * @param string $dest - Destination of files being moved
+	 *
+	 * @return bool true on success
+	 */
+	public static function rmove($src, $dest)
+	{
+		// Only continue if user is a 'global' Admin
+		if (!($GLOBALS['xoopsUser'] instanceof XoopsUser) || !$GLOBALS['xoopsUser']->isAdmin()) {
+			return false;
+		}
+
+		// If source is not a directory stop processing
+		if (!is_dir($src)) {
+			return false;
+		}
+
+		// If the destination directory does not exist and could not be created stop processing
+		if (!is_dir($dest) && !mkdir($dest, 0755)) {
+			return false;
+		}
+
+		// Open the source directory to read in files
+		$iterator = new DirectoryIterator($src);
+		foreach ($iterator as $fObj) {
+			if ($fObj->isFile()) {
+				rename($fObj->getPathname(), "{$dest}/" . $fObj->getFilename());
+			} elseif (!$fObj->isDot() && $fObj->isDir()) {
+				// Try recursively on directory
+				self::rmove($fObj->getPathname(), "{$dest}/" . $fObj->getFilename());
+				//                rmdir($fObj->getPath()); // now delete the directory
+			}
+		}
+		$iterator = null;   // clear iterator Obj to close file/directory
+		return rmdir($src); // remove the directory & return results
+	}
+
+	/**
+	 * Recursively copy directories and files from one directory to another
+	 *
+	 * @param string $src  - Source of files being moved
+	 * @param string $dest - Destination of files being moved
+	 *
+	 * @uses \Xmf\Module\Helper::getHelper()
+	 * @uses \Xmf\Module\Helper::isUserAdmin()
+	 *
+	 * @return bool true on success
+	 */
+	public static function rcopy($src, $dest)
+	{
+		// Only continue if user is a 'global' Admin
+		if (!($GLOBALS['xoopsUser'] instanceof XoopsUser) || !$GLOBALS['xoopsUser']->isAdmin()) {
+			return false;
+		}
+
+		// If source is not a directory stop processing
+		if (!is_dir($src)) {
+			return false;
+		}
+
+		// If the destination directory does not exist and could not be created stop processing
+		if (!is_dir($dest) && !mkdir($dest, 0755)) {
+			return false;
+		}
+
+		// Open the source directory to read in files
+		$iterator = new DirectoryIterator($src);
+		foreach ($iterator as $fObj) {
+			if ($fObj->isFile()) {
+				copy($fObj->getPathname(), "{$dest}/" . $fObj->getFilename());
+			} elseif (!$fObj->isDot() && $fObj->isDir()) {
+				self::rcopy($fObj->getPathname(), "{$dest}/" . $fObj->getFilename());
+			}
+		}
+		return true;
+	}
 }

class/helper.php

Indentation +29 added lines, -29 removed lines

@@ -23,37 +23,37 @@
  */
 class Instruction extends \Xmf\Module\Helper
 {
-    public $debugArray = [];
+	public $debugArray = [];
 
-    /**
-     * @internal param $debug
-     */
-    protected function __construct()
-    {
-        //        $this->debug   = $debug;
-        $this->dirname = basename(dirname(__DIR__));
-    }
+	/**
+	 * @internal param $debug
+	 */
+	protected function __construct()
+	{
+		//        $this->debug   = $debug;
+		$this->dirname = basename(dirname(__DIR__));
+	}
 
-    /**
-     * @param bool $debug
-     *
-     * @return Newbb
-     */
-    public static function getInstance($debug = false)
-    {
-        static $instance;
-        if (null === $instance) {
-            $instance = new static($debug);
-        }
+	/**
+	 * @param bool $debug
+	 *
+	 * @return Newbb
+	 */
+	public static function getInstance($debug = false)
+	{
+		static $instance;
+		if (null === $instance) {
+			$instance = new static($debug);
+		}
 
-        return $instance;
-    }
+		return $instance;
+	}
 
-    /**
-     * @return string
-     */
-    public function getDirname()
-    {
-        return $this->dirname;
-    }
+	/**
+	 * @return string
+	 */
+	public function getDirname()
+	{
+		return $this->dirname;
+	}
 }

class/utility.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;
+	}
 }

xoops_version.php

Indentation +169 added lines, -169 removed lines

@@ -10,62 +10,62 @@ 
 $xoops_url     = parse_url(XOOPS_URL);
 
 $modversion = [
-    'version'             => 1.06,
-    'module_status'       => 'Beta 3',
-    'release_date'        => '2017/05/11',
-    'name'                => _MI_INSTRUCTION_NAME,
-    'description'         => _MI_INSTRUCTION_DESC,
-    'credits'             => 'radio-hobby.org, www.shmel.org',
-    'author'              => 'andrey3761, aerograf',
-    'nickname'            => '',
-    'help'                => 'page=help',
-    'license'             => 'GNU GPL 2.0',
-    'license_url'         => 'www.gnu.org/licenses/gpl-2.0.html/',
-    'official'            => 0,
-    'image'               => 'assets/images/slogo.png',
-    'dirname'             => $moduleDirName,
-    'modicons16'          => 'assets/images/icons/16',
-    'modicons32'          => 'assets/images/icons/32',
-    // О модуле
-    'module_website_url'  => 'radio-hobby.org',
-    'module_website_name' => 'radio-hobby.org',
+	'version'             => 1.06,
+	'module_status'       => 'Beta 3',
+	'release_date'        => '2017/05/11',
+	'name'                => _MI_INSTRUCTION_NAME,
+	'description'         => _MI_INSTRUCTION_DESC,
+	'credits'             => 'radio-hobby.org, www.shmel.org',
+	'author'              => 'andrey3761, aerograf',
+	'nickname'            => '',
+	'help'                => 'page=help',
+	'license'             => 'GNU GPL 2.0',
+	'license_url'         => 'www.gnu.org/licenses/gpl-2.0.html/',
+	'official'            => 0,
+	'image'               => 'assets/images/slogo.png',
+	'dirname'             => $moduleDirName,
+	'modicons16'          => 'assets/images/icons/16',
+	'modicons32'          => 'assets/images/icons/32',
+	// О модуле
+	'module_website_url'  => 'radio-hobby.org',
+	'module_website_name' => 'radio-hobby.org',
 
-    'author_website_url'  => 'radio-hobby.org',
-    'author_website_name' => 'andrey3761',
-    'module_website_url'  => 'www.xoops.org',
-    'module_website_name' => 'Support site',
-    'min_php'             => '5.5',
-    'min_xoops'           => '2.5.8',
-    'min_admin'           => '1.1',
-    'min_db'              => ['mysql' => '5.5'],
-    // Файл базы данных
-    'sqlfile'             => ['mysql' => 'sql/mysql.sql'],
-    // Таблицы
-    'tables'              => [
-        $moduleDirName . '_cat',
-        $moduleDirName . '_instr',
-        $moduleDirName . '_page'
-    ],
-    // Имеет админку
-    'hasAdmin'            => 1,
-    'adminindex'          => 'admin/index.php',
-    'adminmenu'           => 'admin/menu.php',
-    'system_menu'         => 1,
-    // Меню
-    'hasMain'             => 1,
-    // Search
-    'hasSearch'           => 1,
-    'search'              => [
-        'file' => 'include/search.inc.php',
-        'func' => $moduleDirName . '_search',
-    ],
+	'author_website_url'  => 'radio-hobby.org',
+	'author_website_name' => 'andrey3761',
+	'module_website_url'  => 'www.xoops.org',
+	'module_website_name' => 'Support site',
+	'min_php'             => '5.5',
+	'min_xoops'           => '2.5.8',
+	'min_admin'           => '1.1',
+	'min_db'              => ['mysql' => '5.5'],
+	// Файл базы данных
+	'sqlfile'             => ['mysql' => 'sql/mysql.sql'],
+	// Таблицы
+	'tables'              => [
+		$moduleDirName . '_cat',
+		$moduleDirName . '_instr',
+		$moduleDirName . '_page'
+	],
+	// Имеет админку
+	'hasAdmin'            => 1,
+	'adminindex'          => 'admin/index.php',
+	'adminmenu'           => 'admin/menu.php',
+	'system_menu'         => 1,
+	// Меню
+	'hasMain'             => 1,
+	// Search
+	'hasSearch'           => 1,
+	'search'              => [
+		'file' => 'include/search.inc.php',
+		'func' => $moduleDirName . '_search',
+	],
 ];
 //  Help files
 $modversion['helpsection'] = [
-    ['name' => _MI_INSTRUCTION_HELP_OVERVIEW, 'link' => 'page=help'],
-    ['name' => _MI_INSTRUCTION_DISCLAIMER, 'link' => 'page=disclaimer'],
-    ['name' => _MI_INSTRUCTION_LICENSE, 'link' => 'page=license'],
-    ['name' => _MI_INSTRUCTION_SUPPORT, 'link' => 'page=support'],
+	['name' => _MI_INSTRUCTION_HELP_OVERVIEW, 'link' => 'page=help'],
+	['name' => _MI_INSTRUCTION_DISCLAIMER, 'link' => 'page=disclaimer'],
+	['name' => _MI_INSTRUCTION_LICENSE, 'link' => 'page=license'],
+	['name' => _MI_INSTRUCTION_SUPPORT, 'link' => 'page=support'],
 ];
 
 // Comments
@@ -78,144 +78,144 @@ 
 
 // Templates
 $modversion['templates'] = [
-    [
-        'file'        => 'admin/' . $moduleDirName . '_admin_index.tpl',
-        'description' => ''
-    ],
-    [
-        'file'        => 'admin/' . $moduleDirName . '_admin_cat.tpl',
-        'description' => ''
-    ],
-    [
-        'file'        => 'admin/' . $moduleDirName . '_admin_editcat.tpl',
-        'description' => ''
-    ],
-    [
-        'file'        => 'admin/' . $moduleDirName . '_admin_savecat.tpl',
-        'description' => ''
-    ],
-    [
-        'file'        => 'admin/' . $moduleDirName . '_admin_viewcat.tpl',
-        'description' => ''
-    ],
-    [
-        'file'        => 'admin/' . $moduleDirName . '_admin_instr.tpl',
-        'description' => ''
-    ],
-    [
-        'file'        => 'admin/' . $moduleDirName . '_admin_editinstr.tpl',
-        'description' => ''
-    ],
-    [
-        'file'        => 'admin/' . $moduleDirName . '_admin_saveinstr.tpl',
-        'description' => ''
-    ],
-    [
-        'file'        => 'admin/' . $moduleDirName . '_admin_viewinstr.tpl',
-        'description' => ''
-    ],
-    [
-        'file'        => 'admin/' . $moduleDirName . '_admin_editpage.tpl',
-        'description' => ''
-    ],
-    [
-        'file'        => 'admin/' . $moduleDirName . '_admin_savepage.tpl',
-        'description' => ''
-    ],
-    [
-        'file'        => 'admin/' . $moduleDirName . '_admin_perm.tpl',
-        'description' => ''
-    ],
-    [
-        'file'        => 'admin/' . $moduleDirName . '_admin_about.tpl',
-        'description' => ''
-    ],
-    [
-        'file'        => $moduleDirName . '_page.tpl',
-        'description' => ''
-    ],
-    [
-        'file'        => $moduleDirName . '_instr.tpl',
-        'description' => ''
-    ],
-    [
-        'file'        => $moduleDirName . '_index.tpl',
-        'description' => ''
-    ],
-    [
-        'file'        => $moduleDirName . '_editpage.tpl',
-        'description' => ''
-    ],
-    [
-        'file'        => $moduleDirName . '_savepage.tpl',
-        'description' => ''
-    ],
+	[
+		'file'        => 'admin/' . $moduleDirName . '_admin_index.tpl',
+		'description' => ''
+	],
+	[
+		'file'        => 'admin/' . $moduleDirName . '_admin_cat.tpl',
+		'description' => ''
+	],
+	[
+		'file'        => 'admin/' . $moduleDirName . '_admin_editcat.tpl',
+		'description' => ''
+	],
+	[
+		'file'        => 'admin/' . $moduleDirName . '_admin_savecat.tpl',
+		'description' => ''
+	],
+	[
+		'file'        => 'admin/' . $moduleDirName . '_admin_viewcat.tpl',
+		'description' => ''
+	],
+	[
+		'file'        => 'admin/' . $moduleDirName . '_admin_instr.tpl',
+		'description' => ''
+	],
+	[
+		'file'        => 'admin/' . $moduleDirName . '_admin_editinstr.tpl',
+		'description' => ''
+	],
+	[
+		'file'        => 'admin/' . $moduleDirName . '_admin_saveinstr.tpl',
+		'description' => ''
+	],
+	[
+		'file'        => 'admin/' . $moduleDirName . '_admin_viewinstr.tpl',
+		'description' => ''
+	],
+	[
+		'file'        => 'admin/' . $moduleDirName . '_admin_editpage.tpl',
+		'description' => ''
+	],
+	[
+		'file'        => 'admin/' . $moduleDirName . '_admin_savepage.tpl',
+		'description' => ''
+	],
+	[
+		'file'        => 'admin/' . $moduleDirName . '_admin_perm.tpl',
+		'description' => ''
+	],
+	[
+		'file'        => 'admin/' . $moduleDirName . '_admin_about.tpl',
+		'description' => ''
+	],
+	[
+		'file'        => $moduleDirName . '_page.tpl',
+		'description' => ''
+	],
+	[
+		'file'        => $moduleDirName . '_instr.tpl',
+		'description' => ''
+	],
+	[
+		'file'        => $moduleDirName . '_index.tpl',
+		'description' => ''
+	],
+	[
+		'file'        => $moduleDirName . '_editpage.tpl',
+		'description' => ''
+	],
+	[
+		'file'        => $moduleDirName . '_savepage.tpl',
+		'description' => ''
+	],
 ];
 // Конфигурация
 $modversion['config'][] = [
-    'name'        => 'form_options',
-    'title'       => '_MI_INSTRUCTION_FORM_OPTIONS',
-    'description' => '_MI_INSTRUCTION_FORM_OPTIONS_DESC',
-    'formtype'    => 'select',
-    'valuetype'   => 'text',
-    'default'     => 'dhtml',
-    'options'     => array_flip($editorHandler->getList())
+	'name'        => 'form_options',
+	'title'       => '_MI_INSTRUCTION_FORM_OPTIONS',
+	'description' => '_MI_INSTRUCTION_FORM_OPTIONS_DESC',
+	'formtype'    => 'select',
+	'valuetype'   => 'text',
+	'default'     => 'dhtml',
+	'options'     => array_flip($editorHandler->getList())
 ];
 $modversion['config'][] = [
-    'name'        => 'perpageadmin',
-    'title'       => '_MI_INSTRUCTION_PERPAGEADMIN',
-    'description' => '_MI_INSTRUCTION_PERPAGEADMINDSC',
-    'formtype'    => 'textbox',
-    'valuetype'   => 'int',
-    'default'     => 20
+	'name'        => 'perpageadmin',
+	'title'       => '_MI_INSTRUCTION_PERPAGEADMIN',
+	'description' => '_MI_INSTRUCTION_PERPAGEADMINDSC',
+	'formtype'    => 'textbox',
+	'valuetype'   => 'int',
+	'default'     => 20
 ];
 $modversion['config'][] = [
-    'name'        => 'perpagemain',
-    'title'       => '_MI_INSTRUCTION_PERPAGEMAIN',
-    'description' => '_MI_INSTRUCTION_PERPAGEMAINDSC',
-    'formtype'    => 'textbox',
-    'valuetype'   => 'int',
-    'default'     => 20
+	'name'        => 'perpagemain',
+	'title'       => '_MI_INSTRUCTION_PERPAGEMAIN',
+	'description' => '_MI_INSTRUCTION_PERPAGEMAINDSC',
+	'formtype'    => 'textbox',
+	'valuetype'   => 'int',
+	'default'     => 20
 ];
 // Теги
 $modversion['config'][] = [
-    'name'        => 'usetag',
-    'title'       => '_MI_INSTRUCTION_USETAG',
-    'description' => '_MI_INSTRUCTION_USETAGDSC',
-    'formtype'    => 'yesno',
-    'valuetype'   => 'int',
-    'default'     => 0
+	'name'        => 'usetag',
+	'title'       => '_MI_INSTRUCTION_USETAG',
+	'description' => '_MI_INSTRUCTION_USETAGDSC',
+	'formtype'    => 'yesno',
+	'valuetype'   => 'int',
+	'default'     => 0
 ];
 // Оценки
 $modversion['config'][] = [
-    'name'        => 'userat',
-    'title'       => '_MI_INSTRUCTION_USERAT',
-    'description' => '_MI_INSTRUCTION_USERATDSC',
-    'formtype'    => 'yesno',
-    'valuetype'   => 'int',
-    'default'     => 0
+	'name'        => 'userat',
+	'title'       => '_MI_INSTRUCTION_USERAT',
+	'description' => '_MI_INSTRUCTION_USERATDSC',
+	'formtype'    => 'yesno',
+	'valuetype'   => 'int',
+	'default'     => 0
 ];
 
 // Блоки
 // Блок последних страниц
 $modversion['blocks'][] = [
-    'file'        => 'instr_lastpage.php',
-    'name'        => _MI_INSTR_BLOCK_LASTPAGE,
-    'description' => _MI_INSTR_BLOCK_LASTPAGE_DESC,
-    'show_func'   => 'b_instr_lastpage_show',
-    'edit_func'   => 'b_instr_lastpage_edit',
-    'options'     => '10|20',
-    'template'    => $moduleDirName . '_block_lastpage.tpl'
+	'file'        => 'instr_lastpage.php',
+	'name'        => _MI_INSTR_BLOCK_LASTPAGE,
+	'description' => _MI_INSTR_BLOCK_LASTPAGE_DESC,
+	'show_func'   => 'b_instr_lastpage_show',
+	'edit_func'   => 'b_instr_lastpage_edit',
+	'options'     => '10|20',
+	'template'    => $moduleDirName . '_block_lastpage.tpl'
 ];
 // Блок последних инструкций
 $modversion['blocks'][] = [
-    'file'        => 'instr_lastinstr.php',
-    'name'        => _MI_INSTR_BLOCK_LASTINSTR,
-    'description' => _MI_INSTR_BLOCK_LASTINSTR_DESC,
-    'show_func'   => 'b_instr_lastinstr_show',
-    'edit_func'   => 'b_instr_lastinstr_edit',
-    'options'     => '10|20',
-    'template'    => $moduleDirName . '_block_lastinstr.tpl'
+	'file'        => 'instr_lastinstr.php',
+	'name'        => _MI_INSTR_BLOCK_LASTINSTR,
+	'description' => _MI_INSTR_BLOCK_LASTINSTR_DESC,
+	'show_func'   => 'b_instr_lastinstr_show',
+	'edit_func'   => 'b_instr_lastinstr_edit',
+	'options'     => '10|20',
+	'template'    => $moduleDirName . '_block_lastinstr.tpl'
 ];
 
 // Notification

comment_new.php

Indentation +8 added lines, -8 removed lines

@@ -7,12 +7,12 @@
 
 $com_itemid = Request::getInt('com_itemid', 0, 'GET');
 if ($com_itemid > 0) {
-    $itemObj       = $publisher->getHandler('item')->get($com_itemid);
-    $com_replytext = _POSTEDBY . '&nbsp;<strong>' . $itemObj->getLinkedPosterName() . '</strong>&nbsp;' . _DATE . '&nbsp;<strong>' . $itemObj->dateSub() . '</strong><br><br>' . $itemObj->summary();
-    $bodytext      = $itemObj->body();
-    if ('' != $bodytext) {
-        $com_replytext .= '<br><br>' . $bodytext . '';
-    }
-    $com_replytitle = $itemObj->getTitle();
-    include_once $GLOBALS['xoops']->path('include/comment_new.php');
+	$itemObj       = $publisher->getHandler('item')->get($com_itemid);
+	$com_replytext = _POSTEDBY . '&nbsp;<strong>' . $itemObj->getLinkedPosterName() . '</strong>&nbsp;' . _DATE . '&nbsp;<strong>' . $itemObj->dateSub() . '</strong><br><br>' . $itemObj->summary();
+	$bodytext      = $itemObj->body();
+	if ('' != $bodytext) {
+		$com_replytext .= '<br><br>' . $bodytext . '';
+	}
+	$com_replytitle = $itemObj->getTitle();
+	include_once $GLOBALS['xoops']->path('include/comment_new.php');
 }

admin/admin_header.php

Indentation +10 added lines, -10 removed lines

@@ -8,27 +8,27 @@
 $myts = MyTextSanitizer::getInstance();
 
 if ($xoopsUser) {
-    $modulepermHandler = xoops_getHandler('groupperm');
-    if (!$modulepermHandler->checkRight('module_admin', $xoopsModule->getVar('mid'), $xoopsUser->getGroups())) {
-        redirect_header(XOOPS_URL, 1, _NOPERM);
-        exit();
-    }
+	$modulepermHandler = xoops_getHandler('groupperm');
+	if (!$modulepermHandler->checkRight('module_admin', $xoopsModule->getVar('mid'), $xoopsUser->getGroups())) {
+		redirect_header(XOOPS_URL, 1, _NOPERM);
+		exit();
+	}
 } else {
-    redirect_header(XOOPS_URL . '/user.php', 1, _NOPERM);
-    exit();
+	redirect_header(XOOPS_URL . '/user.php', 1, _NOPERM);
+	exit();
 }
 
 /** @var Xmf\Module\Helper $moduleHelper */
 if (false !== ($moduleHelper = Xmf\Module\Helper::getHelper($moduleDirName))) {
 } else {
-    $moduleHelper = Xmf\Module\Helper::getHelper('system');
+	$moduleHelper = Xmf\Module\Helper::getHelper('system');
 }
 /** @var Xmf\Module\Admin $adminObject */
 $adminObject = \Xmf\Module\Admin::getInstance();
 
 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();
 }
 
 $pathIcon16    = Xmf\Module\Admin::iconUrl('', 16);

admin/cat.php

Indentation +267 added lines, -267 removed lines

@@ -27,272 +27,272 @@
 // Выбор
 switch ($op) {
 
-    case 'main':
-
-        // Подключаем трей
-        $moduleDirName = dirname(__DIR__);
-        include_once $moduleDirName . '/class/tree.php';
-        //include_once $GLOBALS['xoops']->path('modules/instruction/class/tree.php');
-
-        // Заголовок админки
-        xoops_cp_header();
-        // Навигация
-        $adminObject->displayNavigation(basename(__FILE__));
-
-        // Находим ID-категории => Число страниц
-        $cidinstrids = [];
-        $sql         = "SELECT `cid`, COUNT( `instrid` ) FROM {$insinstrHandler->table} GROUP BY `cid`";
-        $result      = $GLOBALS['xoopsDB']->query($sql);
-        while (list($cid, $count) = $GLOBALS['xoopsDB']->fetchRow($result)) {
-            // Заполняем массив
-            $cidinstrids[$cid] = $count;
-        }
-
-        // Выбираем категории из БД
-        $criteria = new CriteriaCompo();
-        $criteria->setSort('weight ASC, title');
-        $criteria->setOrder('ASC');
-        $ins_cat = $instructioncatHandler->getall($criteria);
-        unset($criteria);
-
-        // Инициализируем
-        $cattree = new InstructionTree($ins_cat, 'cid', 'pid');
-        // Выводим списко категорий в шаблон
-        $GLOBALS['xoopsTpl']->assign('insListCat', $cattree->makeCatsAdmin('--', $cidinstrids));
-
-        // Создание новой категории
-        $objInstructioncat = $instructioncatHandler->create();
-        $form              = $objInstructioncat->getForm('cat.php');
-        // Форма
-        $GLOBALS['xoopsTpl']->assign('insFormCat', $form->render());
-        // Выводим шаблон
-        $GLOBALS['xoopsTpl']->display('db:admin/instruction_admin_cat.tpl');
-
-        // Текст внизу админки
-        include __DIR__ . '/admin_footer.php';
-        // Подвал админки
-        xoops_cp_footer();
-
-        break;
-
-    // Редактирование категории
-    case 'editcat':
-
-        // Заголовок админки
-        xoops_cp_header();
-        // Навигация
-        $adminObject->displayNavigation(basename(__FILE__));
-
-        $objInstructioncat = $instructioncatHandler->get($cid);
-        $form              = $objInstructioncat->getForm('cat.php');
-        // Форма
-        //$GLOBALS['xoopsTpl']->assign( 'insFormCat', $form->render() );
-        echo $form->render();
-        // Выводим шаблон
-        $GLOBALS['xoopsTpl']->display('db:admin/instruction_admin_editcat.tpl');
-
-        // Текст внизу админки
-        include __DIR__ . '/admin_footer.php';
-        // Подвал админки
-        xoops_cp_footer();
-
-        break;
-
-    // Сохранение категорий
-    case 'savecat':
-
-        // Проверка
-        if (!$GLOBALS['xoopsSecurity']->check()) {
-            redirect_header('cat.php', 3, implode(',', $GLOBALS['xoopsSecurity']->getErrors()));
-        }
-        // Если мы редактируем
-        if ($cid) {
-            $objInstructioncat = $instructioncatHandler->get($cid);
-        } else {
-            $objInstructioncat = $instructioncatHandler->create();
-            // Указываем дату создания
-            $objInstructioncat->setVar('datecreated', $time);
-        }
-
-        $err         = false;
-        $message_err = '';
-
-        // Дата обновления
-        $objInstructioncat->setVar('dateupdated', $time);
-        $objInstructioncat->setVar('pid', $pid);
-        $objInstructioncat->setVar('title', $_POST['title']);
-        $objInstructioncat->setVar('description', $_POST['description']);
-        $objInstructioncat->setVar('weight', $weight);
-        $objInstructioncat->setVar('metakeywords', $_POST['metakeywords']);
-        $objInstructioncat->setVar('metadescription', $_POST['metadescription']);
-
-        // Проверка веса
-        if (0 == $weight) {
-            $err         = true;
-            $message_err .= _AM_INSTRUCTION_ERR_WEIGHT . '<br>';
-        }
-        // Проверка категорий
-        if ($cid && ($cid == $pid)) {
-            $err         = true;
-            $message_err .= _AM_INSTRUCTION_ERR_PCAT . '<br>';
-        }
-        // Если были ошибки
-        if (true === $err) {
-            xoops_cp_header();
-            // Навигация
-            $adminObject->displayNavigation(basename(__FILE__));
-
-            $message_err = '<div class="errorMsg" style="text-align: left;">' . $message_err . '</div>';
-            // Выводим ошибки в шаблон
-            $GLOBALS['xoopsTpl']->assign('insErrorMsg', $message_err);
-            // Если небыло ошибок
-        } else {
-            // Вставляем данные в БД
-            if ($instructioncatHandler->insert($objInstructioncat)) {
-
-                // ID категории. Если редактируем - то не изменяется. Если создаём новую - то получаем ID созданной записи.
-                $new_cid = $cid ?: $objInstructioncat->get_new_enreg();
-
-                // ===============
-                // ==== Права ====
-                // ===============
-
-                $gpermHandler = xoops_getHandler('groupperm');
-
-                // Если мы редактируем категорию, то старые права нужно удалить
-                if ($cid) {
-                    // Права на просмотр
-                    $criteria = new CriteriaCompo();
-                    $criteria->add(new Criteria('gperm_itemid', $new_cid, '='));
-                    $criteria->add(new Criteria('gperm_modid', $GLOBALS['xoopsModule']->getVar('mid'), '='));
-                    $criteria->add(new Criteria('gperm_name', 'instruction_view', '='));
-                    $gpermHandler->deleteAll($criteria);
-                    // Права на добавление
-                    $criteria = new CriteriaCompo();
-                    $criteria->add(new Criteria('gperm_itemid', $new_cid, '='));
-                    $criteria->add(new Criteria('gperm_modid', $GLOBALS['xoopsModule']->getVar('mid'), '='));
-                    $criteria->add(new Criteria('gperm_name', 'instruction_submit', '='));
-                    $gpermHandler->deleteAll($criteria);
-                    // Права на редактирование
-                    $criteria = new CriteriaCompo();
-                    $criteria->add(new Criteria('gperm_itemid', $new_cid, '='));
-                    $criteria->add(new Criteria('gperm_modid', $GLOBALS['xoopsModule']->getVar('mid'), '='));
-                    $criteria->add(new Criteria('gperm_name', 'instruction_edit', '='));
-                    $gpermHandler->deleteAll($criteria);
-                }
-
-                // Добавляем права
-                // Права на просмотр
-                if (isset($_POST['groups_instr_view'])) {
-                    foreach ($_POST['groups_instr_view'] as $onegroup_id) {
-                        $gpermHandler->addRight('instruction_view', $new_cid, $onegroup_id, $GLOBALS['xoopsModule']->getVar('mid'));
-                    }
-                }
-                // Права на добавление
-                if (isset($_POST['groups_instr_submit'])) {
-                    foreach ($_POST['groups_instr_submit'] as $onegroup_id) {
-                        $gpermHandler->addRight('instruction_submit', $new_cid, $onegroup_id, $GLOBALS['xoopsModule']->getVar('mid'));
-                    }
-                }
-                // Права на редактирование
-                if (isset($_POST['groups_instr_edit'])) {
-                    foreach ($_POST['groups_instr_edit'] as $onegroup_id) {
-                        $gpermHandler->addRight('instruction_edit', $new_cid, $onegroup_id, $GLOBALS['xoopsModule']->getVar('mid'));
-                    }
-                }
-
-                //
-                redirect_header('cat.php', 3, _AM_INSTRUCTION_NEWCATADDED);
-            }
-            xoops_cp_header();
-            // Навигация
-            $adminObject->displayNavigation(basename(__FILE__));
-            // Выводим ошибки в шаблон
-            $GLOBALS['xoopsTpl']->assign('insErrorMsg', $objInstructioncat->getHtmlErrors());
-        }
-        // Выводим шаблон
-        $GLOBALS['xoopsTpl']->display('db:admin/instruction_admin_savecat.tpl');
-        // Выводим форму
-        $form = $objInstructioncat->getForm();
-        // Форма
-        echo $form->render();
-        // Текст внизу админки
-        include __DIR__ . '/admin_footer.php';
-        // Подвал админки
-        xoops_cp_footer();
-
-        break;
-
-    // Удаление категории
-    case 'delcat':
-
-        // Находим число инструкций в данной категории
-        // Критерий выборки
-        $criteria = new CriteriaCompo();
-        // Все инструкции в данной категории
-        $criteria->add(new Criteria('cid', $cid, '='));
-        $numrows = $insinstrHandler->getCount($criteria);
-        //
-        unset($criteria);
-        // Если есть хоть одна инструкция
-        if ($numrows) {
-            redirect_header('cat.php', 3, _AM_INSTRUCTION_ERR_CATNOTEMPTY);
-        }
-
-        $objInscat = $instructioncatHandler->get($cid);
-        // Если нет такой категории
-        if (!is_object($objInscat)) {
-            redirect_header('cat.php', 3, _AM_INSTRUCTION_ERR_CATNOTSELECT);
-        }
-
-        // Нельзя удалять пока есть доченрии категории
-        // Подключаем трей
-        include_once $GLOBALS['xoops']->path('class/tree.php');
-        $inscat_arr   = $instructioncatHandler->getall();
-        $mytree       = new XoopsObjectTree($inscat_arr, 'cid', 'pid');
-        $ins_childcat = $mytree->getAllChild($cid);
-        // Если есть дочернии категории
-        if (count($ins_childcat)) {
-            redirect_header('cat.php', 3, _AM_INSTRUCTION_ERR_CATCHILDREN);
-        }
-
-        // Нажали ли мы на кнопку OK
-        $ok = isset($_POST['ok']) ? (int)$_POST['ok'] : 0;
-        // Если мы нажали на кнопку
-        if ($ok) {
-
-            // Проверка
-            if (!$GLOBALS['xoopsSecurity']->check()) {
-                redirect_header('cat.php', 3, implode(',', $GLOBALS['xoopsSecurity']->getErrors()));
-            }
-            // Пытаемся удалить категорию
-            if ($instructioncatHandler->delete($objInscat)) {
-
-                // Удалить права доступа к категории
-                // =================================
-
-                // Редирект
-                redirect_header('cat.php', 3, _AM_INSTRUCTION_CATDELETED);
-                // Если не смогли удалить категорию
-            } else {
-                // Редирект
-                redirect_header('cat.php', 3, _AM_INSTRUCTION_ERR_DELCAT);
-            }
-        } else {
-
-            // Заголовок админки
-            xoops_cp_header();
-            // Навигация
-            $adminObject->displayNavigation(basename(__FILE__));
-
-            xoops_confirm(['ok' => 1, 'cid' => $cid, 'op' => 'delcat'], 'cat.php', sprintf(_AM_INSTRUCTION_FORMDELCAT, $objInscat->getVar('title')));
-
-            // Текст внизу админки
-            include __DIR__ . '/admin_footer.php';
-            // Подвал админки
-            xoops_cp_footer();
-        }
-
-        break;
+	case 'main':
+
+		// Подключаем трей
+		$moduleDirName = dirname(__DIR__);
+		include_once $moduleDirName . '/class/tree.php';
+		//include_once $GLOBALS['xoops']->path('modules/instruction/class/tree.php');
+
+		// Заголовок админки
+		xoops_cp_header();
+		// Навигация
+		$adminObject->displayNavigation(basename(__FILE__));
+
+		// Находим ID-категории => Число страниц
+		$cidinstrids = [];
+		$sql         = "SELECT `cid`, COUNT( `instrid` ) FROM {$insinstrHandler->table} GROUP BY `cid`";
+		$result      = $GLOBALS['xoopsDB']->query($sql);
+		while (list($cid, $count) = $GLOBALS['xoopsDB']->fetchRow($result)) {
+			// Заполняем массив
+			$cidinstrids[$cid] = $count;
+		}
+
+		// Выбираем категории из БД
+		$criteria = new CriteriaCompo();
+		$criteria->setSort('weight ASC, title');
+		$criteria->setOrder('ASC');
+		$ins_cat = $instructioncatHandler->getall($criteria);
+		unset($criteria);
+
+		// Инициализируем
+		$cattree = new InstructionTree($ins_cat, 'cid', 'pid');
+		// Выводим списко категорий в шаблон
+		$GLOBALS['xoopsTpl']->assign('insListCat', $cattree->makeCatsAdmin('--', $cidinstrids));
+
+		// Создание новой категории
+		$objInstructioncat = $instructioncatHandler->create();
+		$form              = $objInstructioncat->getForm('cat.php');
+		// Форма
+		$GLOBALS['xoopsTpl']->assign('insFormCat', $form->render());
+		// Выводим шаблон
+		$GLOBALS['xoopsTpl']->display('db:admin/instruction_admin_cat.tpl');
+
+		// Текст внизу админки
+		include __DIR__ . '/admin_footer.php';
+		// Подвал админки
+		xoops_cp_footer();
+
+		break;
+
+	// Редактирование категории
+	case 'editcat':
+
+		// Заголовок админки
+		xoops_cp_header();
+		// Навигация
+		$adminObject->displayNavigation(basename(__FILE__));
+
+		$objInstructioncat = $instructioncatHandler->get($cid);
+		$form              = $objInstructioncat->getForm('cat.php');
+		// Форма
+		//$GLOBALS['xoopsTpl']->assign( 'insFormCat', $form->render() );
+		echo $form->render();
+		// Выводим шаблон
+		$GLOBALS['xoopsTpl']->display('db:admin/instruction_admin_editcat.tpl');
+
+		// Текст внизу админки
+		include __DIR__ . '/admin_footer.php';
+		// Подвал админки
+		xoops_cp_footer();
+
+		break;
+
+	// Сохранение категорий
+	case 'savecat':
+
+		// Проверка
+		if (!$GLOBALS['xoopsSecurity']->check()) {
+			redirect_header('cat.php', 3, implode(',', $GLOBALS['xoopsSecurity']->getErrors()));
+		}
+		// Если мы редактируем
+		if ($cid) {
+			$objInstructioncat = $instructioncatHandler->get($cid);
+		} else {
+			$objInstructioncat = $instructioncatHandler->create();
+			// Указываем дату создания
+			$objInstructioncat->setVar('datecreated', $time);
+		}
+
+		$err         = false;
+		$message_err = '';
+
+		// Дата обновления
+		$objInstructioncat->setVar('dateupdated', $time);
+		$objInstructioncat->setVar('pid', $pid);
+		$objInstructioncat->setVar('title', $_POST['title']);
+		$objInstructioncat->setVar('description', $_POST['description']);
+		$objInstructioncat->setVar('weight', $weight);
+		$objInstructioncat->setVar('metakeywords', $_POST['metakeywords']);
+		$objInstructioncat->setVar('metadescription', $_POST['metadescription']);
+
+		// Проверка веса
+		if (0 == $weight) {
+			$err         = true;
+			$message_err .= _AM_INSTRUCTION_ERR_WEIGHT . '<br>';
+		}
+		// Проверка категорий
+		if ($cid && ($cid == $pid)) {
+			$err         = true;
+			$message_err .= _AM_INSTRUCTION_ERR_PCAT . '<br>';
+		}
+		// Если были ошибки
+		if (true === $err) {
+			xoops_cp_header();
+			// Навигация
+			$adminObject->displayNavigation(basename(__FILE__));
+
+			$message_err = '<div class="errorMsg" style="text-align: left;">' . $message_err . '</div>';
+			// Выводим ошибки в шаблон
+			$GLOBALS['xoopsTpl']->assign('insErrorMsg', $message_err);
+			// Если небыло ошибок
+		} else {
+			// Вставляем данные в БД
+			if ($instructioncatHandler->insert($objInstructioncat)) {
+
+				// ID категории. Если редактируем - то не изменяется. Если создаём новую - то получаем ID созданной записи.
+				$new_cid = $cid ?: $objInstructioncat->get_new_enreg();
+
+				// ===============
+				// ==== Права ====
+				// ===============
+
+				$gpermHandler = xoops_getHandler('groupperm');
+
+				// Если мы редактируем категорию, то старые права нужно удалить
+				if ($cid) {
+					// Права на просмотр
+					$criteria = new CriteriaCompo();
+					$criteria->add(new Criteria('gperm_itemid', $new_cid, '='));
+					$criteria->add(new Criteria('gperm_modid', $GLOBALS['xoopsModule']->getVar('mid'), '='));
+					$criteria->add(new Criteria('gperm_name', 'instruction_view', '='));
+					$gpermHandler->deleteAll($criteria);
+					// Права на добавление
+					$criteria = new CriteriaCompo();
+					$criteria->add(new Criteria('gperm_itemid', $new_cid, '='));
+					$criteria->add(new Criteria('gperm_modid', $GLOBALS['xoopsModule']->getVar('mid'), '='));
+					$criteria->add(new Criteria('gperm_name', 'instruction_submit', '='));
+					$gpermHandler->deleteAll($criteria);
+					// Права на редактирование
+					$criteria = new CriteriaCompo();
+					$criteria->add(new Criteria('gperm_itemid', $new_cid, '='));
+					$criteria->add(new Criteria('gperm_modid', $GLOBALS['xoopsModule']->getVar('mid'), '='));
+					$criteria->add(new Criteria('gperm_name', 'instruction_edit', '='));
+					$gpermHandler->deleteAll($criteria);
+				}
+
+				// Добавляем права
+				// Права на просмотр
+				if (isset($_POST['groups_instr_view'])) {
+					foreach ($_POST['groups_instr_view'] as $onegroup_id) {
+						$gpermHandler->addRight('instruction_view', $new_cid, $onegroup_id, $GLOBALS['xoopsModule']->getVar('mid'));
+					}
+				}
+				// Права на добавление
+				if (isset($_POST['groups_instr_submit'])) {
+					foreach ($_POST['groups_instr_submit'] as $onegroup_id) {
+						$gpermHandler->addRight('instruction_submit', $new_cid, $onegroup_id, $GLOBALS['xoopsModule']->getVar('mid'));
+					}
+				}
+				// Права на редактирование
+				if (isset($_POST['groups_instr_edit'])) {
+					foreach ($_POST['groups_instr_edit'] as $onegroup_id) {
+						$gpermHandler->addRight('instruction_edit', $new_cid, $onegroup_id, $GLOBALS['xoopsModule']->getVar('mid'));
+					}
+				}
+
+				//
+				redirect_header('cat.php', 3, _AM_INSTRUCTION_NEWCATADDED);
+			}
+			xoops_cp_header();
+			// Навигация
+			$adminObject->displayNavigation(basename(__FILE__));
+			// Выводим ошибки в шаблон
+			$GLOBALS['xoopsTpl']->assign('insErrorMsg', $objInstructioncat->getHtmlErrors());
+		}
+		// Выводим шаблон
+		$GLOBALS['xoopsTpl']->display('db:admin/instruction_admin_savecat.tpl');
+		// Выводим форму
+		$form = $objInstructioncat->getForm();
+		// Форма
+		echo $form->render();
+		// Текст внизу админки
+		include __DIR__ . '/admin_footer.php';
+		// Подвал админки
+		xoops_cp_footer();
+
+		break;
+
+	// Удаление категории
+	case 'delcat':
+
+		// Находим число инструкций в данной категории
+		// Критерий выборки
+		$criteria = new CriteriaCompo();
+		// Все инструкции в данной категории
+		$criteria->add(new Criteria('cid', $cid, '='));
+		$numrows = $insinstrHandler->getCount($criteria);
+		//
+		unset($criteria);
+		// Если есть хоть одна инструкция
+		if ($numrows) {
+			redirect_header('cat.php', 3, _AM_INSTRUCTION_ERR_CATNOTEMPTY);
+		}
+
+		$objInscat = $instructioncatHandler->get($cid);
+		// Если нет такой категории
+		if (!is_object($objInscat)) {
+			redirect_header('cat.php', 3, _AM_INSTRUCTION_ERR_CATNOTSELECT);
+		}
+
+		// Нельзя удалять пока есть доченрии категории
+		// Подключаем трей
+		include_once $GLOBALS['xoops']->path('class/tree.php');
+		$inscat_arr   = $instructioncatHandler->getall();
+		$mytree       = new XoopsObjectTree($inscat_arr, 'cid', 'pid');
+		$ins_childcat = $mytree->getAllChild($cid);
+		// Если есть дочернии категории
+		if (count($ins_childcat)) {
+			redirect_header('cat.php', 3, _AM_INSTRUCTION_ERR_CATCHILDREN);
+		}
+
+		// Нажали ли мы на кнопку OK
+		$ok = isset($_POST['ok']) ? (int)$_POST['ok'] : 0;
+		// Если мы нажали на кнопку
+		if ($ok) {
+
+			// Проверка
+			if (!$GLOBALS['xoopsSecurity']->check()) {
+				redirect_header('cat.php', 3, implode(',', $GLOBALS['xoopsSecurity']->getErrors()));
+			}
+			// Пытаемся удалить категорию
+			if ($instructioncatHandler->delete($objInscat)) {
+
+				// Удалить права доступа к категории
+				// =================================
+
+				// Редирект
+				redirect_header('cat.php', 3, _AM_INSTRUCTION_CATDELETED);
+				// Если не смогли удалить категорию
+			} else {
+				// Редирект
+				redirect_header('cat.php', 3, _AM_INSTRUCTION_ERR_DELCAT);
+			}
+		} else {
+
+			// Заголовок админки
+			xoops_cp_header();
+			// Навигация
+			$adminObject->displayNavigation(basename(__FILE__));
+
+			xoops_confirm(['ok' => 1, 'cid' => $cid, 'op' => 'delcat'], 'cat.php', sprintf(_AM_INSTRUCTION_FORMDELCAT, $objInscat->getVar('title')));
+
+			// Текст внизу админки
+			include __DIR__ . '/admin_footer.php';
+			// Подвал админки
+			xoops_cp_footer();
+		}
+
+		break;
 
 }

admin/menu.php

Indentation +32 added lines, -32 removed lines

@@ -1,12 +1,12 @@ 
 <?php
 
 if (!isset($moduleDirName)) {
-    $moduleDirName = basename(dirname(__DIR__));
+	$moduleDirName = basename(dirname(__DIR__));
 }
 
 if (false !== ($moduleHelper = Xmf\Module\Helper::getHelper($moduleDirName))) {
 } else {
-    $moduleHelper = Xmf\Module\Helper::getHelper('system');
+	$moduleHelper = Xmf\Module\Helper::getHelper('system');
 }
 
 $pathIcon32    = \Xmf\Module\Admin::menuIconPath('');
@@ -17,34 +17,34 @@ 
 
 // Административное меню
 $adminmenu = [
-    [
-        'title' => _MI_INSTRUCTION_ADMIN_HOME,
-        'link'  => 'admin/index.php',
-        'desc'  => _MI_INSTRUCTION_ADMIN_HOME_DESC,
-        'icon'  => $pathIcon32 . '/home.png'
-    ],
-    [
-        'title' => _MI_INSTRUCTION_ADMIN_CAT,
-        'link'  => 'admin/cat.php',
-        'desc'  => _MI_INSTRUCTION_ADMIN_CAT_DESC,
-        'icon'  => $pathIcon32 . '/category.png'
-    ],
-    [
-        'title' => _MI_INSTRUCTION_ADMIN_INSTR,
-        'link'  => 'admin/instr.php',
-        'desc'  => _MI_INSTRUCTION_ADMIN_INSTR_DESC,
-        'icon'  => $pathModIcon32 . '/nav_book.png'
-    ],
-    [
-        'title' => _MI_INSTRUCTION_ADMIN_PERM,
-        'link'  => 'admin/perm.php',
-        'desc'  => _MI_INSTRUCTION_ADMIN_PERM_DESC,
-        'icon'  => $pathIcon32 . '/permissions.png'
-    ],
-    [
-        'title' => _MI_INSTRUCTION_ADMIN_ABOUT,
-        'link'  => 'admin/about.php',
-        'desc'  => _MI_INSTRUCTION_ADMIN_ABOUT_DESC,
-        'icon'  => $pathIcon32 . '/about.png'
-    ]
+	[
+		'title' => _MI_INSTRUCTION_ADMIN_HOME,
+		'link'  => 'admin/index.php',
+		'desc'  => _MI_INSTRUCTION_ADMIN_HOME_DESC,
+		'icon'  => $pathIcon32 . '/home.png'
+	],
+	[
+		'title' => _MI_INSTRUCTION_ADMIN_CAT,
+		'link'  => 'admin/cat.php',
+		'desc'  => _MI_INSTRUCTION_ADMIN_CAT_DESC,
+		'icon'  => $pathIcon32 . '/category.png'
+	],
+	[
+		'title' => _MI_INSTRUCTION_ADMIN_INSTR,
+		'link'  => 'admin/instr.php',
+		'desc'  => _MI_INSTRUCTION_ADMIN_INSTR_DESC,
+		'icon'  => $pathModIcon32 . '/nav_book.png'
+	],
+	[
+		'title' => _MI_INSTRUCTION_ADMIN_PERM,
+		'link'  => 'admin/perm.php',
+		'desc'  => _MI_INSTRUCTION_ADMIN_PERM_DESC,
+		'icon'  => $pathIcon32 . '/permissions.png'
+	],
+	[
+		'title' => _MI_INSTRUCTION_ADMIN_ABOUT,
+		'link'  => 'admin/about.php',
+		'desc'  => _MI_INSTRUCTION_ADMIN_ABOUT_DESC,
+		'icon'  => $pathIcon32 . '/about.png'
+	]
 ];

admin/perm.php

Indentation +21 added lines, -21 removed lines

@@ -36,24 +36,24 @@ 
 $moduleId = $GLOBALS['xoopsModule']->getVar('mid');
 
 switch ($permission) {
-    // Права на просмотр
-    case 1:
-        $formTitle             = _AM_INSTRUCTION_PERM_VIEW;
-        $permissionName        = 'instruction_view';
-        $permissionDescription = _AM_INSTRUCTION_PERM_VIEW_DSC;
-        break;
-    // Права на добавление
-    case 2:
-        $formTitle             = _AM_INSTRUCTION_PERM_SUBMIT;
-        $permissionName        = 'instruction_submit';
-        $permissionDescription = _AM_INSTRUCTION_PERM_SUBMIT_DSC;
-        break;
-    // Права на редактирование
-    case 3:
-        $formTitle             = _AM_INSTRUCTION_PERM_EDIT;
-        $permissionName        = 'instruction_edit';
-        $permissionDescription = _AM_INSTRUCTION_PERM_EDIT_DSC;
-        break;
+	// Права на просмотр
+	case 1:
+		$formTitle             = _AM_INSTRUCTION_PERM_VIEW;
+		$permissionName        = 'instruction_view';
+		$permissionDescription = _AM_INSTRUCTION_PERM_VIEW_DSC;
+		break;
+	// Права на добавление
+	case 2:
+		$formTitle             = _AM_INSTRUCTION_PERM_SUBMIT;
+		$permissionName        = 'instruction_submit';
+		$permissionDescription = _AM_INSTRUCTION_PERM_SUBMIT_DSC;
+		break;
+	// Права на редактирование
+	case 3:
+		$formTitle             = _AM_INSTRUCTION_PERM_EDIT;
+		$permissionName        = 'instruction_edit';
+		$permissionDescription = _AM_INSTRUCTION_PERM_EDIT_DSC;
+		break;
 }
 
 // Права
@@ -62,9 +62,9 @@ 
 $sql    = 'SELECT cid, pid, title FROM ' . $xoopsDB->prefix('instruction_cat') . ' ORDER BY title';
 $result = $xoopsDB->query($sql);
 if ($result) {
-    while ($row = $xoopsDB->fetchArray($result)) {
-        $permissionsForm->addItem($row['cid'], $row['title'], $row['pid']);
-    }
+	while ($row = $xoopsDB->fetchArray($result)) {
+		$permissionsForm->addItem($row['cid'], $row['title'], $row['pid']);
+	}
 }
 
 echo $permissionsForm->render();