XOOPS / XoopsCore

htdocs/modules/banners/admin/index.php

Spacing +4 added lines, -4 removed lines

@@ -19,7 +19,7 @@ 
  * @author          Mage Gregory (AKA Mage)
  * @version         $Id: $
  */
-include __DIR__ . '/header.php';
+include __DIR__.'/header.php';
 $xoops = Xoops::getInstance();
 $helper = Banners::getInstance();
 
@@ -45,13 +45,13 @@ 
 $criteria->add(new Criteria('banner_status', 0));
 $banners_finish = $banner_Handler->getCount($criteria);
 // folder path
-$folder_path = $xoops_root_path . '/uploads/banners';
+$folder_path = $xoops_root_path.'/uploads/banners';
 
 $admin_page = new \Xoops\Module\Admin();
 $admin_page->addInfoBox(_MI_BANNERS_BANNERS);
 $admin_page->addInfoBoxLine(sprintf(_AM_BANNERS_INDEX_NBTOTAL, $banners_banner));
-$admin_page->addInfoBoxLine(sprintf(_AM_BANNERS_INDEX_NBCLIENT, '<span class="green">' . $banners_client . '</span>'));
-$admin_page->addInfoBoxLine(sprintf(_AM_BANNERS_INDEX_NBFINISH, '<span class="red">' . $banners_finish . '</span>'));
+$admin_page->addInfoBoxLine(sprintf(_AM_BANNERS_INDEX_NBCLIENT, '<span class="green">'.$banners_client.'</span>'));
+$admin_page->addInfoBoxLine(sprintf(_AM_BANNERS_INDEX_NBFINISH, '<span class="red">'.$banners_finish.'</span>'));
 $admin_page->addConfigBoxLine($folder_path, 'folder');
 $admin_page->addConfigBoxLine(array($folder_path, '777'), 'chmod');
 $admin_page->displayNavigation('index.php');

htdocs/modules/banners/admin/header.php

Spacing +1 added lines, -1 removed lines

@@ -19,4 +19,4 @@
  * @author          Mage Grégory (AKA Mage)
  * @version         $Id: $
  */
-require_once dirname(dirname(dirname(__DIR__))) . '/include/cp_header.php';
+require_once dirname(dirname(dirname(__DIR__))).'/include/cp_header.php';

htdocs/modules/banners/xoops_version.php

Spacing +1 added lines, -1 removed lines

@@ -44,7 +44,7 @@
 // paypal
 $modversion['paypal']                  = array();
 $modversion['paypal']['business']      = 'xoopsfoundation@gmail.com';
-$modversion['paypal']['item_name']     = 'Donation : ' . _MI_BANNERS_DESC;
+$modversion['paypal']['item_name']     = 'Donation : '._MI_BANNERS_DESC;
 $modversion['paypal']['amount']        = 0;
 $modversion['paypal']['currency_code'] = 'USD';
 

htdocs/modules/banners/preloads/preload.php

Spacing +2 added lines, -2 removed lines

@@ -34,13 +34,13 @@
     {
         $path = dirname(__DIR__);
         XoopsLoad::addMap(array(
-            'banners' => $path . '/class/helper.php',
+            'banners' => $path.'/class/helper.php',
         ));
     }
 
     public static function eventCoreBannerDisplay($args)
     {
-        require_once dirname(__DIR__) . '/class/bannerrender.php';
+        require_once dirname(__DIR__).'/class/bannerrender.php';
         $render = new BannerRender();
         $args[0] = $render->displayBanner();
     }

htdocs/modules/banners/include/install.php

Indentation +2 added lines, -2 removed lines

@@ -84,8 +84,8 @@
     $sql = "DELETE FROM " . $xoopsDB->prefix("config") . " WHERE `conf_name` = 'my_ip'";
     $xoopsDB->queryF($sql);
 */
-	$xoops_root_path = \XoopsBaseConfig::get('root-path');
-	$xoops_upload_url = \XoopsBaseConfig::get('uploads-url');
+    $xoops_root_path = \XoopsBaseConfig::get('root-path');
+    $xoops_upload_url = \XoopsBaseConfig::get('uploads-url');
 	
     // create folder "banners"
     $dir = $xoops_root_path . "/uploads/banners";

