mambax7 / instruction

class/common/FilesManagement.php

Spacing +20 added lines, -20 removed lines

@@ -28,8 +28,8 @@ 
     public static function createFolder($folder)
     {
         try {
-            if (!file_exists($folder)) {
-                if (!mkdir($folder) && !is_dir($folder)) {
+            if ( ! file_exists($folder)) {
+                if ( ! mkdir($folder) && ! is_dir($folder)) {
                     throw new \RuntimeException(sprintf('Unable to create the %s directory', $folder));
                 }
 
@@ -59,7 +59,7 @@ 
     public static function recurseCopy($src, $dst)
     {
         $dir = opendir($src);
-        if (!mkdir($dst) && !is_dir($dst)) {
+        if ( ! mkdir($dst) && ! is_dir($dst)) {
             throw new \RuntimeException('The directory ' . $dst . ' could not be created.');
         }
         while (false !== ($file = readdir($dir))) {
@@ -88,7 +88,7 @@ 
     public static function deleteDirectory($src)
     {
         // Only continue if user is a 'global' Admin
-        if (!($GLOBALS['xoopsUser'] instanceof \XoopsUser) || !$GLOBALS['xoopsUser']->isAdmin()) {
+        if ( ! ($GLOBALS['xoopsUser'] instanceof \XoopsUser) || ! $GLOBALS['xoopsUser']->isAdmin()) {
             return false;
         }
 
@@ -102,12 +102,12 @@ 
                 $fileInfo = new \SplFileInfo("{$src}/{$v}");
                 if ($fileInfo->isDir()) {
                     // recursively handle subdirectories
-                    if (!$success = self::deleteDirectory($fileInfo->getRealPath())) {
+                    if ( ! $success = self::deleteDirectory($fileInfo->getRealPath())) {
                         break;
                     }
                 } else {
                     // delete the file
-                    if (!($success = unlink($fileInfo->getRealPath()))) {
+                    if ( ! ($success = unlink($fileInfo->getRealPath()))) {
                         break;
                     }
                 }
@@ -136,12 +136,12 @@ 
     public static function rrmdir($src)
     {
         // Only continue if user is a 'global' Admin
-        if (!($GLOBALS['xoopsUser'] instanceof \XoopsUser) || !$GLOBALS['xoopsUser']->isAdmin()) {
+        if ( ! ($GLOBALS['xoopsUser'] instanceof \XoopsUser) || ! $GLOBALS['xoopsUser']->isAdmin()) {
             return false;
         }
 
         // If source is not a directory stop processing
-        if (!is_dir($src)) {
+        if ( ! is_dir($src)) {
             return false;
         }
 
@@ -153,15 +153,15 @@ 
             if ($fObj->isFile()) {
                 $filename = $fObj->getPathname();
                 $fObj     = null; // clear this iterator object to close the file
-                if (!unlink($filename)) {
+                if ( ! unlink($filename)) {
                     return false; // couldn't delete the file
                 }
-            } elseif (!$fObj->isDot() && $fObj->isDir()) {
+            } elseif ( ! $fObj->isDot() && $fObj->isDir()) {
                 // Try recursively on directory
                 self::rrmdir($fObj->getPathname());
             }
         }
-        $iterator = null;   // clear iterator Obj to close file/directory
+        $iterator = null; // clear iterator Obj to close file/directory
         return rmdir($src); // remove the directory & return results
     }
 
@@ -176,17 +176,17 @@ 
     public static function rmove($src, $dest)
     {
         // Only continue if user is a 'global' Admin
-        if (!($GLOBALS['xoopsUser'] instanceof \XoopsUser) || !$GLOBALS['xoopsUser']->isAdmin()) {
+        if ( ! ($GLOBALS['xoopsUser'] instanceof \XoopsUser) || ! $GLOBALS['xoopsUser']->isAdmin()) {
             return false;
         }
 
         // If source is not a directory stop processing
-        if (!is_dir($src)) {
+        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)) {
+        if ( ! is_dir($dest) && ! mkdir($dest, 0755)) {
             return false;
         }
 
@@ -195,13 +195,13 @@ 
         foreach ($iterator as $fObj) {
             if ($fObj->isFile()) {
                 rename($fObj->getPathname(), "{$dest}/" . $fObj->getFilename());
-            } elseif (!$fObj->isDot() && $fObj->isDir()) {
+            } 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
+        $iterator = null; // clear iterator Obj to close file/directory
         return rmdir($src); // remove the directory & return results
     }
 
@@ -219,17 +219,17 @@ 
     public static function rcopy($src, $dest)
     {
         // Only continue if user is a 'global' Admin
-        if (!($GLOBALS['xoopsUser'] instanceof \XoopsUser) || !$GLOBALS['xoopsUser']->isAdmin()) {
+        if ( ! ($GLOBALS['xoopsUser'] instanceof \XoopsUser) || ! $GLOBALS['xoopsUser']->isAdmin()) {
             return false;
         }
 
         // If source is not a directory stop processing
-        if (!is_dir($src)) {
+        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)) {
+        if ( ! is_dir($dest) && ! mkdir($dest, 0755)) {
             return false;
         }
 
@@ -238,7 +238,7 @@ 
         foreach ($iterator as $fObj) {
             if ($fObj->isFile()) {
                 copy($fObj->getPathname(), "{$dest}/" . $fObj->getFilename());
-            } elseif (!$fObj->isDot() && $fObj->isDir()) {
+            } elseif ( ! $fObj->isDot() && $fObj->isDir()) {
                 self::rcopy($fObj->getPathname(), "{$dest}/" . $fObj->getFilename());
             }
         }

Indentation +226 added lines, -226 removed lines

@@ -17,230 +17,230 @@
  */
 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
-     * @throws \RuntimeException
-     */
-    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
-     * @throws \RuntimeException
-     */
-    public static function recurseCopy($src, $dst)
-    {
-        $dir = opendir($src);
-        if (!mkdir($dst) && !is_dir($dst)) {
-            throw new \RuntimeException('The directory ' . $dst . ' could not be created.');
-        }
-        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
+	 * @throws \RuntimeException
+	 */
+	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
+	 * @throws \RuntimeException
+	 */
+	public static function recurseCopy($src, $dst)
+	{
+		$dir = opendir($src);
+		if (!mkdir($dst) && !is_dir($dst)) {
+			throw new \RuntimeException('The directory ' . $dst . ' could not be created.');
+		}
+		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;
+	}
 }

admin/ajax.php

Indentation +174 added lines, -174 removed lines

@@ -23,178 +23,178 @@
 
 // Выбор
 switch ($op) {
-    // Сохранение страницы
-    case 'savepage':
-        // Выходной массив
-        $ret = [];
-
-        // Ошибки
-        $err         = false;
-        $message_err = '';
-
-        //
-        $title       = Request::getString('title', '', 'POST');
-        $pid         = Request::getInt('pid', 0, 'POST');
-        $weight      = Request::getInt('weight', 0, 'POST');
-        $hometext    = Request::getString('hometext', '', 'POST');
-        $footnote    = Request::getString('footnote', '', 'POST');
-        $status      = Request::getInt('status', 0, 'POST');
-        $type        = Request::getInt('type', 0, 'POST');
-        $keywords    = Request::getString('keywords', '', 'POST');
-        $description = Request::getString('description', '', 'POST');
-        $dosmiley    = (Request::getInt('dosmiley', 0, 'POST') > 0) ? 1 : 0;
-        $doxcode     = (Request::getInt('doxcode', 0, 'POST') > 0) ? 1 : 0;
-        $dobr        = (Request::getInt('dobr', 0, 'POST') > 0) ? 1 : 0;
-        $dohtml      = (Request::getInt('dohtml', 0, 'POST') > 0) ? 1 : 0;
-        //$dohtml      = Request::getInt( 'dohtml', 0, 'POST' );
-        //$dosmiley    = Request::getInt( 'dosmiley', 0, 'POST' );
-        //$doxcode     = Request::getInt( 'doxcode', 0, 'POST' );
-        //$dobr        = Request::getInt( 'dobr', 0, 'POST' );
-        $pageid  = Request::getInt('pageid', 0, 'POST');
-        $instrid = Request::getInt('instrid', 0, 'POST');
-
-        // Проверка
-        if (!$GLOBALS['xoopsSecurity']->check()) {
-            $err         = true;
-            $err_txt     = implode(', ', $GLOBALS['xoopsSecurity']->getErrors());
-            $message_err .= $err_txt . '<br>' . _AM_INSTR_TRY_AGAIN . '<br>';
-        }
-        // =========================
-        // Устанавливаем новый token
-        $token_name    = 'XOOPS_TOKEN';
-        $token_timeout = 0;
-        // $token_name . '_REQUEST' - название элемента формы
-        $ret['toket'] = $GLOBALS['xoopsSecurity']->createToken($token_timeout, $token_name);
-        // =========================
-
-        // Если мы редактируем
-        if ($pageid) {
-            $objInspage = $pageHandler->get($pageid);
-        } elseif ($instrid) {
-            $objInspage = $pageHandler->create();
-            // Если мы создаём страницу необходимо указать к какой инструкции
-            $objInspage->setVar('instrid', $instrid);
-            // Указываем дату создания
-            $objInspage->setVar('datecreated', $time);
-            // Указываем пользователя
-            $objInspage->setVar('uid', $uid);
-        } else {
-            // Устанавливаем сообщение
-            $ret['message'] = _AM_INSTRUCTION_BADREQUEST;
-            // Возвращаем ответ скрипту через JSON
-            echo json_encode($ret);
-            // Прерываем выполнение
-            exit();
-        }
-
-        // Родительская страница
-        $objInspage->setVar('pid', $pid);
-        // Дата обновления
-        $objInspage->setVar('dateupdated', $time);
-        // Название
-        $objInspage->setVar('title', $title);
-        // Вес
-        $objInspage->setVar('weight', $weight);
-        // Текст
-        $objInspage->setVar('hometext', $hometext);
-        // Сноска
-        $objInspage->setVar('footnote', $footnote);
-        // Статус
-        $objInspage->setVar('status', $status);
-        // Тип
-        $objInspage->setVar('type', $type);
-        // Мета-теги ключевых слов
-        $objInspage->setVar('keywords', $keywords);
-        // Мета-теги описания
-        $objInspage->setVar('description', $description);
-        //
-        $objInspage->setVar('dohtml', $dohtml);
-        $objInspage->setVar('dosmiley', $dosmiley);
-        $objInspage->setVar('doxcode', $doxcode);
-        $objInspage->setVar('dobr', $dobr);
-
-        // Проверка категорий
-        if (!$pageid && !$instrid) {
-            $err         = true;
-            $message_err .= _AM_INSTRUCTION_ERR_INSTR . '<br>';
-        }
-        // Проверка веса
-        if (0 == $weight) {
-            $err         = true;
-            $message_err .= _AM_INSTRUCTION_ERR_WEIGHT . '<br>';
-        }
-        // Проверка родительской страницы
-        if ($pageid && ($pageid == $pid)) {
-            $err         = true;
-            $message_err .= _AM_INSTRUCTION_ERR_PPAGE . '<br>';
-        }
-        // Проверка названия
-        if (!$title) {
-            $err         = true;
-            $message_err .= _AM_INSTR_ERR_TITLE . '<br>';
-        }
-        // Проверка основного текста
-        if (!$hometext) {
-            $err         = true;
-            $message_err .= _AM_INSTR_ERR_HOMETEXT . '<br>';
-        }
-
-        // Если были ошибки
-        if (true === $err) {
-            //
-            $message_err = '<div class="errorMsg" style="text-align: left;">' . $message_err . '</div>';
-            // Устанавливаем сообщение
-            $ret['message'] = $message_err;
-            // Возвращаем ответ скрипту через JSON
-            echo json_encode($ret);
-            // Прерываем выполнение
-            exit();
-            // Если небыло ошибок
-        } else {
-            // Вставляем данные в БД
-            if ($pageHandler->insert($objInspage)) {
-                // Находим ID созданной записи
-                $pageid_new = $pageid ?: $objInspage->getNewInstertId();
-                //
-                $ret['pageid'] = $pageid_new;
-                // Получаем ID инструкции
-                $instrid = $objInspage->getInstrid();
-                // Обновляем в инструкции число страниц и дату
-                $instructionHandler->updatePages($instrid);
-                // Если мы редактируем
-                if ($pageid) {
-                    // Устанавливаем сообщение
-                    $ret['message'] = '<div class="successMsg" style="text-align: left;">' . _AM_INSTRUCTION_PAGEMODIFY . '</div>';
-                    // Возвращаем ответ скрипту через JSON
-                    echo json_encode($ret);
-                    // Прерываем выполнение
-                    exit();
-
-                    // Если мы добавляем
-                } else {
-                    // Инкримент комментов
-                    $pageHandler->updateposts($uid, $status, 'add');
-
-                    // Устанавливаем сообщение
-                    $ret['message'] = '<div class="successMsg" style="text-align: left;">' . _AM_INSTRUCTION_PAGEADDED . '</div>';
-                    // Возвращаем ответ скрипту через JSON
-                    echo json_encode($ret);
-                    // Прерываем выполнение
-                    exit();
-                }
-                // Если не получилось вставить данные
-            } else {
-
-                //
-                $message_err = '<div class="errorMsg" style="text-align: left;">' . $objInspage->getHtmlErrors() . '</div>';
-                // Устанавливаем сообщение
-                $ret['message'] = $message_err;
-                // Возвращаем ответ скрипту через JSON
-                echo json_encode($ret);
-                // Прерываем выполнение
-                exit();
-            }
-        }
-
-        break;
+	// Сохранение страницы
+	case 'savepage':
+		// Выходной массив
+		$ret = [];
+
+		// Ошибки
+		$err         = false;
+		$message_err = '';
+
+		//
+		$title       = Request::getString('title', '', 'POST');
+		$pid         = Request::getInt('pid', 0, 'POST');
+		$weight      = Request::getInt('weight', 0, 'POST');
+		$hometext    = Request::getString('hometext', '', 'POST');
+		$footnote    = Request::getString('footnote', '', 'POST');
+		$status      = Request::getInt('status', 0, 'POST');
+		$type        = Request::getInt('type', 0, 'POST');
+		$keywords    = Request::getString('keywords', '', 'POST');
+		$description = Request::getString('description', '', 'POST');
+		$dosmiley    = (Request::getInt('dosmiley', 0, 'POST') > 0) ? 1 : 0;
+		$doxcode     = (Request::getInt('doxcode', 0, 'POST') > 0) ? 1 : 0;
+		$dobr        = (Request::getInt('dobr', 0, 'POST') > 0) ? 1 : 0;
+		$dohtml      = (Request::getInt('dohtml', 0, 'POST') > 0) ? 1 : 0;
+		//$dohtml      = Request::getInt( 'dohtml', 0, 'POST' );
+		//$dosmiley    = Request::getInt( 'dosmiley', 0, 'POST' );
+		//$doxcode     = Request::getInt( 'doxcode', 0, 'POST' );
+		//$dobr        = Request::getInt( 'dobr', 0, 'POST' );
+		$pageid  = Request::getInt('pageid', 0, 'POST');
+		$instrid = Request::getInt('instrid', 0, 'POST');
+
+		// Проверка
+		if (!$GLOBALS['xoopsSecurity']->check()) {
+			$err         = true;
+			$err_txt     = implode(', ', $GLOBALS['xoopsSecurity']->getErrors());
+			$message_err .= $err_txt . '<br>' . _AM_INSTR_TRY_AGAIN . '<br>';
+		}
+		// =========================
+		// Устанавливаем новый token
+		$token_name    = 'XOOPS_TOKEN';
+		$token_timeout = 0;
+		// $token_name . '_REQUEST' - название элемента формы
+		$ret['toket'] = $GLOBALS['xoopsSecurity']->createToken($token_timeout, $token_name);
+		// =========================
+
+		// Если мы редактируем
+		if ($pageid) {
+			$objInspage = $pageHandler->get($pageid);
+		} elseif ($instrid) {
+			$objInspage = $pageHandler->create();
+			// Если мы создаём страницу необходимо указать к какой инструкции
+			$objInspage->setVar('instrid', $instrid);
+			// Указываем дату создания
+			$objInspage->setVar('datecreated', $time);
+			// Указываем пользователя
+			$objInspage->setVar('uid', $uid);
+		} else {
+			// Устанавливаем сообщение
+			$ret['message'] = _AM_INSTRUCTION_BADREQUEST;
+			// Возвращаем ответ скрипту через JSON
+			echo json_encode($ret);
+			// Прерываем выполнение
+			exit();
+		}
+
+		// Родительская страница
+		$objInspage->setVar('pid', $pid);
+		// Дата обновления
+		$objInspage->setVar('dateupdated', $time);
+		// Название
+		$objInspage->setVar('title', $title);
+		// Вес
+		$objInspage->setVar('weight', $weight);
+		// Текст
+		$objInspage->setVar('hometext', $hometext);
+		// Сноска
+		$objInspage->setVar('footnote', $footnote);
+		// Статус
+		$objInspage->setVar('status', $status);
+		// Тип
+		$objInspage->setVar('type', $type);
+		// Мета-теги ключевых слов
+		$objInspage->setVar('keywords', $keywords);
+		// Мета-теги описания
+		$objInspage->setVar('description', $description);
+		//
+		$objInspage->setVar('dohtml', $dohtml);
+		$objInspage->setVar('dosmiley', $dosmiley);
+		$objInspage->setVar('doxcode', $doxcode);
+		$objInspage->setVar('dobr', $dobr);
+
+		// Проверка категорий
+		if (!$pageid && !$instrid) {
+			$err         = true;
+			$message_err .= _AM_INSTRUCTION_ERR_INSTR . '<br>';
+		}
+		// Проверка веса
+		if (0 == $weight) {
+			$err         = true;
+			$message_err .= _AM_INSTRUCTION_ERR_WEIGHT . '<br>';
+		}
+		// Проверка родительской страницы
+		if ($pageid && ($pageid == $pid)) {
+			$err         = true;
+			$message_err .= _AM_INSTRUCTION_ERR_PPAGE . '<br>';
+		}
+		// Проверка названия
+		if (!$title) {
+			$err         = true;
+			$message_err .= _AM_INSTR_ERR_TITLE . '<br>';
+		}
+		// Проверка основного текста
+		if (!$hometext) {
+			$err         = true;
+			$message_err .= _AM_INSTR_ERR_HOMETEXT . '<br>';
+		}
+
+		// Если были ошибки
+		if (true === $err) {
+			//
+			$message_err = '<div class="errorMsg" style="text-align: left;">' . $message_err . '</div>';
+			// Устанавливаем сообщение
+			$ret['message'] = $message_err;
+			// Возвращаем ответ скрипту через JSON
+			echo json_encode($ret);
+			// Прерываем выполнение
+			exit();
+			// Если небыло ошибок
+		} else {
+			// Вставляем данные в БД
+			if ($pageHandler->insert($objInspage)) {
+				// Находим ID созданной записи
+				$pageid_new = $pageid ?: $objInspage->getNewInstertId();
+				//
+				$ret['pageid'] = $pageid_new;
+				// Получаем ID инструкции
+				$instrid = $objInspage->getInstrid();
+				// Обновляем в инструкции число страниц и дату
+				$instructionHandler->updatePages($instrid);
+				// Если мы редактируем
+				if ($pageid) {
+					// Устанавливаем сообщение
+					$ret['message'] = '<div class="successMsg" style="text-align: left;">' . _AM_INSTRUCTION_PAGEMODIFY . '</div>';
+					// Возвращаем ответ скрипту через JSON
+					echo json_encode($ret);
+					// Прерываем выполнение
+					exit();
+
+					// Если мы добавляем
+				} else {
+					// Инкримент комментов
+					$pageHandler->updateposts($uid, $status, 'add');
+
+					// Устанавливаем сообщение
+					$ret['message'] = '<div class="successMsg" style="text-align: left;">' . _AM_INSTRUCTION_PAGEADDED . '</div>';
+					// Возвращаем ответ скрипту через JSON
+					echo json_encode($ret);
+					// Прерываем выполнение
+					exit();
+				}
+				// Если не получилось вставить данные
+			} else {
+
+				//
+				$message_err = '<div class="errorMsg" style="text-align: left;">' . $objInspage->getHtmlErrors() . '</div>';
+				// Устанавливаем сообщение
+				$ret['message'] = $message_err;
+				// Возвращаем ответ скрипту через JSON
+				echo json_encode($ret);
+				// Прерываем выполнение
+				exit();
+			}
+		}
+
+		break;
 }

Spacing +9 added lines, -9 removed lines

@@ -54,7 +54,7 @@ 
         $instrid = Request::getInt('instrid', 0, 'POST');
 
         // Проверка
-        if (!$GLOBALS['xoopsSecurity']->check()) {
+        if ( ! $GLOBALS['xoopsSecurity']->check()) {
             $err         = true;
             $err_txt     = implode(', ', $GLOBALS['xoopsSecurity']->getErrors());
             $message_err .= $err_txt . '<br>' . _AM_INSTR_TRY_AGAIN . '<br>';
@@ -114,28 +114,28 @@ 
         $objInspage->setVar('dobr', $dobr);
 
         // Проверка категорий
-        if (!$pageid && !$instrid) {
-            $err         = true;
+        if ( ! $pageid && ! $instrid) {
+            $err = true;
             $message_err .= _AM_INSTRUCTION_ERR_INSTR . '<br>';
         }
         // Проверка веса
         if (0 == $weight) {
-            $err         = true;
+            $err = true;
             $message_err .= _AM_INSTRUCTION_ERR_WEIGHT . '<br>';
         }
         // Проверка родительской страницы
         if ($pageid && ($pageid == $pid)) {
-            $err         = true;
+            $err = true;
             $message_err .= _AM_INSTRUCTION_ERR_PPAGE . '<br>';
         }
         // Проверка названия
-        if (!$title) {
-            $err         = true;
+        if ( ! $title) {
+            $err = true;
             $message_err .= _AM_INSTR_ERR_TITLE . '<br>';
         }
         // Проверка основного текста
-        if (!$hometext) {
-            $err         = true;
+        if ( ! $hometext) {
+            $err = true;
             $message_err .= _AM_INSTR_ERR_HOMETEXT . '<br>';
         }
 

admin/cat.php

Indentation +259 added lines, -259 removed lines

@@ -29,264 +29,264 @@
 // Выбор
 switch ($op) {
 
-    case 'main':
-
-        // Подключаем трей
-
-        //        include_once __DIR__  . '/../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 {$instructionHandler->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 = $categoryHandler->getall($criteria);
-        unset($criteria);
-
-        // Инициализируем
-        $cattree = new instruction\Tree($ins_cat, 'cid', 'pid');
-        // Выводим списко категорий в шаблон
-        $GLOBALS['xoopsTpl']->assign('insListCat', $cattree->makeCatsAdmin('--', $cidinstrids));
-
-        // Создание новой категории
-        $objInstructioncat = $categoryHandler->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';
-
-        break;
-
-    // Редактирование категории
-    case 'editcat':
-
-        // Заголовок админки
-        xoops_cp_header();
-        // Навигация
-        $adminObject->displayNavigation(basename(__FILE__));
-
-        $objInstructioncat = $categoryHandler->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';
-
-        break;
-
-    // Сохранение категорий
-    case 'savecat':
-
-        // Проверка
-        if (!$GLOBALS['xoopsSecurity']->check()) {
-            redirect_header('cat.php', 3, implode(',', $GLOBALS['xoopsSecurity']->getErrors()));
-        }
-        // Если мы редактируем
-        if ($cid) {
-            $objInstructioncat = $categoryHandler->get($cid);
-        } else {
-            $objInstructioncat = $categoryHandler->create();
-            // Указываем дату создания
-            $objInstructioncat->setVar('datecreated', $time);
-        }
-
-        $err         = false;
-        $message_err = '';
-
-        // Дата обновления
-        $objInstructioncat->setVar('dateupdated', $time);
-        $objInstructioncat->setVar('pid', $pid);
-        $objInstructioncat->setVar('title', Request::getString('title', '', 'POST'));
-        $objInstructioncat->setVar('description', Request::getString('description', '', 'POST'));
-        $objInstructioncat->setVar('weight', $weight);
-        $objInstructioncat->setVar('metakeywords', Request::getString('metakeywords', '', 'POST'));
-        $objInstructioncat->setVar('metadescription', Request::getString('metadescription', '', 'POST'));
-
-        // Проверка веса
-        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 ($categoryHandler->insert($objInstructioncat)) {
-
-                // ID категории. Если редактируем - то не изменяется. Если создаём новую - то получаем ID созданной записи.
-                $new_cid = $cid ?: $objInstructioncat->getNewInstertId();
-
-                // ===============
-                // ==== Права ====
-                // ===============
-
-                $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 (Request::hasVar('groups_instr_view', 'POST')) {
-                    foreach (Request::getArray('groups_instr_view', '', 'POST') as $onegroup_id) {
-                        $gpermHandler->addRight('instruction_view', $new_cid, $onegroup_id, $GLOBALS['xoopsModule']->getVar('mid'));
-                    }
-                }
-                // Права на добавление
-                if (Request::hasVar('groups_instr_submit', 'POST')) {
-                    foreach (Request::getArray('groups_instr_submit', '', 'POST') as $onegroup_id) {
-                        $gpermHandler->addRight('instruction_submit', $new_cid, $onegroup_id, $GLOBALS['xoopsModule']->getVar('mid'));
-                    }
-                }
-                // Права на редактирование
-                if (Request::hasVar('groups_instr_edit', 'POST')) {
-                    foreach (Request::getArray('groups_instr_edit', '', 'POST') 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';
-
-        break;
-
-    // Удаление категории
-    case 'delcat':
-
-        // Находим число инструкций в данной категории
-        // Критерий выборки
-        $criteria = new \CriteriaCompo();
-        // Все инструкции в данной категории
-        $criteria->add(new \Criteria('cid', $cid, '='));
-        $numrows = $instructionHandler->getCount($criteria);
-        //
-        unset($criteria);
-        // Если есть хоть одна инструкция
-        if ($numrows) {
-            redirect_header('cat.php', 3, _AM_INSTRUCTION_ERR_CATNOTEMPTY);
-        }
-
-        $objInscat = $categoryHandler->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   = $categoryHandler->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 = Request::getInt('ok', 0, 'POST');
-        // Если мы нажали на кнопку
-        if ($ok) {
-
-            // Проверка
-            if (!$GLOBALS['xoopsSecurity']->check()) {
-                redirect_header('cat.php', 3, implode(',', $GLOBALS['xoopsSecurity']->getErrors()));
-            }
-            // Пытаемся удалить категорию
-            if ($categoryHandler->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';
-        }
-
-        break;
+	case 'main':
+
+		// Подключаем трей
+
+		//        include_once __DIR__  . '/../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 {$instructionHandler->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 = $categoryHandler->getall($criteria);
+		unset($criteria);
+
+		// Инициализируем
+		$cattree = new instruction\Tree($ins_cat, 'cid', 'pid');
+		// Выводим списко категорий в шаблон
+		$GLOBALS['xoopsTpl']->assign('insListCat', $cattree->makeCatsAdmin('--', $cidinstrids));
+
+		// Создание новой категории
+		$objInstructioncat = $categoryHandler->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';
+
+		break;
+
+	// Редактирование категории
+	case 'editcat':
+
+		// Заголовок админки
+		xoops_cp_header();
+		// Навигация
+		$adminObject->displayNavigation(basename(__FILE__));
+
+		$objInstructioncat = $categoryHandler->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';
+
+		break;
+
+	// Сохранение категорий
+	case 'savecat':
+
+		// Проверка
+		if (!$GLOBALS['xoopsSecurity']->check()) {
+			redirect_header('cat.php', 3, implode(',', $GLOBALS['xoopsSecurity']->getErrors()));
+		}
+		// Если мы редактируем
+		if ($cid) {
+			$objInstructioncat = $categoryHandler->get($cid);
+		} else {
+			$objInstructioncat = $categoryHandler->create();
+			// Указываем дату создания
+			$objInstructioncat->setVar('datecreated', $time);
+		}
+
+		$err         = false;
+		$message_err = '';
+
+		// Дата обновления
+		$objInstructioncat->setVar('dateupdated', $time);
+		$objInstructioncat->setVar('pid', $pid);
+		$objInstructioncat->setVar('title', Request::getString('title', '', 'POST'));
+		$objInstructioncat->setVar('description', Request::getString('description', '', 'POST'));
+		$objInstructioncat->setVar('weight', $weight);
+		$objInstructioncat->setVar('metakeywords', Request::getString('metakeywords', '', 'POST'));
+		$objInstructioncat->setVar('metadescription', Request::getString('metadescription', '', 'POST'));
+
+		// Проверка веса
+		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 ($categoryHandler->insert($objInstructioncat)) {
+
+				// ID категории. Если редактируем - то не изменяется. Если создаём новую - то получаем ID созданной записи.
+				$new_cid = $cid ?: $objInstructioncat->getNewInstertId();
+
+				// ===============
+				// ==== Права ====
+				// ===============
+
+				$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 (Request::hasVar('groups_instr_view', 'POST')) {
+					foreach (Request::getArray('groups_instr_view', '', 'POST') as $onegroup_id) {
+						$gpermHandler->addRight('instruction_view', $new_cid, $onegroup_id, $GLOBALS['xoopsModule']->getVar('mid'));
+					}
+				}
+				// Права на добавление
+				if (Request::hasVar('groups_instr_submit', 'POST')) {
+					foreach (Request::getArray('groups_instr_submit', '', 'POST') as $onegroup_id) {
+						$gpermHandler->addRight('instruction_submit', $new_cid, $onegroup_id, $GLOBALS['xoopsModule']->getVar('mid'));
+					}
+				}
+				// Права на редактирование
+				if (Request::hasVar('groups_instr_edit', 'POST')) {
+					foreach (Request::getArray('groups_instr_edit', '', 'POST') 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';
+
+		break;
+
+	// Удаление категории
+	case 'delcat':
+
+		// Находим число инструкций в данной категории
+		// Критерий выборки
+		$criteria = new \CriteriaCompo();
+		// Все инструкции в данной категории
+		$criteria->add(new \Criteria('cid', $cid, '='));
+		$numrows = $instructionHandler->getCount($criteria);
+		//
+		unset($criteria);
+		// Если есть хоть одна инструкция
+		if ($numrows) {
+			redirect_header('cat.php', 3, _AM_INSTRUCTION_ERR_CATNOTEMPTY);
+		}
+
+		$objInscat = $categoryHandler->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   = $categoryHandler->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 = Request::getInt('ok', 0, 'POST');
+		// Если мы нажали на кнопку
+		if ($ok) {
+
+			// Проверка
+			if (!$GLOBALS['xoopsSecurity']->check()) {
+				redirect_header('cat.php', 3, implode(',', $GLOBALS['xoopsSecurity']->getErrors()));
+			}
+			// Пытаемся удалить категорию
+			if ($categoryHandler->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';
+		}
+
+		break;
 
 }

Spacing +5 added lines, -5 removed lines

@@ -100,7 +100,7 @@ 
     case 'savecat':
 
         // Проверка
-        if (!$GLOBALS['xoopsSecurity']->check()) {
+        if ( ! $GLOBALS['xoopsSecurity']->check()) {
             redirect_header('cat.php', 3, implode(',', $GLOBALS['xoopsSecurity']->getErrors()));
         }
         // Если мы редактируем
@@ -126,12 +126,12 @@ 
 
         // Проверка веса
         if (0 == $weight) {
-            $err         = true;
+            $err = true;
             $message_err .= _AM_INSTRUCTION_ERR_WEIGHT . '<br>';
         }
         // Проверка категорий
         if ($cid && ($cid == $pid)) {
-            $err         = true;
+            $err = true;
             $message_err .= _AM_INSTRUCTION_ERR_PCAT . '<br>';
         }
         // Если были ошибки
@@ -237,7 +237,7 @@ 
 
         $objInscat = $categoryHandler->get($cid);
         // Если нет такой категории
-        if (!is_object($objInscat)) {
+        if ( ! is_object($objInscat)) {
             redirect_header('cat.php', 3, _AM_INSTRUCTION_ERR_CATNOTSELECT);
         }
 
@@ -258,7 +258,7 @@ 
         if ($ok) {
 
             // Проверка
-            if (!$GLOBALS['xoopsSecurity']->check()) {
+            if ( ! $GLOBALS['xoopsSecurity']->check()) {
                 redirect_header('cat.php', 3, implode(',', $GLOBALS['xoopsSecurity']->getErrors()));
             }
             // Пытаемся удалить категорию

admin/instr.php

Indentation +561 added lines, -561 removed lines

@@ -41,566 +41,566 @@
 // Выбор
 switch ($op) {
 
-    case 'main':
-
-        // Заголовок админки
-        xoops_cp_header();
-        // Меню
-        $adminObject->displayNavigation(basename(__FILE__));
-        $adminObject->addItemButton(_AM_INSTRUCTION_ADDINSTR, 'instr.php?op=editinstr', 'add');
-        $adminObject->displayButton('left', '');
-
-        //
-        $criteria = new \CriteriaCompo();
-
-        // Если была передана категория
-        if ($cid) {
-            // Добавляем в выборку ID категории
-            $criteria->add(new \Criteria('cid', $cid, '='));
-            // Получаем объект категории
-            $objInscat = $categoryHandler->get($cid);
-            // Если нет такой категории
-            if (!is_object($objInscat)) {
-                redirect_header('cat.php', 3, _AM_INSTRUCTION_ERR_CATNOTSELECT);
-            }
-        }
-
-        // Число инструкций, удовлетворяющих данному условию
-        $numrows = $instructionHandler->getCount($criteria);
-
-        // Число выборки
-        $criteria->setLimit($limit);
-        // Начинасть с данного элемента
-        $criteria->setStart($start);
-        // Сортировать по
-        $criteria->setSort('instrid');
-        // Порядок сортировки
-        $criteria->setOrder('DESC');
-        // Находим все справки
-        $instr_arr = $instructionHandler->getall($criteria);
-        // Если записей больше чем $limit, то выводим пагинатор
-        if ($numrows > $limit) {
-            $pagenav = new \XoopsPageNav($numrows, $limit, $start, 'start', 'op=' . $op . '&cid=' . $cid);
-            $pagenav = $pagenav->renderNav(4);
-        } else {
-            $pagenav = '';
-        }
-        // Выводим пагинатор в шаблон
-        $GLOBALS['xoopsTpl']->assign('insPagenav', $pagenav);
-
-        // Если есть записи
-        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_cat = $categoryHandler->get($instr_arr[$i]->getVar('cid'));
-
-                // Выводим в шаблон
-                $GLOBALS['xoopsTpl']->append('insListInstr', ['instrid' => $insinstr_instrid, 'title' => $insinstr_title, 'status' => $insinstr_status, 'pages' => $insinstr_pages, 'ctitle' => $insinstr_cat->getVar('title'), 'cid' => $insinstr_cat->getVar('cid'), 'class' => $class]);
-            }
-
-            //
-            $inshead = isset($objInscat) && is_object($objInscat) ? sprintf(_AM_INSTR_LISTINSTRINCAT, $objInscat->getVar('title')) : _AM_INSTR_LISTINSTRALL;
-            $GLOBALS['xoopsTpl']->assign('insHead', $inshead);
-            // Языковые константы
-            $GLOBALS['xoopsTpl']->assign('lang_title', _AM_INSTRUCTION_TITLE);
-            $GLOBALS['xoopsTpl']->assign('lang_cat', _AM_INSTRUCTION_CAT);
-            $GLOBALS['xoopsTpl']->assign('lang_pages', _AM_INSTRUCTION_PAGES);
-            $GLOBALS['xoopsTpl']->assign('lang_action', _AM_INSTRUCTION_ACTION);
-            $GLOBALS['xoopsTpl']->assign('lang_display', _AM_INSTRUCTION_DISPLAY);
-            $GLOBALS['xoopsTpl']->assign('lang_edit', _AM_INSTRUCTION_EDIT);
-            $GLOBALS['xoopsTpl']->assign('lang_del', _AM_INSTRUCTION_DEL);
-            $GLOBALS['xoopsTpl']->assign('lang_lock', _AM_INSTRUCTION_LOCK);
-            $GLOBALS['xoopsTpl']->assign('lang_unlock', _AM_INSTRUCTION_UNLOCK);
-            $GLOBALS['xoopsTpl']->assign('lang_addpage', _AM_INSTRUCTION_ADDPAGE);
-            $GLOBALS['xoopsTpl']->assign('lang_addinstr', _AM_INSTRUCTION_ADDINSTR);
-        }
-
-        // Выводим шаблон
-        $GLOBALS['xoopsTpl']->display('db:admin/instruction_admin_instr.tpl');
-
-        // Текст внизу админки
-        include __DIR__ . '/admin_footer.php';
-
-        break;
-
-    // Редактирование категории
-    case 'editinstr':
-
-        // Заголовок админки
-        xoops_cp_header();
-        // Меню
-        $adminObject->displayNavigation(basename(__FILE__));
-
-        // Если мы редактируем инструкцию
-        if ($instrid) {
-            $objInsinstr = $instructionHandler->get($instrid);
-            // Создание новой страницы
-        } else {
-            $objInsinstr = $instructionHandler->create();
-        }
-
-        // Выводим шаблон
-        $GLOBALS['xoopsTpl']->display('db:admin/instruction_admin_editinstr.tpl');
-        $form = $objInsinstr->getForm('instr.php');
-        // Форма
-        echo $form->render();
-
-        // Текст внизу админки
-        include __DIR__ . '/admin_footer.php';
-
-        break;
-
-    // Сохранение инструкций
-    case 'saveinstr':
-
-        // Проверка
-        if (!$GLOBALS['xoopsSecurity']->check()) {
-            redirect_header('instr.php', 3, implode(',', $GLOBALS['xoopsSecurity']->getErrors()));
-        }
-        // Если мы редактируем
-        if ($instrid) {
-            $objInsinstr = $instructionHandler->get($instrid);
-        } else {
-            $objInsinstr = $instructionHandler->create();
-            // Указываем дату создания
-            $objInsinstr->setVar('datecreated', $time);
-            // Указываем пользователя
-            $objInsinstr->setVar('uid', $uid);
-        }
-
-        $err         = false;
-        $message_err = '';
-        //
-        $instr_title       = Request::getString('title', '', 'POST');
-        $instr_description = Request::getString('description', '', 'POST');
-
-        // Дата обновления
-        $objInsinstr->setVar('dateupdated', $time);
-        //
-        $objInsinstr->setVar('cid', $cid);
-        $objInsinstr->setVar('title', $instr_title);
-        $objInsinstr->setVar('status', Request::getInt('status', 0));
-        $objInsinstr->setVar('description', $instr_description);
-        $objInsinstr->setVar('metakeywords', Request::getString('metakeywords', ''));
-        $objInsinstr->setVar('metadescription', Request::getString('metadescription', ''));
-
-        // Проверка категорий
-        if (!$cid) {
-            $err         = true;
-            $message_err .= _AM_INSTRUCTION_ERR_CAT . '<br>';
-        }
-        // Проверка названия
-        if (!$instr_title) {
-            $err         = true;
-            $message_err .= _AM_INSTR_ERR_TITLE . '<br>';
-        }
-        // Проверка основного текста
-        if (!$instr_description) {
-            $err         = true;
-            $message_err .= _AM_INSTR_ERR_DESCRIPTION . '<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 ($instructionHandler->insert($objInsinstr)) {
-                // Получаем ID созданной записи
-                $instrid_new = $instrid ?: $objInsinstr->getNewInstertId();
-                // Обновление даты в категории
-                $categoryHandler->updateDateupdated($cid, $time);
-                // Тэги
-                if (xoops_getModuleOption('usetag', 'instruction')) {
-                    $tagHandler = xoops_getModuleHandler('tag', 'tag');
-                    $tagHandler->updateByItem(Request::getArray('tag', '', 'POST'), $instrid_new, $GLOBALS['xoopsModule']->getVar('dirname'), 0);
-                }
-
-                // Если мы редактируем
-                if ($instrid) {
-                    redirect_header('instr.php', 3, _AM_INSTRUCTION_INSTRMODIFY);
-                } else {
-                    redirect_header('instr.php', 3, _AM_INSTRUCTION_INSTRADDED);
-                }
-            }
-            xoops_cp_header();
-            // Меню страницы
-            $adminObject->displayNavigation(basename(__FILE__));
-
-            // Выводим ошибки в шаблон
-            $GLOBALS['xoopsTpl']->assign('insErrorMsg', $objInstructioncat->getHtmlErrors());
-        }
-        // Выводим шаблон
-        $GLOBALS['xoopsTpl']->display('db:admin/instruction_admin_saveinstr.tpl');
-        // Выводим форму
-        $form = $objInsinstr->getForm();
-        // Форма
-        echo $form->render();
-        // Текст внизу админки
-        include __DIR__ . '/admin_footer.php';
-
-        break;
-
-    // Просмотр категории
-    case 'viewinstr':
-
-        // Подключаем трей
-        include_once XOOPS_ROOT_PATH . '/modules/instruction/class/Tree.php';
-
-        // Заголовок админки
-        xoops_cp_header();
-        // Меню
-        $adminObject->displayNavigation(basename(__FILE__));
-        // Кнопки
-        $adminObject->addItemButton(_AM_INSTRUCTION_ADDPAGE, 'instr.php?op=editpage&instrid=' . $instrid, 'add');
-        $adminObject->displayButton('left', '');
-
-        //
-        $objInsinstr = $instructionHandler->get($instrid);
-
-        // Находим все страницы в данной инструкции
-        $criteria = new \CriteriaCompo();
-        $criteria->add(new \Criteria('instrid', $instrid, '='));
-        $criteria->setSort('weight');
-        $criteria->setOrder('ASC');
-        $ins_page = $pageHandler->getall($criteria);
-        //
-        unset($criteria);
-
-        // Инициализируем
-        $instree = new instruction\Tree($ins_page, 'pageid', 'pid');
-        // Выводим список страниц в шаблон
-        $GLOBALS['xoopsTpl']->assign('insListPage', $instree->makePagesAdmin($objInsinstr, '--'));
-
-        // Выводим шаблон
-        $GLOBALS['xoopsTpl']->display('db:admin/instruction_admin_viewinstr.tpl');
-
-        // Текст внизу админки
-        include __DIR__ . '/admin_footer.php';
-
-        break;
-
-    // Удаление категории
-    case 'delinstr':
-
-        // Проверка на instrid
-        // ==================
-        // Объект инструкций
-        $objInsinstr = $instructionHandler->get($instrid);
-
-        // Нажали ли мы на кнопку OK
-        $ok = Request::getInt('ok', 0, 'POST');
-        //
-        if ($ok) {
-
-            // Проверка
-            if (!$GLOBALS['xoopsSecurity']->check()) {
-                redirect_header('instr.php', 3, implode(',', $GLOBALS['xoopsSecurity']->getErrors()));
-            }
-            // Находим все страницы, пренадлежащие этой инструкции
-            $criteria = new \CriteriaCompo();
-            $criteria->add(new \Criteria('instrid', $instrid));
-            $ins_page = $pageHandler->getall($criteria);
-            //
-            unset($criteria);
-            // Перебираем все страницы в данной инструкции
-            foreach (array_keys($ins_page) as $i) {
-                // Декримент комментов
-                // Делает дикримент одного коммента, а не всех в цикле...
-                // Удаляем комментарии
-                xoops_comment_delete($GLOBALS['xoopsModule']->getVar('mid'), $ins_page[$i]->getVar('pageid'));
-                // Декримент страниц (Опционально)
-                // ==============================
-
-                // Удаляем страницу
-                // Сделать проверку на удалённость страницы
-                // ========================================
-                $pageHandler->delete($ins_page[$i]);
-            }
-            // Пытаемся удалить инструкцию
-            if ($instructionHandler->delete($objInsinstr)) {
-                // Редирект
-                redirect_header('instr.php', 3, _AM_INSTRUCTION_INSTRDELETED);
-            } else {
-                // Редирект
-                redirect_header('instr.php', 3, _AM_INSTRUCTION_ERR_DELINSTR);
-            }
-        } else {
-            xoops_cp_header();
-
-            $adminObject->displayNavigation(basename(__FILE__));
-            // Форма
-            xoops_confirm(['ok' => 1, 'instrid' => $instrid, 'op' => 'delinstr'], 'instr.php', sprintf(_AM_INSTRUCTION_FORMDELINSTR, $objInsinstr->getVar('title')));
-            // Текст внизу админки
-            include __DIR__ . '/admin_footer.php';
-        }
-
-        break;
-
-    // Добавление страницы
-    case 'editpage':
-
-        // Заголовок админки
-        xoops_cp_header();
-        // Скрипты
-        $xoTheme->addScript(XOOPS_URL . '/modules/instruction/assets/js/admin.js');
-        // Меню
-        $adminObject->displayNavigation(basename(__FILE__));
-
-        // Если мы редактируем страницу
-        if ($pageid) {
-            // Получаем объект страницы
-            $objInspage = $pageHandler->get($pageid);
-            // ID инструкции
-            $instrid = $objInspage->getVar('instrid');
-            // Создание новой страницы
-        } elseif ($instrid) {
-            // Создаём объект страницы
-            $objInspage = $pageHandler->create();
-            // Устанавливаем родительскую страницу
-            $objInspage->setVar('pid', $pid);
-        } else {
-            redirect_header('instr.php', 3, _AM_INSTRUCTION_BADREQUEST);
-        }
-        // Форма
-        $form = $objInspage->getForm('instr.php', $instrid);
-        // Форма
-        echo $form->render();
-        // Выводим шаблон
-        $GLOBALS['xoopsTpl']->display('db:admin/instruction_admin_editpage.tpl');
-
-        // Текст внизу админки
-        include __DIR__ . '/admin_footer.php';
-
-        break;
-
-    // Сохранение страницы
-    case 'savepage':
-        // Ошибки
-        $err         = false;
-        $message_err = '';
-
-        // Проверка сессии
-        if (!$GLOBALS['xoopsSecurity']->check()) {
-            $err         = true;
-            $err_txt     = implode(', ', $GLOBALS['xoopsSecurity']->getErrors());
-            $message_err .= $err_txt . '<br>';
-        }
-
-        // Если мы редактируем
-        if ($pageid) {
-            $objInspage = $pageHandler->get($pageid);
-        } elseif ($instrid) {
-            $objInspage = $pageHandler->create();
-            // Если мы создаём страницу необходимо указать к какой инструкции
-            $objInspage->setVar('instrid', $instrid);
-            // Указываем дату создания
-            $objInspage->setVar('datecreated', $time);
-            // Указываем пользователя
-            $objInspage->setVar('uid', $uid);
-        } else {
-            redirect_header('instr.php', 3, _AM_INSTRUCTION_BADREQUEST);
-        }
-
-        //
-        $page_title    = Request::getString('title', '', 'POST');
-        $page_hometext = Request::getString('hometext', '', 'POST');
-
-        // Родительская страница
-        $objInspage->setVar('pid', $pid);
-        // Дата обновления
-        $objInspage->setVar('dateupdated', $time);
-        // Название страницы
-        $objInspage->setVar('title', $page_title);
-        // Вес страницы
-        $objInspage->setVar('weight', $weight);
-        // Основной текст
-        $objInspage->setVar('hometext', $page_hometext);
-        // Сноска
-        $objInspage->setVar('footnote', Request::getString('footnote', '', 'POST'));
-        // Статус
-        $objInspage->setVar('status', Request::getInt('status', 0, 'POST'));
-        // Тип
-        $objInspage->setVar('type', Request::getInt('type', 0, 'POST'));
-        // Мета-теги описания
-        $objInspage->setVar('keywords', Request::getString('keywords', '', 'POST'));
-        // Мета-теги ключевых слов
-        $objInspage->setVar('description', Request::getString('description', '', 'POST'));
-        //
-        $dosmiley = (Request::getInt('dosmiley', 0, 'POST') > 0) ? 1 : 0;
-        $doxcode  = (Request::getInt('doxcode', 0, 'POST') > 0) ? 1 : 0;
-        $dobr     = (Request::getInt('dobr', 0, 'POST') > 0) ? 1 : 0;
-        $dohtml   = (Request::getInt('dohtml', 0, 'POST') > 0) ? 1 : 0;
-        //$doimage = ( isset( $_POST['doimage'] ) && intval( $_POST['doimage'] ) > 0 ) ? 1 : 0;
-        $objInspage->setVar('dohtml', $dohtml);
-        $objInspage->setVar('dosmiley', $dosmiley);
-        $objInspage->setVar('doxcode', $doxcode);
-        //$objInspage->setVar( 'doimage', $doimage );
-        $objInspage->setVar('dobr', $dobr);
-
-        //
-        if (!$pageid && !$instrid) {
-            $err         = true;
-            $message_err .= _AM_INSTRUCTION_ERR_INSTR . '<br>';
-        }
-        // Проверка веса
-        if (0 == $weight) {
-            $err         = true;
-            $message_err .= _AM_INSTRUCTION_ERR_WEIGHT . '<br>';
-        }
-        // Проверка родительской страницы
-        if ($pageid && ($pageid == $pid)) {
-            $err         = true;
-            $message_err .= _AM_INSTRUCTION_ERR_PPAGE . '<br>';
-        }
-        // Проверка названия
-        if (!$page_title) {
-            $err         = true;
-            $message_err .= _AM_INSTR_ERR_TITLE . '<br>';
-        }
-        // Проверка основного текста
-        if (!$page_hometext) {
-            $err         = true;
-            $message_err .= _AM_INSTR_ERR_HOMETEXT . '<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 ($pageHandler->insert($objInspage)) {
-                // Ссылка для редиректа
-                $redirect_url = 'instr.php?op=viewinstr&amp;instrid=' . $instrid . '#pageid_' . $pid;
-                // Получаем ID инструкции
-                $instrid = $objInspage->getInstrid();
-                // Обновляем в инструкции число страниц и дату
-                $instructionHandler->updatePages($instrid);
-                // Если мы редактируем
-                if ($pageid) {
-                    // Редирект
-                    redirect_header($redirect_url, 3, _AM_INSTRUCTION_PAGEMODIFY);
-                    // Если мы добавляем
-                } else {
-                    // Инкримент комментов
-                    $pageHandler->updateposts($uid, Request::getInt('status', 0, 'POST'), 'add');
-                    // Редирект
-                    redirect_header($redirect_url, 3, _AM_INSTRUCTION_PAGEADDED);
-                }
-            }
-            xoops_cp_header();
-            // Меню страницы
-            $adminObject->displayNavigation(basename(__FILE__));
-
-            // Выводим ошибки в шаблон
-            $GLOBALS['xoopsTpl']->assign('insErrorMsg', $objInspage->getHtmlErrors());
-        }
-        // Скрипты
-        $xoTheme->addScript(XOOPS_URL . '/modules/instruction/assets/js/admin.js');
-        // Выводим шаблон
-        $GLOBALS['xoopsTpl']->display('db:admin/instruction_admin_savepage.tpl');
-        // Выводим форму
-        $form = $objInspage->getForm('instr.php', $instrid);
-        // Форма
-        echo $form->render();
-        // Текст внизу админки
-        include __DIR__ . '/admin_footer.php';
-
-        break;
-
-    // Удаление страницы
-    case 'delpage':
-
-        // Проверка на pageid
-        // ==================
-
-        $objInspage = $pageHandler->get($pageid);
-        // Нажали ли мы на кнопку OK
-        $ok = Request::getInt('ok', 0, 'POST');
-        // Если мы нажали на кнопку
-        if ($ok) {
-
-            // Проверка
-            if (!$GLOBALS['xoopsSecurity']->check()) {
-                redirect_header('instr.php', 3, implode(',', $GLOBALS['xoopsSecurity']->getErrors()));
-            }
-            // ID инструкции
-            $page_instrid = $objInspage->getVar('instrid');
-            // Декримент комментов
-            $pageHandler->updateposts($objInspage->getVar('uid'), $objInspage->getVar('status'), 'delete');
-            // Пытаемся удалить страницу
-            if ($pageHandler->delete($objInspage)) {
-                // Обновляем в инструкции число страниц и дату
-                $instructionHandler->updatePages($page_instrid);
-                // Удаляем комментарии
-                xoops_comment_delete($GLOBALS['xoopsModule']->getVar('mid'), $pageid);
-                //
-                redirect_header('instr.php?op=viewinstr&amp;instrid=' . $page_instrid, 3, _AM_INSTRUCTION_PAGEDELETED);
-                // Если не смогли удалить страницу
-            } else {
-                redirect_header('instr.php?op=viewinstr&amp;instrid=' . $page_instrid, 3, _AM_INSTRUCTION_ERR_DELPAGE);
-            }
-        } else {
-
-            // Заголовок админки
-            xoops_cp_header();
-            // Меню
-            $adminObject->displayNavigation(basename(__FILE__));
-            // Форма
-            xoops_confirm(['ok' => 1, 'pageid' => $pageid, 'op' => 'delpage'], 'instr.php', sprintf(_AM_INSTRUCTION_FORMDELPAGE, $objInspage->getVar('title')));
-            // Текст внизу админки
-            include __DIR__ . '/admin_footer.php';
-        }
-
-        break;
-
-    // Удаление страницы
-    case 'updpage':
-
-        // Принимаем данные
-        $pageids = Request::getArray('pageids', 0, 'POST');
-        $weights = Request::getArray('weights', 0, 'POST');
-        // Перебираем все значения
-        foreach ($pageids as $key => $pageid) {
-
-            // Объявляем объект
-            $objInspage = $pageHandler->get($pageid);
-            // Устанавливаем вес
-            $objInspage->setVar('weight', $weights[$key]);
-            // Вставляем данные в БД
-            $pageHandler->insert($objInspage);
-            // Удаляем объект
-            unset($objInspage);
-        }
-        // Редирект
-        redirect_header('instr.php?op=viewinstr&instrid=' . $instrid, 3, _AM_INSTRUCTION_PAGESUPDATE);
-
-        break;
+	case 'main':
+
+		// Заголовок админки
+		xoops_cp_header();
+		// Меню
+		$adminObject->displayNavigation(basename(__FILE__));
+		$adminObject->addItemButton(_AM_INSTRUCTION_ADDINSTR, 'instr.php?op=editinstr', 'add');
+		$adminObject->displayButton('left', '');
+
+		//
+		$criteria = new \CriteriaCompo();
+
+		// Если была передана категория
+		if ($cid) {
+			// Добавляем в выборку ID категории
+			$criteria->add(new \Criteria('cid', $cid, '='));
+			// Получаем объект категории
+			$objInscat = $categoryHandler->get($cid);
+			// Если нет такой категории
+			if (!is_object($objInscat)) {
+				redirect_header('cat.php', 3, _AM_INSTRUCTION_ERR_CATNOTSELECT);
+			}
+		}
+
+		// Число инструкций, удовлетворяющих данному условию
+		$numrows = $instructionHandler->getCount($criteria);
+
+		// Число выборки
+		$criteria->setLimit($limit);
+		// Начинасть с данного элемента
+		$criteria->setStart($start);
+		// Сортировать по
+		$criteria->setSort('instrid');
+		// Порядок сортировки
+		$criteria->setOrder('DESC');
+		// Находим все справки
+		$instr_arr = $instructionHandler->getall($criteria);
+		// Если записей больше чем $limit, то выводим пагинатор
+		if ($numrows > $limit) {
+			$pagenav = new \XoopsPageNav($numrows, $limit, $start, 'start', 'op=' . $op . '&amp;cid=' . $cid);
+			$pagenav = $pagenav->renderNav(4);
+		} else {
+			$pagenav = '';
+		}
+		// Выводим пагинатор в шаблон
+		$GLOBALS['xoopsTpl']->assign('insPagenav', $pagenav);
+
+		// Если есть записи
+		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_cat = $categoryHandler->get($instr_arr[$i]->getVar('cid'));
+
+				// Выводим в шаблон
+				$GLOBALS['xoopsTpl']->append('insListInstr', ['instrid' => $insinstr_instrid, 'title' => $insinstr_title, 'status' => $insinstr_status, 'pages' => $insinstr_pages, 'ctitle' => $insinstr_cat->getVar('title'), 'cid' => $insinstr_cat->getVar('cid'), 'class' => $class]);
+			}
+
+			//
+			$inshead = isset($objInscat) && is_object($objInscat) ? sprintf(_AM_INSTR_LISTINSTRINCAT, $objInscat->getVar('title')) : _AM_INSTR_LISTINSTRALL;
+			$GLOBALS['xoopsTpl']->assign('insHead', $inshead);
+			// Языковые константы
+			$GLOBALS['xoopsTpl']->assign('lang_title', _AM_INSTRUCTION_TITLE);
+			$GLOBALS['xoopsTpl']->assign('lang_cat', _AM_INSTRUCTION_CAT);
+			$GLOBALS['xoopsTpl']->assign('lang_pages', _AM_INSTRUCTION_PAGES);
+			$GLOBALS['xoopsTpl']->assign('lang_action', _AM_INSTRUCTION_ACTION);
+			$GLOBALS['xoopsTpl']->assign('lang_display', _AM_INSTRUCTION_DISPLAY);
+			$GLOBALS['xoopsTpl']->assign('lang_edit', _AM_INSTRUCTION_EDIT);
+			$GLOBALS['xoopsTpl']->assign('lang_del', _AM_INSTRUCTION_DEL);
+			$GLOBALS['xoopsTpl']->assign('lang_lock', _AM_INSTRUCTION_LOCK);
+			$GLOBALS['xoopsTpl']->assign('lang_unlock', _AM_INSTRUCTION_UNLOCK);
+			$GLOBALS['xoopsTpl']->assign('lang_addpage', _AM_INSTRUCTION_ADDPAGE);
+			$GLOBALS['xoopsTpl']->assign('lang_addinstr', _AM_INSTRUCTION_ADDINSTR);
+		}
+
+		// Выводим шаблон
+		$GLOBALS['xoopsTpl']->display('db:admin/instruction_admin_instr.tpl');
+
+		// Текст внизу админки
+		include __DIR__ . '/admin_footer.php';
+
+		break;
+
+	// Редактирование категории
+	case 'editinstr':
+
+		// Заголовок админки
+		xoops_cp_header();
+		// Меню
+		$adminObject->displayNavigation(basename(__FILE__));
+
+		// Если мы редактируем инструкцию
+		if ($instrid) {
+			$objInsinstr = $instructionHandler->get($instrid);
+			// Создание новой страницы
+		} else {
+			$objInsinstr = $instructionHandler->create();
+		}
+
+		// Выводим шаблон
+		$GLOBALS['xoopsTpl']->display('db:admin/instruction_admin_editinstr.tpl');
+		$form = $objInsinstr->getForm('instr.php');
+		// Форма
+		echo $form->render();
+
+		// Текст внизу админки
+		include __DIR__ . '/admin_footer.php';
+
+		break;
+
+	// Сохранение инструкций
+	case 'saveinstr':
+
+		// Проверка
+		if (!$GLOBALS['xoopsSecurity']->check()) {
+			redirect_header('instr.php', 3, implode(',', $GLOBALS['xoopsSecurity']->getErrors()));
+		}
+		// Если мы редактируем
+		if ($instrid) {
+			$objInsinstr = $instructionHandler->get($instrid);
+		} else {
+			$objInsinstr = $instructionHandler->create();
+			// Указываем дату создания
+			$objInsinstr->setVar('datecreated', $time);
+			// Указываем пользователя
+			$objInsinstr->setVar('uid', $uid);
+		}
+
+		$err         = false;
+		$message_err = '';
+		//
+		$instr_title       = Request::getString('title', '', 'POST');
+		$instr_description = Request::getString('description', '', 'POST');
+
+		// Дата обновления
+		$objInsinstr->setVar('dateupdated', $time);
+		//
+		$objInsinstr->setVar('cid', $cid);
+		$objInsinstr->setVar('title', $instr_title);
+		$objInsinstr->setVar('status', Request::getInt('status', 0));
+		$objInsinstr->setVar('description', $instr_description);
+		$objInsinstr->setVar('metakeywords', Request::getString('metakeywords', ''));
+		$objInsinstr->setVar('metadescription', Request::getString('metadescription', ''));
+
+		// Проверка категорий
+		if (!$cid) {
+			$err         = true;
+			$message_err .= _AM_INSTRUCTION_ERR_CAT . '<br>';
+		}
+		// Проверка названия
+		if (!$instr_title) {
+			$err         = true;
+			$message_err .= _AM_INSTR_ERR_TITLE . '<br>';
+		}
+		// Проверка основного текста
+		if (!$instr_description) {
+			$err         = true;
+			$message_err .= _AM_INSTR_ERR_DESCRIPTION . '<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 ($instructionHandler->insert($objInsinstr)) {
+				// Получаем ID созданной записи
+				$instrid_new = $instrid ?: $objInsinstr->getNewInstertId();
+				// Обновление даты в категории
+				$categoryHandler->updateDateupdated($cid, $time);
+				// Тэги
+				if (xoops_getModuleOption('usetag', 'instruction')) {
+					$tagHandler = xoops_getModuleHandler('tag', 'tag');
+					$tagHandler->updateByItem(Request::getArray('tag', '', 'POST'), $instrid_new, $GLOBALS['xoopsModule']->getVar('dirname'), 0);
+				}
+
+				// Если мы редактируем
+				if ($instrid) {
+					redirect_header('instr.php', 3, _AM_INSTRUCTION_INSTRMODIFY);
+				} else {
+					redirect_header('instr.php', 3, _AM_INSTRUCTION_INSTRADDED);
+				}
+			}
+			xoops_cp_header();
+			// Меню страницы
+			$adminObject->displayNavigation(basename(__FILE__));
+
+			// Выводим ошибки в шаблон
+			$GLOBALS['xoopsTpl']->assign('insErrorMsg', $objInstructioncat->getHtmlErrors());
+		}
+		// Выводим шаблон
+		$GLOBALS['xoopsTpl']->display('db:admin/instruction_admin_saveinstr.tpl');
+		// Выводим форму
+		$form = $objInsinstr->getForm();
+		// Форма
+		echo $form->render();
+		// Текст внизу админки
+		include __DIR__ . '/admin_footer.php';
+
+		break;
+
+	// Просмотр категории
+	case 'viewinstr':
+
+		// Подключаем трей
+		include_once XOOPS_ROOT_PATH . '/modules/instruction/class/Tree.php';
+
+		// Заголовок админки
+		xoops_cp_header();
+		// Меню
+		$adminObject->displayNavigation(basename(__FILE__));
+		// Кнопки
+		$adminObject->addItemButton(_AM_INSTRUCTION_ADDPAGE, 'instr.php?op=editpage&instrid=' . $instrid, 'add');
+		$adminObject->displayButton('left', '');
+
+		//
+		$objInsinstr = $instructionHandler->get($instrid);
+
+		// Находим все страницы в данной инструкции
+		$criteria = new \CriteriaCompo();
+		$criteria->add(new \Criteria('instrid', $instrid, '='));
+		$criteria->setSort('weight');
+		$criteria->setOrder('ASC');
+		$ins_page = $pageHandler->getall($criteria);
+		//
+		unset($criteria);
+
+		// Инициализируем
+		$instree = new instruction\Tree($ins_page, 'pageid', 'pid');
+		// Выводим список страниц в шаблон
+		$GLOBALS['xoopsTpl']->assign('insListPage', $instree->makePagesAdmin($objInsinstr, '--'));
+
+		// Выводим шаблон
+		$GLOBALS['xoopsTpl']->display('db:admin/instruction_admin_viewinstr.tpl');
+
+		// Текст внизу админки
+		include __DIR__ . '/admin_footer.php';
+
+		break;
+
+	// Удаление категории
+	case 'delinstr':
+
+		// Проверка на instrid
+		// ==================
+		// Объект инструкций
+		$objInsinstr = $instructionHandler->get($instrid);
+
+		// Нажали ли мы на кнопку OK
+		$ok = Request::getInt('ok', 0, 'POST');
+		//
+		if ($ok) {
+
+			// Проверка
+			if (!$GLOBALS['xoopsSecurity']->check()) {
+				redirect_header('instr.php', 3, implode(',', $GLOBALS['xoopsSecurity']->getErrors()));
+			}
+			// Находим все страницы, пренадлежащие этой инструкции
+			$criteria = new \CriteriaCompo();
+			$criteria->add(new \Criteria('instrid', $instrid));
+			$ins_page = $pageHandler->getall($criteria);
+			//
+			unset($criteria);
+			// Перебираем все страницы в данной инструкции
+			foreach (array_keys($ins_page) as $i) {
+				// Декримент комментов
+				// Делает дикримент одного коммента, а не всех в цикле...
+				// Удаляем комментарии
+				xoops_comment_delete($GLOBALS['xoopsModule']->getVar('mid'), $ins_page[$i]->getVar('pageid'));
+				// Декримент страниц (Опционально)
+				// ==============================
+
+				// Удаляем страницу
+				// Сделать проверку на удалённость страницы
+				// ========================================
+				$pageHandler->delete($ins_page[$i]);
+			}
+			// Пытаемся удалить инструкцию
+			if ($instructionHandler->delete($objInsinstr)) {
+				// Редирект
+				redirect_header('instr.php', 3, _AM_INSTRUCTION_INSTRDELETED);
+			} else {
+				// Редирект
+				redirect_header('instr.php', 3, _AM_INSTRUCTION_ERR_DELINSTR);
+			}
+		} else {
+			xoops_cp_header();
+
+			$adminObject->displayNavigation(basename(__FILE__));
+			// Форма
+			xoops_confirm(['ok' => 1, 'instrid' => $instrid, 'op' => 'delinstr'], 'instr.php', sprintf(_AM_INSTRUCTION_FORMDELINSTR, $objInsinstr->getVar('title')));
+			// Текст внизу админки
+			include __DIR__ . '/admin_footer.php';
+		}
+
+		break;
+
+	// Добавление страницы
+	case 'editpage':
+
+		// Заголовок админки
+		xoops_cp_header();
+		// Скрипты
+		$xoTheme->addScript(XOOPS_URL . '/modules/instruction/assets/js/admin.js');
+		// Меню
+		$adminObject->displayNavigation(basename(__FILE__));
+
+		// Если мы редактируем страницу
+		if ($pageid) {
+			// Получаем объект страницы
+			$objInspage = $pageHandler->get($pageid);
+			// ID инструкции
+			$instrid = $objInspage->getVar('instrid');
+			// Создание новой страницы
+		} elseif ($instrid) {
+			// Создаём объект страницы
+			$objInspage = $pageHandler->create();
+			// Устанавливаем родительскую страницу
+			$objInspage->setVar('pid', $pid);
+		} else {
+			redirect_header('instr.php', 3, _AM_INSTRUCTION_BADREQUEST);
+		}
+		// Форма
+		$form = $objInspage->getForm('instr.php', $instrid);
+		// Форма
+		echo $form->render();
+		// Выводим шаблон
+		$GLOBALS['xoopsTpl']->display('db:admin/instruction_admin_editpage.tpl');
+
+		// Текст внизу админки
+		include __DIR__ . '/admin_footer.php';
+
+		break;
+
+	// Сохранение страницы
+	case 'savepage':
+		// Ошибки
+		$err         = false;
+		$message_err = '';
+
+		// Проверка сессии
+		if (!$GLOBALS['xoopsSecurity']->check()) {
+			$err         = true;
+			$err_txt     = implode(', ', $GLOBALS['xoopsSecurity']->getErrors());
+			$message_err .= $err_txt . '<br>';
+		}
+
+		// Если мы редактируем
+		if ($pageid) {
+			$objInspage = $pageHandler->get($pageid);
+		} elseif ($instrid) {
+			$objInspage = $pageHandler->create();
+			// Если мы создаём страницу необходимо указать к какой инструкции
+			$objInspage->setVar('instrid', $instrid);
+			// Указываем дату создания
+			$objInspage->setVar('datecreated', $time);
+			// Указываем пользователя
+			$objInspage->setVar('uid', $uid);
+		} else {
+			redirect_header('instr.php', 3, _AM_INSTRUCTION_BADREQUEST);
+		}
+
+		//
+		$page_title    = Request::getString('title', '', 'POST');
+		$page_hometext = Request::getString('hometext', '', 'POST');
+
+		// Родительская страница
+		$objInspage->setVar('pid', $pid);
+		// Дата обновления
+		$objInspage->setVar('dateupdated', $time);
+		// Название страницы
+		$objInspage->setVar('title', $page_title);
+		// Вес страницы
+		$objInspage->setVar('weight', $weight);
+		// Основной текст
+		$objInspage->setVar('hometext', $page_hometext);
+		// Сноска
+		$objInspage->setVar('footnote', Request::getString('footnote', '', 'POST'));
+		// Статус
+		$objInspage->setVar('status', Request::getInt('status', 0, 'POST'));
+		// Тип
+		$objInspage->setVar('type', Request::getInt('type', 0, 'POST'));
+		// Мета-теги описания
+		$objInspage->setVar('keywords', Request::getString('keywords', '', 'POST'));
+		// Мета-теги ключевых слов
+		$objInspage->setVar('description', Request::getString('description', '', 'POST'));
+		//
+		$dosmiley = (Request::getInt('dosmiley', 0, 'POST') > 0) ? 1 : 0;
+		$doxcode  = (Request::getInt('doxcode', 0, 'POST') > 0) ? 1 : 0;
+		$dobr     = (Request::getInt('dobr', 0, 'POST') > 0) ? 1 : 0;
+		$dohtml   = (Request::getInt('dohtml', 0, 'POST') > 0) ? 1 : 0;
+		//$doimage = ( isset( $_POST['doimage'] ) && intval( $_POST['doimage'] ) > 0 ) ? 1 : 0;
+		$objInspage->setVar('dohtml', $dohtml);
+		$objInspage->setVar('dosmiley', $dosmiley);
+		$objInspage->setVar('doxcode', $doxcode);
+		//$objInspage->setVar( 'doimage', $doimage );
+		$objInspage->setVar('dobr', $dobr);
+
+		//
+		if (!$pageid && !$instrid) {
+			$err         = true;
+			$message_err .= _AM_INSTRUCTION_ERR_INSTR . '<br>';
+		}
+		// Проверка веса
+		if (0 == $weight) {
+			$err         = true;
+			$message_err .= _AM_INSTRUCTION_ERR_WEIGHT . '<br>';
+		}
+		// Проверка родительской страницы
+		if ($pageid && ($pageid == $pid)) {
+			$err         = true;
+			$message_err .= _AM_INSTRUCTION_ERR_PPAGE . '<br>';
+		}
+		// Проверка названия
+		if (!$page_title) {
+			$err         = true;
+			$message_err .= _AM_INSTR_ERR_TITLE . '<br>';
+		}
+		// Проверка основного текста
+		if (!$page_hometext) {
+			$err         = true;
+			$message_err .= _AM_INSTR_ERR_HOMETEXT . '<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 ($pageHandler->insert($objInspage)) {
+				// Ссылка для редиректа
+				$redirect_url = 'instr.php?op=viewinstr&amp;instrid=' . $instrid . '#pageid_' . $pid;
+				// Получаем ID инструкции
+				$instrid = $objInspage->getInstrid();
+				// Обновляем в инструкции число страниц и дату
+				$instructionHandler->updatePages($instrid);
+				// Если мы редактируем
+				if ($pageid) {
+					// Редирект
+					redirect_header($redirect_url, 3, _AM_INSTRUCTION_PAGEMODIFY);
+					// Если мы добавляем
+				} else {
+					// Инкримент комментов
+					$pageHandler->updateposts($uid, Request::getInt('status', 0, 'POST'), 'add');
+					// Редирект
+					redirect_header($redirect_url, 3, _AM_INSTRUCTION_PAGEADDED);
+				}
+			}
+			xoops_cp_header();
+			// Меню страницы
+			$adminObject->displayNavigation(basename(__FILE__));
+
+			// Выводим ошибки в шаблон
+			$GLOBALS['xoopsTpl']->assign('insErrorMsg', $objInspage->getHtmlErrors());
+		}
+		// Скрипты
+		$xoTheme->addScript(XOOPS_URL . '/modules/instruction/assets/js/admin.js');
+		// Выводим шаблон
+		$GLOBALS['xoopsTpl']->display('db:admin/instruction_admin_savepage.tpl');
+		// Выводим форму
+		$form = $objInspage->getForm('instr.php', $instrid);
+		// Форма
+		echo $form->render();
+		// Текст внизу админки
+		include __DIR__ . '/admin_footer.php';
+
+		break;
+
+	// Удаление страницы
+	case 'delpage':
+
+		// Проверка на pageid
+		// ==================
+
+		$objInspage = $pageHandler->get($pageid);
+		// Нажали ли мы на кнопку OK
+		$ok = Request::getInt('ok', 0, 'POST');
+		// Если мы нажали на кнопку
+		if ($ok) {
+
+			// Проверка
+			if (!$GLOBALS['xoopsSecurity']->check()) {
+				redirect_header('instr.php', 3, implode(',', $GLOBALS['xoopsSecurity']->getErrors()));
+			}
+			// ID инструкции
+			$page_instrid = $objInspage->getVar('instrid');
+			// Декримент комментов
+			$pageHandler->updateposts($objInspage->getVar('uid'), $objInspage->getVar('status'), 'delete');
+			// Пытаемся удалить страницу
+			if ($pageHandler->delete($objInspage)) {
+				// Обновляем в инструкции число страниц и дату
+				$instructionHandler->updatePages($page_instrid);
+				// Удаляем комментарии
+				xoops_comment_delete($GLOBALS['xoopsModule']->getVar('mid'), $pageid);
+				//
+				redirect_header('instr.php?op=viewinstr&amp;instrid=' . $page_instrid, 3, _AM_INSTRUCTION_PAGEDELETED);
+				// Если не смогли удалить страницу
+			} else {
+				redirect_header('instr.php?op=viewinstr&amp;instrid=' . $page_instrid, 3, _AM_INSTRUCTION_ERR_DELPAGE);
+			}
+		} else {
+
+			// Заголовок админки
+			xoops_cp_header();
+			// Меню
+			$adminObject->displayNavigation(basename(__FILE__));
+			// Форма
+			xoops_confirm(['ok' => 1, 'pageid' => $pageid, 'op' => 'delpage'], 'instr.php', sprintf(_AM_INSTRUCTION_FORMDELPAGE, $objInspage->getVar('title')));
+			// Текст внизу админки
+			include __DIR__ . '/admin_footer.php';
+		}
+
+		break;
+
+	// Удаление страницы
+	case 'updpage':
+
+		// Принимаем данные
+		$pageids = Request::getArray('pageids', 0, 'POST');
+		$weights = Request::getArray('weights', 0, 'POST');
+		// Перебираем все значения
+		foreach ($pageids as $key => $pageid) {
+
+			// Объявляем объект
+			$objInspage = $pageHandler->get($pageid);
+			// Устанавливаем вес
+			$objInspage->setVar('weight', $weights[$key]);
+			// Вставляем данные в БД
+			$pageHandler->insert($objInspage);
+			// Удаляем объект
+			unset($objInspage);
+		}
+		// Редирект
+		redirect_header('instr.php?op=viewinstr&instrid=' . $instrid, 3, _AM_INSTRUCTION_PAGESUPDATE);
+
+		break;
 
 }

Spacing +19 added lines, -19 removed lines

@@ -60,7 +60,7 @@ 
             // Получаем объект категории
             $objInscat = $categoryHandler->get($cid);
             // Если нет такой категории
-            if (!is_object($objInscat)) {
+            if ( ! is_object($objInscat)) {
                 redirect_header('cat.php', 3, _AM_INSTRUCTION_ERR_CATNOTSELECT);
             }
         }
@@ -166,7 +166,7 @@ 
     case 'saveinstr':
 
         // Проверка
-        if (!$GLOBALS['xoopsSecurity']->check()) {
+        if ( ! $GLOBALS['xoopsSecurity']->check()) {
             redirect_header('instr.php', 3, implode(',', $GLOBALS['xoopsSecurity']->getErrors()));
         }
         // Если мы редактируем
@@ -197,18 +197,18 @@ 
         $objInsinstr->setVar('metadescription', Request::getString('metadescription', ''));
 
         // Проверка категорий
-        if (!$cid) {
-            $err         = true;
+        if ( ! $cid) {
+            $err = true;
             $message_err .= _AM_INSTRUCTION_ERR_CAT . '<br>';
         }
         // Проверка названия
-        if (!$instr_title) {
-            $err         = true;
+        if ( ! $instr_title) {
+            $err = true;
             $message_err .= _AM_INSTR_ERR_TITLE . '<br>';
         }
         // Проверка основного текста
-        if (!$instr_description) {
-            $err         = true;
+        if ( ! $instr_description) {
+            $err = true;
             $message_err .= _AM_INSTR_ERR_DESCRIPTION . '<br>';
         }
 
@@ -313,7 +313,7 @@ 
         if ($ok) {
 
             // Проверка
-            if (!$GLOBALS['xoopsSecurity']->check()) {
+            if ( ! $GLOBALS['xoopsSecurity']->check()) {
                 redirect_header('instr.php', 3, implode(',', $GLOBALS['xoopsSecurity']->getErrors()));
             }
             // Находим все страницы, пренадлежащие этой инструкции
@@ -400,7 +400,7 @@ 
         $message_err = '';
 
         // Проверка сессии
-        if (!$GLOBALS['xoopsSecurity']->check()) {
+        if ( ! $GLOBALS['xoopsSecurity']->check()) {
             $err         = true;
             $err_txt     = implode(', ', $GLOBALS['xoopsSecurity']->getErrors());
             $message_err .= $err_txt . '<br>';
@@ -458,28 +458,28 @@ 
         $objInspage->setVar('dobr', $dobr);
 
         //
-        if (!$pageid && !$instrid) {
-            $err         = true;
+        if ( ! $pageid && ! $instrid) {
+            $err = true;
             $message_err .= _AM_INSTRUCTION_ERR_INSTR . '<br>';
         }
         // Проверка веса
         if (0 == $weight) {
-            $err         = true;
+            $err = true;
             $message_err .= _AM_INSTRUCTION_ERR_WEIGHT . '<br>';
         }
         // Проверка родительской страницы
         if ($pageid && ($pageid == $pid)) {
-            $err         = true;
+            $err = true;
             $message_err .= _AM_INSTRUCTION_ERR_PPAGE . '<br>';
         }
         // Проверка названия
-        if (!$page_title) {
-            $err         = true;
+        if ( ! $page_title) {
+            $err = true;
             $message_err .= _AM_INSTR_ERR_TITLE . '<br>';
         }
         // Проверка основного текста
-        if (!$page_hometext) {
-            $err         = true;
+        if ( ! $page_hometext) {
+            $err = true;
             $message_err .= _AM_INSTR_ERR_HOMETEXT . '<br>';
         }
 
@@ -547,7 +547,7 @@ 
         if ($ok) {
 
             // Проверка
-            if (!$GLOBALS['xoopsSecurity']->check()) {
+            if ( ! $GLOBALS['xoopsSecurity']->check()) {
                 redirect_header('instr.php', 3, implode(',', $GLOBALS['xoopsSecurity']->getErrors()));
             }
             // ID инструкции

instr.php

Indentation +33 added lines, -33 removed lines

@@ -19,8 +19,8 @@ 
 $criteria->add(new \Criteria('instrid', $instrid));
 $criteria->add(new \Criteria('status ', '0', '>'));
 if (0 == $instructionHandler->getCount($criteria)) {
-    redirect_header('index.php', 3, _MD_INSTRUCTION_INSTRNOTEXIST);
-    exit();
+	redirect_header('index.php', 3, _MD_INSTRUCTION_INSTRNOTEXIST);
+	exit();
 }
 //
 unset($criteria);
@@ -42,8 +42,8 @@ 
 // Права на просмотр инструкции
 $categories = Xoopsmodules\instruction\Utility::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();
 }
 
 // Массив данных об инструкции
@@ -56,29 +56,29 @@ 
 $instrs['description'] = $objInsinstr->getVar('description');
 // Если админ, рисуем админлинк
 if (($GLOBALS['xoopsUser'] instanceof \XoopsUser) && $GLOBALS['xoopsUser']->isAdmin($GLOBALS['xoopsModule']->mid())) {
-    $instrs['adminlink'] = '&nbsp;<a href="'
-                           . XOOPS_URL
-                           . '/modules/'
-                           . $moduleDirName
-                           . '/admin/instr.php?op=editinstr&instrid='
-                           . $instrid
-                           . '"><img style="width:16px;" src="'. $pathIcon16 . '/edit.png" alt='
-                           . _EDIT
-                           . ' title='
-                           . _EDIT
-                           . '></a>&nbsp;<a href="'
-                           . XOOPS_URL
-                           . '/modules/'
-                           . $moduleDirName
-                           . '/admin/instr.php?op=delinstr&instrid='
-                           . $instrid
-                           . '"><img style="width:16px;" src="'. $pathIcon16 . '/delete.png" alt='
-                           . _DELETE
-                           . ' title='
-                           . _DELETE
-                           . '></a>&nbsp;';
+	$instrs['adminlink'] = '&nbsp;<a href="'
+						   . XOOPS_URL
+						   . '/modules/'
+						   . $moduleDirName
+						   . '/admin/instr.php?op=editinstr&instrid='
+						   . $instrid
+						   . '"><img style="width:16px;" src="'. $pathIcon16 . '/edit.png" alt='
+						   . _EDIT
+						   . ' title='
+						   . _EDIT
+						   . '></a>&nbsp;<a href="'
+						   . XOOPS_URL
+						   . '/modules/'
+						   . $moduleDirName
+						   . '/admin/instr.php?op=delinstr&instrid='
+						   . $instrid
+						   . '"><img style="width:16px;" src="'. $pathIcon16 . '/delete.png" alt='
+						   . _DELETE
+						   . ' title='
+						   . _DELETE
+						   . '></a>&nbsp;';
 } else {
-    $instrs['adminlink'] = '';
+	$instrs['adminlink'] = '';
 }
 
 // Выводим в шаблон
@@ -102,7 +102,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 .= $objInsinstr->getVar('title');
@@ -129,18 +129,18 @@ 
 
 // Теги
 if (xoops_getModuleOption('usetag', 'instruction')) {
-    include_once $GLOBALS['xoops']->path('modules/tag/include/tagbar.php');
-    $xoopsTpl->assign('tags', true);
-    $xoopsTpl->assign('tagbar', tagBar($instrid, 0));
+	include_once $GLOBALS['xoops']->path('modules/tag/include/tagbar.php');
+	$xoopsTpl->assign('tags', true);
+	$xoopsTpl->assign('tagbar', tagBar($instrid, 0));
 } else {
-    $xoopsTpl->assign('tags', false);
+	$xoopsTpl->assign('tags', false);
 }
 
 // Рейтинг
 if (xoops_getModuleOption('userat', 'instruction')) {
-    $xoopsTpl->assign('insUserat', true);
+	$xoopsTpl->assign('insUserat', true);
 } else {
-    $xoopsTpl->assign('insUserat', false);
+	$xoopsTpl->assign('insUserat', false);
 }
 
 // Подвал

Spacing +3 added lines, -3 removed lines

@@ -41,7 +41,7 @@ 
 
 // Права на просмотр инструкции
 $categories = Xoopsmodules\instruction\Utility::getItemIds();
-if (!in_array($objInsinstr->getVar('cid'), $categories)) {
+if ( ! in_array($objInsinstr->getVar('cid'), $categories)) {
     redirect_header(XOOPS_URL . '/modules/' . $moduleDirName . '/', 3, _NOPERM);
     exit();
 }
@@ -62,7 +62,7 @@ 
                            . $moduleDirName
                            . '/admin/instr.php?op=editinstr&instrid='
                            . $instrid
-                           . '"><img style="width:16px;" src="'. $pathIcon16 . '/edit.png" alt='
+                           . '"><img style="width:16px;" src="' . $pathIcon16 . '/edit.png" alt='
                            . _EDIT
                            . ' title='
                            . _EDIT
@@ -72,7 +72,7 @@ 
                            . $moduleDirName
                            . '/admin/instr.php?op=delinstr&instrid='
                            . $instrid
-                           . '"><img style="width:16px;" src="'. $pathIcon16 . '/delete.png" alt='
+                           . '"><img style="width:16px;" src="' . $pathIcon16 . '/delete.png" alt='
                            . _DELETE
                            . ' title='
                            . _DELETE

page.php

Indentation +43 added lines, -43 removed lines

@@ -29,8 +29,8 @@ 
 $criteria->add(new \Criteria('pageid ', $pageid));
 $criteria->add(new \Criteria('status ', '0', '>'));
 if (0 == $pageHandler->getCount($criteria)) {
-    redirect_header('index.php', 3, _MD_INSTRUCTION_PAGENOTEXIST);
-    exit();
+	redirect_header('index.php', 3, _MD_INSTRUCTION_PAGENOTEXIST);
+	exit();
 }
 //
 unset($criteria);
@@ -42,8 +42,8 @@ 
 
 // Если админ и ссылка на отключение кэша
 if (($GLOBALS['xoopsUser'] instanceof \XoopsUser) && $GLOBALS['xoopsUser']->isAdmin() && $nocache) {
-    // Отключаем кэш
-    $GLOBALS['xoopsConfig']['module_cache'][$GLOBALS['xoopsModule']->getVar('mid')] = 0;
+	// Отключаем кэш
+	$GLOBALS['xoopsConfig']['module_cache'][$GLOBALS['xoopsModule']->getVar('mid')] = 0;
 }
 
 // Задание тайтла
@@ -60,8 +60,8 @@ 
 // Права на просмотр инструкции
 $categories = Xoopsmodules\instruction\Utility::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();
 }
 
 // Массив данных о странице
@@ -78,9 +78,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');
@@ -89,39 +89,39 @@ 
 //
 // Если админ, рисуем админлинк
 if (($GLOBALS['xoopsUser'] instanceof \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="'. $pathIcon16 . '/edit.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="'. $pathIcon16 . '/delete.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="'. $pathIcon16 . '/edit.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="'. $pathIcon16 . '/delete.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="'. $pathIcon16 . '/edit.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="'. $pathIcon16 . '/edit.png" alt=' . _EDIT . ' title=' . _EDIT . '></a>';
+	}
+
+	$pages['adminlink'] .= '&nbsp;';
+	// Если нет админлика
+	if ('[&nbsp;&nbsp;]' === $pages['adminlink']) {
+		$pages['adminlink'] = '';
+	}
 }
 // Выводим в шаблон
 $GLOBALS['xoopsTpl']->assign('insPage', $pages);
@@ -140,7 +140,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>';
@@ -172,9 +172,9 @@ 
 
 // Рейтинг
 if (xoops_getModuleOption('userat', 'instruction')) {
-    $xoopsTpl->assign('insUserat', true);
+	$xoopsTpl->assign('insUserat', true);
 } else {
-    $xoopsTpl->assign('insUserat', false);
+	$xoopsTpl->assign('insUserat', false);
 }
 
 // Мета теги

Spacing +4 added lines, -4 removed lines

@@ -59,7 +59,7 @@ 
 
 // Права на просмотр инструкции
 $categories = Xoopsmodules\instruction\Utility::getItemIds();
-if (!in_array($objInsinstr->getVar('cid'), $categories)) {
+if ( ! in_array($objInsinstr->getVar('cid'), $categories)) {
     redirect_header(XOOPS_URL . '/modules/' . $moduleDirName . '/', 3, _NOPERM);
     exit();
 }
@@ -95,7 +95,7 @@ 
                           . $moduleDirName
                           . '/admin/instr.php?op=editpage&pageid='
                           . $pages['pageid']
-                          . '"><img style="width:16px;" src="'. $pathIcon16 . '/edit.png" alt='
+                          . '"><img style="width:16px;" src="' . $pathIcon16 . '/edit.png" alt='
                           . _EDIT
                           . ' title='
                           . _EDIT
@@ -105,7 +105,7 @@ 
                           . $moduleDirName
                           . '/admin/instr.php?op=delpage&pageid='
                           . $pages['pageid']
-                          . '"><img style="width:16px;" src="'. $pathIcon16 . '/delete.png" alt='
+                          . '"><img style="width:16px;" src="' . $pathIcon16 . '/delete.png" alt='
                           . _DELETE
                           . ' title='
                           . _DELETE
@@ -114,7 +114,7 @@ 
     $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="'. $pathIcon16 . '/edit.png" alt=' . _EDIT . ' title=' . _EDIT . '></a>';
+        $pages['adminlink'] .= '<a href="' . XOOPS_URL . '/modules/' . $moduleDirName . '/submit.php?op=editpage&pageid=' . $pages['pageid'] . '"><img style="width:16px;" src="' . $pathIcon16 . '/edit.png" alt=' . _EDIT . ' title=' . _EDIT . '></a>';
     }
 
     $pages['adminlink'] .= '&nbsp;';