mambax7 / instruction

admin/admin_header.php

Unused Use Statements -2 removed lines

@@ -1,7 +1,5 @@
 <?php
 
-use Xoopsmodules\instruction;
-
 // Автор: andrey3761
 $moduleDirName = basename(dirname(__DIR__));
 require_once __DIR__ . '/../../../include/cp_header.php';

Indentation +6 added lines, -6 removed lines

@@ -12,19 +12,19 @@
 
 //if (is_object($GLOBALS['xoopsUser'])) {
 if ($GLOBALS['xoopsUser'] instanceof XoopsUser) {
-    if (!$helper->isUserAdmin()) {
-        $helper->redirect(XOOPS_URL . '/', 3, _NOPERM);
-    }
+	if (!$helper->isUserAdmin()) {
+		$helper->redirect(XOOPS_URL . '/', 3, _NOPERM);
+	}
 } else {
-    $helper->redirect(XOOPS_URL . '/user.php', 1, _NOPERM);
+	$helper->redirect(XOOPS_URL . '/user.php', 1, _NOPERM);
 }
 
 /** @var Xmf\Module\Admin $adminObject */
 $adminObject = \Xmf\Module\Admin::getInstance();
 
 if (!isset($GLOBALS['xoopsTpl']) || !($GLOBALS['xoopsTpl'] instanceof XoopsTpl)) {
-    require_once $GLOBALS['xoops']->path('class/template.php');
-    $xoopsTpl = new \XoopsTpl();
+	require_once $GLOBALS['xoops']->path('class/template.php');
+	$xoopsTpl = new \XoopsTpl();
 }
 
 $pathIcon16    = Xmf\Module\Admin::iconUrl('', 16);

Spacing +2 added lines, -2 removed lines

