mambax7 / instruction

class/common/breadcrumb.php

Spacing +1 added lines, -1 removed lines

@@ -63,7 +63,7 @@
      */
     public function render()
     {
-        if (!isset($GLOBALS['xoTheme']) || !is_object($GLOBALS['xoTheme'])) {
+        if ( ! isset($GLOBALS['xoTheme']) || ! is_object($GLOBALS['xoTheme'])) {
             require_once $GLOBALS['xoops']->path('class/theme.php');
             $GLOBALS['xoTheme'] = new xos_opal_Theme();
         }

Indentation +39 added lines, -39 removed lines

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

class/common/traitfilesmgmt.php

Indentation +222 added lines, -222 removed lines

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

Spacing +19 added lines, -19 removed lines

@@ -26,8 +26,8 @@ 
     public static function createFolder($folder)
     {
         try {
-            if (!file_exists($folder)) {
-                if (!mkdir($folder) && !is_dir($folder)) {
+            if ( ! file_exists($folder)) {
+                if ( ! mkdir($folder) && ! is_dir($folder)) {
                     throw new \RuntimeException(sprintf('Unable to create the %s directory', $folder));
                 }
 
@@ -82,7 +82,7 @@ 
     public static function deleteDirectory($src)
     {
         // Only continue if user is a 'global' Admin
-        if (!($GLOBALS['xoopsUser'] instanceof XoopsUser) || !$GLOBALS['xoopsUser']->isAdmin()) {
+        if ( ! ($GLOBALS['xoopsUser'] instanceof XoopsUser) || ! $GLOBALS['xoopsUser']->isAdmin()) {
             return false;
         }
 
@@ -96,12 +96,12 @@ 
                 $fileInfo = new SplFileInfo("{$src}/{$v}");
                 if ($fileInfo->isDir()) {
                     // recursively handle subdirectories
-                    if (!$success = self::deleteDirectory($fileInfo->getRealPath())) {
+                    if ( ! $success = self::deleteDirectory($fileInfo->getRealPath())) {
                         break;
                     }
                 } else {
                     // delete the file
-                    if (!($success = unlink($fileInfo->getRealPath()))) {
+                    if ( ! ($success = unlink($fileInfo->getRealPath()))) {
                         break;
                     }
                 }
@@ -130,12 +130,12 @@ 
     public static function rrmdir($src)
     {
         // Only continue if user is a 'global' Admin
-        if (!($GLOBALS['xoopsUser'] instanceof XoopsUser) || !$GLOBALS['xoopsUser']->isAdmin()) {
+        if ( ! ($GLOBALS['xoopsUser'] instanceof XoopsUser) || ! $GLOBALS['xoopsUser']->isAdmin()) {
             return false;
         }
 
         // If source is not a directory stop processing
-        if (!is_dir($src)) {
+        if ( ! is_dir($src)) {
             return false;
         }
 
@@ -147,15 +147,15 @@ 
             if ($fObj->isFile()) {
                 $filename = $fObj->getPathname();
                 $fObj     = null; // clear this iterator object to close the file
-                if (!unlink($filename)) {
+                if ( ! unlink($filename)) {
                     return false; // couldn't delete the file
                 }
-            } elseif (!$fObj->isDot() && $fObj->isDir()) {
+            } elseif ( ! $fObj->isDot() && $fObj->isDir()) {
                 // Try recursively on directory
                 self::rrmdir($fObj->getPathname());
             }
         }
-        $iterator = null;   // clear iterator Obj to close file/directory
+        $iterator = null; // clear iterator Obj to close file/directory
         return rmdir($src); // remove the directory & return results
     }
 
@@ -170,17 +170,17 @@ 
     public static function rmove($src, $dest)
     {
         // Only continue if user is a 'global' Admin
-        if (!($GLOBALS['xoopsUser'] instanceof XoopsUser) || !$GLOBALS['xoopsUser']->isAdmin()) {
+        if ( ! ($GLOBALS['xoopsUser'] instanceof XoopsUser) || ! $GLOBALS['xoopsUser']->isAdmin()) {
             return false;
         }
 
         // If source is not a directory stop processing
-        if (!is_dir($src)) {
+        if ( ! is_dir($src)) {
             return false;
         }
 
         // If the destination directory does not exist and could not be created stop processing
-        if (!is_dir($dest) && !mkdir($dest, 0755)) {
+        if ( ! is_dir($dest) && ! mkdir($dest, 0755)) {
             return false;
         }
 
@@ -189,13 +189,13 @@ 
         foreach ($iterator as $fObj) {
             if ($fObj->isFile()) {
                 rename($fObj->getPathname(), "{$dest}/" . $fObj->getFilename());
-            } elseif (!$fObj->isDot() && $fObj->isDir()) {
+            } elseif ( ! $fObj->isDot() && $fObj->isDir()) {
                 // Try recursively on directory
                 self::rmove($fObj->getPathname(), "{$dest}/" . $fObj->getFilename());
                 //                rmdir($fObj->getPath()); // now delete the directory
             }
         }
-        $iterator = null;   // clear iterator Obj to close file/directory
+        $iterator = null; // clear iterator Obj to close file/directory
         return rmdir($src); // remove the directory & return results
     }
 
@@ -213,17 +213,17 @@ 
     public static function rcopy($src, $dest)
     {
         // Only continue if user is a 'global' Admin
-        if (!($GLOBALS['xoopsUser'] instanceof XoopsUser) || !$GLOBALS['xoopsUser']->isAdmin()) {
+        if ( ! ($GLOBALS['xoopsUser'] instanceof XoopsUser) || ! $GLOBALS['xoopsUser']->isAdmin()) {
             return false;
         }
 
         // If source is not a directory stop processing
-        if (!is_dir($src)) {
+        if ( ! is_dir($src)) {
             return false;
         }
 
         // If the destination directory does not exist and could not be created stop processing
-        if (!is_dir($dest) && !mkdir($dest, 0755)) {
+        if ( ! is_dir($dest) && ! mkdir($dest, 0755)) {
             return false;
         }
 
@@ -232,7 +232,7 @@ 
         foreach ($iterator as $fObj) {
             if ($fObj->isFile()) {
                 copy($fObj->getPathname(), "{$dest}/" . $fObj->getFilename());
-            } elseif (!$fObj->isDot() && $fObj->isDir()) {
+            } elseif ( ! $fObj->isDot() && $fObj->isDir()) {
                 self::rcopy($fObj->getPathname(), "{$dest}/" . $fObj->getFilename());
             }
         }

class/instruction.php

Spacing +3 added lines, -3 removed lines

@@ -96,7 +96,7 @@ 
         $form->addElement(new XoopsFormText(_AM_INSTRUCTION_METADESCRIPTIONC, 'metadescription', 50, 255, $this->getVar('metadescription')), false);
 
         // Если мы редактируем категорию
-        if (!$this->isNew()) {
+        if ( ! $this->isNew()) {
             $form->addElement(new XoopsFormHidden('instrid', $this->getVar('instrid')));
         }
         //
@@ -118,9 +118,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);
     }

Indentation +145 added lines, -145 removed lines

@@ -8,153 +8,153 @@
 
 class InstructionInstruction extends XoopsObject
 {
-    // constructor
-    public function __construct()
-    {
-        //		$this->XoopsObject();
-        $this->initVar('instrid', XOBJ_DTYPE_INT, null, false, 11);
-        $this->initVar('cid', XOBJ_DTYPE_INT, 0, false, 5);
-        $this->initVar('uid', XOBJ_DTYPE_INT, 0, false, 11);
-        $this->initVar('title', XOBJ_DTYPE_TXTBOX, '', false);
-        $this->initVar('status', XOBJ_DTYPE_INT, 0, false, 1);
-        $this->initVar('pages', XOBJ_DTYPE_INT, 0, false, 11);
-        $this->initVar('description', XOBJ_DTYPE_TXTAREA, null, false);
-        $this->initVar('datecreated', XOBJ_DTYPE_INT, 0, false, 10);
-        $this->initVar('dateupdated', XOBJ_DTYPE_INT, 0, false, 10);
-        $this->initVar('metakeywords', XOBJ_DTYPE_TXTBOX, '', false);
-        $this->initVar('metadescription', XOBJ_DTYPE_TXTBOX, '', false);
-
-        // Нет в таблице
-        $this->initVar('dohtml', XOBJ_DTYPE_INT, 1, false);
-        $this->initVar('dobr', XOBJ_DTYPE_INT, 0, false);
-    }
-
-    public function InstructionInstruction()
-    {
-        $this->__construct();
-    }
-
-    public function get_new_enreg()
-    {
-        $new_enreg = $GLOBALS['xoopsDB']->getInsertId();
-        return $new_enreg;
-    }
-
-    // Получаем форму
-    public function getForm($action = false)
-    {
-        global $xoopsDB, $xoopsModule, $xoopsModuleConfig;
-        // Если нет $action
-        if (false === $action) {
-            $action = xoops_getenv('REQUEST_URI');
-        }
-        // Подключаем формы
-        include_once $GLOBALS['xoops']->path('class/xoopsformloader.php');
-
-        // Название формы
-        $title = $this->isNew() ? sprintf(_AM_INSTRUCTION_FORMADDINSTR) : sprintf(_AM_INSTRUCTION_FORMEDITINSTR);
-
-        // Форма
-        $form = new XoopsThemeForm($title, 'forminstr', $action, 'post', true);
-        //$form->setExtra('enctype="multipart/form-data"');
-        // Название инструкции
-        $form->addElement(new XoopsFormText(_AM_INSTRUCTION_TITLEC, 'title', 50, 255, $this->getVar('title')), true);
-        // Категория
-        $instructioncatHandler = xoops_getModuleHandler('category', 'instruction');
-        $criteria              = new CriteriaCompo();
-        $criteria->setSort('weight ASC, title');
-        $criteria->setOrder('ASC');
-        $instructioncat_arr = $instructioncatHandler->getall($criteria);
-        unset($criteria);
-        // Подключаем трей
-        include_once $GLOBALS['xoops']->path('class/tree.php');
-        $mytree = new XoopsObjectTree($instructioncat_arr, 'cid', 'pid');
-
-        // $form->addElement(new XoopsFormLabel(_AM_INSTRUCTION_CATC, $mytree->makeSelBox('cid', 'title', '--', $this->getVar('cid'), true)));
-        $moduleDirName = basename(__DIR__);
-        if (false !== ($moduleHelper = Xmf\Module\Helper::getHelper($moduleDirName))) {
-        } else {
-            $moduleHelper = Xmf\Module\Helper::getHelper('system');
-        }
-        $module = $moduleHelper->getModule();
-
-        if (InstructionUtility::checkVerXoops($module, '2.5.9')) {
-            $mytree_select = $mytree->makeSelectElement('cid', 'title', '--', $this->getVar('cid'), true, 0, '', _AM_INSTRUCTION_CATC);
-            $form->addElement($mytree_select);
-        } else {
-            $form->addElement(new XoopsFormLabel(_AM_INSTRUCTION_CATC, $mytree->makeSelBox('cid', 'title', '--', $this->getVar('cid'), true)));
-        }
-
-        // Описание
-        $form->addElement(InstructionUtility::getWysiwygForm(_AM_INSTRUCTION_DESCRIPTIONC, 'description', $this->getVar('description', 'e')), true);
-        // Статус
-        $form->addElement(new XoopsFormRadioYN(_AM_INSTRUCTION_ACTIVEC, 'status', $this->getVar('status')), false);
-
-        // Теги
-        if (is_dir('../../tag') || is_dir('../tag')) {
-            $dir_tag_ok = true;
-        } else {
-            $dir_tag_ok = false;
-        }
-        // Если влючена поддержка тегов и есть модуль tag
-        if (xoops_getModuleOption('usetag', 'instruction') && $dir_tag_ok) {
-            $itemIdForTag = $this->isNew() ? 0 : $this->getVar('instrid');
-            // Подключаем форму тегов
-            include_once $GLOBALS['xoops']->path('modules/tag/include/formtag.php');
-            // Добавляем элемент в форму
-            $form->addElement(new XoopsFormTag('tag', 60, 255, $itemIdForTag, 0));
-        }
-
-        // Мета-теги ключевых слов
-        $form->addElement(new XoopsFormText(_AM_INSTRUCTION_METAKEYWORDSC, 'metakeywords', 50, 255, $this->getVar('metakeywords')), false);
-        // Мета-теги описания
-        $form->addElement(new XoopsFormText(_AM_INSTRUCTION_METADESCRIPTIONC, 'metadescription', 50, 255, $this->getVar('metadescription')), false);
-
-        // Если мы редактируем категорию
-        if (!$this->isNew()) {
-            $form->addElement(new XoopsFormHidden('instrid', $this->getVar('instrid')));
-        }
-        //
-        $form->addElement(new XoopsFormHidden('op', 'saveinstr'));
-        // Кнопка
-        $form->addElement(new XoopsFormButton('', 'submit', _SUBMIT, 'submit'));
-        return $form;
-    }
+	// constructor
+	public function __construct()
+	{
+		//		$this->XoopsObject();
+		$this->initVar('instrid', XOBJ_DTYPE_INT, null, false, 11);
+		$this->initVar('cid', XOBJ_DTYPE_INT, 0, false, 5);
+		$this->initVar('uid', XOBJ_DTYPE_INT, 0, false, 11);
+		$this->initVar('title', XOBJ_DTYPE_TXTBOX, '', false);
+		$this->initVar('status', XOBJ_DTYPE_INT, 0, false, 1);
+		$this->initVar('pages', XOBJ_DTYPE_INT, 0, false, 11);
+		$this->initVar('description', XOBJ_DTYPE_TXTAREA, null, false);
+		$this->initVar('datecreated', XOBJ_DTYPE_INT, 0, false, 10);
+		$this->initVar('dateupdated', XOBJ_DTYPE_INT, 0, false, 10);
+		$this->initVar('metakeywords', XOBJ_DTYPE_TXTBOX, '', false);
+		$this->initVar('metadescription', XOBJ_DTYPE_TXTBOX, '', false);
+
+		// Нет в таблице
+		$this->initVar('dohtml', XOBJ_DTYPE_INT, 1, false);
+		$this->initVar('dobr', XOBJ_DTYPE_INT, 0, false);
+	}
+
+	public function InstructionInstruction()
+	{
+		$this->__construct();
+	}
+
+	public function get_new_enreg()
+	{
+		$new_enreg = $GLOBALS['xoopsDB']->getInsertId();
+		return $new_enreg;
+	}
+
+	// Получаем форму
+	public function getForm($action = false)
+	{
+		global $xoopsDB, $xoopsModule, $xoopsModuleConfig;
+		// Если нет $action
+		if (false === $action) {
+			$action = xoops_getenv('REQUEST_URI');
+		}
+		// Подключаем формы
+		include_once $GLOBALS['xoops']->path('class/xoopsformloader.php');
+
+		// Название формы
+		$title = $this->isNew() ? sprintf(_AM_INSTRUCTION_FORMADDINSTR) : sprintf(_AM_INSTRUCTION_FORMEDITINSTR);
+
+		// Форма
+		$form = new XoopsThemeForm($title, 'forminstr', $action, 'post', true);
+		//$form->setExtra('enctype="multipart/form-data"');
+		// Название инструкции
+		$form->addElement(new XoopsFormText(_AM_INSTRUCTION_TITLEC, 'title', 50, 255, $this->getVar('title')), true);
+		// Категория
+		$instructioncatHandler = xoops_getModuleHandler('category', 'instruction');
+		$criteria              = new CriteriaCompo();
+		$criteria->setSort('weight ASC, title');
+		$criteria->setOrder('ASC');
+		$instructioncat_arr = $instructioncatHandler->getall($criteria);
+		unset($criteria);
+		// Подключаем трей
+		include_once $GLOBALS['xoops']->path('class/tree.php');
+		$mytree = new XoopsObjectTree($instructioncat_arr, 'cid', 'pid');
+
+		// $form->addElement(new XoopsFormLabel(_AM_INSTRUCTION_CATC, $mytree->makeSelBox('cid', 'title', '--', $this->getVar('cid'), true)));
+		$moduleDirName = basename(__DIR__);
+		if (false !== ($moduleHelper = Xmf\Module\Helper::getHelper($moduleDirName))) {
+		} else {
+			$moduleHelper = Xmf\Module\Helper::getHelper('system');
+		}
+		$module = $moduleHelper->getModule();
+
+		if (InstructionUtility::checkVerXoops($module, '2.5.9')) {
+			$mytree_select = $mytree->makeSelectElement('cid', 'title', '--', $this->getVar('cid'), true, 0, '', _AM_INSTRUCTION_CATC);
+			$form->addElement($mytree_select);
+		} else {
+			$form->addElement(new XoopsFormLabel(_AM_INSTRUCTION_CATC, $mytree->makeSelBox('cid', 'title', '--', $this->getVar('cid'), true)));
+		}
+
+		// Описание
+		$form->addElement(InstructionUtility::getWysiwygForm(_AM_INSTRUCTION_DESCRIPTIONC, 'description', $this->getVar('description', 'e')), true);
+		// Статус
+		$form->addElement(new XoopsFormRadioYN(_AM_INSTRUCTION_ACTIVEC, 'status', $this->getVar('status')), false);
+
+		// Теги
+		if (is_dir('../../tag') || is_dir('../tag')) {
+			$dir_tag_ok = true;
+		} else {
+			$dir_tag_ok = false;
+		}
+		// Если влючена поддержка тегов и есть модуль tag
+		if (xoops_getModuleOption('usetag', 'instruction') && $dir_tag_ok) {
+			$itemIdForTag = $this->isNew() ? 0 : $this->getVar('instrid');
+			// Подключаем форму тегов
+			include_once $GLOBALS['xoops']->path('modules/tag/include/formtag.php');
+			// Добавляем элемент в форму
+			$form->addElement(new XoopsFormTag('tag', 60, 255, $itemIdForTag, 0));
+		}
+
+		// Мета-теги ключевых слов
+		$form->addElement(new XoopsFormText(_AM_INSTRUCTION_METAKEYWORDSC, 'metakeywords', 50, 255, $this->getVar('metakeywords')), false);
+		// Мета-теги описания
+		$form->addElement(new XoopsFormText(_AM_INSTRUCTION_METADESCRIPTIONC, 'metadescription', 50, 255, $this->getVar('metadescription')), false);
+
+		// Если мы редактируем категорию
+		if (!$this->isNew()) {
+			$form->addElement(new XoopsFormHidden('instrid', $this->getVar('instrid')));
+		}
+		//
+		$form->addElement(new XoopsFormHidden('op', 'saveinstr'));
+		// Кнопка
+		$form->addElement(new XoopsFormButton('', 'submit', _SUBMIT, 'submit'));
+		return $form;
+	}
 }
 
 class InstructionInstructionHandler extends XoopsPersistableObjectHandler
 {
-    public function __construct($db)
-    {
-        parent::__construct($db, 'instruction_instr', 'InstructionInstruction', 'instrid', 'title');
-    }
-
-    // Обновление даты обновления инструкций
-    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);
-    }
-
-    // Обновление числа страниц
-    public function updatePages($instrid = 0)
-    {
-        $inspageHandler = xoops_getModuleHandler('page', 'instruction');
-        // Находим число активных страниц
-        $criteria = new CriteriaCompo();
-        $criteria->add(new Criteria('instrid', $instrid, '='));
-        $criteria->add(new Criteria('status ', '0', '>'));
-        // Число страниц
-        $pages = $inspageHandler->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);
-    }
+	public function __construct($db)
+	{
+		parent::__construct($db, 'instruction_instr', 'InstructionInstruction', 'instrid', 'title');
+	}
+
+	// Обновление даты обновления инструкций
+	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);
+	}
+
+	// Обновление числа страниц
+	public function updatePages($instrid = 0)
+	{
+		$inspageHandler = xoops_getModuleHandler('page', 'instruction');
+		// Находим число активных страниц
+		$criteria = new CriteriaCompo();
+		$criteria->add(new Criteria('instrid', $instrid, '='));
+		$criteria->add(new Criteria('status ', '0', '>'));
+		// Число страниц
+		$pages = $inspageHandler->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);
+	}
 }

class/helper.php

Indentation +29 added lines, -29 removed lines

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

class/page.php

Spacing +2 added lines, -2 removed lines

@@ -73,7 +73,7 @@ 
         // Находим все страницы данной инструкции
         $criteria->add(new Criteria('instrid', $instrid_page, '='));
         // Если мы редактируем, то убрать текущую страницу из списка выбора родительской
-        if (!$this->isNew()) {
+        if ( ! $this->isNew()) {
             $criteria->add(new Criteria('pageid', $this->getVar('pageid'), '<>'));
         }
         $criteria->setSort('weight');
@@ -128,7 +128,7 @@ 
         $form->addElement($option_tray);
 
         // Если мы редактируем страницу
-        if (!$this->isNew()) {
+        if ( ! $this->isNew()) {
             $form->addElement(new XoopsFormHidden('pageid', $this->getVar('pageid')));
         } else {
             $form->addElement(new XoopsFormHidden('pageid', 0));

Indentation +214 added lines, -214 removed lines

@@ -8,222 +8,222 @@
 
 class InstructionPage extends XoopsObject
 {
-    // constructor
-    public function __construct()
-    {
-        //	$this->XoopsObject();
-        $this->initVar('pageid', XOBJ_DTYPE_INT, null, false, 11);
-        $this->initVar('pid', XOBJ_DTYPE_INT, 0, false, 11);
-        $this->initVar('instrid', XOBJ_DTYPE_INT, 0, false, 11);
-        $this->initVar('uid', XOBJ_DTYPE_INT, 0, false, 11);
-        $this->initVar('title', XOBJ_DTYPE_TXTBOX, '', false, 255);
-        $this->initVar('status', XOBJ_DTYPE_INT, 1, false, 1);
-        $this->initVar('type', XOBJ_DTYPE_INT, 1, false, 1);
-        $this->initVar('hometext', XOBJ_DTYPE_TXTAREA, null, false);
-        $this->initVar('footnote', XOBJ_DTYPE_TXTAREA, '', false);
-        $this->initVar('weight', XOBJ_DTYPE_INT, 0, false, 11);
-        $this->initVar('keywords', XOBJ_DTYPE_TXTBOX, '', false, 255);
-        $this->initVar('description', XOBJ_DTYPE_TXTBOX, '', false, 255);
-        $this->initVar('comments', XOBJ_DTYPE_INT, 0, false, 11);
-        $this->initVar('datecreated', XOBJ_DTYPE_INT, 0, false, 10);
-        $this->initVar('dateupdated', XOBJ_DTYPE_INT, 0, false, 10);
-        $this->initVar('dohtml', XOBJ_DTYPE_INT, 1, false, 1);
-        $this->initVar('dosmiley', XOBJ_DTYPE_INT, 0, false, 1);
-        $this->initVar('doxcode', XOBJ_DTYPE_INT, 1, false, 1);
-        $this->initVar('dobr', XOBJ_DTYPE_INT, 0, false, 1);
-    }
-
-    public function InstructionPage()
-    {
-        $this->__construct();
-    }
-
-    public function get_new_enreg()
-    {
-        $new_enreg = $GLOBALS['xoopsDB']->getInsertId();
-        return $new_enreg;
-    }
-
-    // Получаем форму
-    public function getForm($action = false, $instrid = 0)
-    {
-        // Если нет $action
-        if (false === $action) {
-            $action = xoops_getenv('REQUEST_URI');
-        }
-
-        // Подключаем формы
-        include_once $GLOBALS['xoops']->path('class/xoopsformloader.php');
-        // Подключаем типы страниц
-        $pagetypes = include $GLOBALS['xoops']->path('modules/instruction/include/pagetypes.inc.php');
-
-        // Название формы
-        $title = $this->isNew() ? sprintf(_AM_INSTRUCTION_FORMADDPAGE) : sprintf(_AM_INSTRUCTION_FORMEDITPAGE);
-
-        // Форма
-        $form = new XoopsThemeForm($title, 'instr_form_page', $action, 'post', true);
-        // Название
-        $form->addElement(new XoopsFormText(_AM_INSTRUCTION_TITLEC, 'title', 50, 255, $this->getVar('title')), true);
-
-        // Родительская страница
-        $inspageHandler = xoops_getModuleHandler('page', 'instruction');
-        $criteria       = new CriteriaCompo();
-        // ID инструкции в которой данная страница
-        $instrid_page = $this->isNew() ? $instrid : $this->getVar('instrid');
-        // Находим все страницы данной инструкции
-        $criteria->add(new Criteria('instrid', $instrid_page, '='));
-        // Если мы редактируем, то убрать текущую страницу из списка выбора родительской
-        if (!$this->isNew()) {
-            $criteria->add(new Criteria('pageid', $this->getVar('pageid'), '<>'));
-        }
-        $criteria->setSort('weight');
-        $criteria->setOrder('ASC');
-        $inspage_arr = $inspageHandler->getall($criteria);
-        unset($criteria);
-        // Подключаем трей
-        include_once $GLOBALS['xoops']->path('class/tree.php');
-        $mytree = new XoopsObjectTree($inspage_arr, 'pageid', 'pid');
-
-        // $form->addElement(new XoopsFormLabel(_AM_INSTRUCTION_PPAGEC, $mytree->makeSelBox('pid', 'title', '--', $this->getVar('pid'), true)));
-        $moduleDirName = basename(__DIR__);
-        if (false !== ($moduleHelper = Xmf\Module\Helper::getHelper($moduleDirName))) {
-        } else {
-            $moduleHelper = Xmf\Module\Helper::getHelper('system');
-        }
-        $module = $moduleHelper->getModule();
-
-        if (InstructionUtility::checkVerXoops($module, '2.5.9')) {
-            $mytree_select = $mytree->makeSelectElement('pid', 'title', '--', $this->getVar('pid'), true, 0, '', _AM_INSTRUCTION_PPAGEC);
-            $form->addElement($mytree_select);
-        } else {
-            $form->addElement(new XoopsFormLabel(_AM_INSTRUCTION_PPAGEC, $mytree->makeSelBox('pid', 'title', '--', $this->getVar('pid'), true)));
-        }
-
-        // Вес
-        $form->addElement(new XoopsFormText(_AM_INSTRUCTION_WEIGHTC, 'weight', 5, 5, $this->getVar('weight')), true);
-        // Основной текст
-        $form->addElement(InstructionUtility::getWysiwygForm(_AM_INSTRUCTION_HOMETEXTC, 'hometext', $this->getVar('hometext', 'e')), true);
-        // Сноска
-        $form_footnote = new XoopsFormTextArea(_AM_INSTRUCTION_FOOTNOTEC, 'footnote', $this->getVar('footnote', 'e'));
-        $form_footnote->setDescription(_AM_INSTRUCTION_FOOTNOTE_DSC);
-        $form->addElement($form_footnote, false);
-        unset($form_footnote);
-        // Статус
-        $form->addElement(new XoopsFormRadioYN(_AM_INSTRUCTION_ACTIVEC, 'status', $this->getVar('status')), false);
-        // Тип страницы
-        $form_type = new XoopsFormSelect(_AM_INSTR_PAGETYPEC, 'type', $this->getVar('type'));
-        $form_type->setDescription(_AM_INSTR_PAGETYPEC_DESC);
-        $form_type->addOptionArray($pagetypes);
-        $form->addElement($form_type, false);
-        // Мета-теги ключевых слов
-        $form->addElement(new XoopsFormText(_AM_INSTRUCTION_METAKEYWORDSC, 'keywords', 50, 255, $this->getVar('keywords')), false);
-        // Мета-теги описания
-        $form->addElement(new XoopsFormText(_AM_INSTRUCTION_METADESCRIPTIONC, 'description', 50, 255, $this->getVar('description')), false);
-
-        // Настройки
-        $option_tray = new XoopsFormElementTray(_OPTIONS, '<br>');
-        // HTML
-        $html_checkbox = new XoopsFormCheckBox('', 'dohtml', $this->getVar('dohtml'));
-        $html_checkbox->addOption(1, _AM_INSTR_DOHTML);
-        $option_tray->addElement($html_checkbox);
-        // Смайлы
-        $smiley_checkbox = new XoopsFormCheckBox('', 'dosmiley', $this->getVar('dosmiley'));
-        $smiley_checkbox->addOption(1, _AM_INSTR_DOSMILEY);
-        $option_tray->addElement($smiley_checkbox);
-        // ББ коды
-        $xcode_checkbox = new XoopsFormCheckBox('', 'doxcode', $this->getVar('doxcode'));
-        $xcode_checkbox->addOption(1, _AM_INSTR_DOXCODE);
-        $option_tray->addElement($xcode_checkbox);
-        //
-        $br_checkbox = new XoopsFormCheckBox('', 'dobr', $this->getVar('dobr'));
-        $br_checkbox->addOption(1, _AM_INSTR_DOAUTOWRAP);
-        $option_tray->addElement($br_checkbox);
-        //
-        $form->addElement($option_tray);
-
-        // Если мы редактируем страницу
-        if (!$this->isNew()) {
-            $form->addElement(new XoopsFormHidden('pageid', $this->getVar('pageid')));
-        } else {
-            $form->addElement(new XoopsFormHidden('pageid', 0));
-        }
-        // ID инструкции
-        if ($instrid) {
-            $form->addElement(new XoopsFormHidden('instrid', $instrid));
-        } else {
-            $form->addElement(new XoopsFormHidden('instrid', 0));
-        }
-        //
-        $form->addElement(new XoopsFormHidden('op', 'savepage'));
-        // Кнопка
-        $button_tray = new XoopsFormElementTray('', '');
-        $button_tray->addElement(new XoopsFormButton('', 'submit', _SUBMIT, 'submit'));
-        $save_btn = new XoopsFormButton('', 'cancel', _AM_INSTR_SAVEFORM);
-        $save_btn->setExtra('onclick="instrSavePage();"');
-        $button_tray->addElement($save_btn);
-        $form->addElement($button_tray);
-
-        return $form;
-    }
-
-    //
-    public function getInstrid()
-    {
-        // Возвращаем ID инструкции
-        return $this->getVar('instrid');
-    }
+	// constructor
+	public function __construct()
+	{
+		//	$this->XoopsObject();
+		$this->initVar('pageid', XOBJ_DTYPE_INT, null, false, 11);
+		$this->initVar('pid', XOBJ_DTYPE_INT, 0, false, 11);
+		$this->initVar('instrid', XOBJ_DTYPE_INT, 0, false, 11);
+		$this->initVar('uid', XOBJ_DTYPE_INT, 0, false, 11);
+		$this->initVar('title', XOBJ_DTYPE_TXTBOX, '', false, 255);
+		$this->initVar('status', XOBJ_DTYPE_INT, 1, false, 1);
+		$this->initVar('type', XOBJ_DTYPE_INT, 1, false, 1);
+		$this->initVar('hometext', XOBJ_DTYPE_TXTAREA, null, false);
+		$this->initVar('footnote', XOBJ_DTYPE_TXTAREA, '', false);
+		$this->initVar('weight', XOBJ_DTYPE_INT, 0, false, 11);
+		$this->initVar('keywords', XOBJ_DTYPE_TXTBOX, '', false, 255);
+		$this->initVar('description', XOBJ_DTYPE_TXTBOX, '', false, 255);
+		$this->initVar('comments', XOBJ_DTYPE_INT, 0, false, 11);
+		$this->initVar('datecreated', XOBJ_DTYPE_INT, 0, false, 10);
+		$this->initVar('dateupdated', XOBJ_DTYPE_INT, 0, false, 10);
+		$this->initVar('dohtml', XOBJ_DTYPE_INT, 1, false, 1);
+		$this->initVar('dosmiley', XOBJ_DTYPE_INT, 0, false, 1);
+		$this->initVar('doxcode', XOBJ_DTYPE_INT, 1, false, 1);
+		$this->initVar('dobr', XOBJ_DTYPE_INT, 0, false, 1);
+	}
+
+	public function InstructionPage()
+	{
+		$this->__construct();
+	}
+
+	public function get_new_enreg()
+	{
+		$new_enreg = $GLOBALS['xoopsDB']->getInsertId();
+		return $new_enreg;
+	}
+
+	// Получаем форму
+	public function getForm($action = false, $instrid = 0)
+	{
+		// Если нет $action
+		if (false === $action) {
+			$action = xoops_getenv('REQUEST_URI');
+		}
+
+		// Подключаем формы
+		include_once $GLOBALS['xoops']->path('class/xoopsformloader.php');
+		// Подключаем типы страниц
+		$pagetypes = include $GLOBALS['xoops']->path('modules/instruction/include/pagetypes.inc.php');
+
+		// Название формы
+		$title = $this->isNew() ? sprintf(_AM_INSTRUCTION_FORMADDPAGE) : sprintf(_AM_INSTRUCTION_FORMEDITPAGE);
+
+		// Форма
+		$form = new XoopsThemeForm($title, 'instr_form_page', $action, 'post', true);
+		// Название
+		$form->addElement(new XoopsFormText(_AM_INSTRUCTION_TITLEC, 'title', 50, 255, $this->getVar('title')), true);
+
+		// Родительская страница
+		$inspageHandler = xoops_getModuleHandler('page', 'instruction');
+		$criteria       = new CriteriaCompo();
+		// ID инструкции в которой данная страница
+		$instrid_page = $this->isNew() ? $instrid : $this->getVar('instrid');
+		// Находим все страницы данной инструкции
+		$criteria->add(new Criteria('instrid', $instrid_page, '='));
+		// Если мы редактируем, то убрать текущую страницу из списка выбора родительской
+		if (!$this->isNew()) {
+			$criteria->add(new Criteria('pageid', $this->getVar('pageid'), '<>'));
+		}
+		$criteria->setSort('weight');
+		$criteria->setOrder('ASC');
+		$inspage_arr = $inspageHandler->getall($criteria);
+		unset($criteria);
+		// Подключаем трей
+		include_once $GLOBALS['xoops']->path('class/tree.php');
+		$mytree = new XoopsObjectTree($inspage_arr, 'pageid', 'pid');
+
+		// $form->addElement(new XoopsFormLabel(_AM_INSTRUCTION_PPAGEC, $mytree->makeSelBox('pid', 'title', '--', $this->getVar('pid'), true)));
+		$moduleDirName = basename(__DIR__);
+		if (false !== ($moduleHelper = Xmf\Module\Helper::getHelper($moduleDirName))) {
+		} else {
+			$moduleHelper = Xmf\Module\Helper::getHelper('system');
+		}
+		$module = $moduleHelper->getModule();
+
+		if (InstructionUtility::checkVerXoops($module, '2.5.9')) {
+			$mytree_select = $mytree->makeSelectElement('pid', 'title', '--', $this->getVar('pid'), true, 0, '', _AM_INSTRUCTION_PPAGEC);
+			$form->addElement($mytree_select);
+		} else {
+			$form->addElement(new XoopsFormLabel(_AM_INSTRUCTION_PPAGEC, $mytree->makeSelBox('pid', 'title', '--', $this->getVar('pid'), true)));
+		}
+
+		// Вес
+		$form->addElement(new XoopsFormText(_AM_INSTRUCTION_WEIGHTC, 'weight', 5, 5, $this->getVar('weight')), true);
+		// Основной текст
+		$form->addElement(InstructionUtility::getWysiwygForm(_AM_INSTRUCTION_HOMETEXTC, 'hometext', $this->getVar('hometext', 'e')), true);
+		// Сноска
+		$form_footnote = new XoopsFormTextArea(_AM_INSTRUCTION_FOOTNOTEC, 'footnote', $this->getVar('footnote', 'e'));
+		$form_footnote->setDescription(_AM_INSTRUCTION_FOOTNOTE_DSC);
+		$form->addElement($form_footnote, false);
+		unset($form_footnote);
+		// Статус
+		$form->addElement(new XoopsFormRadioYN(_AM_INSTRUCTION_ACTIVEC, 'status', $this->getVar('status')), false);
+		// Тип страницы
+		$form_type = new XoopsFormSelect(_AM_INSTR_PAGETYPEC, 'type', $this->getVar('type'));
+		$form_type->setDescription(_AM_INSTR_PAGETYPEC_DESC);
+		$form_type->addOptionArray($pagetypes);
+		$form->addElement($form_type, false);
+		// Мета-теги ключевых слов
+		$form->addElement(new XoopsFormText(_AM_INSTRUCTION_METAKEYWORDSC, 'keywords', 50, 255, $this->getVar('keywords')), false);
+		// Мета-теги описания
+		$form->addElement(new XoopsFormText(_AM_INSTRUCTION_METADESCRIPTIONC, 'description', 50, 255, $this->getVar('description')), false);
+
+		// Настройки
+		$option_tray = new XoopsFormElementTray(_OPTIONS, '<br>');
+		// HTML
+		$html_checkbox = new XoopsFormCheckBox('', 'dohtml', $this->getVar('dohtml'));
+		$html_checkbox->addOption(1, _AM_INSTR_DOHTML);
+		$option_tray->addElement($html_checkbox);
+		// Смайлы
+		$smiley_checkbox = new XoopsFormCheckBox('', 'dosmiley', $this->getVar('dosmiley'));
+		$smiley_checkbox->addOption(1, _AM_INSTR_DOSMILEY);
+		$option_tray->addElement($smiley_checkbox);
+		// ББ коды
+		$xcode_checkbox = new XoopsFormCheckBox('', 'doxcode', $this->getVar('doxcode'));
+		$xcode_checkbox->addOption(1, _AM_INSTR_DOXCODE);
+		$option_tray->addElement($xcode_checkbox);
+		//
+		$br_checkbox = new XoopsFormCheckBox('', 'dobr', $this->getVar('dobr'));
+		$br_checkbox->addOption(1, _AM_INSTR_DOAUTOWRAP);
+		$option_tray->addElement($br_checkbox);
+		//
+		$form->addElement($option_tray);
+
+		// Если мы редактируем страницу
+		if (!$this->isNew()) {
+			$form->addElement(new XoopsFormHidden('pageid', $this->getVar('pageid')));
+		} else {
+			$form->addElement(new XoopsFormHidden('pageid', 0));
+		}
+		// ID инструкции
+		if ($instrid) {
+			$form->addElement(new XoopsFormHidden('instrid', $instrid));
+		} else {
+			$form->addElement(new XoopsFormHidden('instrid', 0));
+		}
+		//
+		$form->addElement(new XoopsFormHidden('op', 'savepage'));
+		// Кнопка
+		$button_tray = new XoopsFormElementTray('', '');
+		$button_tray->addElement(new XoopsFormButton('', 'submit', _SUBMIT, 'submit'));
+		$save_btn = new XoopsFormButton('', 'cancel', _AM_INSTR_SAVEFORM);
+		$save_btn->setExtra('onclick="instrSavePage();"');
+		$button_tray->addElement($save_btn);
+		$form->addElement($button_tray);
+
+		return $form;
+	}
+
+	//
+	public function getInstrid()
+	{
+		// Возвращаем ID инструкции
+		return $this->getVar('instrid');
+	}
 }
 
 class InstructionPageHandler extends XoopsPersistableObjectHandler
 {
-    public function __construct($db)
-    {
-        parent::__construct($db, 'instruction_page', 'InstructionPage', 'pageid', 'title');
-    }
-
-    /**
-     * Generate function for update user post
-     *
-     * @ Update user post count after send approve content
-     * @ Update user post count after change status content
-     * @ Update user post count after delete content
-     */
-    public function updateposts($uid, $status, $action)
-    {
-        //
-        switch ($action) {
-            // Добавление страницы
-            case 'add':
-                if ($uid && $status) {
-                    $user          = new xoopsUser($uid);
-                    $memberHandler = xoops_getHandler('member');
-                    // Добавялем +1 к комментам
-                    $memberHandler->updateUserByField($user, 'posts', $user->getVar('posts') + 1);
-                }
-                break;
-            // Удаление страницы
-            case 'delete':
-                if ($uid && $status) {
-                    $user          = new xoopsUser($uid);
-                    $memberHandler = xoops_getHandler('member');
-                    // Декримент комментов
-                    //$user->setVar( 'posts', $user->getVar( 'posts' ) - 1 );
-                    // Сохраняем
-                    $memberHandler->updateUserByField($user, 'posts', $user->getVar('posts') - 1);
-                }
-                break;
-
-            case 'status':
-                if ($uid) {
-                    $user          = new xoopsUser($uid);
-                    $memberHandler = xoops_getHandler('member');
-                    if ($status) {
-                        $memberHandler->updateUserByField($user, 'posts', $user->getVar('posts') - 1);
-                    } else {
-                        $memberHandler->updateUserByField($user, 'posts', $user->getVar('posts') + 1);
-                    }
-                }
-                break;
-        }
-    }
+	public function __construct($db)
+	{
+		parent::__construct($db, 'instruction_page', 'InstructionPage', 'pageid', 'title');
+	}
+
+	/**
+	 * Generate function for update user post
+	 *
+	 * @ Update user post count after send approve content
+	 * @ Update user post count after change status content
+	 * @ Update user post count after delete content
+	 */
+	public function updateposts($uid, $status, $action)
+	{
+		//
+		switch ($action) {
+			// Добавление страницы
+			case 'add':
+				if ($uid && $status) {
+					$user          = new xoopsUser($uid);
+					$memberHandler = xoops_getHandler('member');
+					// Добавялем +1 к комментам
+					$memberHandler->updateUserByField($user, 'posts', $user->getVar('posts') + 1);
+				}
+				break;
+			// Удаление страницы
+			case 'delete':
+				if ($uid && $status) {
+					$user          = new xoopsUser($uid);
+					$memberHandler = xoops_getHandler('member');
+					// Декримент комментов
+					//$user->setVar( 'posts', $user->getVar( 'posts' ) - 1 );
+					// Сохраняем
+					$memberHandler->updateUserByField($user, 'posts', $user->getVar('posts') - 1);
+				}
+				break;
+
+			case 'status':
+				if ($uid) {
+					$user          = new xoopsUser($uid);
+					$memberHandler = xoops_getHandler('member');
+					if ($status) {
+						$memberHandler->updateUserByField($user, 'posts', $user->getVar('posts') - 1);
+					} else {
+						$memberHandler->updateUserByField($user, 'posts', $user->getVar('posts') + 1);
+					}
+				}
+				break;
+		}
+	}
 }

class/category.php

Spacing +7 added lines, -7 removed lines

@@ -83,7 +83,7 @@ 
         $instructioncatHandler = xoops_getModuleHandler('category', 'instruction');
         $criteria              = new CriteriaCompo();
         // Если мы редактируем, то убрать текущую категорию из списка выбора родительской
-        if (!$this->isNew()) {
+        if ( ! $this->isNew()) {
             $criteria->add(new Criteria('cid', $this->getVar('cid'), '<>'));
         }
         $criteria->setSort('weight ASC, title');
@@ -128,7 +128,7 @@ 
         // Права на просмотр
         $groups_ids = [];
         // Если мы редактируем
-        if (!$this->isNew()) {
+        if ( ! $this->isNew()) {
             $groups_ids        = $gpermHandler->getGroupIds('instruction_view', $this->getVar('cid'), $GLOBALS['xoopsModule']->getVar('mid'));
             $groups_ids        = array_values($groups_ids);
             $groups_instr_view = new XoopsFormCheckBox(_AM_INSTRUCTION_PERM_VIEW, 'groups_instr_view', $groups_ids);
@@ -140,7 +140,7 @@ 
 
         // Права на отправку
         $groups_ids = [];
-        if (!$this->isNew()) {
+        if ( ! $this->isNew()) {
             $groups_ids          = $gpermHandler->getGroupIds('instruction_submit', $this->getVar('cid'), $GLOBALS['xoopsModule']->getVar('mid'));
             $groups_ids          = array_values($groups_ids);
             $groups_instr_submit = new XoopsFormCheckBox(_AM_INSTRUCTION_PERM_SUBMIT, 'groups_instr_submit', $groups_ids);
@@ -152,7 +152,7 @@ 
 
         // Права на редактирование
         $groups_ids = [];
-        if (!$this->isNew()) {
+        if ( ! $this->isNew()) {
             $groups_ids        = $gpermHandler->getGroupIds('instruction_edit', $this->getVar('cid'), $GLOBALS['xoopsModule']->getVar('mid'));
             $groups_ids        = array_values($groups_ids);
             $groups_instr_edit = new XoopsFormCheckBox(_AM_INSTRUCTION_PERM_EDIT, 'groups_instr_edit', $groups_ids);
@@ -166,7 +166,7 @@ 
         // ==========================================================
 
         // Если мы редактируем категорию
-        if (!$this->isNew()) {
+        if ( ! $this->isNew()) {
             $form->addElement(new XoopsFormHidden('cid', $this->getVar('cid')));
             //$form->addElement( new XoopsFormHidden( 'catmodify', true));
         }
@@ -196,9 +196,9 @@ 
     public function updateDateupdated($cid = 0, $time = null)
     {
         // Если не передали время
-        $time = null === $time ? time() : (int)$time;
+        $time = null === $time ? time() : (int) $time;
         //
-        $sql = sprintf('UPDATE `%s` SET `dateupdated` = %u WHERE `cid` = %u', $this->table, $time, (int)$cid);
+        $sql = sprintf('UPDATE `%s` SET `dateupdated` = %u WHERE `cid` = %u', $this->table, $time, (int) $cid);
         //
         return $this->db->query($sql);
     }

Indentation +169 added lines, -169 removed lines

@@ -11,56 +11,56 @@ 
 
 class InstructionCategory extends XoopsObject
 {
-    // constructor
-    public function __construct()
-    {
-        //		$this->XoopsObject();
-        $this->initVar('cid', XOBJ_DTYPE_INT, null, false, 5);
-        $this->initVar('pid', XOBJ_DTYPE_INT, 0, false, 5);
-        $this->initVar('title', XOBJ_DTYPE_TXTBOX, '', false);
-        $this->initVar('imgurl', XOBJ_DTYPE_TXTBOX, '', false);
-        $this->initVar('description', XOBJ_DTYPE_TXTAREA, null, false);
-        $this->initVar('weight', XOBJ_DTYPE_INT, 0, false, 11);
-        $this->initVar('datecreated', XOBJ_DTYPE_INT, 0, false, 10);
-        $this->initVar('dateupdated', XOBJ_DTYPE_INT, 0, false, 10);
-        $this->initVar('metakeywords', XOBJ_DTYPE_TXTBOX, '', false);
-        $this->initVar('metadescription', XOBJ_DTYPE_TXTBOX, '', false);
-    }
-
-    public function InstructionCategory()
-    {
-        $this->__construct();
-    }
-
-    public function get_new_enreg()
-    {
-        $new_enreg = $GLOBALS['xoopsDB']->getInsertId();
-        return $new_enreg;
-    }
-
-    // Получаем форму
-    public function getForm($action = false)
-    {
-        //global $xoopsDB, $xoopsModule, $xoopsModuleConfig;
-        // Если нет $action
-        if (false === $action) {
-            $action = xoops_getenv('REQUEST_URI');
-        }
-        // Подключаем формы
-        include_once $GLOBALS['xoops']->path('class/xoopsformloader.php');
-
-        // Название формы
-        $title = $this->isNew() ? sprintf(_AM_INSTRUCTION_FORMADDCAT) : sprintf(_AM_INSTRUCTION_FORMEDITCAT);
-
-        // Форма
-        $form = new XoopsThemeForm($title, 'formcat', $action, 'post', true);
-        //$form->setExtra('enctype="multipart/form-data"');
-        // Название категории
-        $form->addElement(new XoopsFormText(_AM_INSTRUCTION_TITLEC, 'title', 50, 255, $this->getVar('title')), true);
-        // Редактор
-        $form->addElement(new XoopsFormTextArea(_AM_INSTRUCTION_DSCC, 'description', $this->getVar('description', 'e')), true);
-        //image
-        /*
+	// constructor
+	public function __construct()
+	{
+		//		$this->XoopsObject();
+		$this->initVar('cid', XOBJ_DTYPE_INT, null, false, 5);
+		$this->initVar('pid', XOBJ_DTYPE_INT, 0, false, 5);
+		$this->initVar('title', XOBJ_DTYPE_TXTBOX, '', false);
+		$this->initVar('imgurl', XOBJ_DTYPE_TXTBOX, '', false);
+		$this->initVar('description', XOBJ_DTYPE_TXTAREA, null, false);
+		$this->initVar('weight', XOBJ_DTYPE_INT, 0, false, 11);
+		$this->initVar('datecreated', XOBJ_DTYPE_INT, 0, false, 10);
+		$this->initVar('dateupdated', XOBJ_DTYPE_INT, 0, false, 10);
+		$this->initVar('metakeywords', XOBJ_DTYPE_TXTBOX, '', false);
+		$this->initVar('metadescription', XOBJ_DTYPE_TXTBOX, '', false);
+	}
+
+	public function InstructionCategory()
+	{
+		$this->__construct();
+	}
+
+	public function get_new_enreg()
+	{
+		$new_enreg = $GLOBALS['xoopsDB']->getInsertId();
+		return $new_enreg;
+	}
+
+	// Получаем форму
+	public function getForm($action = false)
+	{
+		//global $xoopsDB, $xoopsModule, $xoopsModuleConfig;
+		// Если нет $action
+		if (false === $action) {
+			$action = xoops_getenv('REQUEST_URI');
+		}
+		// Подключаем формы
+		include_once $GLOBALS['xoops']->path('class/xoopsformloader.php');
+
+		// Название формы
+		$title = $this->isNew() ? sprintf(_AM_INSTRUCTION_FORMADDCAT) : sprintf(_AM_INSTRUCTION_FORMEDITCAT);
+
+		// Форма
+		$form = new XoopsThemeForm($title, 'formcat', $action, 'post', true);
+		//$form->setExtra('enctype="multipart/form-data"');
+		// Название категории
+		$form->addElement(new XoopsFormText(_AM_INSTRUCTION_TITLEC, 'title', 50, 255, $this->getVar('title')), true);
+		// Редактор
+		$form->addElement(new XoopsFormTextArea(_AM_INSTRUCTION_DSCC, 'description', $this->getVar('description', 'e')), true);
+		//image
+		/*
         $downloadscat_img = $this->getVar('imgurl') ? $this->getVar('imgurl') : 'blank.gif';
         $uploadirectory='/uploads/tdmdownloads/images/cats';
         $imgtray = new XoopsFormElementTray(_AM_TDMDOWNLOADS_FORMIMG,'<br>');
@@ -79,127 +79,127 @@ 
         $imgtray->addElement($fileseltray);
         $form->addElement($imgtray);
         */
-        // Родительская категория
-        $instructioncatHandler = xoops_getModuleHandler('category', 'instruction');
-        $criteria              = new CriteriaCompo();
-        // Если мы редактируем, то убрать текущую категорию из списка выбора родительской
-        if (!$this->isNew()) {
-            $criteria->add(new Criteria('cid', $this->getVar('cid'), '<>'));
-        }
-        $criteria->setSort('weight ASC, title');
-        $criteria->setOrder('ASC');
-        $instructioncat_arr = $instructioncatHandler->getall($criteria);
-        unset($criteria);
-        // Подключаем трей
-        include_once $GLOBALS['xoops']->path('class/tree.php');
-        $mytree = new XoopsObjectTree($instructioncat_arr, 'cid', 'pid');
-
-        // $form->addElement(new XoopsFormLabel(_AM_INSTRUCTION_PCATC, $mytree->makeSelBox('pid', 'title', '--', $this->getVar('pid'), true)));
-        $moduleDirName = basename(__DIR__);
-        if (false !== ($moduleHelper = Xmf\Module\Helper::getHelper($moduleDirName))) {
-        } else {
-            $moduleHelper = Xmf\Module\Helper::getHelper('system');
-        }
-        $module = $moduleHelper->getModule();
-
-        if (InstructionUtility::checkVerXoops($module, '2.5.9')) {
-            $mytree_select = $mytree->makeSelectElement('pid', 'title', '--', $this->getVar('pid'), true, 0, '', _AM_INSTRUCTION_PCATC);
-            $form->addElement($mytree_select);
-        } else {
-            $form->addElement(new XoopsFormLabel(_AM_INSTRUCTION_PCATC, $mytree->makeSelBox('pid', 'title', '--', $this->getVar('pid'), true)));
-        }
-
-        // Вес
-        $form->addElement(new XoopsFormText(_AM_INSTRUCTION_WEIGHTC, 'weight', 5, 5, $this->getVar('weight')), true);
-        // Мета-теги ключевых слов
-        $form->addElement(new XoopsFormText(_AM_INSTRUCTION_METAKEYWORDSC, 'metakeywords', 50, 255, $this->getVar('metakeywords')), false);
-        // Мета-теги описания
-        $form->addElement(new XoopsFormText(_AM_INSTRUCTION_METADESCRIPTIONC, 'metadescription', 50, 255, $this->getVar('metadescription')), false);
-
-        // ==========================================================
-        // ==========================================================
-
-        // Права
-        $memberHandler = xoops_getHandler('member');
-        $group_list    = $memberHandler->getGroupList();
-        $gpermHandler  = xoops_getHandler('groupperm');
-        $full_list     = array_keys($group_list);
-
-        // Права на просмотр
-        $groups_ids = [];
-        // Если мы редактируем
-        if (!$this->isNew()) {
-            $groups_ids        = $gpermHandler->getGroupIds('instruction_view', $this->getVar('cid'), $GLOBALS['xoopsModule']->getVar('mid'));
-            $groups_ids        = array_values($groups_ids);
-            $groups_instr_view = new XoopsFormCheckBox(_AM_INSTRUCTION_PERM_VIEW, 'groups_instr_view', $groups_ids);
-        } else {
-            $groups_instr_view = new XoopsFormCheckBox(_AM_INSTRUCTION_PERM_VIEW, 'groups_instr_view', $full_list);
-        }
-        $groups_instr_view->addOptionArray($group_list);
-        $form->addElement($groups_instr_view);
-
-        // Права на отправку
-        $groups_ids = [];
-        if (!$this->isNew()) {
-            $groups_ids          = $gpermHandler->getGroupIds('instruction_submit', $this->getVar('cid'), $GLOBALS['xoopsModule']->getVar('mid'));
-            $groups_ids          = array_values($groups_ids);
-            $groups_instr_submit = new XoopsFormCheckBox(_AM_INSTRUCTION_PERM_SUBMIT, 'groups_instr_submit', $groups_ids);
-        } else {
-            $groups_instr_submit = new XoopsFormCheckBox(_AM_INSTRUCTION_PERM_SUBMIT, 'groups_instr_submit', $full_list);
-        }
-        $groups_instr_submit->addOptionArray($group_list);
-        $form->addElement($groups_instr_submit);
-
-        // Права на редактирование
-        $groups_ids = [];
-        if (!$this->isNew()) {
-            $groups_ids        = $gpermHandler->getGroupIds('instruction_edit', $this->getVar('cid'), $GLOBALS['xoopsModule']->getVar('mid'));
-            $groups_ids        = array_values($groups_ids);
-            $groups_instr_edit = new XoopsFormCheckBox(_AM_INSTRUCTION_PERM_EDIT, 'groups_instr_edit', $groups_ids);
-        } else {
-            $groups_instr_edit = new XoopsFormCheckBox(_AM_INSTRUCTION_PERM_EDIT, 'groups_instr_edit', $full_list);
-        }
-        $groups_instr_edit->addOptionArray($group_list);
-        $form->addElement($groups_instr_edit);
-
-        // ==========================================================
-        // ==========================================================
-
-        // Если мы редактируем категорию
-        if (!$this->isNew()) {
-            $form->addElement(new XoopsFormHidden('cid', $this->getVar('cid')));
-            //$form->addElement( new XoopsFormHidden( 'catmodify', true));
-        }
-        //
-        $form->addElement(new XoopsFormHidden('op', 'savecat'));
-        // Кнопка
-        $button_tray = new XoopsFormElementTray('', '');
-        $submit_btn  = new XoopsFormButton('', 'submit', _SUBMIT, 'submit');
-        $button_tray->addElement($submit_btn);
-        $cancel_btn = new XoopsFormButton('', 'cancel', _CANCEL, 'cancel');
-        $cancel_btn->setExtra('onclick="javascript:history.go(-1);"');
-        $button_tray->addElement($cancel_btn);
-        $form->addElement($button_tray);
-
-        return $form;
-    }
+		// Родительская категория
+		$instructioncatHandler = xoops_getModuleHandler('category', 'instruction');
+		$criteria              = new CriteriaCompo();
+		// Если мы редактируем, то убрать текущую категорию из списка выбора родительской
+		if (!$this->isNew()) {
+			$criteria->add(new Criteria('cid', $this->getVar('cid'), '<>'));
+		}
+		$criteria->setSort('weight ASC, title');
+		$criteria->setOrder('ASC');
+		$instructioncat_arr = $instructioncatHandler->getall($criteria);
+		unset($criteria);
+		// Подключаем трей
+		include_once $GLOBALS['xoops']->path('class/tree.php');
+		$mytree = new XoopsObjectTree($instructioncat_arr, 'cid', 'pid');
+
+		// $form->addElement(new XoopsFormLabel(_AM_INSTRUCTION_PCATC, $mytree->makeSelBox('pid', 'title', '--', $this->getVar('pid'), true)));
+		$moduleDirName = basename(__DIR__);
+		if (false !== ($moduleHelper = Xmf\Module\Helper::getHelper($moduleDirName))) {
+		} else {
+			$moduleHelper = Xmf\Module\Helper::getHelper('system');
+		}
+		$module = $moduleHelper->getModule();
+
+		if (InstructionUtility::checkVerXoops($module, '2.5.9')) {
+			$mytree_select = $mytree->makeSelectElement('pid', 'title', '--', $this->getVar('pid'), true, 0, '', _AM_INSTRUCTION_PCATC);
+			$form->addElement($mytree_select);
+		} else {
+			$form->addElement(new XoopsFormLabel(_AM_INSTRUCTION_PCATC, $mytree->makeSelBox('pid', 'title', '--', $this->getVar('pid'), true)));
+		}
+
+		// Вес
+		$form->addElement(new XoopsFormText(_AM_INSTRUCTION_WEIGHTC, 'weight', 5, 5, $this->getVar('weight')), true);
+		// Мета-теги ключевых слов
+		$form->addElement(new XoopsFormText(_AM_INSTRUCTION_METAKEYWORDSC, 'metakeywords', 50, 255, $this->getVar('metakeywords')), false);
+		// Мета-теги описания
+		$form->addElement(new XoopsFormText(_AM_INSTRUCTION_METADESCRIPTIONC, 'metadescription', 50, 255, $this->getVar('metadescription')), false);
+
+		// ==========================================================
+		// ==========================================================
+
+		// Права
+		$memberHandler = xoops_getHandler('member');
+		$group_list    = $memberHandler->getGroupList();
+		$gpermHandler  = xoops_getHandler('groupperm');
+		$full_list     = array_keys($group_list);
+
+		// Права на просмотр
+		$groups_ids = [];
+		// Если мы редактируем
+		if (!$this->isNew()) {
+			$groups_ids        = $gpermHandler->getGroupIds('instruction_view', $this->getVar('cid'), $GLOBALS['xoopsModule']->getVar('mid'));
+			$groups_ids        = array_values($groups_ids);
+			$groups_instr_view = new XoopsFormCheckBox(_AM_INSTRUCTION_PERM_VIEW, 'groups_instr_view', $groups_ids);
+		} else {
+			$groups_instr_view = new XoopsFormCheckBox(_AM_INSTRUCTION_PERM_VIEW, 'groups_instr_view', $full_list);
+		}
+		$groups_instr_view->addOptionArray($group_list);
+		$form->addElement($groups_instr_view);
+
+		// Права на отправку
+		$groups_ids = [];
+		if (!$this->isNew()) {
+			$groups_ids          = $gpermHandler->getGroupIds('instruction_submit', $this->getVar('cid'), $GLOBALS['xoopsModule']->getVar('mid'));
+			$groups_ids          = array_values($groups_ids);
+			$groups_instr_submit = new XoopsFormCheckBox(_AM_INSTRUCTION_PERM_SUBMIT, 'groups_instr_submit', $groups_ids);
+		} else {
+			$groups_instr_submit = new XoopsFormCheckBox(_AM_INSTRUCTION_PERM_SUBMIT, 'groups_instr_submit', $full_list);
+		}
+		$groups_instr_submit->addOptionArray($group_list);
+		$form->addElement($groups_instr_submit);
+
+		// Права на редактирование
+		$groups_ids = [];
+		if (!$this->isNew()) {
+			$groups_ids        = $gpermHandler->getGroupIds('instruction_edit', $this->getVar('cid'), $GLOBALS['xoopsModule']->getVar('mid'));
+			$groups_ids        = array_values($groups_ids);
+			$groups_instr_edit = new XoopsFormCheckBox(_AM_INSTRUCTION_PERM_EDIT, 'groups_instr_edit', $groups_ids);
+		} else {
+			$groups_instr_edit = new XoopsFormCheckBox(_AM_INSTRUCTION_PERM_EDIT, 'groups_instr_edit', $full_list);
+		}
+		$groups_instr_edit->addOptionArray($group_list);
+		$form->addElement($groups_instr_edit);
+
+		// ==========================================================
+		// ==========================================================
+
+		// Если мы редактируем категорию
+		if (!$this->isNew()) {
+			$form->addElement(new XoopsFormHidden('cid', $this->getVar('cid')));
+			//$form->addElement( new XoopsFormHidden( 'catmodify', true));
+		}
+		//
+		$form->addElement(new XoopsFormHidden('op', 'savecat'));
+		// Кнопка
+		$button_tray = new XoopsFormElementTray('', '');
+		$submit_btn  = new XoopsFormButton('', 'submit', _SUBMIT, 'submit');
+		$button_tray->addElement($submit_btn);
+		$cancel_btn = new XoopsFormButton('', 'cancel', _CANCEL, 'cancel');
+		$cancel_btn->setExtra('onclick="javascript:history.go(-1);"');
+		$button_tray->addElement($cancel_btn);
+		$form->addElement($button_tray);
+
+		return $form;
+	}
 }
 
 class InstructionCategoryHandler extends XoopsPersistableObjectHandler
 {
-    public function __construct($db)
-    {
-        parent::__construct($db, 'instruction_cat', 'InstructionCategory', 'cid', 'title');
-    }
-
-    // Обновление даты обновления категории
-    public function updateDateupdated($cid = 0, $time = null)
-    {
-        // Если не передали время
-        $time = null === $time ? time() : (int)$time;
-        //
-        $sql = sprintf('UPDATE `%s` SET `dateupdated` = %u WHERE `cid` = %u', $this->table, $time, (int)$cid);
-        //
-        return $this->db->query($sql);
-    }
+	public function __construct($db)
+	{
+		parent::__construct($db, 'instruction_cat', 'InstructionCategory', 'cid', 'title');
+	}
+
+	// Обновление даты обновления категории
+	public function updateDateupdated($cid = 0, $time = null)
+	{
+		// Если не передали время
+		$time = null === $time ? time() : (int)$time;
+		//
+		$sql = sprintf('UPDATE `%s` SET `dateupdated` = %u WHERE `cid` = %u', $this->table, $time, (int)$cid);
+		//
+		return $this->db->query($sql);
+	}
 }

class/tree.php

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);
             }

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 +305 added lines, -305 removed lines

@@ -16,78 +16,78 @@ 
 //    {
 //    }
 
-    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 _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;
+	}
 }

index.php

Spacing +3 added lines, -3 removed lines

@@ -21,9 +21,9 @@ 
 // Стили
 $xoTheme->addStylesheet(XOOPS_URL . '/modules/' . $moduleDirName . '/assets/css/style.css');
 //
-$cid = isset($_GET['cid']) ? (int)$_GET['cid'] : 0;
+$cid = isset($_GET['cid']) ? (int) $_GET['cid'] : 0;
 //
-$start = isset($_GET['start']) ? (int)$_GET['start'] : 0;
+$start = isset($_GET['start']) ? (int) $_GET['start'] : 0;
 //
 $limit = xoops_getModuleOption('perpagemain', 'instruction');
 // Права на просмотр
@@ -52,7 +52,7 @@ 
 // Если есть категория
 if ($cid) {
     // Если нельзя просматривать эту категорию
-    if (!in_array($cid, $categories)) {
+    if ( ! in_array($cid, $categories)) {
         redirect_header('index.php', 3, _MD_INSTRUCTION_NOPERM_CAT);
     }
     $criteria->add(new Criteria('cid', $cid, '='));

Indentation +54 added lines, -54 removed lines

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

comment_new.php

Indentation +8 added lines, -8 removed lines

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