mambax7 / chess

class/Common/ServerStats.php

Indentation +62 added lines, -62 removed lines

@@ -19,66 +19,66 @@
  */
 trait ServerStats
 {
-    /**
-     * serverStats()
-     *
-     * @return string
-     */
-    public static function getServerStats()
-    {
-        $moduleDirName = \basename(dirname(__DIR__, 2));
-        $moduleDirNameUpper = mb_strtoupper($moduleDirName);
-        \xoops_loadLanguage('common', $moduleDirName);
-        $html = '';
-
-        //        $sql   = 'SELECT metavalue';
-        //        $sql   .= ' FROM ' . $GLOBALS['xoopsDB']->prefix('wfdownloads_meta');
-        //        $sql   .= " WHERE metakey='version' LIMIT 1";
-        //        $query = $GLOBALS['xoopsDB']->query($sql);
-        //        list($meta) = $GLOBALS['xoopsDB']->fetchRow($query);
-
-        $html .= '<fieldset>';
-        $html .= "<legend style='font-weight: bold; color: #900;'>" . \constant('CO_' . $moduleDirNameUpper . '_IMAGEINFO') . '</legend>';
-        $html .= "<div style='padding: 8px;'>";
-        //        $html .= '<div>' . constant('CO_' . $moduleDirNameUpper . '_METAVERSION') . $meta . "</div>";
-        //        $html .= "<br>";
-        //        $html .= "<br>";
-        $html .= '<div>' . \constant('CO_' . $moduleDirNameUpper . '_SPHPINI') . '</div>';
-        $html .= '<ul>';
-
-        $gdlib = \function_exists('gd_info') ? '<span style="color: #008000;">' . \constant('CO_' . $moduleDirNameUpper . '_GDON') . '</span>' : '<span style="color: #ff0000;">' . \constant('CO_' . $moduleDirNameUpper . '_GDOFF') . '</span>';
-
-        $html .= '<li>' . \constant('CO_' . $moduleDirNameUpper . '_GDLIBSTATUS') . $gdlib;
-
-        if (\function_exists('gd_info')) {
-            if (true === ($gdlib = gd_info())) {
-                $html .= '<li>' . \constant('CO_' . $moduleDirNameUpper . '_GDLIBVERSION') . '<b>' . $gdlib['GD Version'] . '</b>';
-            }
-        }
-
-        //    $safemode = ini_get('safe_mode') ? constant('CO_' . $moduleDirNameUpper . '_ON') . constant('CO_' . $moduleDirNameUpper . '_SAFEMODEPROBLEMS : constant('CO_' . $moduleDirNameUpper . '_OFF');
-
-        //    $html .= '<li>' . constant('CO_' . $moduleDirNameUpper . '_SAFEMODESTATUS . $safemode;
-
-        //    $registerglobals = (!ini_get('register_globals')) ? "<span style=\"color: #008000;\">" . constant('CO_' . $moduleDirNameUpper . '_OFF') . '</span>' : "<span style=\"color: #ff0000;\">" . constant('CO_' . $moduleDirNameUpper . '_ON') . '</span>';
-
-        //    $html .= '<li>' . constant('CO_' . $moduleDirNameUpper . '_REGISTERGLOBALS . $registerglobals;
-
-        $downloads = \ini_get('file_uploads') ? '<span style="color: #008000;">' . \constant('CO_' . $moduleDirNameUpper . '_ON') . '</span>' : '<span style="color: #ff0000;">' . \constant('CO_' . $moduleDirNameUpper . '_OFF') . '</span>';
-
-        $html .= '<li>' . \constant('CO_' . $moduleDirNameUpper . '_SERVERUPLOADSTATUS') . $downloads;
-        $html .= '<li>' . \constant('CO_' . $moduleDirNameUpper . '_MAXUPLOADSIZE') . ' <b><span style="color: #0000ff;">' . \ini_get('upload_max_filesize') . '</span></b>';
-        $html .= '<li>' . \constant('CO_' . $moduleDirNameUpper . '_MAXPOSTSIZE') . ' <b><span style="color: #0000ff;">' . \ini_get('post_max_size') . '</span></b>';
-        $html .= '<li>' . \constant('CO_' . $moduleDirNameUpper . '_MEMORYLIMIT') . ' <b><span style="color: #0000ff;">' . \ini_get('memory_limit') . '</span></b>';
-        $html .= '</ul>';
-        $html .= '<ul>';
-        $html .= '<li>' . \constant('CO_' . $moduleDirNameUpper . '_SERVERPATH') . ' <b>' . XOOPS_ROOT_PATH . '</b>';
-        $html .= '</ul>';
-        $html .= '<br>';
-        $html .= \constant('CO_' . $moduleDirNameUpper . '_UPLOADPATHDSC') . '';
-        $html .= '</div>';
-        $html .= '</fieldset><br>';
-
-        return $html;
-    }
+	/**
+	 * serverStats()
+	 *
+	 * @return string
+	 */
+	public static function getServerStats()
+	{
+		$moduleDirName = \basename(dirname(__DIR__, 2));
+		$moduleDirNameUpper = mb_strtoupper($moduleDirName);
+		\xoops_loadLanguage('common', $moduleDirName);
+		$html = '';
+
+		//        $sql   = 'SELECT metavalue';
+		//        $sql   .= ' FROM ' . $GLOBALS['xoopsDB']->prefix('wfdownloads_meta');
+		//        $sql   .= " WHERE metakey='version' LIMIT 1";
+		//        $query = $GLOBALS['xoopsDB']->query($sql);
+		//        list($meta) = $GLOBALS['xoopsDB']->fetchRow($query);
+
+		$html .= '<fieldset>';
+		$html .= "<legend style='font-weight: bold; color: #900;'>" . \constant('CO_' . $moduleDirNameUpper . '_IMAGEINFO') . '</legend>';
+		$html .= "<div style='padding: 8px;'>";
+		//        $html .= '<div>' . constant('CO_' . $moduleDirNameUpper . '_METAVERSION') . $meta . "</div>";
+		//        $html .= "<br>";
+		//        $html .= "<br>";
+		$html .= '<div>' . \constant('CO_' . $moduleDirNameUpper . '_SPHPINI') . '</div>';
+		$html .= '<ul>';
+
+		$gdlib = \function_exists('gd_info') ? '<span style="color: #008000;">' . \constant('CO_' . $moduleDirNameUpper . '_GDON') . '</span>' : '<span style="color: #ff0000;">' . \constant('CO_' . $moduleDirNameUpper . '_GDOFF') . '</span>';
+
+		$html .= '<li>' . \constant('CO_' . $moduleDirNameUpper . '_GDLIBSTATUS') . $gdlib;
+
+		if (\function_exists('gd_info')) {
+			if (true === ($gdlib = gd_info())) {
+				$html .= '<li>' . \constant('CO_' . $moduleDirNameUpper . '_GDLIBVERSION') . '<b>' . $gdlib['GD Version'] . '</b>';
+			}
+		}
+
+		//    $safemode = ini_get('safe_mode') ? constant('CO_' . $moduleDirNameUpper . '_ON') . constant('CO_' . $moduleDirNameUpper . '_SAFEMODEPROBLEMS : constant('CO_' . $moduleDirNameUpper . '_OFF');
+
+		//    $html .= '<li>' . constant('CO_' . $moduleDirNameUpper . '_SAFEMODESTATUS . $safemode;
+
+		//    $registerglobals = (!ini_get('register_globals')) ? "<span style=\"color: #008000;\">" . constant('CO_' . $moduleDirNameUpper . '_OFF') . '</span>' : "<span style=\"color: #ff0000;\">" . constant('CO_' . $moduleDirNameUpper . '_ON') . '</span>';
+
+		//    $html .= '<li>' . constant('CO_' . $moduleDirNameUpper . '_REGISTERGLOBALS . $registerglobals;
+
+		$downloads = \ini_get('file_uploads') ? '<span style="color: #008000;">' . \constant('CO_' . $moduleDirNameUpper . '_ON') . '</span>' : '<span style="color: #ff0000;">' . \constant('CO_' . $moduleDirNameUpper . '_OFF') . '</span>';
+
+		$html .= '<li>' . \constant('CO_' . $moduleDirNameUpper . '_SERVERUPLOADSTATUS') . $downloads;
+		$html .= '<li>' . \constant('CO_' . $moduleDirNameUpper . '_MAXUPLOADSIZE') . ' <b><span style="color: #0000ff;">' . \ini_get('upload_max_filesize') . '</span></b>';
+		$html .= '<li>' . \constant('CO_' . $moduleDirNameUpper . '_MAXPOSTSIZE') . ' <b><span style="color: #0000ff;">' . \ini_get('post_max_size') . '</span></b>';
+		$html .= '<li>' . \constant('CO_' . $moduleDirNameUpper . '_MEMORYLIMIT') . ' <b><span style="color: #0000ff;">' . \ini_get('memory_limit') . '</span></b>';
+		$html .= '</ul>';
+		$html .= '<ul>';
+		$html .= '<li>' . \constant('CO_' . $moduleDirNameUpper . '_SERVERPATH') . ' <b>' . XOOPS_ROOT_PATH . '</b>';
+		$html .= '</ul>';
+		$html .= '<br>';
+		$html .= \constant('CO_' . $moduleDirNameUpper . '_UPLOADPATHDSC') . '';
+		$html .= '</div>';
+		$html .= '</fieldset><br>';
+
+		return $html;
+	}
 }

Spacing +12 added lines, -12 removed lines

@@ -38,21 +38,21 @@ 
         //        list($meta) = $GLOBALS['xoopsDB']->fetchRow($query);
 
         $html .= '<fieldset>';
-        $html .= "<legend style='font-weight: bold; color: #900;'>" . \constant('CO_' . $moduleDirNameUpper . '_IMAGEINFO') . '</legend>';
+        $html .= "<legend style='font-weight: bold; color: #900;'>".\constant('CO_'.$moduleDirNameUpper.'_IMAGEINFO').'</legend>';
         $html .= "<div style='padding: 8px;'>";
         //        $html .= '<div>' . constant('CO_' . $moduleDirNameUpper . '_METAVERSION') . $meta . "</div>";
         //        $html .= "<br>";
         //        $html .= "<br>";
-        $html .= '<div>' . \constant('CO_' . $moduleDirNameUpper . '_SPHPINI') . '</div>';
+        $html .= '<div>'.\constant('CO_'.$moduleDirNameUpper.'_SPHPINI').'</div>';
         $html .= '<ul>';
 
