mambax7 / instruction

class/tree.php

Doc Comments +18 added lines

@@ -16,6 +16,11 @@ 
     {
     }
 
+    /**
+     * @param integer $key
+     * @param string $ret
+     * @param string $prefix_orig
+     */
     public function _makePagesAdminOptions($key, &$ret, $prefix_orig, $objInsinstr, $class = 'odd', $prefix_curr = '')
     {
         if ($key > 0) {
@@ -119,6 +124,11 @@ 
     // === Дерево категорий в админке ===
     // ==================================
 
+    /**
+     * @param integer $key
+     * @param string $ret
+     * @param string $prefix_orig
+     */
     public function _makeCatsAdminOptions($key, &$ret, $prefix_orig, $cidinstrids = [], &$class = 'odd', $prefix_curr = '')
     {
         if ($key > 0) {
@@ -204,6 +214,10 @@ 
     // Список страниц на стороне пользователя
     // ======================================
 
+    /**
+     * @param integer $key
+     * @param string $ret
+     */
     public function _makePagesUserTree($key, &$ret, $currpageid = 0, &$lastpageids = [], $level = 0)
     {
 
@@ -308,6 +322,10 @@ 
 
     // Находим предыдущую и следующую страницы.
     // Находим последнии страницы на каждом уровне.
+
+    /**
+     * @param integer $key
+     */
     public function _makePagesUserCalc($key, $currpageid = 0, &$prevpages = [], &$nextpages = [], &$lastpageids = [], $level = 0)
     {
 

Indentation +309 added lines, -309 removed lines

@@ -12,82 +12,82 @@ 
 // Наследник класса XoopsObjectTree
 class InstructionTree extends XoopsObjectTree
 {
-    public function __constrcut()
-    {
-    }
-
-    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 __constrcut()
+	{
+	}
+
+	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;
-        }
+			// Устанавливаем префикс
+			$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);
-            }
-        }
-    }
+		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);
+			}
+		}
+	}
 
-    public function makePagesAdmin(&$objInsinstr, $prefix = '-', $key = 0)
-    {
-        $ret = '<form name="inspages" action="instr.php" method="post">
+	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>
@@ -98,10 +98,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') . '" />
@@ -112,52 +112,52 @@ 
   </table>
 </form>';
 
-        return $ret;
-    }
-
-    // ==================================
-    // === Дерево категорий в админке ===
-    // ==================================
-
-    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;
+	}
+
+	// ==================================
+	// === Дерево категорий в админке ===
+	// ==================================
+
+	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>
@@ -168,20 +168,20 @@ 
       </td>
     </tr>';
 
-            // Устанавливаем префикс
-            $prefix_curr .= $prefix_orig;
-        }
+			// Устанавливаем префикс
+			$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);
-            }
-        }
-    }
+		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);
+			}
+		}
+	}
 
-    public function makeCatsAdmin($prefix = '-', $cidinstrids = [], $key = 0)
-    {
-        $ret = '<table width="100%" cellspacing="1" class="outer">
+	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>
@@ -192,193 +192,193 @@ 
       <td class="head" align="center" width="150">' . _AM_INSTRUCTION_ACTION . '</td>
     </tr>';
 
-        // Выводим все страницы
-        $this->_makeCatsAdminOptions($key, $ret, $prefix, $cidinstrids);
-
-        $ret .= '</table>';
-
-        return $ret;
-    }
-
-    // ======================================
-    // Список страниц на стороне пользователя
-    // ======================================
-
-    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 (in_array($currpageid, array_keys($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>';
-        }
-    }
-
-    // Находим предыдущую и следующую страницы.
-    // Находим последнии страницы на каждом уровне.
-    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);
-            }
-        }
-    }
-
-    //
-
-    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;
+	}
+
+	// ======================================
+	// Список страниц на стороне пользователя
+	// ======================================
+
+	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 (in_array($currpageid, array_keys($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>';
+		}
+	}
+
+	// Находим предыдущую и следующую страницы.
+	// Находим последнии страницы на каждом уровне.
+	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);
+			}
+		}
+	}
+
+	//
+
+	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;
+	}
 }