@@ -12,7 +12,7 @@ 
 
 //if (is_object($GLOBALS['xoopsUser'])) {
 if ($GLOBALS['xoopsUser'] instanceof XoopsUser) {
-    if (!$helper->isUserAdmin()) {
+    if ( ! $helper->isUserAdmin()) {
         $helper->redirect(XOOPS_URL . '/', 3, _NOPERM);
     }
 } else {
@@ -22,7 +22,7 @@ 
 /** @var Xmf\Module\Admin $adminObject */
 $adminObject = \Xmf\Module\Admin::getInstance();
 
-if (!isset($GLOBALS['xoopsTpl']) || !($GLOBALS['xoopsTpl'] instanceof XoopsTpl)) {
+if ( ! isset($GLOBALS['xoopsTpl']) || ! ($GLOBALS['xoopsTpl'] instanceof XoopsTpl)) {
     require_once $GLOBALS['xoops']->path('class/template.php');
     $xoopsTpl = new \XoopsTpl();
 }

class/InstructionHandler.php

Doc Comments +1 added lines, -1 removed lines

@@ -15,7 +15,7 @@
 class InstructionHandler extends \XoopsPersistableObjectHandler
 {
     /**
-     * @param null|mixed $db
+     * @param null|\XoopsDatabase $db
      */
     public function __construct(\XoopsDatabase $db = null)
     {

Indentation +42 added lines, -42 removed lines

@@ -14,51 +14,51 @@
  */
 class InstructionHandler extends \XoopsPersistableObjectHandler
 {
-    /**
-     * @param null|mixed $db
-     */
-    public function __construct(\XoopsDatabase $db = null)
-    {
-        parent::__construct($db, 'instruction_instr', Instruction::class, 'instrid', 'title');
-    }
+	/**
+	 * @param null|mixed $db
+	 */
+	public function __construct(\XoopsDatabase $db = null)
+	{
+		parent::__construct($db, 'instruction_instr', Instruction::class, 'instrid', 'title');
+	}
 
-    // Обновление даты обновления инструкций
+	// Обновление даты обновления инструкций
 
-    /**
-     * @param int  $instrid
-     * @param bool|int $time
-     * @return mixed
-     */
-    public function updateDateupdated($instrid = 0, $time = null)
-    {
-        // Если не передали время
-        $time = null === $time ? time() : (int)$time;
-        //
-        $sql = sprintf('UPDATE `%s` SET `dateupdated` = %u WHERE `instrid` = %u', $this->table, $time, (int)$instrid);
-        //
-        return $this->db->query($sql);
-    }
+	/**
+	 * @param int  $instrid
+	 * @param bool|int $time
+	 * @return mixed
+	 */
+	public function updateDateupdated($instrid = 0, $time = null)
+	{
+		// Если не передали время
+		$time = null === $time ? time() : (int)$time;
+		//
+		$sql = sprintf('UPDATE `%s` SET `dateupdated` = %u WHERE `instrid` = %u', $this->table, $time, (int)$instrid);
+		//
+		return $this->db->query($sql);
+	}
 
-    // Обновление числа страниц
+	// Обновление числа страниц
 
-    /**
-     * @param int $instrid
-     * @return mixed
-     */
-    public function updatePages($instrid = 0)
-    {
-        //        $pageHandler = xoops_getModuleHandler('page', 'instruction');
-        // Находим число активных страниц
-        $criteria = new \CriteriaCompo();
-        $criteria->add(new \Criteria('instrid', $instrid, '='));
-        $criteria->add(new \Criteria('status ', '0', '>'));
-        // Число страниц
-        $pages = $pageHandler->getCount($criteria);
-        unset($criteria);
+	/**
+	 * @param int $instrid
+	 * @return mixed
+	 */
+	public function updatePages($instrid = 0)
+	{
+		//        $pageHandler = xoops_getModuleHandler('page', 'instruction');
+		// Находим число активных страниц
+		$criteria = new \CriteriaCompo();
+		$criteria->add(new \Criteria('instrid', $instrid, '='));
+		$criteria->add(new \Criteria('status ', '0', '>'));
+		// Число страниц
+		$pages = $pageHandler->getCount($criteria);
+		unset($criteria);
 
-        // Сохраняем это число
-        $sql = sprintf('UPDATE `%s` SET `pages` = %u, `dateupdated` = %u WHERE `instrid` = %u', $this->table, $pages, time(), $instrid);
-        //
-        return $this->db->query($sql);
-    }
+		// Сохраняем это число
+		$sql = sprintf('UPDATE `%s` SET `pages` = %u, `dateupdated` = %u WHERE `instrid` = %u', $this->table, $pages, time(), $instrid);
+		//
+		return $this->db->query($sql);
+	}
 }

Spacing +2 added lines, -2 removed lines

@@ -32,9 +32,9 @@
     public function updateDateupdated($instrid = 0, $time = null)
     {
         // Если не передали время
-        $time = null === $time ? time() : (int)$time;
+        $time = null === $time ? time() : (int) $time;
         //
-        $sql = sprintf('UPDATE `%s` SET `dateupdated` = %u WHERE `instrid` = %u', $this->table, $time, (int)$instrid);
+        $sql = sprintf('UPDATE `%s` SET `dateupdated` = %u WHERE `instrid` = %u', $this->table, $time, (int) $instrid);
         //
         return $this->db->query($sql);
     }

class/Tree.php

Doc Comments +9 added lines, -9 removed lines

@@ -23,9 +23,9 @@ 
     //    }
 
     /**
-     * @param        $key
-     * @param        $ret
-     * @param        $prefix_orig
+     * @param        integer $key
+     * @param        string $ret
+     * @param        string $prefix_orig
      * @param        $objInsinstr
      * @param string $class
      * @param string $prefix_curr
@@ -140,9 +140,9 @@ 
     // ==================================
 
     /**
-     * @param        $key
-     * @param        $ret
-     * @param        $prefix_orig
+     * @param        integer $key
+     * @param        string $ret
+     * @param        string $prefix_orig
      * @param array  $cidinstrids
      * @param string $class
      * @param string $prefix_curr
@@ -239,8 +239,8 @@ 
     // ======================================
 
     /**
-     * @param       $key
-     * @param       $ret
+     * @param       integer $key
+     * @param       string $ret
      * @param int   $currpageid
      * @param array $lastpageids
      * @param int   $level
@@ -350,7 +350,7 @@ 
     // Находим предыдущую и следующую страницы.
     // Находим последнии страницы на каждом уровне.
     /**
-     * @param       $key
+     * @param       integer $key
      * @param int   $currpageid
      * @param array $prevpages
      * @param array $nextpages

Indentation +363 added lines, -363 removed lines

@@ -18,96 +18,96 @@ 
  */
 class Tree extends \XoopsObjectTree
 {
-    //    public function __construct()
-    //    {
-    //    }
-
-    /**
-     * @param        $key
-     * @param        $ret
-     * @param        $prefix_orig
-     * @param        $objInsinstr
-     * @param string $class
-     * @param string $prefix_curr
-     */
-    public function _makePagesAdminOptions($key, &$ret, $prefix_orig, $objInsinstr, $class = 'odd', $prefix_curr = '')
-    {
-        if ($key > 0) {
-
-            //
-            $class = ('even' === $class) ? 'odd' : 'even';
-            // ID инструкции ( Можно сделать статической )
-            $instrid = $objInsinstr->getVar('instrid');
-
-            // ID страницы
-            $pageid = $this->tree[$key]['obj']->getVar('pageid');
-            // Название страницы
-            $pagetitle = $this->tree[$key]['obj']->getVar('title');
-            // Вес
-            $pageweight = $this->tree[$key]['obj']->getVar('weight');
-            // Статус
-            $pagestatus = $this->tree[$key]['obj']->getVar('status');
-            // Тип страницы
-            $pagetype = $this->tree[$key]['obj']->getVar('type');
-
-            // Дочернии страницы
-            $page_childs = $this->getAllChild($pageid);
-            // Число дочерних страниц
-            $num_childs = count($page_childs);
-
-            // Действие - удаление
-            $act_del = ($num_childs > 0) ? '<img src="../assets/icons/no_delete_mini.png" alt="' . _AM_INSTR_NODELPAGE . '" title="' . _AM_INSTR_NODELPAGE . '" />' : '<a href="instr.php?op=delpage&pageid='
-                                                                                                                                                                      . $pageid
-                                                                                                                                                                      . '"><img src="../assets/icons/delete_mini.png" alt="'
-                                                                                                                                                                      . _AM_INSTRUCTION_DEL
-                                                                                                                                                                      . '" title="'
-                                                                                                                                                                      . _AM_INSTRUCTION_DEL
-                                                                                                                                                                      . '"></a>';
-            //
-            $page_link = '<a name="pageid_' . $pageid . '" ' . ($pagetype ? 'href="' . XOOPS_URL . '/modules/' . INST_DIRNAME . '/page.php?id=' . $pageid . '#pagetext"' : '') . '>' . $pagetitle . '</a>';
-
-            $ret .= '<tr class="' . $class . '">
+	//    public function __construct()
+	//    {
+	//    }
+
+	/**
+	 * @param        $key
+	 * @param        $ret
+	 * @param        $prefix_orig
+	 * @param        $objInsinstr
+	 * @param string $class
+	 * @param string $prefix_curr
+	 */
+	public function _makePagesAdminOptions($key, &$ret, $prefix_orig, $objInsinstr, $class = 'odd', $prefix_curr = '')
+	{
+		if ($key > 0) {
+
+			//
+			$class = ('even' === $class) ? 'odd' : 'even';
+			// ID инструкции ( Можно сделать статической )
+			$instrid = $objInsinstr->getVar('instrid');
+
+			// ID страницы
+			$pageid = $this->tree[$key]['obj']->getVar('pageid');
+			// Название страницы
+			$pagetitle = $this->tree[$key]['obj']->getVar('title');
+			// Вес
+			$pageweight = $this->tree[$key]['obj']->getVar('weight');
+			// Статус
+			$pagestatus = $this->tree[$key]['obj']->getVar('status');
+			// Тип страницы
+			$pagetype = $this->tree[$key]['obj']->getVar('type');
+
+			// Дочернии страницы
+			$page_childs = $this->getAllChild($pageid);
+			// Число дочерних страниц
+			$num_childs = count($page_childs);
+
+			// Действие - удаление
+			$act_del = ($num_childs > 0) ? '<img src="../assets/icons/no_delete_mini.png" alt="' . _AM_INSTR_NODELPAGE . '" title="' . _AM_INSTR_NODELPAGE . '" />' : '<a href="instr.php?op=delpage&pageid='
+																																									  . $pageid
+																																									  . '"><img src="../assets/icons/delete_mini.png" alt="'
+																																									  . _AM_INSTRUCTION_DEL
+																																									  . '" title="'
+																																									  . _AM_INSTRUCTION_DEL
+																																									  . '"></a>';
+			//
+			$page_link = '<a name="pageid_' . $pageid . '" ' . ($pagetype ? 'href="' . XOOPS_URL . '/modules/' . INST_DIRNAME . '/page.php?id=' . $pageid . '#pagetext"' : '') . '>' . $pagetitle . '</a>';
+
+			$ret .= '<tr class="' . $class . '">
       <td>' . $prefix_curr . ' ' . $page_link . '</td>
       <td align="center" width="50">
         <input type="text" name="weights[]" size="2" value="' . $pageweight . '" />
         <input type="hidden" name="pageids[]" value="' . $pageid . '" />
       </td>
       <td align="center" width="180">';
-            // Просмотре без кэша
-            $ret .= ' <a href="' . XOOPS_URL . '/modules/' . INST_DIRNAME . '/page.php?id=' . $pageid . '&amp;nocache=1"><img src="../assets/icons/no_cache.png" alt="' . _AM_INSTR_DISPLAY_NOCACHE . '" title="' . _AM_INSTR_DISPLAY_NOCACHE . '" /></a> ';
-            // Добавить подстраницу
-            $ret .= ' <a href="instr.php?op=editpage&instrid=' . $instrid . '&pid=' . $pageid . '"><img src="../assets/icons/add_mini.png" alt="' . _AM_INSTRUCTION_ADDSUBPAGE . '" title="' . _AM_INSTRUCTION_ADDSUBPAGE . '" /></a> ';
-
-            if ($pagestatus) {
-                $ret .= ' <img src="../assets/icons/lock_mini.png" alt="' . _AM_INSTRUCTION_LOCK . '" title="' . _AM_INSTRUCTION_LOCK . '"> ';
-            } else {
-                $ret .= ' <img src="../assets/icons/unlock_mini.png" alt="' . _AM_INSTRUCTION_UNLOCK . '" title="' . _AM_INSTRUCTION_UNLOCK . '"> ';
-            }
-
-            $ret .= ' <a href="instr.php?op=editpage&pageid=' . $pageid . '"><img src="../assets/icons/edit_mini.png" alt="' . _AM_INSTRUCTION_EDIT . '" title="' . _AM_INSTRUCTION_EDIT . '"></a> ' . $act_del . '
+			// Просмотре без кэша
+			$ret .= ' <a href="' . XOOPS_URL . '/modules/' . INST_DIRNAME . '/page.php?id=' . $pageid . '&amp;nocache=1"><img src="../assets/icons/no_cache.png" alt="' . _AM_INSTR_DISPLAY_NOCACHE . '" title="' . _AM_INSTR_DISPLAY_NOCACHE . '" /></a> ';
+			// Добавить подстраницу
+			$ret .= ' <a href="instr.php?op=editpage&instrid=' . $instrid . '&pid=' . $pageid . '"><img src="../assets/icons/add_mini.png" alt="' . _AM_INSTRUCTION_ADDSUBPAGE . '" title="' . _AM_INSTRUCTION_ADDSUBPAGE . '" /></a> ';
+
+			if ($pagestatus) {
+				$ret .= ' <img src="../assets/icons/lock_mini.png" alt="' . _AM_INSTRUCTION_LOCK . '" title="' . _AM_INSTRUCTION_LOCK . '"> ';
+			} else {
+				$ret .= ' <img src="../assets/icons/unlock_mini.png" alt="' . _AM_INSTRUCTION_UNLOCK . '" title="' . _AM_INSTRUCTION_UNLOCK . '"> ';
+			}
+
+			$ret .= ' <a href="instr.php?op=editpage&pageid=' . $pageid . '"><img src="../assets/icons/edit_mini.png" alt="' . _AM_INSTRUCTION_EDIT . '" title="' . _AM_INSTRUCTION_EDIT . '"></a> ' . $act_del . '
       </td>
     </tr>';
 
-            // Устанавливаем префикс
-            $prefix_curr .= $prefix_orig;
-        }
-
-        if (isset($this->tree[$key]['child']) && !empty($this->tree[$key]['child'])) {
-            foreach ($this->tree[$key]['child'] as $childkey) {
-                $this->_makePagesAdminOptions($childkey, $ret, $prefix_orig, $objInsinstr, $class, $prefix_curr);
-            }
-        }
-    }
-
-    /**
-     * @param        $objInsinstr
-     * @param string $prefix
-     * @param int    $key
-     * @return string
-     */
-    public function makePagesAdmin(&$objInsinstr, $prefix = '-', $key = 0)
-    {
-        $ret = '<form name="inspages" action="instr.php" method="post">
+			// Устанавливаем префикс
+			$prefix_curr .= $prefix_orig;
+		}
+
+		if (isset($this->tree[$key]['child']) && !empty($this->tree[$key]['child'])) {
+			foreach ($this->tree[$key]['child'] as $childkey) {
+				$this->_makePagesAdminOptions($childkey, $ret, $prefix_orig, $objInsinstr, $class, $prefix_curr);
+			}
+		}
+	}
+
+	/**
+	 * @param        $objInsinstr
+	 * @param string $prefix
+	 * @param int    $key
+	 * @return string
+	 */
+	public function makePagesAdmin(&$objInsinstr, $prefix = '-', $key = 0)
+	{
+		$ret = '<form name="inspages" action="instr.php" method="post">
   <table width="100%" cellspacing="1" class="outer">
     <tr>
       <th align="center" colspan="3">' . sprintf(_AM_INSTRUCTION_LISTPAGESININSTR, $objInsinstr->getVar('title')) . '</th>
@@ -118,10 +118,10 @@ 
       <td class="head" align="center" width="180">' . _AM_INSTRUCTION_ACTION . '</td>
     </tr>';
 
-        // Выводим все страницы
-        $this->_makePagesAdminOptions($key, $ret, $prefix, $objInsinstr);
+		// Выводим все страницы
+		$this->_makePagesAdminOptions($key, $ret, $prefix, $objInsinstr);
 
-        $ret .= '<tr class="foot">
+		$ret .= '<tr class="foot">
       <td><a href="instr.php?op=editpage&instrid=' . $objInsinstr->getVar('instrid') . '"><img src="../assets/icons/add_mini.png" alt="' . _AM_INSTRUCTION_ADDPAGE . '" title="' . _AM_INSTRUCTION_ADDPAGE . '"></a></td>
       <td colspan="2">
         <input type="hidden" name="instrid" value="' . $objInsinstr->getVar('instrid') . '" />
@@ -132,60 +132,60 @@ 
   </table>
 </form>';
 
-        return $ret;
-    }
-
-    // ==================================
-    // === Дерево категорий в админке ===
-    // ==================================
-
-    /**
-     * @param        $key
-     * @param        $ret
-     * @param        $prefix_orig
-     * @param array  $cidinstrids
-     * @param string $class
-     * @param string $prefix_curr
-     */
-    public function _makeCatsAdminOptions($key, &$ret, $prefix_orig, $cidinstrids = [], &$class = 'odd', $prefix_curr = '')
-    {
-        if ($key > 0) {
-
-            //
-            $class = ('even' === $class) ? 'odd' : 'even';
-
-            // ID категории
-            $catid = $this->tree[$key]['obj']->getVar('cid');
-            // Название категории
-            $cattitle = $this->tree[$key]['obj']->getVar('title');
-            // Вес
-            $catweight = $this->tree[$key]['obj']->getVar('weight');
-            // Статус
-            $pagestatus = $this->tree[$key]['obj']->getVar('status');
-
-            // Дочернии категории
-            $cat_childs = $this->getAllChild($catid);
-            // Число дочерних категорий
-            $num_childs = count($cat_childs);
-            // Число инструкций
-            $num_instrs = isset($cidinstrids[$catid]) ? $cidinstrids[$catid] : 0;
-
-            // Действие - удаление
-            $act_del = (($num_instrs > 0) || ($num_childs > 0)) ? '<img src="../assets/icons/no_delete_mini.png" alt="' . _AM_INSTR_NODELCAT . '" title="' . _AM_INSTR_NODELCAT . '" />' : '<a href="cat.php?op=delcat&cid='
-                                                                                                                                                                                           . $catid
-                                                                                                                                                                                           . '"><img src="../assets/icons/delete_mini.png" alt="'
-                                                                                                                                                                                           . _AM_INSTRUCTION_DEL
-                                                                                                                                                                                           . '" title="'
-                                                                                                                                                                                           . _AM_INSTRUCTION_DEL
-                                                                                                                                                                                           . '" /></a>';
-            // Действие - просмотр
-            $act_view = ($num_instrs > 0) ? '<a href="instr.php?cid=' . $catid . '"><img src="../assets/icons/view_mini.png" alt="' . _AM_INSTR_VIEWINSTR . '" title="' . _AM_INSTR_VIEWINSTR . '" /></a>' : '<img src="../assets/icons/no_view_mini.png" alt="'
-                                                                                                                                                                                                             . _AM_INSTR_NOVIEWINSTR
-                                                                                                                                                                                                             . '" title="'
-                                                                                                                                                                                                             . _AM_INSTR_NOVIEWINSTR
-                                                                                                                                                                                                             . '" />';
-
-            $ret .= '<tr class="' . $class . '">
+		return $ret;
+	}
+
+	// ==================================
+	// === Дерево категорий в админке ===
+	// ==================================
+
+	/**
+	 * @param        $key
+	 * @param        $ret
+	 * @param        $prefix_orig
+	 * @param array  $cidinstrids
+	 * @param string $class
+	 * @param string $prefix_curr
+	 */
+	public function _makeCatsAdminOptions($key, &$ret, $prefix_orig, $cidinstrids = [], &$class = 'odd', $prefix_curr = '')
+	{
+		if ($key > 0) {
+
+			//
+			$class = ('even' === $class) ? 'odd' : 'even';
+
+			// ID категории
+			$catid = $this->tree[$key]['obj']->getVar('cid');
+			// Название категории
+			$cattitle = $this->tree[$key]['obj']->getVar('title');
+			// Вес
+			$catweight = $this->tree[$key]['obj']->getVar('weight');
+			// Статус
+			$pagestatus = $this->tree[$key]['obj']->getVar('status');
+
+			// Дочернии категории
+			$cat_childs = $this->getAllChild($catid);
+			// Число дочерних категорий
+			$num_childs = count($cat_childs);
+			// Число инструкций
+			$num_instrs = isset($cidinstrids[$catid]) ? $cidinstrids[$catid] : 0;
+
+			// Действие - удаление
+			$act_del = (($num_instrs > 0) || ($num_childs > 0)) ? '<img src="../assets/icons/no_delete_mini.png" alt="' . _AM_INSTR_NODELCAT . '" title="' . _AM_INSTR_NODELCAT . '" />' : '<a href="cat.php?op=delcat&cid='
+																																														   . $catid
+																																														   . '"><img src="../assets/icons/delete_mini.png" alt="'
+																																														   . _AM_INSTRUCTION_DEL
+																																														   . '" title="'
+																																														   . _AM_INSTRUCTION_DEL
+																																														   . '" /></a>';
+			// Действие - просмотр
+			$act_view = ($num_instrs > 0) ? '<a href="instr.php?cid=' . $catid . '"><img src="../assets/icons/view_mini.png" alt="' . _AM_INSTR_VIEWINSTR . '" title="' . _AM_INSTR_VIEWINSTR . '" /></a>' : '<img src="../assets/icons/no_view_mini.png" alt="'
+																																																			 . _AM_INSTR_NOVIEWINSTR
+																																																			 . '" title="'
+																																																			 . _AM_INSTR_NOVIEWINSTR
+																																																			 . '" />';
+
+			$ret .= '<tr class="' . $class . '">
       <td>' . $prefix_curr . ' <a href="' . XOOPS_URL . '/modules/' . INST_DIRNAME . '/index.php?cid=' . $catid . '">' . $cattitle . '</a></td>
       <td align="center" width="50">' . $catweight . '</td>
       <td align="center" width="100">' . $num_instrs . '</td>
@@ -196,26 +196,26 @@ 
       </td>
     </tr>';
 
-            // Устанавливаем префикс
-            $prefix_curr .= $prefix_orig;
-        }
-
-        if (isset($this->tree[$key]['child']) && !empty($this->tree[$key]['child'])) {
-            foreach ($this->tree[$key]['child'] as $childkey) {
-                $this->_makeCatsAdminOptions($childkey, $ret, $prefix_orig, $cidinstrids, $class, $prefix_curr);
-            }
-        }
-    }
-
-    /**
-     * @param string $prefix
-     * @param array  $cidinstrids
-     * @param int    $key
-     * @return string
-     */
-    public function makeCatsAdmin($prefix = '-', $cidinstrids = [], $key = 0)
-    {
-        $ret = '<table width="100%" cellspacing="1" class="outer">
+			// Устанавливаем префикс
+			$prefix_curr .= $prefix_orig;
+		}
+
+		if (isset($this->tree[$key]['child']) && !empty($this->tree[$key]['child'])) {
+			foreach ($this->tree[$key]['child'] as $childkey) {
+				$this->_makeCatsAdminOptions($childkey, $ret, $prefix_orig, $cidinstrids, $class, $prefix_curr);
+			}
+		}
+	}
+
+	/**
+	 * @param string $prefix
+	 * @param array  $cidinstrids
+	 * @param int    $key
+	 * @return string
+	 */
+	public function makeCatsAdmin($prefix = '-', $cidinstrids = [], $key = 0)
+	{
+		$ret = '<table width="100%" cellspacing="1" class="outer">
     <tr>
       <th align="center" colspan="4">' . _AM_INSTR_LISTALLCATS . '</th>
     </tr>
@@ -226,215 +226,215 @@ 
       <td class="head" align="center" width="150">' . _AM_INSTRUCTION_ACTION . '</td>
     </tr>';
 
-        // Выводим все страницы
-        $this->_makeCatsAdminOptions($key, $ret, $prefix, $cidinstrids);
-
-        $ret .= '</table>';
-
-        return $ret;
-    }
-
-    // ======================================
-    // Список страниц на стороне пользователя
-    // ======================================
-
-    /**
-     * @param       $key
-     * @param       $ret
-     * @param int   $currpageid
-     * @param array $lastpageids
-     * @param int   $level
-     */
-    public function _makePagesUserTree($key, &$ret, $currpageid = 0, &$lastpageids = [], $level = 0)
-    {
-
-        // Сохраняем значение предыдущей страницы
-        //static $stat_prevpages;
-
-        if ($key > 0) {
-
-            // ID страницы
-            $pageid = $this->tree[$key]['obj']->getVar('pageid');
-            // Название страницы
-            $pagetitle = $this->tree[$key]['obj']->getVar('title');
-            // Тип страницы
-            $pagetype = $this->tree[$key]['obj']->getVar('type');
-
-            // Дочернии категории
-            $page_childs = $this->getAllChild($pageid);
-            // Число дочерних страниц
-            $num_childs = count($page_childs);
-
-            // Генерируем класс
-            // InstrTreeNode InstrTreeIsRoot InstrTreeExpandClosed InstrTreeIsLast
-            $class = [];
-            // Данный класс должен быть у любого узла
-            $class[] = 'InstrTreeNode';
-            // Если узел нулевого уровня, добавляем InstrTreeIsRoot
-            if (0 === $level) {
-                $class[] = 'InstrTreeIsRoot';
-            }
-            // Тип узла InstrTreeExpandClosed|InstrTreeExpandLeaf
-            // Если у узла нет потомков - InstrTreeExpandLeaf
-            if (0 == $num_childs) {
-                $class[] = 'InstrTreeExpandLeaf';
-                // Если у искомого элемента есть потомки - открываем список
-            } elseif ($currpageid == $pageid) {
-                $class[] = 'InstrTreeExpandOpen';
-                // Если искомый элемент есть в потомках текущего, то ставим класс InstrTreeExpandOpen
-            } elseif (array_key_exists($currpageid, $page_childs)) {
-                $class[] = 'InstrTreeExpandOpen';
-                //
-            } else {
-                $class[] = 'InstrTreeExpandClosed';
-            }
-
-            // Данный класс нужно добавлять последнему узлу в каждом уровне
-
-            if (isset($lastpageids[$level]) && ($pageid == $lastpageids[$level])) {
-                $class[] = 'InstrTreeIsLast';
-            }
-
-            //$class[] = 'InstrTreeIsLast';
-
-            // Test
-            //$ret .= '<div id="' . $pageid . '">';
-
-            // Создаём запись
-            $ret .= '<li class="' . implode(' ', $class) . '">';
-            //
-            $ret .= '<div class="InstrTreeExpand"></div>';
-            //
-            $ret .= '<div class="InstrTreeContent">';
-
-            // Если это лист дерева
-            if (0 == $pagetype) {
-                $ret .= '<span class="InstrTreeEmptyPage">' . $pagetitle . '</span>';
-                //
-            } elseif ($currpageid == $pageid) {
-                $ret .= $pagetitle;
-                //
-            } else {
-                $ret .= '<a href="' . XOOPS_URL . '/modules/' . INST_DIRNAME . '/page.php?id=' . $pageid . '#pagetext">' . $pagetitle . '</a>';
-            }
-
-            $ret .= '</div>';
-
-            // Если есть потомки
-            if ($num_childs > 0) {
-                $ret .= '<ul class="InstrTreeContainer">';
-            }
-
-            // Инкримент уровня
-            $level++;
-        }
-
-        // Рекурсия
-        if (isset($this->tree[$key]['child']) && !empty($this->tree[$key]['child'])) {
-            foreach ($this->tree[$key]['child'] as $childkey) {
-                $this->_makePagesUserTree($childkey, $ret, $currpageid, $lastpageids, $level);
-            }
-        }
-
-        // Test
-        if ($key > 0) {
-            // Если есть потомки
-            if ($num_childs > 0) {
-                $ret .= '</ul>';
-            }
-            // Конец текущей записи
-            $ret .= '</li>';
-        }
-    }
-
-    // Находим предыдущую и следующую страницы.
-    // Находим последнии страницы на каждом уровне.
-    /**
-     * @param       $key
-     * @param int   $currpageid
-     * @param array $prevpages
-     * @param array $nextpages
-     * @param array $lastpageids
-     * @param int   $level
-     */
-    public function _makePagesUserCalc($key, $currpageid = 0, &$prevpages = [], &$nextpages = [], &$lastpageids = [], $level = 0)
-    {
-
-        // Сохраняем значение предыдущей страницы
-        static $stat_prevpages;
-
-        if ($key > 0) {
-            // ID страницы
-            $pageid = $this->tree[$key]['obj']->getVar('pageid');
-            // Название страницы
-            $pagetitle = $this->tree[$key]['obj']->getVar('title');
-            // Тип страницы
-            $pagetype = $this->tree[$key]['obj']->getVar('type');
-
-            // Если мы передали ID текущей страницы, то находить предыдудую и следующую страницы
-            // Не находить предыдущие и следующие для "Пустой страницы"
-            if ($currpageid && $pagetype) {
-                // Если элемент равен текущей странице
-                if (isset($stat_prevpages) && ($currpageid == $pageid)) {
-                    // Забиваем массив предыдущей страницы
-                    $prevpages['pageid'] = $stat_prevpages['pageid'];
-                    $prevpages['title']  = $stat_prevpages['title'];
-
-                    // Если предыдущий равен текущей странице
-                } elseif (isset($stat_prevpages) && ($currpageid == $stat_prevpages['pageid'])) {
-                    // Забиваем массив следующей страницы
-                    $nextpages['pageid'] = $pageid;
-                    $nextpages['title']  = $pagetitle;
-                }
-                // Заносим текущие данные в массив предыдущей страницы
-                $stat_prevpages['pageid'] = $pageid;
-                $stat_prevpages['title']  = $pagetitle;
-            }
-
-            // Заносим текущую страницу в массив "последних страний"
-            $lastpageids[$level] = $pageid;
-
-            // Инкримент уровня
-            $level++;
-        }
-
-        // Рекурсия
-        if (isset($this->tree[$key]['child']) && !empty($this->tree[$key]['child'])) {
-            foreach ($this->tree[$key]['child'] as $childkey) {
-                $this->_makePagesUserCalc($childkey, $currpageid, $prevpages, $nextpages, $lastpageids, $level);
-            }
-        }
-    }
-
-    //
-
-    /**
-     * @param int   $currpageid
-     * @param array $prevpages
-     * @param array $nextpages
-     * @param int   $key
-     * @return string
-     */
-    public function makePagesUser($currpageid = 0, &$prevpages = [], &$nextpages = [], $key = 0)
-    {
-
-        // Массив последней страницы на каждом уровне
-        // level => pageid
-        $lastpageids = [];
-
-        // Расчёт
-        $this->_makePagesUserCalc($key, $currpageid, $prevpages, $nextpages, $lastpageids);
-
-        $ret = '<div onclick="instr_tree_toggle(arguments[0])">
+		// Выводим все страницы
+		$this->_makeCatsAdminOptions($key, $ret, $prefix, $cidinstrids);
+
+		$ret .= '</table>';
+
+		return $ret;
+	}
+
+	// ======================================
+	// Список страниц на стороне пользователя
+	// ======================================
+
+	/**
+	 * @param       $key
+	 * @param       $ret
+	 * @param int   $currpageid
+	 * @param array $lastpageids
+	 * @param int   $level
+	 */
+	public function _makePagesUserTree($key, &$ret, $currpageid = 0, &$lastpageids = [], $level = 0)
+	{
+
+		// Сохраняем значение предыдущей страницы
+		//static $stat_prevpages;
+
+		if ($key > 0) {
+
+			// ID страницы
+			$pageid = $this->tree[$key]['obj']->getVar('pageid');
+			// Название страницы
+			$pagetitle = $this->tree[$key]['obj']->getVar('title');
+			// Тип страницы
+			$pagetype = $this->tree[$key]['obj']->getVar('type');
+
+			// Дочернии категории
+			$page_childs = $this->getAllChild($pageid);
+			// Число дочерних страниц
+			$num_childs = count($page_childs);
+
+			// Генерируем класс
+			// InstrTreeNode InstrTreeIsRoot InstrTreeExpandClosed InstrTreeIsLast
+			$class = [];
+			// Данный класс должен быть у любого узла
+			$class[] = 'InstrTreeNode';
+			// Если узел нулевого уровня, добавляем InstrTreeIsRoot
+			if (0 === $level) {
+				$class[] = 'InstrTreeIsRoot';
+			}
+			// Тип узла InstrTreeExpandClosed|InstrTreeExpandLeaf
+			// Если у узла нет потомков - InstrTreeExpandLeaf
+			if (0 == $num_childs) {
+				$class[] = 'InstrTreeExpandLeaf';
+				// Если у искомого элемента есть потомки - открываем список
+			} elseif ($currpageid == $pageid) {
+				$class[] = 'InstrTreeExpandOpen';
+				// Если искомый элемент есть в потомках текущего, то ставим класс InstrTreeExpandOpen
+			} elseif (array_key_exists($currpageid, $page_childs)) {
+				$class[] = 'InstrTreeExpandOpen';
+				//
+			} else {
+				$class[] = 'InstrTreeExpandClosed';
+			}
+
+			// Данный класс нужно добавлять последнему узлу в каждом уровне
+
+			if (isset($lastpageids[$level]) && ($pageid == $lastpageids[$level])) {
+				$class[] = 'InstrTreeIsLast';
+			}
+
+			//$class[] = 'InstrTreeIsLast';
+
+			// Test
+			//$ret .= '<div id="' . $pageid . '">';
+
+			// Создаём запись
+			$ret .= '<li class="' . implode(' ', $class) . '">';
+			//
+			$ret .= '<div class="InstrTreeExpand"></div>';
+			//
+			$ret .= '<div class="InstrTreeContent">';
+
+			// Если это лист дерева
+			if (0 == $pagetype) {
+				$ret .= '<span class="InstrTreeEmptyPage">' . $pagetitle . '</span>';
+				//
+			} elseif ($currpageid == $pageid) {
+				$ret .= $pagetitle;
+				//
+			} else {
+				$ret .= '<a href="' . XOOPS_URL . '/modules/' . INST_DIRNAME . '/page.php?id=' . $pageid . '#pagetext">' . $pagetitle . '</a>';
+			}
+
+			$ret .= '</div>';
+
+			// Если есть потомки
+			if ($num_childs > 0) {
+				$ret .= '<ul class="InstrTreeContainer">';
+			}
+
+			// Инкримент уровня
+			$level++;
+		}
+
+		// Рекурсия
+		if (isset($this->tree[$key]['child']) && !empty($this->tree[$key]['child'])) {
+			foreach ($this->tree[$key]['child'] as $childkey) {
+				$this->_makePagesUserTree($childkey, $ret, $currpageid, $lastpageids, $level);
+			}
+		}
+
+		// Test
+		if ($key > 0) {
+			// Если есть потомки
+			if ($num_childs > 0) {
+				$ret .= '</ul>';
+			}
+			// Конец текущей записи
+			$ret .= '</li>';
+		}
+	}
+
+	// Находим предыдущую и следующую страницы.
+	// Находим последнии страницы на каждом уровне.
+	/**
+	 * @param       $key
+	 * @param int   $currpageid
+	 * @param array $prevpages
+	 * @param array $nextpages
+	 * @param array $lastpageids
+	 * @param int   $level
+	 */
+	public function _makePagesUserCalc($key, $currpageid = 0, &$prevpages = [], &$nextpages = [], &$lastpageids = [], $level = 0)
+	{
+
+		// Сохраняем значение предыдущей страницы
+		static $stat_prevpages;
+
+		if ($key > 0) {
+			// ID страницы
+			$pageid = $this->tree[$key]['obj']->getVar('pageid');
+			// Название страницы
+			$pagetitle = $this->tree[$key]['obj']->getVar('title');
+			// Тип страницы
+			$pagetype = $this->tree[$key]['obj']->getVar('type');
+
+			// Если мы передали ID текущей страницы, то находить предыдудую и следующую страницы
+			// Не находить предыдущие и следующие для "Пустой страницы"
+			if ($currpageid && $pagetype) {
+				// Если элемент равен текущей странице
+				if (isset($stat_prevpages) && ($currpageid == $pageid)) {
+					// Забиваем массив предыдущей страницы
+					$prevpages['pageid'] = $stat_prevpages['pageid'];
+					$prevpages['title']  = $stat_prevpages['title'];
+
+					// Если предыдущий равен текущей странице
+				} elseif (isset($stat_prevpages) && ($currpageid == $stat_prevpages['pageid'])) {
+					// Забиваем массив следующей страницы
+					$nextpages['pageid'] = $pageid;
+					$nextpages['title']  = $pagetitle;
+				}
+				// Заносим текущие данные в массив предыдущей страницы
+				$stat_prevpages['pageid'] = $pageid;
+				$stat_prevpages['title']  = $pagetitle;
+			}
+
+			// Заносим текущую страницу в массив "последних страний"
+			$lastpageids[$level] = $pageid;
+
+			// Инкримент уровня
+			$level++;
+		}
+
+		// Рекурсия
+		if (isset($this->tree[$key]['child']) && !empty($this->tree[$key]['child'])) {
+			foreach ($this->tree[$key]['child'] as $childkey) {
+				$this->_makePagesUserCalc($childkey, $currpageid, $prevpages, $nextpages, $lastpageids, $level);
+			}
+		}
+	}
+
+	//
+
+	/**
+	 * @param int   $currpageid
+	 * @param array $prevpages
+	 * @param array $nextpages
+	 * @param int   $key
+	 * @return string
+	 */
+	public function makePagesUser($currpageid = 0, &$prevpages = [], &$nextpages = [], $key = 0)
+	{
+
+		// Массив последней страницы на каждом уровне
+		// level => pageid
+		$lastpageids = [];
+
+		// Расчёт
+		$this->_makePagesUserCalc($key, $currpageid, $prevpages, $nextpages, $lastpageids);
+
+		$ret = '<div onclick="instr_tree_toggle(arguments[0])">
 <div>' . _MD_INSTRUCTION_LISTPAGES . '</div>
 <div><ul class="InstrTreeContainer">';
 
-        // Генерируем дерево
-        $this->_makePagesUserTree($key, $ret, $currpageid, $lastpageids);
+		// Генерируем дерево
+		$this->_makePagesUserTree($key, $ret, $currpageid, $lastpageids);
 
-        $ret .= '</ul>
+		$ret .= '</ul>
 </div>';
 
-        return $ret;
-    }
+		return $ret;
+	}
 }

class/Utility.php

Doc Comments +2 added lines, -2 removed lines

@@ -46,8 +46,8 @@
     // Редактор
 
     /**
-     * @param        $caption
-     * @param        $name
+     * @param        string $caption
+     * @param        string $name
      * @param string $value
      * @return bool|\XoopsFormEditor
      */

Unused Use Statements -1 removed lines

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

Indentation +70 added lines, -70 removed lines

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

blocks/instr_lastpage.php

Indentation +55 added lines, -55 removed lines

@@ -12,63 +12,63 @@ 
 function b_instr_lastpage_show($options = [])
 {
 
-    // Подключаем функции
-    $moduleDirName = basename(dirname(__DIR__));
-    include_once $GLOBALS['xoops']->path('modules/' . $moduleDirName . '/class/utility.php');
-    //
-    $myts = MyTextSanitizer::getInstance();
-    //
-    //mb    $instructionHandler = xoops_getModuleHandler('instruction', 'instruction');
-    //mb    $pageHandler  = xoops_getModuleHandler('page', 'instruction');
+	// Подключаем функции
+	$moduleDirName = basename(dirname(__DIR__));
+	include_once $GLOBALS['xoops']->path('modules/' . $moduleDirName . '/class/utility.php');
+	//
+	$myts = MyTextSanitizer::getInstance();
+	//
+	//mb    $instructionHandler = xoops_getModuleHandler('instruction', 'instruction');
+	//mb    $pageHandler  = xoops_getModuleHandler('page', 'instruction');
 
-    $db                 = \XoopsDatabaseFactory::getDatabase();
-    $instructionHandler = new \Xoopsmodules\instruction\InstructionHandler($db);
-    $pageHandler        = new \Xoopsmodules\instruction\PageHandler($db);
+	$db                 = \XoopsDatabaseFactory::getDatabase();
+	$instructionHandler = new \Xoopsmodules\instruction\InstructionHandler($db);
+	$pageHandler        = new \Xoopsmodules\instruction\PageHandler($db);
 
-    // Добавляем стили
-    //global $xoTheme;
-    //$xoTheme->addStylesheet( XOOPS_URL . '/modules/instruction/css/blocks.css' );
+	// Добавляем стили
+	//global $xoTheme;
+	//$xoTheme->addStylesheet( XOOPS_URL . '/modules/instruction/css/blocks.css' );
 
-    // Опции
-    // Количество страниц
-    $limit = $options[0];
-    // Количество символов
-    $numchars = $options[1];
+	// Опции
+	// Количество страниц
+	$limit = $options[0];
+	// Количество символов
+	$numchars = $options[1];
 
-    // Права на просмотр
-    $cat_view = Xoopsmodules\instruction\Utility::getItemIds();
-    // Массив выходных данных
-    $block = [];
+	// Права на просмотр
+	$cat_view = Xoopsmodules\instruction\Utility::getItemIds();
+	// Массив выходных данных
+	$block = [];
 
-    // Если есть категории для прасмотра
-    if (is_array($cat_view) && count($cat_view) > 0) {
+	// Если есть категории для прасмотра
+	if (is_array($cat_view) && count($cat_view) > 0) {
 
-        // Находим последние страницы
-        $sql = "SELECT p.pageid, p.instrid, p.title, p.dateupdated, i.title, i.cid FROM {$pageHandler->table} p, {$instructionHandler->table} i WHERE p.instrid = i.instrid AND i.cid IN (" . implode(', ', $cat_view) . ') AND p.status > 0 AND i.status > 0 ORDER BY p.dateupdated DESC';
-        // Лимит запроса
-        $result = $GLOBALS['xoopsDB']->query($sql, $limit);
-        // Перебираем все значения
-        $i = 0;
-        while (list($pageid, $instrid, $ptitle, $dateupdated, $ititle, $cid) = $GLOBALS['xoopsDB']->fetchRow($result)) {
-            // ID страницы
-            $block[$i]['pageid'] = $pageid;
-            // ID инструкции
-            $block[$i]['instrid'] = $instrid;
-            // Название страницы
-            $block[$i]['ptitle'] = $myts->htmlSpecialChars($ptitle);
-            // Название инструкции
-            $block[$i]['ititle'] = $myts->htmlSpecialChars($ititle);
-            // Дата обновления страницы
-            $block[$i]['dateupdated'] = formatTimeStamp($dateupdated, 's');
-            // Категория инстркции
-            $block[$i]['cid'] = $cid;
-            // Инкримент
-            $i++;
-        }
-    }
+		// Находим последние страницы
+		$sql = "SELECT p.pageid, p.instrid, p.title, p.dateupdated, i.title, i.cid FROM {$pageHandler->table} p, {$instructionHandler->table} i WHERE p.instrid = i.instrid AND i.cid IN (" . implode(', ', $cat_view) . ') AND p.status > 0 AND i.status > 0 ORDER BY p.dateupdated DESC';
+		// Лимит запроса
+		$result = $GLOBALS['xoopsDB']->query($sql, $limit);
+		// Перебираем все значения
+		$i = 0;
+		while (list($pageid, $instrid, $ptitle, $dateupdated, $ititle, $cid) = $GLOBALS['xoopsDB']->fetchRow($result)) {
+			// ID страницы
+			$block[$i]['pageid'] = $pageid;
+			// ID инструкции
+			$block[$i]['instrid'] = $instrid;
+			// Название страницы
+			$block[$i]['ptitle'] = $myts->htmlSpecialChars($ptitle);
+			// Название инструкции
+			$block[$i]['ititle'] = $myts->htmlSpecialChars($ititle);
+			// Дата обновления страницы
+			$block[$i]['dateupdated'] = formatTimeStamp($dateupdated, 's');
+			// Категория инстркции
+			$block[$i]['cid'] = $cid;
+			// Инкримент
+			$i++;
+		}
+	}
 
-    // Возвращаем массив
-    return $block;
+	// Возвращаем массив
+	return $block;
 }
 
 // Редактирование последних страниц
@@ -78,10 +78,10 @@ 
  */
 function b_instr_lastpage_edit($options = [])
 {
-    $form = '';
-    $form .= _MB_INSTR_DISPLAYPAGESC . ' <input name="options[0]" size="5" maxlength="255" value="' . $options[0] . '" type="text" /><br>' . "\n";
-    $form .= _MB_INSTR_NUMCHARSC . ' <input name="options[1]" size="5" maxlength="255" value="' . $options[1] . '" type="text" /><br>' . "\n";
+	$form = '';
+	$form .= _MB_INSTR_DISPLAYPAGESC . ' <input name="options[0]" size="5" maxlength="255" value="' . $options[0] . '" type="text" /><br>' . "\n";
+	$form .= _MB_INSTR_NUMCHARSC . ' <input name="options[1]" size="5" maxlength="255" value="' . $options[1] . '" type="text" /><br>' . "\n";
 
-    // Возвращаем форму
-    return $form;
+	// Возвращаем форму
+	return $form;
 }

blocks/instr_lastinstr.php

Indentation +53 added lines, -53 removed lines

@@ -13,59 +13,59 @@ 
 function b_instr_lastinstr_show($options = [])
 {
 
-    // Подключаем функции
-    //    $moduleDirName = dirname(__DIR__);
-    $moduleDirName = basename(dirname(__DIR__));
-    //    include_once $GLOBALS['xoops']->path('modules/' . $moduleDirName . '/class/utility.php');
-    include_once $GLOBALS['xoops']->path('modules/' . $moduleDirName . '/include/common.php');
-    //
-    $myts = MyTextSanitizer::getInstance();
-    //
-    //mb    $instructionHandler = xoops_getModuleHandler('instruction', 'instruction');
-    $db                 = \XoopsDatabaseFactory::getDatabase();
-    $instructionHandler = new \Xoopsmodules\instruction\InstructionHandler($db);
+	// Подключаем функции
+	//    $moduleDirName = dirname(__DIR__);
+	$moduleDirName = basename(dirname(__DIR__));
+	//    include_once $GLOBALS['xoops']->path('modules/' . $moduleDirName . '/class/utility.php');
+	include_once $GLOBALS['xoops']->path('modules/' . $moduleDirName . '/include/common.php');
+	//
+	$myts = MyTextSanitizer::getInstance();
+	//
+	//mb    $instructionHandler = xoops_getModuleHandler('instruction', 'instruction');
+	$db                 = \XoopsDatabaseFactory::getDatabase();
+	$instructionHandler = new \Xoopsmodules\instruction\InstructionHandler($db);
 
-    // Добавляем стили
-    //global $xoTheme;
-    //$xoTheme->addStylesheet( XOOPS_URL . '/modules/instruction/css/blocks.css' );
+	// Добавляем стили
+	//global $xoTheme;
+	//$xoTheme->addStylesheet( XOOPS_URL . '/modules/instruction/css/blocks.css' );
 
-    // Опции
-    // Количество страниц
-    $limit = $options[0];
-    // Количество символов
-    $numchars = $options[1];
+	// Опции
+	// Количество страниц
+	$limit = $options[0];
+	// Количество символов
+	$numchars = $options[1];
 
-    // Права на просмотр
-    $cat_view = Xoopsmodules\instruction\Utility::getItemIds();
-    // Массив выходных данных
-    $block = [];
+	// Права на просмотр
+	$cat_view = Xoopsmodules\instruction\Utility::getItemIds();
+	// Массив выходных данных
+	$block = [];
 
-    // Если есть категории для прасмотра
-    if (is_array($cat_view) && count($cat_view) > 0) {
+	// Если есть категории для прасмотра
+	if (is_array($cat_view) && count($cat_view) > 0) {
 
-        // Находим последние инструкции
-        $sql = "SELECT `instrid`, `cid`, `title`, `pages`, `dateupdated` FROM {$instructionHandler->table} WHERE `cid` IN (" . implode(', ', $cat_view) . ') AND `status` > 0 ORDER BY `dateupdated` DESC';
-        // Лимит запроса
-        $result = $GLOBALS['xoopsDB']->query($sql, $limit);
-        // Перебираем все значения
-        $i = 0;
-        while (list($instrid, $cid, $ititle, $pages, $dateupdated) = $GLOBALS['xoopsDB']->fetchRow($result)) {
-            // ID инструкции
-            $block[$i]['instrid'] = $instrid;
-            // ID категории
-            $block[$i]['cid'] = $cid;
-            // Название инструкции
-            $block[$i]['ititle'] = $myts->htmlSpecialChars($ititle);
-            // Число страниц
-            $block[$i]['pages'] = $pages;
-            // Дата обновления инструкции
-            $block[$i]['dateupdated'] = formatTimeStamp($dateupdated, 's');
-            // Инкримент
-            $i++;
-        }
-    }
-    // Возвращаем массив
-    return $block;
+		// Находим последние инструкции
+		$sql = "SELECT `instrid`, `cid`, `title`, `pages`, `dateupdated` FROM {$instructionHandler->table} WHERE `cid` IN (" . implode(', ', $cat_view) . ') AND `status` > 0 ORDER BY `dateupdated` DESC';
+		// Лимит запроса
+		$result = $GLOBALS['xoopsDB']->query($sql, $limit);
+		// Перебираем все значения
+		$i = 0;
+		while (list($instrid, $cid, $ititle, $pages, $dateupdated) = $GLOBALS['xoopsDB']->fetchRow($result)) {
+			// ID инструкции
+			$block[$i]['instrid'] = $instrid;
+			// ID категории
+			$block[$i]['cid'] = $cid;
+			// Название инструкции
+			$block[$i]['ititle'] = $myts->htmlSpecialChars($ititle);
+			// Число страниц
+			$block[$i]['pages'] = $pages;
+			// Дата обновления инструкции
+			$block[$i]['dateupdated'] = formatTimeStamp($dateupdated, 's');
+			// Инкримент
+			$i++;
+		}
+	}
+	// Возвращаем массив
+	return $block;
 }
 
 // Редактирование последних инструкций
@@ -75,10 +75,10 @@ 
  */
 function b_instr_lastinstr_edit($options = [])
 {
-    $form = '';
-    $form .= _MB_INSTR_DISPLAYINSTRC . ' <input name="options[0]" size="5" maxlength="255" value="' . $options[0] . '" type="text" /><br>' . "\n";
-    $form .= _MB_INSTR_NUMCHARSC . ' <input name="options[1]" size="5" maxlength="255" value="' . $options[1] . '" type="text" /><br>' . "\n";
+	$form = '';
+	$form .= _MB_INSTR_DISPLAYINSTRC . ' <input name="options[0]" size="5" maxlength="255" value="' . $options[0] . '" type="text" /><br>' . "\n";
+	$form .= _MB_INSTR_NUMCHARSC . ' <input name="options[1]" size="5" maxlength="255" value="' . $options[1] . '" type="text" /><br>' . "\n";
 
-    // Возвращаем форму
-    return $form;
+	// Возвращаем форму
+	return $form;
 }

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 (is_object($GLOBALS['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="./assets/icons/edit_mini.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="./assets/icons/delete_mini.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="./assets/icons/edit_mini.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="./assets/icons/delete_mini.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 +2 added lines, -2 removed lines

@@ -12,7 +12,7 @@ 
 //$categoryHandler   = xoops_getModuleHandler('category', 'instruction');
 //$pageHandler  = xoops_getModuleHandler('page', 'instruction');
 
-$instrid = isset($_GET['id']) ? (int)$_GET['id'] : 0;
+$instrid = isset($_GET['id']) ? (int) $_GET['id'] : 0;
 
 // Существует ли такая инструкция
 $criteria = new \CriteriaCompo();
@@ -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();
 }

xoops_version.php

Indentation +167 added lines, -167 removed lines

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

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', $_POST['title']);
-        $objInstructioncat->setVar('description', $_POST['description']);
-        $objInstructioncat->setVar('weight', $weight);
-        $objInstructioncat->setVar('metakeywords', $_POST['metakeywords']);
-        $objInstructioncat->setVar('metadescription', $_POST['metadescription']);
-
-        // Проверка веса
-        if (0 == $weight) {
-            $err         = true;
-            $message_err .= _AM_INSTRUCTION_ERR_WEIGHT . '<br>';
-        }
-        // Проверка категорий
-        if ($cid && ($cid == $pid)) {
-            $err         = true;
-            $message_err .= _AM_INSTRUCTION_ERR_PCAT . '<br>';
-        }
-        // Если были ошибки
-        if (true === $err) {
-            xoops_cp_header();
-            // Навигация
-            $adminObject->displayNavigation(basename(__FILE__));
-
-            $message_err = '<div class="errorMsg" style="text-align: left;">' . $message_err . '</div>';
-            // Выводим ошибки в шаблон
-            $GLOBALS['xoopsTpl']->assign('insErrorMsg', $message_err);
-            // Если небыло ошибок
-        } else {
-            // Вставляем данные в БД
-            if ($categoryHandler->insert($objInstructioncat)) {
-
-                // ID категории. Если редактируем - то не изменяется. Если создаём новую - то получаем ID созданной записи.
-                $new_cid = $cid ?: $objInstructioncat->get_new_enreg();
-
-                // ===============
-                // ==== Права ====
-                // ===============
-
-                $gpermHandler = xoops_getHandler('groupperm');
-
-                // Если мы редактируем категорию, то старые права нужно удалить
-                if ($cid) {
-                    // Права на просмотр
-                    $criteria = new\ CriteriaCompo();
-                    $criteria->add(new \Criteria('gperm_itemid', $new_cid, '='));
-                    $criteria->add(new \Criteria('gperm_modid', $GLOBALS['xoopsModule']->getVar('mid'), '='));
-                    $criteria->add(new \Criteria('gperm_name', 'instruction_view', '='));
-                    $gpermHandler->deleteAll($criteria);
-                    // Права на добавление
-                    $criteria = new \CriteriaCompo();
-                    $criteria->add(new \Criteria('gperm_itemid', $new_cid, '='));
-                    $criteria->add(new \Criteria('gperm_modid', $GLOBALS['xoopsModule']->getVar('mid'), '='));
-                    $criteria->add(new \Criteria('gperm_name', 'instruction_submit', '='));
-                    $gpermHandler->deleteAll($criteria);
-                    // Права на редактирование
-                    $criteria = new \CriteriaCompo();
-                    $criteria->add(new \Criteria('gperm_itemid', $new_cid, '='));
-                    $criteria->add(new \Criteria('gperm_modid', $GLOBALS['xoopsModule']->getVar('mid'), '='));
-                    $criteria->add(new \Criteria('gperm_name', 'instruction_edit', '='));
-                    $gpermHandler->deleteAll($criteria);
-                }
-
-                // Добавляем права
-                // Права на просмотр
-                if (isset($_POST['groups_instr_view'])) {
-                    foreach ($_POST['groups_instr_view'] as $onegroup_id) {
-                        $gpermHandler->addRight('instruction_view', $new_cid, $onegroup_id, $GLOBALS['xoopsModule']->getVar('mid'));
-                    }
-                }
-                // Права на добавление
-                if (isset($_POST['groups_instr_submit'])) {
-                    foreach ($_POST['groups_instr_submit'] as $onegroup_id) {
-                        $gpermHandler->addRight('instruction_submit', $new_cid, $onegroup_id, $GLOBALS['xoopsModule']->getVar('mid'));
-                    }
-                }
-                // Права на редактирование
-                if (isset($_POST['groups_instr_edit'])) {
-                    foreach ($_POST['groups_instr_edit'] as $onegroup_id) {
-                        $gpermHandler->addRight('instruction_edit', $new_cid, $onegroup_id, $GLOBALS['xoopsModule']->getVar('mid'));
-                    }
-                }
-
-                //
-                redirect_header('cat.php', 3, _AM_INSTRUCTION_NEWCATADDED);
-            }
-            xoops_cp_header();
-            // Навигация
-            $adminObject->displayNavigation(basename(__FILE__));
-            // Выводим ошибки в шаблон
-            $GLOBALS['xoopsTpl']->assign('insErrorMsg', $objInstructioncat->getHtmlErrors());
-        }
-        // Выводим шаблон
-        $GLOBALS['xoopsTpl']->display('db:admin/instruction_admin_savecat.tpl');
-        // Выводим форму
-        $form = $objInstructioncat->getForm();
-        // Форма
-        echo $form->render();
-        // Текст внизу админки
-        include __DIR__ . '/admin_footer.php';
-
-        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 = isset($_POST['ok']) ? (int)$_POST['ok'] : 0;
-        // Если мы нажали на кнопку
-        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', $_POST['title']);
+		$objInstructioncat->setVar('description', $_POST['description']);
+		$objInstructioncat->setVar('weight', $weight);
+		$objInstructioncat->setVar('metakeywords', $_POST['metakeywords']);
+		$objInstructioncat->setVar('metadescription', $_POST['metadescription']);
+
+		// Проверка веса
+		if (0 == $weight) {
+			$err         = true;
+			$message_err .= _AM_INSTRUCTION_ERR_WEIGHT . '<br>';
+		}
+		// Проверка категорий
+		if ($cid && ($cid == $pid)) {
+			$err         = true;
+			$message_err .= _AM_INSTRUCTION_ERR_PCAT . '<br>';
+		}
+		// Если были ошибки
+		if (true === $err) {
+			xoops_cp_header();
+			// Навигация
+			$adminObject->displayNavigation(basename(__FILE__));
+
+			$message_err = '<div class="errorMsg" style="text-align: left;">' . $message_err . '</div>';
+			// Выводим ошибки в шаблон
+			$GLOBALS['xoopsTpl']->assign('insErrorMsg', $message_err);
+			// Если небыло ошибок
+		} else {
+			// Вставляем данные в БД
+			if ($categoryHandler->insert($objInstructioncat)) {
+
+				// ID категории. Если редактируем - то не изменяется. Если создаём новую - то получаем ID созданной записи.
+				$new_cid = $cid ?: $objInstructioncat->get_new_enreg();
+
+				// ===============
+				// ==== Права ====
+				// ===============
+
+				$gpermHandler = xoops_getHandler('groupperm');
+
+				// Если мы редактируем категорию, то старые права нужно удалить
+				if ($cid) {
+					// Права на просмотр
+					$criteria = new\ CriteriaCompo();
+					$criteria->add(new \Criteria('gperm_itemid', $new_cid, '='));
+					$criteria->add(new \Criteria('gperm_modid', $GLOBALS['xoopsModule']->getVar('mid'), '='));
+					$criteria->add(new \Criteria('gperm_name', 'instruction_view', '='));
+					$gpermHandler->deleteAll($criteria);
+					// Права на добавление
+					$criteria = new \CriteriaCompo();
+					$criteria->add(new \Criteria('gperm_itemid', $new_cid, '='));
+					$criteria->add(new \Criteria('gperm_modid', $GLOBALS['xoopsModule']->getVar('mid'), '='));
+					$criteria->add(new \Criteria('gperm_name', 'instruction_submit', '='));
+					$gpermHandler->deleteAll($criteria);
+					// Права на редактирование
+					$criteria = new \CriteriaCompo();
+					$criteria->add(new \Criteria('gperm_itemid', $new_cid, '='));
+					$criteria->add(new \Criteria('gperm_modid', $GLOBALS['xoopsModule']->getVar('mid'), '='));
+					$criteria->add(new \Criteria('gperm_name', 'instruction_edit', '='));
+					$gpermHandler->deleteAll($criteria);
+				}
+
+				// Добавляем права
+				// Права на просмотр
+				if (isset($_POST['groups_instr_view'])) {
+					foreach ($_POST['groups_instr_view'] as $onegroup_id) {
+						$gpermHandler->addRight('instruction_view', $new_cid, $onegroup_id, $GLOBALS['xoopsModule']->getVar('mid'));
+					}
+				}
+				// Права на добавление
+				if (isset($_POST['groups_instr_submit'])) {
+					foreach ($_POST['groups_instr_submit'] as $onegroup_id) {
+						$gpermHandler->addRight('instruction_submit', $new_cid, $onegroup_id, $GLOBALS['xoopsModule']->getVar('mid'));
+					}
+				}
+				// Права на редактирование
+				if (isset($_POST['groups_instr_edit'])) {
+					foreach ($_POST['groups_instr_edit'] as $onegroup_id) {
+						$gpermHandler->addRight('instruction_edit', $new_cid, $onegroup_id, $GLOBALS['xoopsModule']->getVar('mid'));
+					}
+				}
+
+				//
+				redirect_header('cat.php', 3, _AM_INSTRUCTION_NEWCATADDED);
+			}
+			xoops_cp_header();
+			// Навигация
+			$adminObject->displayNavigation(basename(__FILE__));
+			// Выводим ошибки в шаблон
+			$GLOBALS['xoopsTpl']->assign('insErrorMsg', $objInstructioncat->getHtmlErrors());
+		}
+		// Выводим шаблон
+		$GLOBALS['xoopsTpl']->display('db:admin/instruction_admin_savecat.tpl');
+		// Выводим форму
+		$form = $objInstructioncat->getForm();
+		// Форма
+		echo $form->render();
+		// Текст внизу админки
+		include __DIR__ . '/admin_footer.php';
+
+		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 = isset($_POST['ok']) ? (int)$_POST['ok'] : 0;
+		// Если мы нажали на кнопку
+		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 +6 added lines, -6 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);
         }
 
@@ -253,12 +253,12 @@ 
         }
 
         // Нажали ли мы на кнопку OK
-        $ok = isset($_POST['ok']) ? (int)$_POST['ok'] : 0;
+        $ok = isset($_POST['ok']) ? (int) $_POST['ok'] : 0;
         // Если мы нажали на кнопку
         if ($ok) {
 
             // Проверка
-            if (!$GLOBALS['xoopsSecurity']->check()) {
+            if ( ! $GLOBALS['xoopsSecurity']->check()) {
                 redirect_header('cat.php', 3, implode(',', $GLOBALS['xoopsSecurity']->getErrors()));
             }
             // Пытаемся удалить категорию