ggoffy / wggithub

class/Common/Breadcrumb.php

Spacing +1 added lines, -1 removed lines

@@ -73,7 +73,7 @@
         require $GLOBALS['xoops']->path('class/template.php');
         $breadcrumbTpl = new \XoopsTpl();
         $breadcrumbTpl->assign('breadcrumb', $this->bread);
-        $html = $breadcrumbTpl->fetch('db:' . $this->dirname . '_common_breadcrumb.tpl');
+        $html = $breadcrumbTpl->fetch('db:'.$this->dirname.'_common_breadcrumb.tpl');
         unset($breadcrumbTpl);
 
         return $html;

class/Common/Resizer.php

Indentation +2 added lines, -2 removed lines

@@ -65,8 +65,8 @@
 
         if ($width > $this->maxWidth || $height > $this->maxHeight) {
             // recalc image size based on this->maxWidth/this->maxHeight
-			$new_width  = 0;
-			$new_height = 0;
+            $new_width  = 0;
+            $new_height = 0;
             if ($width > $height) {
                 if ($width < $this->maxWidth) {
                     $new_width = $width;

Spacing +14 added lines, -14 removed lines

@@ -146,14 +146,14 @@ 
         $max_height_resize = $this->maxHeight;
         if ($original_w > $original_h) {
             $max_height_ratio = $this->maxHeight / $original_h;
-            $max_width_resize = (int)\round($original_w * $max_height_ratio);
+            $max_width_resize = (int) \round($original_w * $max_height_ratio);
         } else {
             $max_width_ratio   = $this->maxWidth / $original_w;
-            $max_height_resize = (int)\round($original_h * $max_width_ratio);
+            $max_height_resize = (int) \round($original_h * $max_width_ratio);
         }
         if ($max_width_resize < $this->maxWidth) {
             $max_height_ratio  = $this->maxWidth / $max_width_resize;
-            $max_height_resize = (int)\round($this->maxHeight * $max_height_ratio);
+            $max_height_resize = (int) \round($this->maxHeight * $max_height_ratio);
             $max_width_resize  = $this->maxWidth;
         }
 
@@ -168,9 +168,9 @@ 
         $max_width_offset  = 0;
         $max_height_offset = 0;
         if ($this->maxWidth < $max_width_resize) {
-            $max_width_offset = (int)\round(($max_width_resize - $this->maxWidth) / 2);
+            $max_width_offset = (int) \round(($max_width_resize - $this->maxWidth) / 2);
         } else {
-            $max_height_offset = (int)\round(($max_height_resize - $this->maxHeight) / 2);
+            $max_height_offset = (int) \round(($max_height_resize - $this->maxHeight) / 2);
         }
 
         if (!imagecopy($final, $thumb, 0, 0, $max_width_offset, $max_height_offset, $max_width_resize, $max_height_resize)) {
@@ -189,10 +189,10 @@ 
         $dest = \imagecreatefromjpeg($this->endFile);
         $src  = \imagecreatefromjpeg($this->sourceFile);
         if (4 == $this->mergeType) {
-            $imgWidth  = (int)\round($this->maxWidth / 2 - 1);
-            $imgHeight = (int)\round($this->maxHeight / 2 - 1);
-            $posCol2   = (int)\round($this->maxWidth / 2 + 1);
-            $posRow2   = (int)\round($this->maxHeight / 2 + 1);
+            $imgWidth  = (int) \round($this->maxWidth / 2 - 1);
+            $imgHeight = (int) \round($this->maxHeight / 2 - 1);
+            $posCol2   = (int) \round($this->maxWidth / 2 + 1);
+            $posRow2   = (int) \round($this->maxHeight / 2 + 1);
             switch ($this->mergePos) {
                 case 1:
                     imagecopy($dest, $src, 0, 0, 0, 0, $imgWidth, $imgHeight); //top left
@@ -209,11 +209,11 @@ 
             }
         }
         if (6 == $this->mergeType) {
-            $imgWidth  = (int)\round($this->maxWidth / 3 - 1);
-            $imgHeight = (int)\round($this->maxHeight / 2 - 1);
-            $posCol2   = (int)\round($this->maxWidth / 3 + 1);
-            $posCol3   = $posCol2 + (int)\round($this->maxWidth / 3 + 1);
-            $posRow2   = (int)\round($this->maxHeight / 2 + 1);
+            $imgWidth  = (int) \round($this->maxWidth / 3 - 1);
+            $imgHeight = (int) \round($this->maxHeight / 2 - 1);
+            $posCol2   = (int) \round($this->maxWidth / 3 + 1);
+            $posCol3   = $posCol2 + (int) \round($this->maxWidth / 3 + 1);
+            $posRow2   = (int) \round($this->maxHeight / 2 + 1);
 
             switch ($this->mergePos) {
                 case 1:

class/Common/Migrate.php

Spacing +1 added lines, -1 removed lines

@@ -88,7 +88,7 @@
                 $this->synchronizeTable($tableName);
                 $updateTable = $GLOBALS['xoopsDB']->prefix($tableName);
                 $joinTable   = $GLOBALS['xoopsDB']->prefix($srcTableName);
-                $sql         = "UPDATE `$updateTable` t1 INNER JOIN `$joinTable` t2 ON t1.post_id = t2.post_id \n" . "SET t1.dohtml = t2.dohtml,  t1.dosmiley = t2.dosmiley, t1.doxcode = t2.doxcode\n" . '  , t1.doimage = t2.doimage, t1.dobr = t2.dobr';
+                $sql         = "UPDATE `$updateTable` t1 INNER JOIN `$joinTable` t2 ON t1.post_id = t2.post_id \n"."SET t1.dohtml = t2.dohtml,  t1.dosmiley = t2.dosmiley, t1.doxcode = t2.doxcode\n".'  , t1.doimage = t2.doimage, t1.dobr = t2.dobr';
                 $this->tableHandler->addToQueue($sql);
             }
         }

class/Common/Configurator.php

Spacing +1 added lines, -1 removed lines

@@ -51,7 +51,7 @@
         //        require \dirname(\dirname(__DIR__)) . '/config/config.php';
         //        $config = getConfig();
 
-        $config = include dirname(__DIR__, 2) . '/config/config.php';
+        $config = include dirname(__DIR__, 2).'/config/config.php';
 
         $this->name            = $config->name;
         $this->paths           = $config->paths;

class/Common/ModuleFeedback.php

Spacing +15 added lines, -15 removed lines

@@ -67,29 +67,29 @@ 
         $moduleDirNameUpper = \mb_strtoupper($moduleDirName);
         // Get Theme Form
         \xoops_load('XoopsFormLoader');
-        $form = new \XoopsThemeForm(\constant('CO_' . $moduleDirNameUpper . '_' . 'FB_FORM_TITLE'), 'formfeedback', 'feedback.php', 'post', true);
+        $form = new \XoopsThemeForm(\constant('CO_'.$moduleDirNameUpper.'_'.'FB_FORM_TITLE'), 'formfeedback', 'feedback.php', 'post', true);
         $form->setExtra('enctype="multipart/form-data"');
 
-        $recipient = new \XoopsFormText(\constant('CO_' . $moduleDirNameUpper . '_' . 'FB_RECIPIENT'), 'recipient', 50, 255, $GLOBALS['xoopsModule']->getInfo('author_mail'));
+        $recipient = new \XoopsFormText(\constant('CO_'.$moduleDirNameUpper.'_'.'FB_RECIPIENT'), 'recipient', 50, 255, $GLOBALS['xoopsModule']->getInfo('author_mail'));
         $recipient->setExtra('disabled="disabled"');
         $form->addElement($recipient);
-        $your_name = new \XoopsFormText(\constant('CO_' . $moduleDirNameUpper . '_' . 'FB_NAME'), 'your_name', 50, 255, $this->name);
-        $your_name->setExtra('placeholder="' . \constant('CO_' . $moduleDirNameUpper . '_' . 'FB_NAME_PLACEHOLER') . '"');
+        $your_name = new \XoopsFormText(\constant('CO_'.$moduleDirNameUpper.'_'.'FB_NAME'), 'your_name', 50, 255, $this->name);
+        $your_name->setExtra('placeholder="'.\constant('CO_'.$moduleDirNameUpper.'_'.'FB_NAME_PLACEHOLER').'"');
         $form->addElement($your_name);
-        $your_site = new \XoopsFormText(\constant('CO_' . $moduleDirNameUpper . '_' . 'FB_SITE'), 'your_site', 50, 255, $this->site);
-        $your_site->setExtra('placeholder="' . \constant('CO_' . $moduleDirNameUpper . '_' . 'FB_SITE_PLACEHOLER') . '"');
+        $your_site = new \XoopsFormText(\constant('CO_'.$moduleDirNameUpper.'_'.'FB_SITE'), 'your_site', 50, 255, $this->site);
+        $your_site->setExtra('placeholder="'.\constant('CO_'.$moduleDirNameUpper.'_'.'FB_SITE_PLACEHOLER').'"');
         $form->addElement($your_site);
-        $your_mail = new \XoopsFormText(\constant('CO_' . $moduleDirNameUpper . '_' . 'FB_MAIL'), 'your_mail', 50, 255, $this->email);
-        $your_mail->setExtra('placeholder="' . \constant('CO_' . $moduleDirNameUpper . '_' . 'FB_MAIL_PLACEHOLER') . '"');
+        $your_mail = new \XoopsFormText(\constant('CO_'.$moduleDirNameUpper.'_'.'FB_MAIL'), 'your_mail', 50, 255, $this->email);
+        $your_mail->setExtra('placeholder="'.\constant('CO_'.$moduleDirNameUpper.'_'.'FB_MAIL_PLACEHOLER').'"');
         $form->addElement($your_mail);
 
-        $fbtypeSelect = new \XoopsFormSelect(\constant('CO_' . $moduleDirNameUpper . '_' . 'FB_TYPE'), 'fb_type', $this->type);
+        $fbtypeSelect = new \XoopsFormSelect(\constant('CO_'.$moduleDirNameUpper.'_'.'FB_TYPE'), 'fb_type', $this->type);
         $fbtypeSelect->addOption('', '');
-        $fbtypeSelect->addOption(\constant('CO_' . $moduleDirNameUpper . '_' . 'FB_TYPE_SUGGESTION'), \constant('CO_' . $moduleDirNameUpper . '_' . 'FB_TYPE_SUGGESTION'));
-        $fbtypeSelect->addOption(\constant('CO_' . $moduleDirNameUpper . '_' . 'FB_TYPE_BUGS'), \constant('CO_' . $moduleDirNameUpper . '_' . 'FB_TYPE_BUGS'));
-        $fbtypeSelect->addOption(\constant('CO_' . $moduleDirNameUpper . '_' . 'FB_TYPE_TESTIMONIAL'), \constant('CO_' . $moduleDirNameUpper . '_' . 'FB_TYPE_TESTIMONIAL'));
-        $fbtypeSelect->addOption(\constant('CO_' . $moduleDirNameUpper . '_' . 'FB_TYPE_FEATURES'), \constant('CO_' . $moduleDirNameUpper . '_' . 'FB_TYPE_FEATURES'));
-        $fbtypeSelect->addOption(\constant('CO_' . $moduleDirNameUpper . '_' . 'FB_TYPE_OTHERS'), \constant('CO_' . $moduleDirNameUpper . '_' . 'FB_TYPE_OTHERS'));
+        $fbtypeSelect->addOption(\constant('CO_'.$moduleDirNameUpper.'_'.'FB_TYPE_SUGGESTION'), \constant('CO_'.$moduleDirNameUpper.'_'.'FB_TYPE_SUGGESTION'));
+        $fbtypeSelect->addOption(\constant('CO_'.$moduleDirNameUpper.'_'.'FB_TYPE_BUGS'), \constant('CO_'.$moduleDirNameUpper.'_'.'FB_TYPE_BUGS'));
+        $fbtypeSelect->addOption(\constant('CO_'.$moduleDirNameUpper.'_'.'FB_TYPE_TESTIMONIAL'), \constant('CO_'.$moduleDirNameUpper.'_'.'FB_TYPE_TESTIMONIAL'));
+        $fbtypeSelect->addOption(\constant('CO_'.$moduleDirNameUpper.'_'.'FB_TYPE_FEATURES'), \constant('CO_'.$moduleDirNameUpper.'_'.'FB_TYPE_FEATURES'));
+        $fbtypeSelect->addOption(\constant('CO_'.$moduleDirNameUpper.'_'.'FB_TYPE_OTHERS'), \constant('CO_'.$moduleDirNameUpper.'_'.'FB_TYPE_OTHERS'));
         $form->addElement($fbtypeSelect, true);
 
         $editorConfigs           = [];
@@ -104,7 +104,7 @@ 
         $configHandler           = \xoops_getHandler('config');
         $config                  = &$configHandler->getConfigsByCat(0, $module->getVar('mid'));
         $editorConfigs['editor'] = $config['general_editor'];
-        $editor                  = new \XoopsFormEditor(\constant('CO_' . $moduleDirNameUpper . '_' . 'FB_TYPE_CONTENT'), 'fb_content', $editorConfigs);
+        $editor                  = new \XoopsFormEditor(\constant('CO_'.$moduleDirNameUpper.'_'.'FB_TYPE_CONTENT'), 'fb_content', $editorConfigs);
         $form->addElement($editor, true);
 
         $form->addElement(new \XoopsFormHidden('op', 'send'));

class/Common/FilesManagement.php

Spacing +11 added lines, -11 removed lines

@@ -35,7 +35,7 @@ 
                     throw new \RuntimeException(\sprintf('Unable to create the %s directory', $folder));
                 }
 
-                file_put_contents($folder . '/index.html', '<script>history.go(-1);</script>');
+                file_put_contents($folder.'/index.html', '<script>history.go(-1);</script>');
             }
         } catch (\Exception $e) {
             echo 'Caught exception: ', $e->getMessage(), '<br>';
@@ -61,14 +61,14 @@ 
         $dir = \opendir($src);
         //        @\mkdir($dst);
         if (!@\mkdir($dst) && !\is_dir($dst)) {
-            throw new \RuntimeException('The directory ' . $dst . ' could not be created.');
+            throw new \RuntimeException('The directory '.$dst.' could not be created.');
         }
         while (false !== ($file = \readdir($dir))) {
             if (('.' !== $file) && ('..' !== $file)) {
-                if (\is_dir($src . '/' . $file)) {
-                    self::recurseCopy($src . '/' . $file, $dst . '/' . $file);
+                if (\is_dir($src.'/'.$file)) {
+                    self::recurseCopy($src.'/'.$file, $dst.'/'.$file);
                 } else {
-                    \copy($src . '/' . $file, $dst . '/' . $file);
+                    \copy($src.'/'.$file, $dst.'/'.$file);
                 }
             }
         }
@@ -207,7 +207,7 @@ 
                 self::rrmdir($fObj->getPathname());
             }
         }
-        $iterator = null;   // clear iterator Obj to close file/directory
+        $iterator = null; // clear iterator Obj to close file/directory
         return \rmdir($src); // remove the directory & return results
     }
 
