XOOPS / XoopsCore

htdocs/modules/banners/admin/clients.php

Spacing +4 added lines, -4 removed lines

@@ -21,7 +21,7 @@ 
  * @author          Mage Grégory (AKA Mage)
  * @version         $Id$
  */
-include __DIR__ . '/header.php';
+include __DIR__.'/header.php';
 
 // Get main instance
 $xoops = Xoops::getInstance();
@@ -176,9 +176,9 @@ 
                             $banner_arr[$i]->getVar('banner_imageurl'),
                             '',
                             0,
-                            strlen($xoops_url . '/uploads/banners/')
+                            strlen($xoops_url.'/uploads/banners/')
                         );
-                        $urlfile =  $xoops_root_path . '/uploads/banners/' . $namefile;
+                        $urlfile = $xoops_root_path.'/uploads/banners/'.$namefile;
                         if ($banner_Handler->delete($obj)) {
                             // delete banner
                             if (is_file($urlfile)) {
@@ -197,7 +197,7 @@ 
                 echo $xoops->confirm(
                     array("ok" => 1, "cid" => $cid, "op" => "delete"),
                     'clients.php',
-                    sprintf(_AM_BANNERS_CLIENTS_SUREDEL, $obj->getVar("bannerclient_name")) . '<br />'
+                    sprintf(_AM_BANNERS_CLIENTS_SUREDEL, $obj->getVar("bannerclient_name")).'<br />'
                 );
             }
         } else {

htdocs/modules/banners/admin/banners.php

Spacing +16 added lines, -16 removed lines

@@ -21,7 +21,7 @@ 
  * @author          Mage Grégory (AKA Mage)
  * @version         $Id: $
  */
-include __DIR__ . '/header.php';
+include __DIR__.'/header.php';
 // Get main instance
 $xoops = Xoops::getInstance();
 $helper = Banners::getInstance();
@@ -101,7 +101,7 @@ 
                     $percent = substr(100 * $clicks / $impmade, 0, 5);
                 }
                 if ($imptotal == 0) {
-                    $left = "" . _AM_BANNERS_BANNERS_UNLIMIT . "";
+                    $left = ""._AM_BANNERS_BANNERS_UNLIMIT."";
                 } else {
                     $left = $imptotal - $impmade;
                 }
@@ -110,9 +110,9 @@ 
                     $img .= html_entity_decode($htmlcode);
                 } else {
                     if (stristr($imageurl, '.swf')) {
-                        $img .= '<object type="application/x-shockwave-flash" width="468" height="60" data="' . $imageurl . '" style="z-index:100;">' . '<param name="movie" value="' . $imageurl . '" />' . '<param name="wmode" value="opaque" />' . '</object>';
+                        $img .= '<object type="application/x-shockwave-flash" width="468" height="60" data="'.$imageurl.'" style="z-index:100;">'.'<param name="movie" value="'.$imageurl.'" />'.'<param name="wmode" value="opaque" />'.'</object>';
                     } else {
-                        $img .= '<img src="' . $imageurl . '" alt="" />';
+                        $img .= '<img src="'.$imageurl.'" alt="" />';
                     }
                 }
 
@@ -130,7 +130,7 @@ 
         }
         // Display Page Navigation
         if ($banner_count > $nb_banners) {
-            $nav = new XoopsPageNav($banner_count, $nb_banners, $start, 'start', 'startF=' . $startF);
+            $nav = new XoopsPageNav($banner_count, $nb_banners, $start, 'start', 'startF='.$startF);
             $xoops->tpl()->assign('nav_menu_banner', $nav->renderNav(4));
         }
         // Display Finished Banners
@@ -166,9 +166,9 @@ 
                 } else {
                     $img = '<div id="xo-bannerfix">';
                     if (stristr($imageurl, '.swf')) {
-                        $img .= '<object type="application/x-shockwave-flash" width="468" height="60" data="' . $imageurl . '" style="z-index:100;">' . '<param name="movie" value="' . $imageurl . '" />' . '<param name="wmode" value="opaque" />' . '</object>';
+                        $img .= '<object type="application/x-shockwave-flash" width="468" height="60" data="'.$imageurl.'" style="z-index:100;">'.'<param name="movie" value="'.$imageurl.'" />'.'<param name="wmode" value="opaque" />'.'</object>';
                     } else {
-                        $img .= '<img src="' . $imageurl . '" alt="" />';
+                        $img .= '<img src="'.$imageurl.'" alt="" />';
                     }
 
                     $img .= '</div>';
@@ -193,7 +193,7 @@ 
         }
         // Display Page Navigation
         if ($banner_finish_count > $nb_banners) {
-            $nav = new XoopsPageNav($banner_finish_count, $nb_banners, $startF, 'startF', 'start=' . $start);
+            $nav = new XoopsPageNav($banner_finish_count, $nb_banners, $startF, 'startF', 'start='.$start);
             $xoops->tpl()->assign('nav_menu_bannerF', $nav->renderNav(4));
         }
         break;