-        $gdlib = \function_exists('gd_info') ? '<span style="color: #008000;">' . \constant('CO_' . $moduleDirNameUpper . '_GDON') . '</span>' : '<span style="color: #ff0000;">' . \constant('CO_' . $moduleDirNameUpper . '_GDOFF') . '</span>';
+        $gdlib = \function_exists('gd_info') ? '<span style="color: #008000;">'.\constant('CO_'.$moduleDirNameUpper.'_GDON').'</span>' : '<span style="color: #ff0000;">'.\constant('CO_'.$moduleDirNameUpper.'_GDOFF').'</span>';
 
-        $html .= '<li>' . \constant('CO_' . $moduleDirNameUpper . '_GDLIBSTATUS') . $gdlib;
+        $html .= '<li>'.\constant('CO_'.$moduleDirNameUpper.'_GDLIBSTATUS').$gdlib;
 
         if (\function_exists('gd_info')) {
             if (true === ($gdlib = gd_info())) {
-                $html .= '<li>' . \constant('CO_' . $moduleDirNameUpper . '_GDLIBVERSION') . '<b>' . $gdlib['GD Version'] . '</b>';
+                $html .= '<li>'.\constant('CO_'.$moduleDirNameUpper.'_GDLIBVERSION').'<b>'.$gdlib['GD Version'].'</b>';
             }
         }
 
@@ -64,18 +64,18 @@ 
 
         //    $html .= '<li>' . constant('CO_' . $moduleDirNameUpper . '_REGISTERGLOBALS . $registerglobals;
 
-        $downloads = \ini_get('file_uploads') ? '<span style="color: #008000;">' . \constant('CO_' . $moduleDirNameUpper . '_ON') . '</span>' : '<span style="color: #ff0000;">' . \constant('CO_' . $moduleDirNameUpper . '_OFF') . '</span>';
+        $downloads = \ini_get('file_uploads') ? '<span style="color: #008000;">'.\constant('CO_'.$moduleDirNameUpper.'_ON').'</span>' : '<span style="color: #ff0000;">'.\constant('CO_'.$moduleDirNameUpper.'_OFF').'</span>';
 
-        $html .= '<li>' . \constant('CO_' . $moduleDirNameUpper . '_SERVERUPLOADSTATUS') . $downloads;
-        $html .= '<li>' . \constant('CO_' . $moduleDirNameUpper . '_MAXUPLOADSIZE') . ' <b><span style="color: #0000ff;">' . \ini_get('upload_max_filesize') . '</span></b>';
-        $html .= '<li>' . \constant('CO_' . $moduleDirNameUpper . '_MAXPOSTSIZE') . ' <b><span style="color: #0000ff;">' . \ini_get('post_max_size') . '</span></b>';
-        $html .= '<li>' . \constant('CO_' . $moduleDirNameUpper . '_MEMORYLIMIT') . ' <b><span style="color: #0000ff;">' . \ini_get('memory_limit') . '</span></b>';
+        $html .= '<li>'.\constant('CO_'.$moduleDirNameUpper.'_SERVERUPLOADSTATUS').$downloads;
+        $html .= '<li>'.\constant('CO_'.$moduleDirNameUpper.'_MAXUPLOADSIZE').' <b><span style="color: #0000ff;">'.\ini_get('upload_max_filesize').'</span></b>';
+        $html .= '<li>'.\constant('CO_'.$moduleDirNameUpper.'_MAXPOSTSIZE').' <b><span style="color: #0000ff;">'.\ini_get('post_max_size').'</span></b>';
+        $html .= '<li>'.\constant('CO_'.$moduleDirNameUpper.'_MEMORYLIMIT').' <b><span style="color: #0000ff;">'.\ini_get('memory_limit').'</span></b>';
         $html .= '</ul>';
         $html .= '<ul>';
-        $html .= '<li>' . \constant('CO_' . $moduleDirNameUpper . '_SERVERPATH') . ' <b>' . XOOPS_ROOT_PATH . '</b>';
+        $html .= '<li>'.\constant('CO_'.$moduleDirNameUpper.'_SERVERPATH').' <b>'.XOOPS_ROOT_PATH.'</b>';
         $html .= '</ul>';
         $html .= '<br>';
-        $html .= \constant('CO_' . $moduleDirNameUpper . '_UPLOADPATHDSC') . '';
+        $html .= \constant('CO_'.$moduleDirNameUpper.'_UPLOADPATHDSC').'';
         $html .= '</div>';
         $html .= '</fieldset><br>';
 

class/Common/ModuleFeedback.php

Indentation +82 added lines, -82 removed lines

@@ -28,135 +28,135 @@
  */
 class ModuleFeedback extends \XoopsObject
 {
-    public $name = '';
-    public $email = '';
-    public $site = '';
-    public $type = '';
-    public $content = '';
+	public $name = '';
+	public $email = '';
+	public $site = '';
+	public $type = '';
+	public $content = '';
 
-    /**
-     * Constructor
-     *
-     * @param null
-     */
-    public function __construct()
-    {
-    }
+	/**
+	 * Constructor
+	 *
+	 * @param null
+	 */
+	public function __construct()
+	{
+	}
 
-    /**
-     * @static function &getInstance
-     *
-     * @param null
-     */
-    public static function getInstance()
-    {
-        static $instance = false;
+	/**
+	 * @static function &getInstance
+	 *
+	 * @param null
+	 */
+	public static function getInstance()
+	{
+		static $instance = false;
 
-        if (!$instance) {
-            $instance = new self();
-        }
-    }
+		if (!$instance) {
+			$instance = new self();
+		}
+	}
 
-    /**
-     * @public function getFormFeedback:
-     * provide form for sending a feedback to module author
-     * @param bool $action
-     * @return \XoopsThemeForm
-     */
-    public function getFormFeedback($action = false)
-    {
-        if (false === $action) {
-            $action = $_SERVER['REQUEST_URI'];
-        }
+	/**
+	 * @public function getFormFeedback:
+	 * provide form for sending a feedback to module author
+	 * @param bool $action
+	 * @return \XoopsThemeForm
+	 */
+	public function getFormFeedback($action = false)
+	{
+		if (false === $action) {
+			$action = $_SERVER['REQUEST_URI'];
+		}
 
-        $moduleDirName = \basename(dirname(__DIR__, 2));
+		$moduleDirName = \basename(dirname(__DIR__, 2));
 
-        $moduleDirNameUpper = mb_strtoupper($moduleDirName);
+		$moduleDirNameUpper = mb_strtoupper($moduleDirName);
 
-        // Get Theme Form
+		// Get Theme Form
 
-        \xoops_load('XoopsFormLoader');
+		\xoops_load('XoopsFormLoader');
 
-        $form = new \XoopsThemeForm(\constant('CO_' . $moduleDirNameUpper . '_' . 'FB_FORM_TITLE'), 'formfeedback', 'feedback.php', 'post', true);
+		$form = new \XoopsThemeForm(\constant('CO_' . $moduleDirNameUpper . '_' . 'FB_FORM_TITLE'), 'formfeedback', 'feedback.php', 'post', true);
 
-        $form->setExtra('enctype="multipart/form-data"');
+		$form->setExtra('enctype="multipart/form-data"');
 
-        $recipient = new \XoopsFormText(\constant('CO_' . $moduleDirNameUpper . '_' . 'FB_RECIPIENT'), 'recipient', 50, 255, $GLOBALS['xoopsModule']->getInfo('author_mail'));
+		$recipient = new \XoopsFormText(\constant('CO_' . $moduleDirNameUpper . '_' . 'FB_RECIPIENT'), 'recipient', 50, 255, $GLOBALS['xoopsModule']->getInfo('author_mail'));
 
-        $recipient->setExtra('disabled="disabled"');
+		$recipient->setExtra('disabled="disabled"');
 
-        $form->addElement($recipient);
+		$form->addElement($recipient);
 
-        $your_name = new \XoopsFormText(\constant('CO_' . $moduleDirNameUpper . '_' . 'FB_NAME'), 'your_name', 50, 255, $this->name);
+		$your_name = new \XoopsFormText(\constant('CO_' . $moduleDirNameUpper . '_' . 'FB_NAME'), 'your_name', 50, 255, $this->name);
 
-        $your_name->setExtra('placeholder="' . \constant('CO_' . $moduleDirNameUpper . '_' . 'FB_NAME_PLACEHOLER') . '"');
+		$your_name->setExtra('placeholder="' . \constant('CO_' . $moduleDirNameUpper . '_' . 'FB_NAME_PLACEHOLER') . '"');
 
-        $form->addElement($your_name);
+		$form->addElement($your_name);
 
-        $your_site = new \XoopsFormText(\constant('CO_' . $moduleDirNameUpper . '_' . 'FB_SITE'), 'your_site', 50, 255, $this->site);
+		$your_site = new \XoopsFormText(\constant('CO_' . $moduleDirNameUpper . '_' . 'FB_SITE'), 'your_site', 50, 255, $this->site);
 
-        $your_site->setExtra('placeholder="' . \constant('CO_' . $moduleDirNameUpper . '_' . 'FB_SITE_PLACEHOLER') . '"');
+		$your_site->setExtra('placeholder="' . \constant('CO_' . $moduleDirNameUpper . '_' . 'FB_SITE_PLACEHOLER') . '"');
 
-        $form->addElement($your_site);
+		$form->addElement($your_site);
 
-        $your_mail = new \XoopsFormText(\constant('CO_' . $moduleDirNameUpper . '_' . 'FB_MAIL'), 'your_mail', 50, 255, $this->email);
+		$your_mail = new \XoopsFormText(\constant('CO_' . $moduleDirNameUpper . '_' . 'FB_MAIL'), 'your_mail', 50, 255, $this->email);
 
-        $your_mail->setExtra('placeholder="' . \constant('CO_' . $moduleDirNameUpper . '_' . 'FB_MAIL_PLACEHOLER') . '"');
+		$your_mail->setExtra('placeholder="' . \constant('CO_' . $moduleDirNameUpper . '_' . 'FB_MAIL_PLACEHOLER') . '"');
 
-        $form->addElement($your_mail);
+		$form->addElement($your_mail);
 
-        $fbtypeSelect = new \XoopsFormSelect(\constant('CO_' . $moduleDirNameUpper . '_' . 'FB_TYPE'), 'fb_type', $this->type);
+		$fbtypeSelect = new \XoopsFormSelect(\constant('CO_' . $moduleDirNameUpper . '_' . 'FB_TYPE'), 'fb_type', $this->type);
 
-        $fbtypeSelect->addOption('', '');
+		$fbtypeSelect->addOption('', '');
 
-        $fbtypeSelect->addOption(\constant('CO_' . $moduleDirNameUpper . '_' . 'FB_TYPE_SUGGESTION'), \constant('CO_' . $moduleDirNameUpper . '_' . 'FB_TYPE_SUGGESTION'));
+		$fbtypeSelect->addOption(\constant('CO_' . $moduleDirNameUpper . '_' . 'FB_TYPE_SUGGESTION'), \constant('CO_' . $moduleDirNameUpper . '_' . 'FB_TYPE_SUGGESTION'));
 
-        $fbtypeSelect->addOption(\constant('CO_' . $moduleDirNameUpper . '_' . 'FB_TYPE_BUGS'), \constant('CO_' . $moduleDirNameUpper . '_' . 'FB_TYPE_BUGS'));
+		$fbtypeSelect->addOption(\constant('CO_' . $moduleDirNameUpper . '_' . 'FB_TYPE_BUGS'), \constant('CO_' . $moduleDirNameUpper . '_' . 'FB_TYPE_BUGS'));
 
-        $fbtypeSelect->addOption(\constant('CO_' . $moduleDirNameUpper . '_' . 'FB_TYPE_TESTIMONIAL'), \constant('CO_' . $moduleDirNameUpper . '_' . 'FB_TYPE_TESTIMONIAL'));
+		$fbtypeSelect->addOption(\constant('CO_' . $moduleDirNameUpper . '_' . 'FB_TYPE_TESTIMONIAL'), \constant('CO_' . $moduleDirNameUpper . '_' . 'FB_TYPE_TESTIMONIAL'));
 
-        $fbtypeSelect->addOption(\constant('CO_' . $moduleDirNameUpper . '_' . 'FB_TYPE_FEATURES'), \constant('CO_' . $moduleDirNameUpper . '_' . 'FB_TYPE_FEATURES'));
+		$fbtypeSelect->addOption(\constant('CO_' . $moduleDirNameUpper . '_' . 'FB_TYPE_FEATURES'), \constant('CO_' . $moduleDirNameUpper . '_' . 'FB_TYPE_FEATURES'));
 
-        $fbtypeSelect->addOption(\constant('CO_' . $moduleDirNameUpper . '_' . 'FB_TYPE_OTHERS'), \constant('CO_' . $moduleDirNameUpper . '_' . 'FB_TYPE_OTHERS'));
+		$fbtypeSelect->addOption(\constant('CO_' . $moduleDirNameUpper . '_' . 'FB_TYPE_OTHERS'), \constant('CO_' . $moduleDirNameUpper . '_' . 'FB_TYPE_OTHERS'));
 
-        $form->addElement($fbtypeSelect, true);
+		$form->addElement($fbtypeSelect, true);
 
-        $editorConfigs = [];
+		$editorConfigs = [];
 
-        $editorConfigs['name'] = 'fb_content';
+		$editorConfigs['name'] = 'fb_content';
 
-        $editorConfigs['value'] = $this->content;
+		$editorConfigs['value'] = $this->content;
 
-        $editorConfigs['rows'] = 5;
+		$editorConfigs['rows'] = 5;
 
-        $editorConfigs['cols'] = 40;
+		$editorConfigs['cols'] = 40;
 
-        $editorConfigs['width'] = '100%';
+		$editorConfigs['width'] = '100%';
 
-        $editorConfigs['height'] = '400px';
+		$editorConfigs['height'] = '400px';
 
-        /** @var \XoopsModuleHandler $moduleHandler */
+		/** @var \XoopsModuleHandler $moduleHandler */
 
-        $moduleHandler = \xoops_getHandler('module');
+		$moduleHandler = \xoops_getHandler('module');
 
-        $module = $moduleHandler->getByDirname('system');
+		$module = $moduleHandler->getByDirname('system');
 
-        /** @var \XoopsConfigHandler $configHandler */
+		/** @var \XoopsConfigHandler $configHandler */
 
-        $configHandler = \xoops_getHandler('config');
+		$configHandler = \xoops_getHandler('config');
 
-        $config = $configHandler->getConfigsByCat(0, $module->getVar('mid'));
+		$config = $configHandler->getConfigsByCat(0, $module->getVar('mid'));
 
-        $editorConfigs['editor'] = $config['general_editor'];
+		$editorConfigs['editor'] = $config['general_editor'];
 
-        $editor = new \XoopsFormEditor(\constant('CO_' . $moduleDirNameUpper . '_' . 'FB_TYPE_CONTENT'), 'fb_content', $editorConfigs);
+		$editor = new \XoopsFormEditor(\constant('CO_' . $moduleDirNameUpper . '_' . 'FB_TYPE_CONTENT'), 'fb_content', $editorConfigs);
 
-        $form->addElement($editor, true);
+		$form->addElement($editor, true);
 
-        $form->addElement(new \XoopsFormHidden('op', 'send'));
+		$form->addElement(new \XoopsFormHidden('op', 'send'));
 
-        $form->addElement(new \XoopsFormButtonTray('', \_SUBMIT, 'submit', '', false));
+		$form->addElement(new \XoopsFormButtonTray('', \_SUBMIT, 'submit', '', false));
 
-        return $form;
-    }
+		return $form;
+	}
 }