@@ -240,14 +240,14 @@ 
         $iterator = new \DirectoryIterator($src);
         foreach ($iterator as $fObj) {
             if ($fObj->isFile()) {
-                \rename($fObj->getPathname(), "{$dest}/" . $fObj->getFilename());
+                \rename($fObj->getPathname(), "{$dest}/".$fObj->getFilename());
             } elseif (!$fObj->isDot() && $fObj->isDir()) {
                 // Try recursively on directory
-                self::rmove($fObj->getPathname(), "{$dest}/" . $fObj->getFilename());
+                self::rmove($fObj->getPathname(), "{$dest}/".$fObj->getFilename());
                 //                \rmdir($fObj->getPath()); // now delete the directory
             }
         }
-        $iterator = null;   // clear iterator Obj to close file/directory
+        $iterator = null; // clear iterator Obj to close file/directory
         return \rmdir($src); // remove the directory & return results
     }
 
@@ -283,9 +283,9 @@ 
         $iterator = new \DirectoryIterator($src);
         foreach ($iterator as $fObj) {
             if ($fObj->isFile()) {
-                \copy($fObj->getPathname(), "{$dest}/" . $fObj->getFilename());
+                \copy($fObj->getPathname(), "{$dest}/".$fObj->getFilename());
             } elseif (!$fObj->isDot() && $fObj->isDir()) {
-                self::rcopy($fObj->getPathname(), "{$dest}/" . $fObj->getFilename());
+                self::rcopy($fObj->getPathname(), "{$dest}/".$fObj->getFilename());
             }
         }
 