Spacing +4 added lines, -4 removed lines

@@ -78,7 +78,7 @@ 
             $prefix_curr .= $prefix_orig;
         }
 
-        if (isset($this->tree[$key]['child']) && !empty($this->tree[$key]['child'])) {
+        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);
             }
@@ -172,7 +172,7 @@ 
             $prefix_curr .= $prefix_orig;
         }
 
-        if (isset($this->tree[$key]['child']) && !empty($this->tree[$key]['child'])) {
+        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);
             }
@@ -289,7 +289,7 @@ 
         }
 
         // Рекурсия
-        if (isset($this->tree[$key]['child']) && !empty($this->tree[$key]['child'])) {
+        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);
             }
@@ -350,7 +350,7 @@ 
         }
 
         // Рекурсия
-        if (isset($this->tree[$key]['child']) && !empty($this->tree[$key]['child'])) {
+        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);
             }

class/utility.php

Doc Comments +5 added lines

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

Unused Use Statements -2 removed lines

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

Indentation +53 added lines, -53 removed lines

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

Spacing +1 added lines, -1 removed lines

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

include/comment_functions.php

Indentation +4 added lines, -4 removed lines

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

include/config.php

Indentation +49 added lines, -49 removed lines

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

Spacing +2 added lines, -2 removed lines

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

include/pagetypes.inc.php

Indentation +2 added lines, -2 removed lines

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

include/common.php

Indentation +10 added lines, -10 removed lines

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

Spacing +2 added lines, -2 removed lines

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

include/plugin.tag.php

Indentation +53 added lines, -53 removed lines

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

Spacing +3 added lines, -3 removed lines

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

Braces +4 added lines, -2 removed lines

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

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

+               }

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

include/search.inc.php

Indentation +38 added lines, -38 removed lines

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

Spacing +1 added lines, -1 removed lines

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

submit.php

Indentation +191 added lines, -191 removed lines