class/Common/VersionChecks.php

Indentation +114 added lines, -114 removed lines

@@ -19,170 +19,170 @@
  */
 trait VersionChecks
 {
-    /**
-     * Verifies XOOPS version meets minimum requirements for this module
-     * @static
-     *
-     * @param null|string $requiredVer
-     * @return bool true if meets requirements, false if not
-     */
-    public static function checkVerXoops(?\XoopsModule $module = null, $requiredVer = null)
-    {
-        $moduleDirName = \basename(dirname(__DIR__, 2));
+	/**
+	 * Verifies XOOPS version meets minimum requirements for this module
+	 * @static
+	 *
+	 * @param null|string $requiredVer
+	 * @return bool true if meets requirements, false if not
+	 */
+	public static function checkVerXoops(?\XoopsModule $module = null, $requiredVer = null)
+	{
+		$moduleDirName = \basename(dirname(__DIR__, 2));
 
-        $moduleDirNameUpper = mb_strtoupper($moduleDirName);
+		$moduleDirNameUpper = mb_strtoupper($moduleDirName);
 
-        if (null === $module) {
-            $module = \XoopsModule::getByDirname($moduleDirName);
-        }
+		if (null === $module) {
+			$module = \XoopsModule::getByDirname($moduleDirName);
+		}
 
-        \xoops_loadLanguage('admin', $moduleDirName);
+		\xoops_loadLanguage('admin', $moduleDirName);
 
-        \xoops_loadLanguage('common', $moduleDirName);
+		\xoops_loadLanguage('common', $moduleDirName);
 
-        //check for minimum XOOPS version
-        $currentVer = mb_substr(\XOOPS_VERSION, 6); // get the numeric part of string
-        if (null === $requiredVer) {
-            $requiredVer = '' . $module->getInfo('min_xoops'); //making sure it's a string
-        }
+		//check for minimum XOOPS version
+		$currentVer = mb_substr(\XOOPS_VERSION, 6); // get the numeric part of string
+		if (null === $requiredVer) {
+			$requiredVer = '' . $module->getInfo('min_xoops'); //making sure it's a string
+		}
 
-        $success = true;
+		$success = true;
 
-        if (\version_compare($currentVer, $requiredVer, '<')) {
-            $success = false;
+		if (\version_compare($currentVer, $requiredVer, '<')) {
+			$success = false;
 
-            $module->setErrors(\sprintf(\constant('CO_' . $moduleDirNameUpper . '_ERROR_BAD_XOOPS'), $requiredVer, $currentVer));
-        }
+			$module->setErrors(\sprintf(\constant('CO_' . $moduleDirNameUpper . '_ERROR_BAD_XOOPS'), $requiredVer, $currentVer));
+		}
 
-        return $success;
-    }
+		return $success;
+	}
 
-    /**
-     * Verifies PHP version meets minimum requirements for this module
-     * @static
-     *
-     * @return bool true if meets requirements, false if not
-     */
-    public static function checkVerPhp(?\XoopsModule $module = null)
-    {
-        $moduleDirName = \basename(dirname(__DIR__, 2));
+	/**
+	 * Verifies PHP version meets minimum requirements for this module
+	 * @static
+	 *
+	 * @return bool true if meets requirements, false if not
+	 */
+	public static function checkVerPhp(?\XoopsModule $module = null)
+	{
+		$moduleDirName = \basename(dirname(__DIR__, 2));
 
-        $moduleDirNameUpper = mb_strtoupper($moduleDirName);
+		$moduleDirNameUpper = mb_strtoupper($moduleDirName);
 
-        if (null === $module) {
-            $module = \XoopsModule::getByDirname($moduleDirName);
-        }
+		if (null === $module) {
+			$module = \XoopsModule::getByDirname($moduleDirName);
+		}
 
-        \xoops_loadLanguage('admin', $moduleDirName);
+		\xoops_loadLanguage('admin', $moduleDirName);
 
-        \xoops_loadLanguage('common', $moduleDirName);
+		\xoops_loadLanguage('common', $moduleDirName);
 
-        // check for minimum PHP version
+		// check for minimum PHP version
 
-        $success = true;
+		$success = true;
 
-        $verNum = \PHP_VERSION;
+		$verNum = \PHP_VERSION;
 
-        $reqVer = &$module->getInfo('min_php');
+		$reqVer = &$module->getInfo('min_php');
 
-        if (false !== $reqVer && '' !== $reqVer && !\is_array($reqVer)) {
-            if (\version_compare($verNum, $reqVer, '<')) {
-                $module->setErrors(\sprintf(\constant('CO_' . $moduleDirNameUpper . '_ERROR_BAD_PHP'), $reqVer, $verNum));
+		if (false !== $reqVer && '' !== $reqVer && !\is_array($reqVer)) {
+			if (\version_compare($verNum, $reqVer, '<')) {
+				$module->setErrors(\sprintf(\constant('CO_' . $moduleDirNameUpper . '_ERROR_BAD_PHP'), $reqVer, $verNum));
 
-                $success = false;
-            }
-        }
+				$success = false;
+			}
+		}
 
-        return $success;
-    }
+		return $success;
+	}
 
-    /**
-     * compares current module version with latest GitHub release
-     * @static
-     * @param \Xmf\Module\Helper $helper
-     * @param string|null        $source
-     * @param string|null        $default
-     *
-     * @return string|array info about the latest module version, if newer
-     */
-    public static function checkVerModule($helper, $source = 'github', $default = 'master')
-    {
-        $moduleDirName = \basename(dirname(__DIR__, 2));
+	/**
+	 * compares current module version with latest GitHub release
+	 * @static
+	 * @param \Xmf\Module\Helper $helper
+	 * @param string|null        $source
+	 * @param string|null        $default
+	 *
+	 * @return string|array info about the latest module version, if newer
+	 */
+	public static function checkVerModule($helper, $source = 'github', $default = 'master')
+	{
+		$moduleDirName = \basename(dirname(__DIR__, 2));
 
-        $moduleDirNameUpper = mb_strtoupper($moduleDirName);
+		$moduleDirNameUpper = mb_strtoupper($moduleDirName);
 
-        $update = '';
+		$update = '';
 
-        $repository = 'XoopsModules25x/' . $moduleDirName;
+		$repository = 'XoopsModules25x/' . $moduleDirName;
 
-        //        $repository         = 'XoopsModules25x/publisher'; //for testing only
+		//        $repository         = 'XoopsModules25x/publisher'; //for testing only
 
-        $ret = '';
+		$ret = '';
 
-        $infoReleasesUrl = "https://api.github.com/repos/$repository/releases";
+		$infoReleasesUrl = "https://api.github.com/repos/$repository/releases";
 
-        if ('github' === $source) {
-            if (\function_exists('curl_init') && false !== ($curlHandle = \curl_init())) {
-                \curl_setopt($curlHandle, \CURLOPT_URL, $infoReleasesUrl);
+		if ('github' === $source) {
+			if (\function_exists('curl_init') && false !== ($curlHandle = \curl_init())) {
+				\curl_setopt($curlHandle, \CURLOPT_URL, $infoReleasesUrl);
 
-                \curl_setopt($curlHandle, \CURLOPT_RETURNTRANSFER, true);
+				\curl_setopt($curlHandle, \CURLOPT_RETURNTRANSFER, true);
 
-                \curl_setopt($curlHandle, \CURLOPT_SSL_VERIFYPEER, true);
+				\curl_setopt($curlHandle, \CURLOPT_SSL_VERIFYPEER, true);
 
-                \curl_setopt($curlHandle, \CURLOPT_HTTPHEADER, ["User-Agent:Publisher\r\n"]);
+				\curl_setopt($curlHandle, \CURLOPT_HTTPHEADER, ["User-Agent:Publisher\r\n"]);
 
-                $curlReturn = \curl_exec($curlHandle);
+				$curlReturn = \curl_exec($curlHandle);
 
-                if (false === $curlReturn) {
-                    \trigger_error(\curl_error($curlHandle));
-                } elseif (false !== \mb_strpos($curlReturn, 'Not Found')) {
-                    \trigger_error('Repository Not Found: ' . $infoReleasesUrl);
-                } else {
-                    $file = \json_decode($curlReturn, false);
+				if (false === $curlReturn) {
+					\trigger_error(\curl_error($curlHandle));
+				} elseif (false !== \mb_strpos($curlReturn, 'Not Found')) {
+					\trigger_error('Repository Not Found: ' . $infoReleasesUrl);
+				} else {
+					$file = \json_decode($curlReturn, false);
 
-                    $latestVersionLink = \sprintf("https://github.com/$repository/archive/%s.zip", $file ? \reset($file)->tag_name : $default);
+					$latestVersionLink = \sprintf("https://github.com/$repository/archive/%s.zip", $file ? \reset($file)->tag_name : $default);
 
-                    $latestVersion = $file[0]->tag_name;
+					$latestVersion = $file[0]->tag_name;
 
-                    $prerelease = $file[0]->prerelease;
+					$prerelease = $file[0]->prerelease;
 
-                    if ('master' !== $latestVersionLink) {
-                        $update = \constant('CO_' . $moduleDirNameUpper . '_' . 'NEW_VERSION') . $latestVersion;
-                    }
+					if ('master' !== $latestVersionLink) {
+						$update = \constant('CO_' . $moduleDirNameUpper . '_' . 'NEW_VERSION') . $latestVersion;
+					}
 
-                    //"PHP-standardized" version
+					//"PHP-standardized" version
 
-                    $latestVersion = mb_strtolower($latestVersion);
+					$latestVersion = mb_strtolower($latestVersion);
 
-                    if (false !== mb_strpos($latestVersion, 'final')) {
-                        $latestVersion = \str_replace('_', '', mb_strtolower($latestVersion));
+					if (false !== mb_strpos($latestVersion, 'final')) {
+						$latestVersion = \str_replace('_', '', mb_strtolower($latestVersion));
 
-                        $latestVersion = \str_replace('final', '', mb_strtolower($latestVersion));
-                    }
+						$latestVersion = \str_replace('final', '', mb_strtolower($latestVersion));
+					}
 
-                    $moduleVersion = $helper->getModule()->getInfo('version') . '_' . $helper->getModule()->getInfo('module_status');
+					$moduleVersion = $helper->getModule()->getInfo('version') . '_' . $helper->getModule()->getInfo('module_status');
 
-                    //"PHP-standardized" version
+					//"PHP-standardized" version
 
-                    $moduleVersion = \str_replace(' ', '', mb_strtolower($moduleVersion));
+					$moduleVersion = \str_replace(' ', '', mb_strtolower($moduleVersion));
 
-                    //                    $moduleVersion = '1.0'; //for testing only
+					//                    $moduleVersion = '1.0'; //for testing only
 
-                    //                    $moduleDirName = 'publisher'; //for testing only
+					//                    $moduleDirName = 'publisher'; //for testing only
 
-                    if (!$prerelease && \version_compare($moduleVersion, $latestVersion, '<')) {
-                        $ret = [];
+					if (!$prerelease && \version_compare($moduleVersion, $latestVersion, '<')) {
+						$ret = [];
 
-                        $ret[] = $update;
+						$ret[] = $update;
 
-                        $ret[] = $latestVersionLink;
-                    }
-                }
+						$ret[] = $latestVersionLink;
+					}
+				}
 
-                \curl_close($curlHandle);
-            }
-        }
+				\curl_close($curlHandle);
+			}
+		}
 
-        return $ret;
-    }
+		return $ret;
+	}
 }

