mambax7 / pedigree

include/class_eq_pie.php

Spacing +6 added lines, -6 removed lines

@@ -114,12 +114,12 @@ 
 
             $piePart = $value[1];
             if (isset($this->total) && $this->total > 0) {
-                $piePart100 = round($piePart / $this->total * 100, 2);  // value in percentage, the rounding and * 100 for extra accuracy for pie w/o gaps
+                $piePart100 = round($piePart / $this->total * 100, 2); // value in percentage, the rounding and * 100 for extra accuracy for pie w/o gaps
             } else {
                 $piePart100 = 0;
             }
 
-            $piePart360 = $piePart100 * 3.6;                    // in degrees
+            $piePart360 = $piePart100 * 3.6; // in degrees
 
             $colR      = hexdec(substr($value[2], 1, 2));
             $colG      = hexdec(substr($value[2], 3, 2));
@@ -146,11 +146,11 @@ 
         foreach ($EQpieData as $i => $value) {
             $piePart = $value[1];
             if (isset($this->total) && $this->total > 0) {
-                $piePart100 = round($piePart / $this->total * 100, 2);  // value in percentage, the rounding and * 100 for extra accuracy for pie w/o gaps
+                $piePart100 = round($piePart / $this->total * 100, 2); // value in percentage, the rounding and * 100 for extra accuracy for pie w/o gaps
             } else {
                 $piePart100 = 0;
             }
-            $piePart360 = $piePart100 * 3.6;                    // in degrees
+            $piePart360 = $piePart100 * 3.6; // in degrees
 
             $colR      = hexdec(substr($value[2], 1, 2));
             $colG      = hexdec(substr($value[2], 3, 2));
@@ -193,7 +193,7 @@ 
             foreach ($EQpieData as $i => $value) {
                 $piePart = $value[1];
                 if (isset($this->total) && $this->total > 0) {
-                    $piePart100 = round($piePart / $this->total * 100, 2);  // value in percentage, the rounding and * 100 for extra accuracy for pie w/o gaps
+                    $piePart100 = round($piePart / $this->total * 100, 2); // value in percentage, the rounding and * 100 for extra accuracy for pie w/o gaps
                 } else {
                     $piePart100 = 0;
                 }
@@ -205,7 +205,7 @@ 
                 imagefilledrectangle($legendImage, 5, $yOffset + 2, 5 + $box_width, $yOffset + $box_height + 2, $PartColor);
                 imagerectangle($legendImage, 5, $yOffset + 2, 5 + $box_width, $yOffset + $box_height + 2, $borderColor);
 
-                $text = $value[0] . ' ' . $piePart100 . '%';
+                $text = $value[0].' '.$piePart100.'%';
                 imagestring($legendImage, 2, '20', $yOffset, $text, $textColor);
                 $yOffset += 15;
             }

include/color.php

Spacing +2 added lines, -2 removed lines

@@ -216,8 +216,8 @@
      */
     public function changeLightness($degree = 10)
     {
-        $color1 =& $this->color1;
-        $color2 =& $this->color2;
+        $color1 = & $this->color1;
+        $color2 = & $this->color2;
 
         for ($x = 0; $x < 3; ++$x) {
             if (($color1[$x] + $degree) < 256) {

tools.php

Switch Indentation +101 added lines, -101 removed lines

@@ -68,107 +68,107 @@
 }
 
 switch ($op) {
-    case 'lang':
-        lang();
-        break;
-    case 'langsave':
-        langsave();
-        break;
-    case 'colours':
-        colours();
-        break;
-    case 'savecolours':
-        savecolours();
-        break;
-    case 'settings':
-        settings();
-        break;
-    case 'settingssave':
-        settingssave();
-        break;
-    case 'pro':
-        pro();
-        break;
-    case 'userfields':
-        userfields($field);
-        $uf = true;
-        break;
-    case 'listuserfields':
-        listuserfields();
-        $uf = true;
-        break;
-    case 'togglelocked':
-        togglelocked($field);
-        break;
-    case 'fieldmove':
-        fieldmove($field, $_GET['move']);
-        break;
-    case 'deluserfield':
-        deluserfield($_GET['id']);
-        break;
-    case 'restoreuserfield':
-        restoreuserfield($_GET['id']);
-        break;
-    case 'editlookup':
-        editlookup($_GET['id']);
-        break;
-    case 'lookupmove':
-        lookupmove($field, $_GET['id'], $_GET['move']);
-        break;
-    case 'dellookupvalue':
-        dellookupvalue($field, $_GET['id']);
-        break;
-    case 'addlookupvalue':
-        addlookupvalue($field);
-        break;
-    case 'editlookupvalue':
-        editlookupvalue($field, $_GET['id']);
-        break;
-    case 'savelookupvalue':
-        savelookupvalue($field, $_GET['id']);
-        break;
-    case 'deleted':
-        deleted();
-        break;
-    case 'delperm':
-        delperm($_GET['id']);
-        break;
-    case 'delall':
-        delall();
-        break;
-    case 'restore':
-        restore($_GET['id']);
-        break;
-    case 'database':
-        database();
-        $db = true;
-        break;
-    case 'userq':
-        userq();
-        $db = true;
-        break;
-    case 'userqrun':
-        userqrun($_GET['f']);
-        $db = true;
-        break;
-    case 'dbanc':
-        database_oa();
-        $db = true;
-        break;
-    case 'fltypar':
-        database_fp();
-        $db = true;
-        break;
-    case 'credits':
-        credits();
-        break;
-    case 'index':
-        index();
-        break;
-    default :
-        userfields();
-        $uf = true;
-        break;
+        case 'lang':
+            lang();
+            break;
+        case 'langsave':
+            langsave();
+            break;
+        case 'colours':
+            colours();
+            break;
+        case 'savecolours':
+            savecolours();
+            break;
+        case 'settings':
+            settings();
+            break;
+        case 'settingssave':
+            settingssave();
+            break;
+        case 'pro':
+            pro();
+            break;
+        case 'userfields':
+            userfields($field);
+            $uf = true;
+            break;
+        case 'listuserfields':
+            listuserfields();
+            $uf = true;
+            break;
+        case 'togglelocked':
+            togglelocked($field);
+            break;
+        case 'fieldmove':
+            fieldmove($field, $_GET['move']);
+            break;
+        case 'deluserfield':
+            deluserfield($_GET['id']);
+            break;
+        case 'restoreuserfield':
+            restoreuserfield($_GET['id']);
+            break;
+        case 'editlookup':
+            editlookup($_GET['id']);
+            break;
+        case 'lookupmove':
+            lookupmove($field, $_GET['id'], $_GET['move']);
+            break;
+        case 'dellookupvalue':
+            dellookupvalue($field, $_GET['id']);
+            break;
+        case 'addlookupvalue':
+            addlookupvalue($field);
+            break;
+        case 'editlookupvalue':
+            editlookupvalue($field, $_GET['id']);
+            break;
+        case 'savelookupvalue':
+            savelookupvalue($field, $_GET['id']);
+            break;
+        case 'deleted':
+            deleted();
+            break;
+        case 'delperm':
+            delperm($_GET['id']);
+            break;
+        case 'delall':
+            delall();
+            break;
+        case 'restore':
+            restore($_GET['id']);
+            break;
+        case 'database':
+            database();
+            $db = true;
+            break;
+        case 'userq':
+            userq();
+            $db = true;
+            break;
+        case 'userqrun':
+            userqrun($_GET['f']);
+            $db = true;
+            break;
+        case 'dbanc':
+            database_oa();
+            $db = true;
+            break;
+        case 'fltypar':
+            database_fp();
+            $db = true;
+            break;
+        case 'credits':
+            credits();
+            break;
+        case 'index':
+            index();
+            break;
+        default :
+            userfields();
+            $uf = true;
+            break;
 }
 
 //create tools array

Indentation +26 added lines, -26 removed lines

@@ -395,9 +395,9 @@ 
             $form .= "<td><a href='tools.php?op=fieldmove&field={$row['id']}&move=up'>" . "<img src='assets/images/up.gif' alt='move field up'></a></td>\n" . "<td>&nbsp;</td>\n";
         } else { //other rows
             $form .= "<td><a href='tools.php?op=fieldmove&field={$row['id']}&move=up'>"
-                     . "<img src='assets/images/up.gif' alt='move field up'></a></td>\n"
-                     . "<td><a href='tools.php?op=fieldmove&field={$row['id']}&move=down'>\n"
-                     . "<img src='assets/images/down.gif' alt='move field down'></a></td>\n";
+                        . "<img src='assets/images/up.gif' alt='move field up'></a></td>\n"
+                        . "<td><a href='tools.php?op=fieldmove&field={$row['id']}&move=down'>\n"
+                        . "<img src='assets/images/down.gif' alt='move field down'></a></td>\n";
         }
         $form .= "<td><a href='tools.php?op=deluserfield&id={$row['id']}'>" . "<img src='images/delete.png' alt='delete field'></a>&nbsp;" . "<a href='tools.php?op=userfields&field={$row['id']}'>{$row['fieldName']}</a></td>\n";
         //can the filed be shown in a list
@@ -567,33 +567,33 @@ 
         $form .= "  <tr>\n";
         if (0 == $count) { //first row
             $form .= "    <td style='width: 15px;'>&nbsp;</td>\n"
-                     . "    <td style='width: 15px;'>\n"
-                     . "      <a href='tools.php?op=lookupmove&field={$field}&id={$row['id']}&move=down'><img src='assets/images/down.gif'></a>\n"
-                     . "    </td>\n"
-                     . "    <td>\n"
-                     . "      <a href='tools.php?op=dellookupvalue&field={$field}&id={$row['id']}'><img src='images/delete.png'></a>\n"
-                     . "      &nbsp;<a href='tools.php?op=editlookupvalue&field={$field}&id={$row['id']}'>{$row['value']}</a>\n"
-                     . "    </td>\n";
+                        . "    <td style='width: 15px;'>\n"
+                        . "      <a href='tools.php?op=lookupmove&field={$field}&id={$row['id']}&move=down'><img src='assets/images/down.gif'></a>\n"
+                        . "    </td>\n"
+                        . "    <td>\n"
+                        . "      <a href='tools.php?op=dellookupvalue&field={$field}&id={$row['id']}'><img src='images/delete.png'></a>\n"
+                        . "      &nbsp;<a href='tools.php?op=editlookupvalue&field={$field}&id={$row['id']}'>{$row['value']}</a>\n"
+                        . "    </td>\n";
         } elseif ($count == $numrows - 1) { //last row
             $form .= "    <td>\n"
-                     . "      <a href='tools.php?op=lookupmove&field={$field}&id={$row['id']}&move=up'><img src='assets/images/up.gif'></a>\n"
-                     . "    </td>\n"
-                     . "    <td>&nbsp;</td>\n"
-                     . "    <td>\n"
-                     . "      <a href='tools.php?op=dellookupvalue&field={$field}&id={$row['id']}'><img src='assets/images/delete.png'></a>\n"
-                     . "      &nbsp;<a href='tools.php?op=editlookupvalue&field={$field}&id={$row['id']}'>{$row['value']}</a>\n"
-                     . "    </td>\n";
+                        . "      <a href='tools.php?op=lookupmove&field={$field}&id={$row['id']}&move=up'><img src='assets/images/up.gif'></a>\n"
+                        . "    </td>\n"
+                        . "    <td>&nbsp;</td>\n"
+                        . "    <td>\n"
+                        . "      <a href='tools.php?op=dellookupvalue&field={$field}&id={$row['id']}'><img src='assets/images/delete.png'></a>\n"
+                        . "      &nbsp;<a href='tools.php?op=editlookupvalue&field={$field}&id={$row['id']}'>{$row['value']}</a>\n"
+                        . "    </td>\n";
         } else { //other rows
             $form .= "    <td>\n"
-                     . "      <a href='tools.php?op=lookupmove&field={$field}&id={$row['id']}&move=up'><img src='assets/images/up.gif'></a>\n"
-                     . "    </td>\n"
-                     . "    <td>\n"
-                     . "      <a href='tools.php?op=lookupmove&field={$field}&id={$row['id']}&move=down'><img src='assets/images/down.gif'></a>\n"
-                     . "    </td>\n"
-                     . "    <td>\n"
-                     . "      <a href='tools.php?op=dellookupvalue&field={$field}&id={$row['id']}'><img src='images/delete.png'></a>\n"
-                     . "      &nbsp;<a href='tools.php?op=editlookupvalue&field={$field}&id={$row['id']}'>{$row['value']}</a>\n"
-                     . "    </td>\n";
+                        . "      <a href='tools.php?op=lookupmove&field={$field}&id={$row['id']}&move=up'><img src='assets/images/up.gif'></a>\n"
+                        . "    </td>\n"
+                        . "    <td>\n"
+                        . "      <a href='tools.php?op=lookupmove&field={$field}&id={$row['id']}&move=down'><img src='assets/images/down.gif'></a>\n"
+                        . "    </td>\n"
+                        . "    <td>\n"
+                        . "      <a href='tools.php?op=dellookupvalue&field={$field}&id={$row['id']}'><img src='images/delete.png'></a>\n"
+                        . "      &nbsp;<a href='tools.php?op=editlookupvalue&field={$field}&id={$row['id']}'>{$row['value']}</a>\n"
+                        . "    </td>\n";
         }
         $form .= "</tr>\n";
         ++$count;

Spacing +134 added lines, -134 removed lines

@@ -24,7 +24,7 @@ 
 
 
 //require_once  dirname(dirname(__DIR__)) . '/mainfile.php';
-require_once __DIR__ . '/header.php';
+require_once __DIR__.'/header.php';
 $moduleDirName = basename(__DIR__);
 xoops_loadLanguage('main', $moduleDirName);
 $helper = Pedigree\Helper::getInstance();
@@ -34,23 +34,23 @@ 
 include $GLOBALS['xoops']->path('/header.php');
 
 // Include any common code for this module.
-require_once XOOPS_ROOT_PATH . '/modules/' . $moduleDirName . '/include/common.php';
+require_once XOOPS_ROOT_PATH.'/modules/'.$moduleDirName.'/include/common.php';
 
 //check for access
 $xoopsModule = XoopsModule::getByDirname($moduleDirName);
 if (empty($GLOBALS['xoopsUser']) || !($GLOBALS['xoopsUser'] instanceof \XoopsUser)) {
-    redirect_header('index.php', 3, _NOPERM . '<br>' . _MA_PEDIGREE_REGIST);
+    redirect_header('index.php', 3, _NOPERM.'<br>'._MA_PEDIGREE_REGIST);
 }
 
-$xoTheme->addScript(XOOPS_URL . '/browse.php?Frameworks/jquery/jquery.js');
+$xoTheme->addScript(XOOPS_URL.'/browse.php?Frameworks/jquery/jquery.js');
 //$xoTheme->addScript(PEDIGREE_URL . '/assets/js/jquery.ThickBox/thickbox-compressed.js');
 
-$xoTheme->addScript(PEDIGREE_URL . '/assets/js/jquery.magnific-popup.min.js');
-$xoTheme->addScript(PEDIGREE_URL . '/assets/js/colpick.js');
+$xoTheme->addScript(PEDIGREE_URL.'/assets/js/jquery.magnific-popup.min.js');
+$xoTheme->addScript(PEDIGREE_URL.'/assets/js/colpick.js');
 
-$xoTheme->addStylesheet(PEDIGREE_URL . '/assets/css/colpick.css');
-$xoTheme->addStylesheet(PEDIGREE_URL . '/assets/css/magnific-popup.css');
-$xoTheme->addStylesheet(PEDIGREE_URL . '/assets/css/style.css');
+$xoTheme->addStylesheet(PEDIGREE_URL.'/assets/css/colpick.css');
+$xoTheme->addStylesheet(PEDIGREE_URL.'/assets/css/magnific-popup.css');
+$xoTheme->addStylesheet(PEDIGREE_URL.'/assets/css/style.css');
 
 global $field;
 //add JS routines
@@ -201,7 +201,7 @@ 
 $GLOBALS['xoopsTpl']->assign('tools', $tools);
 
 //footer
-include XOOPS_ROOT_PATH . '/footer.php';
+include XOOPS_ROOT_PATH.'/footer.php';
 
 function index()
 {
@@ -260,26 +260,26 @@ 
     </script>
 
     <?php
-    $form = _MA_PEDIGREE_BGCOLOR . '<br><br>';
+    $form = _MA_PEDIGREE_BGCOLOR.'<br><br>';
     $form .= '<FORM NAME="myForm" action=\'tools.php?op=savecolours\' method=\'POST\'>
 <table>
 
 <!--
-    <tr><td>' . _MA_PEDIGREE_TXT_COLOR . '</td><td><INPUT TYPE="text" id="text" name="text" value="' . $text . '" size="11" maxlength="7">
+    <tr><td>' . _MA_PEDIGREE_TXT_COLOR.'</td><td><INPUT TYPE="text" id="text" name="text" value="'.$text.'" size="11" maxlength="7">
     <a href="TCP.popup(document.forms[\'myForm\'].elements[\'text\'])">
     <img width="15" height="13" border="0" alt="Click Here to Pick up the color" src="assets/images/sel.gif"></a></td></tr>
 
 
-    <tr><td>' . _MA_PEDIGREE_LINK_COLOR . '</td><td><INPUT TYPE="text" id="actlink" name="actlink" value="' . $actlink . '" size="11" maxlength="7">
+    <tr><td>' . _MA_PEDIGREE_LINK_COLOR.'</td><td><INPUT TYPE="text" id="actlink" name="actlink" value="'.$actlink.'" size="11" maxlength="7">
     <a href="TCP.popup(document.forms[\'myForm\'].elements[\'actlink\'])">
     <img width="15" height="13" border="0" alt="Click Here to Pick up the color" src="assets/images/sel.gif"></a></td></tr>
 
 
-    <tr><td>' . _MA_PEDIGREE_BACK1_COLOR . '</td><td><INPUT TYPE="text" id="even" name="even" value="' . $even . '" size="11" maxlength="7">
+    <tr><td>' . _MA_PEDIGREE_BACK1_COLOR.'</td><td><INPUT TYPE="text" id="even" name="even" value="'.$even.'" size="11" maxlength="7">
     <a href="TCP.popup(document.forms[\'myForm\'].elements[\'even\'])">
     <img width="15" height="13" border="0" alt="Click Here to Pick up the color" src="assets/images/sel.gif"></a></td></tr>
 
-    <tr><td>' . _MA_PEDIGREE_BACK2_COLOR . '</td><td><INPUT TYPE="text" id="body" name="body" value="' . $body . '" size="11" maxlength="7">
+    <tr><td>' . _MA_PEDIGREE_BACK2_COLOR.'</td><td><INPUT TYPE="text" id="body" name="body" value="'.$body.'" size="11" maxlength="7">
     <a href="TCP.popup(document.forms[\'myForm\'].elements[\'body\'])">
     <img width="15" height="13" border="0" alt="Click Here to Pick up the color" src="assets/images/sel.gif"></a></td></tr>
 
@@ -287,36 +287,36 @@ 
 
 <!--
 
-    <tr><td>' . _MA_PEDIGREE_TXT_COLOR . '</td><td>#<input type="text" id="picker" name="text" value="' . $text . '" size="11" maxlength="7">
+    <tr><td>' . _MA_PEDIGREE_TXT_COLOR.'</td><td>#<input type="text" id="picker" name="text" value="'.$text.'" size="11" maxlength="7">
     </td></tr>
 
-    <tr><td>' . _MA_PEDIGREE_LINK_COLOR . '</td><td>#<input type="text" id="picker" name="actlink" value="' . $actlink . '" size="11" maxlength="7">
+    <tr><td>' . _MA_PEDIGREE_LINK_COLOR.'</td><td>#<input type="text" id="picker" name="actlink" value="'.$actlink.'" size="11" maxlength="7">
     </td></tr>
 
-    <tr><td>' . _MA_PEDIGREE_BACK1_COLOR . '</td><td>#<input type="text" id="picker" name="even" value="' . $even . '" size="11" maxlength="7">
+    <tr><td>' . _MA_PEDIGREE_BACK1_COLOR.'</td><td>#<input type="text" id="picker" name="even" value="'.$even.'" size="11" maxlength="7">
     </td></tr>
 
-        <tr><td>' . _MA_PEDIGREE_BACK2_COLOR . '</td><td>#<input type="text" id="picker" name="body" value="' . $body . '" size="11" maxlength="7">
+        <tr><td>' . _MA_PEDIGREE_BACK2_COLOR.'</td><td>#<input type="text" id="picker" name="body" value="'.$body.'" size="11" maxlength="7">
     </td></tr>
 
 -->
 
-<tr><td>' . _MA_PEDIGREE_TXT_COLOR . '</td><td><input class="color {hash:true,caps:false}" onMouseOver="changeTextColor(\'back4\', this.value)" type="text" name="text" maxlength="7" size="7" id="colorpickerField1" value="' . $text . '">
+<tr><td>' . _MA_PEDIGREE_TXT_COLOR.'</td><td><input class="color {hash:true,caps:false}" onMouseOver="changeTextColor(\'back4\', this.value)" type="text" name="text" maxlength="7" size="7" id="colorpickerField1" value="'.$text.'">
 </td></tr>
 
 
-<tr><td>' . _MA_PEDIGREE_LINK_COLOR . '</td><td><input class="color {hash:true,caps:false}" onMouseOver="changeTextColor(\'back4\', this.value)" type="text" name="actlink" maxlength="7" size="7" id="colorpickerField1" value="' . $actlink . '">
+<tr><td>' . _MA_PEDIGREE_LINK_COLOR.'</td><td><input class="color {hash:true,caps:false}" onMouseOver="changeTextColor(\'back4\', this.value)" type="text" name="actlink" maxlength="7" size="7" id="colorpickerField1" value="'.$actlink.'">
 </td></tr>
 
 
-<tr><td>' . _MA_PEDIGREE_BACK1_COLOR . '</td><td><input class="color {hash:true,caps:false}" onMouseOver="changeTextColor(\'back4\', this.value)" type="text" name="even" maxlength="7" size="7" id="colorpickerField1" value="' . $even . '">
+<tr><td>' . _MA_PEDIGREE_BACK1_COLOR.'</td><td><input class="color {hash:true,caps:false}" onMouseOver="changeTextColor(\'back4\', this.value)" type="text" name="even" maxlength="7" size="7" id="colorpickerField1" value="'.$even.'">
 </td></tr>
 
 
-<tr><td>' . _MA_PEDIGREE_BACK2_COLOR . '</td><td><input class="color {hash:true,caps:false}" onMouseOver="changeTextColor(\'back4\', this.value)" type="text" name="body" maxlength="7" size="7" id="colorpickerField1" value="' . $body . '">
+<tr><td>' . _MA_PEDIGREE_BACK2_COLOR.'</td><td><input class="color {hash:true,caps:false}" onMouseOver="changeTextColor(\'back4\', this.value)" type="text" name="body" maxlength="7" size="7" id="colorpickerField1" value="'.$body.'">
 </td></tr>
 
-    <tr><td><INPUT TYPE="submit" value="' . _MA_PEDIGREE_SUBMIT_BUTTON . '"></td><td>&nbsp;</td></tr>
+    <tr><td><INPUT TYPE="submit" value="' . _MA_PEDIGREE_SUBMIT_BUTTON.'"></td><td>&nbsp;</td></tr>
 
     </table>
 
@@ -338,8 +338,8 @@ 
 {
     $color = new \XoopsModules\Pedigree\ImageColor();
     //create darker link hover colour
-    $actLink = Request::getString('actlink', '', 'POST');  //active link color in hex
-    $even    = Request::getString('even', '', 'POST');     // even color in hex
+    $actLink = Request::getString('actlink', '', 'POST'); //active link color in hex
+    $even    = Request::getString('even', '', 'POST'); // even color in hex
     $text    = Request::getString('text', '', 'POST');
     $body    = Request::getString('body', '', 'POST');
 
@@ -358,7 +358,7 @@ 
     $col = "{$actLink};{$even};#{$female};{$text};#{$dark};#{$head};{$body};{$actLink}";
 
     //  $query = 'UPDATE ' . $GLOBALS['xoopsDB']->prefix('config') . " SET conf_value = '{$col}' WHERE conf_name = 'colourscheme'";
-    $query = 'UPDATE ' . $GLOBALS['xoopsDB']->prefix('config') . " SET conf_value = '" . $GLOBALS['xoopsDB']->escape($col) . "' WHERE conf_name = 'colourscheme'";
+    $query = 'UPDATE '.$GLOBALS['xoopsDB']->prefix('config')." SET conf_value = '".$GLOBALS['xoopsDB']->escape($col)."' WHERE conf_name = 'colourscheme'";
     $GLOBALS['xoopsDB']->query($query);
     redirect_header('tools.php?op=colours', 1, 'Your settings have been saved.');
 }
@@ -373,12 +373,12 @@ 
 {
     global $form;
     $form .= _MA_PEDIGREE_FIELD_EXPLAIN4;
-    $sql     = 'SELECT * FROM ' . $GLOBALS['xoopsDB']->prefix('pedigree_fields') . " WHERE isActive = '1' ORDER BY `order`";
+    $sql     = 'SELECT * FROM '.$GLOBALS['xoopsDB']->prefix('pedigree_fields')." WHERE isActive = '1' ORDER BY `order`";
     $result  = $GLOBALS['xoopsDB']->query($sql);
     $numrows = $GLOBALS['xoopsDB']->getRowsNum($result);
     $count   = 0;
     $form .= _MA_PEDIGREE_FIELD_FORM1;
-    $form    .= "<tr ><td colspan='7'><hr></td></tr>";
+    $form .= "<tr ><td colspan='7'><hr></td></tr>";
     $mark = "<td><span style='font-weight: bold;'>X</span></td>\n";
     while (false !== ($row = $GLOBALS['xoopsDB']->fetchArray($result))) {
         $form .= "<tr>\n";
@@ -390,16 +390,16 @@ 
         }
 
         if (0 == $count) { //first row
-            $form .= "<td style='width: 15px;'>&nbsp;</td><td style='width: 15px;'><a href='tools.php?op=fieldmove&field={$row['id']}&move=down'>" . "<img src='assets/images/down.gif' alt='move field down'></a></td>\n";
+            $form .= "<td style='width: 15px;'>&nbsp;</td><td style='width: 15px;'><a href='tools.php?op=fieldmove&field={$row['id']}&move=down'>"."<img src='assets/images/down.gif' alt='move field down'></a></td>\n";
         } elseif ($count == $numrows - 1) { //last row
-            $form .= "<td><a href='tools.php?op=fieldmove&field={$row['id']}&move=up'>" . "<img src='assets/images/up.gif' alt='move field up'></a></td>\n" . "<td>&nbsp;</td>\n";
+            $form .= "<td><a href='tools.php?op=fieldmove&field={$row['id']}&move=up'>"."<img src='assets/images/up.gif' alt='move field up'></a></td>\n"."<td>&nbsp;</td>\n";
         } else { //other rows
             $form .= "<td><a href='tools.php?op=fieldmove&field={$row['id']}&move=up'>"
                      . "<img src='assets/images/up.gif' alt='move field up'></a></td>\n"
                      . "<td><a href='tools.php?op=fieldmove&field={$row['id']}&move=down'>\n"
                      . "<img src='assets/images/down.gif' alt='move field down'></a></td>\n";
         }
-        $form .= "<td><a href='tools.php?op=deluserfield&id={$row['id']}'>" . "<img src='images/delete.png' alt='delete field'></a>&nbsp;" . "<a href='tools.php?op=userfields&field={$row['id']}'>{$row['fieldName']}</a></td>\n";
+        $form .= "<td><a href='tools.php?op=deluserfield&id={$row['id']}'>"."<img src='images/delete.png' alt='delete field'></a>&nbsp;"."<a href='tools.php?op=userfields&field={$row['id']}'>{$row['fieldName']}</a></td>\n";
         //can the filed be shown in a list
         if (1 == $row['ViewInList']) {
             $form .= $mark;
@@ -414,7 +414,7 @@ 
         }
         //has lookuptable ?
         if (1 == $row['LookupTable']) {
-            $form .= "<td><a href='tools.php?op=editlookup&id={$row['id']}'>" . _EDIT . "</a></td>\n";
+            $form .= "<td><a href='tools.php?op=editlookup&id={$row['id']}'>"._EDIT."</a></td>\n";
         } else {
             $form .= "<td>&nbsp;</td>\n";
         }
@@ -423,7 +423,7 @@ 
         ++$count;
     }
     $form .= "</table>\n";
-    $sql    = 'SELECT * FROM ' . $GLOBALS['xoopsDB']->prefix('pedigree_fields') . " WHERE isActive = '0' ORDER BY 'id'";
+    $sql    = 'SELECT * FROM '.$GLOBALS['xoopsDB']->prefix('pedigree_fields')." WHERE isActive = '0' ORDER BY 'id'";
     $result = $GLOBALS['xoopsDB']->query($sql);
     if ($GLOBALS['xoopsDB']->getRowsNum($result) > 0) {
         $form .= _MA_PEDIGREE_FIELD_EXPLAIN5;
@@ -445,7 +445,7 @@ 
 function togglelocked($id)
 {
     //find current status
-    $sql    = 'SELECT locked FROM ' . $GLOBALS['xoopsDB']->prefix('pedigree_fields') . " WHERE id = '{$id}'";
+    $sql    = 'SELECT locked FROM '.$GLOBALS['xoopsDB']->prefix('pedigree_fields')." WHERE id = '{$id}'";
     $result = $GLOBALS['xoopsDB']->query($sql);
     while (false !== ($row = $GLOBALS['xoopsDB']->fetchArray($result))) {
         if (0 == $row['locked']) { //not locked
@@ -465,7 +465,7 @@ 
 function lock($id)
 {
     $id  = (int)$id;
-    $sql = 'UPDATE ' . $GLOBALS['xoopsDB']->prefix('pedigree_fields') . " SET locked = '1' WHERE id = '{$id}'";
+    $sql = 'UPDATE '.$GLOBALS['xoopsDB']->prefix('pedigree_fields')." SET locked = '1' WHERE id = '{$id}'";
     $GLOBALS['xoopsDB']->queryF($sql);
 }
 
@@ -475,7 +475,7 @@ 
 function unlock($id)
 {
     $id  = (int)$id;
-    $sql = 'UPDATE ' . $GLOBALS['xoopsDB']->prefix('pedigree_fields') . " SET locked = '0' WHERE id = '{$id}'";
+    $sql = 'UPDATE '.$GLOBALS['xoopsDB']->prefix('pedigree_fields')." SET locked = '0' WHERE id = '{$id}'";
     $GLOBALS['xoopsDB']->queryF($sql);
 }
 
@@ -491,7 +491,7 @@ 
 function fieldmove($field, $move)
 {
     //find next id
-    $sql    = 'SELECT * FROM ' . $GLOBALS['xoopsDB']->prefix('pedigree_fields') . " WHERE isActive = '1' ORDER BY `order`";
+    $sql    = 'SELECT * FROM '.$GLOBALS['xoopsDB']->prefix('pedigree_fields')." WHERE isActive = '1' ORDER BY `order`";
     $result = $GLOBALS['xoopsDB']->query($sql);
     while (false !== ($row = $GLOBALS['xoopsDB']->fetchArray($result))) {
         $valorder[] = $row['order'];
@@ -513,12 +513,12 @@ 
         $nextorder = $valorder[$x - 1];
     }
     //move value with ID=nextid to original location
-    $sql = 'UPDATE ' . $GLOBALS['xoopsDB']->prefix('pedigree_fields') . " SET `order` = '127' WHERE `order` = '{$nextorder}'";
+    $sql = 'UPDATE '.$GLOBALS['xoopsDB']->prefix('pedigree_fields')." SET `order` = '127' WHERE `order` = '{$nextorder}'";
     $GLOBALS['xoopsDB']->queryF($sql);
-    $sql = 'UPDATE ' . $GLOBALS['xoopsDB']->prefix('pedigree_fields') . " SET `order` = '{$nextorder}' WHERE `order` = '{$currentorder}'";
+    $sql = 'UPDATE '.$GLOBALS['xoopsDB']->prefix('pedigree_fields')." SET `order` = '{$nextorder}' WHERE `order` = '{$currentorder}'";
     $GLOBALS['xoopsDB']->queryF($sql);
     //move current value into nextvalue's spot
-    $sql = 'UPDATE ' . $GLOBALS['xoopsDB']->prefix('pedigree_fields') . " SET `order` = '{$currentorder}' WHERE `order` = '127'";
+    $sql = 'UPDATE '.$GLOBALS['xoopsDB']->prefix('pedigree_fields')." SET `order` = '{$currentorder}' WHERE `order` = '127'";
     $GLOBALS['xoopsDB']->queryF($sql);
     listuserfields();
 }
@@ -531,7 +531,7 @@ 
 function deluserfield($id)
 {
     $id  = (int)$id;
-    $sql = 'UPDATE ' . $GLOBALS['xoopsDB']->prefix('pedigree_fields') . " SET isActive = '0' WHERE id = {$id}";
+    $sql = 'UPDATE '.$GLOBALS['xoopsDB']->prefix('pedigree_fields')." SET isActive = '0' WHERE id = {$id}";
     $GLOBALS['xoopsDB']->queryF($sql);
     listuserfields();
 }
@@ -545,7 +545,7 @@ 
 function restoreuserfield($id)
 {
     $id  = (int)$id;
-    $sql = 'UPDATE ' . $GLOBALS['xoopsDB']->prefix('pedigree_fields') . " SET isActive = '1' WHERE id = {$id}";
+    $sql = 'UPDATE '.$GLOBALS['xoopsDB']->prefix('pedigree_fields')." SET isActive = '1' WHERE id = {$id}";
     $GLOBALS['xoopsDB']->queryF($sql);
     listuserfields();
 }
@@ -558,7 +558,7 @@ 
 function editlookup($field)
 {
     $form    = _MA_PEDIGREE_LOOKUPFIELD;
-    $sql     = 'SELECT * FROM ' . $GLOBALS['xoopsDB']->prefix('pedigree_lookup' . $field) . ' ORDER BY `order`';
+    $sql     = 'SELECT * FROM '.$GLOBALS['xoopsDB']->prefix('pedigree_lookup'.$field).' ORDER BY `order`';
     $result  = $GLOBALS['xoopsDB']->query($sql);
     $numrows = $GLOBALS['xoopsDB']->getRowsNum($result);
     $count   = 0;
@@ -598,7 +598,7 @@ 
         $form .= "</tr>\n";
         ++$count;
     }
-    $form .= "</table>\n" . "<form method='post' action='tools.php?op=addlookupvalue&field={$field}'>\n" . "<input type='text' name='value' style='width: 140px;'>&nbsp;\n" . "<input type='submit' value='Add value'>\n" . _MA_PEDIGREE_DELVALUE . "\n";
+    $form .= "</table>\n"."<form method='post' action='tools.php?op=addlookupvalue&field={$field}'>\n"."<input type='text' name='value' style='width: 140px;'>&nbsp;\n"."<input type='submit' value='Add value'>\n"._MA_PEDIGREE_DELVALUE."\n";
     //    $form .= '<br><br><input type="submit" name="reset" value=Exit>&nbsp;';
     $GLOBALS['xoopsTpl']->assign('form', $form);
 }
@@ -611,7 +611,7 @@ 
 function lookupmove($field, $id, $move)
 {
     //find next id
-    $sql    = 'SELECT * FROM ' . $GLOBALS['xoopsDB']->prefix('pedigree_lookup' . $field) . ' ORDER BY `order`';
+    $sql    = 'SELECT * FROM '.$GLOBALS['xoopsDB']->prefix('pedigree_lookup'.$field).' ORDER BY `order`';
     $result = $GLOBALS['xoopsDB']->query($sql);
     while (false !== ($row = $GLOBALS['xoopsDB']->fetchArray($result))) {
         $values[] = ['id' => $row['id'], 'content' => $row['value'], 'orderof' => $row['order']];
@@ -637,9 +637,9 @@ 
         $nextid    = $values[$arraylocation - 1]['id'];
         $nextorder = $values[$arraylocation - 1]['orderof'];
     }
-    $sql = 'UPDATE `draaf_pedigree_lookup' . $field . "` SET `order` = '" . $nextorder . "' WHERE `id` = '" . $id . "'";
+    $sql = 'UPDATE `draaf_pedigree_lookup'.$field."` SET `order` = '".$nextorder."' WHERE `id` = '".$id."'";
     $GLOBALS['xoopsDB']->queryF($sql);
-    $sql = 'UPDATE `draaf_pedigree_lookup' . $field . "` SET `order` = '" . $currentorder . "' WHERE `id` = '" . $nextid . "'";
+    $sql = 'UPDATE `draaf_pedigree_lookup'.$field."` SET `order` = '".$currentorder."' WHERE `id` = '".$nextid."'";
     $GLOBALS['xoopsDB']->queryF($sql);
     editlookup($field);
 }
@@ -654,10 +654,10 @@ 
  */
 function editlookupvalue($field, $id)
 {
-    $sql    = 'SELECT * FROM ' . $GLOBALS['xoopsDB']->prefix('pedigree_lookup' . $field) . ' WHERE id =' . $id;
+    $sql    = 'SELECT * FROM '.$GLOBALS['xoopsDB']->prefix('pedigree_lookup'.$field).' WHERE id ='.$id;
     $result = $GLOBALS['xoopsDB']->query($sql);
     while (false !== ($row = $GLOBALS['xoopsDB']->fetchArray($result))) {
-        $form = "<form method='post' action='tools.php?op=savelookupvalue&field={$field}&id={$id}'>" . "<input type='text' name='value' value='{$row['value']}' style='width: 140px;'>&nbsp;" . "<input type='submit' value='Save value'>\n";
+        $form = "<form method='post' action='tools.php?op=savelookupvalue&field={$field}&id={$id}'>"."<input type='text' name='value' value='{$row['value']}' style='width: 140px;'>&nbsp;"."<input type='submit' value='Save value'>\n";
     }
     $GLOBALS['xoopsTpl']->assign('form', $form);
 }
@@ -671,9 +671,9 @@ 
  */
 function savelookupvalue($field, $id)
 {
-    $id    = (int)$id;  //sanitize id
+    $id    = (int)$id; //sanitize id
     $value = Request::getString('value', '', 'POST');
-    $SQL   = 'UPDATE ' . $GLOBALS['xoopsDB']->prefix('pedigree_lookup' . $field) . " SET value = '{$value}' WHERE id = {$id}";
+    $SQL   = 'UPDATE '.$GLOBALS['xoopsDB']->prefix('pedigree_lookup'.$field)." SET value = '{$value}' WHERE id = {$id}";
     $GLOBALS['xoopsDB']->queryF($SQL);
     redirect_header("tools.php?op=editlookup&id={$field}", 2, 'The value has been saved.');
 }
@@ -692,16 +692,16 @@ 
     //    $default     = $fieldObject->defaultvalue;
     $default = $GLOBALS['xoopsDB']->escape($fieldObject->defaultvalue);
     if ($default == $id) {
-        redirect_header('tools.php?op=editlookup&id=' . $field, 3, _MA_PEDIGREE_NO_DELETE . $fieldObject->fieldname);
+        redirect_header('tools.php?op=editlookup&id='.$field, 3, _MA_PEDIGREE_NO_DELETE.$fieldObject->fieldname);
     }
-    $sql = 'DELETE FROM ' . $GLOBALS['xoopsDB']->prefix('pedigree_lookup' . $field) . ' WHERE id = ' . $id;
+    $sql = 'DELETE FROM '.$GLOBALS['xoopsDB']->prefix('pedigree_lookup'.$field).' WHERE id = '.$id;
     $GLOBALS['xoopsDB']->queryF($sql);
     //change current values to default for deleted value
-    $sql = 'UPDATE ' . $GLOBALS['xoopsDB']->prefix('pedigree_tree') . ' SET user' . $field . " = '" . $default . "' WHERE user" . $field . " = '" . $id . "'";
+    $sql = 'UPDATE '.$GLOBALS['xoopsDB']->prefix('pedigree_tree').' SET user'.$field." = '".$default."' WHERE user".$field." = '".$id."'";
     $GLOBALS['xoopsDB']->queryF($sql);
-    $sql = 'UPDATE ' . $GLOBALS['xoopsDB']->prefix('pedigree_temp') . ' SET user' . $field . " = '" . $default . "' WHERE user" . $field . " = '" . $id . "'";
+    $sql = 'UPDATE '.$GLOBALS['xoopsDB']->prefix('pedigree_temp').' SET user'.$field." = '".$default."' WHERE user".$field." = '".$id."'";
     $GLOBALS['xoopsDB']->queryF($sql);
-    $sql = 'UPDATE ' . $GLOBALS['xoopsDB']->prefix('pedigree_trash') . ' SET user' . $field . " = '" . $default . "' WHERE user" . $field . " = '" . $id . "'";
+    $sql = 'UPDATE '.$GLOBALS['xoopsDB']->prefix('pedigree_trash').' SET user'.$field." = '".$default."' WHERE user".$field." = '".$id."'";
     $GLOBALS['xoopsDB']->queryF($sql);
     editlookup($field);
 }
@@ -711,15 +711,15 @@ 
  */
 function addlookupvalue($field)
 {
-    $SQL    = 'SELECT id FROM ' . $GLOBALS['xoopsDB']->prefix('pedigree_lookup' . $field) . ' ORDER BY id DESC LIMIT 1';
+    $SQL    = 'SELECT id FROM '.$GLOBALS['xoopsDB']->prefix('pedigree_lookup'.$field).' ORDER BY id DESC LIMIT 1';
     $result = $GLOBALS['xoopsDB']->query($SQL);
     while (false !== ($row = $GLOBALS['xoopsDB']->fetchArray($result))) {
         $count = $row['id'];
         ++$count;
-        $sql = 'INSERT INTO ' . $GLOBALS['xoopsDB']->prefix('pedigree_lookup' . $field) . " VALUES ('" . $count . "', '" . $_POST['value'] . "', '" . $count . "')";
+        $sql = 'INSERT INTO '.$GLOBALS['xoopsDB']->prefix('pedigree_lookup'.$field)." VALUES ('".$count."', '".$_POST['value']."', '".$count."')";
         $GLOBALS['xoopsDB']->queryF($sql);
     }
-    redirect_header('tools.php?op=editlookup&id=' . $field, 2, 'The value has been added.');
+    redirect_header('tools.php?op=editlookup&id='.$field, 2, 'The value has been added.');
 }
 
 /**
@@ -728,7 +728,7 @@ 
 function userfields($field = 0)
 {
     global $field;
-    require_once __DIR__ . '/include/checkoutwizard.php';
+    require_once __DIR__.'/include/checkoutwizard.php';
 
     $wizard = new CheckoutWizard();
     $action = $wizard->coalesce($_GET['action']);
@@ -745,29 +745,29 @@ 
         } else {
             $form = _MA_PEDIGREE_FIELDPROP_ELSE;
         }
-        $form .= "<form method='post' action='" . $_SERVER['PHP_SELF'] . '?op=userfields&action=' . $wizard->resetAction() . "'>";
-        $form .= "<input type='submit' value='" . _MA_PEDIGREE_FINISH_BUTTON . "'></form>";
+        $form .= "<form method='post' action='".$_SERVER['PHP_SELF'].'?op=userfields&action='.$wizard->resetAction()."'>";
+        $form .= "<input type='submit' value='"._MA_PEDIGREE_FINISH_BUTTON."'></form>";
     } else {
-        $form = "<form method='post' action='" . $_SERVER['PHP_SELF'] . '?op=userfields&action=' . $wizard->getStepName() . "'>";
+        $form = "<form method='post' action='".$_SERVER['PHP_SELF'].'?op=userfields&action='.$wizard->getStepName()."'>";
         if (0 == !$field) {
             $form .= "<input type='hidden' name='field' value='{$field}'>";
         }
         if (0 == !$wizard->getValue('field')) { // field allready exists (editing mode)
-            $form .= '<h2>' . $wizard->getStepProperty('title') . ' - ' . $wizard->getValue('name') . ' - step ' . $wizard->getStepNumber() . '</h2>';
+            $form .= '<h2>'.$wizard->getStepProperty('title').' - '.$wizard->getValue('name').' - step '.$wizard->getStepNumber().'</h2>';
         } else {
-            $form .= '<h2>' . $wizard->getStepProperty('title') . ' - step ' . $wizard->getStepNumber() . '</h2>';
+            $form .= '<h2>'.$wizard->getStepProperty('title').' - step '.$wizard->getStepNumber().'</h2>';
         }
 
         if ('fieldname' === $wizard->getStepName()) {
             $form .= _MA_PEDIGREE_FIELD_FORM2;
-            $form .= "<input type='text' name='name' value='" . htmlspecialchars($wizard->getValue('name'), ENT_QUOTES | ENT_HTML5) . "'>";
+            $form .= "<input type='text' name='name' value='".htmlspecialchars($wizard->getValue('name'), ENT_QUOTES | ENT_HTML5)."'>";
             $form .= "</td><td style='width: 25%;'>";
             if ($wizard->isError('name')) {
                 $form .= $wizard->getError('name');
             }
             $form .= '</td></tr>';
             $form .= _MA_PEDIGREE_FIELD_FORM3;
-            $form .= "<textarea name='explain' rows='5' cols='15'>" . htmlspecialchars($wizard->getValue('explain'), ENT_QUOTES | ENT_HTML5) . '</textarea>';
+            $form .= "<textarea name='explain' rows='5' cols='15'>".htmlspecialchars($wizard->getValue('explain'), ENT_QUOTES | ENT_HTML5).'</textarea>';
             $form .= "</td><td style='width: 25%;'>";
             if ($wizard->isError('explain')) {
                 $form .= $wizard->getError('explain');
@@ -798,10 +798,10 @@ 
             $form .= "<input type='hidden' name='fc' value='{$i}'>";
             $form .= '<table>';
             for ($x = 1; $x < $i; ++$x) {
-                $form .= "<tr><td>Value : ({$x})</td><td>" . htmlspecialchars($wizard->getValue('lookup' . $x), ENT_QUOTES | ENT_HTML5) . '</td></tr>';
+                $form .= "<tr><td>Value : ({$x})</td><td>".htmlspecialchars($wizard->getValue('lookup'.$x), ENT_QUOTES | ENT_HTML5).'</td></tr>';
             }
             $form .= "<tr><td>Value : ({$i})</td><td>";
-            $form .= "<input type='text' name='lookup{$i}' value='" . htmlspecialchars($wizard->getValue('lookup' . $i), ENT_QUOTES | ENT_HTML5) . "'>";
+            $form .= "<input type='text' name='lookup{$i}' value='".htmlspecialchars($wizard->getValue('lookup'.$i), ENT_QUOTES | ENT_HTML5)."'>";
             $form .= "<input type='hidden' name='id{$i}' value='{$i}'>";
             if ($wizard->isError('lookup')) {
                 $form .= $wizard->getError('lookup');
@@ -817,25 +817,25 @@ 
                     if ('hassearch' === $wizard->getValue('hassearch')) {
                         $form .= ' checked =_MA_PEDIGREE_CHECKED ';
                     }
-                    $form .= '>' . _MA_PEDIGREE_FIELDSEARCH . '<br>';
+                    $form .= '>'._MA_PEDIGREE_FIELDSEARCH.'<br>';
                 } else {
-                    $form .= "<input type='checkbox' name='hassearch' disabled='true' value='hassearch'>" . _MA_PEDIGREE_FIELDSEARCH . '<br>';
+                    $form .= "<input type='checkbox' name='hassearch' disabled='true' value='hassearch'>"._MA_PEDIGREE_FIELDSEARCH.'<br>';
                 }
                 //viewinpedigree
                 $form .= "<input type='checkbox' name='viewinpedigree' value='viewinpedigree'";
                 if ('viewinpedigree' === $wizard->getValue('viewinpedigree')) {
                     $form .= ' checked =_MA_PEDIGREE_CHECKED ';
                 }
-                $form .= '>' . _MA_PEDIGREE_SHOWFIELD . '<br>';
+                $form .= '>'._MA_PEDIGREE_SHOWFIELD.'<br>';
                 //viewinadvanced
                 if ('radiobutton' === $fieldtype || 'selectbox' === $fieldtype) {
                     $form .= "<input type='checkbox' name='viewinadvanced' value='viewinadvanced'";
                     if ('viewinadvanced' === $wizard->getValue('viewinadvanced')) {
                         $form .= ' checked=checked';
                     }
-                    $form .= '>' . _MA_PEDIGREE_SHOWFIELD_ADVANCE . '<br>';
+                    $form .= '>'._MA_PEDIGREE_SHOWFIELD_ADVANCE.'<br>';
                 } else {
-                    $form .= "<input type='checkbox' name='viewinadvanced' disabled='true' value='viewinadvanced'>" . _MA_PEDIGREE_SHOWFIELD_ADVANCE . '<br>';
+                    $form .= "<input type='checkbox' name='viewinadvanced' disabled='true' value='viewinadvanced'>"._MA_PEDIGREE_SHOWFIELD_ADVANCE.'<br>';
                 }
                 //viewinpie
                 if ('radiobutton' === $fieldtype || 'selectbox' === $fieldtype) {
@@ -843,9 +843,9 @@ 
                     if ('viewinpie' === $wizard->getValue('viewinpie')) {
                         $form .= ' checked =_MA_PEDIGREE_CHECKED ';
                     }
-                    $form .= '>' . _MA_PEDIGREE_SHOWFIELD_PIECHART . '<br>';
+                    $form .= '>'._MA_PEDIGREE_SHOWFIELD_PIECHART.'<br>';
                 } else {
-                    $form .= "<input type='checkbox' name='viewinpie' disabled='true' value='viewinpie'>" . _MA_PEDIGREE_SHOWFIELD_PIECHART . '<br>';
+                    $form .= "<input type='checkbox' name='viewinpie' disabled='true' value='viewinpie'>"._MA_PEDIGREE_SHOWFIELD_PIECHART.'<br>';
                 }
                 //viewinlist
                 if (in_array($fieldtype, ['textbox', 'dateselect', 'urlfield', 'radiobutton', 'selectbox'])) {
@@ -853,9 +853,9 @@ 
                     if ('viewinlist' === $wizard->getValue('viewinlist')) {
                         $form .= ' checked =_MA_PEDIGREE_CHECKED ';
                     }
-                    $form .= '>' . _MA_PEDIGREE_SHOWFIELD_RESULT . '<br>';
+                    $form .= '>'._MA_PEDIGREE_SHOWFIELD_RESULT.'<br>';
                 } else {
-                    $form .= "<input type='checkbox' name='viewinlist' disabled='true' value='viewinlist'>" . _MA_PEDIGREE_SHOWFIELD_RESULT . '<br>';
+                    $form .= "<input type='checkbox' name='viewinlist' disabled='true' value='viewinlist'>"._MA_PEDIGREE_SHOWFIELD_RESULT.'<br>';
                 }
                 //add a litter
                 $form .= "<input type='checkbox' name='Litter' value='Litter'";
@@ -865,7 +865,7 @@ 
                     //disable if generallitter is active (you cant have both)
                     $form .= " disabled='true'";
                 }
-                $form .= '>' . _MA_PEDIGREE_SHOWFIELD_ADDLITTER . '<br>';
+                $form .= '>'._MA_PEDIGREE_SHOWFIELD_ADDLITTER.'<br>';
 
                 //add a litter general
                 $form .= "<input type='checkbox' name='Generallitter' value='Generallitter'";
@@ -875,7 +875,7 @@ 
                     //disable if add a litter is active (you cant have both)
                     $form .= " disabled='true'";
                 }
-                $form .= '>' . _MA_PEDIGREE_SHOWFIELD_ADDLITTGLOBAL . '<br>';
+                $form .= '>'._MA_PEDIGREE_SHOWFIELD_ADDLITTGLOBAL.'<br>';
             } else {
                 if ('search' === $wizard->getStepName()) {
                     $form .= _MA_PEDIGREE_SEARCH_NAME;
@@ -892,7 +892,7 @@ 
                     }
                     $form .= '</td></tr>';
                     $form .= _MA_PEDIGREE_FIELDEXPLANSEARCH;
-                    $form .= "<textarea name='searchexplain' rows='5' cols='15'>" . htmlspecialchars($wizard->getValue('searchexplain'), ENT_QUOTES | ENT_HTML5) . '</textarea>';
+                    $form .= "<textarea name='searchexplain' rows='5' cols='15'>".htmlspecialchars($wizard->getValue('searchexplain'), ENT_QUOTES | ENT_HTML5).'</textarea>';
                     $form .= "</td><td style='width: 25%;'>";
                     if ($wizard->isError('searchexplain')) {
                         $form .= $wizard->getError('searchexplain');
@@ -906,15 +906,15 @@ 
                         $form .= "Default value : <select size='1' name='defaultvalue'>";
                         $radioarray = $wizard->getValue('radioarray');
                         foreach ($radioarray as $x => $xValue) {
-                            $form .= "<option value='" . $radioarray[$x]['id'] . "'";
+                            $form .= "<option value='".$radioarray[$x]['id']."'";
                             if ($wizard->getValue('defaultvalue') == $radioarray[$x]['id']) {
                                 $form .= ' selected';
                             }
-                            $form .= ' >' . $radioarray[$x]['value'] . '</option>';
+                            $form .= ' >'.$radioarray[$x]['value'].'</option>';
                         }
                         $form .= '</select>';
                     } else {
-                        $form .= '' . _MA_PEDIGREE_FIELDVALUE . " <input type='text' name='defaultvalue' value='" . htmlspecialchars($wizard->getValue('defaultvalue'), ENT_QUOTES | ENT_HTML5) . "'>";
+                        $form .= ''._MA_PEDIGREE_FIELDVALUE." <input type='text' name='defaultvalue' value='".htmlspecialchars($wizard->getValue('defaultvalue'), ENT_QUOTES | ENT_HTML5)."'>";
                     }
                     $form .= _MA_PEDIGREE_DEFAUTVALUE_EXPLAIN;
                 } elseif ('confirm' === $wizard->getStepName()) {
@@ -926,27 +926,27 @@ 
                 }
                 //            }
                 //        }
-                $form .= '<b>' . _MA_PEDIGREE_FIELDCONTROL4 . $wizard->getValue('name') . '</b><br>';
-                $form .= '<b>' . _MA_PEDIGREE_FIELDCONTROL5 . stripslashes($wizard->getValue('explain')) . '</b><br>';
-                $form .= '<b>' . _MA_PEDIGREE_FIELDCONTROL6 . $wizard->getValue('fieldtype') . '</b><br>';
+                $form .= '<b>'._MA_PEDIGREE_FIELDCONTROL4.$wizard->getValue('name').'</b><br>';
+                $form .= '<b>'._MA_PEDIGREE_FIELDCONTROL5.stripslashes($wizard->getValue('explain')).'</b><br>';
+                $form .= '<b>'._MA_PEDIGREE_FIELDCONTROL6.$wizard->getValue('fieldtype').'</b><br>';
                 if ('selectbox' === $wizard->getValue('fieldtype')
                     || 'radiobutton' === $wizard->getValue('fieldtype')) {
                     $count = $wizard->getValue('fc');
                     for ($x = 1; $x < $count + 1; ++$x) {
                         $radioarray[] = [
-                            'id'    => $wizard->getValue('id' . $x),
-                            'value' => $wizard->getValue('lookup' . $x)
+                            'id'    => $wizard->getValue('id'.$x),
+                            'value' => $wizard->getValue('lookup'.$x)
                         ];
                     }
                     $val = $wizard->getValue('defaultvalue');
-                    $form .= '<b>' . _MA_PEDIGREE_FIELDCONTROL7 . $wizard->getValue('lookup' . $val) . '</b><br>';
+                    $form .= '<b>'._MA_PEDIGREE_FIELDCONTROL7.$wizard->getValue('lookup'.$val).'</b><br>';
                 } else {
-                    $form .= '<b>' . _MA_PEDIGREE_FIELDCONTROL7 . $wizard->getValue('defaultvalue') . '</b><br>';
+                    $form .= '<b>'._MA_PEDIGREE_FIELDCONTROL7.$wizard->getValue('defaultvalue').'</b><br>';
                 }
                 if ('hasearch' === $wizard->getValue('hassearch')) {
-                    $form .= _MA_PEDIGREE_FIELDSEARCH . ' : <b>  ' . _YES . '</b><br>';
-                    $form .= '<b>' . _MA_PEDIGREE_FIELDSEARCH . $wizard->getValue('searchname') . '</b><br>';
-                    $form .= '<b>' . _MA_PEDIGREE_FIELDCONTROL5 . $wizard->getValue('searhexplain') . '</b><br>';
+                    $form .= _MA_PEDIGREE_FIELDSEARCH.' : <b>  '._YES.'</b><br>';
+                    $form .= '<b>'._MA_PEDIGREE_FIELDSEARCH.$wizard->getValue('searchname').'</b><br>';
+                    $form .= '<b>'._MA_PEDIGREE_FIELDCONTROL5.$wizard->getValue('searhexplain').'</b><br>';
                 }
                 if ('viewinpedigree' === $wizard->getValue('viewinpedigree')) {
                     $form .= _MA_PEDIGREE_SYNTH1;
@@ -966,13 +966,13 @@ 
                 if ('Generallitter' === $wizard->getValue('generallitter')) {
                     $form .= _MA_PEDIGREE_SYNTH6;
                 }
-                $form .= _MA_PEDIGREE_FIELDCONTROL3 . '<br><br>';
+                $form .= _MA_PEDIGREE_FIELDCONTROL3.'<br><br>';
             }
         }
     }
 
     if (!$wizard->isFirstStep()) {
-        $form .= "<p><input type='submit' name='previous' value='&lt;&lt; " . _MA_PEDIGREE_PREVIOUS_BUTTON . "'>&nbsp;";
+        $form .= "<p><input type='submit' name='previous' value='&lt;&lt; "._MA_PEDIGREE_PREVIOUS_BUTTON."'>&nbsp;";
     }
     $form .= "<input type='submit' name='reset' value=Exit>&nbsp;";
     $last = $wizard->isLastStep() ? _MA_PEDIGREE_FINISH_BUTTON : _MA_PEDIGREE_NEXT_BUTTON;
@@ -991,7 +991,7 @@ 
     $moduleConfig  = $configHandler->getConfigsByCat(0, $module->getVar('mid'));
     */
     $helper = Pedigree\Helper::getInstance(false);
-    $form     = 'Pedigree database module<br><br><li>Programming : James Cotton<br><li>Design & Layout : Ton van der Hagen<br><li>Version : '
+    $form = 'Pedigree database module<br><br><li>Programming : James Cotton<br><li>Design & Layout : Ton van der Hagen<br><li>Version : '
                 . round($helper->getModule()->getVar('version') / 100, 2)
                 . ' '
                 . $helper->getModule()->getVar('module_status')
@@ -1009,7 +1009,7 @@ 
 function userq()
 {
     $form = _MA_PEDIGREE_QUERIE_EXPLAN;
-    $d    = $GLOBALS['xoops']->path('modules/' . $GLOBALS['xoopsModule']->dirname() . '/userqueries/');
+    $d    = $GLOBALS['xoops']->path('modules/'.$GLOBALS['xoopsModule']->dirname().'/userqueries/');
 
     $dir = opendir($d);
     while (false !== ($f = readdir($dir))) {
@@ -1025,7 +1025,7 @@ 
  */
 function userqrun($file)
 {
-    include $GLOBALS['xoops']->path('modules/' . $GLOBALS['xoopsModule']->dirname() . "/userqueries/{$file}");
+    include $GLOBALS['xoops']->path('modules/'.$GLOBALS['xoopsModule']->dirname()."/userqueries/{$file}");
     $GLOBALS['xoopsTpl']->assign('form', $form);
 }
 
@@ -1038,13 +1038,13 @@ 
 {
     $form   = _MA_PEDIGREE_ANCEST_EXPLAN;
     $sql    = 'SELECT d.id AS d_id, d.naam AS d_naam
-            FROM ' . $GLOBALS['xoopsDB']->prefix('pedigree_tree') . ' d
-            LEFT JOIN ' . $GLOBALS['xoopsDB']->prefix('pedigree_tree') . ' m ON m.id = d.mother
-            LEFT JOIN ' . $GLOBALS['xoopsDB']->prefix('pedigree_tree') . ' f ON f.id = d.father
-            LEFT JOIN ' . $GLOBALS['xoopsDB']->prefix('pedigree_tree') . ' mm ON mm.id = m.mother
-            LEFT JOIN ' . $GLOBALS['xoopsDB']->prefix('pedigree_tree') . ' mf ON mf.id = m.father
-            LEFT JOIN ' . $GLOBALS['xoopsDB']->prefix('pedigree_tree') . ' fm ON fm.id = f.mother
-            LEFT JOIN ' . $GLOBALS['xoopsDB']->prefix('pedigree_tree') . ' ff ON ff.id = f.father
+            FROM ' . $GLOBALS['xoopsDB']->prefix('pedigree_tree').' d
+            LEFT JOIN ' . $GLOBALS['xoopsDB']->prefix('pedigree_tree').' m ON m.id = d.mother
+            LEFT JOIN ' . $GLOBALS['xoopsDB']->prefix('pedigree_tree').' f ON f.id = d.father
+            LEFT JOIN ' . $GLOBALS['xoopsDB']->prefix('pedigree_tree').' mm ON mm.id = m.mother
+            LEFT JOIN ' . $GLOBALS['xoopsDB']->prefix('pedigree_tree').' mf ON mf.id = m.father
+            LEFT JOIN ' . $GLOBALS['xoopsDB']->prefix('pedigree_tree').' fm ON fm.id = f.mother
+            LEFT JOIN ' . $GLOBALS['xoopsDB']->prefix('pedigree_tree').' ff ON ff.id = f.father
             WHERE
             d.mother = d.id
             OR d.father = d.id
@@ -1076,8 +1076,8 @@ 
 {
     $form   = _MA_PEDIGREE_GENDER_EXPLAN;
     $sql    = 'SELECT d.id AS d_id, d.naam AS d_naam, d.mother AS d_mother, m.roft AS m_roft
-            FROM ' . $GLOBALS['xoopsDB']->prefix('pedigree_tree') . ' d
-            LEFT JOIN ' . $GLOBALS['xoopsDB']->prefix('pedigree_tree') . " m ON m.id = d.mother
+            FROM ' . $GLOBALS['xoopsDB']->prefix('pedigree_tree').' d
+            LEFT JOIN ' . $GLOBALS['xoopsDB']->prefix('pedigree_tree')." m ON m.id = d.mother
             WHERE
             d.mother = m.id
             AND m.roft = '0' ";
@@ -1085,15 +1085,15 @@ 
     while (false !== ($row = $GLOBALS['xoopsDB']->fetchArray($result))) {
         $form .= "<li><a href='dog.php?id={$row['d_id']}'>{$row['d_naam']}</a> [mother seems to be male]<br>";
     }
-    $sql    = 'SELECT d.id AS d_id, d.naam AS d_naam, d.father AS d_father, f.roft AS f_roft
-            FROM ' . $GLOBALS['xoopsDB']->prefix('pedigree_tree') . ' d
-            LEFT JOIN ' . $GLOBALS['xoopsDB']->prefix('pedigree_tree') . " f ON f.id = d.father
+    $sql = 'SELECT d.id AS d_id, d.naam AS d_naam, d.father AS d_father, f.roft AS f_roft
+            FROM ' . $GLOBALS['xoopsDB']->prefix('pedigree_tree').' d
+            LEFT JOIN ' . $GLOBALS['xoopsDB']->prefix('pedigree_tree')." f ON f.id = d.father
             WHERE
             d.father = f.id
             AND f.roft = '1' ";
     $result = $GLOBALS['xoopsDB']->query($sql);
     while (false !== ($row = $GLOBALS['xoopsDB']->fetchArray($result))) {
-        $form .= '<li><a href="dog.php?id=' . $row['d_id'] . '">' . $row['d_naam'] . '</a> [father seems to be female]<br>';
+        $form .= '<li><a href="dog.php?id='.$row['d_id'].'">'.$row['d_naam'].'</a> [father seems to be female]<br>';
     }
     $GLOBALS['xoopsTpl']->assign('form', $form);
 }
@@ -1118,13 +1118,13 @@ 
 {
     global $helper;
     $form   = "Below the line are the animals which have been deleted from your database.<br><br>By clicking on the name you can reinsert them into the database.<br>By clicking on the 'X' in front of the name you can permanently delete the animal.<hr>";
-    $sql    = 'SELECT id, naam  FROM ' . $GLOBALS['xoopsDB']->prefix('pedigree_trash');
+    $sql    = 'SELECT id, naam  FROM '.$GLOBALS['xoopsDB']->prefix('pedigree_trash');
     $result = $GLOBALS['xoopsDB']->query($sql);
     while (false !== ($row = $GLOBALS['xoopsDB']->fetchArray($result))) {
-        $form .= '<a href="tools.php?op=delperm&id=' . $row['id'] . '"><img src=' . $GLOBALS['pathIcon16'] . '/delete.png></a>&nbsp;<a href="tools.php?op=restore&id=' . $row['id'] . '">' . $row['naam'] . '</a><br>';
+        $form .= '<a href="tools.php?op=delperm&id='.$row['id'].'"><img src='.$GLOBALS['pathIcon16'].'/delete.png></a>&nbsp;<a href="tools.php?op=restore&id='.$row['id'].'">'.$row['naam'].'</a><br>';
     }
     if ($GLOBALS['xoopsDB']->getRowsNum($result) > 0) {
-        $form .= '<hr><a href="tools.php?op=delall">Click here</a> to remove all these ' . $helper->getConfig('animalTypes') . ' permenantly ';
+        $form .= '<hr><a href="tools.php?op=delall">Click here</a> to remove all these '.$helper->getConfig('animalTypes').' permenantly ';
     }
     $GLOBALS['xoopsTpl']->assign('form', $form);
 }
@@ -1134,14 +1134,14 @@ 
  */
 function delperm($id)
 {
-    $sql = 'DELETE FROM ' . $GLOBALS['xoopsDB']->prefix('pedigree_trash') . " WHERE id = {$id}";
+    $sql = 'DELETE FROM '.$GLOBALS['xoopsDB']->prefix('pedigree_trash')." WHERE id = {$id}";
     $GLOBALS['xoopsDB']->queryF($sql);
     deleted();
 }
 
 function delall()
 {
-    $sql = 'DELETE FROM ' . $GLOBALS['xoopsDB']->prefix('pedigree_trash');
+    $sql = 'DELETE FROM '.$GLOBALS['xoopsDB']->prefix('pedigree_trash');
     $GLOBALS['xoopsDB']->queryF($sql);
     deleted();
 }
@@ -1152,18 +1152,18 @@ 
 function restore($id)
 {
     $queryvalues = '';
-    $sql         = 'SELECT * FROM ' . $GLOBALS['xoopsDB']->prefix('pedigree_trash') . ' WHERE id = ' . $id;
+    $sql         = 'SELECT * FROM '.$GLOBALS['xoopsDB']->prefix('pedigree_trash').' WHERE id = '.$id;
     $result      = $GLOBALS['xoopsDB']->query($sql);
     while (false !== ($row = $GLOBALS['xoopsDB']->fetchArray($result))) {
         foreach ($row as $key => $values) {
-            $queryvalues .= "'" . $values . "',";
+            $queryvalues .= "'".$values."',";
         }
         $outgoing = substr_replace($queryvalues, '', -1);
-        $query    = 'INSERT INTO ' . $GLOBALS['xoopsDB']->prefix('pedigree_tree') . ' VALUES (' . $outgoing . ')';
+        $query    = 'INSERT INTO '.$GLOBALS['xoopsDB']->prefix('pedigree_tree').' VALUES ('.$outgoing.')';
         $GLOBALS['xoopsDB']->queryF($query);
-        $delquery = 'DELETE FROM ' . $GLOBALS['xoopsDB']->prefix('pedigree_trash') . ' WHERE id = ' . $id;
+        $delquery = 'DELETE FROM '.$GLOBALS['xoopsDB']->prefix('pedigree_trash').' WHERE id = '.$id;
         $GLOBALS['xoopsDB']->queryF($delquery);
-        $form = '<li><a href="pedigree.php?pedid=' . $row['id'] . '">' . $row['naam'] . '</a> has been restored into the database.<hr>';
+        $form = '<li><a href="pedigree.php?pedid='.$row['id'].'">'.$row['naam'].'</a> has been restored into the database.<hr>';
     }
     if (isset($form)) {
         $GLOBALS['xoopsTpl']->assign('form', $form);
@@ -1174,7 +1174,7 @@ 
 {
     $helper = Pedigree\Helper::getInstance();
 
-    include XOOPS_ROOT_PATH . '/class/xoopsformloader.php';
+    include XOOPS_ROOT_PATH.'/class/xoopsformloader.php';
     $form = new \XoopsThemeForm(_MA_PEDIGREE_BLOCK_SETTING, 'settings', 'tools.php?op=settingssave', 'POST', 1);
     $form->addElement(new \XoopsFormHiddenToken($name = 'XOOPS_TOKEN_REQUEST', $timeout = 360));
     $select  = new \XoopsFormSelect(_MA_PEDIGREE_RESULT, 'perpage', $value = $helper->getConfig('perpage'), $size = 1, $multiple = false);
@@ -1214,7 +1214,7 @@ 
     $radiolit->addOption(0, $name = _MA_PEDIGREE_NO);
     $form->addElement($radiolit);
     $form->addElement(new \XoopsFormLabel(_MA_PEDIGREE_EXPLAIN, _MA_PEDIGREE_USE_LITTER_EXPLAIN));
-    $radioch = new \XoopsFormRadio(PED_SHOW_KITT_A . $helper->getConfig('children') . _MA_PEDIGREE_SHOW_KITT_B, 'pups', $value = $helper->getConfig('pups'));
+    $radioch = new \XoopsFormRadio(PED_SHOW_KITT_A.$helper->getConfig('children')._MA_PEDIGREE_SHOW_KITT_B, 'pups', $value = $helper->getConfig('pups'));
     $radioch->addOption(1, $name = _MA_PEDIGREE_YES);
     $radioch->addOption(0, $name = _MA_PEDIGREE_NO);
     $form->addElement($radioch);
@@ -1239,7 +1239,7 @@ 
     foreach ($_POST as $key => $values) {
         if (in_array($key, $settings)) {
             //            $query = 'UPDATE ' . $GLOBALS['xoopsDB']->prefix('config') . " SET conf_value = '" . $values . "' WHERE conf_name = '" . $key . "'";
-            $query = 'UPDATE ' . $GLOBALS['xoopsDB']->prefix('config') . " SET conf_value = '" . $GLOBALS['xoopsDB']->escape($values) . "' WHERE conf_name = '" . $GLOBALS['xoopsDB']->escape($key) . "'";
+            $query = 'UPDATE '.$GLOBALS['xoopsDB']->prefix('config')." SET conf_value = '".$GLOBALS['xoopsDB']->escape($values)."' WHERE conf_name = '".$GLOBALS['xoopsDB']->escape($key)."'";
             $GLOBALS['xoopsDB']->query($query);
         }
     }
@@ -1249,28 +1249,28 @@ 
 function lang()
 {
     global $helper;
-    include XOOPS_ROOT_PATH . '/class/xoopsformloader.php';
+    include XOOPS_ROOT_PATH.'/class/xoopsformloader.php';
     $form = new \XoopsThemeForm(_MA_PEDIGREE_BLOCK_NAME, 'language', 'tools.php?op=langsave', 'post', true);
     $form->addElement(new \XoopsFormHiddenToken($name = 'XOOPS_TOKEN_REQUEST', $timeout = 360));
     $form->addElement(new \XoopsFormText(_MA_PEDIGREE_TYPE_AN, 'animalType', $size = 50, $maxsize = 255, $value = $helper->getConfig('animalType')));
-    $form->addElement(new \XoopsFormLabel(_MA_PEDIGREE_EXPLAIN, _MA_PEDIGREE_FIELD_EXPLAIN . $helper->getConfig('animalType') . _MA_PEDIGREE_SEARCH_FORM . $helper->getConfig('animalType') . '</b>.'));
+    $form->addElement(new \XoopsFormLabel(_MA_PEDIGREE_EXPLAIN, _MA_PEDIGREE_FIELD_EXPLAIN.$helper->getConfig('animalType')._MA_PEDIGREE_SEARCH_FORM.$helper->getConfig('animalType').'</b>.'));
     $form->addElement(new \XoopsFormText(_MA_PEDIGREE_TYPE_AN, 'animalTypes', $size = 50, $maxsize = 255, $value = $value = $helper->getConfig('animalTypes')));
-    $form->addElement(new \XoopsFormLabel(_MA_PEDIGREE_EXPLAIN, _MA_PEDIGREE_FIELD_EXPLAIN2 . $helper->getConfig('animalTypes') . _MA_PEDIGREE_FIELD_EXPLAIN3));
+    $form->addElement(new \XoopsFormLabel(_MA_PEDIGREE_EXPLAIN, _MA_PEDIGREE_FIELD_EXPLAIN2.$helper->getConfig('animalTypes')._MA_PEDIGREE_FIELD_EXPLAIN3));
     $form->addElement(new \XoopsFormText(_MA_PEDIGREE_MALE, 'male', $size = 50, $maxsize = 255, $value = $helper->getConfig('male')));
     $form->addElement(new \XoopsFormLabel(_MA_PEDIGREE_EXPLAIN, _MA_PEDIGREE_MALE_EXPLAIN));
     $form->addElement(new \XoopsFormText(_MA_PEDIGREE_FEMALE, 'female', $size = 50, $maxsize = 255, $value = $helper->getConfig('female')));
     $form->addElement(new \XoopsFormLabel(_MA_PEDIGREE_EXPLAIN, _MA_PEDIGREE_FEMALE_EXPLAIN));
     $form->addElement(new \XoopsFormText(_MA_PEDIGREE_CHILDREN, 'children', $size = 50, $maxsize = 255, $value = $helper->getConfig('children')));
-    $form->addElement(new \XoopsFormLabel(_MA_PEDIGREE_EXPLAIN, _MA_PEDIGREE_KITTEN_EXPLAIN1 . $helper->getConfig('animalTypes') . _MA_PEDIGREE_KITTEN_EXPLAIN2));
+    $form->addElement(new \XoopsFormLabel(_MA_PEDIGREE_EXPLAIN, _MA_PEDIGREE_KITTEN_EXPLAIN1.$helper->getConfig('animalTypes')._MA_PEDIGREE_KITTEN_EXPLAIN2));
     $form->addElement(new \XoopsFormText(_MA_PEDIGREE_MOTHER, 'mother', $size = 50, $maxsize = 255, $value = $helper->getConfig('mother')));
-    $form->addElement(new \XoopsFormLabel(_MA_PEDIGREE_EXPLAIN, _MA_PEDIGREE_MOTHER1 . $helper->getConfig('animalTypes') . _MA_PEDIGREE_MOTHER2));
+    $form->addElement(new \XoopsFormLabel(_MA_PEDIGREE_EXPLAIN, _MA_PEDIGREE_MOTHER1.$helper->getConfig('animalTypes')._MA_PEDIGREE_MOTHER2));
     $form->addElement(new \XoopsFormText(_MA_PEDIGREE_FATHER, 'father', $size = 50, $maxsize = 255, $value = $helper->getConfig('father')));
-    $form->addElement(new \XoopsFormLabel(_MA_PEDIGREE_EXPLAIN, _MA_PEDIGREE_FATHER1 . $helper->getConfig('animalTypes') . _MA_PEDIGREE_FATHER2));
+    $form->addElement(new \XoopsFormLabel(_MA_PEDIGREE_EXPLAIN, _MA_PEDIGREE_FATHER1.$helper->getConfig('animalTypes')._MA_PEDIGREE_FATHER2));
     $form->addElement(new \XoopsFormText(_MA_PEDIGREE_LITTER, 'litter', $size = 50, $maxsize = 255, $value = $helper->getConfig('litter')));
     $form->addElement(new \XoopsFormLabel(_MA_PEDIGREE_EXPLAIN, _MA_PEDIGREE_LITTER1));
     $form->addElement(new \XoopsFormTextArea(_MA_PEDIGREE_WELC_TEXT, 'welcome', $value = $helper->getConfig('welcome'), $rows = 15, $cols = 50));
 
-    $form->addElement(new \XoopsFormLabel(_MA_PEDIGREE_EXPLAIN, _MA_PEDIGREE_WELC_TXT_EXPLAIN . $helper->getConfig('animalType') . '<br>[animalTypes] = ' . $helper->getConfig('animalTypes') . _MA_PEDIGREE_WELC_TXT_EXPLAIN2));
+    $form->addElement(new \XoopsFormLabel(_MA_PEDIGREE_EXPLAIN, _MA_PEDIGREE_WELC_TXT_EXPLAIN.$helper->getConfig('animalType').'<br>[animalTypes] = '.$helper->getConfig('animalTypes')._MA_PEDIGREE_WELC_TXT_EXPLAIN2));
     $form->addElement(new \XoopsFormButton('', 'button_id', 'Submit', 'submit'));
     $GLOBALS['xoopsTpl']->assign('form', $form->render());
 }
@@ -1296,7 +1296,7 @@ 
     ];
     foreach ($_POST as $key => $values) {
         if (in_array($key, $settings)) {
-            $query = 'UPDATE ' . $GLOBALS['xoopsDB']->prefix('config') . " SET conf_value = '{$values}' WHERE conf_name = '{$key}'";
+            $query = 'UPDATE '.$GLOBALS['xoopsDB']->prefix('config')." SET conf_value = '{$values}' WHERE conf_name = '{$key}'";
             $GLOBALS['xoopsDB']->query($query);
         }
     }

index.php

Indentation +6 added lines, -6 removed lines

@@ -87,12 +87,12 @@
 
 //add data to smarty template
 $GLOBALS['xoopsTpl']->assign([
-                                 'sselect'    => strtr(_MA_PEDIGREE_SELECT, ['[animalType]' => $moduleConfig['animalType']]),
-                                 'explain'     => _MA_PEDIGREE_EXPLAIN,
-                                 'sname'       => _MA_PEDIGREE_SEARCHNAME,
-                                 'snameex'    => strtr(_MA_PEDIGREE_SEARCHNAME_EX, ['[animalTypes]' => $moduleConfig['animalTypes']]),
-                                 'usersearch' => isset($usersearch) ? $usersearch : ''
-                             ]);
+                                    'sselect'    => strtr(_MA_PEDIGREE_SELECT, ['[animalType]' => $moduleConfig['animalType']]),
+                                    'explain'     => _MA_PEDIGREE_EXPLAIN,
+                                    'sname'       => _MA_PEDIGREE_SEARCHNAME,
+                                    'snameex'    => strtr(_MA_PEDIGREE_SEARCHNAME_EX, ['[animalTypes]' => $moduleConfig['animalTypes']]),
+                                    'usersearch' => isset($usersearch) ? $usersearch : ''
+                                ]);
 $GLOBALS['xoopsTpl']->assign('showwelcome', $moduleConfig['showwelcome']);
 //$GLOBALS['xoopsTpl']->assign('welcome', $GLOBALS['myts']->displayTarea($moduleConfig['welcome']));
 //$word = $myts->displayTarea(strtr($helper->getConfig('welcome'), array('[numanimals]' => $numdogs, '[animalType]' => $helper->getConfig('animalType'), '[animalTypes]' => $helper->getConfig('animalTypes'))));

Spacing +14 added lines, -14 removed lines

@@ -3,35 +3,35 @@ 
 
 use XoopsModules\Pedigree;
 
-require_once __DIR__ . '/header.php';
+require_once __DIR__.'/header.php';
 
 $moduleDirName = basename(__DIR__);
 xoops_loadLanguage('main', $moduleDirName);
 
 // Include any common code for this module.
-require_once XOOPS_ROOT_PATH . '/modules/' . $moduleDirName . '/include/common.php';
-require_once __DIR__ . '/welcome.php';
+require_once XOOPS_ROOT_PATH.'/modules/'.$moduleDirName.'/include/common.php';
+require_once __DIR__.'/welcome.php';
 
 $GLOBALS['xoopsOption']['template_main'] = 'pedigree_index.tpl';
 
 include $GLOBALS['xoops']->path('/header.php');
 
 //load javascript
-$xoTheme->addScript(XOOPS_URL . '/browse.php?Frameworks/jquery/jquery.js');
+$xoTheme->addScript(XOOPS_URL.'/browse.php?Frameworks/jquery/jquery.js');
 //$xoTheme->addScript(PEDIGREE_URL . '/assets/js/jquery.ThickBox/thickbox-compressed.js');
 
-$xoTheme->addScript(PEDIGREE_URL . '/assets/js/jquery.magnific-popup.min.js');
-$xoTheme->addScript(PEDIGREE_URL . '/assets/js/colpick.js');
+$xoTheme->addScript(PEDIGREE_URL.'/assets/js/jquery.magnific-popup.min.js');
+$xoTheme->addScript(PEDIGREE_URL.'/assets/js/colpick.js');
 
 //load CSS style sheets
-$xoTheme->addStylesheet(PEDIGREE_URL . '/assets/css/colpick.css');
-$xoTheme->addStylesheet(PEDIGREE_URL . '/assets/css/magnific-popup.css');
-$xoTheme->addStylesheet(PEDIGREE_URL . '/assets/css/style.css');
+$xoTheme->addStylesheet(PEDIGREE_URL.'/assets/css/colpick.css');
+$xoTheme->addStylesheet(PEDIGREE_URL.'/assets/css/magnific-popup.css');
+$xoTheme->addStylesheet(PEDIGREE_URL.'/assets/css/style.css');
 
 //$xoTheme->addStylesheet(PEDIGREE_URL . '/assets/css/jquery.ThickBox/thickbox.css');
 //$xoTheme->addStylesheet(PEDIGREE_URL . '/module.css');
 
-$GLOBALS['xoopsTpl']->assign('pedigree_url', PEDIGREE_URL . '/');
+$GLOBALS['xoopsTpl']->assign('pedigree_url', PEDIGREE_URL.'/');
 
 // Breadcrumb
 $breadcrumb = new Pedigree\Breadcrumb();
@@ -58,11 +58,11 @@ 
     if ($userField->isActive() && $userField->hasSearch()) {
         $fieldType   = $userField->getSetting('FieldType');
         $fieldObject = new $fieldType($userField, $animal);
-        $function    = 'user' . $iValue . $fieldObject->getSearchString();
+        $function    = 'user'.$iValue.$fieldObject->getSearchString();
         //echo $function."<br>";
         $usersearch[] = [
             'title'       => $userField->getSetting('SearchName'),
-            'searchid'    => 'user' . $iValue,
+            'searchid'    => 'user'.$iValue,
             'function'    => $function,
             'explanation' => $userField->getSetting('SearchExplanation'),
             'searchfield' => $fieldObject->searchfield()
@@ -71,7 +71,7 @@ 
 }
 
 //$catarray['letters']          = Pedigree\Utility::lettersChoice();
-$letter              = '';
+$letter = '';
 $myObject     = Pedigree\Helper::getInstance();
 $activeObject = 'Tree';
 $name         = 'naam';
@@ -79,7 +79,7 @@ 
 $link2        = '%25';
 
 $criteria = $myObject->getHandler('Tree')->getActiveCriteria();
-$criteria->setGroupby('UPPER(LEFT(' . $name . ',1))');
+$criteria->setGroupby('UPPER(LEFT('.$name.',1))');
 $catarray['letters'] = Pedigree\Utility::lettersChoice($myObject, $activeObject, $criteria, $name, $link, $link2);
 //$catarray['toolbar']          = pedigree_toolbar();
 $xoopsTpl->assign('catarray', $catarray);

dog.php

Spacing +38 added lines, -38 removed lines

@@ -5,11 +5,11 @@ 
 use XoopsModules\Pedigree;
 
 //require_once  dirname(dirname(__DIR__)) . '/mainfile.php';
-require_once __DIR__ . '/header.php';
+require_once __DIR__.'/header.php';
 $moduleDirName = basename(__DIR__);
 xoops_loadLanguage('main', $moduleDirName);
 // Include any common code for this module.
-require_once XOOPS_ROOT_PATH . '/modules/' . $moduleDirName . '/include/common.php';
+require_once XOOPS_ROOT_PATH.'/modules/'.$moduleDirName.'/include/common.php';
 
 // Get all HTTP post or get parameters into global variables that are prefixed with "param_"
 //import_request_variables("gp", "param_");
@@ -17,7 +17,7 @@ 
 extract($_POST, EXTR_PREFIX_ALL, 'param');
 
 $GLOBALS['xoopsOption']['template_main'] = 'pedigree_dog.tpl';
-include XOOPS_ROOT_PATH . '/header.php';
+include XOOPS_ROOT_PATH.'/header.php';
 
 global $xoopsUser, $xoopsTpl, $xoopsDB, $xoopsModuleConfig, $xoopsModule;
 global $numofcolumns, $numMatch, $pages, $columns, $dogs;
@@ -43,78 +43,78 @@ 
 }
 
 if (isset($_GET['delpicture']) && 'true' === $_GET['delpicture']) {
-    $delpicsql = 'UPDATE ' . $GLOBALS['xoopsDB']->prefix('pedigree_tree') . " SET foto = '' WHERE id = '" . $id . "'";
+    $delpicsql = 'UPDATE '.$GLOBALS['xoopsDB']->prefix('pedigree_tree')." SET foto = '' WHERE id = '".$id."'";
     $GLOBALS['xoopsDB']->query($delpicsql);
 }
 //query
-$queryString = 'SELECT * FROM ' . $GLOBALS['xoopsDB']->prefix('pedigree_tree') . ' WHERE id=' . $id;
+$queryString = 'SELECT * FROM '.$GLOBALS['xoopsDB']->prefix('pedigree_tree').' WHERE id='.$id;
 $result      = $GLOBALS['xoopsDB']->query($queryString);
 
 while (false !== ($row = $GLOBALS['xoopsDB']->fetchArray($result))) {
     //name
     $naam = stripslashes($row['naam']);
-    $xoopsTpl->assign('xoops_pagetitle', $naam . ' -- detailed information');
+    $xoopsTpl->assign('xoops_pagetitle', $naam.' -- detailed information');
     //owner
     if ('0' != $row['id_owner']) {
-        $queryeig = 'SELECT id, lastname, firstname FROM ' . $GLOBALS['xoopsDB']->prefix('pedigree_owner') . ' WHERE id=' . $row['id_owner'];
+        $queryeig = 'SELECT id, lastname, firstname FROM '.$GLOBALS['xoopsDB']->prefix('pedigree_owner').' WHERE id='.$row['id_owner'];
         $reseig   = $GLOBALS['xoopsDB']->query($queryeig);
         while (false !== ($roweig = $GLOBALS['xoopsDB']->fetchArray($reseig))) {
-            $eig = '<a href="owner.php?ownid=' . $roweig['id'] . '">' . $roweig['firstname'] . ' ' . $roweig['lastname'] . '</a>';
+            $eig = '<a href="owner.php?ownid='.$roweig['id'].'">'.$roweig['firstname'].' '.$roweig['lastname'].'</a>';
         }
     } else {
-        $eig = '<a href="update.php?id=' . $row['id'] . '&fld=ow">' . _MA_PEDIGREE_UNKNOWN . '</a>';
+        $eig = '<a href="update.php?id='.$row['id'].'&fld=ow">'._MA_PEDIGREE_UNKNOWN.'</a>';
     }
     //breeder
     if ('0' != $row['id_breeder']) {
-        $queryfok = 'SELECT id, lastname, firstname FROM ' . $GLOBALS['xoopsDB']->prefix('pedigree_owner') . ' WHERE id=' . $row['id_breeder'];
+        $queryfok = 'SELECT id, lastname, firstname FROM '.$GLOBALS['xoopsDB']->prefix('pedigree_owner').' WHERE id='.$row['id_breeder'];
         $resfok   = $GLOBALS['xoopsDB']->query($queryfok);
         while (false !== ($rowfok = $GLOBALS['xoopsDB']->fetchArray($resfok))) {
-            $fok = '<a href="owner.php?ownid=' . $rowfok['id'] . '">' . $rowfok['firstname'] . ' ' . $rowfok['lastname'] . '</a>';
+            $fok = '<a href="owner.php?ownid='.$rowfok['id'].'">'.$rowfok['firstname'].' '.$rowfok['lastname'].'</a>';
         }
     } else {
-        $fok = '<a href="update.php?id=' . $row['id'] . '&fld=br">' . _MA_PEDIGREE_UNKNOWN . '</a>';
+        $fok = '<a href="update.php?id='.$row['id'].'&fld=br">'._MA_PEDIGREE_UNKNOWN.'</a>';
     }
     //gender
     if (0 == $row['roft']) {
-        $gender = '<img src="assets/images/male.gif"> ' . strtr(_MA_PEDIGREE_FLD_MALE, ['[male]' => $moduleConfig['male']]);
+        $gender = '<img src="assets/images/male.gif"> '.strtr(_MA_PEDIGREE_FLD_MALE, ['[male]' => $moduleConfig['male']]);
     } else {
-        $gender = '<img src="assets/images/female.gif"> ' . strtr(_MA_PEDIGREE_FLD_FEMA, ['[female]' => $moduleConfig['female']]);
+        $gender = '<img src="assets/images/female.gif"> '.strtr(_MA_PEDIGREE_FLD_FEMA, ['[female]' => $moduleConfig['female']]);
     }
     //Sire
     if (0 != $row['father']) {
-        $querysire = 'SELECT naam FROM ' . $GLOBALS['xoopsDB']->prefix('pedigree_tree') . ' WHERE id=' . $row['father'];
+        $querysire = 'SELECT naam FROM '.$GLOBALS['xoopsDB']->prefix('pedigree_tree').' WHERE id='.$row['father'];
         $ressire   = $GLOBALS['xoopsDB']->query($querysire);
         while (false !== ($rowsire = $GLOBALS['xoopsDB']->fetchArray($ressire))) {
-            $sire = '<img src="assets/images/male.gif"><a href="dog.php?id=' . $row['father'] . '">' . stripslashes($rowsire['naam']) . '</a>';
+            $sire = '<img src="assets/images/male.gif"><a href="dog.php?id='.$row['father'].'">'.stripslashes($rowsire['naam']).'</a>';
         }
     } else {
-        $sire = '<img src="assets/images/male.gif"><a href="seldog.php?curval=' . $row['id'] . '&gend=0&letter=A">' . _MA_PEDIGREE_UNKNOWN . '</a>';
+        $sire = '<img src="assets/images/male.gif"><a href="seldog.php?curval='.$row['id'].'&gend=0&letter=A">'._MA_PEDIGREE_UNKNOWN.'</a>';
     }
     //Dam
     if (0 != $row['mother']) {
-        $querydam = 'SELECT naam FROM ' . $GLOBALS['xoopsDB']->prefix('pedigree_tree') . ' WHERE id=' . $row['mother'];
+        $querydam = 'SELECT naam FROM '.$GLOBALS['xoopsDB']->prefix('pedigree_tree').' WHERE id='.$row['mother'];
         $resdam   = $GLOBALS['xoopsDB']->query($querydam);
         while (false !== ($rowdam = $GLOBALS['xoopsDB']->fetchArray($resdam))) {
-            $dam = '<img src="assets/images/female.gif"><a href="dog.php?id=' . $row['mother'] . '">' . stripslashes($rowdam['naam']) . '</a>';
+            $dam = '<img src="assets/images/female.gif"><a href="dog.php?id='.$row['mother'].'">'.stripslashes($rowdam['naam']).'</a>';
         }
     } else {
-        $dam = '<img src="assets/images/female.gif"><a href="seldog.php?curval=' . $row['id'] . '&gend=1&letter=A">' . _MA_PEDIGREE_UNKNOWN . '</a>';
+        $dam = '<img src="assets/images/female.gif"><a href="seldog.php?curval='.$row['id'].'&gend=1&letter=A">'._MA_PEDIGREE_UNKNOWN.'</a>';
     }
     //picture
     if ('' != $row['foto']) {
-        $picture = '<img src=' . PEDIGREE_UPLOAD_URL . '/images/thumbnails/' . $row['foto'] . '_400.jpeg>';
+        $picture = '<img src='.PEDIGREE_UPLOAD_URL.'/images/thumbnails/'.$row['foto'].'_400.jpeg>';
     } else {
-        $picture = '<a href="update.php?id=' . $row['id'] . '&fld=pc">' . _MA_PEDIGREE_UNKNOWN . '</a>';
+        $picture = '<a href="update.php?id='.$row['id'].'&fld=pc">'._MA_PEDIGREE_UNKNOWN.'</a>';
     }
     //inbred precentage
     if ('' == $row['coi']) {
         if (0 != $row['father'] && 0 != $row['mother']) {
-            $inbred = '<a href="coi.php?s=' . $row['father'] . '&d=' . $row['mother'] . '&dogid=' . $row['id'] . '&detail=1">' . strtr(_MA_PEDIGREE_COI_WAIT, ['[animalType]' => $moduleConfig['animalType']]) . '</a>';
+            $inbred = '<a href="coi.php?s='.$row['father'].'&d='.$row['mother'].'&dogid='.$row['id'].'&detail=1">'.strtr(_MA_PEDIGREE_COI_WAIT, ['[animalType]' => $moduleConfig['animalType']]).'</a>';
         } else {
             $inbred = _MA_PEDIGREE_COI_MORE;
         }
     } else {
-        $inbred = '<a href="coi.php?s=' . $row['father'] . '&d=' . $row['mother'] . '&dogid=' . $row['id'] . '&detail=1" title="' . strtr(_MA_PEDIGREE_COI_WAIT, ['[animalType]' => $moduleConfig['animalType']]) . '">' . $row['coi'] . ' %</a>';
+        $inbred = '<a href="coi.php?s='.$row['father'].'&d='.$row['mother'].'&dogid='.$row['id'].'&detail=1" title="'.strtr(_MA_PEDIGREE_COI_WAIT, ['[animalType]' => $moduleConfig['animalType']]).'">'.$row['coi'].' %</a>';
     }
     //brothers and sisters
     $bas = Pedigree\Utility::bas($id, $row['father'], $row['mother']);
@@ -137,46 +137,46 @@ 
     //name
     $items[] = [
         'header' => _MA_PEDIGREE_FLD_NAME,
-        'data'   => '<a href="pedigree.php?pedid=' . $row['id'] . '">' . $naam . '</a> (click to view pedigree)',
-        'edit'   => '<a href="update.php?id=' . $row['id'] . "&fld=nm\"><img src=' " . $pathIcon16 . "/edit.png' border='0' alt=_EDIT title=_EDIT></a>"
+        'data'   => '<a href="pedigree.php?pedid='.$row['id'].'">'.$naam.'</a> (click to view pedigree)',
+        'edit'   => '<a href="update.php?id='.$row['id']."&fld=nm\"><img src=' ".$pathIcon16."/edit.png' border='0' alt=_EDIT title=_EDIT></a>"
     ];
     if ('1' == $moduleConfig['ownerbreeder']) {
         //owner
         $items[] = [
             'header' => _MA_PEDIGREE_FLD_OWNE,
             'data'   => $eig,
-            'edit'   => '<a href="update.php?id=' . $row['id'] . "&fld=ow\"><img src=' " . $pathIcon16 . "/edit.png' border='0' alt=_EDIT title=_EDIT></a>"
+            'edit'   => '<a href="update.php?id='.$row['id']."&fld=ow\"><img src=' ".$pathIcon16."/edit.png' border='0' alt=_EDIT title=_EDIT></a>"
         ];
         //breeder
         $items[] = [
             'header' => _MA_PEDIGREE_FLD_BREE,
             'data'   => $fok,
-            'edit'   => '<a href="update.php?id=' . $row['id'] . "&fld=br\"><img src=' " . $pathIcon16 . "/edit.png' border='0' alt=_EDIT title=_EDIT></a>"
+            'edit'   => '<a href="update.php?id='.$row['id']."&fld=br\"><img src=' ".$pathIcon16."/edit.png' border='0' alt=_EDIT title=_EDIT></a>"
         ];
     }
     //gender
     $items[] = [
         'header' => _MA_PEDIGREE_FLD_GEND,
         'data'   => $gender,
-        'edit'   => '<a href="update.php?id=' . $row['id'] . "&fld=sx\"><img src=' " . $pathIcon16 . "/edit.png' border='0' alt=_EDIT title=_EDIT></a>"
+        'edit'   => '<a href="update.php?id='.$row['id']."&fld=sx\"><img src=' ".$pathIcon16."/edit.png' border='0' alt=_EDIT title=_EDIT></a>"
     ];
     //sire
     $items[] = [
         'header' => strtr(_MA_PEDIGREE_FLD_FATH, ['[father]' => $moduleConfig['father']]),
         'data'   => $sire,
-        'edit'   => '<a href="seldog.php?curval=' . $row['id'] . "&gend=0&letter=A\"><img src=' " . $pathIcon16 . "/edit.png' border='0' alt=_EDIT title=_EDIT></a>"
+        'edit'   => '<a href="seldog.php?curval='.$row['id']."&gend=0&letter=A\"><img src=' ".$pathIcon16."/edit.png' border='0' alt=_EDIT title=_EDIT></a>"
     ];
     //dam
     $items[] = [
         'header' => strtr(_MA_PEDIGREE_FLD_MOTH, ['[mother]' => $moduleConfig['mother']]),
         'data'   => $dam,
-        'edit'   => '<a href="seldog.php?curval=' . $row['id'] . "&gend=1&letter=A\"><img src=' " . $pathIcon16 . "/edit.png' border='0' alt=_EDIT title=_EDIT></a>"
+        'edit'   => '<a href="seldog.php?curval='.$row['id']."&gend=1&letter=A\"><img src=' ".$pathIcon16."/edit.png' border='0' alt=_EDIT title=_EDIT></a>"
     ];
     //picture
     $items[] = [
         'header' => _MA_PEDIGREE_FLD_PICT,
         'data'   => $picture,
-        'edit'   => '<a href="update.php?id=' . $row['id'] . "&fld=pc\"><img src=' " . $pathIcon16 . "/edit.png' border='0' alt=_EDIT title=_EDIT></a><a href=\"dog.php?id=" . $row['id'] . "&delpicture=true\"><img src=' " . $pathIcon16 . "/delete.png' border='0' alt=_DELETE title=_DELETE></a>"
+        'edit'   => '<a href="update.php?id='.$row['id']."&fld=pc\"><img src=' ".$pathIcon16."/edit.png' border='0' alt=_EDIT title=_EDIT></a><a href=\"dog.php?id=".$row['id']."&delpicture=true\"><img src=' ".$pathIcon16."/delete.png' border='0' alt=_DELETE title=_DELETE></a>"
     ];
 
     //userdefined fields
@@ -202,7 +202,7 @@ 
                 $items[] = [
                     'header' => $userField->getSetting('FieldName'),
                     'data'   => $fieldObject->showValue(),
-                    'edit'   => '<a href="update.php?id=' . $row['id'] . '&fld=' . $iValue . "\"><img src=' " . $pathIcon16 . "/edit.png' border='0' alt=_EDIT title=_EDIT></a>"
+                    'edit'   => '<a href="update.php?id='.$row['id'].'&fld='.$iValue."\"><img src=' ".$pathIcon16."/edit.png' border='0' alt=_EDIT title=_EDIT></a>"
                 ];
             }
         }
@@ -255,7 +255,7 @@ 
     if ('1' == $moduleConfig['proversion']) {
         $items[] = [
             'header' => 'Inbred Pedigree',
-            'data'   => '<a href="mpedigree.php?pedid=' . $row['id'] . '">Inbreeding pedigree</a>',
+            'data'   => '<a href="mpedigree.php?pedid='.$row['id'].'">Inbreeding pedigree</a>',
             'edit'   => ''
         ];
     }
@@ -268,13 +268,13 @@ 
 $xoopsTpl->assign('dogs', $dogs);
 $xoopsTpl->assign('columns', $columns);
 $xoopsTpl->assign('numofcolumns', $numofcolumns);
-$xoopsTpl->assign('nummatch', $numMatch . ' Animals found.');
+$xoopsTpl->assign('nummatch', $numMatch.' Animals found.');
 
 //bas
 $xoopsTpl->assign('dogs1', $dogs1);
 $xoopsTpl->assign('columns1', $columns1);
 $xoopsTpl->assign('numofcolumns1', $numofcolumns1);
-$xoopsTpl->assign('nummatch1', $nummatch1 . ' Animals found.');
+$xoopsTpl->assign('nummatch1', $nummatch1.' Animals found.');
 
 //both pups and bas
 $xoopsTpl->assign('width', 100 / $numofcolumns);
@@ -293,5 +293,5 @@ 
 $xoopsTpl->assign('delete', _DELETE);
 
 //comments and footer
-include XOOPS_ROOT_PATH . '/include/comment_view.php';
-include XOOPS_ROOT_PATH . '/footer.php';
+include XOOPS_ROOT_PATH.'/include/comment_view.php';
+include XOOPS_ROOT_PATH.'/footer.php';

phpthumb/phpThumb.php

Indentation +2 added lines, -2 removed lines

@@ -263,7 +263,7 @@ 
             $phpThumb->ErrorImage('ERROR: "' . $SourceFilename . '" cannot be read');
         }
     }
-   if (\Xmf\Request::hasVar('HTTP_REFERER', 'SERVER')) {
+    if (\Xmf\Request::hasVar('HTTP_REFERER', 'SERVER')) {
         $phpThumb->ErrorImage('&md5s=' . $md5s);
     } else {
         die('&md5s=' . $md5s);
@@ -584,7 +584,7 @@ 
                 $phpThumb->ErrorImage('Headers already sent (' . basename(__FILE__) . ' line ' . __LINE__ . ')');
                 exit;
             }
-           if (\Xmf\Request::hasVar('phpThumbDebug', 'GET')) {
+            if (\Xmf\Request::hasVar('phpThumbDebug', 'GET')) {
                 $phpThumb->DebugTimingMessage('skipped direct $SourceFilename passthru', __FILE__, __LINE__);
                 $phpThumb->DebugMessage('Would have passed "' . $SourceFilename . '" through directly, but skipping due to phpThumbDebug', __FILE__, __LINE__);
                 break;

Spacing +84 added lines, -84 removed lines

@@ -33,10 +33,10 @@ 
         return false;
     }
     global $phpThumb;
-    $downloadfilename = phpthumb_functions::SanitizeFilename(!empty($_GET['sia']) ? $_GET['sia'] : (!empty($_GET['down']) ? $_GET['down'] : 'phpThumb_generated_thumbnail.' . (!empty($_GET['f']) ? $_GET['f'] : 'jpg')));
+    $downloadfilename = phpthumb_functions::SanitizeFilename(!empty($_GET['sia']) ? $_GET['sia'] : (!empty($_GET['down']) ? $_GET['down'] : 'phpThumb_generated_thumbnail.'.(!empty($_GET['f']) ? $_GET['f'] : 'jpg')));
     if (!empty($downloadfilename)) {
-        $phpThumb->DebugMessage('SendSaveAsFileHeaderIfNeeded() sending header: Content-Disposition: ' . (!empty($_GET['down']) ? 'attachment' : 'inline') . '; filename="' . $downloadfilename . '"', __FILE__, __LINE__);
-        header('Content-Disposition: ' . (!empty($_GET['down']) ? 'attachment' : 'inline') . '; filename="' . $downloadfilename . '"');
+        $phpThumb->DebugMessage('SendSaveAsFileHeaderIfNeeded() sending header: Content-Disposition: '.(!empty($_GET['down']) ? 'attachment' : 'inline').'; filename="'.$downloadfilename.'"', __FILE__, __LINE__);
+        header('Content-Disposition: '.(!empty($_GET['down']) ? 'attachment' : 'inline').'; filename="'.$downloadfilename.'"');
     }
 
     return true;
@@ -72,43 +72,43 @@ 
     } elseif ($phpThumb->phpThumbDebug) {
         $phpThumb->DebugTimingMessage('skipped using cached image', __FILE__, __LINE__);
         $phpThumb->DebugMessage('Would have used cached file, but skipping due to phpThumbDebug', __FILE__, __LINE__);
-        $phpThumb->DebugMessage('* Would have sent headers (1): Last-Modified: ' . gmdate('D, d M Y H:i:s', $nModified) . ' GMT', __FILE__, __LINE__);
+        $phpThumb->DebugMessage('* Would have sent headers (1): Last-Modified: '.gmdate('D, d M Y H:i:s', $nModified).' GMT', __FILE__, __LINE__);
         if ($getimagesize = @getimagesize($phpThumb->cache_filename)) {
-            $phpThumb->DebugMessage('* Would have sent headers (2): Content-Type: ' . phpthumb_functions::ImageTypeToMIMEtype($getimagesize[2]), __FILE__, __LINE__);
+            $phpThumb->DebugMessage('* Would have sent headers (2): Content-Type: '.phpthumb_functions::ImageTypeToMIMEtype($getimagesize[2]), __FILE__, __LINE__);
         }
-        if (preg_match('#^' . preg_quote($nice_docroot) . '(.*)$#', $nice_cachefile, $matches)) {
-            $phpThumb->DebugMessage('* Would have sent headers (3): Location: ' . dirname($matches[1]) . '/' . urlencode(basename($matches[1])), __FILE__, __LINE__);
+        if (preg_match('#^'.preg_quote($nice_docroot).'(.*)$#', $nice_cachefile, $matches)) {
+            $phpThumb->DebugMessage('* Would have sent headers (3): Location: '.dirname($matches[1]).'/'.urlencode(basename($matches[1])), __FILE__, __LINE__);
         } else {
-            $phpThumb->DebugMessage('* Would have sent data: readfile(' . $phpThumb->cache_filename . ')', __FILE__, __LINE__);
+            $phpThumb->DebugMessage('* Would have sent data: readfile('.$phpThumb->cache_filename.')', __FILE__, __LINE__);
         }
     } else {
         if (headers_sent()) {
-            $phpThumb->ErrorImage('Headers already sent (' . basename(__FILE__) . ' line ' . __LINE__ . ')');
+            $phpThumb->ErrorImage('Headers already sent ('.basename(__FILE__).' line '.__LINE__.')');
             exit;
         }
         SendSaveAsFileHeaderIfNeeded();
 
         header('Cache-Control: private');
         header('Pragma: private');
-        header('Cache-Control: max-age=' . $phpThumb->getParameter('config_cache_maxage'));
-        header('Expires: ' . date(DATE_RFC1123, time() + $phpThumb->getParameter('config_cache_maxage')));
+        header('Cache-Control: max-age='.$phpThumb->getParameter('config_cache_maxage'));
+        header('Expires: '.date(DATE_RFC1123, time() + $phpThumb->getParameter('config_cache_maxage')));
         if (!empty($_SERVER['HTTP_IF_MODIFIED_SINCE']) && ($nModified == strtotime($_SERVER['HTTP_IF_MODIFIED_SINCE']))
             && !empty($_SERVER['SERVER_PROTOCOL'])) {
-            header('Last-Modified: ' . gmdate('D, d M Y H:i:s', $nModified) . ' GMT');
-            header($_SERVER['SERVER_PROTOCOL'] . ' 304 Not Modified');
+            header('Last-Modified: '.gmdate('D, d M Y H:i:s', $nModified).' GMT');
+            header($_SERVER['SERVER_PROTOCOL'].' 304 Not Modified');
             exit;
         }
-        header('Last-Modified: ' . gmdate('D, d M Y H:i:s', $nModified) . ' GMT');
-        header('ETag: "' . md5_file($phpThumb->cache_filename) . '"');
+        header('Last-Modified: '.gmdate('D, d M Y H:i:s', $nModified).' GMT');
+        header('ETag: "'.md5_file($phpThumb->cache_filename).'"');
         if ($getimagesize = @getimagesize($phpThumb->cache_filename)) {
-            header('Content-Type: ' . phpthumb_functions::ImageTypeToMIMEtype($getimagesize[2]));
+            header('Content-Type: '.phpthumb_functions::ImageTypeToMIMEtype($getimagesize[2]));
         } elseif (preg_match('#\\.ico$#i', $phpThumb->cache_filename)) {
             header('Content-Type: image/x-icon');
         }
-        header('Content-Length: ' . filesize($phpThumb->cache_filename));
+        header('Content-Length: '.filesize($phpThumb->cache_filename));
         if (empty($phpThumb->config_cache_force_passthru)
-            && preg_match('#^' . preg_quote($nice_docroot) . '(.*)$#', $nice_cachefile, $matches)) {
-            header('Location: ' . dirname($matches[1]) . '/' . urlencode(basename($matches[1])));
+            && preg_match('#^'.preg_quote($nice_docroot).'(.*)$#', $nice_cachefile, $matches)) {
+            header('Location: '.dirname($matches[1]).'/'.urlencode(basename($matches[1])));
         } else {
             @readfile($phpThumb->cache_filename);
         }
@@ -120,9 +120,9 @@ 
 
 // instantiate a new phpThumb() object
 ob_start();
-if (!require_once __DIR__ . '/phpthumb.class.php') {
+if (!require_once __DIR__.'/phpthumb.class.php') {
     ob_end_flush();
-    die('failed to include_once("' . realpath(__DIR__ . '/phpthumb.class.php') . '")');
+    die('failed to include_once("'.realpath(__DIR__.'/phpthumb.class.php').'")');
 }
 ob_end_clean();
 $phpThumb = new phpThumb();
@@ -130,36 +130,36 @@ 
 $phpThumb->setParameter('config_error_die_on_error', true);
 
 if (!phpthumb_functions::FunctionIsDisabled('set_time_limit')) {
-    set_time_limit(60);  // shouldn't take nearly this long in most cases, but with many filters and/or a slow server...
+    set_time_limit(60); // shouldn't take nearly this long in most cases, but with many filters and/or a slow server...
 }
 
 // phpThumbDebug[0] used to be here, but may reveal too much
 // info when high_security_mode should be enabled (not set yet)
 
-if (file_exists(__DIR__ . '/phpThumb.config.php')) {
+if (file_exists(__DIR__.'/phpThumb.config.php')) {
     ob_start();
-    if (require_once __DIR__ . '/phpThumb.config.php') {
+    if (require_once __DIR__.'/phpThumb.config.php') {
         // great
     } else {
         ob_end_flush();
         $phpThumb->config_disable_debug = false; // otherwise error message won't print
-        $phpThumb->ErrorImage('failed to include_once(' . __DIR__ . '/phpThumb.config.php) - realpath="' . realpath(__DIR__ . '/phpThumb.config.php') . '"');
+        $phpThumb->ErrorImage('failed to include_once('.__DIR__.'/phpThumb.config.php) - realpath="'.realpath(__DIR__.'/phpThumb.config.php').'"');
     }
     ob_end_clean();
-} elseif (file_exists(__DIR__ . '/phpThumb.config.php.default')) {
+} elseif (file_exists(__DIR__.'/phpThumb.config.php.default')) {
     $phpThumb->config_disable_debug = false; // otherwise error message won't print
     $phpThumb->ErrorImage('Please rename "phpThumb.config.php.default" to "phpThumb.config.php"');
 } else {
     $phpThumb->config_disable_debug = false; // otherwise error message won't print
-    $phpThumb->ErrorImage('failed to include_once(' . __DIR__ . '/phpThumb.config.php) - realpath="' . realpath(__DIR__ . '/phpThumb.config.php') . '"');
+    $phpThumb->ErrorImage('failed to include_once('.__DIR__.'/phpThumb.config.php) - realpath="'.realpath(__DIR__.'/phpThumb.config.php').'"');
 }
 
 if (!empty($PHPTHUMB_CONFIG)) {
     foreach ($PHPTHUMB_CONFIG as $key => $value) {
-        $keyname = 'config_' . $key;
+        $keyname = 'config_'.$key;
         $phpThumb->setParameter($keyname, $value);
         if (!preg_match('#(password|mysql)#i', $key)) {
-            $phpThumb->DebugMessage('setParameter(' . $keyname . ', ' . $phpThumb->phpThumbDebugVarDump($value) . ')', __FILE__, __LINE__);
+            $phpThumb->DebugMessage('setParameter('.$keyname.', '.$phpThumb->phpThumbDebugVarDump($value).')', __FILE__, __LINE__);
         }
     }
     if (!$phpThumb->config_disable_debug) {
@@ -176,10 +176,10 @@ 
     $_SERVER['PHP_SELF'] = str_replace($_SERVER['PATH_INFO'], '', @$_SERVER['PHP_SELF']);
 
     $args = explode(';', substr($_SERVER['PATH_INFO'], 1));
-    $phpThumb->DebugMessage('PATH_INFO.$args set to (' . implode(')(', $args) . ')', __FILE__, __LINE__);
+    $phpThumb->DebugMessage('PATH_INFO.$args set to ('.implode(')(', $args).')', __FILE__, __LINE__);
     if (!empty($args)) {
         $_GET['src'] = @$args[count($args) - 1];
-        $phpThumb->DebugMessage('PATH_INFO."src" = "' . $_GET['src'] . '"', __FILE__, __LINE__);
+        $phpThumb->DebugMessage('PATH_INFO."src" = "'.$_GET['src'].'"', __FILE__, __LINE__);
         if (preg_match('#^new\=([a-z0-9]+)#i', $_GET['src'], $matches)) {
             unset($_GET['src']);
             $_GET['new'] = $matches[1];
@@ -188,17 +188,17 @@ 
     if (preg_match('#^([0-9]*)x?([0-9]*)$#i', @$args[count($args) - 2], $matches)) {
         $_GET['w'] = $matches[1];
         $_GET['h'] = $matches[2];
-        $phpThumb->DebugMessage('PATH_INFO."w"x"h" set to "' . $_GET['w'] . '"x"' . $_GET['h'] . '"', __FILE__, __LINE__);
+        $phpThumb->DebugMessage('PATH_INFO."w"x"h" set to "'.$_GET['w'].'"x"'.$_GET['h'].'"', __FILE__, __LINE__);
     }
     for ($i = 0; $i < count($args) - 2; $i++) {
         @list($key, $value) = explode('=', @$args[$i]);
         if ('[]' === substr($key, -2)) {
             $array_key_name          = substr($key, 0, -2);
             $_GET[$array_key_name][] = $value;
-            $phpThumb->DebugMessage('PATH_INFO."' . $array_key_name . '[]" = "' . $value . '"', __FILE__, __LINE__);
+            $phpThumb->DebugMessage('PATH_INFO."'.$array_key_name.'[]" = "'.$value.'"', __FILE__, __LINE__);
         } else {
             $_GET[$key] = $value;
-            $phpThumb->DebugMessage('PATH_INFO."' . $key . '" = "' . $value . '"', __FILE__, __LINE__);
+            $phpThumb->DebugMessage('PATH_INFO."'.$key.'" = "'.$value.'"', __FILE__, __LINE__);
         }
     }
 }
@@ -210,7 +210,7 @@ 
     } elseif (phpthumb_functions::PasswordStrength($phpThumb->config_high_security_password) < 20) {
         $phpThumb->config_disable_debug = false; // otherwise error message won't print
         $phpThumb->ErrorImage('ERROR: $PHPTHUMB_CONFIG[high_security_password] is not complex enough');
-    } elseif ($_GET['hash'] != md5(str_replace($phpThumb->config_high_security_url_separator . 'hash=' . $_GET['hash'], '', $_SERVER['QUERY_STRING']) . $phpThumb->config_high_security_password)) {
+    } elseif ($_GET['hash'] != md5(str_replace($phpThumb->config_high_security_url_separator.'hash='.$_GET['hash'], '', $_SERVER['QUERY_STRING']).$phpThumb->config_high_security_password)) {
         header('HTTP/1.0 403 Forbidden');
         sleep(10); // deliberate delay to discourage password-guessing
         $phpThumb->ErrorImage('ERROR: invalid hash');
@@ -253,26 +253,26 @@ 
                 $md5s = md5($rawImageData);
             }
         } else {
-            $phpThumb->ErrorImage('only FTP and HTTP/HTTPS protocols are allowed, "' . $protocol_matches[1] . '" is not');
+            $phpThumb->ErrorImage('only FTP and HTTP/HTTPS protocols are allowed, "'.$protocol_matches[1].'" is not');
         }
     } else {
         $SourceFilename = $phpThumb->ResolveFilenameToAbsolute($_GET['src']);
         if (is_readable($SourceFilename)) {
             $md5s = phpthumb_functions::md5_file_safe($SourceFilename);
         } else {
-            $phpThumb->ErrorImage('ERROR: "' . $SourceFilename . '" cannot be read');
+            $phpThumb->ErrorImage('ERROR: "'.$SourceFilename.'" cannot be read');
         }
     }
    if (\Xmf\Request::hasVar('HTTP_REFERER', 'SERVER')) {
-        $phpThumb->ErrorImage('&md5s=' . $md5s);
+        $phpThumb->ErrorImage('&md5s='.$md5s);
     } else {
-        die('&md5s=' . $md5s);
+        die('&md5s='.$md5s);
     }
 }
 
 if (!empty($_GET['src']) && empty($phpThumb->config_allow_local_http_src)
-    && preg_match('#^http://' . @$_SERVER['HTTP_HOST'] . '(.+)#i', $_GET['src'], $matches)) {
-    $phpThumb->ErrorImage('It is MUCH better to specify the "src" parameter as "' . $matches[1] . '" instead of "' . $matches[0] . '".' . "\n\n" . 'If you really must do it this way, enable "allow_local_http_src" in phpThumb.config.php');
+    && preg_match('#^http://'.@$_SERVER['HTTP_HOST'].'(.+)#i', $_GET['src'], $matches)) {
+    $phpThumb->ErrorImage('It is MUCH better to specify the "src" parameter as "'.$matches[1].'" instead of "'.$matches[0].'".'."\n\n".'If you really must do it this way, enable "allow_local_http_src" in phpThumb.config.php');
 }
 
 ////////////////////////////////////////////////////////////////
@@ -286,7 +286,7 @@ 
 $parsed_url_referer = phpthumb_functions::ParseURLbetter(@$_SERVER['HTTP_REFERER']);
 if ($phpThumb->config_nooffsitelink_require_refer
     && !in_array(@$parsed_url_referer['host'], $phpThumb->config_nohotlink_valid_domains)) {
-    $phpThumb->ErrorImage('config_nooffsitelink_require_refer enabled and ' . (@$parsed_url_referer['host'] ? '"' . $parsed_url_referer['host'] . '" is not an allowed referer' : 'no HTTP_REFERER exists'));
+    $phpThumb->ErrorImage('config_nooffsitelink_require_refer enabled and '.(@$parsed_url_referer['host'] ? '"'.$parsed_url_referer['host'].'" is not an allowed referer' : 'no HTTP_REFERER exists'));
 }
 $parsed_url_src = phpthumb_functions::ParseURLbetter(@$_GET['src']);
 if ($phpThumb->config_nohotlink_enabled && $phpThumb->config_nohotlink_erase_image
@@ -305,11 +305,11 @@ 
             }
         }
         if ($found_missing_function) {
-            $phpThumb->ErrorImage('SQL function unavailable: ' . $found_missing_function);
+            $phpThumb->ErrorImage('SQL function unavailable: '.$found_missing_function);
         } else {
             $mysqli = new mysqli($phpThumb->config_mysql_hostname, $phpThumb->config_mysql_username, $phpThumb->config_mysql_password, $phpThumb->config_mysql_database);
             if ($mysqli->connect_error) {
-                $phpThumb->ErrorImage('MySQLi connect error (' . $mysqli->connect_errno . ') ' . $mysqli->connect_error);
+                $phpThumb->ErrorImage('MySQLi connect error ('.$mysqli->connect_errno.') '.$mysqli->connect_error);
             } else {
                 if ($result = $mysqli->query($phpThumb->config_mysql_query)) {
                     if ($row = $result->fetch_array()) {
@@ -324,7 +324,7 @@ 
                     }
                 } else {
                     $mysqli->close();
-                    $phpThumb->ErrorImage('Error in MySQL query: "' . $mysqli->error . '"');
+                    $phpThumb->ErrorImage('Error in MySQL query: "'.$mysqli->error.'"');
                 }
             }
             unset($_GET['id']);
@@ -339,7 +339,7 @@ 
             }
         }
         if ($found_missing_function) {
-            $phpThumb->ErrorImage('SQL function unavailable: ' . $found_missing_function);
+            $phpThumb->ErrorImage('SQL function unavailable: '.$found_missing_function);
         } else {
             if ($cid = @mysql_connect($phpThumb->config_mysql_hostname, $phpThumb->config_mysql_username, $phpThumb->config_mysql_password)) {
                 if (@mysqli_select_db($GLOBALS['xoopsDB']->conn, $phpThumb->config_mysql_database, $cid)) {
@@ -356,11 +356,11 @@ 
                         }
                     } else {
                         $GLOBALS['xoopsDB']->close($cid);
-                        $phpThumb->ErrorImage('Error in MySQL query: "' . $GLOBALS['xoopsDB']->error($cid) . '"');
+                        $phpThumb->ErrorImage('Error in MySQL query: "'.$GLOBALS['xoopsDB']->error($cid).'"');
                     }
                 } else {
                     $GLOBALS['xoopsDB']->close($cid);
-                    $phpThumb->ErrorImage('cannot select MySQL database: "' . $GLOBALS['xoopsDB']->error($cid) . '"');
+                    $phpThumb->ErrorImage('cannot select MySQL database: "'.$GLOBALS['xoopsDB']->error($cid).'"');
                 }
             } else {
                 $phpThumb->ErrorImage('cannot connect to MySQL server');
@@ -425,24 +425,24 @@ 
 foreach ($_GET as $key => $value) {
     if (!empty($PHPTHUMB_DEFAULTS_DISABLEGETPARAMS) && ('src' !== $key)) {
         // disabled, do not set parameter
-        $phpThumb->DebugMessage('ignoring $_GET[' . $key . '] because of $PHPTHUMB_DEFAULTS_DISABLEGETPARAMS', __FILE__, __LINE__);
+        $phpThumb->DebugMessage('ignoring $_GET['.$key.'] because of $PHPTHUMB_DEFAULTS_DISABLEGETPARAMS', __FILE__, __LINE__);
     } elseif (in_array($key, $allowedGETparameters)) {
-        $phpThumb->DebugMessage('setParameter(' . $key . ', ' . $phpThumb->phpThumbDebugVarDump($value) . ')', __FILE__, __LINE__);
+        $phpThumb->DebugMessage('setParameter('.$key.', '.$phpThumb->phpThumbDebugVarDump($value).')', __FILE__, __LINE__);
         $phpThumb->setParameter($key, $value);
     } else {
-        $phpThumb->ErrorImage('Forbidden parameter: ' . $key);
+        $phpThumb->ErrorImage('Forbidden parameter: '.$key);
     }
 }
 
 if (!empty($PHPTHUMB_DEFAULTS) && is_array($PHPTHUMB_DEFAULTS)) {
-    $phpThumb->DebugMessage('setting $PHPTHUMB_DEFAULTS[' . implode(';', array_keys($PHPTHUMB_DEFAULTS)) . ']', __FILE__, __LINE__);
+    $phpThumb->DebugMessage('setting $PHPTHUMB_DEFAULTS['.implode(';', array_keys($PHPTHUMB_DEFAULTS)).']', __FILE__, __LINE__);
     foreach ($PHPTHUMB_DEFAULTS as $key => $value) {
         if (!$PHPTHUMB_DEFAULTS_GETSTRINGOVERRIDE
             || !isset($_GET[$key])) { // set parameter to default value if config is set to allow _GET to override default, OR if no value is passed via _GET for this parameter
             //$_GET[$key] = $value;
             //$phpThumb->DebugMessage('PHPTHUMB_DEFAULTS assigning ('.(is_array($value) ? print_r($value, true) : $value).') to $_GET['.$key.']', __FILE__, __LINE__);
             $phpThumb->setParameter($key, $value);
-            $phpThumb->DebugMessage('setParameter(' . $key . ', ' . $phpThumb->phpThumbDebugVarDump($value) . ') from $PHPTHUMB_DEFAULTS', __FILE__, __LINE__);
+            $phpThumb->DebugMessage('setParameter('.$key.', '.$phpThumb->phpThumbDebugVarDump($value).') from $PHPTHUMB_DEFAULTS', __FILE__, __LINE__);
         }
     }
 }
@@ -474,13 +474,13 @@ 
 } elseif (preg_match('#^http\://[^\\?&]+\\.(jpe?g|gif|png)$#i', $phpThumb->src)) {
     // assume is ok to passthru if no other parameters specified
 } elseif (preg_match('#^(f|ht)tp\://#i', $phpThumb->src)) {
-    $phpThumb->DebugMessage('$CanPassThroughDirectly=false because preg_match("#^(f|ht)tp\://#i", ' . $phpThumb->src . ')', __FILE__, __LINE__);
+    $phpThumb->DebugMessage('$CanPassThroughDirectly=false because preg_match("#^(f|ht)tp\://#i", '.$phpThumb->src.')', __FILE__, __LINE__);
     $CanPassThroughDirectly = false;
 } elseif (!@is_readable($phpThumb->sourceFilename)) {
-    $phpThumb->DebugMessage('$CanPassThroughDirectly=false because !@is_readable(' . $phpThumb->sourceFilename . ')', __FILE__, __LINE__);
+    $phpThumb->DebugMessage('$CanPassThroughDirectly=false because !@is_readable('.$phpThumb->sourceFilename.')', __FILE__, __LINE__);
     $CanPassThroughDirectly = false;
 } elseif (!@is_file($phpThumb->sourceFilename)) {
-    $phpThumb->DebugMessage('$CanPassThroughDirectly=false because !@is_file(' . $phpThumb->sourceFilename . ')', __FILE__, __LINE__);
+    $phpThumb->DebugMessage('$CanPassThroughDirectly=false because !@is_file('.$phpThumb->sourceFilename.')', __FILE__, __LINE__);
     $CanPassThroughDirectly = false;
 }
 foreach ($_GET as $key => $value) {
@@ -511,7 +511,7 @@ 
     }
 }
 if (!empty($UnAllowedGET)) {
-    $phpThumb->DebugMessage('$CanPassThroughDirectly=false because $_GET[' . implode(';', array_unique($UnAllowedGET)) . '] are set', __FILE__, __LINE__);
+    $phpThumb->DebugMessage('$CanPassThroughDirectly=false because $_GET['.implode(';', array_unique($UnAllowedGET)).'] are set', __FILE__, __LINE__);
 }
 
 ////////////////////////////////////////////////////////////////
@@ -522,13 +522,13 @@ 
 }
 ////////////////////////////////////////////////////////////////
 
-$phpThumb->DebugMessage('$CanPassThroughDirectly="' . (int)$CanPassThroughDirectly . '" && $phpThumb->src="' . $phpThumb->src . '"', __FILE__, __LINE__);
+$phpThumb->DebugMessage('$CanPassThroughDirectly="'.(int)$CanPassThroughDirectly.'" && $phpThumb->src="'.$phpThumb->src.'"', __FILE__, __LINE__);
 while ($CanPassThroughDirectly && $phpThumb->src) {
     // no parameters set, passthru
 
     if (preg_match('#^http\://[^\\?&]+\.(jpe?g|gif|png)$#i', $phpThumb->src)) {
-        $phpThumb->DebugMessage('Passing HTTP source through directly as Location: redirect (' . $phpThumb->src . ')', __FILE__, __LINE__);
-        header('Location: ' . $phpThumb->src);
+        $phpThumb->DebugMessage('Passing HTTP source through directly as Location: redirect ('.$phpThumb->src.')', __FILE__, __LINE__);
+        header('Location: '.$phpThumb->src);
         exit;
     }
 
@@ -536,20 +536,20 @@ 
 
     // security and size checks
     if ($phpThumb->getimagesizeinfo = @getimagesize($SourceFilename)) {
-        $phpThumb->DebugMessage('Direct passthru getimagesize() returned [w=' . $phpThumb->getimagesizeinfo[0] . ';h=' . $phpThumb->getimagesizeinfo[1] . ';t=' . $phpThumb->getimagesizeinfo[2] . ']', __FILE__, __LINE__);
+        $phpThumb->DebugMessage('Direct passthru getimagesize() returned [w='.$phpThumb->getimagesizeinfo[0].';h='.$phpThumb->getimagesizeinfo[1].';t='.$phpThumb->getimagesizeinfo[2].']', __FILE__, __LINE__);
 
         if (!@$_GET['w'] && !@$_GET['wp'] && !@$_GET['wl'] && !@$_GET['ws'] && !@$_GET['h'] && !@$_GET['hp']
             && !@$_GET['hl']
             && !@$_GET['hs']) {
             // no resizing needed
-            $phpThumb->DebugMessage('Passing "' . $SourceFilename . '" through directly, no resizing required ("' . $phpThumb->getimagesizeinfo[0] . '"x"' . $phpThumb->getimagesizeinfo[1] . '")', __FILE__, __LINE__);
+            $phpThumb->DebugMessage('Passing "'.$SourceFilename.'" through directly, no resizing required ("'.$phpThumb->getimagesizeinfo[0].'"x"'.$phpThumb->getimagesizeinfo[1].'")', __FILE__, __LINE__);
         } elseif (($phpThumb->getimagesizeinfo[0] <= @$_GET['w']) && ($phpThumb->getimagesizeinfo[1] <= @$_GET['h'])
                   && ((@$_GET['w'] == $phpThumb->getimagesizeinfo[0])
                       || (@$_GET['h'] == $phpThumb->getimagesizeinfo[1]))) {
             // image fits into 'w'x'h' box, and at least one dimension matches exactly, therefore no resizing needed
-            $phpThumb->DebugMessage('Passing "' . $SourceFilename . '" through directly, no resizing required ("' . $phpThumb->getimagesizeinfo[0] . '"x"' . $phpThumb->getimagesizeinfo[1] . '" fits inside "' . @$_GET['w'] . '"x"' . @$_GET['h'] . '")', __FILE__, __LINE__);
+            $phpThumb->DebugMessage('Passing "'.$SourceFilename.'" through directly, no resizing required ("'.$phpThumb->getimagesizeinfo[0].'"x"'.$phpThumb->getimagesizeinfo[1].'" fits inside "'.@$_GET['w'].'"x"'.@$_GET['h'].'")', __FILE__, __LINE__);
         } else {
-            $phpThumb->DebugMessage('Not passing "' . $SourceFilename . '" through directly because resizing required (from "' . $phpThumb->getimagesizeinfo[0] . '"x"' . $phpThumb->getimagesizeinfo[1] . '" to "' . @$_GET['w'] . '"x"' . @$_GET['h'] . '")', __FILE__, __LINE__);
+            $phpThumb->DebugMessage('Not passing "'.$SourceFilename.'" through directly because resizing required (from "'.$phpThumb->getimagesizeinfo[0].'"x"'.$phpThumb->getimagesizeinfo[1].'" to "'.@$_GET['w'].'"x"'.@$_GET['h'].'")', __FILE__, __LINE__);
             break;
         }
         switch ($phpThumb->getimagesizeinfo[2]) {
@@ -560,11 +560,11 @@ 
                 break;
             default:
                 // browser probably can't handle format, remangle it to JPEG/PNG/GIF
-                $phpThumb->DebugMessage('Not passing "' . $SourceFilename . '" through directly because $phpThumb->getimagesizeinfo[2] = "' . $phpThumb->getimagesizeinfo[2] . '"', __FILE__, __LINE__);
+                $phpThumb->DebugMessage('Not passing "'.$SourceFilename.'" through directly because $phpThumb->getimagesizeinfo[2] = "'.$phpThumb->getimagesizeinfo[2].'"', __FILE__, __LINE__);
                 break 2;
         }
 
-        $ImageCreateFunctions   = [
+        $ImageCreateFunctions = [
             1 => 'imagecreatefromgif',
             2 => 'imagecreatefromjpeg',
             3 => 'imagecreatefrompng'
@@ -581,28 +581,28 @@ 
             }
 
             if (headers_sent()) {
-                $phpThumb->ErrorImage('Headers already sent (' . basename(__FILE__) . ' line ' . __LINE__ . ')');
+                $phpThumb->ErrorImage('Headers already sent ('.basename(__FILE__).' line '.__LINE__.')');
                 exit;
             }
            if (\Xmf\Request::hasVar('phpThumbDebug', 'GET')) {
                 $phpThumb->DebugTimingMessage('skipped direct $SourceFilename passthru', __FILE__, __LINE__);
-                $phpThumb->DebugMessage('Would have passed "' . $SourceFilename . '" through directly, but skipping due to phpThumbDebug', __FILE__, __LINE__);
+                $phpThumb->DebugMessage('Would have passed "'.$SourceFilename.'" through directly, but skipping due to phpThumbDebug', __FILE__, __LINE__);
                 break;
             }
 
             SendSaveAsFileHeaderIfNeeded();
-            header('Last-Modified: ' . gmdate('D, d M Y H:i:s', @filemtime($SourceFilename)) . ' GMT');
+            header('Last-Modified: '.gmdate('D, d M Y H:i:s', @filemtime($SourceFilename)).' GMT');
             if ($contentType = phpthumb_functions::ImageTypeToMIMEtype(@$phpThumb->getimagesizeinfo[2])) {
-                header('Content-Type: ' . $contentType);
+                header('Content-Type: '.$contentType);
             }
             @readfile($SourceFilename);
             exit;
         } else {
-            $phpThumb->DebugMessage('Not passing "' . $SourceFilename . '" through directly because ($phpThumb->config_disable_onlycreateable_passthru = "' . $phpThumb->config_disable_onlycreateable_passthru . '") and ' . $theImageCreateFunction . '() failed', __FILE__, __LINE__);
+            $phpThumb->DebugMessage('Not passing "'.$SourceFilename.'" through directly because ($phpThumb->config_disable_onlycreateable_passthru = "'.$phpThumb->config_disable_onlycreateable_passthru.'") and '.$theImageCreateFunction.'() failed', __FILE__, __LINE__);
             break;
         }
     } else {
-        $phpThumb->DebugMessage('Not passing "' . $SourceFilename . '" through directly because getimagesize() failed', __FILE__, __LINE__);
+        $phpThumb->DebugMessage('Not passing "'.$SourceFilename.'" through directly because getimagesize() failed', __FILE__, __LINE__);
         break;
     }
     break;
@@ -621,7 +621,7 @@ 
 if (@is_readable($phpThumb->cache_filename)) {
     RedirectToCachedFile();
 } else {
-    $phpThumb->DebugMessage('Cached file "' . $phpThumb->cache_filename . '" does not exist, processing as normal', __FILE__, __LINE__);
+    $phpThumb->DebugMessage('Cached file "'.$phpThumb->cache_filename.'" does not exist, processing as normal', __FILE__, __LINE__);
 }
 
 ////////////////////////////////////////////////////////////////
@@ -656,30 +656,30 @@ 
         $new_background_color = phpthumb_functions::ImageHexColorAllocate($phpThumb->gdimg_source, $bghexcolor, false, $alpha);
         imagefilledrectangle($phpThumb->gdimg_source, 0, 0, $phpThumb->w, $phpThumb->h, $new_background_color);
     } else {
-        $phpThumb->ErrorImage('failed to create "new" image (' . $phpThumb->w . 'x' . $phpThumb->h . ')');
+        $phpThumb->ErrorImage('failed to create "new" image ('.$phpThumb->w.'x'.$phpThumb->h.')');
     }
 } elseif (!$phpThumb->src) {
-    $phpThumb->ErrorImage('Usage: ' . $_SERVER['PHP_SELF'] . '?src=/path/and/filename.jpg' . "\n" . 'read Usage comments for details');
+    $phpThumb->ErrorImage('Usage: '.$_SERVER['PHP_SELF'].'?src=/path/and/filename.jpg'."\n".'read Usage comments for details');
 } elseif (preg_match('#^([a-z0-9]+)://#i', $_GET['src'], $protocol_matches)) {
     if (preg_match('#^(f|ht)tps?://#i', $_GET['src'])) {
-        $phpThumb->DebugMessage('$phpThumb->src (' . $phpThumb->src . ') is remote image, attempting to download', __FILE__, __LINE__);
+        $phpThumb->DebugMessage('$phpThumb->src ('.$phpThumb->src.') is remote image, attempting to download', __FILE__, __LINE__);
         if ($phpThumb->config_http_user_agent) {
-            $phpThumb->DebugMessage('Setting "user_agent" to "' . $phpThumb->config_http_user_agent . '"', __FILE__, __LINE__);
+            $phpThumb->DebugMessage('Setting "user_agent" to "'.$phpThumb->config_http_user_agent.'"', __FILE__, __LINE__);
             ini_set('user_agent', $phpThumb->config_http_user_agent);
         }
         $cleanedupurl = phpthumb_functions::CleanUpURLencoding($phpThumb->src);
-        $phpThumb->DebugMessage('CleanUpURLencoding(' . $phpThumb->src . ') returned "' . $cleanedupurl . '"', __FILE__, __LINE__);
+        $phpThumb->DebugMessage('CleanUpURLencoding('.$phpThumb->src.') returned "'.$cleanedupurl.'"', __FILE__, __LINE__);
         $phpThumb->src = $cleanedupurl;
         unset($cleanedupurl);
         if ($rawImageData = phpthumb_functions::SafeURLread($phpThumb->src, $error, $phpThumb->config_http_fopen_timeout, $phpThumb->config_http_follow_redirect)) {
-            $phpThumb->DebugMessage('SafeURLread(' . $phpThumb->src . ') succeeded' . ($error ? ' with messsages: "' . $error . '"' : ''), __FILE__, __LINE__);
-            $phpThumb->DebugMessage('Setting source data from URL "' . $phpThumb->src . '"', __FILE__, __LINE__);
+            $phpThumb->DebugMessage('SafeURLread('.$phpThumb->src.') succeeded'.($error ? ' with messsages: "'.$error.'"' : ''), __FILE__, __LINE__);
+            $phpThumb->DebugMessage('Setting source data from URL "'.$phpThumb->src.'"', __FILE__, __LINE__);
             $phpThumb->setSourceData($rawImageData, urlencode($phpThumb->src));
         } else {
             $phpThumb->ErrorImage($error);
         }
     } else {
-        $phpThumb->ErrorImage('only FTP and HTTP/HTTPS protocols are allowed, "' . $protocol_matches[1] . '" is not');
+        $phpThumb->ErrorImage('only FTP and HTTP/HTTPS protocols are allowed, "'.$protocol_matches[1].'" is not');
     }
 }
 
@@ -714,10 +714,10 @@ 
             chmod($phpThumb->cache_filename, 0644);
             RedirectToCachedFile();
         } else {
-            $phpThumb->DebugMessage('Failed: RenderToFile(' . $phpThumb->cache_filename . ')', __FILE__, __LINE__);
+            $phpThumb->DebugMessage('Failed: RenderToFile('.$phpThumb->cache_filename.')', __FILE__, __LINE__);
         }
     } else {
-        $phpThumb->DebugMessage('Cannot write to $phpThumb->cache_filename (' . $phpThumb->cache_filename . ') because that directory (' . dirname($phpThumb->cache_filename) . ') is not writable', __FILE__, __LINE__);
+        $phpThumb->DebugMessage('Cannot write to $phpThumb->cache_filename ('.$phpThumb->cache_filename.') because that directory ('.dirname($phpThumb->cache_filename).') is not writable', __FILE__, __LINE__);
     }
 }
 
@@ -730,7 +730,7 @@ 
 ////////////////////////////////////////////////////////////////
 
 if (!$phpThumb->OutputThumbnail()) {
-    $phpThumb->ErrorImage('Error in OutputThumbnail():' . "\n" . $phpThumb->debugmessages[count($phpThumb->debugmessages) - 1]);
+    $phpThumb->ErrorImage('Error in OutputThumbnail():'."\n".$phpThumb->debugmessages[count($phpThumb->debugmessages) - 1]);
 }
 
 ////////////////////////////////////////////////////////////////

phpthumb/phpthumb.gif.php

Spacing +14 added lines, -14 removed lines

@@ -141,9 +141,9 @@ 
 {
     // JPEG output that does not require cjpeg added by James Heinrich <info@silisoftware.com> - December 10, 2003
     if (('WIN' !== strtoupper(substr(PHP_OS, 0, 3))) && (file_exists('/usr/local/bin/cjpeg') || shell_exec('which cjpeg'))) {
-        if (gif_outputAsBmp($gif, $lpszFileName . '.bmp', $bgColor)) {
-            exec('cjpeg ' . $lpszFileName . '.bmp >' . $lpszFileName . ' 2>/dev/null');
-            @unlink($lpszFileName . '.bmp');
+        if (gif_outputAsBmp($gif, $lpszFileName.'.bmp', $bgColor)) {
+            exec('cjpeg '.$lpszFileName.'.bmp >'.$lpszFileName.' 2>/dev/null');
+            @unlink($lpszFileName.'.bmp');
 
             if (@file_exists($lpszFileName)) {
                 if (@filesize($lpszFileName) > 0) {
@@ -500,9 +500,9 @@ 
         $ret = '';
 
         for ($i = 0; $i < $this->m_nColors; $i++) {
-            $ret .= chr($this->m_arColors[$i] & 0x000000FF) . // R
-                    chr(($this->m_arColors[$i] & 0x0000FF00) >> 8) . // G
-                    chr(($this->m_arColors[$i] & 0x00FF0000) >> 16);  // B
+            $ret .= chr($this->m_arColors[$i] & 0x000000FF).// R
+                    chr(($this->m_arColors[$i] & 0x0000FF00) >> 8).// G
+                    chr(($this->m_arColors[$i] & 0x00FF0000) >> 16); // B
         }
 
         return $ret;
@@ -518,9 +518,9 @@ 
         $ret = '';
 
         for ($i = 0; $i < $this->m_nColors; $i++) {
-            $ret .= chr(($this->m_arColors[$i] & 0x00FF0000) >> 16) . // B
-                    chr(($this->m_arColors[$i] & 0x0000FF00) >> 8) . // G
-                    chr($this->m_arColors[$i] & 0x000000FF) . // R
+            $ret .= chr(($this->m_arColors[$i] & 0x00FF0000) >> 16).// B
+                    chr(($this->m_arColors[$i] & 0x0000FF00) >> 8).// G
+                    chr($this->m_arColors[$i] & 0x000000FF).// R
                     "\x00";
         }
 
@@ -791,14 +791,14 @@ 
                     if (!$this->m_gih->load($data, $len = 0)) {
                         return false;
                     }
-                    $data   = substr($data, $len);
+                    $data = substr($data, $len);
                     $datLen += $len;
 
                     // ALLOC BUFFER
                     if (!($this->m_data = $this->m_lzw->deCompress($data, $len = 0))) {
                         return false;
                     }
-                    $data   = substr($data, $len);
+                    $data = substr($data, $len);
                     $datLen += $len;
 
                     if ($this->m_gih->m_bInterlace) {
@@ -913,7 +913,7 @@ 
                 $lne          = substr($this->m_data, 0, $this->m_gih->m_nWidth);
                 $this->m_data = substr($this->m_data, $this->m_gih->m_nWidth);
 
-                $data = substr($data, 0, $y * $this->m_gih->m_nWidth) . $lne . substr($data, ($y + 1) * $this->m_gih->m_nWidth);
+                $data = substr($data, 0, $y * $this->m_gih->m_nWidth).$lne.substr($data, ($y + 1) * $this->m_gih->m_nWidth);
             }
         }
 
@@ -1286,7 +1286,7 @@ 
     {
         $val = (int)$val;
 
-        return chr($val & 0xFF) . chr(($val & 0xFF00) >> 8) . chr(($val & 0xFF0000) >> 16) . chr(($val & 0xFF000000) >> 24);
+        return chr($val & 0xFF).chr(($val & 0xFF00) >> 8).chr(($val & 0xFF0000) >> 16).chr(($val & 0xFF000000) >> 24);
     }
 
     ///////////////////////////////////////////////////////////////////////////
@@ -1299,7 +1299,7 @@ 
     {
         $val = (int)$val;
 
-        return chr(($val & 0xFF000000) >> 24) . chr(($val & 0xFF0000) >> 16) . chr(($val & 0xFF00) >> 8) . chr($val & 0xFF);
+        return chr(($val & 0xFF000000) >> 24).chr(($val & 0xFF0000) >> 16).chr(($val & 0xFF00) >> 8).chr($val & 0xFF);
     }
 
     ///////////////////////////////////////////////////////////////////////////

phpthumb/phpThumb.config.php

Spacing +49 added lines, -49 removed lines

@@ -9,11 +9,11 @@ 
 //            THIS CONFIG FILE ONLY APPLIES TO phpThumb.php //
 //                                                         ///
 //////////////////////////////////////////////////////////////
-require dirname(dirname(__DIR__)) . '/header.php';
+require dirname(dirname(__DIR__)).'/header.php';
 ob_start();
-if (!file_exists(__DIR__ . '/phpthumb.functions.php') || !require_once __DIR__ . '/phpthumb.functions.php') {
+if (!file_exists(__DIR__.'/phpthumb.functions.php') || !require_once __DIR__.'/phpthumb.functions.php') {
     ob_end_flush();
-    die('failed to include_once(phpthumb.functions.php) - realpath="' . realpath(__DIR__ . '/phpthumb.functions.php') . '"');
+    die('failed to include_once(phpthumb.functions.php) - realpath="'.realpath(__DIR__.'/phpthumb.functions.php').'"');
 }
 ob_end_clean();
 
@@ -32,7 +32,7 @@ 
 // * Cache directory configuration (choose only one of these - leave the other lines commented-out):
 // Note: this directory must be writable (usually chmod 777 is neccesary) for caching to work.
 // If the directory is not writable no error will be generated but caching will be disabled.
-$PHPTHUMB_CONFIG['cache_directory'] = XOOPS_CACHE_PATH;                            // set the cache directory relative to the phpThumb() installation
+$PHPTHUMB_CONFIG['cache_directory'] = XOOPS_CACHE_PATH; // set the cache directory relative to the phpThumb() installation
 //$PHPTHUMB_CONFIG['cache_directory'] = $PHPTHUMB_CONFIG['document_root'].'/phpthumb/cache/'; // set the cache directory to an absolute directory for all source images
 //$PHPTHUMB_CONFIG['cache_directory'] = './cache/';                                           // set the cache directory relative to the source image - must start with '.' (will not work to cache URL- or database-sourced images, please use an absolute directory name)
 //$PHPTHUMB_CONFIG['cache_directory'] = null;                                                 // disable thumbnail caching (not recommended)
@@ -50,35 +50,35 @@ 
 //   based on last-access date and/or number of files and/or total filesize.
 
 //$PHPTHUMB_CONFIG['cache_maxage'] = null;            // never delete cached thumbnails based on last-access time
-$PHPTHUMB_CONFIG['cache_maxage'] = 86400 * 30;        // delete cached thumbnails that haven't been accessed in more than [30 days] (value is maximum time since last access in seconds to avoid deletion)
+$PHPTHUMB_CONFIG['cache_maxage'] = 86400 * 30; // delete cached thumbnails that haven't been accessed in more than [30 days] (value is maximum time since last access in seconds to avoid deletion)
 
 //$PHPTHUMB_CONFIG['cache_maxsize'] = null;           // never delete cached thumbnails based on byte size of cache directory
 $PHPTHUMB_CONFIG['cache_maxsize'] = 10 * 1024 * 1024; // delete least-recently-accessed cached thumbnails when more than [10MB] of cached files are present (value is maximum bytesize of all cached files)
 
 //$PHPTHUMB_CONFIG['cache_maxfiles'] = null;          // never delete cached thumbnails based on number of cached files
-$PHPTHUMB_CONFIG['cache_maxfiles'] = 200;             // delete least-recently-accessed cached thumbnails when more than [200] cached files are present (value is maximum number of cached files to keep)
+$PHPTHUMB_CONFIG['cache_maxfiles'] = 200; // delete least-recently-accessed cached thumbnails when more than [200] cached files are present (value is maximum number of cached files to keep)
 
 // * Source image cache configuration
-$PHPTHUMB_CONFIG['cache_source_enabled']   = true;                               // if true, source images obtained via HTTP are cached to $PHPTHUMB_CONFIG['cache_source_directory']
-$PHPTHUMB_CONFIG['cache_source_directory'] = XOOPS_CACHE_PATH;  // set the cache directory for unprocessed source images
+$PHPTHUMB_CONFIG['cache_source_enabled']   = true; // if true, source images obtained via HTTP are cached to $PHPTHUMB_CONFIG['cache_source_directory']
+$PHPTHUMB_CONFIG['cache_source_directory'] = XOOPS_CACHE_PATH; // set the cache directory for unprocessed source images
 
 // * cache source modification date configuration
 $PHPTHUMB_CONFIG['cache_source_filemtime_ignore_local']  = false; // if true, local source images will not be checked for modification date and cached image will be used if available, even if source image is changed or removed
-$PHPTHUMB_CONFIG['cache_source_filemtime_ignore_remote'] = true;  // if true, remote source images will not be checked for modification date and cached image will be used if available, even if source image is changed or removed. WARNING: cached performance MUCH slower if this is set to false.
+$PHPTHUMB_CONFIG['cache_source_filemtime_ignore_remote'] = true; // if true, remote source images will not be checked for modification date and cached image will be used if available, even if source image is changed or removed. WARNING: cached performance MUCH slower if this is set to false.
 
 // * Simplified cache filename configuration
 // Instead of creating unique cache filenames for all parameter combinations, create "simple" cache files (eg: "pic_thumb.jpg")
 // If cache_default_only_suffix is non-empty, GETstring parameters (except 'src') are ignored and only $PHPTHUMB_DEFAULTS
 // parameters (set at the bottom of phpThumb.config.php) are used for processing.
 // The '*' character MUST be used to represent the source image name
-$PHPTHUMB_CONFIG['cache_default_only_suffix'] = '';           // cached in normal phpThumb manner
+$PHPTHUMB_CONFIG['cache_default_only_suffix'] = ''; // cached in normal phpThumb manner
 //$PHPTHUMB_CONFIG['cache_default_only_suffix'] = '*_thumb';  // cache 'pic.jpg' becomes 'pic_thumb.jpg' (or 'pic_thumb.png' if PNG output is selected, etc)
 //$PHPTHUMB_CONFIG['cache_default_only_suffix'] = 'small-*';  // cache 'pic.jpg' becomes 'small-pic.jpg' (or 'small-pic.png' if PNG output is selected, etc)
 
-$PHPTHUMB_CONFIG['cache_prefix'] = 'phpThumb_cache_' . (isset($_SERVER['SERVER_NAME']) ? str_replace('www.', '', $_SERVER['SERVER_NAME']) . '_' : ''); // keep cache file separate by domain
+$PHPTHUMB_CONFIG['cache_prefix'] = 'phpThumb_cache_'.(isset($_SERVER['SERVER_NAME']) ? str_replace('www.', '', $_SERVER['SERVER_NAME']).'_' : ''); // keep cache file separate by domain
 //$PHPTHUMB_CONFIG['cache_prefix'] = 'phpThumb_cache';                                                                                             // allow phpThumb to share 1 set of cached files even if accessed under different servername/domains on same server
 
-$PHPTHUMB_CONFIG['cache_force_passthru'] = true;  // if true, cached image data will always be passed to browser; if false, HTTP redirect will be used instead
+$PHPTHUMB_CONFIG['cache_force_passthru'] = true; // if true, cached image data will always be passed to browser; if false, HTTP redirect will be used instead
 
 // * Temp directory configuration
 // phpThumb() may need to create temp files. Usually the system temp dir is writable and can be used.
@@ -86,7 +86,7 @@ 
 // you should change this to a full pathname to a directory you do have write access to.
 //$PHPTHUMB_CONFIG['temp_directory'] = null;                               // attempt to auto-detect
 //$PHPTHUMB_CONFIG['temp_directory'] = '/tmp/persistent/phpthumb/cache/';  // set to absolute path
-$PHPTHUMB_CONFIG['temp_directory'] = $PHPTHUMB_CONFIG['cache_directory'];  // set to same as cache directory
+$PHPTHUMB_CONFIG['temp_directory'] = $PHPTHUMB_CONFIG['cache_directory']; // set to same as cache directory
 
 // NOTE: "max_source_pixels" only affects GD-resized thumbnails. If you have ImageMagick
 //       installed it will bypass most of these limits
@@ -104,7 +104,7 @@ 
 if (phpthumb_functions::version_compare_replacement(PHP_VERSION, '4.3.2', '>=') && !defined('memory_get_usage')
     && !@ini_get('memory_limit')) {
     // memory_get_usage() will only be defined if your PHP is compiled with the --enable-memory-limit configuration option.
-    $PHPTHUMB_CONFIG['max_source_pixels'] = 0;         // no memory limit
+    $PHPTHUMB_CONFIG['max_source_pixels'] = 0; // no memory limit
 } else {
     // calculate default max_source_pixels as 1/6 of memory limit configuration
     $PHPTHUMB_CONFIG['max_source_pixels'] = round(max((int)ini_get('memory_limit'), (int)get_cfg_var('memory_limit')) * 1048576 / 6);
@@ -115,8 +115,8 @@ 
 }
 
 // ImageMagick configuration
-$PHPTHUMB_CONFIG['prefer_imagemagick']        = true;  // If true, use ImageMagick to resize thumbnails if possible, since it is usually faster than GD functions; if false only use ImageMagick if PHP memory limit is too low.
-$PHPTHUMB_CONFIG['imagemagick_use_thumbnail'] = true;  // If true, use ImageMagick's "-thumbnail" resizing parameter (if available) which removes extra non-image metadata (profiles, EXIF info, etc) resulting in much smaller filesize; if false, use "-resize" paramter which retains this info
+$PHPTHUMB_CONFIG['prefer_imagemagick']        = true; // If true, use ImageMagick to resize thumbnails if possible, since it is usually faster than GD functions; if false only use ImageMagick if PHP memory limit is too low.
+$PHPTHUMB_CONFIG['imagemagick_use_thumbnail'] = true; // If true, use ImageMagick's "-thumbnail" resizing parameter (if available) which removes extra non-image metadata (profiles, EXIF info, etc) resulting in much smaller filesize; if false, use "-resize" paramter which retains this info
 if ('WIN' === strtoupper(substr(PHP_OS, 0, 3))) {
     // Windows: set absolute pathname
     $PHPTHUMB_CONFIG['imagemagick_path'] = 'C:/ImageMagick/convert.exe';
@@ -128,41 +128,41 @@ 
 
 // * Default output configuration:
 $PHPTHUMB_CONFIG['output_format']    = 'png'; // default output format ('jpeg', 'png' or 'gif') - thumbnail will be output in this format (if available in your version of GD or ImageMagick). This is only used if the "f" parameter is not specified, and if the thumbnail can't be output in the input format.
-$PHPTHUMB_CONFIG['output_maxwidth']  = 0;      // default maximum thumbnail width.  If this is zero then default width  is the width  of the source image. This is always overridden by ?w=___ GETstring parameter
-$PHPTHUMB_CONFIG['output_maxheight'] = 0;      // default maximum thumbnail height. If this is zero then default height is the height of the source image. This is always overridden by ?h=___ GETstring parameter
-$PHPTHUMB_CONFIG['output_interlace'] = true;   // if true: interlaced output for GIF/PNG, progressive output for JPEG; if false: non-interlaced for GIF/PNG, baseline for JPEG.
+$PHPTHUMB_CONFIG['output_maxwidth']  = 0; // default maximum thumbnail width.  If this is zero then default width  is the width  of the source image. This is always overridden by ?w=___ GETstring parameter
+$PHPTHUMB_CONFIG['output_maxheight'] = 0; // default maximum thumbnail height. If this is zero then default height is the height of the source image. This is always overridden by ?h=___ GETstring parameter
+$PHPTHUMB_CONFIG['output_interlace'] = true; // if true: interlaced output for GIF/PNG, progressive output for JPEG; if false: non-interlaced for GIF/PNG, baseline for JPEG.
 
 // * Error message configuration
-$PHPTHUMB_CONFIG['error_image_width']           = 300;      // default width for error images
-$PHPTHUMB_CONFIG['error_image_height']          = 100;      // default height for error images
-$PHPTHUMB_CONFIG['error_message_image_default'] = '';       // Set this to the name of a generic error image (e.g. '/images/error.png') that you want displayed in place of any error message that may occur. This setting is overridden by the 'err' parameter, which does the same thing.
+$PHPTHUMB_CONFIG['error_image_width']           = 300; // default width for error images
+$PHPTHUMB_CONFIG['error_image_height']          = 100; // default height for error images
+$PHPTHUMB_CONFIG['error_message_image_default'] = ''; // Set this to the name of a generic error image (e.g. '/images/error.png') that you want displayed in place of any error message that may occur. This setting is overridden by the 'err' parameter, which does the same thing.
 $PHPTHUMB_CONFIG['error_bgcolor']               = 'CCCCFF'; // background color of error message images
 $PHPTHUMB_CONFIG['error_textcolor']             = 'FF0000'; // color of text in error messages
-$PHPTHUMB_CONFIG['error_fontsize']              = 1;        // size of text in error messages, from 1 (smallest) to 5 (largest)
-$PHPTHUMB_CONFIG['error_die_on_error']          = true;     // die with error message on any fatal error (recommended with standalone phpThumb.php)
-$PHPTHUMB_CONFIG['error_silent_die_on_error']   = false;    // simply die with no output of any kind on fatal errors (not recommended)
-$PHPTHUMB_CONFIG['error_die_on_source_failure'] = true;     // die with error message if source image cannot be processed by phpThumb() (usually because source image is corrupt in some way). If false the source image will be passed through unprocessed, if true (default) an error message will be displayed.
+$PHPTHUMB_CONFIG['error_fontsize']              = 1; // size of text in error messages, from 1 (smallest) to 5 (largest)
+$PHPTHUMB_CONFIG['error_die_on_error']          = true; // die with error message on any fatal error (recommended with standalone phpThumb.php)
+$PHPTHUMB_CONFIG['error_silent_die_on_error']   = false; // simply die with no output of any kind on fatal errors (not recommended)
+$PHPTHUMB_CONFIG['error_die_on_source_failure'] = true; // die with error message if source image cannot be processed by phpThumb() (usually because source image is corrupt in some way). If false the source image will be passed through unprocessed, if true (default) an error message will be displayed.
 
 // * Off-server Thumbnailing Configuration:
-$PHPTHUMB_CONFIG['nohotlink_enabled']       = false;                                    // If false will allow thumbnailing from any source domain
-$PHPTHUMB_CONFIG['nohotlink_valid_domains'] = [@$_SERVER['HTTP_HOST']];            // This is the list of domains for which thumbnails are allowed to be created. Note: domain only, do not include port numbers. The default value of the current domain should be fine in most cases, but if neccesary you can add more domains in here, in the format "www.example.com"
-$PHPTHUMB_CONFIG['nohotlink_erase_image']   = true;                                     // if true thumbnail is covered up with $PHPTHUMB_CONFIG['nohotlink_fill_color'] before text is applied, if false text is written over top of thumbnail
+$PHPTHUMB_CONFIG['nohotlink_enabled']       = false; // If false will allow thumbnailing from any source domain
+$PHPTHUMB_CONFIG['nohotlink_valid_domains'] = [@$_SERVER['HTTP_HOST']]; // This is the list of domains for which thumbnails are allowed to be created. Note: domain only, do not include port numbers. The default value of the current domain should be fine in most cases, but if neccesary you can add more domains in here, in the format "www.example.com"
+$PHPTHUMB_CONFIG['nohotlink_erase_image']   = true; // if true thumbnail is covered up with $PHPTHUMB_CONFIG['nohotlink_fill_color'] before text is applied, if false text is written over top of thumbnail
 $PHPTHUMB_CONFIG['nohotlink_text_message']  = 'Off-server thumbnailing is not allowed'; // text of error message
 
 // * Off-server Linking Configuration:
-$PHPTHUMB_CONFIG['nooffsitelink_enabled']       = false;                                       // If false will allow thumbnails to be linked to from any domain, if true only domains listed below in 'nooffsitelink_valid_domains' will be allowed.
-$PHPTHUMB_CONFIG['nooffsitelink_valid_domains'] = [@$_SERVER['HTTP_HOST']];              // This is the list of domains for which thumbnails are allowed to be created. The default value of the current domain should be fine in most cases, but if neccesary you can add more domains in here, in the format 'www.example.com'
-$PHPTHUMB_CONFIG['nooffsitelink_require_refer'] = false;                                      // If false will allow standalone calls to phpThumb(). If true then only requests with a Request::getString('HTTP_REFERER', '', 'SERVER') value in 'nooffsitelink_valid_domains' are allowed.
-$PHPTHUMB_CONFIG['nooffsitelink_erase_image']   = false;                                      // if true thumbnail is covered up with $PHPTHUMB_CONFIG['nohotlink_fill_color'] before text is applied, if false text is written over top of thumbnail
-$PHPTHUMB_CONFIG['nooffsitelink_watermark_src'] = '/demo/images/watermark.png';                // webroot-relative image to overlay on hotlinked images
-$PHPTHUMB_CONFIG['nooffsitelink_text_message']  = 'Image taken from ' . @$_SERVER['HTTP_HOST']; // text of error message (used if [nooffsitelink_watermark_src] is not a valid image)
+$PHPTHUMB_CONFIG['nooffsitelink_enabled']       = false; // If false will allow thumbnails to be linked to from any domain, if true only domains listed below in 'nooffsitelink_valid_domains' will be allowed.
+$PHPTHUMB_CONFIG['nooffsitelink_valid_domains'] = [@$_SERVER['HTTP_HOST']]; // This is the list of domains for which thumbnails are allowed to be created. The default value of the current domain should be fine in most cases, but if neccesary you can add more domains in here, in the format 'www.example.com'
+$PHPTHUMB_CONFIG['nooffsitelink_require_refer'] = false; // If false will allow standalone calls to phpThumb(). If true then only requests with a Request::getString('HTTP_REFERER', '', 'SERVER') value in 'nooffsitelink_valid_domains' are allowed.
+$PHPTHUMB_CONFIG['nooffsitelink_erase_image']   = false; // if true thumbnail is covered up with $PHPTHUMB_CONFIG['nohotlink_fill_color'] before text is applied, if false text is written over top of thumbnail
+$PHPTHUMB_CONFIG['nooffsitelink_watermark_src'] = '/demo/images/watermark.png'; // webroot-relative image to overlay on hotlinked images
+$PHPTHUMB_CONFIG['nooffsitelink_text_message']  = 'Image taken from '.@$_SERVER['HTTP_HOST']; // text of error message (used if [nooffsitelink_watermark_src] is not a valid image)
 
 // * Border & Background default colors
 $PHPTHUMB_CONFIG['border_hexcolor']     = '000000'; // Default border color - usual HTML-style hex color notation (overridden with 'bc' parameter)
 $PHPTHUMB_CONFIG['background_hexcolor'] = 'FFFFFF'; // Default background color when thumbnail aspect ratio does not match fixed-dimension box - usual HTML-style hex color notation (overridden with 'bg' parameter)
 
 // * Watermark configuration
-$PHPTHUMB_CONFIG['ttf_directory'] = __DIR__ . '/fonts'; // Base directory for TTF font files
+$PHPTHUMB_CONFIG['ttf_directory'] = __DIR__.'/fonts'; // Base directory for TTF font files
 //$PHPTHUMB_CONFIG['ttf_directory'] = 'c:/windows/fonts';
 
 // * MySQL configuration
@@ -180,13 +180,13 @@ 
 $PHPTHUMB_CONFIG['mysql_database'] = '';
 
 // * Security configuration
-$PHPTHUMB_CONFIG['high_security_enabled']    = false;  // if enabled, requires 'high_security_password' set to be set and requires the use of phpThumbURL() function (at the bottom of phpThumb.config.php) to generate hashed URLs
-$PHPTHUMB_CONFIG['high_security_password']   = '';     // required if 'high_security_enabled' is true, and must be at complex (uppercase, lowercase, numbers, punctuation, etc -- punctuation is strongest, lowercase is weakest; see PasswordStrength() in phpThumb.php). You can use a password generator like http://silisoftware.com/tools/password-random.php to generate a strong password
-$PHPTHUMB_CONFIG['disable_debug']            = true;   // prevent phpThumb from displaying any information about your system. If true, phpThumbDebug and error messages will be disabled
-$PHPTHUMB_CONFIG['allow_src_above_docroot']  = true;  // if true, allow src to be anywhere in filesystem; if false (default) only allow src within document_root
-$PHPTHUMB_CONFIG['allow_src_above_phpthumb'] = true;   // if true (default), allow src to be anywhere in filesystem; if false only allow src within sub-directory of phpThumb installation
-$PHPTHUMB_CONFIG['allow_parameter_file']     = false;  // if true, allow use of 'file' parameter; if false (default) the 'file' parameter is disabled/ignored
-$PHPTHUMB_CONFIG['allow_parameter_goto']     = false;  // if true, allow use of 'goto' parameter; if false (default) the 'goto' parameter is disabled/ignored
+$PHPTHUMB_CONFIG['high_security_enabled']    = false; // if enabled, requires 'high_security_password' set to be set and requires the use of phpThumbURL() function (at the bottom of phpThumb.config.php) to generate hashed URLs
+$PHPTHUMB_CONFIG['high_security_password']   = ''; // required if 'high_security_enabled' is true, and must be at complex (uppercase, lowercase, numbers, punctuation, etc -- punctuation is strongest, lowercase is weakest; see PasswordStrength() in phpThumb.php). You can use a password generator like http://silisoftware.com/tools/password-random.php to generate a strong password
+$PHPTHUMB_CONFIG['disable_debug']            = true; // prevent phpThumb from displaying any information about your system. If true, phpThumbDebug and error messages will be disabled
+$PHPTHUMB_CONFIG['allow_src_above_docroot']  = true; // if true, allow src to be anywhere in filesystem; if false (default) only allow src within document_root
+$PHPTHUMB_CONFIG['allow_src_above_phpthumb'] = true; // if true (default), allow src to be anywhere in filesystem; if false only allow src within sub-directory of phpThumb installation
+$PHPTHUMB_CONFIG['allow_parameter_file']     = false; // if true, allow use of 'file' parameter; if false (default) the 'file' parameter is disabled/ignored
+$PHPTHUMB_CONFIG['allow_parameter_goto']     = false; // if true, allow use of 'goto' parameter; if false (default) the 'goto' parameter is disabled/ignored
 
 // * HTTP UserAgent configuration
 //$PHPTHUMB_CONFIG['http_user_agent'] = '';                                                                                      // PHP default: none
@@ -194,12 +194,12 @@ 
 $PHPTHUMB_CONFIG['http_user_agent'] = 'Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.7.12) Gecko/20050915 Firefox/1.0.7'; // Windows XP, Firefox
 
 // * Compatibility settings
-$PHPTHUMB_CONFIG['disable_pathinfo_parsing']        = false;  // if true, $_SERVER[PATH_INFO] is not parsed. May be needed on some server configurations to allow normal behavior.
-$PHPTHUMB_CONFIG['disable_imagecopyresampled']      = false;  // if true, ImageCopyResampled is replaced with ImageCopyResampleBicubic. May be needed for buggy versions of PHP-GD.
-$PHPTHUMB_CONFIG['disable_onlycreateable_passthru'] = true;   // if true, any image that can be parsed by getimagesize() can be passed through; if false, only images that can be converted to GD by ImageCreateFrom(JPEG|GIF|PNG) functions are allowed
+$PHPTHUMB_CONFIG['disable_pathinfo_parsing']        = false; // if true, $_SERVER[PATH_INFO] is not parsed. May be needed on some server configurations to allow normal behavior.
+$PHPTHUMB_CONFIG['disable_imagecopyresampled']      = false; // if true, ImageCopyResampled is replaced with ImageCopyResampleBicubic. May be needed for buggy versions of PHP-GD.
+$PHPTHUMB_CONFIG['disable_onlycreateable_passthru'] = true; // if true, any image that can be parsed by getimagesize() can be passed through; if false, only images that can be converted to GD by ImageCreateFrom(JPEG|GIF|PNG) functions are allowed
 
 // * HTTP remote file opening settings
-$PHPTHUMB_CONFIG['http_fopen_timeout']   = 10;   // timeout (in seconds) for fopen / curl / fsockopen
+$PHPTHUMB_CONFIG['http_fopen_timeout']   = 10; // timeout (in seconds) for fopen / curl / fsockopen
 $PHPTHUMB_CONFIG['http_follow_redirect'] = true; // if true (default), follow "302 Found" redirects to new URL; if false, return error message
 
 // * Speed optimizations configuration
@@ -213,7 +213,7 @@ 
 // START DEFAULT PARAMETERS SECTION
 // If any parameters are constant across ALL images, you can set them here
 
-$PHPTHUMB_DEFAULTS_GETSTRINGOVERRIDE = true;  // if true, any parameters in the URL will override the defaults set here; if false, any parameters set here cannot be overridden in the URL
+$PHPTHUMB_DEFAULTS_GETSTRINGOVERRIDE = true; // if true, any parameters in the URL will override the defaults set here; if false, any parameters set here cannot be overridden in the URL
 $PHPTHUMB_DEFAULTS_DISABLEGETPARAMS  = false; // if true, GETstring parameters will be ignored (except for 'src') and only below default parameters will be used; if false, both default and GETstring parameters will be used (depending on $PHPTHUMB_DEFAULTS_GETSTRINGOVERRIDE). Will be auto-set true if !empty($PHPTHUMB_CONFIG['cache_default_only_suffix'])
 
 //$PHPTHUMB_DEFAULTS['w']    = 200;
@@ -236,7 +236,7 @@ 
 {
     global $PHPTHUMB_CONFIG;
 
-    return str_replace(@$PHPTHUMB_CONFIG['document_root'], '', __DIR__) . DIRECTORY_SEPARATOR . 'phpThumb.php?' . $ParameterString . '&hash=' . md5($ParameterString . @$PHPTHUMB_CONFIG['high_security_password']);
+    return str_replace(@$PHPTHUMB_CONFIG['document_root'], '', __DIR__).DIRECTORY_SEPARATOR.'phpThumb.php?'.$ParameterString.'&hash='.md5($ParameterString.@$PHPTHUMB_CONFIG['high_security_password']);
 }
 
 ///////////////////////////////////////////////////////////////////////////////

phpthumb/phpthumb.unsharp.php

Spacing +4 added lines, -4 removed lines

@@ -92,12 +92,12 @@
             // Move copies of the image around one pixel at the time and merge them with weight
             // according to the matrix. The same matrix is simply repeated for higher radii.
             for ($i = 0; $i < $radius; $i++) {
-                imagecopy($imgBlur, $img, 0, 0, 1, 0, $w - 1, $h);               // left
-                imagecopymerge($imgBlur, $img, 1, 0, 0, 0, $w, $h, 50);       // right
-                imagecopymerge($imgBlur, $img, 0, 0, 0, 0, $w, $h, 50);       // center
+                imagecopy($imgBlur, $img, 0, 0, 1, 0, $w - 1, $h); // left
+                imagecopymerge($imgBlur, $img, 1, 0, 0, 0, $w, $h, 50); // right
+                imagecopymerge($imgBlur, $img, 0, 0, 0, 0, $w, $h, 50); // center
                 imagecopy($imgCanvas, $imgBlur, 0, 0, 0, 0, $w, $h);
                 imagecopymerge($imgBlur, $imgCanvas, 0, 0, 0, 1, $w, $h - 1, 33.33333); // up
-                imagecopymerge($imgBlur, $imgCanvas, 0, 1, 0, 0, $w, $h, 25);       // down
+                imagecopymerge($imgBlur, $imgCanvas, 0, 1, 0, 0, $w, $h, 25); // down
             }
         }