Passed
Pull Request — master (#610)
by Richard
14:27
created
htdocs/modules/phpmailer/admin/header.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -17,4 +17,4 @@
 block discarded – undo
17 17
  * @package   phpmailer
18 18
  * @author    Richard Griffith <[email protected]>
19 19
  */
20
-require_once dirname(dirname(dirname(__DIR__))) . '/include/cp_header.php';
20
+require_once dirname(dirname(dirname(__DIR__))).'/include/cp_header.php';
Please login to merge, or discard this patch.
htdocs/modules/phpmailer/xoops_version.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -43,7 +43,7 @@
 block discarded – undo
43 43
 // paypal
44 44
 $modversion['paypal'] = array(
45 45
     'business'      => '[email protected]',
46
-    'item_name'     => 'Donation : ' . _MI_PHPMAILER_NAME,
46
+    'item_name'     => 'Donation : '._MI_PHPMAILER_NAME,
47 47
     'amount'        => 0,
48 48
     'currency_code' => 'USD',
49 49
 );
Please login to merge, or discard this patch.
htdocs/modules/phpmailer/preloads/preload.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -34,7 +34,7 @@  discard block
 block discarded – undo
34 34
      */
35 35
     public static function eventCoreServiceLocateEmail(Provider $provider)
36 36
     {
37
-        $path = dirname(__DIR__) . '/class/PhpMailerEmailProvider.php';
37
+        $path = dirname(__DIR__).'/class/PhpMailerEmailProvider.php';
38 38
         require $path;
39 39
         $object = new PhpMailerEmailProvider();
40 40
         $provider->register($object);
@@ -49,7 +49,7 @@  discard block
 block discarded – undo
49 49
      */
50 50
     public static function eventCoreServiceLocateUserEmailMessage(Provider $provider)
51 51
     {
52
-        $path = dirname(__DIR__) . '/class/PhpMailerMessageProvider.php';
52
+        $path = dirname(__DIR__).'/class/PhpMailerMessageProvider.php';
53 53
         require $path;
54 54
         $object = new PhpMailerMessageProvider();
55 55
         $provider->register($object);
Please login to merge, or discard this patch.
htdocs/modules/phpmailer/language/english/modinfo.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -15,8 +15,8 @@
 block discarded – undo
15 15
  * @author    Richard Griffith <[email protected]>
16 16
  */
17 17
 
18
-define('_MI_PHPMAILER_NAME','PHPMailer');
19
-define('_MI_PHPMAILER_DESC','Use PHPMailer for Email service');
18
+define('_MI_PHPMAILER_NAME', 'PHPMailer');
19
+define('_MI_PHPMAILER_DESC', 'Use PHPMailer for Email service');
20 20
 
21 21
 define('_MI_PHPMAILER_CONF_SMTPHOST', 'SMTP host(s)');
22 22
 define('_MI_PHPMAILER_CONF_SMTPHOST_DESC', 'List of SMTP servers to try to connect to separated by a ";". Example: localhost:25;tls://smtp.example.com:587');
Please login to merge, or discard this patch.
htdocs/modules/phpmailer/class/PhpMailerEmailProvider.php 1 patch
Spacing   +7 added lines, -7 removed lines patch added patch discarded remove patch
@@ -73,11 +73,11 @@  discard block
 block discarded – undo
73 73
     {
74 74
         // Addresses
75 75
         $address = $email->getFromAddress();
76
-        $mailer->setFrom($address->getEmail(), (string) $address->getDisplayName());
76
+        $mailer->setFrom($address->getEmail(), (string)$address->getDisplayName());
77 77
 
78 78
         $list = $email->getToAddresses();
79 79
         foreach ($list->getEachAddress() as $address) {
80
-            $mailer->addAddress($address->getEmail(), (string) $address->getDisplayName());
80
+            $mailer->addAddress($address->getEmail(), (string)$address->getDisplayName());
81 81
         }
82 82
 
83 83
         $list = $email->getReplyToAddresses();
@@ -107,8 +107,8 @@  discard block
 block discarded – undo
107 107
             foreach ($attachmentSet->getEachAttachment() as $attachment) {
108 108
                 $file = $attachment->getFilename();
109 109
                 $body = $attachment->getStringBody();
110
-                $name = (string) $attachment->getName();
111
-                $type = (string) $attachment->getMimeType();
110
+                $name = (string)$attachment->getName();
111
+                $type = (string)$attachment->getMimeType();
112 112
                 $inline = $attachment->getInlineAttribute();
113 113
                 if (null !== $file && !$inline) {
114 114
                     $mailer->addAttachment($file, $name, 'base64', $type);
@@ -133,7 +133,7 @@  discard block
 block discarded – undo
133 133
         }
134 134
 
135 135
         $mailer->isHTML(false);
136
-        $mailer->Body= $email->getBody();
136
+        $mailer->Body = $email->getBody();
137 137
 
138 138
         return $mailer;
139 139
     }
@@ -163,8 +163,8 @@  discard block
 block discarded – undo
163 163
             case 'smtp':
164 164
                 $mailer->isSMTP();
165 165
                 $mailer->Host = $helper->getConfig('smtp_host', $mailer->Host);
166
-                $mailer->SMTPAutoTLS = (bool) $helper->getConfig('smtp_usetls', true);
167
-                $mailer->SMTPDebug = (int) $helper->getConfig('smtp_debug', 0);
166
+                $mailer->SMTPAutoTLS = (bool)$helper->getConfig('smtp_usetls', true);
167
+                $mailer->SMTPDebug = (int)$helper->getConfig('smtp_debug', 0);
168 168
                 break;
169 169
             case 'mail':
170 170
             default:
Please login to merge, or discard this patch.
htdocs/modules/banners/admin/about.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -19,7 +19,7 @@
 block discarded – undo
19 19
  * @author          Mage Gregory (AKA Mage)
20 20
  * @version         $Id: $
21 21
  */
22
-include __DIR__ . '/header.php';
22
+include __DIR__.'/header.php';
23 23
 $xoops = Xoops::getInstance();
24 24
 $xoops->header();
25 25
 $aboutAdmin = new \Xoops\Module\Admin();
Please login to merge, or discard this patch.
htdocs/modules/banners/admin/clients.php 1 patch
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -21,7 +21,7 @@  discard block
 block discarded – undo
21 21
  * @author          Mage Grégory (AKA Mage)
22 22
  * @version         $Id$
23 23
  */
24
-include __DIR__ . '/header.php';
24
+include __DIR__.'/header.php';
25 25
 
26 26
 // Get main instance
27 27
 $xoops = Xoops::getInstance();
@@ -176,9 +176,9 @@  discard block
 block discarded – undo
176 176
                             $banner_arr[$i]->getVar('banner_imageurl'),
177 177
                             '',
178 178
                             0,
179
-                            strlen($xoops_url . '/uploads/banners/')
179
+                            strlen($xoops_url.'/uploads/banners/')
180 180
                         );
181
-                        $urlfile =  $xoops_root_path . '/uploads/banners/' . $namefile;
181
+                        $urlfile = $xoops_root_path.'/uploads/banners/'.$namefile;
182 182
                         if ($banner_Handler->delete($obj)) {
183 183
                             // delete banner
184 184
                             if (is_file($urlfile)) {
@@ -197,7 +197,7 @@  discard block
 block discarded – undo
197 197
                 echo $xoops->confirm(
198 198
                     array("ok" => 1, "cid" => $cid, "op" => "delete"),
199 199
                     'clients.php',
200
-                    sprintf(_AM_BANNERS_CLIENTS_SUREDEL, $obj->getVar("bannerclient_name")) . '<br />'
200
+                    sprintf(_AM_BANNERS_CLIENTS_SUREDEL, $obj->getVar("bannerclient_name")).'<br />'
201 201
                 );
202 202
             }
203 203
         } else {
Please login to merge, or discard this patch.
htdocs/modules/banners/admin/banners.php 1 patch
Spacing   +16 added lines, -16 removed lines patch added patch discarded remove patch
@@ -21,7 +21,7 @@  discard block
 block discarded – undo
21 21
  * @author          Mage Grégory (AKA Mage)
22 22
  * @version         $Id: $
23 23
  */
24
-include __DIR__ . '/header.php';
24
+include __DIR__.'/header.php';
25 25
 // Get main instance
26 26
 $xoops = Xoops::getInstance();
27 27
 $helper = Banners::getInstance();
@@ -101,7 +101,7 @@  discard block
 block discarded – undo
101 101
                     $percent = substr(100 * $clicks / $impmade, 0, 5);
102 102
                 }
103 103
                 if ($imptotal == 0) {
104
-                    $left = "" . _AM_BANNERS_BANNERS_UNLIMIT . "";
104
+                    $left = ""._AM_BANNERS_BANNERS_UNLIMIT."";
105 105
                 } else {
106 106
                     $left = $imptotal - $impmade;
107 107
                 }
@@ -110,9 +110,9 @@  discard block
 block discarded – undo
110 110
                     $img .= html_entity_decode($htmlcode);
111 111
                 } else {
112 112
                     if (stristr($imageurl, '.swf')) {
113
-                        $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>';
113
+                        $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>';
114 114
                     } else {
115
-                        $img .= '<img src="' . $imageurl . '" alt="" />';
115
+                        $img .= '<img src="'.$imageurl.'" alt="" />';
116 116
                     }
117 117
                 }
118 118
 
@@ -130,7 +130,7 @@  discard block
 block discarded – undo
130 130
         }
131 131
         // Display Page Navigation
132 132
         if ($banner_count > $nb_banners) {
133
-            $nav = new XoopsPageNav($banner_count, $nb_banners, $start, 'start', 'startF=' . $startF);
133
+            $nav = new XoopsPageNav($banner_count, $nb_banners, $start, 'start', 'startF='.$startF);
134 134
             $xoops->tpl()->assign('nav_menu_banner', $nav->renderNav(4));
135 135
         }
136 136
         // Display Finished Banners
@@ -166,9 +166,9 @@  discard block
 block discarded – undo
166 166
                 } else {
167 167
                     $img = '<div id="xo-bannerfix">';
168 168
                     if (stristr($imageurl, '.swf')) {
169
-                        $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>';
169
+                        $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>';
170 170
                     } else {
171
-                        $img .= '<img src="' . $imageurl . '" alt="" />';
171
+                        $img .= '<img src="'.$imageurl.'" alt="" />';
172 172
                     }
173 173
 
174 174
                     $img .= '</div>';
@@ -193,7 +193,7 @@  discard block
 block discarded – undo
193 193
         }