@@ -36,195 +36,195 @@
 
 switch ($op) {
 
-    case 'editpage':
-
-        // Задание тайтла
-        $xoopsOption['xoops_pagetitle'] = '';
-        // Шаблон
-        $GLOBALS['xoopsOption']['template_main'] = $moduleDirName . '_editpage.tpl';
-        // Заголовок
-        include_once $GLOBALS['xoops']->path('header.php');
-
-        // Если мы редактируем страницу
-        if ($pageid) {
-            // Получаем объект страницы
-            $objInspage = $inspageHandler->get($pageid);
-            // ID инструкции
-            $instrid = $objInspage->getVar('instrid');
-            // Объект инструкции
-            $objInsinstr = $insinstrHandler->get($instrid);
-            // Можно ли редактировать инструкцию в данной категории
-            if (!in_array($objInsinstr->getVar('cid'), $cat_edit)) {
-                redirect_header('index.php', 3, _MD_INSTRUCTION_NOPERM_EDITPAGE);
-            }
-            // Создание новой страницы
-        } elseif ($instrid) {
-
-            // Если нельзя добавлять не в одну категорию
-            //if( ! count( $cat_submit ) ) redirect_header( 'index.php', 3, _MD_INSTRUCTION_NOPERM_SUBMIT_PAGE );
-            // Создаём объект страницы
-            $objInspage = $inspageHandler->create();
-            // Объект инструкции
-            $objInsinstr = $insinstrHandler->get($instrid);
-            // Можно ли добавлять инструкции в данной категории
-            if (!in_array($objInsinstr->getVar('cid'), $cat_submit)) {
-                redirect_header('index.php', 3, _MD_INSTRUCTION_NOPERM_SUBMITPAGE);
-            }
-        } else {
-            redirect_header('index.php', 3, _MD_INSTRUCTION_BADREQUEST);
-        }
-
-        // Информация об инструкции
-
-        // Массив данных об инструкции
-        $instrs = [];
-        // ID инструкции
-        $instrs['instrid'] = $objInsinstr->getVar('instrid');
-        // Название страницы
-        $instrs['title'] = $objInsinstr->getVar('title');
-        // Описание
-        $instrs['description'] = $objInsinstr->getVar('description');
-
-        // Выводим в шаблон
-        $GLOBALS['xoopsTpl']->assign('insInstr', $instrs);
-
-        //
-
-        $form = $objInspage->getForm('submit.php', $instrid);
-        // Форма
-        $GLOBALS['xoopsTpl']->assign('insFormPage', $form->render());
-
-        // Подвал
-        include_once $GLOBALS['xoops']->path('footer.php');
-
-        break;
-    // Сохранение страницы
-    case 'savepage':
-
-        // Проверка
-        if (!$GLOBALS['xoopsSecurity']->check()) {
-            redirect_header('index.php', 3, implode(',', $GLOBALS['xoopsSecurity']->getErrors()));
-        }
-
-        $err         = false;
-        $message_err = '';
-
-        // Если мы редактируем
-        if ($pageid) {
-            $objInspage = $inspageHandler->get($pageid);
-            // Объект инструкции
-            $objInsinstr = $insinstrHandler->get($objInspage->getVar('instrid'));
-            // Можно ли редактировать инструкцию в данной категории
-            if (!in_array($objInsinstr->getVar('cid'), $cat_edit)) {
-                redirect_header('index.php', 3, _MD_INSTRUCTION_NOPERM_EDITPAGE);
-            }
-        } elseif ($instrid) {
-            $objInspage = $inspageHandler->create();
-            // Объект инструкции
-            $objInsinstr = $insinstrHandler->get($instrid);
-            // Можно ли добавлять инструкции в данной категории
-            if (!in_array($objInsinstr->getVar('cid'), $cat_submit)) {
-                redirect_header('index.php', 3, _MD_INSTRUCTION_NOPERM_SUBMITPAGE);
-            }
-
-            // Если мы создаём страницу необходимо указать к какой инструкции
-            $objInspage->setVar('instrid', $instrid);
-            // Указываем дату создания
-            $objInspage->setVar('datecreated', $time);
-            // Указываем пользователя
-            $objInspage->setVar('uid', $uid);
-        } else {
-            redirect_header('index.php', 3, _MD_INSTRUCTION_BADREQUEST);
-        }
-
-        // Родительская страница
-        $objInspage->setVar('pid', $pid);
-        // Дата обновления
-        $objInspage->setVar('dateupdated', $time);
-        //
-        $objInspage->setVar('title', $_POST['title']);
-        $objInspage->setVar('weight', $weight);
-        $objInspage->setVar('hometext', $_POST['hometext']);
-        // Сноска
-        $objInspage->setVar('footnote', $_POST['footnote']);
-        $objInspage->setVar('status', $_POST['status']);
-        $objInspage->setVar('keywords', $_POST['keywords']);
-        $objInspage->setVar('description', $_POST['description']);
-
-        // Проверка категорий
-        if (!$pageid && !$instrid) {
-            $err         = true;
-            $message_err .= _MD_INSTRUCTION_ERR_INSTR . '<br>';
-        }
-        // Проверка веса
-        if (0 == $weight) {
-            $err         = true;
-            $message_err .= _MD_INSTRUCTION_ERR_WEIGHT . '<br>';
-        }
-        // Проверка родительской страницы
-        if ($pageid && ($pageid == $pid)) {
-            $err         = true;
-            $message_err .= _MD_INSTRUCTION_ERR_PPAGE . '<br>';
-        }
-        // Если были ошибки
-        if (true === $err) {
-            // Задание тайтла
-            $xoopsOption['xoops_pagetitle'] = '';
-            // Шаблон
-            $GLOBALS['xoopsOption']['template_main'] = $moduleDirName . '_savepage.tpl';
-            // Заголовок
-            include_once $GLOBALS['xoops']->path('header.php');
-            // Сообщение об ошибке
-            $message_err = '<div class="errorMsg" style="text-align: left;">' . $message_err . '</div>';
-            // Выводим ошибки в шаблон
-            $GLOBALS['xoopsTpl']->assign('insErrorMsg', $message_err);
-            // Если небыло ошибок
-        } else {
-            // Вставляем данные в БД
-            if ($inspageHandler->insert($objInspage)) {
-                // Если мы редактируем
-                if ($pageid) {
-                    // Обновление даты
-                    $sql = sprintf('UPDATE %s SET `dateupdated` = %u WHERE `instrid` = %u', $GLOBALS['xoopsDB']->prefix($moduleDirName . '_instr'), $time, $instrid);
-                    $GLOBALS['xoopsDB']->query($sql);
-                    // Запись в лог
-                    xoops_loadLanguage('main', 'userslog');
-                    //userslog_insert( $objInsinstr->getVar('title') . ': ' . $objInspage->getVar('title'), _MD_USERSLOG_MODIFY_PAGE );
-                    //
-                    redirect_header('index.php', 3, _MD_INSTRUCTION_PAGEMODIFY);
-                    // Если мы добавляем
-                } else {
-                    // Инкримент комментов
-                    $inspageHandler->updateposts($uid, $_POST['status'], 'add');
-                    // Инкремент страниц и обновление даты
-                    $sql = sprintf('UPDATE %s SET `pages` = `pages` + 1, `dateupdated` = %u WHERE `instrid` = %u', $GLOBALS['xoopsDB']->prefix($moduleDirName . '_instr'), $time, $instrid);
-                    $GLOBALS['xoopsDB']->query($sql);
-                    // Запись в лог
-                    xoops_loadLanguage('main', 'userslog');
-                    //userslog_insert( $objInsinstr->getVar('title') . ': ' . $objInspage->getVar('title'), _MD_USERSLOG_SUBMIT_PAGE );
-                    //
-                    redirect_header('index.php', 3, _MD_INSTRUCTION_PAGEADDED);
-                }
-            }
-
-            // Задание тайтла
-            $xoopsOption['xoops_pagetitle'] = '';
-            // Шаблон
-            $GLOBALS['xoopsOption']['template_main'] = $moduleDirName . '_savepage.tpl';
-            // Заголовок
-            include_once $GLOBALS['xoops']->path('header.php');
-
-            // Выводим ошибки в шаблон
-            $GLOBALS['xoopsTpl']->assign('insErrorMsg', $objInspage->getHtmlErrors());
-        }
-        // Получаем форму
-        $form = $objInspage->getForm('submit.php', $instrid);
-
-        // Форма
-        $GLOBALS['xoopsTpl']->assign('insFormPage', $form->render());
-
-        // Подвал
-        include_once $GLOBALS['xoops']->path('footer.php');
-
-        break;
+	case 'editpage':
+
+		// Задание тайтла
+		$xoopsOption['xoops_pagetitle'] = '';
+		// Шаблон
+		$GLOBALS['xoopsOption']['template_main'] = $moduleDirName . '_editpage.tpl';
+		// Заголовок
+		include_once $GLOBALS['xoops']->path('header.php');
+
+		// Если мы редактируем страницу
+		if ($pageid) {
+			// Получаем объект страницы
+			$objInspage = $inspageHandler->get($pageid);
+			// ID инструкции
+			$instrid = $objInspage->getVar('instrid');
+			// Объект инструкции
+			$objInsinstr = $insinstrHandler->get($instrid);
+			// Можно ли редактировать инструкцию в данной категории
+			if (!in_array($objInsinstr->getVar('cid'), $cat_edit)) {
+				redirect_header('index.php', 3, _MD_INSTRUCTION_NOPERM_EDITPAGE);
+			}
+			// Создание новой страницы
+		} elseif ($instrid) {
+
+			// Если нельзя добавлять не в одну категорию
+			//if( ! count( $cat_submit ) ) redirect_header( 'index.php', 3, _MD_INSTRUCTION_NOPERM_SUBMIT_PAGE );
+			// Создаём объект страницы
+			$objInspage = $inspageHandler->create();
+			// Объект инструкции
+			$objInsinstr = $insinstrHandler->get($instrid);
+			// Можно ли добавлять инструкции в данной категории
+			if (!in_array($objInsinstr->getVar('cid'), $cat_submit)) {
+				redirect_header('index.php', 3, _MD_INSTRUCTION_NOPERM_SUBMITPAGE);
+			}
+		} else {
+			redirect_header('index.php', 3, _MD_INSTRUCTION_BADREQUEST);
+		}
+
+		// Информация об инструкции
+
+		// Массив данных об инструкции
+		$instrs = [];
+		// ID инструкции
+		$instrs['instrid'] = $objInsinstr->getVar('instrid');
+		// Название страницы
+		$instrs['title'] = $objInsinstr->getVar('title');
+		// Описание
+		$instrs['description'] = $objInsinstr->getVar('description');
+
+		// Выводим в шаблон
+		$GLOBALS['xoopsTpl']->assign('insInstr', $instrs);
+
+		//
+
+		$form = $objInspage->getForm('submit.php', $instrid);
+		// Форма
+		$GLOBALS['xoopsTpl']->assign('insFormPage', $form->render());
+
+		// Подвал
+		include_once $GLOBALS['xoops']->path('footer.php');
+
+		break;
+	// Сохранение страницы
+	case 'savepage':
+
+		// Проверка
+		if (!$GLOBALS['xoopsSecurity']->check()) {
+			redirect_header('index.php', 3, implode(',', $GLOBALS['xoopsSecurity']->getErrors()));
+		}
+
+		$err         = false;
+		$message_err = '';
+
+		// Если мы редактируем
+		if ($pageid) {
+			$objInspage = $inspageHandler->get($pageid);
+			// Объект инструкции
+			$objInsinstr = $insinstrHandler->get($objInspage->getVar('instrid'));
+			// Можно ли редактировать инструкцию в данной категории
+			if (!in_array($objInsinstr->getVar('cid'), $cat_edit)) {
+				redirect_header('index.php', 3, _MD_INSTRUCTION_NOPERM_EDITPAGE);
+			}
+		} elseif ($instrid) {
+			$objInspage = $inspageHandler->create();
+			// Объект инструкции
+			$objInsinstr = $insinstrHandler->get($instrid);
+			// Можно ли добавлять инструкции в данной категории
+			if (!in_array($objInsinstr->getVar('cid'), $cat_submit)) {
+				redirect_header('index.php', 3, _MD_INSTRUCTION_NOPERM_SUBMITPAGE);
+			}
+
+			// Если мы создаём страницу необходимо указать к какой инструкции
+			$objInspage->setVar('instrid', $instrid);
+			// Указываем дату создания
+			$objInspage->setVar('datecreated', $time);
+			// Указываем пользователя
+			$objInspage->setVar('uid', $uid);
+		} else {
+			redirect_header('index.php', 3, _MD_INSTRUCTION_BADREQUEST);
+		}
+
+		// Родительская страница
+		$objInspage->setVar('pid', $pid);
+		// Дата обновления
+		$objInspage->setVar('dateupdated', $time);
+		//
+		$objInspage->setVar('title', $_POST['title']);
+		$objInspage->setVar('weight', $weight);
+		$objInspage->setVar('hometext', $_POST['hometext']);
+		// Сноска
+		$objInspage->setVar('footnote', $_POST['footnote']);
+		$objInspage->setVar('status', $_POST['status']);
+		$objInspage->setVar('keywords', $_POST['keywords']);
+		$objInspage->setVar('description', $_POST['description']);
+
+		// Проверка категорий
+		if (!$pageid && !$instrid) {
+			$err         = true;
+			$message_err .= _MD_INSTRUCTION_ERR_INSTR . '<br>';
+		}
+		// Проверка веса
+		if (0 == $weight) {
+			$err         = true;
+			$message_err .= _MD_INSTRUCTION_ERR_WEIGHT . '<br>';
+		}
+		// Проверка родительской страницы
+		if ($pageid && ($pageid == $pid)) {
+			$err         = true;
+			$message_err .= _MD_INSTRUCTION_ERR_PPAGE . '<br>';
+		}
+		// Если были ошибки
+		if (true === $err) {
+			// Задание тайтла
+			$xoopsOption['xoops_pagetitle'] = '';
+			// Шаблон
+			$GLOBALS['xoopsOption']['template_main'] = $moduleDirName . '_savepage.tpl';
+			// Заголовок
+			include_once $GLOBALS['xoops']->path('header.php');
+			// Сообщение об ошибке
+			$message_err = '<div class="errorMsg" style="text-align: left;">' . $message_err . '</div>';
+			// Выводим ошибки в шаблон
+			$GLOBALS['xoopsTpl']->assign('insErrorMsg', $message_err);
+			// Если небыло ошибок
+		} else {
+			// Вставляем данные в БД
+			if ($inspageHandler->insert($objInspage)) {
+				// Если мы редактируем
+				if ($pageid) {
+					// Обновление даты
+					$sql = sprintf('UPDATE %s SET `dateupdated` = %u WHERE `instrid` = %u', $GLOBALS['xoopsDB']->prefix($moduleDirName . '_instr'), $time, $instrid);
+					$GLOBALS['xoopsDB']->query($sql);
+					// Запись в лог
+					xoops_loadLanguage('main', 'userslog');
+					//userslog_insert( $objInsinstr->getVar('title') . ': ' . $objInspage->getVar('title'), _MD_USERSLOG_MODIFY_PAGE );
+					//
+					redirect_header('index.php', 3, _MD_INSTRUCTION_PAGEMODIFY);
+					// Если мы добавляем
+				} else {
+					// Инкримент комментов
+					$inspageHandler->updateposts($uid, $_POST['status'], 'add');
+					// Инкремент страниц и обновление даты
+					$sql = sprintf('UPDATE %s SET `pages` = `pages` + 1, `dateupdated` = %u WHERE `instrid` = %u', $GLOBALS['xoopsDB']->prefix($moduleDirName . '_instr'), $time, $instrid);
+					$GLOBALS['xoopsDB']->query($sql);
+					// Запись в лог
+					xoops_loadLanguage('main', 'userslog');
+					//userslog_insert( $objInsinstr->getVar('title') . ': ' . $objInspage->getVar('title'), _MD_USERSLOG_SUBMIT_PAGE );
+					//
+					redirect_header('index.php', 3, _MD_INSTRUCTION_PAGEADDED);
+				}
+			}
+
+			// Задание тайтла
+			$xoopsOption['xoops_pagetitle'] = '';
+			// Шаблон
+			$GLOBALS['xoopsOption']['template_main'] = $moduleDirName . '_savepage.tpl';
+			// Заголовок
+			include_once $GLOBALS['xoops']->path('header.php');
+
+			// Выводим ошибки в шаблон
+			$GLOBALS['xoopsTpl']->assign('insErrorMsg', $objInspage->getHtmlErrors());
+		}
+		// Получаем форму
+		$form = $objInspage->getForm('submit.php', $instrid);
+
+		// Форма
+		$GLOBALS['xoopsTpl']->assign('insFormPage', $form->render());
+
+		// Подвал
+		include_once $GLOBALS['xoops']->path('footer.php');
+
+		break;
 }

