mambax7 / contact

include/common.php

Spacing +1 added lines, -1 removed lines

@@ -27,7 +27,7 @@
     define('CONTACT_UPLOAD_URL', XOOPS_UPLOAD_URL.'/'.CONTACT_DIRNAME);
     define('CONTACT_IMAGE_PATH', CONTACT_PATH.'/assets/images');
     define('CONTACT_IMAGE_URL', CONTACT_URL.'/assets/images/');
-    define('CONTACT_ADMIN', CONTACT_URL . '/admin/index.php');
+    define('CONTACT_ADMIN', CONTACT_URL.'/admin/index.php');
 }
 // module information
 $copyright = "<a href='http://xoops.wedega.com' title='WEDEGA Webdesign Gabor' target='_blank'>

include/functions_update.php

Spacing +10 added lines, -10 removed lines

@@ -26,7 +26,7 @@ 
     $xoopsDB = XoopsDatabaseFactory::getDatabaseConnection();
 
     if ($version < 180) {
-        $sql = 'CREATE TABLE ' . $xoopsDB->prefix('contact') . ' (
+        $sql = 'CREATE TABLE '.$xoopsDB->prefix('contact').' (
         contact_id int(10) unsigned NOT NULL auto_increment,
         contact_uid int(10) NOT NULL,
         contact_cid int(10) NOT NULL,
@@ -51,31 +51,31 @@ 
 
     if ($version < 181) {
         // Add contact_platform
-        $sql = 'ALTER TABLE `' . $xoopsDB->prefix('contact') . "` ADD `contact_platform` ENUM('Android','Ios','Web') NOT NULL DEFAULT 'Web'";
+        $sql = 'ALTER TABLE `'.$xoopsDB->prefix('contact')."` ADD `contact_platform` ENUM('Android','Ios','Web') NOT NULL DEFAULT 'Web'";
         $xoopsDB->query($sql);
         // Add contact_type
-        $sql = 'ALTER TABLE `' . $xoopsDB->prefix('contact') . "` ADD `contact_type` ENUM('Contact','Phone','Mail') NOT NULL DEFAULT 'Contact'";
+        $sql = 'ALTER TABLE `'.$xoopsDB->prefix('contact')."` ADD `contact_type` ENUM('Contact','Phone','Mail') NOT NULL DEFAULT 'Contact'";
         $xoopsDB->query($sql);
         // Add index contact_uid
-        $sql = 'ALTER TABLE `' . $xoopsDB->prefix('contact') . '` ADD INDEX `contact_uid` ( `contact_uid` )';
+        $sql = 'ALTER TABLE `'.$xoopsDB->prefix('contact').'` ADD INDEX `contact_uid` ( `contact_uid` )';
         $xoopsDB->query($sql);
         // Add index contact_cid
-        $sql = 'ALTER TABLE `' . $xoopsDB->prefix('contact') . '` ADD INDEX `contact_cid` ( `contact_cid` )';
+        $sql = 'ALTER TABLE `'.$xoopsDB->prefix('contact').'` ADD INDEX `contact_cid` ( `contact_cid` )';
         $xoopsDB->query($sql);
         // Add index contact_create
-        $sql = 'ALTER TABLE `' . $xoopsDB->prefix('contact') . '` ADD INDEX `contact_create` ( `contact_create` )';
+        $sql = 'ALTER TABLE `'.$xoopsDB->prefix('contact').'` ADD INDEX `contact_create` ( `contact_create` )';
         $xoopsDB->query($sql);
         // Add index contact_mail
-        $sql = 'ALTER TABLE `' . $xoopsDB->prefix('contact') . '` ADD INDEX `contact_mail` ( `contact_mail` )';
+        $sql = 'ALTER TABLE `'.$xoopsDB->prefix('contact').'` ADD INDEX `contact_mail` ( `contact_mail` )';
         $xoopsDB->query($sql);
         // Add index contact_phone
-        $sql = 'ALTER TABLE `' . $xoopsDB->prefix('contact') . '` ADD INDEX `contact_phone` ( `contact_phone` )';
+        $sql = 'ALTER TABLE `'.$xoopsDB->prefix('contact').'` ADD INDEX `contact_phone` ( `contact_phone` )';
         $xoopsDB->query($sql);
         // Add index contact_platform
-        $sql = 'ALTER TABLE `' . $xoopsDB->prefix('contact') . '` ADD INDEX `contact_platform` ( `contact_platform` )';
+        $sql = 'ALTER TABLE `'.$xoopsDB->prefix('contact').'` ADD INDEX `contact_platform` ( `contact_platform` )';
         $xoopsDB->query($sql);
         // Add index contact_type
-        $sql = 'ALTER TABLE `' . $xoopsDB->prefix('contact') . '` ADD INDEX `contact_type` ( `contact_type` )';
+        $sql = 'ALTER TABLE `'.$xoopsDB->prefix('contact').'` ADD INDEX `contact_type` ( `contact_type` )';
         $xoopsDB->query($sql);
     }
 }

