txmodxoops / tdmcreate

admin/menu.php

Spacing +13 added lines, -13 removed lines

@@ -23,7 +23,7 @@ 
  * @version         $Id: menu.php 11084 2013-02-23 15:44:20Z timgno $
  */
 
-include dirname(__DIR__) . '/preloads/autoloader.php';
+include dirname(__DIR__).'/preloads/autoloader.php';
 
 $moduleDirName      = basename(dirname(__DIR__));
 $moduleDirNameUpper = mb_strtoupper($moduleDirName);
@@ -47,62 +47,62 @@ 
 $adminmenu[] = [
     'title' => _MI_TDMCREATE_ADMENU1,
     'link'  => 'admin/index.php',
-    'icon'  => $pathIcon32 . '/dashboard.png',
+    'icon'  => $pathIcon32.'/dashboard.png',
 ];
 
 $adminmenu[] = [
     'title' => _MI_TDMCREATE_ADMENU2,
     'link'  => 'admin/settings.php',
-    'icon'  => $modPathIcon32 . '/settings.png',
+    'icon'  => $modPathIcon32.'/settings.png',
 ];
 
 $adminmenu[] = [
     'title' => _MI_TDMCREATE_ADMENU3,
     'link'  => 'admin/modules.php',
-    'icon'  => $modPathIcon32 . '/addmodule.png',
+    'icon'  => $modPathIcon32.'/addmodule.png',
 ];
 
 $adminmenu[] = [
     'title' => _MI_TDMCREATE_ADMENU4,
     'link'  => 'admin/tables.php',
-    'icon'  => $modPathIcon32 . '/addtable.png',
+    'icon'  => $modPathIcon32.'/addtable.png',
 ];
 
 $adminmenu[] = [
     'title' => _MI_TDMCREATE_ADMENU5,
     'link'  => 'admin/fields.php',
-    'icon'  => $modPathIcon32 . '/fields.png',
+    'icon'  => $modPathIcon32.'/fields.png',
 ];
 
 $adminmenu[] = [
     'title' => _MI_TDMCREATE_ADMENU6,
     'link'  => 'admin/morefiles.php',
-    'icon'  => $modPathIcon32 . '/files.png',
+    'icon'  => $modPathIcon32.'/files.png',
 ];
 
 $adminmenu[] = [
     'title' => _MI_TDMCREATE_ADMENU7,
     'link'  => 'admin/building.php',
-    'icon'  => $modPathIcon32 . '/builder.png',
+    'icon'  => $modPathIcon32.'/builder.png',
 ];
 
 //Feedback
 $adminmenu[] = [
-    'title' => constant('CO_' . $moduleDirNameUpper . '_' . 'ADMENU_FEEDBACK'),
+    'title' => constant('CO_'.$moduleDirNameUpper.'_'.'ADMENU_FEEDBACK'),
     'link'  => 'admin/feedback.php',
-    'icon'  => $pathIcon32 . 'mail_foward.png',
+    'icon'  => $pathIcon32.'mail_foward.png',
 ];
 
 if ($helper->getConfig('displayDeveloperTools')) {
     $adminmenu[] = [
-        'title' => constant('CO_' . $moduleDirNameUpper . '_' . 'ADMENU_MIGRATE'),
+        'title' => constant('CO_'.$moduleDirNameUpper.'_'.'ADMENU_MIGRATE'),
         'link' => 'admin/migrate.php',
-        'icon' => $pathIcon32 . 'database_go.png',
+        'icon' => $pathIcon32.'database_go.png',
     ];
 }
 
 $adminmenu[] = [
     'title' => _MI_TDMCREATE_ABOUT,
     'link'  => 'admin/about.php',
-    'icon'  => $pathIcon32 . 'about.png',
+    'icon'  => $pathIcon32.'about.png',
 ];

admin/footer.php

Indentation +5 added lines, -5 removed lines

@@ -25,10 +25,10 @@
 $pathIcon32 = Xmf\Module\Admin::iconUrl('', 32);
 
 echo "<div class='adminfooter'>\n"