Spacing +16 added lines, -16 removed lines

@@ -14,17 +14,17 @@ 
 $time = time();
 
 // ID инструкции
-$instrid = isset($_GET['instrid']) ? (int)$_GET['instrid'] : 0;
-$instrid = isset($_POST['instrid']) ? (int)$_POST['instrid'] : $instrid;
+$instrid = isset($_GET['instrid']) ? (int) $_GET['instrid'] : 0;
+$instrid = isset($_POST['instrid']) ? (int) $_POST['instrid'] : $instrid;
 // ID страницы
-$pageid = isset($_GET['pageid']) ? (int)$_GET['pageid'] : 0;
-$pageid = isset($_POST['pageid']) ? (int)$_POST['pageid'] : $pageid;
+$pageid = isset($_GET['pageid']) ? (int) $_GET['pageid'] : 0;
+$pageid = isset($_POST['pageid']) ? (int) $_POST['pageid'] : $pageid;
 // ID категории
-$cid = isset($_POST['cid']) ? (int)$_POST['cid'] : 0;
+$cid = isset($_POST['cid']) ? (int) $_POST['cid'] : 0;
 // Вес
-$weight = isset($_POST['weight']) ? (int)$_POST['weight'] : 0;
+$weight = isset($_POST['weight']) ? (int) $_POST['weight'] : 0;
 //
