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'>

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/contact.php

Spacing +10 added lines, -10 removed lines

@@ -81,7 +81,7 @@ 
         $form->addElement(new XoopsFormLabel(_AM_CONTACT_TO, '', ''));
         $form->addElement(new XoopsFormText(_AM_CONTACT_NAMETO, 'contact_nameto', 50, 255, $this->getVar('contact_name')), true);
         $form->addElement(new XoopsFormText(_AM_CONTACT_MAILTO, 'contact_mailto', 50, 255, $this->getVar('contact_mail')), true);
-        $form->addElement(new XoopsFormText(_AM_CONTACT_SUBJECT, 'contact_subject', 50, 255, _RE . $this->getVar('contact_subject')), true);
+        $form->addElement(new XoopsFormText(_AM_CONTACT_SUBJECT, 'contact_subject', 50, 255, _RE.$this->getVar('contact_subject')), true);
         $form->addElement(new XoopsFormTextArea(_AM_CONTACT_MESSAGE, 'contact_message', '', 5, 60), true);
         $form->addElement(new XoopsFormButton('', 'submit', _SUBMIT, 'submit'));
 
@@ -96,7 +96,7 @@ 
     public function toArray()
     {
         $ret  = array();
-        $vars =& $this->getVars();
+        $vars = & $this->getVars();
         foreach (array_keys($vars) as $i) {
             $ret [$i] = $this->getVar($i);
         }
@@ -210,7 +210,7 @@ 
      */
     public function contactSendMail($contact)
     {
-        $xoopsMailer =& xoops_getMailer();
+        $xoopsMailer = & xoops_getMailer();
         $xoopsMailer->useMail();
         $xoopsMailer->setToEmails($this->contactToEmails($contact['contact_department']));
         $xoopsMailer->setFromEmail($contact['contact_mail']);
@@ -218,9 +218,9 @@ 
 
         $subjectPrefix = '';
         if ($GLOBALS['xoopsModuleConfig']['form_dept'] && $GLOBALS['xoopsModuleConfig']['subject_prefix'] && $GLOBALS['xoopsModuleConfig']['contact_dept']) {
-            $subjectPrefix = '[' . $GLOBALS['xoopsModuleConfig']['prefix_text'] . ' ' . $contact['contact_department'] . ']: ';
+            $subjectPrefix = '['.$GLOBALS['xoopsModuleConfig']['prefix_text'].' '.$contact['contact_department'].']: ';
         }
-        $xoopsMailer->setSubject($subjectPrefix . html_entity_decode($contact['contact_subject'], ENT_QUOTES, 'UTF-8'));
+        $xoopsMailer->setSubject($subjectPrefix.html_entity_decode($contact['contact_subject'], ENT_QUOTES, 'UTF-8'));
         $xoopsMailer->setBody(html_entity_decode($contact['contact_message'], ENT_QUOTES, 'UTF-8'));
         if ($xoopsMailer->send()) {
             $message = _MD_CONTACT_MES_SEND;
@@ -237,7 +237,7 @@ 
      */
     public function contactSendMailConfirm($contact)
     {
-        $xoopsMailer =& xoops_getMailer();
+        $xoopsMailer = & xoops_getMailer();
         $xoopsMailer->useMail();
         $xoopsMailer->setToEmails($contact['contact_mail']);
         $xoopsMailer->setFromEmail($this->contactToEmails($contact['contact_department']));
@@ -263,7 +263,7 @@ 
      */
     public function contactReplyMail($contact)
     {
-        $xoopsMailer =& xoops_getMailer();
+        $xoopsMailer = & xoops_getMailer();
         $xoopsMailer->useMail();
         $xoopsMailer->setToEmails($contact['contact_mailto']);
         $xoopsMailer->setFromEmail($contact['contact_mail']);
@@ -324,7 +324,7 @@ 
         $criteria = new CriteriaCompo();
         $criteria->add(new Criteria('contact_cid', $contact_id));
         $criteria->add(new Criteria('contact_type', 'Contact'));
-        $contacts =& $this->getObjects($criteria, false);
+        $contacts = & $this->getObjects($criteria, false);
         if ($contacts) {
             $ret = array();
             /** @var Contact $root */
@@ -357,7 +357,7 @@ 
         $criteria->setOrder($contact['order']);
         $criteria->setStart($contact['start']);
         $criteria->setLimit($contact['limit']);
-        $contacts =& $this->getObjects($criteria, false);
+        $contacts = & $this->getObjects($criteria, false);
         if ($contacts) {
             /** @var Contact $root */
             foreach ($contacts as $root) {
@@ -497,7 +497,7 @@ 
         }
         $criteria->setSort('contact_create');
         $criteria->setOrder('DESC');
-        $contacts =& $this->getObjects($criteria, false);
+        $contacts = & $this->getObjects($criteria, false);
         if ($contacts) {
             /** @var Contact $root */
             foreach ($contacts as $root) {

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';