class/Common/FileChecker.php

Indentation +135 added lines, -135 removed lines

@@ -36,146 +36,146 @@
  */
 class FileChecker
 {
-    /**
-     * @param string      $file_path
-     * @param string|null $original_file_path
-     * @param string      $redirectFile
-     * @return bool|string
-     */
-    public static function getFileStatus($file_path, $original_file_path, $redirectFile)
-    {
-        global $pathIcon16;
+	/**
+	 * @param string      $file_path
+	 * @param string|null $original_file_path
+	 * @param string      $redirectFile
+	 * @return bool|string
+	 */
+	public static function getFileStatus($file_path, $original_file_path, $redirectFile)
+	{
+		global $pathIcon16;
 
-        if (empty($file_path)) {
-            return false;
-        }
+		if (empty($file_path)) {
+			return false;
+		}
 
-        if (null === $redirectFile) {
-            $redirectFile = $_SERVER['SCRIPT_NAME'];
-        }
-
-        $moduleDirName = \basename(dirname(__DIR__, 2));
-
-        $moduleDirNameUpper = mb_strtoupper($moduleDirName);
-
-        if (null === $original_file_path) {
-            if (self::fileExists($file_path)) {
-                $path_status = "<img src='$pathIcon16/1.png' >";
-
-                $path_status .= "$file_path (" . \constant('CO_' . $moduleDirNameUpper . '_' . 'FC_AVAILABLE') . ') ';
-            } else {
-                $path_status = "<img src='$pathIcon16/0.png' >";
-
-                $path_status .= "$file_path (" . \constant('CO_' . $moduleDirNameUpper . '_' . 'FC_NOTAVAILABLE') . ') ';
-            }
-        } else {
-            if (self::compareFiles($file_path, $original_file_path)) {
-                $path_status = "<img src='$pathIcon16/1.png' >";
-
-                $path_status .= "$file_path (" . \constant('CO_' . $moduleDirNameUpper . '_' . 'FC_AVAILABLE') . ') ';
-            } else {
-                $path_status = "<img src='$pathIcon16/0.png' >";
-
-                $path_status .= "$file_path (" . \constant('CO_' . $moduleDirNameUpper . '_' . 'FC_NOTAVAILABLE') . ') ';
-
-                $path_status .= "<form action='" . $_SERVER['SCRIPT_NAME'] . "' method='post'>";
-
-                $path_status .= "<input type='hidden' name='op' value='copyfile'>";
-
-                $path_status .= "<input type='hidden' name='file_path' value='$file_path'>";
-
-                $path_status .= "<input type='hidden' name='original_file_path' value='$original_file_path'>";
-
-                $path_status .= "<input type='hidden' name='redirect' value='$redirectFile'>";
-
-                $path_status .= "<button class='submit' onClick='this.form.submit();'>" . \constant('CO_' . $moduleDirNameUpper . '_' . 'FC_CREATETHEFILE') . '</button>';
-
-                $path_status .= '</form>';
-            }
-        }
-
-        return $path_status;
-    }
-
-    /**
-     * @param   $source_path
-     * @param   $destination_path
-     *
-     * @return bool
-     */
-    public static function copyFile($source_path, $destination_path)
-    {
-        $source_path = \str_replace('..', '', $source_path);
-
-        $destination_path = \str_replace('..', '', $destination_path);
-
-        return @\copy($source_path, $destination_path);
-    }
-
-    /**
-     * @param   $file1_path
-     * @param   $file2_path
-     *
-     * @return bool
-     */
-    public static function compareFiles($file1_path, $file2_path)
-    {
-        if (!self::fileExists($file1_path) || !self::fileExists($file2_path)) {
-            return false;
-        }
-
-        if (\filetype($file1_path) !== \filetype($file2_path)) {
-            return false;
-        }
-
-        if (\filesize($file1_path) !== \filesize($file2_path)) {
-            return false;
-        }
-
-        $crc1 = mb_strtoupper(\dechex(\crc32(file_get_contents($file1_path))));
-
-        $crc2 = mb_strtoupper(\dechex(\crc32(file_get_contents($file2_path))));
-
-        return !($crc1 !== $crc2);
-    }
-
-    /**
-     * @param   $file_path
-     *
-     * @return bool
-     */
-    public static function fileExists($file_path)
-    {
-        return \is_file($file_path);
-    }
+		if (null === $redirectFile) {
+			$redirectFile = $_SERVER['SCRIPT_NAME'];
+		}
+
+		$moduleDirName = \basename(dirname(__DIR__, 2));
+
+		$moduleDirNameUpper = mb_strtoupper($moduleDirName);
+
+		if (null === $original_file_path) {
+			if (self::fileExists($file_path)) {
+				$path_status = "<img src='$pathIcon16/1.png' >";
+
+				$path_status .= "$file_path (" . \constant('CO_' . $moduleDirNameUpper . '_' . 'FC_AVAILABLE') . ') ';
+			} else {
+				$path_status = "<img src='$pathIcon16/0.png' >";
+
+				$path_status .= "$file_path (" . \constant('CO_' . $moduleDirNameUpper . '_' . 'FC_NOTAVAILABLE') . ') ';
+			}
+		} else {
+			if (self::compareFiles($file_path, $original_file_path)) {
+				$path_status = "<img src='$pathIcon16/1.png' >";
+
+				$path_status .= "$file_path (" . \constant('CO_' . $moduleDirNameUpper . '_' . 'FC_AVAILABLE') . ') ';
+			} else {
+				$path_status = "<img src='$pathIcon16/0.png' >";
+
+				$path_status .= "$file_path (" . \constant('CO_' . $moduleDirNameUpper . '_' . 'FC_NOTAVAILABLE') . ') ';
+
+				$path_status .= "<form action='" . $_SERVER['SCRIPT_NAME'] . "' method='post'>";
+
+				$path_status .= "<input type='hidden' name='op' value='copyfile'>";
+
+				$path_status .= "<input type='hidden' name='file_path' value='$file_path'>";
+
+				$path_status .= "<input type='hidden' name='original_file_path' value='$original_file_path'>";
+
+				$path_status .= "<input type='hidden' name='redirect' value='$redirectFile'>";
+
+				$path_status .= "<button class='submit' onClick='this.form.submit();'>" . \constant('CO_' . $moduleDirNameUpper . '_' . 'FC_CREATETHEFILE') . '</button>';
+
+				$path_status .= '</form>';
+			}
+		}
+
+		return $path_status;
+	}
+
+	/**
+	 * @param   $source_path
+	 * @param   $destination_path
+	 *
+	 * @return bool
+	 */
+	public static function copyFile($source_path, $destination_path)
+	{
+		$source_path = \str_replace('..', '', $source_path);
+
+		$destination_path = \str_replace('..', '', $destination_path);
+
+		return @\copy($source_path, $destination_path);
+	}
+
+	/**
+	 * @param   $file1_path
+	 * @param   $file2_path
+	 *
+	 * @return bool
+	 */
+	public static function compareFiles($file1_path, $file2_path)
+	{
+		if (!self::fileExists($file1_path) || !self::fileExists($file2_path)) {
+			return false;
+		}
+
+		if (\filetype($file1_path) !== \filetype($file2_path)) {
+			return false;
+		}
+
+		if (\filesize($file1_path) !== \filesize($file2_path)) {
+			return false;
+		}
+
+		$crc1 = mb_strtoupper(\dechex(\crc32(file_get_contents($file1_path))));
+
+		$crc2 = mb_strtoupper(\dechex(\crc32(file_get_contents($file2_path))));
+
+		return !($crc1 !== $crc2);
+	}
+
+	/**
+	 * @param   $file_path
+	 *
+	 * @return bool
+	 */
+	public static function fileExists($file_path)
+	{
+		return \is_file($file_path);
+	}
 
-    /**
-     * @param     $target
-     * @param int $mode
-     *
-     * @return bool
-     */
-    public static function setFilePermissions($target, $mode = 0777)
-    {
-        $target = \str_replace('..', '', $target);
-
-        return @\chmod($target, (int)$mode);
-    }
+	/**
+	 * @param     $target
+	 * @param int $mode
+	 *
+	 * @return bool
+	 */
+	public static function setFilePermissions($target, $mode = 0777)
+	{
+		$target = \str_replace('..', '', $target);
+
+		return @\chmod($target, (int)$mode);
+	}
 }
 
 $op = Request::getString('op', '', 'POST');
 switch ($op) {
-    case 'copyfile':
-        if (\Xmf\Request::hasVar('original_file_path', 'POST')) {
-            $original_file_path = $_POST['original_file_path'];
-        }
-        if (\Xmf\Request::hasVar('file_path', 'POST')) {
-            $file_path = $_POST['file_path'];
-        }
-        if (\Xmf\Request::hasVar('redirect', 'POST')) {
-            $redirect = $_POST['redirect'];
-        }
-        $msg = FileChecker::copyFile($original_file_path, $file_path) ? \constant('CO_' . $moduleDirNameUpper . '_' . 'FC_FILECOPIED') : \constant('CO_' . $moduleDirNameUpper . '_' . 'FC_FILENOTCOPIED');
-        \redirect_header($redirect, 2, $msg . ': ' . $file_path);
-        break;
+	case 'copyfile':
+		if (\Xmf\Request::hasVar('original_file_path', 'POST')) {
+			$original_file_path = $_POST['original_file_path'];
+		}
+		if (\Xmf\Request::hasVar('file_path', 'POST')) {
+			$file_path = $_POST['file_path'];
+		}
+		if (\Xmf\Request::hasVar('redirect', 'POST')) {
+			$redirect = $_POST['redirect'];
+		}
+		$msg = FileChecker::copyFile($original_file_path, $file_path) ? \constant('CO_' . $moduleDirNameUpper . '_' . 'FC_FILECOPIED') : \constant('CO_' . $moduleDirNameUpper . '_' . 'FC_FILENOTCOPIED');
+		\redirect_header($redirect, 2, $msg . ': ' . $file_path);
+		break;
 }