class/Common/VersionChecks.php

Spacing +7 added lines, -7 removed lines

@@ -40,13 +40,13 @@ 
         //check for minimum XOOPS version
         $currentVer = mb_substr(XOOPS_VERSION, 6); // get the numeric part of string
         if (null === $requiredVer) {
-            $requiredVer = '' . $module->getInfo('min_xoops'); //making sure it's a string
+            $requiredVer = ''.$module->getInfo('min_xoops'); //making sure it's a string
         }
         $success = true;
 
         if (\version_compare($currentVer, $requiredVer, '<')) {
             $success = false;
-            $module->setErrors(\sprintf(\constant('CO_' . $moduleDirNameUpper . '_ERROR_BAD_XOOPS'), $requiredVer, $currentVer));
+            $module->setErrors(\sprintf(\constant('CO_'.$moduleDirNameUpper.'_ERROR_BAD_XOOPS'), $requiredVer, $currentVer));
         }
 
         return $success;
@@ -75,7 +75,7 @@ 
 
         if (false !== $reqVer && '' !== $reqVer) {
             if (\version_compare($verNum, $reqVer, '<')) {
-                $module->setErrors(\sprintf(\constant('CO_' . $moduleDirNameUpper . '_ERROR_BAD_PHP'), $reqVer, $verNum));
+                $module->setErrors(\sprintf(\constant('CO_'.$moduleDirNameUpper.'_ERROR_BAD_PHP'), $reqVer, $verNum));
                 $success = false;
             }
         }