194 194
         // Display Page Navigation
195 195
         if ($banner_finish_count > $nb_banners) {
196
-            $nav = new XoopsPageNav($banner_finish_count, $nb_banners, $startF, 'startF', 'start=' . $start);
196
+            $nav = new XoopsPageNav($banner_finish_count, $nb_banners, $startF, 'startF', 'start='.$start);
197 197
             $xoops->tpl()->assign('nav_menu_bannerF', $nav->renderNav(4));
198 198
         }
199 199
         break;
@@ -237,7 +237,7 @@  discard block
 block discarded – undo
237 237
         $error_msg = '';
238 238
         $obj->setVar("banner_cid", Request::getInt('cid', 0));
239 239
         if (preg_match('/^[0-9]*[0-9]+$|^[0-9]+[0-9]*$/', $_POST["imptotal"]) == false) {
240
-            $error_msg .= XoopsLocale::E_YOU_NEED_A_POSITIVE_INTEGER . '<br />';
240
+            $error_msg .= XoopsLocale::E_YOU_NEED_A_POSITIVE_INTEGER.'<br />';
241 241
             $obj->setVar("banner_imptotal", 0);
242 242
         } else {
243 243
             $obj->setVar("banner_imptotal", Request::getInt('imptotal', 0));
@@ -246,7 +246,7 @@  discard block
 block discarded – undo
246 246
         $obj->setVar("banner_htmlbanner", Request::getInt('htmlbanner', 0));
247 247
         $obj->setVar("banner_htmlcode", Request::getString('htmlcode', ''));
248 248
 		
249
-        $uploader_banners_img = new XoopsMediaUploader($xoops_upload_path . '/banners', $mimetypes, $upload_size, null, null);
249
+        $uploader_banners_img = new XoopsMediaUploader($xoops_upload_path.'/banners', $mimetypes, $upload_size, null, null);
250 250
 		
251 251
         if ($uploader_banners_img->fetchMedia("banners_imageurl")) {
252 252
             $uploader_banners_img->setPrefix("banner");
@@ -254,13 +254,13 @@  discard block
 block discarded – undo
254 254
             if (!$uploader_banners_img->upload()) {
255 255
                 $error_msg .= $uploader_banners_img->getErrors();
256 256
             } else {
257
-                $obj->setVar("banner_imageurl", $xoops_upload_url . '/banners/' . $uploader_banners_img->getSavedFileName());
257
+                $obj->setVar("banner_imageurl", $xoops_upload_url.'/banners/'.$uploader_banners_img->getSavedFileName());
258 258
             }
259 259
         } else {
260 260
             if ($_POST["banners_imageurl"] === 'blank.gif') {
261 261
                 $obj->setVar("banner_imageurl", Request::getString('imageurl', ''));
262 262
             } else {
263
-                $obj->setVar("banner_imageurl", $xoops_upload_url . '/banners/' . Request::getString('banners_imageurl', ''));
263
+                $obj->setVar("banner_imageurl", $xoops_upload_url.'/banners/'.Request::getString('banners_imageurl', ''));
264 264
             }
265 265
         }
266 266
 
@@ -286,8 +286,8 @@  discard block
 block discarded – undo
286 286
                 if (!$xoops->security()->check()) {
287 287
                     $xoops->redirect("banners.php", 3, implode(",", $xoops->security()->getErrors()));
288 288
                 }
289
-                $namefile = substr_replace($obj->getVar('imageurl'), '', 0, strlen($xoops_url . '/uploads/banners/'));
290
-                $urlfile =  $xoops_root_path . '/uploads/banners/' . $namefile;
289
+                $namefile = substr_replace($obj->getVar('imageurl'), '', 0, strlen($xoops_url.'/uploads/banners/'));
290
+                $urlfile = $xoops_root_path.'/uploads/banners/'.$namefile;
291 291
                 if ($banner_Handler->delete($obj)) {
292 292
                     // delete banner
293 293
                     if (is_file($urlfile)) {
@@ -311,10 +311,10 @@  discard block
 block discarded – undo
311 311
                         $img .= "</embed>";
312 312
                         $img .= "</object>";
313 313
                     } else {
314
-                        $img .= "<img src='" . $imageurl . "' alt='' />";
314
+                        $img .= "<img src='".$imageurl."' alt='' />";
315 315
                     }
316 316
                 }
317
-                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 \>');
317
+                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 \>');
318 318
             }