Spacing +10 added lines, -10 removed lines

@@ -25,7 +25,7 @@ 
 
 //defined('XOOPS_ROOT_PATH') || die('XOOPS root path not defined');
 
-require_once dirname(__DIR__, 4) . '/mainfile.php';
+require_once dirname(__DIR__, 4).'/mainfile.php';
 $moduleDirName      = \basename(dirname(__DIR__, 2));
 $moduleDirNameUpper = mb_strtoupper($moduleDirName);
 \xoops_loadLanguage('filechecker', $moduleDirName);
@@ -62,23 +62,23 @@ 
             if (self::fileExists($file_path)) {
                 $path_status = "<img src='$pathIcon16/1.png' >";
 
-                $path_status .= "$file_path (" . \constant('CO_' . $moduleDirNameUpper . '_' . 'FC_AVAILABLE') . ') ';
+                $path_status .= "$file_path (".\constant('CO_'.$moduleDirNameUpper.'_'.'FC_AVAILABLE').') ';
             } else {
                 $path_status = "<img src='$pathIcon16/0.png' >";
 
-                $path_status .= "$file_path (" . \constant('CO_' . $moduleDirNameUpper . '_' . 'FC_NOTAVAILABLE') . ') ';
+                $path_status .= "$file_path (".\constant('CO_'.$moduleDirNameUpper.'_'.'FC_NOTAVAILABLE').') ';
             }
         } else {
             if (self::compareFiles($file_path, $original_file_path)) {
                 $path_status = "<img src='$pathIcon16/1.png' >";
 
-                $path_status .= "$file_path (" . \constant('CO_' . $moduleDirNameUpper . '_' . 'FC_AVAILABLE') . ') ';
+                $path_status .= "$file_path (".\constant('CO_'.$moduleDirNameUpper.'_'.'FC_AVAILABLE').') ';
             } else {
                 $path_status = "<img src='$pathIcon16/0.png' >";
 
-                $path_status .= "$file_path (" . \constant('CO_' . $moduleDirNameUpper . '_' . 'FC_NOTAVAILABLE') . ') ';
+                $path_status .= "$file_path (".\constant('CO_'.$moduleDirNameUpper.'_'.'FC_NOTAVAILABLE').') ';
 
-                $path_status .= "<form action='" . $_SERVER['SCRIPT_NAME'] . "' method='post'>";
+                $path_status .= "<form action='".$_SERVER['SCRIPT_NAME']."' method='post'>";
 
                 $path_status .= "<input type='hidden' name='op' value='copyfile'>";
 
@@ -88,7 +88,7 @@ 
 
                 $path_status .= "<input type='hidden' name='redirect' value='$redirectFile'>";
 
-                $path_status .= "<button class='submit' onClick='this.form.submit();'>" . \constant('CO_' . $moduleDirNameUpper . '_' . 'FC_CREATETHEFILE') . '</button>';
+                $path_status .= "<button class='submit' onClick='this.form.submit();'>".\constant('CO_'.$moduleDirNameUpper.'_'.'FC_CREATETHEFILE').'</button>';
 
                 $path_status .= '</form>';
             }
@@ -159,7 +159,7 @@ 
     {
         $target = \str_replace('..', '', $target);
 
-        return @\chmod($target, (int)$mode);
+        return @\chmod($target, (int) $mode);
     }
 }
 
@@ -175,7 +175,7 @@ 
         if (\Xmf\Request::hasVar('redirect', 'POST')) {
             $redirect = $_POST['redirect'];
         }
-        $msg = FileChecker::copyFile($original_file_path, $file_path) ? \constant('CO_' . $moduleDirNameUpper . '_' . 'FC_FILECOPIED') : \constant('CO_' . $moduleDirNameUpper . '_' . 'FC_FILENOTCOPIED');
-        \redirect_header($redirect, 2, $msg . ': ' . $file_path);
+        $msg = FileChecker::copyFile($original_file_path, $file_path) ? \constant('CO_'.$moduleDirNameUpper.'_'.'FC_FILECOPIED') : \constant('CO_'.$moduleDirNameUpper.'_'.'FC_FILENOTCOPIED');
+        \redirect_header($redirect, 2, $msg.': '.$file_path);
         break;
 }

class/Common/Configurator.php

Indentation +36 added lines, -36 removed lines