-     . "  <div style='text-align: center;'>\n"
-     . "    <a href='https://xoops.org' rel='external'><img src='{$pathIcon32}/xoopsmicrobutton.gif' alt='XOOPS' title='XOOPS'></a>\n"
-     . "  </div>\n"
-     . '  ' . _AM_MODULEADMIN_ADMIN_FOOTER . "\n"
-     . '</div>';
+        . "  <div style='text-align: center;'>\n"
+        . "    <a href='https://xoops.org' rel='external'><img src='{$pathIcon32}/xoopsmicrobutton.gif' alt='XOOPS' title='XOOPS'></a>\n"
+        . "  </div>\n"
+        . '  ' . _AM_MODULEADMIN_ADMIN_FOOTER . "\n"
+        . '</div>';
 
 xoops_cp_footer();

Spacing +1 added lines, -1 removed lines

@@ -28,7 +28,7 @@
      . "  <div style='text-align: center;'>\n"
      . "    <a href='https://xoops.org' rel='external'><img src='{$pathIcon32}/xoopsmicrobutton.gif' alt='XOOPS' title='XOOPS'></a>\n"
      . "  </div>\n"
-     . '  ' . _AM_MODULEADMIN_ADMIN_FOOTER . "\n"
+     . '  '._AM_MODULEADMIN_ADMIN_FOOTER."\n"
      . '</div>';
 
 xoops_cp_footer();

admin/header.php

Spacing +4 added lines, -4 removed lines

@@ -22,10 +22,10 @@ 
  * @version         $Id: header.php 12258 2014-01-02 09:33:29Z timgno $
  */
 
-include dirname(__DIR__) . '/preloads/autoloader.php';
+include dirname(__DIR__).'/preloads/autoloader.php';
 
-include_once dirname(dirname(dirname(__DIR__))) . '/include/cp_header.php';
-include_once dirname(__DIR__) . '/include/common.php';
+include_once dirname(dirname(dirname(__DIR__))).'/include/cp_header.php';
+include_once dirname(__DIR__).'/include/common.php';
 
 $thisDirname = $GLOBALS['xoopsModule']->getVar('dirname');
 // Link System Icons
@@ -42,7 +42,7 @@ 
 // MyTextSanitizer
 $myts = MyTextSanitizer::getInstance();
 if (!isset($xoopsTpl) || !is_object($xoopsTpl)) {
-    include_once XOOPS_ROOT_PATH . '/class/template.php';
+    include_once XOOPS_ROOT_PATH.'/class/template.php';
     $xoopsTpl = new \XoopsTpl();
 }
 // System Icons

include/functions.php

Spacing +2 added lines, -2 removed lines

