mambax7 / contact

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

send.php

Spacing +6 added lines, -6 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';
 
 /** reCaptcha by google **/
 global $xoopsConfig, $xoopsModuleConfig;
@@ -35,7 +35,7 @@ 
 if (!$captcha && $xoopsModuleConfig['recaptchause']) {
     redirect_header('index.php', 2, _MD_CONTACT_MES_NOCAPTCHA);
 } else {
-    $response=file_get_contents('https://www.google.com/recaptcha/api/siteverify?secret=' . $xoopsModuleConfig['recaptchakey'] . '&response=' . $captcha . '&remoteip=' . $_SERVER['REMOTE_ADDR']);
+    $response = file_get_contents('https://www.google.com/recaptcha/api/siteverify?secret='.$xoopsModuleConfig['recaptchakey'].'&response='.$captcha.'&remoteip='.$_SERVER['REMOTE_ADDR']);
     if ($response.success === false && $xoopsModuleConfig['recaptchause']) {
         redirect_header('index.php', 2, _MD_CONTACT_MES_CAPTCHAINCORRECT);
     } else {
@@ -43,11 +43,11 @@ 
         $op         = XoopsRequest::getString('op', 'form', 'POST');
         $department = XoopsRequest::getString('department', '', 'GET');
         if ($op === 'save') {
-            if (''== XoopsRequest::getString('submit', '', 'POST')) {
+            if ('' == XoopsRequest::getString('submit', '', 'POST')) {
                 redirect_header(XOOPS_URL, 3, _MD_CONTACT_MES_ERROR);
             } else {
                 // check email
-                if (''== XoopsRequest::getString('contact_mail', '', 'POST')) {
+                if ('' == XoopsRequest::getString('contact_mail', '', 'POST')) {
                     redirect_header('index.php', 1, _MD_CONTACT_MES_NOVALIDEMAIL);
                 }
             
@@ -82,4 +82,4 @@ 
 }
 
 
-include XOOPS_ROOT_PATH . '/footer.php';
+include XOOPS_ROOT_PATH.'/footer.php';

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

admin/admin_header.php

Spacing +9 added lines, -9 removed lines

@@ -21,11 +21,11 @@ 
 
 $path = dirname(dirname(dirname(__DIR__)));
 
-include_once $path . '/mainfile.php';
-include_once XOOPS_ROOT_PATH . '/include/cp_functions.php';
-include_once XOOPS_ROOT_PATH . '/include/cp_header.php';
-include_once XOOPS_ROOT_PATH . '/class/pagenav.php';
-include_once XOOPS_ROOT_PATH . '/class/xoopsformloader.php';
+include_once $path.'/mainfile.php';
+include_once XOOPS_ROOT_PATH.'/include/cp_functions.php';
+include_once XOOPS_ROOT_PATH.'/include/cp_header.php';
+include_once XOOPS_ROOT_PATH.'/class/pagenav.php';
+include_once XOOPS_ROOT_PATH.'/class/xoopsformloader.php';
 global $xoopsModule;
 
 xoops_load('XoopsRequest');
@@ -37,10 +37,10 @@ 
 xoops_loadLanguage('modinfo', $moduleDirName);
 xoops_loadLanguage('main', $moduleDirName);
 
-$pathIcon16 = XOOPS_URL . '/' . $xoopsModule->getInfo('icons16');
-$pathIcon32 = XOOPS_URL . '/' . $xoopsModule->getInfo('icons32');
-$pathModuleAdmin = XOOPS_ROOT_PATH . '/' . $xoopsModule->getInfo('dirmoduleadmin');
-require_once $pathModuleAdmin . '/moduleadmin/moduleadmin.php';
+$pathIcon16 = XOOPS_URL.'/'.$xoopsModule->getInfo('icons16');
+$pathIcon32 = XOOPS_URL.'/'.$xoopsModule->getInfo('icons32');
+$pathModuleAdmin = XOOPS_ROOT_PATH.'/'.$xoopsModule->getInfo('dirmoduleadmin');
+require_once $pathModuleAdmin.'/moduleadmin/moduleadmin.php';
 
 $adminObject = new ModuleAdmin();
 

admin/main.php

Spacing +9 added lines, -9 removed lines

@@ -20,7 +20,7 @@ 
  */
 
 // Call header
-require __DIR__ . '/admin_header.php';
+require __DIR__.'/admin_header.php';
 // Display Admin header
 xoops_cp_header();
 // Define default value
@@ -31,11 +31,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':
@@ -55,7 +55,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 = '';
@@ -85,7 +85,7 @@ 
 
     case 'doreply':
         // check email
-        if (''== XoopsRequest::getString('contact_mailto', '', 'POST')) {
+        if ('' == XoopsRequest::getString('contact_mailto', '', 'POST')) {
             redirect_header('main.php', 3, _MD_CONTACT_MES_NOVALIDEMAIL);
         }
 
@@ -188,6 +188,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';

admin/about.php

Spacing +3 added lines, -3 removed lines

@@ -20,7 +20,7 @@ 
  */
 
 // Call header
-require __DIR__ . '/admin_header.php';
+require __DIR__.'/admin_header.php';
 // Display Admin header
 xoops_cp_header();
 
@@ -28,6 +28,6 @@ 
 $GLOBALS['xoopsTpl']->assign('renderabout', $adminObject->renderAbout('6KJ7RW5DR3VTJ', false));
 
 // Call template file
-$GLOBALS['xoopsTpl']->display(XOOPS_ROOT_PATH . '/modules/contact/templates/admin/contact_about.tpl');
+$GLOBALS['xoopsTpl']->display(XOOPS_ROOT_PATH.'/modules/contact/templates/admin/contact_about.tpl');
 // Call footer
-require __DIR__ . '/admin_footer.php';
+require __DIR__.'/admin_footer.php';

admin/index.php

Spacing +3 added lines, -3 removed lines

@@ -20,7 +20,7 @@ 
  */
 
 // Call header
-require __DIR__ . '/admin_header.php';
+require __DIR__.'/admin_header.php';
 // Display Admin header
 xoops_cp_header();
 
@@ -31,6 +31,6 @@ 
 $GLOBALS['xoopsTpl']->assign('renderindex', $adminObject->renderIndex());
 
 // Call template file
-$GLOBALS['xoopsTpl']->display(XOOPS_ROOT_PATH . '/modules/contact/templates/admin/contact_index.tpl');
+$GLOBALS['xoopsTpl']->display(XOOPS_ROOT_PATH.'/modules/contact/templates/admin/contact_index.tpl');
 // Call footer
-require __DIR__ . '/admin_footer.php';
+require __DIR__.'/admin_footer.php';

admin/menu.php

Spacing +6 added lines, -6 removed lines

@@ -25,40 +25,40 @@
 $moduleHandler = xoops_getHandler('module');
 /** @var XoopsModule $module */
 $module        = $moduleHandler->getByDirname(basename(dirname(__DIR__)));
-$pathIcon32    = '../../' . $module->getInfo('icons32');
+$pathIcon32    = '../../'.$module->getInfo('icons32');
 xoops_loadLanguage('modinfo', $module->dirname());
 
 $adminmenu[] = array(
     'title' => _MI_CONTACT_MENU_HOME,
     'desc'  => _MI_CONTACT_MENU_HOME_DESC,
-    'icon'  => $pathIcon32 . '/home.png',
+    'icon'  => $pathIcon32.'/home.png',
     'link'  => 'admin/index.php'
 );
 
 $adminmenu[] = array(
     'title' => _MI_CONTACT_MENU_CONTACT,
     'desc'  => _MI_CONTACT_MENU_CONTACT_DESC,
-    'icon'  => $pathIcon32 . '/content.png',
+    'icon'  => $pathIcon32.'/content.png',
     'link'  => 'admin/main.php'
 );
 
 $adminmenu[] = array(
     'title' => _MI_CONTACT_MENU_LOGS,
     'desc'  => _MI_CONTACT_MENU_LOGS_DESC,
-    'icon'  => $pathIcon32 . '/identity.png',
+    'icon'  => $pathIcon32.'/identity.png',
     'link'  => 'admin/log.php'
 );
 
 $adminmenu[] = array(
     'title' => _MI_CONTACT_MENU_TOOLS,
     'desc'  => _MI_CONTACT_MENU_TOOLS_DESC,
-    'icon'  => $pathIcon32 . '/delete.png',
+    'icon'  => $pathIcon32.'/delete.png',
     'link'  => 'admin/tools.php'
 );
 
 $adminmenu[] = array(
     'title' => _MI_CONTACT_MENU_ABOUT,
     'desc'  => _MI_CONTACT_MENU_ABOUT_DESC,
-    'icon'  => $pathIcon32 . '/about.png',
+    'icon'  => $pathIcon32.'/about.png',
     'link'  => 'admin/about.php'
 );