@@ -28,56 +28,56 @@
  */
 class Configurator
 {
-    public $name;
-    public $paths = [];
-    public $uploadFolders = [];
-    public $copyBlankFiles = [];
-    public $copyTestFolders = [];
-    public $templateFolders = [];
-    public $oldFiles = [];
-    public $oldFolders = [];
-    public $renameTables = [];
-    public $moduleStats = [];
-    public $modCopyright;
-    public $icons;
+	public $name;
+	public $paths = [];
+	public $uploadFolders = [];
+	public $copyBlankFiles = [];
+	public $copyTestFolders = [];
+	public $templateFolders = [];
+	public $oldFiles = [];
+	public $oldFolders = [];
+	public $renameTables = [];
+	public $moduleStats = [];
+	public $modCopyright;
+	public $icons;
 
-    /**
-     * Configurator constructor.
-     */
-    public function __construct()
-    {
-        //        $moduleDirName      = basename(dirname(dirname(__DIR__)));
-        //        $moduleDirNameUpper = mb_strtoupper($moduleDirName);
+	/**
+	 * Configurator constructor.
+	 */
+	public function __construct()
+	{
+		//        $moduleDirName      = basename(dirname(dirname(__DIR__)));
+		//        $moduleDirNameUpper = mb_strtoupper($moduleDirName);
 
-        //        require dirname(dirname(__DIR__)) . '/config/config.php';
-        //        $config = getConfig();
+		//        require dirname(dirname(__DIR__)) . '/config/config.php';
+		//        $config = getConfig();
 
-        $config = include dirname(__DIR__, 2) . '/config/config.php';
+		$config = include dirname(__DIR__, 2) . '/config/config.php';
 
-        $this->name = $config->name;
+		$this->name = $config->name;
 
-        $this->paths = $config->paths;
+		$this->paths = $config->paths;
 
-        $this->uploadFolders = $config->uploadFolders;
+		$this->uploadFolders = $config->uploadFolders;
 
-        $this->copyBlankFiles = $config->copyBlankFiles;
+		$this->copyBlankFiles = $config->copyBlankFiles;
 
-        $this->copyTestFolders = $config->copyTestFolders;
+		$this->copyTestFolders = $config->copyTestFolders;
 
-        $this->templateFolders = $config->templateFolders;
+		$this->templateFolders = $config->templateFolders;
 
-        $this->oldFiles = $config->oldFiles;
+		$this->oldFiles = $config->oldFiles;
 
-        $this->oldFolders = $config->oldFolders;
+		$this->oldFolders = $config->oldFolders;
 
-        $this->renameTables = $config->renameTables;
+		$this->renameTables = $config->renameTables;
 
-        $this->moduleStats = $config->moduleStats;
+		$this->moduleStats = $config->moduleStats;
 
-        $this->modCopyright = $config->modCopyright;
+		$this->modCopyright = $config->modCopyright;
 
-        $this->icons = include dirname(__DIR__, 2) . '/config/icons.php';
+		$this->icons = include dirname(__DIR__, 2) . '/config/icons.php';
 
-        $this->paths = include dirname(__DIR__, 2) . '/config/paths.php';
-    }
+		$this->paths = include dirname(__DIR__, 2) . '/config/paths.php';
+	}
 }

Spacing +3 added lines, -3 removed lines

@@ -52,7 +52,7 @@ 
         //        require dirname(dirname(__DIR__)) . '/config/config.php';
         //        $config = getConfig();
 
-        $config = include dirname(__DIR__, 2) . '/config/config.php';
+        $config = include dirname(__DIR__, 2).'/config/config.php';
 
         $this->name = $config->name;
 
@@ -76,8 +76,8 @@ 
 
         $this->modCopyright = $config->modCopyright;
 
-        $this->icons = include dirname(__DIR__, 2) . '/config/icons.php';
+        $this->icons = include dirname(__DIR__, 2).'/config/icons.php';
 
-        $this->paths = include dirname(__DIR__, 2) . '/config/paths.php';
+        $this->paths = include dirname(__DIR__, 2).'/config/paths.php';
     }
 }

class/Common/DirectoryChecker.php

Indentation +111 added lines, -111 removed lines

@@ -36,158 +36,158 @@
  */
 class DirectoryChecker
 {
-    /**
-     * @param     $path
-     * @param int $mode
-     * @param     $redirectFile
-     *
-     * @return bool|string
-     */
-    public static function getDirectoryStatus($path, $mode = 0777, $redirectFile = null)
-    {
-        global $pathIcon16;
+	/**
+	 * @param     $path
+	 * @param int $mode
+	 * @param     $redirectFile
+	 *
+	 * @return bool|string
+	 */
+	public static function getDirectoryStatus($path, $mode = 0777, $redirectFile = null)
+	{
+		global $pathIcon16;
 
-        if (empty($path)) {
-            return false;
-        }
+		if (empty($path)) {
+			return false;
+		}
 
-        if (null === $redirectFile) {
-            $redirectFile = $_SERVER['SCRIPT_NAME'];
-        }
+		if (null === $redirectFile) {
+			$redirectFile = $_SERVER['SCRIPT_NAME'];
+		}
 
-        $moduleDirName = \basename(dirname(__DIR__, 2));
+		$moduleDirName = \basename(dirname(__DIR__, 2));
 
-        $moduleDirNameUpper = mb_strtoupper($moduleDirName);
+		$moduleDirNameUpper = mb_strtoupper($moduleDirName);
 
-        if (!@\is_dir($path)) {
-            $path_status = "<img src='$pathIcon16/0.png' >";
+		if (!@\is_dir($path)) {
+			$path_status = "<img src='$pathIcon16/0.png' >";
 
-            $path_status .= "$path (" . \constant('CO_' . $moduleDirNameUpper . '_' . 'DC_NOTAVAILABLE') . ') ';
+			$path_status .= "$path (" . \constant('CO_' . $moduleDirNameUpper . '_' . 'DC_NOTAVAILABLE') . ') ';
 
-            $path_status .= "<form action='" . $_SERVER['SCRIPT_NAME'] . "' method='post'>";
+			$path_status .= "<form action='" . $_SERVER['SCRIPT_NAME'] . "' method='post'>";
 
-            $path_status .= "<input type='hidden' name='op' value='createdir'>";
+			$path_status .= "<input type='hidden' name='op' value='createdir'>";
 
-            $path_status .= "<input type='hidden' name='path' value='$path'>";
+			$path_status .= "<input type='hidden' name='path' value='$path'>";
 
-            $path_status .= "<input type='hidden' name='redirect' value='$redirectFile'>";
+			$path_status .= "<input type='hidden' name='redirect' value='$redirectFile'>";
 
-            $path_status .= "<button class='submit' onClick='this.form.submit();'>" . \constant('CO_' . $moduleDirNameUpper . '_' . 'DC_CREATETHEDIR') . '</button>';
+			$path_status .= "<button class='submit' onClick='this.form.submit();'>" . \constant('CO_' . $moduleDirNameUpper . '_' . 'DC_CREATETHEDIR') . '</button>';
 
-            $path_status .= '</form>';
-        } elseif (@\is_writable($path)) {
-            $path_status = "<img src='$pathIcon16/1.png' >";
+			$path_status .= '</form>';
+		} elseif (@\is_writable($path)) {
+			$path_status = "<img src='$pathIcon16/1.png' >";
 
-            $path_status .= "$path (" . \constant('CO_' . $moduleDirNameUpper . '_' . 'DC_AVAILABLE') . ') ';
+			$path_status .= "$path (" . \constant('CO_' . $moduleDirNameUpper . '_' . 'DC_AVAILABLE') . ') ';
 
-            $currentMode = mb_substr(\decoct(\fileperms($path)), 2);
+			$currentMode = mb_substr(\decoct(\fileperms($path)), 2);
 
-            if ($currentMode != \decoct($mode)) {
-                $path_status = "<img src='$pathIcon16/0.png' >";
+			if ($currentMode != \decoct($mode)) {
+				$path_status = "<img src='$pathIcon16/0.png' >";
 
-                $path_status .= $path . \sprintf(\constant('CO_' . $moduleDirNameUpper . '_' . 'DC_NOTWRITABLE'), \decoct($mode), $currentMode);
+				$path_status .= $path . \sprintf(\constant('CO_' . $moduleDirNameUpper . '_' . 'DC_NOTWRITABLE'), \decoct($mode), $currentMode);
 
-                $path_status .= "<form action='" . $_SERVER['SCRIPT_NAME'] . "' method='post'>";
+				$path_status .= "<form action='" . $_SERVER['SCRIPT_NAME'] . "' method='post'>";
 
-                $path_status .= "<input type='hidden' name='op' value='setdirperm'>";
+				$path_status .= "<input type='hidden' name='op' value='setdirperm'>";
 
-                $path_status .= "<input type='hidden' name='mode' value='$mode'>";
+				$path_status .= "<input type='hidden' name='mode' value='$mode'>";
 
-                $path_status .= "<input type='hidden' name='path' value='$path'>";
+				$path_status .= "<input type='hidden' name='path' value='$path'>";
 
-                $path_status .= "<input type='hidden' name='redirect' value='$redirectFile'>";
+				$path_status .= "<input type='hidden' name='redirect' value='$redirectFile'>";
 
-                $path_status .= "<button class='submit' onClick='this.form.submit();'>" . \constant('CO_' . $moduleDirNameUpper . '_' . 'DC_SETMPERM') . '</button>';
+				$path_status .= "<button class='submit' onClick='this.form.submit();'>" . \constant('CO_' . $moduleDirNameUpper . '_' . 'DC_SETMPERM') . '</button>';
 
-                $path_status .= '</form>';
-            }
-        } else {
-            $currentMode = mb_substr(\decoct(\fileperms($path)), 2);
+				$path_status .= '</form>';
+			}
+		} else {
+			$currentMode = mb_substr(\decoct(\fileperms($path)), 2);
 
-            $path_status = "<img src='$pathIcon16/0.png' >";
+			$path_status = "<img src='$pathIcon16/0.png' >";
 
-            $path_status .= $path . \sprintf(\constant('CO_' . $moduleDirNameUpper . '_' . 'DC_NOTWRITABLE'), \decoct($mode), $currentMode);
+			$path_status .= $path . \sprintf(\constant('CO_' . $moduleDirNameUpper . '_' . 'DC_NOTWRITABLE'), \decoct($mode), $currentMode);
 
-            $path_status .= "<form action='" . $_SERVER['SCRIPT_NAME'] . "' method='post'>";
+			$path_status .= "<form action='" . $_SERVER['SCRIPT_NAME'] . "' method='post'>";
 
-            $path_status .= "<input type='hidden' name='op' value='setdirperm'>";
+			$path_status .= "<input type='hidden' name='op' value='setdirperm'>";
 
-            $path_status .= "<input type='hidden' name='mode' value='$mode'>";
+			$path_status .= "<input type='hidden' name='mode' value='$mode'>";
 
-            $path_status .= "<input type='hidden' name='path' value='$path'>";
+			$path_status .= "<input type='hidden' name='path' value='$path'>";
 
-            $path_status .= "<input type='hidden' name='redirect' value='$redirectFile'>";
+			$path_status .= "<input type='hidden' name='redirect' value='$redirectFile'>";
 
-            $path_status .= "<button class='submit' onClick='this.form.submit();'>" . \constant('CO_' . $moduleDirNameUpper . '_' . 'DC_SETMPERM') . '</button>';
+			$path_status .= "<button class='submit' onClick='this.form.submit();'>" . \constant('CO_' . $moduleDirNameUpper . '_' . 'DC_SETMPERM') . '</button>';
 
-            $path_status .= '</form>';
-        }
+			$path_status .= '</form>';
+		}
 
-        return $path_status;
-    }
+		return $path_status;
+	}
 
-    /**
-     * @param     $target
-     * @param int $mode
-     *
-     * @return bool
-     */
-    public static function createDirectory($target, $mode = 0777)
-    {
-        $target = \str_replace('..', '', $target);
+	/**
+	 * @param     $target
+	 * @param int $mode
+	 *
+	 * @return bool
+	 */
+	public static function createDirectory($target, $mode = 0777)
+	{
+		$target = \str_replace('..', '', $target);
 
-        // http://www.php.net/manual/en/function.mkdir.php
+		// http://www.php.net/manual/en/function.mkdir.php
 
-        return \is_dir($target) || (self::createDirectory(\dirname($target), $mode) && !\mkdir($target, $mode) && !\is_dir($target));
-    }
+		return \is_dir($target) || (self::createDirectory(\dirname($target), $mode) && !\mkdir($target, $mode) && !\is_dir($target));
+	}
 
-    /**
-     * @param     $target
-     * @param int $mode
-     *
-     * @return bool
-     */
-    public static function setDirectoryPermissions($target, $mode = 0777)
-    {
-        $target = \str_replace('..', '', $target);
+	/**
+	 * @param     $target
+	 * @param int $mode
+	 *
+	 * @return bool
+	 */
+	public static function setDirectoryPermissions($target, $mode = 0777)
+	{
+		$target = \str_replace('..', '', $target);
 
-        return @\chmod($target, (int)$mode);
-    }
+		return @\chmod($target, (int)$mode);
+	}
 
-    /**
-     * @param   $dir_path
-     *
-     * @return bool
-     */
-    public static function dirExists($dir_path)
-    {
-        return \is_dir($dir_path);
-    }
+	/**
+	 * @param   $dir_path
+	 *
+	 * @return bool
+	 */
+	public static function dirExists($dir_path)
+	{
+		return \is_dir($dir_path);
+	}
 }
 
 $op = Request::getString('op', '', 'POST');
 switch ($op) {
-    case 'createdir':
-        if (\Xmf\Request::hasVar('path', 'POST')) {
-            $path = $_POST['path'];
-        }
-        if (\Xmf\Request::hasVar('redirect', 'POST')) {
-            $redirect = $_POST['redirect'];
-        }
-        $msg = DirectoryChecker::createDirectory($path) ? \constant('CO_' . $moduleDirNameUpper . '_' . 'DC_DIRCREATED') : \constant('CO_' . $moduleDirNameUpper . '_' . 'DC_DIRNOTCREATED');
-        \redirect_header($redirect, 2, $msg . ': ' . $path);
-        break;
-    case 'setdirperm':
-        if (\Xmf\Request::hasVar('path', 'POST')) {
-            $path = $_POST['path'];
-        }
-        if (\Xmf\Request::hasVar('redirect', 'POST')) {
-            $redirect = $_POST['redirect'];
-        }
-        if (\Xmf\Request::hasVar('mode', 'POST')) {
-            $mode = $_POST['mode'];
-        }
-        $msg = DirectoryChecker::setDirectoryPermissions($path, $mode) ? \constant('CO_' . $moduleDirNameUpper . '_' . 'DC_PERMSET') : \constant('CO_' . $moduleDirNameUpper . '_' . 'DC_PERMNOTSET');
-        \redirect_header($redirect, 2, $msg . ': ' . $path);
-        break;
+	case 'createdir':
+		if (\Xmf\Request::hasVar('path', 'POST')) {
+			$path = $_POST['path'];
+		}
+		if (\Xmf\Request::hasVar('redirect', 'POST')) {
+			$redirect = $_POST['redirect'];
+		}
+		$msg = DirectoryChecker::createDirectory($path) ? \constant('CO_' . $moduleDirNameUpper . '_' . 'DC_DIRCREATED') : \constant('CO_' . $moduleDirNameUpper . '_' . 'DC_DIRNOTCREATED');
+		\redirect_header($redirect, 2, $msg . ': ' . $path);
+		break;
+	case 'setdirperm':
+		if (\Xmf\Request::hasVar('path', 'POST')) {
+			$path = $_POST['path'];
+		}
+		if (\Xmf\Request::hasVar('redirect', 'POST')) {
+			$redirect = $_POST['redirect'];
+		}
+		if (\Xmf\Request::hasVar('mode', 'POST')) {
+			$mode = $_POST['mode'];
+		}
+		$msg = DirectoryChecker::setDirectoryPermissions($path, $mode) ? \constant('CO_' . $moduleDirNameUpper . '_' . 'DC_PERMSET') : \constant('CO_' . $moduleDirNameUpper . '_' . 'DC_PERMNOTSET');
+		\redirect_header($redirect, 2, $msg . ': ' . $path);
+		break;
 }

