mambax7 / pedigree

class/pedigree.php

Indentation +2 added lines, -2 removed lines

@@ -38,8 +38,8 @@
     protected function __construct($debug)
     {
         $this->debug   = $debug;
-       $moduleDirName = basename(dirname(__DIR__));
-       parent::__construct($moduleDirName);
+        $moduleDirName = basename(dirname(__DIR__));
+        parent::__construct($moduleDirName);
     }
 
     /**

Spacing +7 added lines, -7 removed lines

@@ -37,7 +37,7 @@ 
      */
     protected function __construct($debug)
     {
-        $this->debug   = $debug;
+        $this->debug = $debug;
        $moduleDirName = basename(dirname(__DIR__));
        parent::__construct($moduleDirName);
     }
@@ -88,7 +88,7 @@ 
 
             return null;
         }
-        $this->addLog("Getting config '{$name}' : " . print_r($this->config[$name], true));
+        $this->addLog("Getting config '{$name}' : ".print_r($this->config[$name], true));
 
         return $this->config[$name];
     }
@@ -105,7 +105,7 @@ 
             $this->initConfig();
         }
         $this->config[$name] = $value;
-        $this->addLog("Setting config '{$name}' : " . $this->config[$name]);
+        $this->addLog("Setting config '{$name}' : ".$this->config[$name]);
 
         return $this->config[$name];
     }