@@ -97,7 +97,7 @@ 
         $moduleDirName      = \basename(dirname(__DIR__, 2));
         $moduleDirNameUpper = \mb_strtoupper($moduleDirName);
         $update             = '';
-        $repository         = 'XoopsModules25x/' . $moduleDirName;
+        $repository         = 'XoopsModules25x/'.$moduleDirName;
         //        $repository         = 'XoopsModules25x/publisher'; //for testing only
         $ret             = '';
         $infoReleasesUrl = "https://api.github.com/repos/$repository/releases";
@@ -111,14 +111,14 @@ 
                 if (false === $curlReturn) {
                     \trigger_error(\curl_error($curlHandle));
                 } elseif (mb_strpos($curlReturn, 'Not Found')) {
-                    \trigger_error('Repository Not Found: ' . $infoReleasesUrl);
+                    \trigger_error('Repository Not Found: '.$infoReleasesUrl);
                 } else {
                     $file              = \json_decode($curlReturn, false);
                     $latestVersionLink = \sprintf("https://github.com/$repository/archive/%s.zip", $file ? reset($file)->tag_name : $default);
                     $latestVersion     = $file[0]->tag_name;
                     $prerelease        = $file[0]->prerelease;
                     if ('master' !== $latestVersionLink) {
-                        $update = \constant('CO_' . $moduleDirNameUpper . '_' . 'NEW_VERSION') . $latestVersion;
+                        $update = \constant('CO_'.$moduleDirNameUpper.'_'.'NEW_VERSION').$latestVersion;
                     }
                     //"PHP-standardized" version
                     $latestVersion = \mb_strtolower($latestVersion);
@@ -126,7 +126,7 @@ 
                         $latestVersion = \str_replace('_', '', \mb_strtolower($latestVersion));
                         $latestVersion = \str_replace('final', '', \mb_strtolower($latestVersion));
                     }