Spacing +16 added lines, -16 removed lines

@@ -25,7 +25,7 @@ 
 
 //defined('XOOPS_ROOT_PATH') || die('XOOPS root path not defined');
 
-require_once dirname(__DIR__, 4) . '/mainfile.php';
+require_once dirname(__DIR__, 4).'/mainfile.php';
 $moduleDirName      = \basename(dirname(__DIR__, 2));
 $moduleDirNameUpper = mb_strtoupper($moduleDirName);
 \xoops_loadLanguage('directorychecker', $moduleDirName);
@@ -62,9 +62,9 @@ 
         if (!@\is_dir($path)) {
             $path_status = "<img src='$pathIcon16/0.png' >";
 
-            $path_status .= "$path (" . \constant('CO_' . $moduleDirNameUpper . '_' . 'DC_NOTAVAILABLE') . ') ';
+            $path_status .= "$path (".\constant('CO_'.$moduleDirNameUpper.'_'.'DC_NOTAVAILABLE').') ';
 
-            $path_status .= "<form action='" . $_SERVER['SCRIPT_NAME'] . "' method='post'>";
+            $path_status .= "<form action='".$_SERVER['SCRIPT_NAME']."' method='post'>";
 
             $path_status .= "<input type='hidden' name='op' value='createdir'>";
 
@@ -72,22 +72,22 @@ 
 
             $path_status .= "<input type='hidden' name='redirect' value='$redirectFile'>";
 
-            $path_status .= "<button class='submit' onClick='this.form.submit();'>" . \constant('CO_' . $moduleDirNameUpper . '_' . 'DC_CREATETHEDIR') . '</button>';
+            $path_status .= "<button class='submit' onClick='this.form.submit();'>".\constant('CO_'.$moduleDirNameUpper.'_'.'DC_CREATETHEDIR').'</button>';
 
             $path_status .= '</form>';
         } elseif (@\is_writable($path)) {
             $path_status = "<img src='$pathIcon16/1.png' >";
 
-            $path_status .= "$path (" . \constant('CO_' . $moduleDirNameUpper . '_' . 'DC_AVAILABLE') . ') ';
+            $path_status .= "$path (".\constant('CO_'.$moduleDirNameUpper.'_'.'DC_AVAILABLE').') ';
 
             $currentMode = mb_substr(\decoct(\fileperms($path)), 2);
 
             if ($currentMode != \decoct($mode)) {
                 $path_status = "<img src='$pathIcon16/0.png' >";
 
-                $path_status .= $path . \sprintf(\constant('CO_' . $moduleDirNameUpper . '_' . 'DC_NOTWRITABLE'), \decoct($mode), $currentMode);
+                $path_status .= $path.\sprintf(\constant('CO_'.$moduleDirNameUpper.'_'.'DC_NOTWRITABLE'), \decoct($mode), $currentMode);
 
-                $path_status .= "<form action='" . $_SERVER['SCRIPT_NAME'] . "' method='post'>";
+                $path_status .= "<form action='".$_SERVER['SCRIPT_NAME']."' method='post'>";
 
                 $path_status .= "<input type='hidden' name='op' value='setdirperm'>";
 
@@ -97,7 +97,7 @@ 
 
                 $path_status .= "<input type='hidden' name='redirect' value='$redirectFile'>";
 
-                $path_status .= "<button class='submit' onClick='this.form.submit();'>" . \constant('CO_' . $moduleDirNameUpper . '_' . 'DC_SETMPERM') . '</button>';
+                $path_status .= "<button class='submit' onClick='this.form.submit();'>".\constant('CO_'.$moduleDirNameUpper.'_'.'DC_SETMPERM').'</button>';
 
                 $path_status .= '</form>';
             }
@@ -106,9 +106,9 @@ 
 
             $path_status = "<img src='$pathIcon16/0.png' >";
 
-            $path_status .= $path . \sprintf(\constant('CO_' . $moduleDirNameUpper . '_' . 'DC_NOTWRITABLE'), \decoct($mode), $currentMode);
+            $path_status .= $path.\sprintf(\constant('CO_'.$moduleDirNameUpper.'_'.'DC_NOTWRITABLE'), \decoct($mode), $currentMode);
 
-            $path_status .= "<form action='" . $_SERVER['SCRIPT_NAME'] . "' method='post'>";
+            $path_status .= "<form action='".$_SERVER['SCRIPT_NAME']."' method='post'>";
 
             $path_status .= "<input type='hidden' name='op' value='setdirperm'>";
 
@@ -118,7 +118,7 @@ 
 
             $path_status .= "<input type='hidden' name='redirect' value='$redirectFile'>";
 
-            $path_status .= "<button class='submit' onClick='this.form.submit();'>" . \constant('CO_' . $moduleDirNameUpper . '_' . 'DC_SETMPERM') . '</button>';
+            $path_status .= "<button class='submit' onClick='this.form.submit();'>".\constant('CO_'.$moduleDirNameUpper.'_'.'DC_SETMPERM').'</button>';
 
             $path_status .= '</form>';
         }