@@ -117,12 +117,12 @@ 
      */
     public function getHandler($name)
     {
-        if (!isset($this->handler[$name . 'Handler'])) {
+        if (!isset($this->handler[$name.'Handler'])) {
             $this->initHandler($name);
         }
         $this->addLog("Getting handler '{$name}'");
 
-        return $this->handler[$name . 'Handler'];
+        return $this->handler[$name.'Handler'];
     }
 
     public function initModule()
@@ -149,8 +149,8 @@ 
      */
     public function initHandler($name)
     {
-        $this->addLog('INIT ' . $name . ' HANDLER');
-        $this->handler[$name . 'Handler'] = xoops_getModuleHandler($name, $this->dirname);
+        $this->addLog('INIT '.$name.' HANDLER');
+        $this->handler[$name.'Handler'] = xoops_getModuleHandler($name, $this->dirname);
     }
 
     /**

class/SelectBox.php

Spacing +6 added lines, -6 removed lines

@@ -45,11 +45,11 @@ 
     {
         $this->fieldnumber  = $parentObject->getId();
         $this->fieldname    = $parentObject->fieldname;
-        $this->value        = $animalObject->{'user' . $this->fieldnumber};
+        $this->value        = $animalObject->{'user'.$this->fieldnumber};
         $this->defaultvalue = $parentObject->defaultvalue;
         $this->lookuptable  = $parentObject->lookuptable;
         if (0 == $this->lookuptable) {
-            echo "<span style='color: red;'><h3>A lookuptable must be specified for userfield" . $this->fieldnumber . '</h3></span>';
+            echo "<span style='color: red;'><h3>A lookuptable must be specified for userfield".$this->fieldnumber.'</h3></span>';
         }
     }
 
@@ -58,7 +58,7 @@ 
      */
     public function editField()
     {
-        $select         = new \XoopsFormSelect('<b>' . $this->fieldname . '</b>', 'user' . $this->fieldnumber, $value = $this->value, $size = 1, $multiple = false);
+        $select         = new \XoopsFormSelect('<b>'.$this->fieldname.'</b>', 'user'.$this->fieldnumber, $value = $this->value, $size = 1, $multiple = false);
         $lookupcontents = parent::lookupField($this->fieldnumber);
         $lcCount        = count($lookupcontents);
         foreach ($lookupcontents as $i => $iValue) {
@@ -75,7 +75,7 @@ 
      */
     public function newField($name = '')
     {
-        $select         = new \XoopsFormSelect('<b>' . $this->fieldname . '</b>', $name . 'user' . $this->fieldnumber, $value = $this->defaultvalue, $size = 1, $multiple = false);
+        $select         = new \XoopsFormSelect('<b>'.$this->fieldname.'</b>', $name.'user'.$this->fieldnumber, $value = $this->defaultvalue, $size = 1, $multiple = false);
         $lookupcontents = parent::lookupField($this->fieldnumber);
         $lcCount        = count($lookupcontents);
         foreach ($lookupcontents as $i => $iValue) {
@@ -116,7 +116,7 @@ 
             }
         }
 
-        return $this->fieldname . ' : ' . $choosenvalue;
+        return $this->fieldname.' : '.$choosenvalue;
     }
 
     /**
@@ -145,7 +145,7 @@ 
         $lookupcontents = parent::lookupField($this->fieldnumber);
         $lcCount        = count($lookupcontents);
         foreach ($lookupcontents as $i => $iValue) {
-            $select .= "<option value='" . $lookupcontents[$i]['id'] . "'>" . $lookupcontents[$i]['value'] . '</option>';
+            $select .= "<option value='".$lookupcontents[$i]['id']."'>".$lookupcontents[$i]['value'].'</option>';
         }
         $select .= '</select>';
 

class/DateSelect.php

Spacing +6 added lines, -6 removed lines

@@ -47,16 +47,16 @@ 
         //@todo move language strings to language file
         $this->fieldnumber  = $parentObject->getId();
         $this->fieldname    = $parentObject->fieldname;
-        $this->value        = $animalObject->{'user' . $this->fieldnumber};
+        $this->value        = $animalObject->{'user'.$this->fieldnumber};
         $this->defaultvalue = $parentObject->defaultvalue;
         if ($parentObject->hasLookup()) {
-            xoops_error('No lookuptable may be specified for userfield ' . $this->fieldnumber, get_class($this));
+            xoops_error('No lookuptable may be specified for userfield '.$this->fieldnumber, get_class($this));
         }
         if ($parentObject->inAdvanced()) {
-            xoops_error('userfield ' . $this->fieldnumber . ' cannot be shown in advanced info', get_class($this));
+            xoops_error('userfield '.$this->fieldnumber.' cannot be shown in advanced info', get_class($this));
         }
         if ($parentObject->inPie()) {
-            xoops_error('A Pie-chart cannot be specified for userfield ' . $this->fieldnumber, get_class($this));
+            xoops_error('A Pie-chart cannot be specified for userfield '.$this->fieldnumber, get_class($this));
         }
     }
 
@@ -65,7 +65,7 @@ 
      */
     public function editField()
     {
-        $textarea = new \XoopsFormTextDateSelect('<b>' . $this->fieldname . '</b>', 'user' . $this->fieldnumber, $size = 15, $this->value);
+        $textarea = new \XoopsFormTextDateSelect('<b>'.$this->fieldname.'</b>', 'user'.$this->fieldnumber, $size = 15, $this->value);
 
         return $textarea;
     }
@@ -77,7 +77,7 @@ 
      */
     public function newField($name = '')
     {
-        $textarea = new \XoopsFormTextDateSelect('<b>' . $this->fieldname . '</b>', $name . 'user' . $this->fieldnumber, $size = 15, $this->defaultvalue);
+        $textarea = new \XoopsFormTextDateSelect('<b>'.$this->fieldname.'</b>', $name.'user'.$this->fieldnumber, $size = 15, $this->defaultvalue);
 
         return $textarea;
     }

class/Picture.php

Spacing +11 added lines, -11 removed lines

@@ -34,23 +34,23 @@ 
     {
         $this->fieldnumber  = $parentObject->getId();
         $this->fieldname    = $parentObject->fieldname;
-        $this->value        = $animalObject->{'user' . $this->fieldnumber};
+        $this->value        = $animalObject->{'user'.$this->fieldnumber};
         $this->defaultvalue = $parentObject->defaultvalue;
         $this->lookuptable  = $parentObject->hasLookup();
         if ($this->lookuptable) {
-            xoops_error('No lookuptable may be specified for userfield ' . $this->fieldnumber);
+            xoops_error('No lookuptable may be specified for userfield '.$this->fieldnumber);
         }
         if ($parentObject->inAdvanced()) {
-            xoops_error('userfield ' . $this->fieldnumber . ' cannot be shown in advanced info', get_class($this));
+            xoops_error('userfield '.$this->fieldnumber.' cannot be shown in advanced info', get_class($this));
         }
         if ($parentObject->inPie()) {
-            xoops_error('A Pie-chart cannot be specified for userfield ' . $this->fieldnumber, get_class($this));
+            xoops_error('A Pie-chart cannot be specified for userfield '.$this->fieldnumber, get_class($this));
         }
         if ('1' == $parentObject->viewinlist) {
-            xoops_error('userfield ' . $this->fieldnumber . ' cannot be included in listview', get_class($this));
+            xoops_error('userfield '.$this->fieldnumber.' cannot be included in listview', get_class($this));
         }
         if ('1' == $parentObject->hassearch) {
-            xoops_error('Search cannot be defined for userfield ' . $this->fieldnumber, get_class($this));
+            xoops_error('Search cannot be defined for userfield '.$this->fieldnumber, get_class($this));
         }
     }
 
@@ -59,7 +59,7 @@ 
      */
     public function editField()
     {
-        $picturefield = new \XoopsFormFile($this->fieldname, 'user' . $this->fieldnumber, 1024000);
+        $picturefield = new \XoopsFormFile($this->fieldname, 'user'.$this->fieldnumber, 1024000);
         $picturefield->setExtra("size ='50'");
 
         return $picturefield;
@@ -72,7 +72,7 @@ 
      */
     public function newField($name = '')
     {
-        $picturefield = new \XoopsFormFile($this->fieldname, $name . 'user' . $this->fieldnumber, 1024000);
+        $picturefield = new \XoopsFormFile($this->fieldname, $name.'user'.$this->fieldnumber, 1024000);
         $picturefield->setExtra("size ='50'");
 
         return $picturefield;
@@ -83,7 +83,7 @@ 
      */
     public function viewField()
     {
-        $view = new \XoopsFormLabel($this->fieldname, '<img src="' . PEDIGREE_UPLOAD_URL . '/images/thumbnails/' . $this->value . '_400.jpeg">');
+        $view = new \XoopsFormLabel($this->fieldname, '<img src="'.PEDIGREE_UPLOAD_URL.'/images/thumbnails/'.$this->value.'_400.jpeg">');
 
         return $view;
     }
@@ -93,7 +93,7 @@ 
      */
     public function showField()
     {
-        return '<img src="' . PEDIGREE_UPLOAD_URL . '/images/thumbnails/' . $this->value . '_150.jpeg">';
+        return '<img src="'.PEDIGREE_UPLOAD_URL.'/images/thumbnails/'.$this->value.'_150.jpeg">';
     }
 
     /**
@@ -101,7 +101,7 @@ 
      */
     public function showValue()
     {
-        return '<img src="' . PEDIGREE_UPLOAD_URL . '/images/thumbnails/' . $this->value . '_400.jpeg">';
+        return '<img src="'.PEDIGREE_UPLOAD_URL.'/images/thumbnails/'.$this->value.'_400.jpeg">';
     }
 
     /**

class/Utility.php

Indentation +2 added lines, -2 removed lines

@@ -823,9 +823,9 @@
     {
         $helper = Pedigree\Helper::getInstance();
 
-       $pedigree_isAdmin = $helper->isUserAdmin();
+        $pedigree_isAdmin = $helper->isUserAdmin();
 
-         return $pedigree_isAdmin;
+            return $pedigree_isAdmin;
     }
 
     public static function getXoopsCpHeader()

Spacing +41 added lines, -41 removed lines

@@ -28,7 +28,7 @@ 
         //        $filteredFolder = XoopsFilterInput::clean($folder, 'PATH');
         if (!is_dir($folder)) {
             mkdir($folder);
-            file_put_contents($folder . '/index.html', '<script>history.go(-1);</script>');
+            file_put_contents($folder.'/index.html', '<script>history.go(-1);</script>');
         }
         //        chmod($filteredFolder, 0777);
     }
@@ -65,7 +65,7 @@ 
         $max_imgheight     = $GLOBALS['xoopsModuleConfig']['maximgheight']; //1000;
         $allowed_mimetypes = ['image/gif', 'image/jpeg', 'image/pjpeg', 'image/x-png', 'image/png'];
         //    $img_dir = XOOPS_ROOT_PATH . "/modules/" . $GLOBALS['xoopsModule']->dirname() . "/images" ;
-        $img_dir = $GLOBALS['xoopsModuleConfig']['uploaddir'] . '/images';
+        $img_dir = $GLOBALS['xoopsModuleConfig']['uploaddir'].'/images';
         require_once $GLOBALS['xoops']->path('class/uploader.php');
         $field = $_POST['xoops_upload_file'][$num];
         if (!empty($field) || '' != $field) {
@@ -120,7 +120,7 @@ 
     */
 
         // load the image
-        require_once $GLOBALS['xoops']->path('modules/' . $GLOBALS['xoopsModule']->dirname() . '/library/Zebra_Image.php');
+        require_once $GLOBALS['xoops']->path('modules/'.$GLOBALS['xoopsModule']->dirname().'/library/Zebra_Image.php');
         $thumbnail_widths = [150, 400];
 
         // indicate a target image
@@ -132,12 +132,12 @@ 
         // create a new instance of the class
         $image = new \Zebra_Image();
         // indicate a source image (a GIF, PNG or JPEG file)
-        $image->source_path = PEDIGREE_UPLOAD_PATH . "/images/{$filename}";
+        $image->source_path = PEDIGREE_UPLOAD_PATH."/images/{$filename}";
 
         foreach ($thumbnail_widths as $thumbnail_width) {
 
             // generate & output thumbnail
-            $output_filename    = PEDIGREE_UPLOAD_PATH . '/images/thumbnails/' . basename($filename) . "_{$thumbnail_width}.{$config_output_format}";
+            $output_filename    = PEDIGREE_UPLOAD_PATH.'/images/thumbnails/'.basename($filename)."_{$thumbnail_width}.{$config_output_format}";
             $image->target_path = $output_filename;
             // since in this example we're going to have a jpeg file, let's set the output
             // image's quality
@@ -293,17 +293,17 @@ 
                 $x = $columns[$i]['columnnumber'];
                 if (is_array($columns[$i]['lookupval'])) {
                     foreach ($columns[$i]['lookupval'] as $key => $keyvalue) {
-                        if ($keyvalue['id'] == $rowres['user' . $x]) {
+                        if ($keyvalue['id'] == $rowres['user'.$x]) {
                             $value = $keyvalue['value'];
                         }
                     }
                     //debug information
                     ///echo $columns[$i]['columnname']."is an array !";
                 } //format value - cant use object because of query count
-                elseif (0 === strncmp($rowres['user' . $x], 'http://', 7)) {
-                    $value = '<a href="' . $rowres['user' . $x] . '">' . $rowres['user' . $x] . '</a>';
+                elseif (0 === strncmp($rowres['user'.$x], 'http://', 7)) {
+                    $value = '<a href="'.$rowres['user'.$x].'">'.$rowres['user'.$x].'</a>';
                 } else {
-                    $value = $rowres['user' . $x];
+                    $value = $rowres['user'.$x];
                 }
                 $columnvalue[] = ['value' => $value];
             }
@@ -312,7 +312,7 @@ 
                 'id'          => $rowres['d_id'],
                 'name'        => $name,
                 'gender'      => $gender,
-                'link'        => '<a href="dog.php?id=' . $rowres['d_id'] . '">' . $name . '</a>',
+                'link'        => '<a href="dog.php?id='.$rowres['d_id'].'">'.$name.'</a>',
                 'colour'      => '',
                 'number'      => '',
                 'usercolumns' => $columnvalue
@@ -333,9 +333,9 @@ 
     {
         global $numofcolumns1, $nummatch1, $pages1, $columns1, $dogs1;
         if ('0' == $pa && '0' == $ma) {
-            $sqlquery = 'SELECT * FROM ' . $GLOBALS['xoopsDB']->prefix('pedigree_tree') . ' WHERE father = ' . $pa . ' AND mother = ' . $ma . ' AND id != ' . $oid . " AND father != '0' AND mother !='0' ORDER BY naam";
+            $sqlquery = 'SELECT * FROM '.$GLOBALS['xoopsDB']->prefix('pedigree_tree').' WHERE father = '.$pa.' AND mother = '.$ma.' AND id != '.$oid." AND father != '0' AND mother !='0' ORDER BY naam";
         } else {
-            $sqlquery = 'SELECT * FROM ' . $GLOBALS['xoopsDB']->prefix('pedigree_tree') . ' WHERE father = ' . $pa . ' AND mother = ' . $ma . ' AND id != ' . $oid . ' ORDER BY naam';
+            $sqlquery = 'SELECT * FROM '.$GLOBALS['xoopsDB']->prefix('pedigree_tree').' WHERE father = '.$pa.' AND mother = '.$ma.' AND id != '.$oid.' ORDER BY naam';
         }
         $queryresult = $GLOBALS['xoopsDB']->query($sqlquery);
         $nummatch1   = $GLOBALS['xoopsDB']->getRowsNum($queryresult);
@@ -382,17 +382,17 @@ 
                 $x = $columns1[$i]['columnnumber'];
                 if (is_array($columns1[$i]['lookupval'])) {
                     foreach ($columns1[$i]['lookupval'] as $key => $keyvalue) {
-                        if ($keyvalue['id'] == $rowres['user' . $x]) {
+                        if ($keyvalue['id'] == $rowres['user'.$x]) {
                             $value = $keyvalue['value'];
                         }
                     }
                     //debug information
                     ///echo $columns[$i]['columnname']."is an array !";
                 } //format value - cant use object because of query count
-                elseif (0 === strncmp($rowres['user' . $x], 'http://', 7)) {
-                    $value = '<a href="' . $rowres['user' . $x] . '">' . $rowres['user' . $x] . '</a>';
+                elseif (0 === strncmp($rowres['user'.$x], 'http://', 7)) {
+                    $value = '<a href="'.$rowres['user'.$x].'">'.$rowres['user'.$x].'</a>';
                 } else {
-                    $value = $rowres['user' . $x];
+                    $value = $rowres['user'.$x];
                 }
                 $columnvalue1[] = ['value' => $value];
             }
@@ -400,7 +400,7 @@ 
                 'id'          => $rowres['id'],
                 'name'        => $name,
                 'gender'      => $gender,
-                'link'        => '<a href="dog.php?id=' . $rowres['id'] . '">' . $name . '</a>',
+                'link'        => '<a href="dog.php?id='.$rowres['id'].'">'.$name.'</a>',
                 'colour'      => '',
                 'number'      => '',
                 'usercolumns' => $columnvalue1
@@ -421,9 +421,9 @@ 
         $content = '';
 
         if (0 == $breeder) {
-            $sqlquery = 'SELECT id, naam, roft FROM ' . $GLOBALS['xoopsDB']->prefix('pedigree_tree') . " WHERE id_owner = '" . $oid . "' ORDER BY naam";
+            $sqlquery = 'SELECT id, naam, roft FROM '.$GLOBALS['xoopsDB']->prefix('pedigree_tree')." WHERE id_owner = '".$oid."' ORDER BY naam";
         } else {
-            $sqlquery = 'SELECT id, naam, roft FROM ' . $GLOBALS['xoopsDB']->prefix('pedigree_tree') . " WHERE id_breeder = '" . $oid . "' ORDER BY naam";
+            $sqlquery = 'SELECT id, naam, roft FROM '.$GLOBALS['xoopsDB']->prefix('pedigree_tree')." WHERE id_breeder = '".$oid."' ORDER BY naam";
         }
         $queryresult = $GLOBALS['xoopsDB']->query($sqlquery);
         while (false !== ($rowres = $GLOBALS['xoopsDB']->fetchArray($queryresult))) {
@@ -432,8 +432,8 @@ 
             } else {
                 $gender = '<img src="assets/images/female.gif">';
             }
-            $link    = '<a href="dog.php?id=' . $rowres['id'] . '">' . stripslashes($rowres['naam']) . '</a>';
-            $content .= $gender . ' ' . $link . '<br>';
+            $link = '<a href="dog.php?id='.$rowres['id'].'">'.stripslashes($rowres['naam']).'</a>';
+            $content .= $gender.' '.$link.'<br>';
         }
 
         return $content;
@@ -447,7 +447,7 @@ 
     public static function getName($oid)
     {
         $oid         = (int)$oid;
-        $sqlquery    = 'SELECT naam FROM ' . $GLOBALS['xoopsDB']->prefix('pedigree_tree') . " WHERE id = '{$oid}'";
+        $sqlquery    = 'SELECT naam FROM '.$GLOBALS['xoopsDB']->prefix('pedigree_tree')." WHERE id = '{$oid}'";
         $queryresult = $GLOBALS['xoopsDB']->query($sqlquery);
         while (false !== ($rowres = $GLOBALS['xoopsDB']->fetchArray($queryresult))) {
             $an = stripslashes($rowres['naam']);
@@ -462,7 +462,7 @@ 
      */
     public static function showParent($PA)
     {
-        $sqlquery    = 'SELECT naam FROM ' . $GLOBALS['xoopsDB']->prefix('pedigree_tree') . " WHERE id='" . $PA . "'";
+        $sqlquery    = 'SELECT naam FROM '.$GLOBALS['xoopsDB']->prefix('pedigree_tree')." WHERE id='".$PA."'";
         $queryresult = $GLOBALS['xoopsDB']->query($sqlquery);
         while (false !== ($rowres = $GLOBALS['xoopsDB']->fetchArray($queryresult))) {
             $result = $rowres['naam'];
@@ -481,7 +481,7 @@ 
      */
     public static function findId($naam_hond)
     {
-        $sqlquery    = 'SELECT id FROM ' . $GLOBALS['xoopsDB']->prefix('pedigree_tree') . " WHERE naam= '$naam_hond'";
+        $sqlquery    = 'SELECT id FROM '.$GLOBALS['xoopsDB']->prefix('pedigree_tree')." WHERE naam= '$naam_hond'";
         $queryresult = $GLOBALS['xoopsDB']->query($sqlquery);
         while (false !== ($rowres = $GLOBALS['xoopsDB']->fetchArray($queryresult))) {
             $result = $rowres['id'];
@@ -534,8 +534,8 @@ 
             $gender = '';
             if ((!empty($GLOBALS['xoopsUser']) && $GLOBALS['xoopsUser'] instanceof \XoopsUser)
                 && ($row['user'] == $GLOBALS['xoopsUser']->getVar('uid') || true === $modadmin)) {
-                $gender = "<a href='dog.php?id={$row['id']}'><img src='assets/images/edit.png' alt='" . _EDIT . "'></a>
-                     . <a href='delete.php?id={$row['id']}'><img src='assets/images/delete.png' alt='" . _DELETE . "'></a>";
+                $gender = "<a href='dog.php?id={$row['id']}'><img src='assets/images/edit.png' alt='"._EDIT."'></a>
+                     . <a href='delete.php?id={$row['id']}'><img src='assets/images/delete.png' alt='"._DELETE."'></a>";
             }
 
             $genImg = (0 == $row['roft']) ? 'male.gif' : 'female.gif';
@@ -546,7 +546,7 @@ 
             } else {
                 $camera = '';
             }
-            $name = stripslashes($row['naam']) . $camera;
+            $name = stripslashes($row['naam']).$camera;
             unset($columnvalue);
 
             //fill array
@@ -555,22 +555,22 @@ 
                 $lookuparray = $columns[$i]['lookuparray'];
                 if (is_array($lookuparray)) {
                     foreach ($lookuparray as $index => $indexValue) {
-                        if ($lookuparray[$index]['id'] == $row['user' . $x]) {
+                        if ($lookuparray[$index]['id'] == $row['user'.$x]) {
                             //echo "<h1>".$lookuparray[$index]['id']."</h1>";
                             $value = $lookuparray[$index]['value'];
                         }
                     }
                 } //format value - cant use object because of query count
-                elseif (0 === strncmp($row['user' . $x], 'http://', 7)) {
-                    $value = '<a href="' . $row['user' . $x] . '">' . $row['user' . $x] . '</a>';
+                elseif (0 === strncmp($row['user'.$x], 'http://', 7)) {
+                    $value = '<a href="'.$row['user'.$x].'">'.$row['user'.$x].'</a>';
                 } else {
-                    $value = $row['user' . $x];
+                    $value = $row['user'.$x];
                 }
                 $columnvalue[] = ['value' => $value];
                 unset($value);
             }
 
-            $linkto = '<a href="' . $link . $row[$element] . '">' . $name . '</a>';
+            $linkto = '<a href="'.$link.$row[$element].'">'.$name.'</a>';
             //create array
             $dogs[] = [
                 'id'          => $row['id'],
@@ -602,7 +602,7 @@ 
         $cat_sql = '';
         if (is_array($cats)) {
             $cats    = array_map('intval', $cats); // make sure all cats are numbers
-            $cat_sql = '(' . implode(',', $cats) . ')';
+            $cat_sql = '('.implode(',', $cats).')';
         /*
                     $cat_sql = '(' . current($cats);
                     array_shift($cats);
@@ -612,7 +612,7 @@ 
                     $cat_sql .= ')';
         */
         } else {
-            $cat_sql = '(' . (int)$cats . ')'; // not efficient but at least creates valid SQL statement
+            $cat_sql = '('.(int)$cats.')'; // not efficient but at least creates valid SQL statement
         }
 
         return $cat_sql;
@@ -751,7 +751,7 @@ 
         //        xoops_load('XoopsLocal');
 
         //        $criteria = $myObject->getHandler($activeObject)->getActiveCriteria();
-        $criteria->setGroupby('UPPER(LEFT(' . $name . ',1))');
+        $criteria->setGroupby('UPPER(LEFT('.$name.',1))');
         $countsByLetters = $myObject->getHandler($activeObject)->getCounts($criteria);
         // Fill alphabet array
 
@@ -791,7 +791,7 @@ 
                 $letter_array['count']  = $countsByLetters[$letter];
                 //            $letter_array['url']    = "" . XOOPS_URL . "/modules/" . $helper->getModule()->dirname() . "/viewcat.php?list={$letter}";
                 //                $letter_array['url'] = '' . XOOPS_URL . '/modules/' . $myObject->getModule()->dirname() . '/'.$file.'?f='.$name."&amp;l=1&amp;w={$letter}%25&amp;o=".$name;
-                $letter_array['url'] = '' . XOOPS_URL . '/modules/' . $myObject->getModule()->dirname() . '/' . $link . $letter . $link2;
+                $letter_array['url'] = ''.XOOPS_URL.'/modules/'.$myObject->getModule()->dirname().'/'.$link.$letter.$link2;
             } else {
                 $letter_array['letter'] = $letter;
                 $letter_array['count']  = 0;
@@ -810,7 +810,7 @@ 
         $letterschoiceTpl          = new \XoopsTpl();
         $letterschoiceTpl->caching = false; // Disable cache
         $letterschoiceTpl->assign('alphabet', $alphabet_array);
-        $html = $letterschoiceTpl->fetch('db:' . $myObject->getModule()->dirname() . '_common_letterschoice.tpl');
+        $html = $letterschoiceTpl->fetch('db:'.$myObject->getModule()->dirname().'_common_letterschoice.tpl');
         unset($letterschoiceTpl);
 
         return $html;
@@ -847,7 +847,7 @@ 
         if (!$withLink) {
             return $pedigreeModuleName;
         } else {
-            return '<a href="' . PEDIGREE_URL . '/">{$pedigreeModuleName}</a>';
+            return '<a href="'.PEDIGREE_URL.'/">{$pedigreeModuleName}</a>';
         }
     }
 
@@ -868,7 +868,7 @@ 
         $GLOBALS['xoopsDB'] = \XoopsDatabaseFactory::getDatabaseConnection();
         $realName           = $GLOBALS['xoopsDB']->prefix($table);
 
-        $sql = 'SHOW TABLES FROM ' . XOOPS_DB_NAME;
+        $sql = 'SHOW TABLES FROM '.XOOPS_DB_NAME;
         $ret = $GLOBALS['xoopsDB']->queryF($sql);
 
         while (false !== (list($m_table) = $GLOBALS['xoopsDB']->fetchRow($ret))) {
@@ -972,10 +972,10 @@ 
         $queryString = $_SERVER['QUERY_STRING'];
 
         if ('' != $queryString) {
-            $queryString = '?' . $queryString;
+            $queryString = '?'.$queryString;
         }
 
-        $currentURL = $http . $httpHost . $phpSelf . $queryString;
+        $currentURL = $http.$httpHost.$phpSelf.$queryString;
 
         $urls                = [];
         $urls['http']        = $http;

class/HtmlInputAbstract.php

Spacing +1 added lines, -1 removed lines

@@ -85,7 +85,7 @@
     {
         $ret = [];
         global $xoopsDB;
-        $SQL    = 'SELECT * FROM ' . $GLOBALS['xoopsDB']->prefix('pedigree_lookup' . $fieldnumber) . " ORDER BY 'order'";
+        $SQL    = 'SELECT * FROM '.$GLOBALS['xoopsDB']->prefix('pedigree_lookup'.$fieldnumber)." ORDER BY 'order'";
         $result = $GLOBALS['xoopsDB']->query($SQL);
         while (false !== ($row = $GLOBALS['xoopsDB']->fetchArray($result))) {
             $ret[] = ['id' => $row['id'], 'value' => $row['value']];

class/TextBox.php

Spacing +6 added lines, -6 removed lines

@@ -44,17 +44,17 @@ 
     {
         $this->fieldnumber  = $parentObject->getId();
         $this->fieldname    = $parentObject->fieldname;
-        $this->value        = $animalObject->{'user' . $this->fieldnumber};
+        $this->value        = $animalObject->{'user'.$this->fieldnumber};
         $this->defaultvalue = $parentObject->defaultvalue;
         $this->lookuptable  = $parentObject->lookuptable;
         if ('1' == $this->lookuptable) {
-            xoops_error('No lookuptable may be specified for userfield ' . $this->fieldnumber, get_class($this));
+            xoops_error('No lookuptable may be specified for userfield '.$this->fieldnumber, get_class($this));
         }
         if ('1' == $parentObject->viewinadvanced) {
-            xoops_error('userfield ' . $this->fieldnumber . ' cannot be shown in advanced info', get_class($this));
+            xoops_error('userfield '.$this->fieldnumber.' cannot be shown in advanced info', get_class($this));
         }
         if ('1' == $parentObject->viewinpie) {
-            xoops_error('A Pie-chart cannot be specified for userfield ' . $this->fieldnumber, get_class($this));
+            xoops_error('A Pie-chart cannot be specified for userfield '.$this->fieldnumber, get_class($this));
         }
     }
 
@@ -64,7 +64,7 @@ 
      */
     public function editField()
     {
-        $textbox = new \XoopsFormText('<b>' . $this->fieldname . '</b>', 'user' . $this->fieldnumber, $size = 50, $maxsize = 50, $value = $this->value);
+        $textbox = new \XoopsFormText('<b>'.$this->fieldname.'</b>', 'user'.$this->fieldnumber, $size = 50, $maxsize = 50, $value = $this->value);
 
         return $textbox;
     }
@@ -76,7 +76,7 @@ 
      */
     public function newField($name = '')
     {
-        $textbox = new \XoopsFormText('<b>' . $this->fieldname . '</b>', $name . 'user' . $this->fieldnumber, $size = 50, $maxsize = 50, $value = $this->defaultvalue);
+        $textbox = new \XoopsFormText('<b>'.$this->fieldname.'</b>', $name.'user'.$this->fieldnumber, $size = 50, $maxsize = 50, $value = $this->defaultvalue);
 
         return $textbox;
     }

class/Fields.php

Indentation +3 added lines, -3 removed lines

@@ -96,9 +96,9 @@
         $currentType = $this->getVar('litter') ? 'litter' : 'generallitter';
         $litterRadio = new \XoopsFormRadio(_AM_PEDIGREE_PEDIGREE_CONFIG_LITTER_TYPE, 'litterType', $currentType);
         $litterRadio->addOptionArray([
-                                         'litter'        => _AM_PEDIGREE_PEDIGREE_CONFIG_LITTER,
-                                         'generallitter' => _AM_PEDIGREE_PEDIGREE_CONFIG_GENERALLITTER
-                                     ]);
+                                            'litter'        => _AM_PEDIGREE_PEDIGREE_CONFIG_LITTER,
+                                            'generallitter' => _AM_PEDIGREE_PEDIGREE_CONFIG_GENERALLITTER
+                                        ]);
         $form->addElement($litterRadio, false);
         //        $form->addElement(new \XoopsFormRadioYN(_AM_PEDIGREE_PEDIGREE_CONFIG_LITTER, "Litter", $this->getVar("Litter")), false);
         //        $form->addElement(new \XoopsFormRadioYN(_AM_PEDIGREE_PEDIGREE_CONFIG_GENERALLITTER, "Generallitter", $this->getVar("Generallitter")), false);

Spacing +10 added lines, -10 removed lines

@@ -164,7 +164,7 @@ 
      */
     public function isActive()
     {
-        return (1 == $this->getVar('isActive'));// ? true : false;
+        return (1 == $this->getVar('isActive')); // ? true : false;
     }
 
     /**
@@ -172,7 +172,7 @@ 
      */
     public function inAdvanced()
     {
-        return (1 == $this->getVar('viewInAdvanced'));// ? true : false;
+        return (1 == $this->getVar('viewInAdvanced')); // ? true : false;
     }
 
     /**
@@ -180,7 +180,7 @@ 
      */
     public function isLocked()
     {
-        return (1 == $this->getVar('locked'));// ? true : false;
+        return (1 == $this->getVar('locked')); // ? true : false;
     }
 
     /**
@@ -188,7 +188,7 @@ 
      */
     public function hasSearch()
     {
-        return (1 == $this->getVar('hasSearch'));// ? true : false;
+        return (1 == $this->getVar('hasSearch')); // ? true : false;
     }
 
     /**
@@ -196,7 +196,7 @@ 
      */
     public function addLitter()
     {
-        return (1 == $this->getVar('litter'));// ? true : false;
+        return (1 == $this->getVar('litter')); // ? true : false;
     }
 
     /**
@@ -204,7 +204,7 @@ 
      */
     public function generalLitter()
     {
-        return (1 == $this->getVar('generalLitter'));// ? true : false;
+        return (1 == $this->getVar('generalLitter')); // ? true : false;
     }
 
     /**
@@ -212,7 +212,7 @@ 
      */
     public function hasLookup()
     {
-        return (1 == $this->getVar('lookupTable'));// ? true : false;
+        return (1 == $this->getVar('lookupTable')); // ? true : false;
     }
 
     /**
@@ -228,7 +228,7 @@ 
      */
     public function inPie()
     {
-        return (1 == $this->getVar('viewInPie'));// ? true : false;
+        return (1 == $this->getVar('viewInPie')); // ? true : false;
     }
 
     /**
@@ -236,7 +236,7 @@ 
      */
     public function inPedigree()
     {
-        return (1 == $this->getVar('viewInPedigree'));// ? true : false;
+        return (1 == $this->getVar('viewInPedigree')); // ? true : false;
     }
 
     /**
@@ -244,7 +244,7 @@ 
      */
     public function inList()
     {
-        return (1 == $this->getVar('viewInList'));// ? true : false;
+        return (1 == $this->getVar('viewInList')); // ? true : false;
     }
 
     /**

class/Field.php

Spacing +2 added lines, -2 removed lines

@@ -161,7 +161,7 @@ 
     {
         $ret = [];
         global $xoopsDB;
-        $SQL    = 'SELECT * FROM ' . $GLOBALS['xoopsDB']->prefix("pedigree_lookup{$fieldnumber}") . " ORDER BY 'order'";
+        $SQL    = 'SELECT * FROM '.$GLOBALS['xoopsDB']->prefix("pedigree_lookup{$fieldnumber}")." ORDER BY 'order'";
         $result = $GLOBALS['xoopsDB']->query($SQL);
         while (false !== ($row = $GLOBALS['xoopsDB']->fetchArray($result))) {
             $ret[] = ['id' => $row['id'], 'value' => $row['value']];
@@ -186,7 +186,7 @@ 
      */
     public function showField()
     {
-        return $this->fieldname . ' : ' . $this->value;
+        return $this->fieldname.' : '.$this->value;
     }
 
     /**