Spacing +11 added lines, -11 removed lines

@@ -32,7 +32,7 @@ 
     $xoops = Xoops::getInstance();
     //$xoops->db();
     //global $xoopsDB;
-    XoopsLoad::addMap(array('banners' => dirname(__DIR__) . '/class/helper.php'));
+    XoopsLoad::addMap(array('banners' => dirname(__DIR__).'/class/helper.php'));
     $helper = Banners::getInstance();
     // Get handler
     $banner_Handler = $helper->getHandlerBanner();
@@ -88,25 +88,25 @@ 
 	$xoops_upload_url = \XoopsBaseConfig::get('uploads-url');
 	
     // create folder "banners"
-    $dir = $xoops_root_path . "/uploads/banners";
+    $dir = $xoops_root_path."/uploads/banners";
     if (!is_dir($dir)) {
         mkdir($dir, 0777);
         chmod($dir, 0777);
     }
     //Copy index.html
-    $file = $xoops_root_path . "/uploads/banners/index.html";
+    $file = $xoops_root_path."/uploads/banners/index.html";
     if (!is_file($file)) {
-        copy($xoops_root_path . "/modules/banners/images/index.html", $file);
+        copy($xoops_root_path."/modules/banners/images/index.html", $file);
     }
     //Copy blank.gif
-    $file = $xoops_root_path . "/uploads/banners/blank.gif";
+    $file = $xoops_root_path."/uploads/banners/blank.gif";
     if (!is_file($file)) {
-        copy($xoops_root_path . "/uploads/blank.gif", $file);
+        copy($xoops_root_path."/uploads/blank.gif", $file);
     }
     //Copy .htaccess
-    $file = $xoops_root_path . "/uploads/banners/.htaccess";
+    $file = $xoops_root_path."/uploads/banners/.htaccess";
     if (!is_file($file)) {
-        copy($xoops_root_path . "/uploads/.htaccess", $file);
+        copy($xoops_root_path."/uploads/.htaccess", $file);
     }
 
 /* this should be in system upgrade, not module install
@@ -179,15 +179,15 @@ 
     );
     foreach ($banners as $k => $v) {
         //Copy banner
-        $file = $xoops_root_path . "/uploads/banners/" . $k;
-        $copy_file = $xoops_root_path . "/modules/banners/images/" . $k;
+        $file = $xoops_root_path."/uploads/banners/".$k;
+        $copy_file = $xoops_root_path."/modules/banners/images/".$k;
         if (!is_file($file) && is_file($copy_file)) {
             copy($copy_file, $file);
         }
         $obj = $banner_Handler->create();
         $obj->setVar("banner_cid", $newclient_id);
         $obj->setVar("banner_clickurl", $v);
-        $obj->setVar("banner_imageurl", $xoops_upload_url . '/banners/' . $k);
+        $obj->setVar("banner_imageurl", $xoops_upload_url.'/banners/'.$k);
         $obj->setVar("banner_datestart", time());
         $obj->setVar("banner_dateend", 0);
         $obj->setVar("banner_status", 1);

htdocs/modules/banners/language/english/blocks.php

Spacing +9 added lines, -9 removed lines

@@ -19,12 +19,12 @@
  * @author          Mage Grégory (AKA Mage)
  * @version         $Id$
  */
-define("_MB_BANNERS_DISP","Display");
-define("_MB_BANNERS","Banners");
-define("_MB_BANNERS_ALIGNEMENT","Alignement");
-define("_MB_BANNERS_ALIGNEMENT_H","horizontal");
-define("_MB_BANNERS_ALIGNEMENT_V","vertical");
-define("_MB_BANNERS_ALLCLIENTS","All clients");
-define("_MB_BANNERS_CLIENTSTODISPLAY","Select displayed clients");
-define("_MB_BANNERS_IDDISPLAY","Indicate the ID of the banner to be displayed");
-define("_MB_BANNERS_SEP","separated with ','.");
+define("_MB_BANNERS_DISP", "Display");
+define("_MB_BANNERS", "Banners");
+define("_MB_BANNERS_ALIGNEMENT", "Alignement");
+define("_MB_BANNERS_ALIGNEMENT_H", "horizontal");
+define("_MB_BANNERS_ALIGNEMENT_V", "vertical");
+define("_MB_BANNERS_ALLCLIENTS", "All clients");
+define("_MB_BANNERS_CLIENTSTODISPLAY", "Select displayed clients");
+define("_MB_BANNERS_IDDISPLAY", "Indicate the ID of the banner to be displayed");
+define("_MB_BANNERS_SEP", "separated with ','.");

