mambax7 / XOOPS_equipment_manage

admin/customer.php

Unused Use Statements -3 removed lines

@@ -23,9 +23,6 @@
  */
 
 use Xmf\Module\Admin;
-use Xmf\Database\Tables;
-use Xmf\Debug;
-use Xmf\Module\Helper;
 use Xmf\Module\Helper\Permission;
 use Xmf\Request;
 

Indentation +4 added lines, -4 removed lines

@@ -342,10 +342,10 @@
             }
         } else {
             xoops_confirm(array(
-                              'ok' => 1,
-                              'id' => Request::getString('id', ''),
-                              'op' => 'delete'
-                          ), Request::getCmd('REQUEST_URI', '', 'SERVER'), sprintf(AM_EQUIPMENT_FORMSUREDEL, $customerObject->getVar('last')));
+                                'ok' => 1,
+                                'id' => Request::getString('id', ''),
+                                'op' => 'delete'
+                            ), Request::getCmd('REQUEST_URI', '', 'SERVER'), sprintf(AM_EQUIPMENT_FORMSUREDEL, $customerObject->getVar('last')));
         }
         break;
 

Spacing +1 added lines, -1 removed lines

@@ -56,7 +56,7 @@
         $criteria->setLimit($customerPaginationLimit);
         $criteria->setStart($start);
         $customerTempRows  = $customerHandler->getCount();
-        $customerTempArray = $customerHandler->getAll($criteria);/*
+        $customerTempArray = $customerHandler->getAll($criteria); /*
 //
 // 
                     <th class='center width5'>".AM_EQUIPMENT_FORM_ACTION."</th>

admin/equipment.php

Unused Use Statements -3 removed lines

@@ -23,9 +23,6 @@
  */
 
 use Xmf\Module\Admin;
-use Xmf\Database\Tables;
-use Xmf\Debug;
-use Xmf\Module\Helper;
 use Xmf\Module\Helper\Permission;
 use Xmf\Request;
 

Indentation +4 added lines, -4 removed lines

@@ -357,10 +357,10 @@
             }
         } else {
             xoops_confirm(array(
-                              'ok' => 1,
-                              'id' => Request::getString('id', ''),
-                              'op' => 'delete'
-                          ), Request::getCmd('REQUEST_URI', '', 'SERVER'), sprintf(AM_EQUIPMENT_FORMSUREDEL, $equipmentObject->getVar('name')));
+                                'ok' => 1,
+                                'id' => Request::getString('id', ''),
+                                'op' => 'delete'
+                            ), Request::getCmd('REQUEST_URI', '', 'SERVER'), sprintf(AM_EQUIPMENT_FORMSUREDEL, $equipmentObject->getVar('name')));
         }
         break;
 

Spacing +1 added lines, -1 removed lines

@@ -56,7 +56,7 @@
         $criteria->setLimit($equipmentPaginationLimit);
         $criteria->setStart($start);
         $equipmentTempRows  = $equipmentHandler->getCount();
-        $equipmentTempArray = $equipmentHandler->getAll($criteria);/*
+        $equipmentTempArray = $equipmentHandler->getAll($criteria); /*
 //
 // 
                     <th class='center width5'>".AM_EQUIPMENT_FORM_ACTION."</th>

class/form/customer.php

Doc Comments +1 added lines, -1 removed lines

@@ -44,7 +44,7 @@
     /**
      * Constructor
      *
-     * @param $target
+     * @param EquipmentCustomer $target
      */
     public function __construct($target)
     {

Unused Use Statements -1 removed lines

@@ -22,7 +22,6 @@
  * @since           1.0.0
  */
 
-use Xmf\Request;
 use Xmf\Module\Helper;
 use Xmf\Module\Helper\Permission;
 

class/form/equipment.php

Doc Comments +1 added lines, -1 removed lines

@@ -44,7 +44,7 @@
     /**
      * Constructor
      *
-     * @param $target
+     * @param EquipmentEquipment $target
      */
     public function __construct($target)
     {

Unused Use Statements -1 removed lines

@@ -22,7 +22,6 @@
  * @since           1.0.0
  */
 
-use Xmf\Request;
 use Xmf\Module\Helper;
 use Xmf\Module\Helper\Permission;
 

class/utility.php

Doc Comments +3 added lines

@@ -296,6 +296,9 @@
         return $success;
     }
 
+    /**
+     * @param string $sql
+     */
     public static function getQueryDataToJSON($sql)
     {
         global $xoopsDB;

Braces +1 added lines, -2 removed lines

@@ -186,8 +186,7 @@
                 }
                 file_put_contents($folder . '/index.html', '<script>history.go(-1);</script>');
             }