-$pid = isset($_POST['pid']) ? (int)$_POST['pid'] : 0;
+$pid = isset($_POST['pid']) ? (int) $_POST['pid'] : 0;
 
 // Права на добавление
 $cat_submit = InstructionUtility::getItemIds($moduleDirName . '_submit');
@@ -54,7 +54,7 @@ 
             // Объект инструкции
             $objInsinstr = $insinstrHandler->get($instrid);
             // Можно ли редактировать инструкцию в данной категории
-            if (!in_array($objInsinstr->getVar('cid'), $cat_edit)) {
+            if ( ! in_array($objInsinstr->getVar('cid'), $cat_edit)) {
                 redirect_header('index.php', 3, _MD_INSTRUCTION_NOPERM_EDITPAGE);
             }
             // Создание новой страницы
@@ -67,7 +67,7 @@ 
             // Объект инструкции
             $objInsinstr = $insinstrHandler->get($instrid);
             // Можно ли добавлять инструкции в данной категории
-            if (!in_array($objInsinstr->getVar('cid'), $cat_submit)) {
+            if ( ! in_array($objInsinstr->getVar('cid'), $cat_submit)) {
                 redirect_header('index.php', 3, _MD_INSTRUCTION_NOPERM_SUBMITPAGE);
             }
         } else {
@@ -102,7 +102,7 @@ 
     case 'savepage':
 
         // Проверка
-        if (!$GLOBALS['xoopsSecurity']->check()) {
+        if ( ! $GLOBALS['xoopsSecurity']->check()) {
             redirect_header('index.php', 3, implode(',', $GLOBALS['xoopsSecurity']->getErrors()));
         }
 
@@ -115,7 +115,7 @@ 
             // Объект инструкции
             $objInsinstr = $insinstrHandler->get($objInspage->getVar('instrid'));
             // Можно ли редактировать инструкцию в данной категории
-            if (!in_array($objInsinstr->getVar('cid'), $cat_edit)) {
+            if ( ! in_array($objInsinstr->getVar('cid'), $cat_edit)) {
                 redirect_header('index.php', 3, _MD_INSTRUCTION_NOPERM_EDITPAGE);
             }
         } elseif ($instrid) {
@@ -123,7 +123,7 @@ 
             // Объект инструкции
             $objInsinstr = $insinstrHandler->get($instrid);
             // Можно ли добавлять инструкции в данной категории
-            if (!in_array($objInsinstr->getVar('cid'), $cat_submit)) {
+            if ( ! in_array($objInsinstr->getVar('cid'), $cat_submit)) {
                 redirect_header('index.php', 3, _MD_INSTRUCTION_NOPERM_SUBMITPAGE);
             }
 
@@ -152,18 +152,18 @@ 
         $objInspage->setVar('description', $_POST['description']);
 
         // Проверка категорий
-        if (!$pageid && !$instrid) {
-            $err         = true;
+        if ( ! $pageid && ! $instrid) {
+            $err = true;
             $message_err .= _MD_INSTRUCTION_ERR_INSTR . '<br>';
         }
         // Проверка веса
         if (0 == $weight) {
-            $err         = true;
+            $err = true;
             $message_err .= _MD_INSTRUCTION_ERR_WEIGHT . '<br>';
         }
         // Проверка родительской страницы
         if ($pageid && ($pageid == $pid)) {
-            $err         = true;
+            $err = true;
             $message_err .= _MD_INSTRUCTION_ERR_PPAGE . '<br>';
         }
         // Если были ошибки