index.php

Indentation +1 added lines, -1 removed lines

@@ -18,7 +18,7 @@
  * @author      Trabis <lusopoemas@gmail.com>
  * @author      Hossein Azizabadi (AKA Voltan)
  * @author      Mirza (AKA Bleekk)
-  */
+ */
 include __DIR__ . '/header.php';
 $GLOBALS['xoopsOption']['template_main'] = 'contact_index.tpl';
 //unset($_SESSION);

Spacing +4 added lines, -4 removed lines

@@ -19,10 +19,10 @@ 
  * @author      Hossein Azizabadi (AKA Voltan)
  * @author      Mirza (AKA Bleekk)
   */
-include __DIR__ . '/header.php';
+include __DIR__.'/header.php';
 $GLOBALS['xoopsOption']['template_main'] = 'contact_index.tpl';
 //unset($_SESSION);
-include XOOPS_ROOT_PATH . '/header.php';
+include XOOPS_ROOT_PATH.'/header.php';
 global $xoopsModuleConfig, $xoopsModule;
 /*Modules Options*/
 if ($xoopsModuleConfig['form_dept'] == 1) {
@@ -49,7 +49,7 @@ 
 $GLOBALS['xoopsTpl']->assign('map', $xoopsModuleConfig['embed_maps']);
 /*end Modules options*/
 
-$GLOBALS['xoopsTpl']->assign('breadcrumb', '<li><a href="' . XOOPS_URL . '">' . _YOURHOME . '</a></li> <li class="active">' . $xoopsModule->name().'</li>');
+$GLOBALS['xoopsTpl']->assign('breadcrumb', '<li><a href="'.XOOPS_URL.'">'._YOURHOME.'</a></li> <li class="active">'.$xoopsModule->name().'</li>');
 $GLOBALS['xoopsTpl']->assign('info', xoops_getModuleOption('contact_info', 'contact'));
 $GLOBALS['xoopsTpl']->assign('contact_default', xoops_getModuleOption('contact_default', 'contact'));
 
@@ -80,4 +80,4 @@ 
 $GLOBALS['xoopsTpl']->assign('lng_subject_info', _MD_CONTACT_SUBJECT_INFO);
 $GLOBALS['xoopsTpl']->assign('lng_message_info', _MD_CONTACT_MESSAGE_INFO);
                
-include XOOPS_ROOT_PATH . '/footer.php';
+include XOOPS_ROOT_PATH.'/footer.php';

ajax.php

Spacing +1 added lines, -1 removed lines

@@ -19,7 +19,7 @@
  * @author      Hossein Azizabadi (AKA Voltan)
  */
 
-include __DIR__ . '/header.php';
+include __DIR__.'/header.php';
 
 if (!empty($_POST)) {
     // Info Processing

class/helper.php

Doc Comments +3 added lines, -3 removed lines

@@ -55,7 +55,7 @@ 
     */
     /**
      * ContactHelper constructor.
-     * @param $debug
+     * @param boolean $debug
      */
     protected function __construct($debug)
     {
@@ -124,7 +124,7 @@ 
     /**
      * @param null $name
      * @param null $value
-     * @return mixed
+     * @return string
      */
     public function setConfig($name = null, $value = null)
     {
@@ -141,7 +141,7 @@ 
     */
     /**
      * @param $name
-     * @return mixed
+     * @return string
      */
     public function getHandler($name)
     {

Spacing +7 added lines, -7 removed lines

@@ -60,7 +60,7 @@ 
     protected function __construct($debug)
     {
         $this->debug = $debug;
-        $this->dirname =  basename(dirname(__DIR__));
+        $this->dirname = basename(dirname(__DIR__));
     }
     /*
     *  @static function getInstance
@@ -113,7 +113,7 @@ 
             $this->addLog("ERROR :: CONFIG '{$name}' does not exist");
             return null;
         }
-        $this->addLog("Getting config '{$name}' : " . $this->config[$name]);
+        $this->addLog("Getting config '{$name}' : ".$this->config[$name]);
         return $this->config[$name];
     }
     /*
@@ -132,7 +132,7 @@ 
             $this->initConfig();
         }
         $this->config[$name] = $value;
-        $this->addLog("Setting config '{$name}' : " . $this->config[$name]);
+        $this->addLog("Setting config '{$name}' : ".$this->config[$name]);
         return $this->config[$name];
     }
     /*
@@ -145,11 +145,11 @@ 
      */
     public function getHandler($name)
     {
-        if (!isset($this->handler[$name . 'Handler'])) {
+        if (!isset($this->handler[$name.'Handler'])) {
             $this->initHandler($name);
         }
         $this->addLog("Getting handler '{$name}'");
-        return $this->handler[$name . 'Handler'];
+        return $this->handler[$name.'Handler'];
     }
     /*
     *  @static function initModule
@@ -187,8 +187,8 @@ 
      */
     public function initHandler($name)
     {
-        $this->addLog('INIT ' . $name . ' HANDLER');
-        $this->handler[$name . 'Handler'] = xoops_getModuleHandler($name, $this->dirname);
+        $this->addLog('INIT '.$name.' HANDLER');
+        $this->handler[$name.'Handler'] = xoops_getModuleHandler($name, $this->dirname);
     }
     /*
     *  @static function addLog

class/utility.php

Spacing +7 added lines, -7 removed lines

@@ -29,7 +29,7 @@ 
                 if (!mkdir($folder) && !is_dir($folder)) {
                     throw new \RuntimeException(sprintf('Unable to create the %s directory', $folder));
                 } else {
-                    file_put_contents($folder . '/index.html', '<script>history.go(-1);</script>');
+                    file_put_contents($folder.'/index.html', '<script>history.go(-1);</script>');
                 }
             }
         } catch (Exception $e) {
@@ -67,10 +67,10 @@ 
         //    @mkdir($dst);
         while (false !== ($file = readdir($dir))) {
             if (($file !== '.') && ($file !== '..')) {
-                if (is_dir($src . '/' . $file)) {
-                    self::recurseCopy($src . '/' . $file, $dst . '/' . $file);
+                if (is_dir($src.'/'.$file)) {
+                    self::recurseCopy($src.'/'.$file, $dst.'/'.$file);
                 } else {
-                    copy($src . '/' . $file, $dst . '/' . $file);
+                    copy($src.'/'.$file, $dst.'/'.$file);
                 }
             }
         }
@@ -91,7 +91,7 @@ 
         //check for minimum XOOPS version
         $currentVer  = substr(XOOPS_VERSION, 6); // get the numeric part of string
         $currArray   = explode('.', $currentVer);
-        $requiredVer = '' . $module->getInfo('min_xoops'); //making sure it's a string
+        $requiredVer = ''.$module->getInfo('min_xoops'); //making sure it's a string
         $reqArray    = explode('.', $requiredVer);
         $success     = true;
         foreach ($reqArray as $k => $v) {
@@ -105,7 +105,7 @@ 
                     break;
                 }
             } else {
-                if ((int)$v > 0) { // handles things like x.x.x.0_RC2
+                if ((int) $v > 0) { // handles things like x.x.x.0_RC2
                     $success = false;
                     break;
                 }
@@ -133,7 +133,7 @@ 
         // check for minimum PHP version
         $success = true;
         $verNum  = PHP_VERSION;
-        $reqVer  =& $module->getInfo('min_php');
+        $reqVer  = & $module->getInfo('min_php');
         if (false !== $reqVer && '' !== $reqVer) {
             if (version_compare($verNum, $reqVer, '<')) {
                 $module->setErrors(sprintf(_AM_CONTACT_ERROR_BAD_PHP, $reqVer, $verNum));

admin/tools.php

Spacing +3 added lines, -3 removed lines

@@ -22,7 +22,7 @@ 
 use Xmf\Request;
 
 // Call header
-require __DIR__ . '/admin_header.php';
+require __DIR__.'/admin_header.php';
 // Display Admin header
 xoops_cp_header();
 // Define default value
@@ -55,6 +55,6 @@ 
 
 $GLOBALS['xoopsTpl']->assign('navigation', $adminObject->displayNavigation(basename(__FILE__)));
 // Call template file
-$GLOBALS['xoopsTpl']->display(XOOPS_ROOT_PATH . '/modules/contact/templates/admin/contact_tools.tpl');
+$GLOBALS['xoopsTpl']->display(XOOPS_ROOT_PATH.'/modules/contact/templates/admin/contact_tools.tpl');
 // Call footer
-require __DIR__ . '/admin_footer.php';
+require __DIR__.'/admin_footer.php';

admin/admin_header.php

Spacing +3 added lines, -3 removed lines

@@ -19,9 +19,9 @@
  * @author      Hossein Azizabadi (AKA Voltan)
  */
 
-require_once __DIR__ . '/../../../include/cp_header.php';
-require_once XOOPS_ROOT_PATH . '/class/pagenav.php';
-require_once XOOPS_ROOT_PATH . '/class/xoopsformloader.php';
+require_once __DIR__.'/../../../include/cp_header.php';
+require_once XOOPS_ROOT_PATH.'/class/pagenav.php';
+require_once XOOPS_ROOT_PATH.'/class/xoopsformloader.php';
 global $xoopsModule;
 
 $moduleDirName = $GLOBALS['xoopsModule']->getVar('dirname');

admin/main.php

Spacing +9 added lines, -9 removed lines

@@ -22,12 +22,12 @@ 
 use Xmf\Request;
 
 // Call header
-require __DIR__ . '/admin_header.php';
+require __DIR__.'/admin_header.php';
 // Display Admin header
 xoops_cp_header();
 global $xoopsModuleConfig;
 // Define default value
-$level      = '';
+$level = '';
 
 $saveinfo = $xoopsModuleConfig['saveinfo'];
 $sendmail = $xoopsModuleConfig['sendmail'];
@@ -38,11 +38,11 @@ 
 // Define scripts
 $GLOBALS['xoTheme']->addScript('browse.php?Frameworks/jquery/jquery.js');
 $GLOBALS['xoTheme']->addScript('browse.php?Frameworks/jquery/plugins/jquery.ui.js');
-$GLOBALS['xoTheme']->addScript(XOOPS_URL . '/modules/contact/assets/js/admin.js');
+$GLOBALS['xoTheme']->addScript(XOOPS_URL.'/modules/contact/assets/js/admin.js');
 // Add module stylesheet
-$GLOBALS['xoTheme']->addStylesheet(XOOPS_URL . '/modules/contact/assets/css/admin.css');
-$GLOBALS['xoTheme']->addStylesheet(XOOPS_URL . '/modules/system/css/ui/' . xoops_getModuleOption('jquery_theme', 'system') . '/ui.all.css');
-$GLOBALS['xoTheme']->addStylesheet(XOOPS_URL . '/modules/system/css/admin.css');
+$GLOBALS['xoTheme']->addStylesheet(XOOPS_URL.'/modules/contact/assets/css/admin.css');
+$GLOBALS['xoTheme']->addStylesheet(XOOPS_URL.'/modules/system/css/ui/'.xoops_getModuleOption('jquery_theme', 'system').'/ui.all.css');
+$GLOBALS['xoTheme']->addStylesheet(XOOPS_URL.'/modules/system/css/admin.css');
 
 switch ($op) {
     case 'list':
@@ -61,7 +61,7 @@ 
         $contacts        = $contactHandler->contactGetAdminList($contact, 'contact_cid');
 
         if ($contact_numrows > $contact['limit']) {
-            $contact_pagenav = new XoopsPageNav($contact_numrows, $contact['limit'], $contact['start'], 'start', 'limit=' . $contact['limit']);
+            $contact_pagenav = new XoopsPageNav($contact_numrows, $contact['limit'], $contact['start'], 'start', 'limit='.$contact['limit']);
             $contact_pagenav = $contact_pagenav->renderNav(4);
         } else {
             $contact_pagenav = '';
@@ -194,6 +194,6 @@ 
 $GLOBALS['xoopsTpl']->assign('level', $level);
 
 // Call template file
-$GLOBALS['xoopsTpl']->display(XOOPS_ROOT_PATH . '/modules/contact/templates/admin/contact_main.tpl');
+$GLOBALS['xoopsTpl']->display(XOOPS_ROOT_PATH.'/modules/contact/templates/admin/contact_main.tpl');
 // Call footer
-require __DIR__ . '/admin_footer.php';
+require __DIR__.'/admin_footer.php';