-        }
-        catch (Exception $e) {
+        } catch (Exception $e) {
             echo 'Caught exception: ', $e->getMessage(), '', '<br>';
         }
     }

include/onupdate.php

Unused Use Statements -2 removed lines

@@ -22,8 +22,6 @@
  * @since           1.0.0
  */
 
-use Xmf\Language;
-
 if ((!defined('XOOPS_ROOT_PATH')) || !$GLOBALS['xoopsUser'] instanceof XoopsUser
     || !$GLOBALS['xoopsUser']->IsAdmin()) {
     exit('Restricted access' . PHP_EOL);

Spacing +1 added lines, -1 removed lines

@@ -95,7 +95,7 @@
             foreach ($configurator['templateFolders'] as $folder) {
                 $templateFolder = $GLOBALS['xoops']->path('modules/' . $moduleDirName . $folder);
                 if (is_dir($templateFolder)) {
-                    $templateList = array_diff(scandir($templateFolder,SCANDIR_SORT_NONE), array('..', '.'));
+                    $templateList = array_diff(scandir($templateFolder, SCANDIR_SORT_NONE), array('..', '.'));
                     foreach ($templateList as $k => $v) {
                         $fileInfo = new SplFileInfo($templateFolder . $v);
                         if ($fileInfo->getExtension() === 'html' && $fileInfo->getFilename() !== 'index.html') {

class/rentals.php

Indentation +1 added lines, -1 removed lines

@@ -121,7 +121,7 @@
 //                    INNER JOIN `%2$s` ON `%1$s`.`id`=`%2$s`.`id` WHERE `%1$s`.`borrower`="%3$s"', $xoopsDB->prefix('equipment_rentals'), $xoopsDB->prefix('equipment_equipment'), $xoopsUser->uname());
 
         $borrow_sql = 'SELECT r.quantity, e.name, c.first, c.last FROM ' . $xoopsDB->prefix('equipment_equipment') . ' AS e, ' . $xoopsDB->prefix('equipment_customer') . ' AS c, '
-                      . $xoopsDB->prefix('equipment_rentals') . ' AS r WHERE e.id = r.equipmentid AND c.id = r.customerid';
+                        . $xoopsDB->prefix('equipment_rentals') . ' AS r WHERE e.id = r.equipmentid AND c.id = r.customerid';
 
         $borrow_data = EquipmentUtility::getQueryDataToJSON($borrow_sql);
 

blocks/equipment.php

Spacing +1 added lines, -1 removed lines

@@ -88,7 +88,7 @@
     $form           .= MB_EQUIPMENT_CATTODISPLAY . "<br><select name='options[]' multiple='multiple' size='5'>";
     $form           .= "<option value='0' " . (in_array(0, $options) === false ? '' : "selected='selected'") . '>' . MB_EQUIPMENT_ALLCAT . '</option>';
     foreach (array_keys($equipmentArray) as $i) {
-        $id   = $equipmentArray[$i]->getVar('id');
+        $id = $equipmentArray[$i]->getVar('id');
         $form .= "<option value='" . $id . "' " . (in_array($id, $options) === false ? '' : "selected='selected'") . '>' . $equipmentArray[$i]->getVar('name') . '</option>';
     }
     $form .= '</select>';

blocks/customer.php

Spacing +1 added lines, -1 removed lines

@@ -86,7 +86,7 @@
     $form          .= MB_EQUIPMENT_CATTODISPLAY . "<br><select name='options[]' multiple='multiple' size='5'>";
     $form          .= "<option value='0' " . (in_array(0, $options) === false ? '' : "selected='selected'") . '>' . MB_EQUIPMENT_ALLCAT . '</option>';
     foreach (array_keys($customerArray) as $i) {
-        $id   = $customerArray[$i]->getVar('id');
+        $id = $customerArray[$i]->getVar('id');
         $form .= "<option value='" . $id . "' " . (in_array($id, $options) === false ? '' : "selected='selected'") . '>' . $customerArray[$i]->getVar('last') . '</option>';
     }
     $form .= '</select>';