@@ -151,7 +151,7 @@ 
     {
         $target = \str_replace('..', '', $target);
 
-        return @\chmod($target, (int)$mode);
+        return @\chmod($target, (int) $mode);
     }
 
     /**
@@ -174,8 +174,8 @@ 
         if (\Xmf\Request::hasVar('redirect', 'POST')) {
             $redirect = $_POST['redirect'];
         }
-        $msg = DirectoryChecker::createDirectory($path) ? \constant('CO_' . $moduleDirNameUpper . '_' . 'DC_DIRCREATED') : \constant('CO_' . $moduleDirNameUpper . '_' . 'DC_DIRNOTCREATED');
-        \redirect_header($redirect, 2, $msg . ': ' . $path);
+        $msg = DirectoryChecker::createDirectory($path) ? \constant('CO_'.$moduleDirNameUpper.'_'.'DC_DIRCREATED') : \constant('CO_'.$moduleDirNameUpper.'_'.'DC_DIRNOTCREATED');
+        \redirect_header($redirect, 2, $msg.': '.$path);
         break;
     case 'setdirperm':
         if (\Xmf\Request::hasVar('path', 'POST')) {
@@ -187,7 +187,7 @@ 
         if (\Xmf\Request::hasVar('mode', 'POST')) {
             $mode = $_POST['mode'];
         }
-        $msg = DirectoryChecker::setDirectoryPermissions($path, $mode) ? \constant('CO_' . $moduleDirNameUpper . '_' . 'DC_PERMSET') : \constant('CO_' . $moduleDirNameUpper . '_' . 'DC_PERMNOTSET');
-        \redirect_header($redirect, 2, $msg . ': ' . $path);
+        $msg = DirectoryChecker::setDirectoryPermissions($path, $mode) ? \constant('CO_'.$moduleDirNameUpper.'_'.'DC_PERMSET') : \constant('CO_'.$moduleDirNameUpper.'_'.'DC_PERMNOTSET');
+        \redirect_header($redirect, 2, $msg.': '.$path);
         break;
 }

class/Common/Breadcrumb.php

Indentation +30 added lines, -30 removed lines

@@ -38,40 +38,40 @@ 
  */
 class Breadcrumb
 {
-    public $dirname;
-    private $bread = [];
-
-    public function __construct()
-    {
-        $this->dirname = \basename(dirname(__DIR__, 2));
-    }
-
-    /**
-     * Add link to breadcrumb
-     *
-     * @param string $title
-     * @param string $link
-     */
-    public function addLink($title = '', $link = '')
-    {
-        $this->bread[] = [
-            'link'  => $link,
-            'title' => $title,
-        ];
-    }
-
-    /**
-     * Render BreadCrumb
-     */
-    public function render()
-    {
-        /*
+	public $dirname;
+	private $bread = [];
+
+	public function __construct()
+	{
+		$this->dirname = \basename(dirname(__DIR__, 2));
+	}
+
+	/**
+	 * Add link to breadcrumb
+	 *
+	 * @param string $title
+	 * @param string $link
+	 */
+	public function addLink($title = '', $link = '')
+	{
+		$this->bread[] = [
+			'link'  => $link,
+			'title' => $title,
+		];
+	}
+
+	/**
+	 * Render BreadCrumb
+	 */
+	public function render()
+	{
+		/*
         TODO if you want to use the render code below,
         1) create ./templates/chess_common_breadcrumb.tpl)
         2) add declaration to  xoops_version.php
         */
 
-        /*
+		/*
         if (!isset($GLOBALS['xoTheme']) || !\is_object($GLOBALS['xoTheme'])) {
             require $GLOBALS['xoops']->path('class/theme.php');
 
@@ -90,5 +90,5 @@ 
 
         return $html;
         */
-    }
+	}
 }

class/Common/Migrate.php

Indentation +106 added lines, -106 removed lines

@@ -25,110 +25,110 @@
  */
 class Migrate extends \Xmf\Database\Migrate
 {
-    private $renameTables;
-
-    /**
-     * Migrate constructor.
-     * @param Common\Configurator $configurator
-     * @throws \RuntimeException
-     * @throws \InvalidArgumentException
-     */
-    public function __construct(?Common\Configurator $configurator = null)
-    {
-        if (null !== $configurator) {
-            $this->renameTables = $configurator->renameTables;
-
-            $moduleDirName = \basename(dirname(__DIR__, 2));
-
-            parent::__construct($moduleDirName);
-        }
-    }
-
-    /**
-     * change table prefix if needed
-     */
-    private function changePrefix()
-    {
-        foreach ($this->renameTables as $oldName => $newName) {
-            if ($this->tableHandler->useTable($oldName) && !$this->tableHandler->useTable($newName)) {
-                $this->tableHandler->renameTable($oldName, $newName);
-            }
-        }
-    }
-
-    /**
-     * Change integer IPv4 column to varchar IPv6 capable
-     *
-     * @param string $tableName  table to convert
-     * @param string $columnName column with IP address
-     */
-    private function convertIPAddresses($tableName, $columnName)
-    {
-        if ($this->tableHandler->useTable($tableName)) {
-            $attributes = $this->tableHandler->getColumnAttributes($tableName, $columnName);
-
-            if (false !== mb_strpos($attributes, ' int(')) {
-                if (false === mb_strpos($attributes, 'unsigned')) {
-                    $this->tableHandler->alterColumn($tableName, $columnName, " bigint(16) NOT NULL  DEFAULT '0' ");
-
-                    $this->tableHandler->update($tableName, [$columnName => "4294967296 + $columnName"], "WHERE $columnName < 0", false);
-                }
-
-                $this->tableHandler->alterColumn($tableName, $columnName, " varchar(45)  NOT NULL  DEFAULT '' ");
-
-                $this->tableHandler->update($tableName, [$columnName => "INET_NTOA($columnName)"], '', false);
-            }
-        }
-    }
-
-    /**
-     * Move do* columns from newbb_posts to newbb_posts_text table
-     */
-    private function moveDoColumns()
-    {
-        $tableName = 'newbb_posts_text';
-
-        $srcTableName = 'newbb_posts';
-
-        if ($this->tableHandler->useTable($tableName)
-            && $this->tableHandler->useTable($srcTableName)) {
-            $attributes = $this->tableHandler->getColumnAttributes($tableName, 'dohtml');
-
-            if (false === $attributes) {
-                $this->synchronizeTable($tableName);
-
-                $updateTable = $GLOBALS['xoopsDB']->prefix($tableName);
-
-                $joinTable = $GLOBALS['xoopsDB']->prefix($srcTableName);
-
-                $sql = "UPDATE `$updateTable` t1 INNER JOIN `$joinTable` t2 ON t1.post_id = t2.post_id \n" . "SET t1.dohtml = t2.dohtml,  t1.dosmiley = t2.dosmiley, t1.doxcode = t2.doxcode\n" . '  , t1.doimage = t2.doimage, t1.dobr = t2.dobr';
-
-                $this->tableHandler->addToQueue($sql);
-            }
-        }
-    }
-
-    /**
-     * Perform any upfront actions before synchronizing the schema
-     *
-     * Some typical uses include
-     *   table and column renames
-     *   data conversions
-     */
-    protected function preSyncActions()
-    {
-        // change 'bb' table prefix to 'newbb'
-
-        $this->changePrefix();
-
-        // columns dohtml, dosmiley, doxcode, doimage and dobr moved between tables as some point
-
-        $this->moveDoColumns();
-
-        // Convert IP address columns from int to readable varchar(45) for IPv6
-
-        $this->convertIPAddresses('newbb_posts', 'poster_ip');
-
-        $this->convertIPAddresses('newbb_report', 'reporter_ip');
-    }
+	private $renameTables;
+
+	/**
+	 * Migrate constructor.
+	 * @param Common\Configurator $configurator
+	 * @throws \RuntimeException
+	 * @throws \InvalidArgumentException
+	 */
+	public function __construct(?Common\Configurator $configurator = null)
+	{
+		if (null !== $configurator) {
+			$this->renameTables = $configurator->renameTables;
+
+			$moduleDirName = \basename(dirname(__DIR__, 2));
+
+			parent::__construct($moduleDirName);
+		}
+	}
+
+	/**
+	 * change table prefix if needed
+	 */
+	private function changePrefix()
+	{
+		foreach ($this->renameTables as $oldName => $newName) {
+			if ($this->tableHandler->useTable($oldName) && !$this->tableHandler->useTable($newName)) {
+				$this->tableHandler->renameTable($oldName, $newName);
+			}
+		}
+	}
+
+	/**
+	 * Change integer IPv4 column to varchar IPv6 capable
+	 *
+	 * @param string $tableName  table to convert
+	 * @param string $columnName column with IP address
+	 */
+	private function convertIPAddresses($tableName, $columnName)
+	{
+		if ($this->tableHandler->useTable($tableName)) {
+			$attributes = $this->tableHandler->getColumnAttributes($tableName, $columnName);
+
+			if (false !== mb_strpos($attributes, ' int(')) {
+				if (false === mb_strpos($attributes, 'unsigned')) {
+					$this->tableHandler->alterColumn($tableName, $columnName, " bigint(16) NOT NULL  DEFAULT '0' ");
+
+					$this->tableHandler->update($tableName, [$columnName => "4294967296 + $columnName"], "WHERE $columnName < 0", false);
+				}
+
+				$this->tableHandler->alterColumn($tableName, $columnName, " varchar(45)  NOT NULL  DEFAULT '' ");
+
+				$this->tableHandler->update($tableName, [$columnName => "INET_NTOA($columnName)"], '', false);
+			}
+		}
+	}
+
+	/**
+	 * Move do* columns from newbb_posts to newbb_posts_text table
+	 */
+	private function moveDoColumns()
+	{
+		$tableName = 'newbb_posts_text';
+
+		$srcTableName = 'newbb_posts';
+
+		if ($this->tableHandler->useTable($tableName)
+			&& $this->tableHandler->useTable($srcTableName)) {
+			$attributes = $this->tableHandler->getColumnAttributes($tableName, 'dohtml');
+
+			if (false === $attributes) {
+				$this->synchronizeTable($tableName);
+
+				$updateTable = $GLOBALS['xoopsDB']->prefix($tableName);
+
+				$joinTable = $GLOBALS['xoopsDB']->prefix($srcTableName);
+
+				$sql = "UPDATE `$updateTable` t1 INNER JOIN `$joinTable` t2 ON t1.post_id = t2.post_id \n" . "SET t1.dohtml = t2.dohtml,  t1.dosmiley = t2.dosmiley, t1.doxcode = t2.doxcode\n" . '  , t1.doimage = t2.doimage, t1.dobr = t2.dobr';
+
+				$this->tableHandler->addToQueue($sql);
+			}
+		}
+	}
+
+	/**
+	 * Perform any upfront actions before synchronizing the schema
+	 *
+	 * Some typical uses include
+	 *   table and column renames
+	 *   data conversions
+	 */
+	protected function preSyncActions()
+	{
+		// change 'bb' table prefix to 'newbb'
+
+		$this->changePrefix();
+
+		// columns dohtml, dosmiley, doxcode, doimage and dobr moved between tables as some point
+
+		$this->moveDoColumns();
+
+		// Convert IP address columns from int to readable varchar(45) for IPv6
+
+		$this->convertIPAddresses('newbb_posts', 'poster_ip');
+
+		$this->convertIPAddresses('newbb_report', 'reporter_ip');
+	}
 }

comment_post.php

Spacing +2 added lines, -2 removed lines

@@ -35,6 +35,6 @@
 
 /**#@+
  */
-require dirname(__DIR__, 2) . '/mainfile.php';
-require XOOPS_ROOT_PATH . '/include/comment_post.php';
+require dirname(__DIR__, 2).'/mainfile.php';
+require XOOPS_ROOT_PATH.'/include/comment_post.php';
 /**#@-*/