@@ -237,7 +237,7 @@ 
         $error_msg = '';
         $obj->setVar("banner_cid", Request::getInt('cid', 0));
         if (preg_match('/^[0-9]*[0-9]+$|^[0-9]+[0-9]*$/', $_POST["imptotal"]) == false) {
-            $error_msg .= XoopsLocale::E_YOU_NEED_A_POSITIVE_INTEGER . '<br />';
+            $error_msg .= XoopsLocale::E_YOU_NEED_A_POSITIVE_INTEGER.'<br />';
             $obj->setVar("banner_imptotal", 0);
         } else {
             $obj->setVar("banner_imptotal", Request::getInt('imptotal', 0));
@@ -246,7 +246,7 @@ 
         $obj->setVar("banner_htmlbanner", Request::getInt('htmlbanner', 0));
         $obj->setVar("banner_htmlcode", Request::getString('htmlcode', ''));
 		
-        $uploader_banners_img = new XoopsMediaUploader($xoops_upload_path . '/banners', $mimetypes, $upload_size, null, null);
+        $uploader_banners_img = new XoopsMediaUploader($xoops_upload_path.'/banners', $mimetypes, $upload_size, null, null);
 		
         if ($uploader_banners_img->fetchMedia("banners_imageurl")) {
             $uploader_banners_img->setPrefix("banner");
@@ -254,13 +254,13 @@ 
             if (!$uploader_banners_img->upload()) {
                 $error_msg .= $uploader_banners_img->getErrors();
             } else {
-                $obj->setVar("banner_imageurl", $xoops_upload_url . '/banners/' . $uploader_banners_img->getSavedFileName());
+                $obj->setVar("banner_imageurl", $xoops_upload_url.'/banners/'.$uploader_banners_img->getSavedFileName());
             }
         } else {
             if ($_POST["banners_imageurl"] === 'blank.gif') {
                 $obj->setVar("banner_imageurl", Request::getString('imageurl', ''));
             } else {
-                $obj->setVar("banner_imageurl", $xoops_upload_url . '/banners/' . Request::getString('banners_imageurl', ''));
+                $obj->setVar("banner_imageurl", $xoops_upload_url.'/banners/'.Request::getString('banners_imageurl', ''));
             }
         }
 
@@ -286,8 +286,8 @@ 
                 if (!$xoops->security()->check()) {
                     $xoops->redirect("banners.php", 3, implode(",", $xoops->security()->getErrors()));
                 }
-                $namefile = substr_replace($obj->getVar('imageurl'), '', 0, strlen($xoops_url . '/uploads/banners/'));
-                $urlfile =  $xoops_root_path . '/uploads/banners/' . $namefile;
+                $namefile = substr_replace($obj->getVar('imageurl'), '', 0, strlen($xoops_url.'/uploads/banners/'));
+                $urlfile = $xoops_root_path.'/uploads/banners/'.$namefile;
                 if ($banner_Handler->delete($obj)) {
                     // delete banner
                     if (is_file($urlfile)) {
@@ -311,10 +311,10 @@ 
                         $img .= "</embed>";
                         $img .= "</object>";
                     } else {
-                        $img .= "<img src='" . $imageurl . "' alt='' />";
+                        $img .= "<img src='".$imageurl."' alt='' />";
                     }
                 }
-                echo $xoops->confirm(array("ok" => 1, "bid" => $bid, "op" => "delete"), 'banners.php', XoopsLocale::Q_ARE_YOU_SURE_YOU_WANT_TO_DELETE_THIS_ITEM . '<br \>' . $img . '<br \>');
+                echo $xoops->confirm(array("ok" => 1, "bid" => $bid, "op" => "delete"), 'banners.php', XoopsLocale::Q_ARE_YOU_SURE_YOU_WANT_TO_DELETE_THIS_ITEM.'<br \>'.$img.'<br \>');
             }
         } else {
             $xoops->redirect('banners.php', 1, XoopsLocale::E_DATABASE_NOT_UPDATED);

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