htdocs/modules/banners/language/english/main.php

Spacing +1 added lines, -1 removed lines

@@ -23,7 +23,7 @@
 //define("_MD_BANNERS_INDEX_CLIENTNAME", "Displaying active banners for client: %s");
 define("_MD_BANNERS_INDEX_DBERROR", "Database was not updated due to some error!");
 define("_MD_BANNERS_INDEX_EMAIL", "Send E-mail stats");
-define("_MD_BANNERS_INDEX_NO_ID","No valid ID detected");
+define("_MD_BANNERS_INDEX_NO_ID", "No valid ID detected");
 define("_MD_BANNERS_INDEX_NO_REFERER", "No referer detected");
 define("_MD_BANNERS_INDEX_ID", "ID");
 define("_MD_BANNERS_INDEX_MAIL_MESSAGE", "Available Banner Statistics for the selected Banner at %s :\n\n\n

htdocs/modules/banners/language/english/admin.php

Spacing +45 added lines, -45 removed lines

@@ -20,64 +20,64 @@
  * @version         $Id$
  */
 //generic
-define("_AM_BANNERS_ACTION","Action");
-define("_AM_BANNERS_VIEW","View");
-define("_AM_BANNERS_DELETE","Delete");
-define("_AM_BANNERS_EDIT","Edit");
-define("_AM_BANNERS_DBUPDATED","Database updated successfully!");
+define("_AM_BANNERS_ACTION", "Action");
+define("_AM_BANNERS_VIEW", "View");
+define("_AM_BANNERS_DELETE", "Delete");
+define("_AM_BANNERS_EDIT", "Edit");
+define("_AM_BANNERS_DBUPDATED", "Database updated successfully!");
 
 //index.php
-define("_AM_BANNERS_INDEX_NBTOTAL","There are %s banners in our database");
-define("_AM_BANNERS_INDEX_NBCLIENT","There are %s client in our database");
-define("_AM_BANNERS_INDEX_NBFINISH","There are %s banners finished");
+define("_AM_BANNERS_INDEX_NBTOTAL", "There are %s banners in our database");
+define("_AM_BANNERS_INDEX_NBCLIENT", "There are %s client in our database");
+define("_AM_BANNERS_INDEX_NBFINISH", "There are %s banners finished");
 
 //clients.php
-define("_AM_BANNERS_CLIENTS_ACTIVEBANNERS","Active Banners");
-define("_AM_BANNERS_CLIENTS_ADD","Add a new client");
-define("_AM_BANNERS_CLIENTS_EDIT","Edit client");
-define("_AM_BANNERS_CLIENTS_EXTRAINFO","Extra Info");
-define("_AM_BANNERS_CLIENTS_LIST","List of clients");
-define("_AM_BANNERS_CLIENTS_MAIL","Client email");
-define("_AM_BANNERS_CLIENTS_NAME","Client name");
-define("_AM_BANNERS_CLIENTS_SUREDEL","Are you sure to delete this client '<strong>%s</strong>' and ALL its banners?");
-define("_AM_BANNERS_CLIENTS_UNAME","User name");
-define("_AM_BANNERS_CLIENTS_UNAME_NO","Do not use a member");
-define("_AM_BANNERS_CLIENTS_UNAME_YES","Use a member");
+define("_AM_BANNERS_CLIENTS_ACTIVEBANNERS", "Active Banners");
+define("_AM_BANNERS_CLIENTS_ADD", "Add a new client");
+define("_AM_BANNERS_CLIENTS_EDIT", "Edit client");
+define("_AM_BANNERS_CLIENTS_EXTRAINFO", "Extra Info");
+define("_AM_BANNERS_CLIENTS_LIST", "List of clients");
+define("_AM_BANNERS_CLIENTS_MAIL", "Client email");
+define("_AM_BANNERS_CLIENTS_NAME", "Client name");
+define("_AM_BANNERS_CLIENTS_SUREDEL", "Are you sure to delete this client '<strong>%s</strong>' and ALL its banners?");
+define("_AM_BANNERS_CLIENTS_UNAME", "User name");
+define("_AM_BANNERS_CLIENTS_UNAME_NO", "Do not use a member");
+define("_AM_BANNERS_CLIENTS_UNAME_YES", "Use a member");
 
 //banners.php
-define("_AM_BANNERS_BANNERS_ADD","Add a new banner");
-define("_AM_BANNERS_BANNERS_CLICKS","Clicks");
-define("_AM_BANNERS_BANNERS_CLICKURL","Click URL");
-define("_AM_BANNERS_BANNERS_CODEHTML","Code HTML");
-define("_AM_BANNERS_BANNERS_CURRENT","Current active banners");
-define("_AM_BANNERS_BANNERS_EDIT","Edit banner");
-define("_AM_BANNERS_BANNERS_ENDDATE","Date ended");
-define("_AM_BANNERS_BANNERS_FINISH","Finished banners");
-define("_AM_BANNERS_BANNERS_IMAGE","Image");
-define("_AM_BANNERS_BANNERS_IMAGE_PATH","Files exist in: %s");
-define("_AM_BANNERS_BANNERS_IMGURL","Image URL:");
-define("_AM_BANNERS_BANNERS_IMPRESIONLEFT","Impressions left");
-define("_AM_BANNERS_BANNERS_IMPRESSIONS","Impressions");
-define("_AM_BANNERS_BANNERS_IMPRESSIONSP","Impressions purchased");
-define("_AM_BANNERS_BANNERS_IMPRESSIONSP_PATTERN","You must enter an integer");
-define("_AM_BANNERS_BANNERS_LIST","List of banners");
-define("_AM_BANNERS_BANNERS_NCLICKS","% Clicks");
-define("_AM_BANNERS_BANNERS_RELOAD","Reload banner");
-define("_AM_BANNERS_BANNERS_STARTDATE","Date started");
-define("_AM_BANNERS_BANNERS_SUREDEL","Are you sure to delete this Banner?");
-define("_AM_BANNERS_BANNERS_UNLIMIT","Unlimited");
-define("_AM_BANNERS_BANNERS_UPLOADS","Upload");
-define("_AM_BANNERS_BANNERS_USEHTML","Use HTML?");
+define("_AM_BANNERS_BANNERS_ADD", "Add a new banner");
+define("_AM_BANNERS_BANNERS_CLICKS", "Clicks");
+define("_AM_BANNERS_BANNERS_CLICKURL", "Click URL");
+define("_AM_BANNERS_BANNERS_CODEHTML", "Code HTML");
+define("_AM_BANNERS_BANNERS_CURRENT", "Current active banners");
+define("_AM_BANNERS_BANNERS_EDIT", "Edit banner");
+define("_AM_BANNERS_BANNERS_ENDDATE", "Date ended");
+define("_AM_BANNERS_BANNERS_FINISH", "Finished banners");
+define("_AM_BANNERS_BANNERS_IMAGE", "Image");
+define("_AM_BANNERS_BANNERS_IMAGE_PATH", "Files exist in: %s");
+define("_AM_BANNERS_BANNERS_IMGURL", "Image URL:");
+define("_AM_BANNERS_BANNERS_IMPRESIONLEFT", "Impressions left");
+define("_AM_BANNERS_BANNERS_IMPRESSIONS", "Impressions");
+define("_AM_BANNERS_BANNERS_IMPRESSIONSP", "Impressions purchased");
+define("_AM_BANNERS_BANNERS_IMPRESSIONSP_PATTERN", "You must enter an integer");
+define("_AM_BANNERS_BANNERS_LIST", "List of banners");
+define("_AM_BANNERS_BANNERS_NCLICKS", "% Clicks");
+define("_AM_BANNERS_BANNERS_RELOAD", "Reload banner");
+define("_AM_BANNERS_BANNERS_STARTDATE", "Date started");
+define("_AM_BANNERS_BANNERS_SUREDEL", "Are you sure to delete this Banner?");
+define("_AM_BANNERS_BANNERS_UNLIMIT", "Unlimited");
+define("_AM_BANNERS_BANNERS_UPLOADS", "Upload");
+define("_AM_BANNERS_BANNERS_USEHTML", "Use HTML?");
 
-define("_AM_BANNERS_BANNERS_ERROR_NOCLIENT","Please create first a client");
+define("_AM_BANNERS_BANNERS_ERROR_NOCLIENT", "Please create first a client");
 
 
 // Tips
 define("_AM_BANNERS_TIPS_CLIENTS", "<ul><li>Add, modify and update clients.</li></ul>");
 define("_AM_BANNERS_TIPS_BANNERS", "<ul><li>Add, modify and update banners.</li></ul>");
 // Alert
-define("_AM_BANNERS_ALERT_INFO_TITLE_UPLOADS","Information for uploads");
-define("_AM_BANNERS_ALERT_INFO_TITLE","Information");
+define("_AM_BANNERS_ALERT_INFO_TITLE_UPLOADS", "Information for uploads");
+define("_AM_BANNERS_ALERT_INFO_TITLE", "Information");
 define("_AM_BANNERS_ALERT_INFO_CLIENT_ADDEDIT", "If you do not put a user of the site, the customer will not have access to these banners.");
 define("_AM_BANNERS_ALERT_INFO_MIMETYPES", "Authorized mime types: %s");
 define("_AM_BANNERS_ALERT_INFO_MAXFILE", "Max uploaded files size: %s [KB]");

htdocs/modules/banners/blocks/banners_blocks.php

Spacing +17 added lines, -17 removed lines

@@ -22,7 +22,7 @@ 
 function banners_blocks_show($options) {
     $block = array();
     $xoops = Xoops::getInstance();
-    require_once dirname(__DIR__) . '/class/bannerrender.php';
+    require_once dirname(__DIR__).'/class/bannerrender.php';
     $render = new BannerRender();
     switch ($options[0]) {
         case 'random':
@@ -49,10 +49,10 @@ 
     switch ($options[0]) {
 
         case 'random':
-            $form .= _MB_BANNERS_DISP . " \n";
-            $form .= "<input type=\"hidden\" name=\"options[0]\" value=\"" . $options[0] . "\" />\n";
-            $form .= "<input name=\"options[1]\" size=\"5\" maxlength=\"255\" value=\"" . $options[1] . "\" type=\"text\" />&nbsp;" . _MB_BANNERS . "<br />\n";
-            switch ($options[2]){
+            $form .= _MB_BANNERS_DISP."&nbsp;\n";
+            $form .= "<input type=\"hidden\" name=\"options[0]\" value=\"".$options[0]."\" />\n";
+            $form .= "<input name=\"options[1]\" size=\"5\" maxlength=\"255\" value=\"".$options[1]."\" type=\"text\" />&nbsp;"._MB_BANNERS."<br />\n";
+            switch ($options[2]) {
                 case 'H':
                     $checked_H = 'checked="checked"';
                     $checked_V = '';
@@ -63,30 +63,30 @@ 
                     $checked_V = 'checked="checked"';
                     break;
             }
-            $form .= _MB_BANNERS_ALIGNEMENT . " : <input name=\"options[2]\" value=\"H\" type=\"radio\" " . $checked_H . "/>" . _MB_BANNERS_ALIGNEMENT_H . "&nbsp;\n";
-            $form .= "<input name=\"options[2]\" value=\"V\" type=\"radio\" " . $checked_V . "/>" . _MB_BANNERS_ALIGNEMENT_V . "<br />\n";
+            $form .= _MB_BANNERS_ALIGNEMENT." : <input name=\"options[2]\" value=\"H\" type=\"radio\" ".$checked_H."/>"._MB_BANNERS_ALIGNEMENT_H."&nbsp;\n";
+            $form .= "<input name=\"options[2]\" value=\"V\" type=\"radio\" ".$checked_V."/>"._MB_BANNERS_ALIGNEMENT_V."<br />\n";
             array_shift($options);
             array_shift($options);
             array_shift($options);
-            $form .= _MB_BANNERS_CLIENTSTODISPLAY . "<br /><select name=\"options[]\" multiple=\"multiple\" size=\"5\">\n";
+            $form .= _MB_BANNERS_CLIENTSTODISPLAY."<br /><select name=\"options[]\" multiple=\"multiple\" size=\"5\">\n";
             $xoops = Xoops::getInstance();
-            $client_Handler = $xoops->getModuleHandler('bannerclient','banners');
+            $client_Handler = $xoops->getModuleHandler('bannerclient', 'banners');
             $criteria = new CriteriaCompo();
             $criteria->setSort('bannerclient_name');
             $criteria->setOrder('ASC');
             $client_arr = $client_Handler->getAll($criteria);
-            $form .= "<option value=\"0\" " . (array_search(0, $options) === false ? '' : 'selected="selected"') . ">" . _MB_BANNERS_ALLCLIENTS . "</option>\n";
+            $form .= "<option value=\"0\" ".(array_search(0, $options) === false ? '' : 'selected="selected"').">"._MB_BANNERS_ALLCLIENTS."</option>\n";
             foreach (array_keys($client_arr) as $i) {
-                $form .= "<option value=\"" . $client_arr[$i]->getVar('cid') . "\" " . (array_search($client_arr[$i]->getVar('cid'), $options) === false ? '' : 'selected="selected"') . ">" . $client_arr[$i]->getVar('name')."</option>\n";
+                $form .= "<option value=\"".$client_arr[$i]->getVar('cid')."\" ".(array_search($client_arr[$i]->getVar('cid'), $options) === false ? '' : 'selected="selected"').">".$client_arr[$i]->getVar('name')."</option>\n";
             }
             $form .= "</select>\n";
             break;
 
         case 'id':
-            $form .= _MB_BANNERS_IDDISPLAY . "&nbsp;\n";
-            $form .= "<input type=\"hidden\" name=\"options[0]\" value=\"" . $options[0] . "\" />\n";
-            $form .= "<input name=\"options[1]\" size=\"20\" maxlength=\"255\" value=\"" . $options[1] . "\" type=\"text\" />&nbsp;" . _MB_BANNERS_SEP . "<br />\n";
-            switch ($options[2]){
+            $form .= _MB_BANNERS_IDDISPLAY."&nbsp;\n";
+            $form .= "<input type=\"hidden\" name=\"options[0]\" value=\"".$options[0]."\" />\n";
+            $form .= "<input name=\"options[1]\" size=\"20\" maxlength=\"255\" value=\"".$options[1]."\" type=\"text\" />&nbsp;"._MB_BANNERS_SEP."<br />\n";
+            switch ($options[2]) {
                 case 'H':
                     $checked_H = 'checked="checked"';
                     $checked_V = '';
@@ -97,8 +97,8 @@ 
                     $checked_V = 'checked="checked"';
                     break;
             }
-            $form .= _MB_BANNERS_ALIGNEMENT . " : <input name=\"options[2]\" value=\"H\" type=\"radio\" " . $checked_H . "/>" . _MB_BANNERS_ALIGNEMENT_H . "&nbsp;\n";
-            $form .= "<input name=\"options[2]\" value=\"V\" type=\"radio\" " . $checked_V . "/>" . _MB_BANNERS_ALIGNEMENT_V . "<br />\n";
+            $form .= _MB_BANNERS_ALIGNEMENT." : <input name=\"options[2]\" value=\"H\" type=\"radio\" ".$checked_H."/>"._MB_BANNERS_ALIGNEMENT_H."&nbsp;\n";
+            $form .= "<input name=\"options[2]\" value=\"V\" type=\"radio\" ".$checked_V."/>"._MB_BANNERS_ALIGNEMENT_V."<br />\n";
             break;
     }
     return $form;