-                    $moduleVersion = ($helper->getModule()->getInfo('version') . '_' . $helper->getModule()->getInfo('module_status'));
+                    $moduleVersion = ($helper->getModule()->getInfo('version').'_'.$helper->getModule()->getInfo('module_status'));
                     //"PHP-standardized" version
                     $moduleVersion = \str_replace(' ', '', \mb_strtolower($moduleVersion));
                     //                    $moduleVersion = '1.0'; //for testing only

class/RequestsHandler.php

Indentation +90 added lines, -90 removed lines

@@ -31,96 +31,96 @@
  */
 class RequestsHandler extends \XoopsPersistableObjectHandler
 {
-	/**
-	 * Constructor
-	 *
-	 * @param \XoopsDatabase $db
-	 */
-	public function __construct(\XoopsDatabase $db)
-	{
-		parent::__construct($db, 'wggithub_requests', Requests::class, 'req_id', 'req_request');
-	}
-
-	/**
-	 * @param bool $isNew
-	 *
-	 * @return object
-	 */
-	public function create($isNew = true)
-	{
-		return parent::create($isNew);
-	}
-
-	/**
-	 * retrieve a field
-	 *
-	 * @param int $i field id
-	 * @param null fields
-	 * @return mixed reference to the {@link Get} object
-	 */
-	public function get($i = null, $fields = null)
-	{
-		return parent::get($i, $fields);
-	}
-
-	/**
-	 * get inserted id
-	 *
-	 * @param null
-	 * @return int reference to the {@link Get} object
-	 */
-	public function getInsertId()
-	{
-		return $this->db->getInsertId();
-	}
-
-	/**
-	 * Get Count Requests in the database
-	 * @param int    $start
-	 * @param int    $limit
-	 * @param string $sort
-	 * @param string $order
-	 * @return int
-	 */
-	public function getCountRequests($start = 0, $limit = 0, $sort = 'req_id ASC, req_request', $order = 'ASC')
-	{
-		$crCountRequests = new \CriteriaCompo();
-		$crCountRequests = $this->getRequestsCriteria($crCountRequests, $start, $limit, $sort, $order);
-		return $this->getCount($crCountRequests);
-	}
-
-	/**
-	 * Get All Requests in the database
-	 * @param int    $start
-	 * @param int    $limit
-	 * @param string $sort
-	 * @param string $order
-	 * @return array
-	 */
-	public function getAllRequests($start = 0, $limit = 0, $sort = 'req_id ASC, req_request', $order = 'ASC')
-	{
-		$crAllRequests = new \CriteriaCompo();
-		$crAllRequests = $this->getRequestsCriteria($crAllRequests, $start, $limit, $sort, $order);
-		return $this->getAll($crAllRequests);
-	}
-
-	/**
-	 * Get Criteria Requests
-	 * @param        $crRequests
-	 * @param int    $start
-	 * @param int    $limit
-	 * @param string $sort
-	 * @param string $order
-	 * @return int
-	 */
-	private function getRequestsCriteria($crRequests, $start, $limit, $sort, $order)
-	{
-		$crRequests->setStart($start);
-		$crRequests->setLimit($limit);
-		$crRequests->setSort($sort);
-		$crRequests->setOrder($order);
-		return $crRequests;
-	}
+    /**
+     * Constructor
+     *
+     * @param \XoopsDatabase $db
+     */
+    public function __construct(\XoopsDatabase $db)
+    {
+        parent::__construct($db, 'wggithub_requests', Requests::class, 'req_id', 'req_request');
+    }
+
+    /**
+     * @param bool $isNew
+     *
+     * @return object
+     */
+    public function create($isNew = true)
+    {
+        return parent::create($isNew);
+    }
+
+    /**
+     * retrieve a field
+     *
+     * @param int $i field id
+     * @param null fields
+     * @return mixed reference to the {@link Get} object
+     */
+    public function get($i = null, $fields = null)
+    {
+        return parent::get($i, $fields);
+    }
+
+    /**
+     * get inserted id
+     *
+     * @param null
+     * @return int reference to the {@link Get} object
+     */
+    public function getInsertId()
+    {
+        return $this->db->getInsertId();
+    }
+
+    /**
+     * Get Count Requests in the database
+     * @param int    $start
+     * @param int    $limit
+     * @param string $sort
+     * @param string $order
+     * @return int
+     */
+    public function getCountRequests($start = 0, $limit = 0, $sort = 'req_id ASC, req_request', $order = 'ASC')
+    {
+        $crCountRequests = new \CriteriaCompo();
+        $crCountRequests = $this->getRequestsCriteria($crCountRequests, $start, $limit, $sort, $order);
+        return $this->getCount($crCountRequests);
+    }
+
+    /**
+     * Get All Requests in the database
+     * @param int    $start
+     * @param int    $limit
+     * @param string $sort
+     * @param string $order
+     * @return array
+     */
+    public function getAllRequests($start = 0, $limit = 0, $sort = 'req_id ASC, req_request', $order = 'ASC')
+    {
+        $crAllRequests = new \CriteriaCompo();
+        $crAllRequests = $this->getRequestsCriteria($crAllRequests, $start, $limit, $sort, $order);
+        return $this->getAll($crAllRequests);
+    }
+
+    /**
+     * Get Criteria Requests
+     * @param        $crRequests
+     * @param int    $start
+     * @param int    $limit
+     * @param string $sort
+     * @param string $order
+     * @return int
+     */
+    private function getRequestsCriteria($crRequests, $start, $limit, $sort, $order)
+    {
+        $crRequests->setStart($start);
+        $crRequests->setLimit($limit);
+        $crRequests->setSort($sort);
+        $crRequests->setOrder($order);
+        return $crRequests;
+    }
 
     /**
      * Update table requests

Spacing +1 added lines, -1 removed lines

@@ -140,7 +140,7 @@
         $requestsObj = $requestsHandler->create();
         $requestsObj->setVar('req_request', $request);
         $requestsObj->setVar('req_result', $result);
-        $requestsObj->setVar('req_datecreated',time());
+        $requestsObj->setVar('req_datecreated', time());
         $requestsObj->setVar('req_submitter', $submitter);
         // Insert Data
         if ($requestsHandler->insert($requestsObj)) {

class/Github/GithubClient.php

Spacing +12 added lines, -12 removed lines

@@ -45,8 +45,8 @@ 
         $ch = \curl_init();
         //set the url, number of POST vars, POST data
         \curl_setopt($ch, \CURLOPT_URL, $url);
-        \curl_setopt($ch, \CURLOPT_HTTPAUTH, 'token ' . $this->tokenAuth);
-        \curl_setopt($ch, \CURLOPT_USERPWD, $this->userAuth . ':' . $this->tokenAuth);
+        \curl_setopt($ch, \CURLOPT_HTTPAUTH, 'token '.$this->tokenAuth);
+        \curl_setopt($ch, \CURLOPT_USERPWD, $this->userAuth.':'.$this->tokenAuth);
         \curl_setopt($ch, \CURLOPT_CUSTOMREQUEST, 'GET');
         \curl_setopt($ch, \CURLOPT_RETURNTRANSFER, true);
         \curl_setopt($ch, \CURLOPT_CONNECTTIMEOUT, 25);
@@ -60,7 +60,7 @@ 
         \curl_setopt($ch, \CURLOPT_POST, 1);
         $result = \curl_exec($ch);
         $info = \curl_getinfo($ch);
-        if (($errMsg = \curl_error($ch)) || !\in_array((int)$info['http_code'], [200, 201], true)) {
+        if (($errMsg = \curl_error($ch)) || !\in_array((int) $info['http_code'], [200, 201], true)) {
             $error = $throwError;
         }
         \curl_close($ch);
@@ -78,7 +78,7 @@ 
         if ($requestsHandler->insert($requestsObj, true)) {
             $reqId = $requestsObj->getNewInsertedIdRequests();
         } else {
-            throw new \RuntimeException('"' . \_MA_WGGITHUB_READGH_ERROR_INSERTREQ . '"');
+            throw new \RuntimeException('"'.\_MA_WGGITHUB_READGH_ERROR_INSERTREQ.'"');
         }
         unset($requestsObj);
 
@@ -90,19 +90,19 @@ 
             $requestsObj = $requestsHandler->get($reqId);
             $requestsObj->setVar('req_result', $result);
             if (!$requestsHandler->insert($requestsObj, true)) {
-                throw new \RuntimeException('"' . \_MA_WGGITHUB_READGH_ERROR_INSERTREQ . '"');
+                throw new \RuntimeException('"'.\_MA_WGGITHUB_READGH_ERROR_INSERTREQ.'"');
             }
             unset($requestsObj);
 
             if ($this->debug) {
-                echo '<br>Error:' . $result;
+                echo '<br>Error:'.$result;
             }
 
             //catch common errors
-            switch ((int)$info['http_code']) {
+            switch ((int) $info['http_code']) {
                 case 401:
                     $this->apiErrorMisc = true;
-                    throw new \RuntimeException('"' . \_MA_WGGITHUB_READGH_ERROR_API_401 . '"');
+                    throw new \RuntimeException('"'.\_MA_WGGITHUB_READGH_ERROR_API_401.'"');
                     break;
                 case 403:
                     if (\strpos($result, 'API rate limit exceeded') > 0) {
@@ -111,21 +111,21 @@ 
                         return false;
                     } else {
                         $this->apiErrorMisc = true;
-                        throw new \RuntimeException('"' . \_MA_WGGITHUB_READGH_ERROR_API_403 . '"');
+                        throw new \RuntimeException('"'.\_MA_WGGITHUB_READGH_ERROR_API_403.'"');
                     }
                     break;
                 case 404:
                     $this->apiErrorMisc = true;
-                    throw new \RuntimeException('"' . \_MA_WGGITHUB_READGH_ERROR_API_404 . '"');
+                    throw new \RuntimeException('"'.\_MA_WGGITHUB_READGH_ERROR_API_404.'"');
                     break;
                 case 405:
                     $this->apiErrorMisc = true;
-                    throw new \RuntimeException('"' . \_MA_WGGITHUB_READGH_ERROR_API_405 . '"');
+                    throw new \RuntimeException('"'.\_MA_WGGITHUB_READGH_ERROR_API_405.'"');
                     break;
                 case 0:
                 default:
                     $this->apiErrorMisc = true;
-                    throw new \RuntimeException('"' . \_MA_WGGITHUB_READGH_ERROR_API . ': ' . $errMsg . '"');
+                    throw new \RuntimeException('"'.\_MA_WGGITHUB_READGH_ERROR_API.': '.$errMsg.'"');
                     break;
             }
         } else {