@@ -32,7 +32,7 @@ 
 {
     $donationform = [
         0 => '<form name="donation" id="donation" action="http://www.txmodxoops.org/modules/xdonations/" method="post" onsubmit="return xoopsFormValidate_donation();">',
-        1 => '<table class="outer" cellspacing="1" width="100%"><tbody><tr><th colspan="2">' . _AM_TDMCREATE_ABOUT_MAKE_DONATION . '</th></tr><tr align="left" valign="top"><td class="head"><div class="xoops-form-element-caption-required"><span class="caption-text">' . _AM_TDMCREATE_DONATION_AMOUNT . '</span><span class="caption-marker">*</span></div></td><td class="even"><select size="1" name="item[A][amount]" id="item[A][amount]" title="Donation Amount"><option value="5">5.00 EUR</option><option value="10">10.00 EUR</option><option value="20">20.00 EUR</option><option value="40">40.00 EUR</option><option value="60">60.00 EUR</option><option value="80">80.00 EUR</option><option value="90">90.00 EUR</option><option value="100">100.00 EUR</option><option value="200">200.00 EUR</option></select></td></tr><tr align="left" valign="top"><td class="head"></td><td class="even"><input class="formButton" name="submit" id="submit" value="' . _SUBMIT . '" title="' . _SUBMIT . '" type="submit"></td></tr></tbody></table>',
+        1 => '<table class="outer" cellspacing="1" width="100%"><tbody><tr><th colspan="2">'._AM_TDMCREATE_ABOUT_MAKE_DONATION.'</th></tr><tr align="left" valign="top"><td class="head"><div class="xoops-form-element-caption-required"><span class="caption-text">'._AM_TDMCREATE_DONATION_AMOUNT.'</span><span class="caption-marker">*</span></div></td><td class="even"><select size="1" name="item[A][amount]" id="item[A][amount]" title="Donation Amount"><option value="5">5.00 EUR</option><option value="10">10.00 EUR</option><option value="20">20.00 EUR</option><option value="40">40.00 EUR</option><option value="60">60.00 EUR</option><option value="80">80.00 EUR</option><option value="90">90.00 EUR</option><option value="100">100.00 EUR</option><option value="200">200.00 EUR</option></select></td></tr><tr align="left" valign="top"><td class="head"></td><td class="even"><input class="formButton" name="submit" id="submit" value="'._SUBMIT.'" title="'._SUBMIT.'" type="submit"></td></tr></tbody></table>',
         2 => '<input name="op" id="op" value="createinvoice" type="hidden"><input name="plugin" id="plugin" value="donations" type="hidden"><input name="donation" id="donation" value="1" type="hidden"><input name="drawfor" id="drawfor" value="Chronolabs Co-Operative" type="hidden"><input name="drawto" id="drawto" value="%s" type="hidden"><input name="drawto_email" id="drawto_email" value="%s" type="hidden"><input name="key" id="key" value="%s" type="hidden"><input name="currency" id="currency" value="EUR" type="hidden"><input name="weight_unit" id="weight_unit" value="kgs" type="hidden"><input name="item[A][cat]" id="item[A][cat]" value="XDN%s" type="hidden"><input name="item[A][name]" id="item[A][name]" value="Donation for %s" type="hidden"><input name="item[A][quantity]" id="item[A][quantity]" value="1" type="hidden"><input name="item[A][shipping]" id="item[A][shipping]" value="0" type="hidden"><input name="item[A][handling]" id="item[A][handling]" value="0" type="hidden"><input name="item[A][weight]" id="item[A][weight]" value="0" type="hidden"><input name="item[A][tax]" id="item[A][tax]" value="0" type="hidden"><input name="return" id="return" value="http://www.txmodxoops.org/modules/xdonations/success.php" type="hidden"><input name="cancel" id="cancel" value="http://www.txmodxoops.org/modules/xdonations/success.php" type="hidden"></form>', 'D' => '',
         3 => '',
         4 => '<!-- Start Form Validation JavaScript //-->
@@ -55,7 +55,7 @@ 
     for ($key = 0; $key <= 4; ++$key) {
         switch ($key) {
             case 2:
-                $donationform[$key] = sprintf($donationform[$key], $GLOBALS['xoopsConfig']['sitename'] . ' - ' . ('' != $GLOBALS['xoopsUser']->getVar('name') ? $GLOBALS['xoopsUser']->getVar('name') . ' [' . $GLOBALS['xoopsUser']->getVar('uname') . ']' : $GLOBALS['xoopsUser']->getVar('uname')), $GLOBALS['xoopsUser']->getVar('email'), XOOPS_LICENSE_KEY, mb_strtoupper($GLOBALS['xoopsModule']->getVar('dirname')), mb_strtoupper($GLOBALS['xoopsModule']->getVar('dirname')) . ' ' . $GLOBALS['xoopsModule']->getVar('name'));
+                $donationform[$key] = sprintf($donationform[$key], $GLOBALS['xoopsConfig']['sitename'].' - '.('' != $GLOBALS['xoopsUser']->getVar('name') ? $GLOBALS['xoopsUser']->getVar('name').' ['.$GLOBALS['xoopsUser']->getVar('uname').']' : $GLOBALS['xoopsUser']->getVar('uname')), $GLOBALS['xoopsUser']->getVar('email'), XOOPS_LICENSE_KEY, mb_strtoupper($GLOBALS['xoopsModule']->getVar('dirname')), mb_strtoupper($GLOBALS['xoopsModule']->getVar('dirname')).' '.$GLOBALS['xoopsModule']->getVar('name'));
                 break;
         }
     }

include/update.php

Spacing +6 added lines, -6 removed lines

@@ -56,7 +56,7 @@ 
 {
     global $xoopsDB;
     $result = $xoopsDB->query(
-        'SELECT t1.tpl_id FROM ' . $xoopsDB->prefix('tplfile') . ' t1, ' . $xoopsDB->prefix('tplfile')
+        'SELECT t1.tpl_id FROM '.$xoopsDB->prefix('tplfile').' t1, '.$xoopsDB->prefix('tplfile')
         . ' t2 WHERE t1.tpl_refid = t2.tpl_refid AND t1.tpl_module = t2.tpl_module AND t1.tpl_tplset=t2.tpl_tplset AND t1.tpl_file = t2.tpl_file AND t1.tpl_type = t2.tpl_type AND t1.tpl_id > t2.tpl_id'
     );
     $tplids = [];
@@ -66,7 +66,7 @@ 
     if (count($tplids) > 0) {
         $tplfileHandler = xoops_getHandler('tplfile');
         $duplicate_files = $tplfileHandler->getObjects(
-            new \Criteria('tpl_id', '(' . implode(',', $tplids) . ')', 'IN')
+            new \Criteria('tpl_id', '('.implode(',', $tplids).')', 'IN')
         );
 
         if (count($duplicate_files) > 0) {
@@ -75,9 +75,9 @@ 
             }
         }
     }
-    $sql = 'SHOW INDEX FROM ' . $xoopsDB->prefix('tplfile') . " WHERE KEY_NAME = 'tpl_refid_module_set_file_type'";
+    $sql = 'SHOW INDEX FROM '.$xoopsDB->prefix('tplfile')." WHERE KEY_NAME = 'tpl_refid_module_set_file_type'";
     if (!$result = $xoopsDB->queryF($sql)) {
-        xoops_error($this->db->error() . '<br />' . $sql);
+        xoops_error($this->db->error().'<br />'.$sql);
 
         return false;
     }
@@ -92,10 +92,10 @@ 
 
         return true;
     }
-    $sql = 'ALTER TABLE ' . $xoopsDB->prefix('tplfile')
+    $sql = 'ALTER TABLE '.$xoopsDB->prefix('tplfile')
         . ' ADD UNIQUE tpl_refid_module_set_file_type ( tpl_refid, tpl_module, tpl_tplset, tpl_file, tpl_type )';
     if (!$result = $xoopsDB->queryF($sql)) {
-        xoops_error($xoopsDB->error() . '<br />' . $sql);
+        xoops_error($xoopsDB->error().'<br />'.$sql);
         $module->setErrors(
             "'tpl_refid_module_set_file_type' unique index is not added to 'tplfile' table. Warning: do not use XOOPS until you add this unique index."
         );

config/icons.php

Spacing +11 added lines, -11 removed lines

@@ -3,20 +3,20 @@
 function getIcons()
 {
     $pathIcon16 = \Xmf\Module\Admin::iconUrl('', 16);
-    $moduleDirName      = basename(dirname(__DIR__));
+    $moduleDirName = basename(dirname(__DIR__));
 
     return (object)[
-        'name'          => mb_strtoupper($moduleDirName) . ' IconConfigurator',
+        'name'          => mb_strtoupper($moduleDirName).' IconConfigurator',
         'icons'         => [
-            'edit' => "<img src='" . $pathIcon16 . "/edit.png'  alt=" . _EDIT . "' align='middle'>",
-            'delete' => "<img src='" . $pathIcon16 . "/delete.png' alt='" . _DELETE . "' align='middle'>",
-            'clone' => "<img src='" . $pathIcon16 . "/editcopy.png' alt='" . _CLONE . "' align='middle'>",
-            'preview' => "<img src='" . $pathIcon16 . "/view.png' alt='" . _PREVIEW . "' align='middle'>",
-            'print' => "<img src='" . $pathIcon16 . "/printer.png' alt='" . _CLONE . "' align='middle'>",
-            'pdf' => "<img src='" . $pathIcon16 . "/pdf.png' alt='" . _CLONE . "' align='middle'>",
-            'add' => "<img src='" . $pathIcon16 . "/add.png' alt='" . _ADD . "' align='middle'>",
-            '0' => "<img src='" . $pathIcon16 . "/0.png' alt='" . 0 . "' align='middle'>",
-            '1' => "<img src='" . $pathIcon16 . "/1.png' alt='" . 1 . "' align='middle'>",
+            'edit' => "<img src='".$pathIcon16."/edit.png'  alt="._EDIT."' align='middle'>",
+            'delete' => "<img src='".$pathIcon16."/delete.png' alt='"._DELETE."' align='middle'>",
+            'clone' => "<img src='".$pathIcon16."/editcopy.png' alt='"._CLONE."' align='middle'>",
+            'preview' => "<img src='".$pathIcon16."/view.png' alt='"._PREVIEW."' align='middle'>",
+            'print' => "<img src='".$pathIcon16."/printer.png' alt='"._CLONE."' align='middle'>",
+            'pdf' => "<img src='".$pathIcon16."/pdf.png' alt='"._CLONE."' align='middle'>",
+            'add' => "<img src='".$pathIcon16."/add.png' alt='"._ADD."' align='middle'>",
+            '0' => "<img src='".$pathIcon16."/0.png' alt='".0."' align='middle'>",
+            '1' => "<img src='".$pathIcon16."/1.png' alt='".1."' align='middle'>",
         ],
     ];
 }

config/config.php

Spacing +13 added lines, -13 removed lines

@@ -21,30 +21,30 @@ 
 $moduleDirNameUpper = mb_strtoupper($moduleDirName);
 
 return (object)[
-    'name'           => mb_strtoupper($moduleDirName) . ' ModuleConfigurator',
+    'name'           => mb_strtoupper($moduleDirName).' ModuleConfigurator',
     'paths'          => [
         'dirname'    => $moduleDirName,
-        'admin'      => XOOPS_ROOT_PATH . '/modules/' . $moduleDirName . '/admin',
-        'modPath'    => XOOPS_ROOT_PATH . '/modules/' . $moduleDirName,
-        'modUrl'     => XOOPS_URL . '/modules/' . $moduleDirName,
-        'uploadPath' => XOOPS_UPLOAD_PATH . '/' . $moduleDirName,
-        'uploadUrl'  => XOOPS_UPLOAD_URL . '/' . $moduleDirName,
+        'admin'      => XOOPS_ROOT_PATH.'/modules/'.$moduleDirName.'/admin',
+        'modPath'    => XOOPS_ROOT_PATH.'/modules/'.$moduleDirName,
+        'modUrl'     => XOOPS_URL.'/modules/'.$moduleDirName,
+        'uploadPath' => XOOPS_UPLOAD_PATH.'/'.$moduleDirName,
+        'uploadUrl'  => XOOPS_UPLOAD_URL.'/'.$moduleDirName,
     ],
     'uploadFolders'  => [
-        XOOPS_UPLOAD_PATH . '/' . $moduleDirName,
-        XOOPS_UPLOAD_PATH . '/' . $moduleDirName . '/screenshots',
+        XOOPS_UPLOAD_PATH.'/'.$moduleDirName,
+        XOOPS_UPLOAD_PATH.'/'.$moduleDirName.'/screenshots',
         //XOOPS_UPLOAD_PATH . '/flags'
     ],
     'copyBlankFiles' => [
-        XOOPS_UPLOAD_PATH . '/' . $moduleDirName,
-        XOOPS_UPLOAD_PATH . '/' . $moduleDirName . '/screenshots',
+        XOOPS_UPLOAD_PATH.'/'.$moduleDirName,
+        XOOPS_UPLOAD_PATH.'/'.$moduleDirName.'/screenshots',
         //XOOPS_UPLOAD_PATH . '/flags'
     ],
 
     'copyTestFolders' => [
         [
-            XOOPS_ROOT_PATH . '/modules/' . $moduleDirName . '/testdata/uploads',
-            XOOPS_UPLOAD_PATH . '/' . $moduleDirName,
+            XOOPS_ROOT_PATH.'/modules/'.$moduleDirName.'/testdata/uploads',
+            XOOPS_UPLOAD_PATH.'/'.$moduleDirName,
         ],
         //            [
         //                XOOPS_ROOT_PATH . '/modules/' . $moduleDirName . '/testdata/thumbs',
@@ -81,6 +81,6 @@ 
         //            'totalsubmitted'  => $helper->getHandler('Item')->getItemsCount(-1, [Constants::PUBLISHER_STATUS_SUBMITTED]),
     ],
     'modCopyright' => "<a href='https://xoops.org' title='XOOPS Project' target='_blank'>
-                     <img src='" . constant($moduleDirNameUpper . '_AUTHOR_LOGOIMG') . "' alt='XOOPS Project'></a>",
+                     <img src='" . constant($moduleDirNameUpper.'_AUTHOR_LOGOIMG')."' alt='XOOPS Project'></a>",
 ];
 

config/paths.php

Spacing +9 added lines, -9 removed lines

@@ -6,19 +6,19 @@
     $moduleDirNameUpper = mb_strtoupper($moduleDirName);
 
     return (object)[
-        'name'          => mb_strtoupper($moduleDirName) . ' PathConfigurator',
+        'name'          => mb_strtoupper($moduleDirName).' PathConfigurator',
         'paths'         => [
             'dirname'    => $moduleDirName,
-            'admin'      => XOOPS_ROOT_PATH . '/modules/' . $moduleDirName . '/admin',
-            'modPath'    => XOOPS_ROOT_PATH . '/modules/' . $moduleDirName,
-            'modUrl'     => XOOPS_URL . '/modules/' . $moduleDirName,
-            'uploadPath' => XOOPS_UPLOAD_PATH . '/' . $moduleDirName,
-            'uploadUrl'  => XOOPS_UPLOAD_URL . '/' . $moduleDirName,
+            'admin'      => XOOPS_ROOT_PATH.'/modules/'.$moduleDirName.'/admin',
+            'modPath'    => XOOPS_ROOT_PATH.'/modules/'.$moduleDirName,
+            'modUrl'     => XOOPS_URL.'/modules/'.$moduleDirName,
+            'uploadPath' => XOOPS_UPLOAD_PATH.'/'.$moduleDirName,
+            'uploadUrl'  => XOOPS_UPLOAD_URL.'/'.$moduleDirName,
         ],
         'uploadFolders' => [
-            XOOPS_UPLOAD_PATH . '/' . $moduleDirName,
-            XOOPS_UPLOAD_PATH . '/' . $moduleDirName . '/category',
-            XOOPS_UPLOAD_PATH . '/' . $moduleDirName . '/screenshots',
+            XOOPS_UPLOAD_PATH.'/'.$moduleDirName,
+            XOOPS_UPLOAD_PATH.'/'.$moduleDirName.'/category',
+            XOOPS_UPLOAD_PATH.'/'.$moduleDirName.'/screenshots',
             //XOOPS_UPLOAD_PATH . '/flags'
         ],
     ];

class/Settings.php

Spacing +8 added lines, -8 removed lines

@@ -145,7 +145,7 @@ 
         $isNew = $this->isNew();
         $title = $isNew ? sprintf(_AM_TDMCREATE_SETTING_NEW) : sprintf(_AM_TDMCREATE_SETTING_EDIT);
 
-        include_once XOOPS_ROOT_PATH . '/class/xoopsformloader.php';
+        include_once XOOPS_ROOT_PATH.'/class/xoopsformloader.php';
 
         $form = new \XoopsThemeForm($title, 'settingform', $action, 'post', true);
         $form->setExtra('enctype="multipart/form-data"');
@@ -174,7 +174,7 @@ 
         $checkbox = new \XoopsFormCheckbox(' ', 'setting_option', $settingOption, '<br>');
         $checkbox->setDescription(_AM_TDMCREATE_OPTIONS_DESC);
         foreach ($this->options as $option) {
-            $checkbox->addOption($option, self::getDefinedLanguage('_AM_TDMCREATE_SETTING_' . mb_strtoupper($option)));
+            $checkbox->addOption($option, self::getDefinedLanguage('_AM_TDMCREATE_SETTING_'.mb_strtoupper($option)));
         }
         $optionsTray->addElement($checkbox);
 
@@ -183,17 +183,17 @@ 
         $modImage = $this->getVar('set_image');
         $modImage = $modImage ?: $set['image'];
 
-        $uploadDirectory = 'uploads/' . $GLOBALS['xoopsModule']->dirname() . '/images/modules';
+        $uploadDirectory = 'uploads/'.$GLOBALS['xoopsModule']->dirname().'/images/modules';
         $imgtray = new \XoopsFormElementTray(_AM_TDMCREATE_SETTING_IMAGE, '<br>');
-        $imgpath = sprintf(_AM_TDMCREATE_FORMIMAGE_PATH, './' . mb_strtolower($uploadDirectory) . '/');
+        $imgpath = sprintf(_AM_TDMCREATE_FORMIMAGE_PATH, './'.mb_strtolower($uploadDirectory).'/');
         $imageSelect = new \XoopsFormSelect($imgpath, 'set_image', $modImage);
         $modImage_array = \XoopsLists::getImgListAsArray(TDMC_UPLOAD_IMGMOD_PATH);
         foreach ($modImage_array as $image) {
             $imageSelect->addOption($image, $image);
         }
-        $imageSelect->setExtra("onchange='showImgSelected(\"image3\", \"set_image\", \"" . $uploadDirectory . '", "", "' . XOOPS_URL . "\")'");
+        $imageSelect->setExtra("onchange='showImgSelected(\"image3\", \"set_image\", \"".$uploadDirectory.'", "", "'.XOOPS_URL."\")'");
         $imgtray->addElement($imageSelect);
-        $imgtray->addElement(new \XoopsFormLabel('', "<br><img src='" . TDMC_UPLOAD_IMGMOD_URL . '/' . $modImage . "' name='image3' id='image3' alt='' /><br>"));
+        $imgtray->addElement(new \XoopsFormLabel('', "<br><img src='".TDMC_UPLOAD_IMGMOD_URL.'/'.$modImage."' name='image3' id='image3' alt='' /><br>"));
 
         $fileseltray = new \XoopsFormElementTray('', '<br>');
         $fileseltray->addElement(new \XoopsFormFile(_AM_TDMCREATE_FORMUPLOAD, 'attachedfile', $helper->getConfig('maxsize')));
@@ -221,7 +221,7 @@ 
         $form->addElement(new \XoopsFormText(_AM_TDMCREATE_SETTING_PAYPAL_BUTTON, 'set_donations', 50, 255, $this->getVar('set_donations')));
         $form->addElement(new \XoopsFormText(_AM_TDMCREATE_SETTING_SUBVERSION, 'set_subversion', 50, 255, $this->getVar('set_subversion')));
 
-        $buttonTray = new \XoopsFormElementTray(_REQUIRED . ' <sup class="red bold">*</sup>', '');
+        $buttonTray = new \XoopsFormElementTray(_REQUIRED.' <sup class="red bold">*</sup>', '');
         $buttonTray->addElement(new \XoopsFormHidden('op', 'save'));
         $buttonTray->addElement(new \XoopsFormButton('', 'submit', _SUBMIT, 'submit'));
         $form->addElement($buttonTray);
@@ -261,7 +261,7 @@ 
     {
         $retSet = [];
         foreach ($this->options as $option) {
-            if (1 == $this->getVar('set_' . $option)) {
+            if (1 == $this->getVar('set_'.$option)) {
                 $retSet[] = $option;
             }
         }