319 319
         } else {
320 320
             $xoops->redirect('banners.php', 1, XoopsLocale::E_DATABASE_NOT_UPDATED);
Please login to merge, or discard this patch.
htdocs/modules/banners/admin/index.php 1 patch
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -19,7 +19,7 @@  discard block
 block discarded – undo
19 19
  * @author          Mage Gregory (AKA Mage)
20 20
  * @version         $Id: $
21 21
  */
22
-include __DIR__ . '/header.php';
22
+include __DIR__.'/header.php';
23 23
 $xoops = Xoops::getInstance();
24 24
 $helper = Banners::getInstance();
25 25
 
@@ -45,13 +45,13 @@  discard block
 block discarded – undo
45 45
 $criteria->add(new Criteria('banner_status', 0));
46 46
 $banners_finish = $banner_Handler->getCount($criteria);
47 47
 // folder path
48
-$folder_path = $xoops_root_path . '/uploads/banners';
48
+$folder_path = $xoops_root_path.'/uploads/banners';
49 49
 
50 50
 $admin_page = new \Xoops\Module\Admin();
51 51
 $admin_page->addInfoBox(_MI_BANNERS_BANNERS);
52 52
 $admin_page->addInfoBoxLine(sprintf(_AM_BANNERS_INDEX_NBTOTAL, $banners_banner));
53
-$admin_page->addInfoBoxLine(sprintf(_AM_BANNERS_INDEX_NBCLIENT, '<span class="green">' . $banners_client . '</span>'));
54
-$admin_page->addInfoBoxLine(sprintf(_AM_BANNERS_INDEX_NBFINISH, '<span class="red">' . $banners_finish . '</span>'));
53
+$admin_page->addInfoBoxLine(sprintf(_AM_BANNERS_INDEX_NBCLIENT, '<span class="green">'.$banners_client.'</span>'));
54
+$admin_page->addInfoBoxLine(sprintf(_AM_BANNERS_INDEX_NBFINISH, '<span class="red">'.$banners_finish.'</span>'));
55 55
 $admin_page->addConfigBoxLine($folder_path, 'folder');
56 56
 $admin_page->addConfigBoxLine(array($folder_path, '777'), 'chmod');
57 57
 $admin_page->displayNavigation('index.php');
Please login to merge, or discard this patch.