Use standard model transformer for asset model API response

Signed-off-by: snipe <snipe@snipe.net>

app/Console/Commands/LdapSync.php

@@ -182,7 +182,7 @@ class LdapSync extends Command
             // Inject location information fields
             for ($i = 0; $i < $results['count']; $i++) {
                 $results[$i]['ldap_location_override'] = false;
-                $results[$i]['location_id'] = 0;
+                $results[$i]['location_id'] = null;
             }
 
             // Grab subsets based on location-specific DNs, and overwrite location for these users.

app/Console/Commands/RestoreFromBackup.php

@@ -329,9 +329,9 @@ class RestoreFromBackup extends Command
                     }
                 }
             }
-            $good_extensions = ['png', 'gif', 'jpg', 'svg', 'jpeg', 'doc', 'docx', 'pdf', 'txt',
-                                'zip', 'rar', 'xls', 'xlsx', 'lic', 'xml', 'rtf', 'webp', 'key', 'ico', 'avif'
-            ];
+
+            $good_extensions = config('filesystems.allowed_upload_extensions_array');
+
             foreach (array_merge($private_files, $public_files) as $file) {
                 $has_wildcard = (strpos($file, '*') !== false);
                 if ($has_wildcard) {

app/Helpers/Helper.php

@@ -13,6 +13,7 @@ use App\Models\Setting;
 use App\Models\Statuslabel;
 use App\Models\License;
 use App\Models\Location;
+use Illuminate\Http\RedirectResponse;
 use Illuminate\Support\Facades\Crypt;
 use Illuminate\Contracts\Encryption\DecryptException;
 use Carbon\Carbon;
@@ -876,6 +877,48 @@ class Helper
         return false;
     }
 
+    /**
+     * Check if the file is a video, so we can show a preview
+     *
+     * @param File $file
+     * @return string | Boolean
+     * @author [B. Wetherington] [<bwetherington@grokability.com>]
+     * @since [v8.1.18]
+     */
+    public static function checkUploadIsVideo($file)
+    {
+        $finfo = @finfo_open(FILEINFO_MIME_TYPE); // return mime type ala mimetype extension
+        $filetype = @finfo_file($finfo, $file);
+        finfo_close($finfo);
+
+        if (($filetype == 'video/mp4') || ($filetype == 'video/quicktime') || ($filetype == 'video/mpeg') || ($filetype == 'video/ogg') || ($filetype == 'video/webm') || ($filetype == 'video/x-msvide')) {
+            return $filetype;
+        }
+
+        return false;
+    }
+
+    /**
+     * Check if the file is audio, so we can show a preview
+     *
+     * @param File $file
+     * @return string | Boolean
+     * @author [A. Gianotto] [<snipe@snipe.net>]
+     * @since [v3.0]
+     */
+    public static function checkUploadIsAudio($file)
+    {
+        $finfo = @finfo_open(FILEINFO_MIME_TYPE); // return mime type ala mimetype extension
+        $filetype = @finfo_file($finfo, $file);
+        finfo_close($finfo);
+
+        if (($filetype == 'audio/mpeg') || ($filetype == 'audio/ogg')) {
+            return $filetype;
+        }
+
+        return false;
+    }
+
     /**
      * Walks through the permissions in the permissions config file and determines if
      * permissions are granted based on a $selected_arr array.
@@ -1170,6 +1213,15 @@ class Helper
             // Misc
             'pdf'   => 'far fa-file-pdf',
             'lic'   => 'far fa-save',
+
+            // video
+            'mov'   => 'fa-solid fa-video',
+            'mp4'   => 'fa-solid fa-video',
+
+            // audio
+            'ogg'   => 'fa-solid fa-file-audio',
+            'mp3'   => 'fa-solid fa-file-audio',
+            'wav'   => 'fa-solid fa-file-audio',
         ];
 
         if ($extension && array_key_exists($extension, $allowedExtensionMap)) {
@@ -1480,68 +1532,64 @@ class Helper
     }
 
 
-    static public function getRedirectOption($request, $id, $table, $item_id = null)
+    static public function getRedirectOption($request, $id, $table, $item_id = null) : RedirectResponse
     {
 
-        $redirect_option = Session::get('redirect_option');
-        $checkout_to_type = Session::get('checkout_to_type');
+        $redirect_option = Session::get('redirect_option') ?? $request->redirect_option;
+        $checkout_to_type = Session::get('checkout_to_type') ?? null;
         $checkedInFrom = Session::get('checkedInFrom');
         $other_redirect = Session::get('other_redirect');
+        $backUrl = Session::pull('back_url', route('home'));
+
+       // return to previous page
+        if ($redirect_option === 'back') {
+            if ($backUrl === route('home')) {
+                return redirect()->to($backUrl)
+                ->with('warning', trans('general.page_error'));
+             }
+
+            return redirect()->to($backUrl);
+        }
 
         // return to index
         if ($redirect_option == 'index') {
-            switch ($table) {
-                case "Assets":
-                    return route('hardware.index');
-                case "Users":
-                    return route('users.index');
-                case "Licenses":
-                    return route('licenses.index');
-                case "Accessories":
-                    return route('accessories.index');
-                case "Components":
-                    return route('components.index');
-                case "Consumables":
-                    return route('consumables.index');
-            }
+            return match ($table) {
+                'Assets' => redirect()->route('hardware.index'),
+                'Users' => redirect()->route('users.index'),
+                'Licenses' => redirect()->route('licenses.index'),
+                'Accessories' => redirect()->route('accessories.index'),
+                'Components' => redirect()->route('components.index'),
+                'Consumables' => redirect()->route('consumables.index'),
+            };
         }
 
         // return to thing being assigned
         if ($redirect_option == 'item') {
-            switch ($table) {
-                case "Assets":
-                    return route('hardware.show', $id ?? $item_id);
-                case "Users":
-                    return route('users.show', $id ?? $item_id);
-                case "Licenses":
-                    return route('licenses.show', $id ?? $item_id);
-                case "Accessories":
-                    return route('accessories.show', $id ?? $item_id);
-                case "Components":
-                    return route('components.show', $id ?? $item_id);
-                case "Consumables":
-                    return route('consumables.show', $id ?? $item_id);
-            }
+            return match ($table) {
+                'Assets'      => redirect()->route('hardware.show', $id ?? $item_id),
+                'Users'       => redirect()->route('users.show', $id ?? $item_id),
+                'Licenses'    => redirect()->route('licenses.show', $id ?? $item_id),
+                'Accessories' => redirect()->route('accessories.show', $id ?? $item_id),
+                'Components'  => redirect()->route('components.show', $id ?? $item_id),
+                'Consumables' => redirect()->route('consumables.show', $id ?? $item_id),
+            };
         }
 
         // return to assignment target
         if ($redirect_option == 'target') {
-            switch ($checkout_to_type) {
-                case 'user':
-                    return route('users.show', $request->assigned_user ?? $checkedInFrom);
-                case 'location':
-                    return route('locations.show', $request->assigned_location ?? $checkedInFrom);
-                case 'asset':
-                    return route('hardware.show', $request->assigned_asset ?? $checkedInFrom);
-            }
+            return match ($checkout_to_type) {
+                'user'     => redirect()->route('users.show', $request->assigned_user ?? $checkedInFrom),
+                'location' => redirect()->route('locations.show', $request->assigned_location ?? $checkedInFrom),
+                'asset'    => redirect()->route('hardware.show', $request->assigned_asset ?? $checkedInFrom),
+            };
         }
 
         // return to somewhere else
         if ($redirect_option == 'other_redirect') {
-            switch ($other_redirect) {
-                case 'audit':
-                    return route('assets.audit.due');
-            }
+            return match ($other_redirect) {
+                'audit' => redirect()->route('assets.audit.due'),
+                'model' => redirect()->route('models.show', $request->model_id),
+            };
 
         }
 

app/Helpers/StorageHelper.php

@@ -41,15 +41,21 @@ class StorageHelper
     {
 
         $allowed_inline = [
-            'pdf',
-            'svg',
-            'jpg',
-            'gif',
-            'svg',
             'avif',
-            'webp',
-            'png',
             'gif',
+            'gif',
+            'jpg',
+            'mov',
+            'mp3',
+            'mp4',
+            'ogg',
+            'pdf',
+            'png',
+            'svg',
+            'svg',
+            'wav',
+            'webm',
+            'webp',
         ];
 
 

app/Http/Controllers/Accessories/AccessoriesController.php

@@ -83,7 +83,8 @@ class AccessoriesController extends Controller
         // Was the accessory created?
         if ($accessory->save()) {
             // Redirect to the new accessory  page
-            return redirect()->to(Helper::getRedirectOption($request, $accessory->id, 'Accessories'))->with('success', trans('admin/accessories/message.create.success'));
+            return Helper::getRedirectOption($request, $accessory->id, 'Accessories')
+                ->with('success', trans('admin/accessories/message.create.success'));
         }
 
         return redirect()->back()->withInput()->withErrors($accessory->getErrors());
@@ -167,7 +168,8 @@ class AccessoriesController extends Controller
             session()->put(['redirect_option' => $request->get('redirect_option')]);
 
             if ($accessory->save()) {
-                return redirect()->to(Helper::getRedirectOption($request, $accessory->id, 'Accessories'))->with('success', trans('admin/accessories/message.update.success'));
+                return Helper::getRedirectOption($request, $accessory->id, 'Accessories')
+                    ->with('success', trans('admin/accessories/message.update.success'));
             }
         } else {
             return redirect()->route('accessories.index')->with('error', trans('admin/accessories/message.does_not_exist'));

app/Http/Controllers/Accessories/AccessoryCheckinController.php

@@ -78,7 +78,8 @@ class AccessoryCheckinController extends Controller
 
             session()->put(['redirect_option' => $request->get('redirect_option')]);
 
-            return redirect()->to(Helper::getRedirectOption($request, $accessory->id, 'Accessories'))->with('success', trans('admin/accessories/message.checkin.success'));
+            return Helper::getRedirectOption($request, $accessory->id, 'Accessories')
+                ->with('success', trans('admin/accessories/message.checkin.success'));
         }
         // Redirect to the accessory management page with error
         return redirect()->route('accessories.index')->with('error', trans('admin/accessories/message.checkin.error'));

app/Http/Controllers/Accessories/AccessoryCheckoutController.php

@@ -97,7 +97,7 @@ class AccessoryCheckoutController extends Controller
 
 
         // Redirect to the new accessory page
-        return redirect()->to(Helper::getRedirectOption($request, $accessory->id, 'Accessories'))
+        return Helper::getRedirectOption($request, $accessory->id, 'Accessories')
             ->with('success', trans('admin/accessories/message.checkout.success'));
     }
 }

app/Http/Controllers/Account/AcceptanceController.php

@@ -21,6 +21,7 @@ use App\Models\License;
 use App\Models\Component;
 use App\Models\Consumable;
 use App\Notifications\AcceptanceAssetAcceptedNotification;
+use App\Notifications\AcceptanceAssetAcceptedToUserNotification;
 use App\Notifications\AcceptanceAssetDeclinedNotification;
 use Exception;
 use Illuminate\Http\Request;
@@ -151,6 +152,8 @@ class AcceptanceController extends Controller
                 }
             }
 
+
+            $assigned_user = User::find($acceptance->assigned_to_id);
             // this is horrible
             switch($acceptance->checkoutable_type){
                 case 'App\Models\Asset':
@@ -160,35 +163,30 @@ class AcceptanceController extends Controller
                             return redirect()->back()->with('error', trans('admin/models/message.does_not_exist'));
                         }
                         $display_model = $asset_model->name;
-                        $assigned_to = User::find($acceptance->assigned_to_id)->present()->fullName;
                 break;
 
                 case 'App\Models\Accessory':
                         $pdf_view_route ='account.accept.accept-accessory-eula';
                         $accessory = Accessory::find($item->id);
                         $display_model = $accessory->name;
-                        $assigned_to = User::find($acceptance->assigned_to_id)->present()->fullName;
                 break;
 
                 case 'App\Models\LicenseSeat':
                         $pdf_view_route ='account.accept.accept-license-eula';
                         $license = License::find($item->license_id);
                         $display_model = $license->name;
-                        $assigned_to = User::find($acceptance->assigned_to_id)->present()->fullName;
                 break;
 
                 case 'App\Models\Component':
                         $pdf_view_route ='account.accept.accept-component-eula';
                         $component = Component::find($item->id);
                         $display_model = $component->name;
-                        $assigned_to = User::find($acceptance->assigned_to_id)->present()->fullName;
                 break;
 
                 case 'App\Models\Consumable':
                         $pdf_view_route ='account.accept.accept-consumable-eula';
                         $consumable = Consumable::find($item->id);
                         $display_model = $consumable->name;
-                        $assigned_to = User::find($acceptance->assigned_to_id)->present()->fullName;
                 break;
             }
 //            if ($acceptance->checkoutable_type == 'App\Models\Asset') {
@@ -229,7 +227,7 @@ class AcceptanceController extends Controller
                 'note' => $request->input('note'),
                 'check_out_date' => Carbon::parse($acceptance->created_at)->format('Y-m-d'),
                 'accepted_date' => Carbon::parse($acceptance->accepted_at)->format('Y-m-d'),
-                'assigned_to' => $assigned_to,
+                'assigned_to' => $assigned_user->present()->fullName,
                 'company_name' => $branding_settings->site_name,
                 'signature' => ($sig_filename) ? storage_path() . '/private_uploads/signatures/' . $sig_filename : null,
                 'logo' => $path_logo,
@@ -243,6 +241,19 @@ class AcceptanceController extends Controller
             }
 
             $acceptance->accept($sig_filename, $item->getEula(), $pdf_filename, $request->input('note'));
+
+            // Send the PDF to the signing user
+            if (($request->input('send_copy') == '1') && ($assigned_user->email !='')) {
+
+                // Add the attachment for the signing user into the $data array
+                $data['file'] = $pdf_filename;
+
+                try {
+                    $assigned_user->notify(new AcceptanceAssetAcceptedToUserNotification($data));
+                } catch (\Exception $e) {
+                    Log::warning($e);
+                }
+            }
             try {
                 $acceptance->notify(new AcceptanceAssetAcceptedNotification($data));
             } catch (\Exception $e) {

app/Http/Controllers/Api/AssetModelsController.php

@@ -154,7 +154,7 @@ class AssetModelsController extends Controller
         $assetmodel = $request->handleImages($assetmodel);
 
         if ($assetmodel->save()) {
-            return response()->json(Helper::formatStandardApiResponse('success', $assetmodel, trans('admin/models/message.create.success')));
+            return response()->json(Helper::formatStandardApiResponse('success', (new AssetModelsTransformer)->transformAssetModel($assetmodel), trans('admin/models/message.create.success')));
         }
         return response()->json(Helper::formatStandardApiResponse('error', null, $assetmodel->getErrors()));
 
@@ -207,7 +207,7 @@ class AssetModelsController extends Controller
         $assetmodel = AssetModel::findOrFail($id);
         $assetmodel->fill($request->all());
         $assetmodel = $request->handleImages($assetmodel);
-        
+
         /**
          * Allow custom_fieldset_id to override and populate fieldset_id.
          * This is stupid, but required for legacy API support.
@@ -222,7 +222,7 @@ class AssetModelsController extends Controller
 
 
         if ($assetmodel->save()) {
-            return response()->json(Helper::formatStandardApiResponse('success', $assetmodel, trans('admin/models/message.update.success')));
+            return response()->json(Helper::formatStandardApiResponse('success', (new AssetModelsTransformer)->transformAssetModel($assetmodel), trans('admin/models/message.update.success')));
         }
 
         return response()->json(Helper::formatStandardApiResponse('error', null, $assetmodel->getErrors()));

app/Http/Controllers/Api/CompaniesController.php

@@ -43,7 +43,10 @@ class CompaniesController extends Controller
 
         $companies = Company::withCount(['assets as assets_count'  => function ($query) {
             $query->AssetsForShow();
-        }])->withCount('licenses as licenses_count', 'accessories as accessories_count', 'consumables as consumables_count', 'components as components_count', 'users as users_count');
+        }])
+            ->with('adminuser')
+            ->withCount('licenses as licenses_count', 'accessories as accessories_count', 'consumables as consumables_count', 'components as components_count', 'users as users_count');
+
 
         if ($request->filled('search')) {
             $companies->TextSearch($request->input('search'));
@@ -119,6 +122,7 @@ class CompaniesController extends Controller
     {
         $this->authorize('view', Company::class);
         $company = Company::findOrFail($id);
+        $this->authorize('view', $company);
         return (new CompaniesTransformer)->transformCompany($company);
 
     }
@@ -136,6 +140,7 @@ class CompaniesController extends Controller
     {
         $this->authorize('update', Company::class);
         $company = Company::findOrFail($id);
+        $this->authorize('update', $company);
         $company->fill($request->all());
         $company = $request->handleImages($company);
 
@@ -188,6 +193,7 @@ class CompaniesController extends Controller
             'companies.image',
         ]);
 
+
         if ($request->filled('search')) {
             $companies = $companies->where('companies.name', 'LIKE', '%'.$request->get('search').'%');
         }

app/Http/Controllers/Api/LicenseSeatsController.php

@@ -29,6 +29,15 @@ class LicenseSeatsController extends Controller
             $seats = LicenseSeat::with('license', 'user', 'asset', 'user.department')
                 ->where('license_seats.license_id', $licenseId);
 
+            if ($request->input('status') == 'available') {
+                $seats->whereNull('license_seats.assigned_to');
+            }
+
+            if ($request->input('status') == 'assigned') {
+                $seats->ByAssigned();
+            }
+
+
             $order = $request->input('order') === 'asc' ? 'asc' : 'desc';
 
             if ($request->input('sort') == 'department') {

app/Http/Controllers/Api/UploadedFilesController.php

@@ -33,6 +33,7 @@ class UploadedFilesController extends Controller
         'assets' => Asset::class,
         'components' => Component::class,
         'consumables' => Consumable::class,
+        'hardware' => Asset::class,
         'licenses' => License::class,
         'locations' => Location::class,
         'models' => AssetModel::class,
@@ -44,6 +45,7 @@ class UploadedFilesController extends Controller
         'assets' => 'private_uploads/assets/',
         'components' => 'private_uploads/components/',
         'consumables' => 'private_uploads/consumables/',
+        'hardware' => 'private_uploads/assets/',
         'licenses' => 'private_uploads/licenses/',
         'locations' => 'private_uploads/locations/',
         'models' => 'private_uploads/assetmodels/',
@@ -55,6 +57,7 @@ class UploadedFilesController extends Controller
         'assets' => 'asset',
         'components' => 'component',
         'consumables' => 'consumable',
+        'hardware' => 'asset',
         'licenses' => 'license',
         'locations' => 'location',
         'models' => 'model',

app/Http/Controllers/Assets/AssetCheckinController.php

@@ -153,7 +153,8 @@ class AssetCheckinController extends Controller
         if ($asset->save()) {
 
             event(new CheckoutableCheckedIn($asset, $target, auth()->user(), $request->input('note'), $checkin_at, $originalValues));
-            return redirect()->to(Helper::getRedirectOption($request, $asset->id, 'Assets'))->with('success', trans('admin/hardware/message.checkin.success'));
+            return Helper::getRedirectOption($request, $asset->id, 'Assets')
+                ->with('success', trans('admin/hardware/message.checkin.success'));
         }
         // Redirect to the asset management page with error
         return redirect()->route('hardware.index')->with('error', trans('admin/hardware/message.checkin.error').$asset->getErrors());

app/Http/Controllers/Assets/AssetCheckoutController.php

@@ -123,7 +123,7 @@ class AssetCheckoutController extends Controller
             session()->put(['redirect_option' => $request->get('redirect_option'), 'checkout_to_type' => $request->get('checkout_to_type')]);
 
             if ($asset->checkOut($target, $admin, $checkout_at, $expected_checkin, $request->get('note'), $request->get('name'))) {
-                return redirect()->to(Helper::getRedirectOption($request, $asset->id, 'Assets'))
+                return Helper::getRedirectOption($request, $asset->id, 'Assets')
                     ->with('success', trans('admin/hardware/message.checkout.success'));
             }
             // Redirect to the asset management page with error

app/Http/Controllers/Assets/AssetsController.php

@@ -233,18 +233,18 @@ class AssetsController extends Controller
         if ($successes) {
             if ($failures) {
                 //some succeeded, some failed
-                return redirect()->to(Helper::getRedirectOption($request, $asset->id, 'Assets')) //FIXME - not tested
+                return Helper::getRedirectOption($request, $asset->id, 'Assets') //FIXME - not tested
                 ->with('success-unescaped', trans_choice('admin/hardware/message.create.multi_success_linked', $successes, ['links' => join(", ", $successes)]))
                     ->with('warning', trans_choice('admin/hardware/message.create.partial_failure', $failures, ['failures' => join("; ", $failures)]));
             } else {
                 if (count($successes) == 1) {
                     //the most common case, keeping it so we don't have to make every use of that translation string be trans_choice'ed
                     //and re-translated
-                    return redirect()->to(Helper::getRedirectOption($request, $asset->id, 'Assets'))
+                    return Helper::getRedirectOption($request, $asset->id, 'Assets')
                         ->with('success-unescaped', trans('admin/hardware/message.create.success_linked', ['link' => route('hardware.show', $asset), 'id', 'tag' => e($asset->asset_tag)]));
                 } else {
                     //multi-success
-                    return redirect()->to(Helper::getRedirectOption($request, $asset->id, 'Assets'))
+                    return Helper::getRedirectOption($request, $asset->id, 'Assets')
                         ->with('success-unescaped', trans_choice('admin/hardware/message.create.multi_success_linked', $successes, ['links' => join(", ", $successes)]));
                 }
             }
@@ -265,6 +265,7 @@ class AssetsController extends Controller
     public function edit(Asset $asset) : View | RedirectResponse
     {
         $this->authorize($asset);
+        session()->put('back_url', url()->previous());
         return view('hardware/edit')
             ->with('item', $asset)
             ->with('statuslabel_list', Helper::statusLabelList())
@@ -427,11 +428,15 @@ class AssetsController extends Controller
                 }
             }
         }
+        session()->put([
+            'redirect_option' => $request->get('redirect_option'),
+            'checkout_to_type' => $request->get('checkout_to_type'),
+            'other_redirect' => $request->get('redirect_option') === 'other_redirect' ? 'model' : null,
+        ]);
 
-        session()->put(['redirect_option' => $request->get('redirect_option'), 'checkout_to_type' => $request->get('checkout_to_type')]);
 
         if ($asset->save()) {
-            return redirect()->to(Helper::getRedirectOption($request, $asset->id, 'Assets'))
+            return Helper::getRedirectOption($request, $asset->id, 'Assets')
                 ->with('success', trans('admin/hardware/message.update.success'));
         }
 
@@ -996,7 +1001,7 @@ class AssetsController extends Controller
             }
 
             $asset->logAudit($request->input('note'), $request->input('location_id'), $file_name, $originalValues);
-            return redirect()->to(Helper::getRedirectOption($request, $asset->id, 'Assets'))->with('success', trans('admin/hardware/message.audit.success'));
+            return Helper::getRedirectOption($request, $asset->id, 'Assets')->with('success', trans('admin/hardware/message.audit.success'));
         }
 
         return redirect()->back()->withInput()->withErrors($asset->getErrors());

app/Http/Controllers/Assets/BulkAssetsController.php

@@ -112,11 +112,47 @@ class BulkAssetsController extends Controller
         // This handles all of the pivot sorting below (versus the assets.* fields in the allowed_columns array)
         $column_sort = in_array($sort_override, $allowed_columns) ? $sort_override : 'assets.id';
 
-        $assets = Asset::with('assignedTo', 'location', 'model')
+        $query = Asset::with('assignedTo', 'location', 'model')
                 ->whereIn('assets.id', $asset_ids)
                 ->withTrashed();
 
-        $assets = $assets->get();
+
+        switch ($sort_override) {
+            case 'model':
+                $query->OrderModels($order);
+                break;
+            case 'model_number':
+                $query->OrderModelNumber($order);
+                break;
+            case 'category':
+                $query->OrderCategory($order);
+                break;
+            case 'manufacturer':
+                $query->OrderManufacturer($order);
+                break;
+            case 'company':
+                $query->OrderCompany($order);
+                break;
+            case 'location':
+                $query->OrderLocation($order);
+                break;
+            case 'rtd_location':
+                $query->OrderRtdLocation($order);
+                break;
+            case 'status_label':
+                $query->OrderStatus($order);
+                break;
+            case 'supplier':
+                $query->OrderSupplier($order);
+                break;
+            case 'assigned_to':
+                $query->OrderAssigned($order);
+                break;
+            default:
+                $query->orderBy($column_sort, $order);
+                break;
+        }
+        $assets = $query->get();
 
         if ($assets->isEmpty()) {
             Log::debug('No assets were found for the provided IDs', ['ids' => $asset_ids]);
@@ -169,40 +205,7 @@ class BulkAssetsController extends Controller
             }
         }
 
-        switch ($sort_override) {
-            case 'model':
-                $assets->OrderModels($order);
-                break;
-            case 'model_number':
-                $assets->OrderModelNumber($order);
-                break;
-            case 'category':
-                $assets->OrderCategory($order);
-                break;
-            case 'manufacturer':
-                $assets->OrderManufacturer($order);
-                break;
-            case 'company':
-                $assets->OrderCompany($order);
-                break;
-            case 'location':
-                $assets->OrderLocation($order);
-            case 'rtd_location':
-                $assets->OrderRtdLocation($order);
-                break;
-            case 'status_label':
-                $assets->OrderStatus($order);
-                break;
-            case 'supplier':
-                $assets->OrderSupplier($order);
-                break;
-            case 'assigned_to':
-                $assets->OrderAssigned($order);
-                break;
-            default:
-                $assets->orderBy($column_sort, $order);
-                break;
-        }
+
 
         return redirect()->back()->with('error', 'No action selected');
     }

app/Http/Controllers/CompaniesController.php

@@ -123,11 +123,13 @@ final class CompaniesController extends Controller
      */
     public function destroy($companyId) : RedirectResponse
     {
+
         if (is_null($company = Company::find($companyId))) {
             return redirect()->route('companies.index')
                 ->with('error', trans('admin/companies/message.not_found'));
         }
 
+
         $this->authorize('delete', $company);
         if (! $company->isDeletable()) {
             return redirect()->route('companies.index')

app/Http/Controllers/Components/ComponentCheckinController.php

@@ -100,8 +100,8 @@ class ComponentCheckinController extends Controller
 
             session()->put(['redirect_option' => $request->get('redirect_option')]);
 
-            return redirect()->to(Helper::getRedirectOption($request, $component->id, 'Components'))->with('success',
-                trans('admin/components/message.checkin.success'));
+            return Helper::getRedirectOption($request, $component->id, 'Components')
+                ->with('success', trans('admin/components/message.checkin.success'));
         }
 
         return redirect()->route('components.index')->with('error', trans('admin/components/message.does_not_exist'));

app/Http/Controllers/Components/ComponentCheckoutController.php

@@ -120,6 +120,7 @@ class ComponentCheckoutController extends Controller
 
         session()->put(['redirect_option' => $request->get('redirect_option'), 'checkout_to_type' => $request->get('checkout_to_type')]);
 
-        return redirect()->to(Helper::getRedirectOption($request, $component->id, 'Components'))->with('success', trans('admin/components/message.checkout.success'));
+        return Helper::getRedirectOption($request, $component->id, 'Components')
+            ->with('success', trans('admin/components/message.checkout.success'));
     }
 }

app/Http/Controllers/Components/ComponentsController.php

@@ -91,7 +91,8 @@ class ComponentsController extends Controller
         session()->put(['redirect_option' => $request->get('redirect_option')]);
 
         if ($component->save()) {
-            return redirect()->to(Helper::getRedirectOption($request, $component->id, 'Components'))->with('success', trans('admin/components/message.create.success'));
+            return Helper::getRedirectOption($request, $component->id, 'Components')
+                ->with('success', trans('admin/components/message.create.success'));
         }
 
         return redirect()->back()->withInput()->withErrors($component->getErrors());
@@ -111,6 +112,7 @@ class ComponentsController extends Controller
     {
 
             $this->authorize('update', $component);
+            session()->put('back_url', url()->previous());
             return view('components/edit')
                 ->with('item', $component)
                 ->with('category_type', 'component');
@@ -164,7 +166,8 @@ class ComponentsController extends Controller
         session()->put(['redirect_option' => $request->get('redirect_option')]);
 
         if ($component->save()) {
-            return redirect()->to(Helper::getRedirectOption($request, $component->id, 'Components'))->with('success', trans('admin/components/message.update.success'));
+            return Helper::getRedirectOption($request, $component->id, 'Components')
+                ->with('success', trans('admin/components/message.update.success'));
         }
 
         return redirect()->back()->withInput()->withErrors($component->getErrors());

app/Http/Controllers/Consumables/ConsumableCheckoutController.php

@@ -111,6 +111,7 @@ class ConsumableCheckoutController extends Controller
 
 
         // Redirect to the new consumable page
-        return redirect()->to(Helper::getRedirectOption($request, $consumable->id, 'Consumables'))->with('success', trans('admin/consumables/message.checkout.success'));
+        return Helper::getRedirectOption($request, $consumable->id, 'Consumables')
+            ->with('success', trans('admin/consumables/message.checkout.success'));
     }
 }

app/Http/Controllers/Consumables/ConsumablesController.php

@@ -90,7 +90,8 @@ class ConsumablesController extends Controller
         session()->put(['redirect_option' => $request->get('redirect_option')]);
 
         if ($consumable->save()) {
-            return redirect()->to(Helper::getRedirectOption($request, $consumable->id, 'Consumables'))->with('success', trans('admin/consumables/message.create.success'));
+            return Helper::getRedirectOption($request, $consumable->id, 'Consumables')
+                ->with('success', trans('admin/consumables/message.create.success'));
         }
 
         return redirect()->back()->withInput()->withErrors($consumable->getErrors());
@@ -107,6 +108,7 @@ class ConsumablesController extends Controller
     public function edit(Consumable $consumable) : View | RedirectResponse
     {
             $this->authorize($consumable);
+            session()->put('back_url', url()->previous());
             return view('consumables/edit')
                 ->with('item', $consumable)
                 ->with('category_type', 'consumable');
@@ -160,7 +162,8 @@ class ConsumablesController extends Controller
         session()->put(['redirect_option' => $request->get('redirect_option')]);
 
         if ($consumable->save()) {
-            return redirect()->to(Helper::getRedirectOption($request, $consumable->id, 'Consumables'))->with('success', trans('admin/consumables/message.update.success'));
+            return Helper::getRedirectOption($request, $consumable->id, 'Consumables')
+                ->with('success', trans('admin/consumables/message.update.success'));
         }
 
         return redirect()->back()->withInput()->withErrors($consumable->getErrors());

app/Http/Controllers/Licenses/LicenseCheckinController.php

@@ -98,14 +98,17 @@ class LicenseCheckinController extends Controller
         $licenseSeat->notes = $request->input('notes');
 
         session()->put(['redirect_option' => $request->get('redirect_option')]);
-
+        if ($request->get('redirect_option') === 'target'){
+            session()->put(['checkout_to_type' => 'user']);
+        }
 
         // Was the asset updated?
         if ($licenseSeat->save()) {
             event(new CheckoutableCheckedIn($licenseSeat, $return_to, auth()->user(), $request->input('notes')));
 
 
-            return redirect()->to(Helper::getRedirectOption($request, $license->id, 'Licenses'))->with('success', trans('admin/licenses/message.checkin.success'));
+            return Helper::getRedirectOption($request, $license->id, 'Licenses')
+                ->with('success', trans('admin/licenses/message.checkin.success'));
         }
 
         // Redirect to the license page with error

app/Http/Controllers/Licenses/LicenseCheckoutController.php

@@ -89,7 +89,8 @@ class LicenseCheckoutController extends Controller
 
 
         if ($checkoutTarget) {
-            return redirect()->to(Helper::getRedirectOption($request, $license->id, 'Licenses'))->with('success', trans('admin/licenses/message.checkout.success'));
+            return Helper::getRedirectOption($request, $license->id, 'Licenses')
+                ->with('success', trans('admin/licenses/message.checkout.success'));
         }
 
 

app/Http/Controllers/Licenses/LicensesController.php

@@ -105,7 +105,8 @@ class LicensesController extends Controller
         session()->put(['redirect_option' => $request->get('redirect_option')]);
 
         if ($license->save()) {
-            return redirect()->to(Helper::getRedirectOption($request, $license->id, 'Licenses'))->with('success', trans('admin/licenses/message.create.success'));
+            return Helper::getRedirectOption($request, $license->id, 'Licenses')
+                ->with('success', trans('admin/licenses/message.create.success'));
         }
 
         return redirect()->back()->withInput()->withErrors($license->getErrors());
@@ -125,7 +126,7 @@ class LicensesController extends Controller
     {
 
         $this->authorize('update', $license);
-
+        session()->put('back_url', url()->previous());
         $maintained_list = [
             '' => 'Maintained',
             '1' => 'Yes',
@@ -181,7 +182,8 @@ class LicensesController extends Controller
         session()->put(['redirect_option' => $request->get('redirect_option')]);
 
         if ($license->save()) {
-            return redirect()->to(Helper::getRedirectOption($request, $license->id, 'Licenses'))->with('success', trans('admin/licenses/message.update.success'));
+            return Helper::getRedirectOption($request, $license->id, 'Licenses')
+                ->with('success', trans('admin/licenses/message.update.success'));
         }
         // If we can't adjust the number of seats, the error is flashed to the session by the event handler in License.php
         return redirect()->back()->withInput()->withErrors($license->getErrors());

app/Http/Controllers/Users/UsersController.php

@@ -154,7 +154,8 @@ class UsersController extends Controller
                 $user->notify(new WelcomeNotification($data));
             }
 
-            return redirect()->to(Helper::getRedirectOption($request, $user->id, 'Users'))->with('success', trans('admin/users/message.success.create'));
+            return Helper::getRedirectOption($request, $user->id, 'Users')
+                ->with('success', trans('admin/users/message.success.create'));
         }
 
         return redirect()->back()->withInput()->withErrors($user->getErrors());
@@ -186,6 +187,7 @@ class UsersController extends Controller
     {
 
         $this->authorize('update', User::class);
+        session()->put('back_url', url()->previous());
         $user = User::with(['assets', 'assets.model', 'consumables', 'accessories', 'licenses', 'userloc'])->withTrashed()->find($user->id);
 
         if ($user) {
@@ -312,7 +314,7 @@ class UsersController extends Controller
 
         if ($user->save()) {
             // Redirect to the user page
-            return redirect()->to(Helper::getRedirectOption($request, $user->id, 'Users'))
+            return Helper::getRedirectOption($request, $user->id, 'Users')
                 ->with('success', trans('admin/users/message.success.update'));
         }
         return redirect()->back()->withInput()->withErrors($user->getErrors());
@@ -510,6 +512,8 @@ class UsersController extends Controller
                         trans('admin/companies/table.title'),
                         trans('admin/users/table.title'),
                         trans('general.employee_number'),
+                        trans('admin/users/table.first_name'),
+                        trans('admin/users/table.last_name'),
                         trans('admin/users/table.name'),
                         trans('admin/users/table.username'),
                         trans('admin/users/table.email'),
@@ -555,6 +559,8 @@ class UsersController extends Controller
                             ($user->company) ? $user->company->name : '',
                             $user->jobtitle,
                             $user->employee_num,
+                            $user->first_name,
+                            $user->last_name,
                             $user->present()->fullName(),
                             $user->username,
                             $user->email,

app/Http/Requests/UploadFileRequest.php

@@ -31,7 +31,7 @@ class UploadFileRequest extends Request
         $max_file_size = Helper::file_upload_max_size();
 
         return [
-            'file.*' => 'required|mimes:png,gif,jpg,svg,jpeg,doc,docx,pdf,txt,zip,rar,xls,xlsx,lic,xml,rtf,json,webp,avif|max:'.$max_file_size,
+            'file.*' => 'required|mimes:'.config('filesystems.allowed_upload_extensions_for_validator').'|max:'.$max_file_size,
         ];
     }
 

app/Http/Transformers/ActionlogsTransformer.php

@@ -113,8 +113,8 @@ class ActionlogsTransformer
 
                                         // Display the changes if the user is an admin or superadmin
                                         if (Gate::allows('admin')) {
-                                            $clean_meta[$fieldname]['old'] = ($enc_old) ? unserialize($enc_old): '';
-                                            $clean_meta[$fieldname]['new'] = ($enc_new) ? unserialize($enc_new): '';
+                                            $clean_meta[$fieldname]['old'] = ($enc_old) ? unserialize($enc_old, ['allowed_classes' => false]) : '';
+                                            $clean_meta[$fieldname]['new'] = ($enc_new) ? unserialize($enc_new, ['allowed_classes' => false]) : '';
                                         }
 
                                     }
@@ -198,8 +198,8 @@ class ActionlogsTransformer
             'note'          => ($actionlog->note) ? Helper::parseEscapedMarkedownInline($actionlog->note): null,
             'signature_file'   => ($actionlog->accept_signature) ? route('log.signature.view', ['filename' => $actionlog->accept_signature ]) : null,
             'log_meta'          => ((isset($clean_meta)) && (is_array($clean_meta))) ? $clean_meta: null,
-            'remote_ip'          => ($actionlog->remote_ip) ??  null,
-            'user_agent'          => ($actionlog->user_agent) ??  null,
+            'remote_ip' => e($actionlog->remote_ip) ?? null,
+            'user_agent' => e($actionlog->user_agent) ?? null,
             'action_source'          => ($actionlog->action_source) ??  null,
             'action_date'   => ($actionlog->action_date) ? Helper::getFormattedDateObject($actionlog->action_date, 'datetime'): Helper::getFormattedDateObject($actionlog->created_at, 'datetime'),
         ];

app/Http/Transformers/DepreciationsTransformer.php

@@ -26,11 +26,11 @@ class DepreciationsTransformer
         $array = [
             'id' => (int) $depreciation->id,
             'name' => e($depreciation->name),
-            'months' => $depreciation->months.' '.trans('general.months'),
+            'months' => trans_choice('general.months_plural', $depreciation->months),
             'depreciation_min' => $depreciation->depreciation_type === 'percent' ? $depreciation->depreciation_min.'%' : $depreciation->depreciation_min,
-            'assets_count' => $depreciation->assets_count,
-            'models_count' => $depreciation->models_count,
-            'licenses_count' => $depreciation->licenses_count,
+            'assets_count' => ($depreciation->assets_count > 0) ? (int) $depreciation->assets_count : 0,
+            'models_count' => ($depreciation->models_count > 0) ? (int) $depreciation->models_count : 0,
+            'licenses_count' => ($depreciation->licenses_count > 0) ? (int) $depreciation->licenses_count : 0,
             'created_by' => ($depreciation->adminuser) ? [
                 'id' => (int) $depreciation->adminuser->id,
                 'name'=> e($depreciation->adminuser->present()->fullName()),

app/Models/Accessory.php

@@ -15,7 +15,7 @@ use Watson\Validating\ValidatingTrait;
 /**
  * Model for Accessories.
  *
- * @version    v1.0
+ * @version v1.0
  */
 class Accessory extends SnipeModel
 {
@@ -56,8 +56,8 @@ class Accessory extends SnipeModel
     ];
 
     /**
-    * Accessory validation rules
-    */
+     * Accessory validation rules
+     */
     public $rules = [
         'name'              => 'required|min:3|max:255',
         'qty'               => 'required|integer|min:1',
@@ -71,12 +71,12 @@ class Accessory extends SnipeModel
 
 
     /**
-    * Whether the model should inject it's identifier to the unique
-    * validation rules before attempting validation. If this property
-    * is not set in the model it will default to true.
-    *
+     * Whether the model should inject it's identifier to the unique
+     * validation rules before attempting validation. If this property
+     * is not set in the model it will default to true.
+     *
      * @var bool
-    */
+     */
     protected $injectUniqueIdentifier = true;
     use ValidatingTrait;
 
@@ -108,7 +108,7 @@ class Accessory extends SnipeModel
      * Establishes the accessory -> supplier relationship
      *
      * @author [A. Gianotto] [<snipe@snipe.net>]
-     * @since [v3.0]
+     * @since  [v3.0]
      * @return \Illuminate\Database\Eloquent\Relations\Relation
      */
     public function supplier()
@@ -121,7 +121,7 @@ class Accessory extends SnipeModel
      * Sets the requestable attribute on the accessory
      *
      * @author [A. Gianotto] [<snipe@snipe.net>]
-     * @since [v4.0]
+     * @since  [v4.0]
      * @return void
      */
     public function setRequestableAttribute($value)
@@ -136,7 +136,7 @@ class Accessory extends SnipeModel
      * Establishes the accessory -> company relationship
      *
      * @author [A. Gianotto] [<snipe@snipe.net>]
-     * @since [v3.0]
+     * @since  [v3.0]
      * @return \Illuminate\Database\Eloquent\Relations\Relation
      */
     public function company()
@@ -148,7 +148,7 @@ class Accessory extends SnipeModel
      * Establishes the accessory -> location relationship
      *
      * @author [A. Gianotto] [<snipe@snipe.net>]
-     * @since [v3.0]
+     * @since  [v3.0]
      * @return \Illuminate\Database\Eloquent\Relations\Relation
      */
     public function location()
@@ -160,7 +160,7 @@ class Accessory extends SnipeModel
      * Establishes the accessory -> category relationship
      *
      * @author [A. Gianotto] [<snipe@snipe.net>]
-     * @since [v3.0]
+     * @since  [v3.0]
      * @return \Illuminate\Database\Eloquent\Relations\Relation
      */
     public function category()
@@ -172,7 +172,7 @@ class Accessory extends SnipeModel
      * Returns the action logs associated with the accessory
      *
      * @author [A. Gianotto] [<snipe@snipe.net>]
-     * @since [v3.0]
+     * @since  [v3.0]
      * @return \Illuminate\Database\Eloquent\Relations\Relation
      */
     public function assetlog()
@@ -201,8 +201,8 @@ class Accessory extends SnipeModel
      *
      * It's super-mega-assy, but it's the best I could do for now.
      *
-     * @author  A. Gianotto <snipe@snipe.net>
-     * @since v5.0.0
+     * @author A. Gianotto <snipe@snipe.net>
+     * @since  v5.0.0
      *
      * @see \App\Http\Controllers\Api\AccessoriesController\checkedout()
      */
@@ -219,7 +219,7 @@ class Accessory extends SnipeModel
      * presenter or service provider
      *
      * @author [A. Gianotto] [<snipe@snipe.net>]
-     * @since [v3.0]
+     * @since  [v3.0]
      * @return string
      */
     public function getImageUrl()
@@ -235,7 +235,7 @@ class Accessory extends SnipeModel
      * Establishes the accessory -> users relationship
      *
      * @author [A. Gianotto] [<snipe@snipe.net>]
-     * @since [v3.0]
+     * @since  [v3.0]
      * @return \Illuminate\Database\Eloquent\Relations\Relation
      */
     public function checkouts()
@@ -248,7 +248,7 @@ class Accessory extends SnipeModel
      * Establishes the accessory -> admin user relationship
      *
      * @author A. Gianotto <snipe@snipe.net>
-     * @since [v7.0.13]
+     * @since  [v7.0.13]
      * @return \Illuminate\Database\Eloquent\Relations\Relation
      */
     public function adminuser()
@@ -260,7 +260,7 @@ class Accessory extends SnipeModel
      * Checks whether or not the accessory has users
      *
      * @author [A. Gianotto] [<snipe@snipe.net>]
-     * @since [v3.0]
+     * @since  [v3.0]
      * @return int
      */
     public function hasUsers()
@@ -274,7 +274,7 @@ class Accessory extends SnipeModel
      * Establishes the accessory -> manufacturer relationship
      *
      * @author [A. Gianotto] [<snipe@snipe.net>]
-     * @since [v3.0]
+     * @since  [v3.0]
      * @return \Illuminate\Database\Eloquent\Relations\Relation
      */
     public function manufacturer()
@@ -287,7 +287,7 @@ class Accessory extends SnipeModel
      * accessory based on the category it belongs to.
      *
      * @author [A. Gianotto] [<snipe@snipe.net>]
-     * @since [v3.0]
+     * @since  [v3.0]
      * @return bool
      */
     public function checkin_email()
@@ -300,7 +300,7 @@ class Accessory extends SnipeModel
      * accept it via email.
      *
      * @author [A. Gianotto] [<snipe@snipe.net>]
-     * @since [v3.0]
+     * @since  [v3.0]
      * @return bool
      */
     public function requireAcceptance()
@@ -313,7 +313,7 @@ class Accessory extends SnipeModel
      * checks for a settings level EULA
      *
      * @author [A. Gianotto] [<snipe@snipe.net>]
-     * @since [v3.0]
+     * @since  [v3.0]
      * @return string
      */
     public function getEula()
@@ -333,7 +333,7 @@ class Accessory extends SnipeModel
      * Check how many items within an accessory are checked out
      *
      * @author [A. Gianotto] [<snipe@snipe.net>]
-     * @since [v5.0]
+     * @since  [v5.0]
      * @return int
      */
     public function numCheckedOut()
@@ -350,7 +350,7 @@ class Accessory extends SnipeModel
      * bad things happen.
      *
      * @author [A. Gianotto] [<snipe@snipe.net>]
-     * @since [v3.0]
+     * @since  [v3.0]
      * @return int
      */
     public function numRemaining()
@@ -365,8 +365,8 @@ class Accessory extends SnipeModel
     /**
      * Run after the checkout acceptance was declined by the user
      * 
-     * @param  User   $acceptedBy
-     * @param  string $signature
+     * @param User   $acceptedBy
+     * @param string $signature
      */
     public function declinedCheckout(User $declinedBy, $signature)
     {
@@ -392,8 +392,8 @@ class Accessory extends SnipeModel
      * This simply checks that there is a value for quantity, and if there isn't, set it to 0.
      *
      * @author A. Gianotto <snipe@snipe.net>
-     * @since v6.3.4
-     * @param $value
+     * @since  v6.3.4
+     * @param  $value
      * @return void
      */
     public function setQtyAttribute($value)
@@ -410,7 +410,6 @@ class Accessory extends SnipeModel
 
     /**
      * Query builder scope to order on created_by name
-     *
      */
     public function scopeOrderByCreatedByName($query, $order)
     {
@@ -418,68 +417,68 @@ class Accessory extends SnipeModel
     }
 
     /**
-    * Query builder scope to order on company
-    *
-    * @param  \Illuminate\Database\Query\Builder  $query  Query builder instance
-    * @param  text                              $order       Order
-    *
-    * @return \Illuminate\Database\Query\Builder          Modified query builder
-    */
+     * Query builder scope to order on company
+     *
+     * @param \Illuminate\Database\Query\Builder $query Query builder instance
+     * @param text                               $order Order
+     *
+     * @return \Illuminate\Database\Query\Builder          Modified query builder
+     */
     public function scopeOrderCompany($query, $order)
     {
         return $query->leftJoin('companies', 'accessories.company_id', '=', 'companies.id')
-        ->orderBy('companies.name', $order);
+            ->orderBy('companies.name', $order);
     }
 
     /**
-    * Query builder scope to order on category
-    *
-    * @param  \Illuminate\Database\Query\Builder  $query  Query builder instance
-    * @param  text                              $order       Order
-    *
-    * @return \Illuminate\Database\Query\Builder          Modified query builder
-    */
+     * Query builder scope to order on category
+     *
+     * @param \Illuminate\Database\Query\Builder $query Query builder instance
+     * @param text                               $order Order
+     *
+     * @return \Illuminate\Database\Query\Builder          Modified query builder
+     */
     public function scopeOrderCategory($query, $order)
     {
         return $query->leftJoin('categories', 'accessories.category_id', '=', 'categories.id')
-        ->orderBy('categories.name', $order);
+            ->orderBy('categories.name', $order);
     }
 
     /**
-    * Query builder scope to order on location
-    *
-    * @param  \Illuminate\Database\Query\Builder  $query  Query builder instance
-    * @param  text                              $order       Order
-    *
-    * @return \Illuminate\Database\Query\Builder          Modified query builder
-    */
+     * Query builder scope to order on location
+     *
+     * @param \Illuminate\Database\Query\Builder $query Query builder instance
+     * @param text                               $order Order
+     *
+     * @return \Illuminate\Database\Query\Builder          Modified query builder
+     */
     public function scopeOrderLocation($query, $order)
     {
         return $query->leftJoin('locations', 'accessories.location_id', '=', 'locations.id')
-        ->orderBy('locations.name', $order);
+            ->orderBy('locations.name', $order);
     }
 
     /**
-    * Query builder scope to order on manufacturer
-    *
-    * @param  \Illuminate\Database\Query\Builder  $query  Query builder instance
-    * @param  text                              $order       Order
-    *
-    * @return \Illuminate\Database\Query\Builder          Modified query builder
-    */
+     * Query builder scope to order on manufacturer
+     *
+     * @param \Illuminate\Database\Query\Builder $query Query builder instance
+     * @param text                               $order Order
+     *
+     * @return \Illuminate\Database\Query\Builder          Modified query builder
+     */
     public function scopeOrderManufacturer($query, $order)
     {
         return $query->leftJoin('manufacturers', 'accessories.manufacturer_id', '=', 'manufacturers.id')->orderBy('manufacturers.name', $order);
     }
 
     /**
-    * Query builder scope to order on supplier
-    *
-    * @param  \Illuminate\Database\Query\Builder  $query  Query builder instance
-    * @param  text                              $order       Order
-    *
-    * @return \Illuminate\Database\Query\Builder          Modified query builder
-    */
+     * Query builder scope to order on supplier
+     *
+     * @param \Illuminate\Database\Query\Builder $query Query builder instance
+     * @param text                               $order Order
+     *
+     * @return \Illuminate\Database\Query\Builder          Modified query builder
+     */
     public function scopeOrderSupplier($query, $order)
     {
         return $query->leftJoin('suppliers', 'accessories.supplier_id', '=', 'suppliers.id')->orderBy('suppliers.name', $order);

app/Models/AccessoryCheckout.php

@@ -16,7 +16,7 @@ use Watson\Validating\ValidatingTrait;
 /**
  * Model for Accessories.
  *
- * @version    v1.0
+ * @version v1.0
  */
 class AccessoryCheckout extends Model
 {
@@ -36,7 +36,7 @@ class AccessoryCheckout extends Model
      * Establishes the accessory checkout -> accessory relationship
      *
      * @author [A. Kroeger]
-     * @since [v7.0.9]
+     * @since  [v7.0.9]
      * @return \Illuminate\Database\Eloquent\Relations\Relation
      */
     public function accessory()
@@ -52,7 +52,7 @@ class AccessoryCheckout extends Model
      * Establishes the accessory checkout -> user relationship
      *
      * @author [A. Kroeger]
-     * @since [v7.0.9]
+     * @since  [v7.0.9]
      * @return \Illuminate\Database\Eloquent\Relations\Relation
      */
     public function adminuser()
@@ -64,7 +64,7 @@ class AccessoryCheckout extends Model
      * Get the target this asset is checked out to
      *
      * @author [A. Kroeger]
-     * @since [v7.0]
+     * @since  [v7.0]
      * @return \Illuminate\Database\Eloquent\Relations\Relation
      */
     public function assignedTo()
@@ -76,7 +76,7 @@ class AccessoryCheckout extends Model
      * Gets the lowercased name of the type of target the asset is assigned to
      *
      * @author [A. Gianotto] [<snipe@snipe.net>]
-     * @since [v4.0]
+     * @since  [v4.0]
      * @return string
      */
     public function assignedType()
@@ -91,7 +91,7 @@ class AccessoryCheckout extends Model
      * this method is an easy way of seeing if we are checked out to a user.
      *
      * @author [A. Kroeger]
-     * @since [v7.0]
+     * @since  [v7.0]
      */
     public function checkedOutToUser(): bool
     {
@@ -127,50 +127,64 @@ class AccessoryCheckout extends Model
      * Run additional, advanced searches.
      *
      * @param  \Illuminate\Database\Eloquent\Builder $query
-     * @param  array  $terms The search terms
+     * @param  array                                 $terms The search terms
      * @return \Illuminate\Database\Eloquent\Builder
      */
     public function advancedTextSearch(Builder $query, array $terms)
     {
 
-        $userQuery = User::where(function ($query) use ($terms) {
-            foreach ($terms as $term) {
-                $search_str = '%' . $term . '%';
-                $query->where('first_name', 'like', $search_str)
-                    ->orWhere('last_name', 'like', $search_str)
-                    ->orWhere('note', 'like', $search_str);
+        $userQuery = User::where(
+            function ($query) use ($terms) {
+                foreach ($terms as $term) {
+                    $search_str = '%' . $term . '%';
+                    $query->where('first_name', 'like', $search_str)
+                        ->orWhere('last_name', 'like', $search_str)
+                        ->orWhere('note', 'like', $search_str);
+                }
             }
-        })->select('id');
+        )->select('id');
 
-        $locationQuery = Location::where(function ($query) use ($terms) {
-            foreach ($terms as $term) {
-                $search_str = '%' . $term . '%';
-                $query->where('name', 'like', $search_str);
+        $locationQuery = Location::where(
+            function ($query) use ($terms) {
+                foreach ($terms as $term) {
+                    $search_str = '%' . $term . '%';
+                    $query->where('name', 'like', $search_str);
+                }
             }
-        })->select('id');
+        )->select('id');
 
-        $assetQuery = Asset::where(function ($query) use ($terms) {
-            foreach ($terms as $term) {
-                $search_str = '%' . $term . '%';
-                $query->where('name', 'like', $search_str);
+        $assetQuery = Asset::where(
+            function ($query) use ($terms) {
+                foreach ($terms as $term) {
+                    $search_str = '%' . $term . '%';
+                    $query->where('name', 'like', $search_str);
+                }
             }
-        })->select('id');
+        )->select('id');
 
-        $query->where(function ($query) use ($userQuery) {
-            $query->where('assigned_type', User::class)
-            ->whereIn('assigned_to', $userQuery);
-        })->orWhere(function($query) use ($locationQuery) {
-            $query->where('assigned_type', Location::class)
-            ->whereIn('assigned_to', $locationQuery);
-        })->orWhere(function($query) use ($assetQuery) {
-            $query->where('assigned_type', Asset::class)
-            ->whereIn('assigned_to', $assetQuery);
-        })->orWhere(function($query) use ($terms) {
-            foreach ($terms as $term) {
-                $search_str = '%' . $term . '%';
-                $query->where('note', 'like', $search_str);
+        $query->where(
+            function ($query) use ($userQuery) {
+                $query->where('assigned_type', User::class)
+                    ->whereIn('assigned_to', $userQuery);
             }
-        });
+        )->orWhere(
+            function ($query) use ($locationQuery) {
+                $query->where('assigned_type', Location::class)
+                    ->whereIn('assigned_to', $locationQuery);
+            }
+        )->orWhere(
+            function ($query) use ($assetQuery) {
+                $query->where('assigned_type', Asset::class)
+                    ->whereIn('assigned_to', $assetQuery);
+            }
+        )->orWhere(
+            function ($query) use ($terms) {
+                foreach ($terms as $term) {
+                    $search_str = '%' . $term . '%';
+                    $query->where('note', 'like', $search_str);
+                }
+            }
+        );
 
         return $query;
     }

app/Models/Actionlog.php

@@ -12,7 +12,7 @@ use Illuminate\Database\Eloquent\SoftDeletes;
  * Model for the Actionlog (the table that keeps a historical log of
  * checkouts, checkins, and updates).
  *
- * @version    v1.0
+ * @version v1.0
  */
 class Actionlog extends SnipeModel
 {
@@ -96,29 +96,31 @@ class Actionlog extends SnipeModel
      * Override from Builder to automatically add the company
      *
      * @author [A. Gianotto] [<snipe@snipe.net>]
-     * @since [v3.0]
+     * @since  [v3.0]
      * @return \Illuminate\Database\Eloquent\Relations\Relation
      */
     public static function boot()
     {
         parent::boot();
-        static::creating(function (self $actionlog) {
-            // If the admin is a superadmin, let's see if the target instead has a company.
-            if (auth()->user() && auth()->user()->isSuperUser()) {
-                if ($actionlog->target) {
-                    $actionlog->company_id = $actionlog->target->company_id;
-                } elseif ($actionlog->item) {
-                    $actionlog->company_id = $actionlog->item->company_id;
+        static::creating(
+            function (self $actionlog) {
+                // If the admin is a superadmin, let's see if the target instead has a company.
+                if (auth()->user() && auth()->user()->isSuperUser()) {
+                    if ($actionlog->target) {
+                        $actionlog->company_id = $actionlog->target->company_id;
+                    } elseif ($actionlog->item) {
+                        $actionlog->company_id = $actionlog->item->company_id;
+                    }
+                } elseif (auth()->user() && auth()->user()->company) {
+                    $actionlog->company_id = auth()->user()->company_id;
                 }
-            } elseif (auth()->user() && auth()->user()->company) {
-                $actionlog->company_id = auth()->user()->company_id;
-            }
 
-            if ($actionlog->action_date == '') {
-                $actionlog->action_date = Carbon::now();
-            }
+                if ($actionlog->action_date == '') {
+                    $actionlog->action_date = Carbon::now();
+                }
 
-        });
+            }
+        );
     }
 
 
@@ -126,7 +128,7 @@ class Actionlog extends SnipeModel
      * Establishes the actionlog -> item relationship
      *
      * @author [A. Gianotto] [<snipe@snipe.net>]
-     * @since [v3.0]
+     * @since  [v3.0]
      * @return \Illuminate\Database\Eloquent\Relations\Relation
      */
     public function item()
@@ -138,7 +140,7 @@ class Actionlog extends SnipeModel
      * Establishes the actionlog -> company relationship
      *
      * @author [A. Gianotto] [<snipe@snipe.net>]
-     * @since [v3.0]
+     * @since  [v3.0]
      * @return \Illuminate\Database\Eloquent\Relations\Relation
      */
     public function company()
@@ -151,7 +153,7 @@ class Actionlog extends SnipeModel
      * Establishes the actionlog -> asset relationship
      *
      * @author [A. Gianotto] [<snipe@snipe.net>]
-     * @since [v3.0]
+     * @since  [v3.0]
      * @return \Illuminate\Database\Eloquent\Relations\Relation
      */
     public function assets()
@@ -163,7 +165,7 @@ class Actionlog extends SnipeModel
      * Establishes the actionlog -> license relationship
      *
      * @author [A. Gianotto] [<snipe@snipe.net>]
-     * @since [v3.0]
+     * @since  [v3.0]
      * @return \Illuminate\Database\Eloquent\Relations\Relation
      */
     public function licenses()
@@ -175,7 +177,7 @@ class Actionlog extends SnipeModel
      * Establishes the actionlog -> consumable relationship
      *
      * @author [A. Gianotto] [<snipe@snipe.net>]
-     * @since [v3.0]
+     * @since  [v3.0]
      * @return \Illuminate\Database\Eloquent\Relations\Relation
      */
     public function consumables()
@@ -187,7 +189,7 @@ class Actionlog extends SnipeModel
      * Establishes the actionlog -> consumable relationship
      *
      * @author [A. Gianotto] [<snipe@snipe.net>]
-     * @since [v3.0]
+     * @since  [v3.0]
      * @return \Illuminate\Database\Eloquent\Relations\Relation
      */
     public function accessories()
@@ -199,7 +201,7 @@ class Actionlog extends SnipeModel
      * Establishes the actionlog -> components relationship
      *
      * @author [A. Gianotto] [<snipe@snipe.net>]
-     * @since [v3.0]
+     * @since  [v3.0]
      * @return \Illuminate\Database\Eloquent\Relations\Relation
      */
     public function components()
@@ -211,7 +213,7 @@ class Actionlog extends SnipeModel
      * Establishes the actionlog -> item type relationship
      *
      * @author [A. Gianotto] [<snipe@snipe.net>]
-     * @since [v3.0]
+     * @since  [v3.0]
      * @return \Illuminate\Database\Eloquent\Relations\Relation
      */
     public function itemType()
@@ -227,7 +229,7 @@ class Actionlog extends SnipeModel
      * Establishes the actionlog -> target type relationship
      *
      * @author [A. Gianotto] [<snipe@snipe.net>]
-     * @since [v3.0]
+     * @since  [v3.0]
      * @return \Illuminate\Database\Eloquent\Relations\Relation
      */
     public function targetType()
@@ -244,7 +246,7 @@ class Actionlog extends SnipeModel
      * Establishes the actionlog -> uploads relationship
      *
      * @author [A. Gianotto] [<snipe@snipe.net>]
-     * @since [v3.0]
+     * @since  [v3.0]
      * @return \Illuminate\Database\Eloquent\Relations\Relation
      */
     public function uploads()
@@ -258,7 +260,7 @@ class Actionlog extends SnipeModel
      * Establishes the actionlog -> userlog relationship
      *
      * @author [A. Gianotto] [<snipe@snipe.net>]
-     * @since [v3.0]
+     * @since  [v3.0]
      * @return \Illuminate\Database\Eloquent\Relations\Relation
      */
     public function userlog()
@@ -270,7 +272,7 @@ class Actionlog extends SnipeModel
      * Establishes the actionlog -> admin user relationship
      *
      * @author [A. Gianotto] [<snipe@snipe.net>]
-     * @since [v3.0]
+     * @since  [v3.0]
      * @return \Illuminate\Database\Eloquent\Relations\Relation
      */
     public function adminuser()
@@ -283,7 +285,7 @@ class Actionlog extends SnipeModel
      * Establishes the actionlog -> user relationship
      *
      * @author [A. Gianotto] [<snipe@snipe.net>]
-     * @since [v3.0]
+     * @since  [v3.0]
      * @return \Illuminate\Database\Eloquent\Relations\Relation
      */
     public function user()
@@ -296,7 +298,7 @@ class Actionlog extends SnipeModel
      * Establishes the actionlog -> target relationship
      *
      * @author [A. Gianotto] [<snipe@snipe.net>]
-     * @since [v3.0]
+     * @since  [v3.0]
      * @return \Illuminate\Database\Eloquent\Relations\Relation
      */
     public function target()
@@ -308,7 +310,7 @@ class Actionlog extends SnipeModel
      * Establishes the actionlog -> location relationship
      *
      * @author [A. Gianotto] [<snipe@snipe.net>]
-     * @since [v3.0]
+     * @since  [v3.0]
      * @return \Illuminate\Database\Eloquent\Relations\Relation
      */
     public function location()
@@ -321,7 +323,7 @@ class Actionlog extends SnipeModel
      * Check if the file exists, and if it does, force a download
      *
      * @author [A. Gianotto] [<snipe@snipe.net>]
-     * @since [v3.0]
+     * @since  [v3.0]
      * @return string | false
      */
     public function get_src($type = 'assets', $fieldname = 'filename')
@@ -339,7 +341,7 @@ class Actionlog extends SnipeModel
      * Saves the log record with the action type
      *
      * @author [A. Gianotto] [<snipe@snipe.net>]
-     * @since [v3.0]
+     * @since  [v3.0]
      * @return bool
      */
     public function logaction($actiontype)
@@ -360,7 +362,7 @@ class Actionlog extends SnipeModel
      * Calculate the number of days until the next audit
      *
      * @author [A. Gianotto] [<snipe@snipe.net>]
-     * @since [v4.0]
+     * @since  [v4.0]
      * @return int
      */
     public function daysUntilNextAudit($monthInterval = 12, $asset = null)
@@ -389,7 +391,7 @@ class Actionlog extends SnipeModel
      * Calculate the date of the next audit
      *
      * @author [A. Gianotto] [<snipe@snipe.net>]
-     * @since [v4.0]
+     * @since  [v4.0]
      * @return \Datetime
      */
     public function calcNextAuditDate($monthInterval = 12, $asset = null)
@@ -406,8 +408,8 @@ class Actionlog extends SnipeModel
     /**
      * Gets action logs in chronological order, excluding uploads
      *
-     * @author  Vincent Sposato <vincent.sposato@gmail.com>
-     * @since v1.0
+     * @author Vincent Sposato <vincent.sposato@gmail.com>
+     * @since  v1.0
      * @return \Illuminate\Database\Eloquent\Collection
      */
     public function getListingOfActionLogsChronologicalOrder()
@@ -423,7 +425,7 @@ class Actionlog extends SnipeModel
      * Determines what the type of request is so we can log it to the action_log
      *
      * @author A. Gianotto <snipe@snipe.net>
-     * @since v6.3.0
+     * @since  v6.3.0
      * @return string
      */
     public function determineActionSource(): string
@@ -435,7 +437,8 @@ class Actionlog extends SnipeModel
 
         // This is an API call
         if (((request()->header('content-type') && (request()->header('accept'))=='application/json'))
-            && (starts_with(request()->header('authorization'), 'Bearer '))) {
+            && (starts_with(request()->header('authorization'), 'Bearer '))
+        ) {
             return 'api';
         }
 
@@ -453,24 +456,24 @@ class Actionlog extends SnipeModel
     {
 
         switch ($this->item_type) {
-            case Accessory::class:
-                return route('show.accessoryfile', [$this->item_id, $this->id]);
-            case Asset::class:
-                return route('show/assetfile', [$this->item_id, $this->id]);
-            case AssetModel::class:
-                return route('show/modelfile', [$this->item_id, $this->id]);
-            case Consumable::class:
-                return route('show/locationsfile', [$this->item_id, $this->id]);
-            case Component::class:
-                return route('show.componentfile', [$this->item_id, $this->id]);
-            case License::class:
-                return route('show.licensefile', [$this->item_id, $this->id]);
-            case Location::class:
-                return route('show/locationsfile', [$this->item_id, $this->id]);
-            case User::class:
-                return route('show/userfile', [$this->item_id, $this->id]);
-            default:
-                return null;
+        case Accessory::class:
+            return route('show.accessoryfile', [$this->item_id, $this->id]);
+        case Asset::class:
+            return route('show/assetfile', [$this->item_id, $this->id]);
+        case AssetModel::class:
+            return route('show/modelfile', [$this->item_id, $this->id]);
+        case Consumable::class:
+            return route('show/locationsfile', [$this->item_id, $this->id]);
+        case Component::class:
+            return route('show.componentfile', [$this->item_id, $this->id]);
+        case License::class:
+            return route('show.licensefile', [$this->item_id, $this->id]);
+        case Location::class:
+            return route('show/locationsfile', [$this->item_id, $this->id]);
+        case User::class:
+            return route('show/userfile', [$this->item_id, $this->id]);
+        default:
+            return null;
         }
     }
 
@@ -478,24 +481,24 @@ class Actionlog extends SnipeModel
     {
 
         switch ($this->item_type) {
-            case Accessory::class:
-                return 'private_uploads/accessories/'.$this->filename;
-            case Asset::class:
-                return 'private_uploads/assets/'.$this->filename;
-            case AssetModel::class:
-                return 'private_uploads/assetmodels/'.$this->filename;
-            case Consumable::class:
-                return 'private_uploads/consumables/'.$this->filename;
-            case Component::class:
-                return 'private_uploads/components/'.$this->filename;
-            case License::class:
-                return 'private_uploads/licenses/'.$this->filename;
-            case Location::class:
-                return 'private_uploads/locations/'.$this->filename;
-            case User::class:
-                return 'private_uploads/users/'.$this->filename;
-            default:
-                return null;
+        case Accessory::class:
+            return 'private_uploads/accessories/'.$this->filename;
+        case Asset::class:
+            return 'private_uploads/assets/'.$this->filename;
+        case AssetModel::class:
+            return 'private_uploads/assetmodels/'.$this->filename;
+        case Consumable::class:
+            return 'private_uploads/consumables/'.$this->filename;
+        case Component::class:
+            return 'private_uploads/components/'.$this->filename;
+        case License::class:
+            return 'private_uploads/licenses/'.$this->filename;
+        case Location::class:
+            return 'private_uploads/locations/'.$this->filename;
+        case User::class:
+            return 'private_uploads/users/'.$this->filename;
+        default:
+            return null;
         }
     }
 

app/Models/AssetMaintenance.php

@@ -12,7 +12,7 @@ use Watson\Validating\ValidatingTrait;
 /**
  * Model for Asset Maintenances.
  *
- * @version    v1.0
+ * @version v1.0
  */
 class AssetMaintenance extends Model implements ICompanyableChild
 {
@@ -93,7 +93,7 @@ class AssetMaintenance extends Model implements ICompanyableChild
     /**
      * getImprovementOptions
      *
-     * @return array
+     * @return  array
      * @author  Vincent Sposato <vincent.sposato@gmail.com>
      * @version v1.0
      */
@@ -157,21 +157,21 @@ class AssetMaintenance extends Model implements ICompanyableChild
      * asset
      * Get asset for this improvement
      *
-     * @return mixed
+     * @return  mixed
      * @author  Vincent Sposato <vincent.sposato@gmail.com>
      * @version v1.0
      */
     public function asset()
     {
         return $this->belongsTo(\App\Models\Asset::class, 'asset_id')
-                    ->withTrashed();
+            ->withTrashed();
     }
     
 
     /**
      * Get the admin who created the maintenance
      *
-     * @return mixed
+     * @return  mixed
      * @author  A. Gianotto <snipe@snipe.net>
      * @version v3.0
      */
@@ -184,7 +184,7 @@ class AssetMaintenance extends Model implements ICompanyableChild
     public function supplier()
     {
         return $this->belongsTo(\App\Models\Supplier::class, 'supplier_id')
-                    ->withTrashed();
+            ->withTrashed();
     }
 
     /**
@@ -196,8 +196,8 @@ class AssetMaintenance extends Model implements ICompanyableChild
     /**
      * Query builder scope to order on a supplier
      *
-     * @param  \Illuminate\Database\Query\Builder  $query  Query builder instance
-     * @param  string                              $order       Order
+     * @param \Illuminate\Database\Query\Builder $query Query builder instance
+     * @param string                             $order Order
      *
      * @return \Illuminate\Database\Query\Builder          Modified query builder
      */
@@ -212,8 +212,8 @@ class AssetMaintenance extends Model implements ICompanyableChild
     /**
      * Query builder scope to order on asset tag
      *
-     * @param  \Illuminate\Database\Query\Builder  $query  Query builder instance
-     * @param  string                              $order       Order
+     * @param \Illuminate\Database\Query\Builder $query Query builder instance
+     * @param string                             $order Order
      *
      * @return \Illuminate\Database\Query\Builder          Modified query builder
      */
@@ -226,8 +226,8 @@ class AssetMaintenance extends Model implements ICompanyableChild
     /**
      * Query builder scope to order on asset tag
      *
-     * @param  \Illuminate\Database\Query\Builder  $query  Query builder instance
-     * @param  string                              $order       Order
+     * @param \Illuminate\Database\Query\Builder $query Query builder instance
+     * @param string                             $order Order
      *
      * @return \Illuminate\Database\Query\Builder          Modified query builder
      */
@@ -240,8 +240,8 @@ class AssetMaintenance extends Model implements ICompanyableChild
     /**
      * Query builder scope to order on serial
      *
-     * @param  \Illuminate\Database\Query\Builder  $query  Query builder instance
-     * @param  string                              $order       Order
+     * @param \Illuminate\Database\Query\Builder $query Query builder instance
+     * @param string                             $order Order
      *
      * @return \Illuminate\Database\Query\Builder          Modified query builder
      */
@@ -254,8 +254,8 @@ class AssetMaintenance extends Model implements ICompanyableChild
     /**
      * Query builder scope to order on status label name
      *
-     * @param  \Illuminate\Database\Query\Builder  $query  Query builder instance
-     * @param  text                              $order         Order
+     * @param \Illuminate\Database\Query\Builder $query Query builder instance
+     * @param text                               $order Order
      *
      * @return \Illuminate\Database\Query\Builder          Modified query builder
      */
@@ -269,8 +269,8 @@ class AssetMaintenance extends Model implements ICompanyableChild
     /**
      * Query builder scope to order on status label name
      *
-     * @param  \Illuminate\Database\Query\Builder  $query  Query builder instance
-     * @param  text                              $order         Order
+     * @param \Illuminate\Database\Query\Builder $query Query builder instance
+     * @param text                               $order Order
      *
      * @return \Illuminate\Database\Query\Builder          Modified query builder
      */

app/Models/AssetModel.php

@@ -17,7 +17,7 @@ use App\Http\Traits\TwoColumnUniqueUndeletedTrait;
  * Model for Asset Models. Asset Models contain higher level
  * attributes that are common among the same type of asset.
  *
- * @version    v1.0
+ * @version v1.0
  */
 class AssetModel extends SnipeModel
 {
@@ -105,7 +105,7 @@ class AssetModel extends SnipeModel
      * Establishes the model -> assets relationship
      *
      * @author [A. Gianotto] [<snipe@snipe.net>]
-     * @since [v1.0]
+     * @since  [v1.0]
      * @return \Illuminate\Database\Eloquent\Relations\Relation
      */
     public function assets()
@@ -117,7 +117,7 @@ class AssetModel extends SnipeModel
      * Establishes the model -> category relationship
      *
      * @author [A. Gianotto] [<snipe@snipe.net>]
-     * @since [v1.0]
+     * @since  [v1.0]
      * @return \Illuminate\Database\Eloquent\Relations\Relation
      */
     public function category()
@@ -129,7 +129,7 @@ class AssetModel extends SnipeModel
      * Establishes the model -> depreciation relationship
      *
      * @author [A. Gianotto] [<snipe@snipe.net>]
-     * @since [v1.0]
+     * @since  [v1.0]
      * @return \Illuminate\Database\Eloquent\Relations\Relation
      */
     public function depreciation()
@@ -141,7 +141,7 @@ class AssetModel extends SnipeModel
      * Establishes the model -> manufacturer relationship
      *
      * @author [A. Gianotto] [<snipe@snipe.net>]
-     * @since [v1.0]
+     * @since  [v1.0]
      * @return \Illuminate\Database\Eloquent\Relations\Relation
      */
     public function manufacturer()
@@ -153,7 +153,7 @@ class AssetModel extends SnipeModel
      * Establishes the model -> fieldset relationship
      *
      * @author [A. Gianotto] [<snipe@snipe.net>]
-     * @since [v2.0]
+     * @since  [v2.0]
      * @return \Illuminate\Database\Eloquent\Relations\Relation
      */
     public function fieldset()
@@ -163,14 +163,14 @@ class AssetModel extends SnipeModel
    
     public function customFields()
     {
-       return $this->fieldset()->first()->fields(); 
+        return $this->fieldset()->first()->fields(); 
     }
 
     /**
      * Establishes the model -> custom field default values relationship
      *
      * @author hannah tinkler
-     * @since [v4.3]
+     * @since  [v4.3]
      * @return \Illuminate\Database\Eloquent\Relations\Relation
      */
     public function defaultValues()
@@ -184,7 +184,7 @@ class AssetModel extends SnipeModel
      * @todo this should probably be moved
      *
      * @author [A. Gianotto] [<snipe@snipe.net>]
-     * @since [v2.0]
+     * @since  [v2.0]
      * @return \Illuminate\Database\Eloquent\Relations\Relation
      */
     public function getImageUrl()
@@ -201,7 +201,7 @@ class AssetModel extends SnipeModel
      * Checks if the model is deletable
      *
      * @author A. Gianotto <snipe@snipe.net>
-     * @since [v6.3.4]
+     * @since  [v6.3.4]
      * @return bool
      */
     public function isDeletable()
@@ -216,7 +216,7 @@ class AssetModel extends SnipeModel
      * Get user who created the item
      *
      * @author [A. Gianotto] [<snipe@snipe.net>]
-     * @since [v1.0]
+     * @since  [v1.0]
      * @return \Illuminate\Database\Eloquent\Relations\Relation
      */
     public function adminuser()
@@ -235,10 +235,10 @@ class AssetModel extends SnipeModel
      * scopeInCategory
      * Get all models that are in the array of category ids
      *
-     * @param       $query
+     * @param $query
      * @param array $categoryIdListing
      *
-     * @return mixed
+     * @return  mixed
      * @author  Vincent Sposato <vincent.sposato@gmail.com>
      * @version v1.0
      */
@@ -251,9 +251,9 @@ class AssetModel extends SnipeModel
      * scopeRequestable
      * Get all models that are requestable by a user.
      *
-     * @param       $query
+     * @param $query
      *
-     * @return $query
+     * @return  $query
      * @author  Daniel Meltzer <dmeltzer.devel@gmail.com>
      * @version v3.5
      */
@@ -265,8 +265,8 @@ class AssetModel extends SnipeModel
     /**
      * Query builder scope to search on text, including catgeory and manufacturer name
      *
-     * @param  Illuminate\Database\Query\Builder  $query  Query builder instance
-     * @param  text                              $search      Search term
+     * @param Illuminate\Database\Query\Builder $query  Query builder instance
+     * @param text                              $search Search term
      *
      * @return Illuminate\Database\Query\Builder          Modified query builder
      */
@@ -274,23 +274,31 @@ class AssetModel extends SnipeModel
     {
         return $query->where('models.name', 'LIKE', "%$search%")
             ->orWhere('model_number', 'LIKE', "%$search%")
-            ->orWhere(function ($query) use ($search) {
-                $query->whereHas('category', function ($query) use ($search) {
-                    $query->where('categories.name', 'LIKE', '%'.$search.'%');
-                });
-            })
-            ->orWhere(function ($query) use ($search) {
-                $query->whereHas('manufacturer', function ($query) use ($search) {
-                    $query->where('manufacturers.name', 'LIKE', '%'.$search.'%');
-                });
-            });
+            ->orWhere(
+                function ($query) use ($search) {
+                    $query->whereHas(
+                        'category', function ($query) use ($search) {
+                            $query->where('categories.name', 'LIKE', '%'.$search.'%');
+                        }
+                    );
+                }
+            )
+            ->orWhere(
+                function ($query) use ($search) {
+                    $query->whereHas(
+                        'manufacturer', function ($query) use ($search) {
+                            $query->where('manufacturers.name', 'LIKE', '%'.$search.'%');
+                        }
+                    );
+                }
+            );
     }
 
     /**
      * Query builder scope to order on manufacturer
      *
-     * @param  \Illuminate\Database\Query\Builder  $query  Query builder instance
-     * @param  text                              $order       Order
+     * @param \Illuminate\Database\Query\Builder $query Query builder instance
+     * @param text                               $order Order
      *
      * @return \Illuminate\Database\Query\Builder          Modified query builder
      */
@@ -302,8 +310,8 @@ class AssetModel extends SnipeModel
     /**
      * Query builder scope to order on category name
      *
-     * @param  \Illuminate\Database\Query\Builder  $query  Query builder instance
-     * @param  text                              $order       Order
+     * @param \Illuminate\Database\Query\Builder $query Query builder instance
+     * @param text                               $order Order
      *
      * @return \Illuminate\Database\Query\Builder          Modified query builder
      */
@@ -319,7 +327,6 @@ class AssetModel extends SnipeModel
 
     /**
      * Query builder scope to order on created_by name
-     *
      */
     public function scopeOrderByCreatedByName($query, $order)
     {

app/Models/Category.php

@@ -18,7 +18,7 @@ use Illuminate\Support\Str;
  * to require acceptance from the user, whether or not to
  * send a EULA to the user, etc.
  *
- * @version    v1.0
+ * @version v1.0
  */
 class Category extends SnipeModel
 {
@@ -96,7 +96,7 @@ class Category extends SnipeModel
      * Checks if category can be deleted
      *
      * @author [Dan Meltzer] [<dmeltzer.devel@gmail.com>]
-     * @since [v5.0]
+     * @since  [v5.0]
      * @return bool
      */
     public function isDeletable()
@@ -119,7 +119,7 @@ class Category extends SnipeModel
      * Establishes the category -> accessories relationship
      *
      * @author [A. Gianotto] [<snipe@snipe.net>]
-     * @since [v2.0]
+     * @since  [v2.0]
      * @return \Illuminate\Database\Eloquent\Relations\Relation
      */
     public function accessories()
@@ -131,7 +131,7 @@ class Category extends SnipeModel
      * Establishes the category -> licenses relationship
      *
      * @author [A. Gianotto] [<snipe@snipe.net>]
-     * @since [v4.3]
+     * @since  [v4.3]
      * @return \Illuminate\Database\Eloquent\Relations\Relation
      */
     public function licenses()
@@ -143,7 +143,7 @@ class Category extends SnipeModel
      * Establishes the category -> consumables relationship
      *
      * @author [A. Gianotto] [<snipe@snipe.net>]
-     * @since [v3.0]
+     * @since  [v3.0]
      * @return \Illuminate\Database\Eloquent\Relations\Relation
      */
     public function consumables()
@@ -155,7 +155,7 @@ class Category extends SnipeModel
      * Establishes the category -> consumables relationship
      *
      * @author [A. Gianotto] [<snipe@snipe.net>]
-     * @since [v3.0]
+     * @since  [v3.0]
      * @return \Illuminate\Database\Eloquent\Relations\Relation
      */
     public function components()
@@ -170,7 +170,7 @@ class Category extends SnipeModel
      * It should only be used in a single category context.
      *
      * @author [A. Gianotto] [<snipe@snipe.net>]
-     * @since [v2.0]
+     * @since  [v2.0]
      * @return int
      */
     public function itemCount()
@@ -181,18 +181,18 @@ class Category extends SnipeModel
         }
 
         switch ($this->category_type) {
-            case 'asset':
-                return $this->assets->count();
-            case 'accessory':
-                return $this->accessories->count();
-            case 'component':
-                return $this->components->count();
-            case 'consumable':
-                return $this->consumables->count();
-            case 'license':
-                return $this->licenses->count();
-            default:
-                return 0;
+        case 'asset':
+            return $this->assets->count();
+        case 'accessory':
+            return $this->accessories->count();
+        case 'component':
+            return $this->components->count();
+        case 'consumable':
+            return $this->consumables->count();
+        case 'license':
+            return $this->licenses->count();
+        default:
+            return 0;
         }
 
     }
@@ -201,7 +201,7 @@ class Category extends SnipeModel
      * Establishes the category -> assets relationship
      *
      * @author [A. Gianotto] [<snipe@snipe.net>]
-     * @since [v2.0]
+     * @since  [v2.0]
      * @return \Illuminate\Database\Eloquent\Relations\Relation
      */
     public function assets()
@@ -218,8 +218,8 @@ class Category extends SnipeModel
      * by their category.
      *
      * @author [A. Gianotto] [<snipe@snipe.net>]
-     * @since [v6.1.0]
-     * @see \App\Models\Asset::scopeAssetsForShow()
+     * @since  [v6.1.0]
+     * @see    \App\Models\Asset::scopeAssetsForShow()
      * @return \Illuminate\Database\Eloquent\Relations\Relation
      */
     public function showableAssets()
@@ -231,7 +231,7 @@ class Category extends SnipeModel
      * Establishes the category -> models relationship
      *
      * @author [A. Gianotto] [<snipe@snipe.net>]
-     * @since [v2.0]
+     * @since  [v2.0]
      * @return \Illuminate\Database\Eloquent\Relations\Relation
      */
     public function models()
@@ -249,7 +249,7 @@ class Category extends SnipeModel
      * checks for a settings level EULA
      *
      * @author [A. Gianotto] [<snipe@snipe.net>]
-     * @since [v2.0]
+     * @since  [v2.0]
      * @return string | null
      */
     public function getEula()
@@ -276,7 +276,7 @@ class Category extends SnipeModel
      *
      * This will also correctly parse a 1/0 if "true"/"false" is passed.
      *
-     * @param $value
+     * @param  $value
      * @return void
      */
     public function setCheckinEmailAttribute($value)
@@ -293,9 +293,9 @@ class Category extends SnipeModel
     /**
      * Query builder scope for whether or not the category requires acceptance
      *
-     * @author  Vincent Sposato <vincent.sposato@gmail.com>
+     * @author Vincent Sposato <vincent.sposato@gmail.com>
      *
-     * @param  \Illuminate\Database\Query\Builder  $query  Query builder instance
+     * @param  \Illuminate\Database\Query\Builder $query Query builder instance
      * @return \Illuminate\Database\Query\Builder          Modified query builder
      */
     public function scopeRequiresAcceptance($query)

app/Models/CheckoutAcceptance.php

@@ -66,7 +66,7 @@ class CheckoutAcceptance extends Model
     /**
      * Was the checkoutable checked out to this user?
      *
-     * @param  User    $user
+     * @param  User $user
      * @return bool
      */
     public function isCheckedOutTo(User $user)
@@ -79,7 +79,7 @@ class CheckoutAcceptance extends Model
      * Do not add stuff here that doesn't have a corresponding column in the
      * checkout_acceptances table or you'll get an error.
      *
-     * @param  string $signature_filename
+     * @param string $signature_filename
      */
     public function accept($signature_filename, $eula = null, $filename = null, $note = null)
     {
@@ -99,7 +99,7 @@ class CheckoutAcceptance extends Model
     /**
      * Decline the checkout acceptance
      *
-     * @param  string $signature_filename
+     * @param string $signature_filename
      */
     public function decline($signature_filename, $note = null)
     {
@@ -116,8 +116,9 @@ class CheckoutAcceptance extends Model
 
     /**
      * Filter checkout acceptences by the user
+     *
      * @param  Illuminate\Database\Eloquent\Builder $query
-     * @param  User    $user
+     * @param  User                                 $user
      * @return \Illuminate\Database\Eloquent\Builder
      */
     public function scopeForUser(Builder $query, User $user)
@@ -127,6 +128,7 @@ class CheckoutAcceptance extends Model
 
     /**
      * Filter to only get pending acceptances
+     *
      * @param  Illuminate\Database\Eloquent\Builder $query
      * @return \Illuminate\Database\Eloquent\Builder
      */

app/Models/Company.php

@@ -13,11 +13,13 @@ use Illuminate\Support\Facades\Schema;
 /**
  * Model for Companies.
  *
- * @version    v1.8
+ * @version v1.8
  */
 final class Company extends SnipeModel
 {
     use HasFactory;
+    use CompanyableTrait;
+
 
     protected $table = 'companies';
 
@@ -26,19 +28,19 @@ final class Company extends SnipeModel
         'name' => 'required|min:1|max:255|unique:companies,name',
         'fax' => 'min:7|max:35|nullable',
         'phone' => 'min:7|max:35|nullable',
-		'email' => 'email|max:150|nullable',
+    'email' => 'email|max:150|nullable',
     ];
 
     protected $presenter = \App\Presenters\CompanyPresenter::class;
     use Presentable;
 
     /**
-    * Whether the model should inject it's identifier to the unique
-    * validation rules before attempting validation. If this property
-    * is not set in the model it will default to true.
-    *
+     * Whether the model should inject it's identifier to the unique
+     * validation rules before attempting validation. If this property
+     * is not set in the model it will default to true.
+     *
      * @var bool
-    */
+     */
     protected $injectUniqueIdentifier = true;
     use ValidatingTrait;
     use Searchable;
@@ -100,7 +102,7 @@ final class Company extends SnipeModel
      * account the full multiple company support setting
      * and if the current user is a super user.
      *
-     * @param $unescaped_input
+     * @param  $unescaped_input
      * @return int|mixed|string|null
      */
     public static function getIdForCurrentUser($unescaped_input)
@@ -127,7 +129,7 @@ final class Company extends SnipeModel
      * Check to see if the current user should have access to the model.
      * I hate this method and I think it should be refactored.
      *
-     * @param $companyable
+     * @param  $companyable
      * @return bool|void
      */
     public static function isCurrentUserHasAccess($companyable)
@@ -146,10 +148,10 @@ final class Company extends SnipeModel
         if (!is_string($companyable)) {
             $company_table = $companyable->getModel()->getTable();
             try {
-                // This is primary for the gate:allows-check in location->isDeletable()
+                // This is primarily for the gate:allows-check in location->isDeletable()
                 // Locations don't have a company_id so without this it isn't possible to delete locations with FullMultipleCompanySupport enabled
                 // because this function is called by SnipePermissionsPolicy->before()
-                if (!$companyable instanceof Company && !Schema::hasColumn($company_table, 'company_id')) {
+                if (!Schema::hasColumn($company_table, 'company_id')) {
                     return true;
                 }
 
@@ -163,9 +165,16 @@ final class Company extends SnipeModel
             // Log::warning('Companyable is '.$companyable);
             $current_user_company_id = auth()->user()->company_id;
             $companyable_company_id = $companyable->company_id;
-            return $current_user_company_id == null || $current_user_company_id == $companyable_company_id || auth()->user()->isSuperUser();
+
+            // Set this to check companyable on company
+            if ($companyable instanceof Company) {
+                $companyable_company_id = $companyable->id;
+            }
+            return ($current_user_company_id == null) || ($current_user_company_id == $companyable_company_id) || auth()->user()->isSuperUser();
         }
 
+        return false;
+
     }
 
     public static function isCurrentUserAuthorized()
@@ -183,7 +192,7 @@ final class Company extends SnipeModel
      * Checks if company can be deleted
      *
      * @author [Dan Meltzer] [<dmeltzer.devel@gmail.com>]
-     * @since [v5.0]
+     * @since  [v5.0]
      * @return bool
      */
     public function isDeletable()
@@ -200,7 +209,7 @@ final class Company extends SnipeModel
     }
 
     /**
-     * @param $unescaped_input
+     * @param  $unescaped_input
      * @return int|mixed|string|null
      */
     public static function getIdForUser($unescaped_input)
@@ -256,14 +265,14 @@ final class Company extends SnipeModel
      * @todo - refactor that trait to handle the user's model as well.
      *
      * @author [A. Gianotto] <snipe@snipe.net>
-     * @param $query
-     * @param $column
-     * @param $table_name
+     * @param  $query
+     * @param  $column
+     * @param  $table_name
      * @return mixed
      */
     public static function scopeCompanyables($query, $column = 'company_id', $table_name = null)
     {
-        // If not logged in and hitting this, assume we are on the command line and don't scope?'
+        // If not logged in and hitting this, assume we are on the command line and don't scope?
         if (! static::isFullMultipleCompanySupportEnabled() || (Auth::hasUser() && auth()->user()->isSuperUser()) || (! Auth::hasUser())) {
             return $query;
         } else {
@@ -280,11 +289,16 @@ final class Company extends SnipeModel
     private static function scopeCompanyablesDirectly($query, $column = 'company_id', $table_name = null)
     {
 
+        $company_id = null;
         // Get the company ID of the logged-in user, or set it to null if there is no company associated with the user
         if (Auth::hasUser()) {
             $company_id = auth()->user()->company_id;
-        } else {
-            $company_id = null;
+        }
+
+
+        // If we are scoping the companies table itself, look for the company.id
+        if ($query->getModel()->getTable() == 'companies') {
+            return $query->where('companies.id', '=', $company_id);
         }
 
 
@@ -297,6 +311,8 @@ final class Company extends SnipeModel
             return $query->where($table.$column, '=', $company_id);
         }
 
+
+
     }
 
     public function adminuser()
@@ -311,8 +327,8 @@ final class Company extends SnipeModel
      * This gets invoked by CompanyableChildScope, but I'm not sure what it does.
      *
      * @author [A. Gianotto] <snipe@snipe.net>
-     * @param array $companyable_names
-     * @param $query
+     * @param  array $companyable_names
+     * @param  $query
      * @return mixed
      */
     public static function scopeCompanyableChildren(array $companyable_names, $query)
@@ -324,17 +340,18 @@ final class Company extends SnipeModel
             return $query;
         } else {
             $f = function ($q) {
-                Log::debug('scopeCompanyablesDirectly firing ');
                 static::scopeCompanyablesDirectly($q);
             };
 
-            $q = $query->where(function ($q) use ($companyable_names, $f) {
-                $q2 = $q->whereHas($companyable_names[0], $f);
+            $q = $query->where(
+                function ($q) use ($companyable_names, $f) {
+                    $q2 = $q->whereHas($companyable_names[0], $f);
 
-                for ($i = 1; $i < count($companyable_names); $i++) {
-                    $q2 = $q2->orWhereHas($companyable_names[$i], $f);
+                    for ($i = 1; $i < count($companyable_names); $i++) {
+                        $q2 = $q2->orWhereHas($companyable_names[$i], $f);
+                    }
                 }
-            });
+            );
 
             return $q;
         }

app/Models/CompanyableChildScope.php

@@ -9,15 +9,15 @@ use Illuminate\Database\Eloquent\Scope;
 /**
  * Handle query scoping for full company support.
  *
- * @todo Move this to a more Laravel 5.2 esque way
- * @version    v1.0
+ * @todo    Move this to a more Laravel 5.2 esque way
+ * @version v1.0
  */
 final class CompanyableChildScope implements Scope
 {
     /**
      * Apply the scope to a given Eloquent query builder.
      *
-     * @param  \Illuminate\Database\Eloquent\Builder  $builder
+     * @param  \Illuminate\Database\Eloquent\Builder $builder
      * @return void
      */
     public function apply(Builder $builder, Model $model)
@@ -31,7 +31,7 @@ final class CompanyableChildScope implements Scope
      * @todo IMPLEMENT
      * Remove the scope from the given Eloquent query builder.
      *
-     * @param  \Illuminate\Database\Eloquent\Builder  $builder
+     * @param  \Illuminate\Database\Eloquent\Builder $builder
      * @return void
      */
     public function remove(Builder $builder)

app/Models/CompanyableScope.php

@@ -9,15 +9,15 @@ use Illuminate\Database\Eloquent\Scope;
 /**
  * Handle query scoping for full company support.
  *
- * @todo Move this to a more Laravel 5.2 esque way
- * @version    v1.0
+ * @todo    Move this to a more Laravel 5.2 esque way
+ * @version v1.0
  */
 final class CompanyableScope implements Scope
 {
     /**
      * Apply the scope to a given Eloquent query builder.
      *
-     * @param  \Illuminate\Database\Eloquent\Builder  $builder
+     * @param  \Illuminate\Database\Eloquent\Builder $builder
      * @return void
      */
     public function apply(Builder $builder, Model $model)
@@ -29,7 +29,7 @@ final class CompanyableScope implements Scope
      * @todo IMPLEMENT
      * Remove the scope from the given Eloquent query builder.
      *
-     * @param  \Illuminate\Database\Eloquent\Builder  $builder
+     * @param  \Illuminate\Database\Eloquent\Builder $builder
      * @return void
      */
     public function remove(Builder $builder)

app/Models/CompanyableTrait.php

@@ -8,7 +8,7 @@ trait CompanyableTrait
      * This trait is used to scope models to the current company. To use this scope on companyable models,
      * we use the "use Companyable;" statement at the top of the mode.
      *
-     * @see \App\Models\Company\Company::scopeCompanyables()
+     * @see    \App\Models\Company\Company::scopeCompanyables()
      * @return void
      */
     public static function bootCompanyableTrait()

app/Models/Component.php

@@ -13,7 +13,7 @@ use Watson\Validating\ValidatingTrait;
 /**
  * Model for Components.
  *
- * @version    v1.0
+ * @version v1.0
  */
 class Component extends SnipeModel
 {
@@ -121,7 +121,7 @@ class Component extends SnipeModel
      * Establishes the component -> location relationship
      *
      * @author [A. Gianotto] [<snipe@snipe.net>]
-     * @since [v3.0]
+     * @since  [v3.0]
      * @return \Illuminate\Database\Eloquent\Relations\Relation
      */
     public function location()
@@ -133,7 +133,7 @@ class Component extends SnipeModel
      * Establishes the component -> assets relationship
      *
      * @author [A. Gianotto] [<snipe@snipe.net>]
-     * @since [v3.0]
+     * @since  [v3.0]
      * @return \Illuminate\Database\Eloquent\Relations\Relation
      */
     public function assets()
@@ -147,7 +147,7 @@ class Component extends SnipeModel
      * @todo this is probably not needed - refactor
      *
      * @author [A. Gianotto] [<snipe@snipe.net>]
-     * @since [v3.0]
+     * @since  [v3.0]
      * @return \Illuminate\Database\Eloquent\Relations\Relation
      */
     public function adminuser()
@@ -159,7 +159,7 @@ class Component extends SnipeModel
      * Establishes the component -> company relationship
      *
      * @author [A. Gianotto] [<snipe@snipe.net>]
-     * @since [v3.0]
+     * @since  [v3.0]
      * @return \Illuminate\Database\Eloquent\Relations\Relation
      */
     public function company()
@@ -171,7 +171,7 @@ class Component extends SnipeModel
      * Establishes the component -> category relationship
      *
      * @author [A. Gianotto] [<snipe@snipe.net>]
-     * @since [v3.0]
+     * @since  [v3.0]
      * @return \Illuminate\Database\Eloquent\Relations\Relation
      */
     public function category()
@@ -183,7 +183,7 @@ class Component extends SnipeModel
      * Establishes the item -> supplier relationship
      *
      * @author [A. Gianotto] [<snipe@snipe.net>]
-     * @since [v6.1.1]
+     * @since  [v6.1.1]
      * @return \Illuminate\Database\Eloquent\Relations\Relation
      */
     public function supplier()
@@ -196,7 +196,7 @@ class Component extends SnipeModel
      * Establishes the item -> manufacturer relationship
      *
      * @author [A. Gianotto] [<snipe@snipe.net>]
-     * @since [v3.0]
+     * @since  [v3.0]
      * @return \Illuminate\Database\Eloquent\Relations\Relation
      */
     public function manufacturer()
@@ -208,7 +208,7 @@ class Component extends SnipeModel
      * Establishes the component -> action logs relationship
      *
      * @author [A. Gianotto] [<snipe@snipe.net>]
-     * @since [v3.0]
+     * @since  [v3.0]
      * @return \Illuminate\Database\Eloquent\Relations\Relation
      */
     public function assetlog()
@@ -220,7 +220,7 @@ class Component extends SnipeModel
      * Check how many items within a component are checked out
      *
      * @author [A. Gianotto] [<snipe@snipe.net>]
-     * @since [v5.0]
+     * @since  [v5.0]
      * @return int
      */
     public function numCheckedOut()
@@ -239,11 +239,12 @@ class Component extends SnipeModel
      *
      * This allows us to get the assets with assigned components without the company restriction
      */
-    public function uncontrainedAssets() {
+    public function uncontrainedAssets()
+    {
 
         return $this->belongsToMany(\App\Models\Asset::class, 'components_assets')
-                ->withPivot('id', 'assigned_qty', 'created_at', 'created_by', 'note')
-                ->withoutGlobalScope(new CompanyableScope);
+            ->withPivot('id', 'assigned_qty', 'created_at', 'created_by', 'note')
+            ->withoutGlobalScope(new CompanyableScope);
 
     }
 
@@ -252,7 +253,7 @@ class Component extends SnipeModel
      * Check how many items within a component are remaining
      *
      * @author [A. Gianotto] [<snipe@snipe.net>]
-     * @since [v3.0]
+     * @since  [v3.0]
      * @return int
      */
     public function numRemaining()
@@ -275,8 +276,8 @@ class Component extends SnipeModel
      * This simply checks that there is a value for quantity, and if there isn't, set it to 0.
      *
      * @author A. Gianotto <snipe@snipe.net>
-     * @since v6.3.4
-     * @param $value
+     * @since  v6.3.4
+     * @param  $value
      * @return void
      */
     public function setQtyAttribute($value)
@@ -294,8 +295,8 @@ class Component extends SnipeModel
     /**
      * Query builder scope to order on company
      *
-     * @param  \Illuminate\Database\Query\Builder  $query  Query builder instance
-     * @param  string                              $order       Order
+     * @param \Illuminate\Database\Query\Builder $query Query builder instance
+     * @param string                             $order Order
      *
      * @return \Illuminate\Database\Query\Builder          Modified query builder
      */
@@ -307,8 +308,8 @@ class Component extends SnipeModel
     /**
      * Query builder scope to order on company
      *
-     * @param  \Illuminate\Database\Query\Builder  $query  Query builder instance
-     * @param  string                              $order       Order
+     * @param \Illuminate\Database\Query\Builder $query Query builder instance
+     * @param string                             $order Order
      *
      * @return \Illuminate\Database\Query\Builder          Modified query builder
      */
@@ -320,8 +321,8 @@ class Component extends SnipeModel
     /**
      * Query builder scope to order on company
      *
-     * @param  \Illuminate\Database\Query\Builder  $query  Query builder instance
-     * @param  string                              $order       Order
+     * @param \Illuminate\Database\Query\Builder $query Query builder instance
+     * @param string                             $order Order
      *
      * @return \Illuminate\Database\Query\Builder          Modified query builder
      */
@@ -333,8 +334,8 @@ class Component extends SnipeModel
     /**
      * Query builder scope to order on supplier
      *
-     * @param  \Illuminate\Database\Query\Builder  $query  Query builder instance
-     * @param  text                              $order       Order
+     * @param \Illuminate\Database\Query\Builder $query Query builder instance
+     * @param text                               $order Order
      *
      * @return \Illuminate\Database\Query\Builder          Modified query builder
      */
@@ -346,8 +347,8 @@ class Component extends SnipeModel
     /**
      * Query builder scope to order on manufacturer
      *
-     * @param  \Illuminate\Database\Query\Builder  $query  Query builder instance
-     * @param  text                              $order       Order
+     * @param \Illuminate\Database\Query\Builder $query Query builder instance
+     * @param text                               $order Order
      *
      * @return \Illuminate\Database\Query\Builder          Modified query builder
      */

app/Models/Consumable.php

@@ -125,7 +125,7 @@ class Consumable extends SnipeModel
      * @todo Update this comment once it's been implemented
      *
      * @author [A. Gianotto] [<snipe@snipe.net>]
-     * @since [v3.0]
+     * @since  [v3.0]
      * @return \Illuminate\Database\Eloquent\Relations\Relation
      */
     public function setRequestableAttribute($value)
@@ -140,7 +140,7 @@ class Consumable extends SnipeModel
      * Establishes the consumable -> admin user relationship
      *
      * @author [A. Gianotto] [<snipe@snipe.net>]
-     * @since [v3.0]
+     * @since  [v3.0]
      * @return \Illuminate\Database\Eloquent\Relations\Relation
      */
     public function adminuser()
@@ -152,7 +152,7 @@ class Consumable extends SnipeModel
      * Establishes the component -> assignments relationship
      *
      * @author [A. Gianotto] [<snipe@snipe.net>]
-     * @since [v3.0]
+     * @since  [v3.0]
      * @return \Illuminate\Database\Eloquent\Relations\Relation
      */
     public function consumableAssignments()
@@ -164,7 +164,7 @@ class Consumable extends SnipeModel
      * Establishes the component -> company relationship
      *
      * @author [A. Gianotto] [<snipe@snipe.net>]
-     * @since [v3.0]
+     * @since  [v3.0]
      * @return \Illuminate\Database\Eloquent\Relations\Relation
      */
     public function company()
@@ -176,7 +176,7 @@ class Consumable extends SnipeModel
      * Establishes the component -> manufacturer relationship
      *
      * @author [A. Gianotto] [<snipe@snipe.net>]
-     * @since [v3.0]
+     * @since  [v3.0]
      * @return \Illuminate\Database\Eloquent\Relations\Relation
      */
     public function manufacturer()
@@ -188,7 +188,7 @@ class Consumable extends SnipeModel
      * Establishes the component -> location relationship
      *
      * @author [A. Gianotto] [<snipe@snipe.net>]
-     * @since [v3.0]
+     * @since  [v3.0]
      * @return \Illuminate\Database\Eloquent\Relations\Relation
      */
     public function location()
@@ -200,7 +200,7 @@ class Consumable extends SnipeModel
      * Establishes the component -> category relationship
      *
      * @author [A. Gianotto] [<snipe@snipe.net>]
-     * @since [v3.0]
+     * @since  [v3.0]
      * @return \Illuminate\Database\Eloquent\Relations\Relation
      */
     public function category()
@@ -213,7 +213,7 @@ class Consumable extends SnipeModel
      * Establishes the component -> action logs relationship
      *
      * @author [A. Gianotto] [<snipe@snipe.net>]
-     * @since [v3.0]
+     * @since  [v3.0]
      * @return \Illuminate\Database\Eloquent\Relations\Relation
      */
     public function assetlog()
@@ -225,7 +225,7 @@ class Consumable extends SnipeModel
      * Gets the full image url for the consumable
      *
      * @author [A. Gianotto] [<snipe@snipe.net>]
-     * @since [v3.0]
+     * @since  [v3.0]
      * @return string | false
      */
     public function getImageUrl()
@@ -241,7 +241,7 @@ class Consumable extends SnipeModel
      * Establishes the component -> users relationship
      *
      * @author [A. Gianotto] [<snipe@snipe.net>]
-     * @since [v3.0]
+     * @since  [v3.0]
      */
     public function users() : Relation
     {
@@ -252,7 +252,7 @@ class Consumable extends SnipeModel
      * Establishes the item -> supplier relationship
      *
      * @author [A. Gianotto] [<snipe@snipe.net>]
-     * @since [v6.1.1]
+     * @since  [v6.1.1]
      * @return \Illuminate\Database\Eloquent\Relations\Relation
      */
     public function supplier()
@@ -266,7 +266,7 @@ class Consumable extends SnipeModel
      * asset model category
      *
      * @author [A. Gianotto] [<snipe@snipe.net>]
-     * @since [v4.0]
+     * @since  [v4.0]
      * @return bool
      */
     public function checkin_email()
@@ -278,7 +278,7 @@ class Consumable extends SnipeModel
      * Determine whether this asset requires acceptance by the assigned user
      *
      * @author [A. Gianotto] [<snipe@snipe.net>]
-     * @since [v4.0]
+     * @since  [v4.0]
      * @return bool
      */
     public function requireAcceptance()
@@ -291,7 +291,7 @@ class Consumable extends SnipeModel
      * checks for a settings level EULA
      *
      * @author [A. Gianotto] [<snipe@snipe.net>]
-     * @since [v4.0]
+     * @since  [v4.0]
      * @return string | false
      */
     public function getEula()
@@ -309,7 +309,7 @@ class Consumable extends SnipeModel
      * Check how many items within a consumable are checked out
      *
      * @author [A. Gianotto] [<snipe@snipe.net>]
-     * @since [v5.0]
+     * @since  [v5.0]
      * @return int
      */
     public function numCheckedOut()
@@ -321,7 +321,7 @@ class Consumable extends SnipeModel
      * Checks the number of available consumables
      *
      * @author [A. Gianotto] [<snipe@snipe.net>]
-     * @since [v4.0]
+     * @since  [v4.0]
      * @return int
      */
     public function numRemaining()
@@ -347,8 +347,8 @@ class Consumable extends SnipeModel
      * This simply checks that there is a value for quantity, and if there isn't, set it to 0.
      *
      * @author A. Gianotto <snipe@snipe.net>
-     * @since v6.3.4
-     * @param $value
+     * @since  v6.3.4
+     * @param  $value
      * @return void
      */
     public function setQtyAttribute($value)
@@ -365,8 +365,8 @@ class Consumable extends SnipeModel
     /**
      * Query builder scope to order on company
      *
-     * @param  \Illuminate\Database\Query\Builder  $query  Query builder instance
-     * @param  string                              $order       Order
+     * @param \Illuminate\Database\Query\Builder $query Query builder instance
+     * @param string                             $order Order
      *
      * @return \Illuminate\Database\Query\Builder          Modified query builder
      */
@@ -378,8 +378,8 @@ class Consumable extends SnipeModel
     /**
      * Query builder scope to order on location
      *
-     * @param  \Illuminate\Database\Query\Builder  $query  Query builder instance
-     * @param  text                              $order       Order
+     * @param \Illuminate\Database\Query\Builder $query Query builder instance
+     * @param text                               $order Order
      *
      * @return \Illuminate\Database\Query\Builder          Modified query builder
      */
@@ -391,8 +391,8 @@ class Consumable extends SnipeModel
     /**
      * Query builder scope to order on manufacturer
      *
-     * @param  \Illuminate\Database\Query\Builder  $query  Query builder instance
-     * @param  string   $order       Order
+     * @param \Illuminate\Database\Query\Builder $query Query builder instance
+     * @param string                             $order Order
      *
      * @return \Illuminate\Database\Query\Builder          Modified query builder
      */
@@ -404,8 +404,8 @@ class Consumable extends SnipeModel
     /**
      * Query builder scope to order on company
      *
-     * @param  \Illuminate\Database\Query\Builder  $query  Query builder instance
-     * @param  string                              $order       Order
+     * @param \Illuminate\Database\Query\Builder $query Query builder instance
+     * @param string                             $order Order
      *
      * @return \Illuminate\Database\Query\Builder          Modified query builder
      */
@@ -417,8 +417,8 @@ class Consumable extends SnipeModel
     /**
      * Query builder scope to order on remaining
      *
-     * @param  \Illuminate\Database\Query\Builder  $query  Query builder instance
-     * @param  string                              $order       Order
+     * @param \Illuminate\Database\Query\Builder $query Query builder instance
+     * @param string                             $order Order
      *
      * @return \Illuminate\Database\Query\Builder          Modified query builder
      */
@@ -431,8 +431,8 @@ class Consumable extends SnipeModel
     /**
      * Query builder scope to order on supplier
      *
-     * @param  \Illuminate\Database\Query\Builder  $query  Query builder instance
-     * @param  text                              $order       Order
+     * @param \Illuminate\Database\Query\Builder $query Query builder instance
+     * @param text                               $order Order
      *
      * @return \Illuminate\Database\Query\Builder          Modified query builder
      */

app/Models/CustomField.php

@@ -92,7 +92,7 @@ class CustomField extends Model
      * table instead of the assets table.
      *
      * @author [Brady Wetherington] [<uberbrady@gmail.com>]
-     * @since [v3.0]
+     * @since  [v3.0]
      */
     public static $table_name = 'assets';
 
@@ -103,7 +103,7 @@ class CustomField extends Model
      * do with previously existing values. - @snipe
      *
      * @author [A. Gianotto] [<snipe@snipe.net>]
-     * @since [v3.4]
+     * @since  [v3.4]
      * @return string
      */
     public static function name_to_db_name($name)
@@ -120,66 +120,78 @@ class CustomField extends Model
      * to do it in the controllers.
      *
      * @author [A. Gianotto] [<snipe@snipe.net>]
-     * @since [v3.4]
+     * @since  [v3.4]
      * @return bool
      */
     public static function boot()
     {
         parent::boot();
-        self::created(function ($custom_field) {
+        self::created(
+            function ($custom_field) {
 
-            // Column already exists on the assets table - nothing to do here.
-            // This *shouldn't* happen in the wild.
-            if (Schema::hasColumn(self::$table_name, $custom_field->db_column)) {
-                return false;
+                // Column already exists on the assets table - nothing to do here.
+                // This *shouldn't* happen in the wild.
+                if (Schema::hasColumn(self::$table_name, $custom_field->db_column)) {
+                    return false;
+                }
+
+                // Update the column name in the assets table
+                Schema::table(
+                    self::$table_name, function ($table) use ($custom_field) {
+                        $table->text($custom_field->convertUnicodeDbSlug())->nullable();
+                    }
+                );
+
+                // Update the db_column property in the custom fields table
+                $custom_field->db_column = $custom_field->convertUnicodeDbSlug();
+                $custom_field->save();
             }
+        );
 
-            // Update the column name in the assets table
-            Schema::table(self::$table_name, function ($table) use ($custom_field) {
-                $table->text($custom_field->convertUnicodeDbSlug())->nullable();
-            });
+        self::updating(
+            function ($custom_field) {
 
-            // Update the db_column property in the custom fields table
-            $custom_field->db_column = $custom_field->convertUnicodeDbSlug();
-            $custom_field->save();
-        });
+                // Column already exists on the assets table - nothing to do here.
+                if ($custom_field->isDirty('name')) {
+                    if (Schema::hasColumn(self::$table_name, $custom_field->convertUnicodeDbSlug())) {
+                        return true;
+                    }
 
-        self::updating(function ($custom_field) {
+                    // Rename the field if the name has changed
+                    Schema::table(
+                        self::$table_name, function ($table) use ($custom_field) {
+                            $table->renameColumn($custom_field->convertUnicodeDbSlug($custom_field->getOriginal('name')), $custom_field->convertUnicodeDbSlug());
+                        }
+                    );
+
+                    // Save the updated column name to the custom fields table
+                    $custom_field->db_column = $custom_field->convertUnicodeDbSlug();
+                    $custom_field->save();
 
-            // Column already exists on the assets table - nothing to do here.
-            if ($custom_field->isDirty('name')) {
-                if (Schema::hasColumn(self::$table_name, $custom_field->convertUnicodeDbSlug())) {
                     return true;
                 }
 
-                // Rename the field if the name has changed
-                Schema::table(self::$table_name, function ($table) use ($custom_field) {
-                    $table->renameColumn($custom_field->convertUnicodeDbSlug($custom_field->getOriginal('name')), $custom_field->convertUnicodeDbSlug());
-                });
-
-                // Save the updated column name to the custom fields table
-                $custom_field->db_column = $custom_field->convertUnicodeDbSlug();
-                $custom_field->save();
-
                 return true;
             }
-
-            return true;
-        });
+        );
 
         // Drop the assets column if we've deleted it from custom fields
-        self::deleting(function ($custom_field) {
-            return Schema::table(self::$table_name, function ($table) use ($custom_field) {
-                $table->dropColumn($custom_field->db_column);
-            });
-        });
+        self::deleting(
+            function ($custom_field) {
+                return Schema::table(
+                    self::$table_name, function ($table) use ($custom_field) {
+                        $table->dropColumn($custom_field->db_column);
+                    }
+                );
+            }
+        );
     }
 
     /**
      * Establishes the customfield -> fieldset relationship
      *
      * @author [A. Gianotto] [<snipe@snipe.net>]
-     * @since [v3.0]
+     * @since  [v3.0]
      * @return \Illuminate\Database\Eloquent\Relations\Relation
      */
     public function fieldset()
@@ -214,26 +226,26 @@ class CustomField extends Model
     public function displayFieldInCurrentForm($form_type = null)
     {
         switch ($form_type) {
-            case 'audit':
-                return $this->displayFieldInAuditForm();
-            case 'checkin':
-                return $this->displayFieldInCheckinForm();
-            case 'checkout':
-                return $this->displayFieldInCheckoutForm();
+        case 'audit':
+            return $this->displayFieldInAuditForm();
+        case 'checkin':
+            return $this->displayFieldInCheckinForm();
+        case 'checkout':
+            return $this->displayFieldInCheckoutForm();
         }
     }
 
 
     public function assetModels()
     {
-       return $this->fieldset()->with('models')->get()->pluck('models')->flatten()->unique('id'); 
+        return $this->fieldset()->with('models')->get()->pluck('models')->flatten()->unique('id'); 
     }
 
     /**
      * Establishes the customfield -> admin user relationship
      *
      * @author [A. Gianotto] [<snipe@snipe.net>]
-     * @since [v3.0]
+     * @since  [v3.0]
      * @return \Illuminate\Database\Eloquent\Relations\Relation
      */
     public function user()
@@ -245,7 +257,7 @@ class CustomField extends Model
      * Establishes the customfield -> default values relationship
      *
      * @author Hannah Tinkler
-     * @since [v3.0]
+     * @since  [v3.0]
      * @return \Illuminate\Database\Eloquent\Relations\Relation
      */
     public function defaultValues()
@@ -262,19 +274,23 @@ class CustomField extends Model
      */
     public function defaultValue($modelId)
     {
-        return $this->defaultValues->filter(function ($item) use ($modelId) {
-            return $item->pivot->asset_model_id == $modelId;
-        })->map(function ($item) {
-            return $item->pivot->default_value;
-        })->first();
+        return $this->defaultValues->filter(
+            function ($item) use ($modelId) {
+                return $item->pivot->asset_model_id == $modelId;
+            }
+        )->map(
+            function ($item) {
+                return $item->pivot->default_value;
+            }
+        )->first();
     }
 
     /**
      * Checks the format of the attribute
      *
      * @author [A. Gianotto] [<snipe@snipe.net>]
-     * @param $value string
-     * @since [v3.0]
+     * @param  $value string
+     * @since  [v3.0]
      * @return bool
      */
     public function check_format($value)
@@ -286,7 +302,7 @@ class CustomField extends Model
      * Gets the DB column name.
      *
      * @author [A. Gianotto] [<snipe@snipe.net>]
-     * @since [v3.0]
+     * @since  [v3.0]
      * @return string
      */
     public function db_column_name()
@@ -302,7 +318,7 @@ class CustomField extends Model
      * user-friendly text in the dropdowns, and in the custom fields display.
      *
      * @author [A. Gianotto] [<snipe@snipe.net>]
-     * @since [v3.4]
+     * @since  [v3.4]
      * @return string
      */
     public function getFormatAttribute($value)
@@ -320,7 +336,7 @@ class CustomField extends Model
      * Format a value string as an array for select boxes and checkboxes.
      *
      * @author [A. Gianotto] [<snipe@snipe.net>]
-     * @since [v3.4]
+     * @since  [v3.4]
      * @return array
      */
     public function setFormatAttribute($value)
@@ -336,7 +352,7 @@ class CustomField extends Model
      * Format a value string as an array for select boxes and checkboxes.
      *
      * @author [A. Gianotto] [<snipe@snipe.net>]
-     * @since [v3.4]
+     * @since  [v3.4]
      * @return array
      */
     public function formatFieldValuesAsArray()
@@ -366,7 +382,7 @@ class CustomField extends Model
      * Check whether the field is encrypted
      *
      * @author [A. Gianotto] [<snipe@snipe.net>]
-     * @since [v3.4]
+     * @since  [v3.4]
      * @return bool
      */
     public function isFieldDecryptable($string)
@@ -383,7 +399,7 @@ class CustomField extends Model
      * won't break the database.
      *
      * @author [A. Gianotto] [<snipe@snipe.net>]
-     * @since [v3.4]
+     * @since  [v3.4]
      * @return string
      */
     public function convertUnicodeDbSlug($original = null)
@@ -392,7 +408,7 @@ class CustomField extends Model
         $id = $this->id ? $this->id : 'xx';
 
         if (! function_exists('transliterator_transliterate')) {
-            $long_slug = '_snipeit_'.str_slug(mb_convert_encoding(trim($name),"UTF-8"), '_');
+            $long_slug = '_snipeit_'.str_slug(mb_convert_encoding(trim($name), "UTF-8"), '_');
         } else {
             $long_slug = '_snipeit_'.Utf8Slugger::slugify($name, '_');
         }
@@ -402,9 +418,10 @@ class CustomField extends Model
 
     /**
      * Get validation rules for custom fields to use with Validator
+     *
      * @author [V. Cordes] [<volker@fdatek.de>]
-     * @param int $id
-     * @since [v4.1.10]
+     * @param  int $id
+     * @since  [v4.1.10]
      * @return array
      */
     public function validationRules($regex_format = null)
@@ -418,6 +435,7 @@ class CustomField extends Model
 
     /**
      * Check to see if there is a custom regex format type
+     *
      * @see https://github.com/grokability/snipe-it/issues/5896
      *
      * @author Wes Hulette <jwhulette@gmail.com>

app/Models/CustomFieldset.php

@@ -20,6 +20,7 @@ class CustomFieldset extends Model
 
     /**
      * Validation rules
+     *
      * @var array
      */
     public $rules = [
@@ -39,7 +40,7 @@ class CustomFieldset extends Model
      * Establishes the fieldset -> field relationship
      *
      * @author [Brady Wetherington] [<uberbrady@gmail.com>]
-     * @since [v3.0]
+     * @since  [v3.0]
      * @return \Illuminate\Database\Eloquent\Relations\Relation
      */
     public function fields()
@@ -51,7 +52,7 @@ class CustomFieldset extends Model
      * Establishes the fieldset -> models relationship
      *
      * @author [Brady Wetherington] [<uberbrady@gmail.com>]
-     * @since [v3.0]
+     * @since  [v3.0]
      * @return \Illuminate\Database\Eloquent\Relations\Relation
      */
     public function models()
@@ -63,7 +64,7 @@ class CustomFieldset extends Model
      * Establishes the fieldset -> admin user relationship
      *
      * @author [Brady Wetherington] [<uberbrady@gmail.com>]
-     * @since [v3.0]
+     * @since  [v3.0]
      * @return \Illuminate\Database\Eloquent\Relations\Relation
      */
     public function user()
@@ -76,14 +77,14 @@ class CustomFieldset extends Model
         if ($this->fields) {
 
             switch ($form_type) {
-                case 'audit':
-                    return $this->fields->where('display_audit', '1')->count() > 0;
-                case 'checkin':
-                    return $this->fields->where('display_checkin', '1')->count() > 0;
-                case 'checkout':
-                    return $this->fields->where('display_checkout', '1')->count() > 0;
-                default:
-                    return true;
+            case 'audit':
+                return $this->fields->where('display_audit', '1')->count() > 0;
+            case 'checkin':
+                return $this->fields->where('display_checkin', '1')->count() > 0;
+            case 'checkout':
+                return $this->fields->where('display_checkout', '1')->count() > 0;
+            default:
+                return true;
             }
         }
 
@@ -95,7 +96,7 @@ class CustomFieldset extends Model
      * custom field format
      *
      * @author [A. Gianotto] [<snipe@snipe.net>]
-     * @since [v3.0]
+     * @since  [v3.0]
      * @return array
      */
     public function validation_rules()
@@ -104,8 +105,9 @@ class CustomFieldset extends Model
         foreach ($this->fields as $field) {
             $rule = [];
 
-            if (($field->field_encrypted != '1') ||
-                  (($field->field_encrypted == '1') && (Gate::allows('admin')))) {
+            if (($field->field_encrypted != '1') 
+                || (($field->field_encrypted == '1') && (Gate::allows('admin')))
+            ) {
                     $rule[] = ($field->pivot->required == '1') ? 'required' : 'nullable';
             }
 

app/Models/Department.php

@@ -72,7 +72,7 @@ class Department extends SnipeModel
      * Establishes the department -> company relationship
      *
      * @author A. Gianotto <snipe@snipe.net>
-     * @since [v4.0]
+     * @since  [v4.0]
      * @return \Illuminate\Database\Eloquent\Relations\Relation
      */
     public function company()
@@ -84,7 +84,7 @@ class Department extends SnipeModel
      * Establishes the department -> users relationship
      *
      * @author A. Gianotto <snipe@snipe.net>
-     * @since [v4.0]
+     * @since  [v4.0]
      * @return \Illuminate\Database\Eloquent\Relations\Relation
      */
     public function users()
@@ -96,7 +96,7 @@ class Department extends SnipeModel
      * Establishes the department -> manager relationship
      *
      * @author A. Gianotto <snipe@snipe.net>
-     * @since [v4.0]
+     * @since  [v4.0]
      * @return \Illuminate\Database\Eloquent\Relations\Relation
      */
     public function manager()
@@ -108,7 +108,7 @@ class Department extends SnipeModel
      * Establishes the department -> location relationship
      *
      * @author A. Gianotto <snipe@snipe.net>
-     * @since [v4.0]
+     * @since  [v4.0]
      * @return \Illuminate\Database\Eloquent\Relations\Relation
      */
     public function location()
@@ -119,8 +119,8 @@ class Department extends SnipeModel
     /**
      * Query builder scope to order on location name
      *
-     * @param  \Illuminate\Database\Query\Builder  $query  Query builder instance
-     * @param  text                              $order       Order
+     * @param \Illuminate\Database\Query\Builder $query Query builder instance
+     * @param text                               $order Order
      *
      * @return \Illuminate\Database\Query\Builder          Modified query builder
      */
@@ -132,8 +132,8 @@ class Department extends SnipeModel
     /**
      * Query builder scope to order on manager name
      *
-     * @param  \Illuminate\Database\Query\Builder  $query  Query builder instance
-     * @param  text                              $order       Order
+     * @param \Illuminate\Database\Query\Builder $query Query builder instance
+     * @param text                               $order Order
      *
      * @return \Illuminate\Database\Query\Builder          Modified query builder
      */
@@ -145,8 +145,8 @@ class Department extends SnipeModel
     /**
      * Query builder scope to order on company
      *
-     * @param  \Illuminate\Database\Query\Builder  $query  Query builder instance
-     * @param  text                              $order       Order
+     * @param \Illuminate\Database\Query\Builder $query Query builder instance
+     * @param text                               $order Order
      *
      * @return \Illuminate\Database\Query\Builder          Modified query builder
      */

app/Models/Depreciable.php

@@ -48,15 +48,15 @@ class Depreciable extends SnipeModel
         $depreciation = 0;
         $setting = Setting::getSettings();
         switch ($setting->depreciation_method) {
-            case 'half_1':
+        case 'half_1':
             $depreciation = $this->getHalfYearDepreciatedValue(true);
             break;
 
-            case 'half_2':
+        case 'half_2':
             $depreciation = $this->getHalfYearDepreciatedValue(false);
             break;
 
-            default:
+        default:
             $depreciation = $this->getLinearDepreciatedValue();
         }
 
@@ -74,7 +74,7 @@ class Depreciable extends SnipeModel
             return null;
         }
 
-        if ($months_passed >= $this->get_depreciation()->months){
+        if ($months_passed >= $this->get_depreciation()->months) {
             //if there is a floor use it
             if($this->get_depreciation()->depreciation_min) {
 
@@ -93,14 +93,15 @@ class Depreciable extends SnipeModel
         return $current_value;
     }
 
-    public function getMonthlyDepreciation(){
+    public function getMonthlyDepreciation()
+    {
 
         return ($this->purchase_cost-$this->calculateDepreciation())/$this->get_depreciation()->months;
 
     }
 
     /**
-     * @param onlyHalfFirstYear Boolean always applied only second half of the first year
+     * @param  onlyHalfFirstYear Boolean always applied only second half of the first year
      * @return float|int
      */
     public function getHalfYearDepreciatedValue($onlyHalfFirstYear = false)
@@ -131,7 +132,7 @@ class Depreciable extends SnipeModel
     }
 
     /**
-     * @param \DateTime $date
+     * @param  \DateTime $date
      * @return int
      */
     protected function get_fiscal_year($date)
@@ -146,7 +147,7 @@ class Depreciable extends SnipeModel
     }
 
     /**
-     * @param \DateTime $date
+     * @param  \DateTime $date
      * @return bool
      */
     protected function is_first_half_of_year($date)

app/Models/Depreciation.php

@@ -16,7 +16,7 @@ class Depreciation extends SnipeModel
     // Declare the rules for the form validation
     protected $rules = [
         'name' => 'required|min:3|max:255|unique:depreciations,name',
-        'months' => 'required|max:3600|integer|gt:0',
+        'months' => 'required|max:3600|integer',
     ];
 
     /**
@@ -56,7 +56,7 @@ class Depreciation extends SnipeModel
      * Establishes the depreciation -> models relationship
      *
      * @author A. Gianotto <snipe@snipe.net>
-     * @since [v5.0]
+     * @since  [v5.0]
      * @return \Illuminate\Database\Eloquent\Relations\Relation
      */
     public function models()
@@ -68,7 +68,7 @@ class Depreciation extends SnipeModel
      * Establishes the depreciation -> licenses relationship
      *
      * @author A. Gianotto <snipe@snipe.net>
-     * @since [v5.0]
+     * @since  [v5.0]
      * @return \Illuminate\Database\Eloquent\Relations\Relation
      */
     public function licenses()
@@ -80,7 +80,7 @@ class Depreciation extends SnipeModel
      * Establishes the depreciation -> assets relationship
      *
      * @author A. Gianotto <snipe@snipe.net>
-     * @since [v5.0]
+     * @since  [v5.0]
      * @return \Illuminate\Database\Eloquent\Relations\Relation
      */
     public function assets()
@@ -92,7 +92,7 @@ class Depreciation extends SnipeModel
      * Get the user that created the depreciation
      *
      * @author A. Gianotto <snipe@snipe.net>
-     * @since [v7.0.13]
+     * @since  [v7.0.13]
      * @return \Illuminate\Database\Eloquent\Relations\Relation
      */
     public function adminuser()

app/Models/Group.php

@@ -51,7 +51,7 @@ class Group extends SnipeModel
      * Establishes the groups -> users relationship
      *
      * @author A. Gianotto <snipe@snipe.net>
-     * @since [v1.0]
+     * @since  [v1.0]
      * @return \Illuminate\Database\Eloquent\Relations\Relation
      */
     public function users()
@@ -63,7 +63,7 @@ class Group extends SnipeModel
      * Get the user that created the group
      *
      * @author A. Gianotto <snipe@snipe.net>
-     * @since [v6.3.0]
+     * @since  [v6.3.0]
      * @return \Illuminate\Database\Eloquent\Relations\Relation
      */
     public function adminuser()
@@ -75,7 +75,7 @@ class Group extends SnipeModel
      * Decode JSON permissions into array
      *
      * @author A. Gianotto <snipe@snipe.net>
-     * @since [v1.0]
+     * @since  [v1.0]
      * @return array | \stdClass
      */
     public function decodePermissions()

app/Models/Import.php

@@ -19,7 +19,7 @@ class Import extends Model
      * Establishes the license -> admin user relationship
      *
      * @author A. Gianotto <snipe@snipe.net>
-     * @since [v2.0]
+     * @since  [v2.0]
      * @return \Illuminate\Database\Eloquent\Relations\Relation
      */
     public function adminuser()

app/Models/Labels/DefaultLabel.php

@@ -38,7 +38,8 @@ class DefaultLabel extends RectangleSheet
     private int $rows;
 
 
-    public function __construct() {
+    public function __construct()
+    {
         $settings = Setting::getSettings();
 
         $this->textSize = Helper::convertUnit($settings->labels_fontsize, 'pt', 'in');
@@ -74,41 +75,116 @@ class DefaultLabel extends RectangleSheet
 
     }
 
-    public function getUnit()   { return 'in'; }
+    public function getUnit()
+    {
+        return 'in'; 
+    }
 
-    public function getPageWidth()  { return $this->pageWidth; }
-    public function getPageHeight() { return $this->pageHeight; }
+    public function getPageWidth()
+    {
+        return $this->pageWidth; 
+    }
+    public function getPageHeight()
+    {
+        return $this->pageHeight; 
+    }
 
-    public function getPageMarginTop()    { return $this->pageMarginTop; }
-    public function getPageMarginBottom() { return $this->pageMarginBottom; }
-    public function getPageMarginLeft()   { return $this->pageMarginLeft; }
-    public function getPageMarginRight()  { return $this->pageMarginRight; }
+    public function getPageMarginTop()
+    {
+        return $this->pageMarginTop; 
+    }
+    public function getPageMarginBottom()
+    {
+        return $this->pageMarginBottom; 
+    }
+    public function getPageMarginLeft()
+    {
+        return $this->pageMarginLeft; 
+    }
+    public function getPageMarginRight()
+    {
+        return $this->pageMarginRight; 
+    }
 
-    public function getColumns() { return $this->columns; }
-    public function getRows()    { return $this->rows; }
-    public function getLabelBorder() { return 0; }
+    public function getColumns()
+    {
+        return $this->columns; 
+    }
+    public function getRows()
+    {
+        return $this->rows; 
+    }
+    public function getLabelBorder()
+    {
+        return 0; 
+    }
 
-    public function getLabelWidth()  { return $this->labelWidth; }
-    public function getLabelHeight() { return $this->labelHeight; }
+    public function getLabelWidth()
+    {
+        return $this->labelWidth; 
+    }
+    public function getLabelHeight()
+    {
+        return $this->labelHeight; 
+    }
 
-    public function getLabelMarginTop()    { return 0; }
-    public function getLabelMarginBottom() { return 0; }
-    public function getLabelMarginLeft()   { return 0; }
-    public function getLabelMarginRight()  { return 0; }
+    public function getLabelMarginTop()
+    {
+        return 0; 
+    }
+    public function getLabelMarginBottom()
+    {
+        return 0; 
+    }
+    public function getLabelMarginLeft()
+    {
+        return 0; 
+    }
+    public function getLabelMarginRight()
+    {
+        return 0; 
+    }
 
-    public function getLabelColumnSpacing() { return $this->labelSpacingH; }
-    public function getLabelRowSpacing()    { return $this->labelSpacingV; }
+    public function getLabelColumnSpacing()
+    {
+        return $this->labelSpacingH; 
+    }
+    public function getLabelRowSpacing()
+    {
+        return $this->labelSpacingV; 
+    }
 
-    public function getSupportAssetTag()  { return false; }
-    public function getSupport1DBarcode() { return true; }
-    public function getSupport2DBarcode() { return true; }
-    public function getSupportFields()    { return 4; }
-    public function getSupportTitle()     { return true; }
-    public function getSupportLogo()      { return true; }
+    public function getSupportAssetTag()
+    {
+        return false; 
+    }
+    public function getSupport1DBarcode()
+    {
+        return true; 
+    }
+    public function getSupport2DBarcode()
+    {
+        return true; 
+    }
+    public function getSupportFields()
+    {
+        return 4; 
+    }
+    public function getSupportTitle()
+    {
+        return true; 
+    }
+    public function getSupportLogo()
+    {
+        return true; 
+    }
 
-    public function preparePDF($pdf) {}
+    public function preparePDF($pdf)
+    {
+    }
 
-    public function write($pdf, $record) {
+    public function write($pdf, $record)
+    {
 
         $asset = $record->get('asset');
         $settings = Setting::getSettings();

app/Models/Labels/Field.php

@@ -5,21 +5,30 @@ namespace App\Models\Labels;
 use App\Models\Asset;
 use Illuminate\Support\Collection;
 
-class Field {
+class Field
+{
     protected Collection $options;
-    public function getOptions() { return $this->options; }
-    public function setOptions($options) {
+    public function getOptions()
+    {
+        return $this->options; 
+    }
+    public function setOptions($options)
+    {
         $tempCollect = collect($options);
         if (!$tempCollect->contains(fn($o) => !is_subclass_of($o, FieldOption::class))) {
             $this->options = $options;
         }
     }
     
-    public function toArray(Asset $asset) { return Field::makeArray($this, $asset); }
+    public function toArray(Asset $asset)
+    {
+        return Field::makeArray($this, $asset); 
+    }
 
     /* Statics */
 
-    public static function makeArray(Field $field, Asset $asset) {
+    public static function makeArray(Field $field, Asset $asset)
+    {
         return $field->getOptions()
             // filter out any FieldOptions that are accidentally null
             ->filter()
@@ -27,11 +36,13 @@ class Field {
             ->filter(fn($result) => $result['value'] != null);
     }
 
-    public static function makeString(Field $option) {
+    public static function makeString(Field $option)
+    {
         return implode('|', $option->getOptions());
     }
 
-    public static function fromString(string $theString) {
+    public static function fromString(string $theString)
+    {
         $field = new Field();
         $field->options = collect(explode('|', $theString))
             ->filter(fn($optionString) => !empty($optionString))

app/Models/Labels/FieldOption.php

@@ -5,14 +5,22 @@ namespace App\Models\Labels;
 use App\Models\Asset;
 use Illuminate\Support\Collection;
 
-class FieldOption {
+class FieldOption
+{
     protected string $label;
-    public function getLabel() { return $this->label; }
+    public function getLabel()
+    {
+        return $this->label; 
+    }
 
     protected string $dataSource;
-    public function getDataSource() { return $this->dataSource; }
+    public function getDataSource()
+    {
+        return $this->dataSource; 
+    }
 
-    public function getValue(Asset $asset) {
+    public function getValue(Asset $asset)
+    {
         $dataPath = collect(explode('.', $this->dataSource));
 
         // assignedTo directly on the asset is a special case where
@@ -33,18 +41,29 @@ class FieldOption {
             return $asset->purchase_date ? $asset->purchase_date->format('Y-m-d') : null;
         }
         
-        return $dataPath->reduce(function ($myValue, $path) {
-            try { return $myValue ? $myValue->{$path} : ${$myValue}; }
-            catch (\Exception $e) { return $myValue; }
-        }, $asset);
+        return $dataPath->reduce(
+            function ($myValue, $path) {
+                try { return $myValue ? $myValue->{$path} : ${$myValue}; 
+                }
+                catch (\Exception $e) { return $myValue; 
+                }
+            }, $asset
+        );
     }
 
-    public function toArray(Asset $asset=null) { return FieldOption::makeArray($this, $asset); }
-    public function toString() { return FieldOption::makeString($this); }
+    public function toArray(Asset $asset=null)
+    {
+        return FieldOption::makeArray($this, $asset); 
+    }
+    public function toString()
+    {
+        return FieldOption::makeString($this); 
+    }
 
     /* Statics */
 
-    public static function makeArray(FieldOption $option, Asset $asset=null) {
+    public static function makeArray(FieldOption $option, Asset $asset=null)
+    {
         return [
             'label' => $option->getLabel(),
             'dataSource' => $option->getDataSource(),
@@ -52,11 +71,13 @@ class FieldOption {
         ];
     }
 
-    public static function makeString(FieldOption $option) {
+    public static function makeString(FieldOption $option)
+    {
         return $option->getLabel() . '=' . $option->getDataSource();
     }
 
-    public static function fromString(string $theString) {
+    public static function fromString(string $theString)
+    {
         $parts = explode('=', $theString);
         if (count($parts) == 2) {
             $option = new FieldOption();

app/Models/Labels/Label.php

@@ -13,7 +13,7 @@ use Illuminate\Support\Facades\Log;
 /**
  * Model for Labels.
  *
- * @version    v1.0
+ * @version v1.0
  */
 abstract class Label
 {
@@ -32,7 +32,8 @@ abstract class Label
      *
      * @return int
      */
-    public function getRotation() {
+    public function getRotation()
+    {
         return 0;
     }
 
@@ -123,29 +124,32 @@ abstract class Label
     /**
      * Make changes to the PDF properties here. OPTIONAL.
      * 
-     * @param  TCPDF  $pdf  The TCPDF instance
+     * @param TCPDF $pdf The TCPDF instance
      */
     public abstract function preparePDF(TCPDF $pdf);
 
     /**
      * Write single data record as content here.
      * 
-     * @param  TCPDF       $pdf     The TCPDF instance
-     * @param  Collection  $record  A data record
+     * @param TCPDF      $pdf    The TCPDF instance
+     * @param Collection $record A data record
      */
     public abstract function write(TCPDF $pdf, Collection $record);
 
     /**
      * Handle the data here. Override for multiple-per-page handling
      * 
-     * @param  TCPDF       $pdf   The TCPDF instance
-     * @param  Collection  $data  The data
+     * @param TCPDF      $pdf  The TCPDF instance
+     * @param Collection $data The data
      */
-    public function writeAll(TCPDF $pdf, Collection $data) {
-        $data->each(function ($record, $index) use ($pdf) {
-            $pdf->AddPage();
-            $this->write($pdf, $record);
-        });
+    public function writeAll(TCPDF $pdf, Collection $data)
+    {
+        $data->each(
+            function ($record, $index) use ($pdf) {
+                $pdf->AddPage();
+                $this->write($pdf, $record);
+            }
+        );
     }
 
     /**
@@ -153,7 +157,8 @@ abstract class Label
      *
      * @return string
      */
-    public final function getName() {
+    public final function getName()
+    {
         $refClass = new \ReflectionClass(Label::class);
         return str_replace($refClass->getNamespaceName() . '\\', '', get_class($this));
     }
@@ -165,7 +170,8 @@ abstract class Label
      *
      * @return string
      */
-    public final function getOrientation() {
+    public final function getOrientation()
+    {
         return ($this->getWidth() >= $this->getHeight()) ? 'L' : 'P';
     }
 
@@ -174,7 +180,8 @@ abstract class Label
      *
      * @return object [ 'x1'=>0.00, 'y1'=>0.00, 'x2'=>0.00, 'y2'=>0.00, 'w'=>0.00, 'h'=>0.00 ]
      */
-    public final function getPrintableArea() {
+    public final function getPrintableArea()
+    {
         return (object)[
             'x1' => $this->getMarginLeft(),
             'y1' => $this->getMarginTop(),
@@ -188,21 +195,22 @@ abstract class Label
     /**
      * Write a text cell.
      * 
-     * @param  TCPDF   $pdf     The TCPDF instance
-     * @param  string  $text    The text to write. Supports 'some **bold** text'.
-     * @param  float   $x       X position of top-left
-     * @param  float   $y       Y position of top-left
-     * @param  string  $font    The font family
-     * @param  string  $style   The font style
-     * @param  int     $size    The font size in getUnit() units
-     * @param  string  $align   Align text in the box. 'L' left, 'R' right, 'C' center.
-     * @param  float   $width   Force text box width. NULL to auto-fit.
-     * @param  float   $height  Force text box height. NULL to auto-fit.
-     * @param  bool    $squash  Squash text if it's too big
-     * @param  int     $border  Thickness of border. Default = 0.
-     * @param  int     $spacing Letter spacing. Default = 0.
+     * @param TCPDF  $pdf     The TCPDF instance
+     * @param string $text    The text to write. Supports 'some **bold** text'.
+     * @param float  $x       X position of top-left
+     * @param float  $y       Y position of top-left
+     * @param string $font    The font family
+     * @param string $style   The font style
+     * @param int    $size    The font size in getUnit() units
+     * @param string $align   Align text in the box. 'L' left, 'R' right, 'C' center.
+     * @param float  $width   Force text box width. NULL to auto-fit.
+     * @param float  $height  Force text box height. NULL to auto-fit.
+     * @param bool   $squash  Squash text if it's too big
+     * @param int    $border  Thickness of border. Default = 0.
+     * @param int    $spacing Letter spacing. Default = 0.
      */
-    public final function writeText(TCPDF $pdf, $text, $x, $y, $font=null, $style=null, $size=null, $align='L', $width=null, $height=null, $squash=false, $border=0, $spacing=0) {
+    public final function writeText(TCPDF $pdf, $text, $x, $y, $font=null, $style=null, $size=null, $align='L', $width=null, $height=null, $squash=false, $border=0, $spacing=0)
+    {
         $prevFamily = $pdf->getFontFamily();
         $prevStyle = $pdf->getFontStyle();
         $prevSizePt = $pdf->getFontSizePt();
@@ -211,33 +219,42 @@ abstract class Label
 
         $fontFamily = !empty($font) ? $font : $prevFamily;
         $fontStyle = !empty($style) ? $style : $prevStyle;
-        if ($size) $fontSizePt = Helper::convertUnit($size, $this->getUnit(), 'pt', true);
-        else $fontSizePt = $prevSizePt;
+        if ($size) { $fontSizePt = Helper::convertUnit($size, $this->getUnit(), 'pt', true);
+        } else { $fontSizePt = $prevSizePt;
+        }
 
         $pdf->SetFontSpacing($spacing);
 
         $parts = collect(explode('**', $text))
-            ->map(function ($part, $index) use ($pdf, $fontFamily, $fontStyle, $fontSizePt) {
-                $modStyle = ($index % 2 == 1) ? 'B' : $fontStyle;
-                $pdf->setFont($fontFamily, $modStyle, $fontSizePt);
-                return [
+            ->map(
+                function ($part, $index) use ($pdf, $fontFamily, $fontStyle, $fontSizePt) {
+                    $modStyle = ($index % 2 == 1) ? 'B' : $fontStyle;
+                    $pdf->setFont($fontFamily, $modStyle, $fontSizePt);
+                    return [
                     'text' => $part,
                     'text_width' => $pdf->GetStringWidth($part),
                     'font_family' => $fontFamily,
                     'font_style' => $modStyle,
                     'font_size' => $fontSizePt,
-                ];
-            });
+                    ];
+                }
+            );
 
-        $textWidth = $parts->reduce(function ($carry, $part) { return $carry += $part['text_width']; });
+        $textWidth = $parts->reduce(
+            function ($carry, $part) {
+                return $carry += $part['text_width']; 
+            }
+        );
         $cellWidth = !empty($width) ? $width : $textWidth;
 
         if ($squash && ($textWidth > 0)) {
             $scaleFactor = min(1.0, $cellWidth / $textWidth);
-            $parts = $parts->map(function ($part, $index) use ($scaleFactor) {
-                $part['text_width'] = $part['text_width'] * $scaleFactor;
-                return $part;
-            });
+            $parts = $parts->map(
+                function ($part, $index) use ($scaleFactor) {
+                    $part['text_width'] = $part['text_width'] * $scaleFactor;
+                    return $part;
+                }
+            );
         }
         $cellHeight = !empty($height) ? $height : Helper::convertUnit($fontSizePt, 'pt', $this->getUnit());
 
@@ -249,18 +266,23 @@ abstract class Label
         }
 
         switch($align) {
-            case 'R': $startX = ($x + $cellWidth) - min($cellWidth, $textWidth); break;
-            case 'C': $startX = ($x + ($cellWidth / 2)) - (min($cellWidth, $textWidth) / 2); break;
-            case 'L':
-            default: $startX = $x; break;
+        case 'R': $startX = ($x + $cellWidth) - min($cellWidth, $textWidth); 
+            break;
+        case 'C': $startX = ($x + ($cellWidth / 2)) - (min($cellWidth, $textWidth) / 2); 
+            break;
+        case 'L':
+        default: $startX = $x; 
+            break;
         }
 
-        $parts->reduce(function ($currentX, $part) use ($pdf, $y, $cellHeight) {
-            $pdf->SetXY($currentX, $y);
-            $pdf->setFont($part['font_family'], $part['font_style'], $part['font_size']);
-            $pdf->Cell($part['text_width'], $cellHeight, $part['text'], 0, 0, '', false, '', 1, true);
-            return $currentX += $part['text_width'];
-        }, $startX);
+        $parts->reduce(
+            function ($currentX, $part) use ($pdf, $y, $cellHeight) {
+                $pdf->SetXY($currentX, $y);
+                $pdf->setFont($part['font_family'], $part['font_style'], $part['font_size']);
+                $pdf->Cell($part['text_width'], $cellHeight, $part['text'], 0, 0, '', false, '', 1, true);
+                return $currentX += $part['text_width'];
+            }, $startX
+        );
 
         $pdf->SetFont($prevFamily, $prevStyle, $prevSizePt);
         $pdf->SetFontSpacing(0);
@@ -269,27 +291,30 @@ abstract class Label
     /**
      * Write an image.
      * 
-     * @param  TCPDF   $pdf     The TCPDF instance
-     * @param  string  $image   The image to write
-     * @param  float   $x       X position of top-left
-     * @param  float   $y       Y position of top-left
-     * @param  float   $width   The container width
-     * @param  float   $height  The container height
-     * @param  string  $halign  Align text in the box. 'L' left, 'R' right, 'C' center. Default 'L'.
-     * @param  string  $valign  Align text in the box. 'T' top, 'B' bottom, 'C' center. Default 'T'.
-     * @param  int     $dpi     Pixels per inch
-     * @param  bool    $resize  Resize to fit container
-     * @param  bool    $stretch Stretch (vs Scale) to fit container
-     * @param  int     $border  Thickness of border. Default = 0.
+     * @param TCPDF  $pdf     The TCPDF instance
+     * @param string $image   The image to write
+     * @param float  $x       X position of top-left
+     * @param float  $y       Y position of top-left
+     * @param float  $width   The container width
+     * @param float  $height  The container height
+     * @param string $halign  Align text in the box. 'L' left, 'R' right, 'C' center. Default 'L'.
+     * @param string $valign  Align text in the box. 'T' top, 'B' bottom, 'C' center. Default 'T'.
+     * @param int    $dpi     Pixels per inch
+     * @param bool   $resize  Resize to fit container
+     * @param bool   $stretch Stretch (vs Scale) to fit container
+     * @param int    $border  Thickness of border. Default = 0.
      * 
      * @return array   Returns the final calculated size [w,h]
      */
-    public final function writeImage(TCPDF $pdf, $image, $x, $y, $width=null, $height=null, $halign='L', $valign='L', $dpi=300, $resize=false, $stretch=false, $border=0) {
+    public final function writeImage(TCPDF $pdf, $image, $x, $y, $width=null, $height=null, $halign='L', $valign='L', $dpi=300, $resize=false, $stretch=false, $border=0)
+    {
 
-        if (empty($image)) return [0,0];
+        if (empty($image)) { return [0,0];
+        }
         
         $imageInfo = getimagesize($image);
-        if (!$imageInfo) return [0,0]; // TODO: SVG or other
+        if (!$imageInfo) { return [0,0]; // TODO: SVG or other
+        }
         
         $imageWidthPx = $imageInfo[0];
         $imageHeightPx = $imageInfo[1];
@@ -342,18 +367,24 @@ abstract class Label
 
         // Horizontal Position
         switch ($halign) {
-            case 'R': $originX = ($x + $containerWidth) - $outputWidth; break;
-            case 'C': $originX = ($x + ($containerWidth / 2)) - ($outputWidth / 2); break;
-            case 'L':
-            default: $originX = $x; break;
+        case 'R': $originX = ($x + $containerWidth) - $outputWidth; 
+            break;
+        case 'C': $originX = ($x + ($containerWidth / 2)) - ($outputWidth / 2); 
+            break;
+        case 'L':
+        default: $originX = $x; 
+            break;
         }
         
         // Vertical Position
         switch ($valign) {
-            case 'B': $originY = ($y + $containerHeight) - $outputHeight; break;
-            case 'C': $originY = ($y + ($containerHeight / 2)) - ($outputHeight / 2); break;
-            case 'T':
-            default: $originY = $y; break;
+        case 'B': $originY = ($y + $containerHeight) - $outputHeight; 
+            break;
+        case 'C': $originY = ($y + ($containerHeight / 2)) - ($outputHeight / 2); 
+            break;
+        case 'T':
+        default: $originY = $y; 
+            break;
         }
 
         // Actual Image
@@ -373,16 +404,18 @@ abstract class Label
     /**
      * Write a 1D barcode.
      * 
-     * @param  TCPDF   $pdf     The TCPDF instance
-     * @param  string  $value   The barcode content
-     * @param  string  $type    The barcode type
-     * @param  float   $x       X position of top-left
-     * @param  float   $y       Y position of top-left
-     * @param  float   $width   The container width
-     * @param  float   $height  The container height
+     * @param TCPDF  $pdf    The TCPDF instance
+     * @param string $value  The barcode content
+     * @param string $type   The barcode type
+     * @param float  $x      X position of top-left
+     * @param float  $y      Y position of top-left
+     * @param float  $width  The container width
+     * @param float  $height The container height
      */
-    public final function write1DBarcode(TCPDF $pdf, $value, $type, $x, $y, $width, $height) {
-        if (empty($value)) return;
+    public final function write1DBarcode(TCPDF $pdf, $value, $type, $x, $y, $width, $height)
+    {
+        if (empty($value)) { return;
+        }
         try {
             $pdf->write1DBarcode($value, $type, $x, $y, $width, $height, null, ['stretch'=>true]);
         } catch (\Exception|TypeError $e) {
@@ -393,16 +426,18 @@ abstract class Label
     /**
      * Write a 2D barcode.
      * 
-     * @param  TCPDF   $pdf     The TCPDF instance
-     * @param  string  $value   The barcode content
-     * @param  string  $type    The barcode type
-     * @param  float   $x       X position of top-left
-     * @param  float   $y       Y position of top-left
-     * @param  float   $width   The container width
-     * @param  float   $height  The container height
+     * @param TCPDF  $pdf    The TCPDF instance
+     * @param string $value  The barcode content
+     * @param string $type   The barcode type
+     * @param float  $x      X position of top-left
+     * @param float  $y      Y position of top-left
+     * @param float  $width  The container width
+     * @param float  $height The container height
      */
-    public final function write2DBarcode(TCPDF $pdf, $value, $type, $x, $y, $width, $height) {
-        if (empty($value)) return;
+    public final function write2DBarcode(TCPDF $pdf, $value, $type, $x, $y, $width, $height)
+    {
+        if (empty($value)) { return;
+        }
         $pdf->write2DBarcode($value, $type, $x, $y, $width, $height, null, ['stretch'=>true]);
     }
 
@@ -411,127 +446,180 @@ abstract class Label
     /**
      * Checks the template is internally valid
      */
-    public final function validate() : void {
+    public final function validate() : void
+    {
         $this->validateUnits();
         $this->validateSize();
         $this->validateMargins();
         $this->validateSupport();
     }
 
-    private function validateUnits() : void {
+    private function validateUnits() : void
+    {
         $validUnits = [ 'pt', 'mm', 'cm', 'in' ];
         $unit = $this->getUnit();
         if (!in_array(strtolower($unit), $validUnits)) {
-            throw new \UnexpectedValueException(trans('admin/labels/message.invalid_return_value', [
-                'name' => 'getUnit()',
-                'expected' => '[ \''.implode('\', \'', $validUnits).'\' ]',
-                'actual' => '\''.$unit.'\''
-            ]));
+            throw new \UnexpectedValueException(
+                trans(
+                    'admin/labels/message.invalid_return_value', [
+                    'name' => 'getUnit()',
+                    'expected' => '[ \''.implode('\', \'', $validUnits).'\' ]',
+                    'actual' => '\''.$unit.'\''
+                    ]
+                )
+            );
         }
     }
 
-    private function validateSize() : void {
+    private function validateSize() : void
+    {
         $width = $this->getWidth();
         if (!is_numeric($width) || is_string($width)) {
-            throw new \UnexpectedValueException(trans('admin/labels/message.invalid_return_type', [
-                'name' => 'getWidth()',
-                'expected' => 'float',
-                'actual' => gettype($width)
-            ]));
+            throw new \UnexpectedValueException(
+                trans(
+                    'admin/labels/message.invalid_return_type', [
+                    'name' => 'getWidth()',
+                    'expected' => 'float',
+                    'actual' => gettype($width)
+                    ]
+                )
+            );
         }
         
         $height = $this->getHeight();
         if (!is_numeric($height) || is_string($height)) {
-            throw new \UnexpectedValueException(trans('admin/labels/message.invalid_return_type', [
-                'name' => 'getHeight()',
-                'expected' => 'float',
-                'actual' => gettype($height)
-            ]));
+            throw new \UnexpectedValueException(
+                trans(
+                    'admin/labels/message.invalid_return_type', [
+                    'name' => 'getHeight()',
+                    'expected' => 'float',
+                    'actual' => gettype($height)
+                    ]
+                )
+            );
         }
     }
 
-    private function validateMargins() : void {
+    private function validateMargins() : void
+    {
         $marginTop = $this->getMarginTop();
         if (!is_numeric($marginTop) || is_string($marginTop)) {
-            throw new \UnexpectedValueException(trans('admin/labels/message.invalid_return_type', [
-                'name' => 'getMarginTop()',
-                'expected' => 'float',
-                'actual' => gettype($marginTop)
-            ]));
+            throw new \UnexpectedValueException(
+                trans(
+                    'admin/labels/message.invalid_return_type', [
+                    'name' => 'getMarginTop()',
+                    'expected' => 'float',
+                    'actual' => gettype($marginTop)
+                    ]
+                )
+            );
         }
         
         $marginBottom = $this->getMarginBottom();
         if (!is_numeric($marginBottom) || is_string($marginBottom)) {
-            throw new \UnexpectedValueException(trans('admin/labels/message.invalid_return_type', [
-                'name' => 'getMarginBottom()',
-                'expected' => 'float',
-                'actual' => gettype($marginBottom)
-            ]));
+            throw new \UnexpectedValueException(
+                trans(
+                    'admin/labels/message.invalid_return_type', [
+                    'name' => 'getMarginBottom()',
+                    'expected' => 'float',
+                    'actual' => gettype($marginBottom)
+                    ]
+                )
+            );
         }
 
         $marginLeft = $this->getMarginLeft();
         if (!is_numeric($marginLeft) || is_string($marginLeft)) {
-            throw new \UnexpectedValueException(trans('admin/labels/message.invalid_return_type', [
-                'name' => 'getMarginLeft()',
-                'expected' => 'float',
-                'actual' => gettype($marginLeft)
-            ]));
+            throw new \UnexpectedValueException(
+                trans(
+                    'admin/labels/message.invalid_return_type', [
+                    'name' => 'getMarginLeft()',
+                    'expected' => 'float',
+                    'actual' => gettype($marginLeft)
+                    ]
+                )
+            );
         }
         
         $marginRight = $this->getMarginRight();
         if (!is_numeric($marginRight) || is_string($marginRight)) {
-            throw new \UnexpectedValueException(trans('admin/labels/message.invalid_return_type', [
-                'name' => 'getMarginRight()',
-                'expected' => 'float',
-                'actual' => gettype($marginRight)
-            ]));
+            throw new \UnexpectedValueException(
+                trans(
+                    'admin/labels/message.invalid_return_type', [
+                    'name' => 'getMarginRight()',
+                    'expected' => 'float',
+                    'actual' => gettype($marginRight)
+                    ]
+                )
+            );
         }
     }
 
-    private function validateSupport() : void {
+    private function validateSupport() : void
+    {
         $support1D = $this->getSupport1DBarcode();
         if (!is_bool($support1D)) {
-            throw new \UnexpectedValueException(trans('admin/labels/message.invalid_return_type', [
-                'name' => 'getSupport1DBarcode()',
-                'expected' => 'boolean',
-                'actual' => gettype($support1D)
-            ]));
+            throw new \UnexpectedValueException(
+                trans(
+                    'admin/labels/message.invalid_return_type', [
+                    'name' => 'getSupport1DBarcode()',
+                    'expected' => 'boolean',
+                    'actual' => gettype($support1D)
+                    ]
+                )
+            );
         }
 
         $support2D = $this->getSupport2DBarcode();
         if (!is_bool($support2D)) {
-            throw new \UnexpectedValueException(trans('admin/labels/message.invalid_return_type', [
-                'name' => 'getSupport2DBarcode()',
-                'expected' => 'boolean',
-                'actual' => gettype($support2D)
-            ]));
+            throw new \UnexpectedValueException(
+                trans(
+                    'admin/labels/message.invalid_return_type', [
+                    'name' => 'getSupport2DBarcode()',
+                    'expected' => 'boolean',
+                    'actual' => gettype($support2D)
+                    ]
+                )
+            );
         }
 
         $supportFields = $this->getSupportFields();
         if (!is_int($supportFields)) {
-            throw new \UnexpectedValueException(trans('admin/labels/message.invalid_return_type', [
-                'name' => 'getSupportFields()',
-                'expected' => 'integer',
-                'actual' => gettype($supportFields)
-            ]));
+            throw new \UnexpectedValueException(
+                trans(
+                    'admin/labels/message.invalid_return_type', [
+                    'name' => 'getSupportFields()',
+                    'expected' => 'integer',
+                    'actual' => gettype($supportFields)
+                    ]
+                )
+            );
         }
 
         $supportLogo = $this->getSupportLogo();
         if (!is_bool($supportLogo)) {
-            throw new \UnexpectedValueException(trans('admin/labels/message.invalid_return_type', [
-                'name' => 'getSupportLogo()',
-                'expected' => 'boolean',
-                'actual' => gettype($supportLogo)
-            ]));
+            throw new \UnexpectedValueException(
+                trans(
+                    'admin/labels/message.invalid_return_type', [
+                    'name' => 'getSupportLogo()',
+                    'expected' => 'boolean',
+                    'actual' => gettype($supportLogo)
+                    ]
+                )
+            );
         }
 
         $supportTitle = $this->getSupportTitle();
         if (!is_bool($supportTitle)) {
-            throw new \UnexpectedValueException(trans('admin/labels/message.invalid_return_type', [
-                'name' => 'getSupportTitle()',
-                'expected' => 'boolean',
-                'actual' => gettype($supportTitle)
-            ]));
+            throw new \UnexpectedValueException(
+                trans(
+                    'admin/labels/message.invalid_return_type', [
+                    'name' => 'getSupportTitle()',
+                    'expected' => 'boolean',
+                    'actual' => gettype($supportTitle)
+                    ]
+                )
+            );
         }
     }
 
@@ -539,23 +627,26 @@ abstract class Label
 
     /**
      * Public Static Functions
-    */
+     */
 
     /**
      * Find size of a page by its format.
      * 
-     * @param  string  $format       Format name (eg: 'A4', 'LETTER', etc.)
-     * @param  string  $orientation  'L' for Landscape, 'P' for Portrait ('L' default)
-     * @param  string  $unit         Unit of measure to return in ('mm' default)
+     * @param string $format      Format name (eg: 'A4', 'LETTER', etc.)
+     * @param string $orientation 'L' for Landscape, 'P' for Portrait ('L' default)
+     * @param string $unit        Unit of measure to return in ('mm' default)
      * 
      * @return object  (object)[ 'width' => (float)123.4, 'height' => (float)123.4 ]
      */
-    public static function fromFormat($format, $orientation='L', $unit='mm', $round=false) {
+    public static function fromFormat($format, $orientation='L', $unit='mm', $round=false)
+    {
         $size = collect(TCPDF_STATIC::getPageSizeFromFormat(strtoupper($format)))
             ->sort()
-            ->map(function ($value) use ($unit) {
-                return Helper::convertUnit($value, 'pt', $unit);
-            })
+            ->map(
+                function ($value) use ($unit) {
+                    return Helper::convertUnit($value, 'pt', $unit);
+                }
+            )
             ->toArray();
         $width = ($orientation == 'L') ? $size[1] : $size[0];
         $height = ($orientation == 'L') ? $size[0] : $size[1];
@@ -571,16 +662,19 @@ abstract class Label
      * Unlike most Models, these are defined by their existence as non-
      * abstract classes stored in Models\Labels.
      *
-     * @param  string|Arrayable|array|null  $path  Label path[s]
+     * @param  string|Arrayable|array|null $path Label path[s]
      * @return Collection|Label|null
      */
-    public static function find($name=null) {
+    public static function find($name=null)
+    {
         // Find many
         if (is_array($name) || $name instanceof Arrayable) {
             $labels = collect($name)
-                ->map(function ($thisname) {
-                    return static::find($thisname);
-                })
+                ->map(
+                    function ($thisname) {
+                        return static::find($thisname);
+                    }
+                )
                 ->whereNotNull();
             return ($labels->count() > 0) ? $labels : null;
         }
@@ -588,26 +682,36 @@ abstract class Label
         // Find one
         if ($name !== null) {
             return static::find()
-                ->sole(function ($label) use ($name) {
-                    return $label->getName() == $name;
-                });
+                ->sole(
+                    function ($label) use ($name) {
+                        return $label->getName() == $name;
+                    }
+                );
         }
 
         // Find all
         return collect(File::allFiles(__DIR__))
-            ->map(function ($file) {
-                preg_match_all('/\/*(.+?)(?:\/|\.)/', $file->getRelativePathName(), $matches);
-                return __NAMESPACE__ . '\\' . implode('\\', $matches[1]);
-            })
-            ->filter(function ($name) {
-                if (!class_exists($name)) return false;
-                $refClass = new \ReflectionClass($name);
-                if ($refClass->isAbstract()) return false;
-                return $refClass->isSubclassOf(Label::class);
-            })
-            ->map(function ($name) {
-                return new $name();
-            });
+            ->map(
+                function ($file) {
+                    preg_match_all('/\/*(.+?)(?:\/|\.)/', $file->getRelativePathName(), $matches);
+                    return __NAMESPACE__ . '\\' . implode('\\', $matches[1]);
+                }
+            )
+            ->filter(
+                function ($name) {
+                    if (!class_exists($name)) { return false;
+                    }
+                    $refClass = new \ReflectionClass($name);
+                    if ($refClass->isAbstract()) { return false;
+                    }
+                    return $refClass->isSubclassOf(Label::class);
+                }
+            )
+            ->map(
+                function ($name) {
+                    return new $name();
+                }
+            );
     }
 
     

app/Models/Labels/RectangleSheet.php

@@ -33,9 +33,13 @@ abstract class RectangleSheet extends Sheet
     public abstract function getLabelRowSpacing();
 
 
-    public function getLabelsPerPage()  { return $this->getColumns() * $this->getRows(); }
+    public function getLabelsPerPage()
+    {
+        return $this->getColumns() * $this->getRows(); 
+    }
 
-    public function getLabelPosition($index)  {
+    public function getLabelPosition($index)
+    {
         $printIndex = $index + $this->getLabelIndexOffset();
         $row = (int)($printIndex / $this->getColumns());
         $col = $printIndex - ($row * $this->getColumns());

app/Models/Labels/Sheet.php

@@ -6,12 +6,30 @@ abstract class Sheet extends Label
 {
     protected int $indexOffset = 0;
 
-    public function getWidth()        { return $this->getPageWidth(); }
-    public function getHeight()       { return $this->getPageHeight(); }
-    public function getMarginTop()    { return $this->getPageMarginTop(); }
-    public function getMarginBottom() { return $this->getPageMarginBottom(); }
-    public function getMarginLeft()   { return $this->getPageMarginLeft(); }
-    public function getMarginRight()  { return $this->getPageMarginRight(); }
+    public function getWidth()
+    {
+        return $this->getPageWidth(); 
+    }
+    public function getHeight()
+    {
+        return $this->getPageHeight(); 
+    }
+    public function getMarginTop()
+    {
+        return $this->getPageMarginTop(); 
+    }
+    public function getMarginBottom()
+    {
+        return $this->getPageMarginBottom(); 
+    }
+    public function getMarginLeft()
+    {
+        return $this->getPageMarginLeft(); 
+    }
+    public function getMarginRight()
+    {
+        return $this->getPageMarginRight(); 
+    }
 
     /**
      * Returns the page width in getUnit() units
@@ -107,7 +125,7 @@ abstract class Sheet extends Label
     /**
      * Returns label position based on its index
      * 
-     * @param  int  $index
+     * @param int $index
      * 
      * @return array  [x,y]
      */
@@ -123,10 +141,11 @@ abstract class Sheet extends Label
     /**
      * Handle the data here. Override for multiple-per-page handling
      * 
-     * @param  TCPDF       $pdf   The TCPDF instance
-     * @param  Collection  $data  The data
+     * @param TCPDF      $pdf  The TCPDF instance
+     * @param Collection $data The data
      */
-    public function writeAll($pdf, $data) {
+    public function writeAll($pdf, $data)
+    {
         $prevPageNumber = -1;
         
         foreach ($data->toArray() as $recordIndex => $record) {
@@ -170,7 +189,8 @@ abstract class Sheet extends Label
      *
      * @return string
      */
-    public final function getLabelOrientation() {
+    public final function getLabelOrientation()
+    {
         return ($this->getLabelWidth() >= $this->getLabelHeight()) ? 'L' : 'P';
     }
 
@@ -179,7 +199,8 @@ abstract class Sheet extends Label
      *
      * @return object [ 'x1'=>0.00, 'y1'=>0.00, 'x2'=>0.00, 'y2'=>0.00, 'w'=>0.00, 'h'=>0.00 ]
      */
-    public final function getLabelPrintableArea() {
+    public final function getLabelPrintableArea()
+    {
         return (object)[
             'x1' => $this->getLabelMarginLeft(),
             'y1' => $this->getLabelMarginTop(),
@@ -195,15 +216,20 @@ abstract class Sheet extends Label
      * 
      * @return int
      */
-    public function getLabelIndexOffset() { return $this->indexOffset; }
+    public function getLabelIndexOffset()
+    {
+        return $this->indexOffset; 
+    }
 
     /**
      * Sets label index offset (skip positions)
      * 
-     * @param  int  $offset
-     * 
+     * @param int $offset
      */
-    public function setLabelIndexOffset(int $offset) { $this->indexOffset = $offset; }
+    public function setLabelIndexOffset(int $offset)
+    {
+        $this->indexOffset = $offset; 
+    }
 }
 
 ?>

app/Models/Labels/Sheets/Avery/L7162.php

@@ -31,7 +31,8 @@ abstract class L7162 extends RectangleSheet
     private float $labelWidth;
     private float $labelHeight;
 
-    public function __construct() {
+    public function __construct()
+    {
         $paperSize = static::fromFormat(self::PAPER_FORMAT, self::PAPER_ORIENTATION, $this->getUnit(), 0);
         $this->pageWidth = $paperSize->width;
         $this->pageHeight = $paperSize->height;
@@ -48,24 +49,63 @@ abstract class L7162 extends RectangleSheet
         $this->labelHeight = Helper::convertUnit(self::LABEL_H, 'pt', $this->getUnit());
     }
 
-    public function getPageWidth()  { return $this->pageWidth; }
-    public function getPageHeight() { return $this->pageHeight; }
+    public function getPageWidth()
+    {
+        return $this->pageWidth; 
+    }
+    public function getPageHeight()
+    {
+        return $this->pageHeight; 
+    }
 
-    public function getPageMarginTop()    { return $this->pageMarginTop; }
-    public function getPageMarginBottom() { return $this->pageMarginTop; }
-    public function getPageMarginLeft()   { return $this->pageMarginLeft; }
-    public function getPageMarginRight()  { return $this->pageMarginLeft; }
+    public function getPageMarginTop()
+    {
+        return $this->pageMarginTop; 
+    }
+    public function getPageMarginBottom()
+    {
+        return $this->pageMarginTop; 
+    }
+    public function getPageMarginLeft()
+    {
+        return $this->pageMarginLeft; 
+    }
+    public function getPageMarginRight()
+    {
+        return $this->pageMarginLeft; 
+    }
 
-    public function getColumns() { return 2; }
-    public function getRows()    { return 8; }
+    public function getColumns()
+    {
+        return 2; 
+    }
+    public function getRows()
+    {
+        return 8; 
+    }
 
-    public function getLabelColumnSpacing() { return $this->columnSpacing; }
-    public function getLabelRowSpacing()    { return $this->rowSpacing; }
+    public function getLabelColumnSpacing()
+    {
+        return $this->columnSpacing; 
+    }
+    public function getLabelRowSpacing()
+    {
+        return $this->rowSpacing; 
+    }
 
-    public function getLabelWidth()  { return $this->labelWidth; }
-    public function getLabelHeight() { return $this->labelHeight; }
+    public function getLabelWidth()
+    {
+        return $this->labelWidth; 
+    }
+    public function getLabelHeight()
+    {
+        return $this->labelHeight; 
+    }
 
-    public function getLabelBorder() { return 0; }
+    public function getLabelBorder()
+    {
+        return 0; 
+    }
 }
 
 ?>

app/Models/Labels/Sheets/Avery/L7162_A.php

@@ -14,23 +14,59 @@ class L7162_A extends L7162
     private const FIELD_SIZE     =   4.60;
     private const FIELD_MARGIN   =   0.30;
 
-    public function getUnit() { return 'mm'; }
+    public function getUnit()
+    {
+        return 'mm'; 
+    }
 
-    public function getLabelMarginTop()    { return 1.0; }
-    public function getLabelMarginBottom() { return 1.0; }
-    public function getLabelMarginLeft()   { return 1.0; }
-    public function getLabelMarginRight()  { return 1.0; }
+    public function getLabelMarginTop()
+    {
+        return 1.0; 
+    }
+    public function getLabelMarginBottom()
+    {
+        return 1.0; 
+    }
+    public function getLabelMarginLeft()
+    {
+        return 1.0; 
+    }
+    public function getLabelMarginRight()
+    {
+        return 1.0; 
+    }
 
-    public function getSupportAssetTag()  { return true; }
-    public function getSupport1DBarcode() { return false; }
-    public function getSupport2DBarcode() { return true; }
-    public function getSupportFields()    { return 4; }
-    public function getSupportLogo()      { return false; }
-    public function getSupportTitle()     { return true; }
+    public function getSupportAssetTag()
+    {
+        return true; 
+    }
+    public function getSupport1DBarcode()
+    {
+        return false; 
+    }
+    public function getSupport2DBarcode()
+    {
+        return true; 
+    }
+    public function getSupportFields()
+    {
+        return 4; 
+    }
+    public function getSupportLogo()
+    {
+        return false; 
+    }
+    public function getSupportTitle()
+    {
+        return true; 
+    }
 
-    public function preparePDF($pdf) {}
+    public function preparePDF($pdf)
+    {
+    }
 
-    public function write($pdf, $record) {
+    public function write($pdf, $record)
+    {
         $pa = $this->getLabelPrintableArea();
 
         $usableWidth = $pa->w;

app/Models/Labels/Sheets/Avery/L7162_B.php

@@ -17,23 +17,59 @@ class L7162_B extends L7162
     private const FIELD_SIZE     =   4.20;
     private const FIELD_MARGIN   =   0.30;
 
-    public function getUnit() { return 'mm'; }
+    public function getUnit()
+    {
+        return 'mm'; 
+    }
 
-    public function getLabelMarginTop()    { return 1.0; }
-    public function getLabelMarginBottom() { return 0; }
-    public function getLabelMarginLeft()   { return 1.0; }
-    public function getLabelMarginRight()  { return 1.0; }
+    public function getLabelMarginTop()
+    {
+        return 1.0; 
+    }
+    public function getLabelMarginBottom()
+    {
+        return 0; 
+    }
+    public function getLabelMarginLeft()
+    {
+        return 1.0; 
+    }
+    public function getLabelMarginRight()
+    {
+        return 1.0; 
+    }
 
-    public function getSupportAssetTag()  { return true; }
-    public function getSupport1DBarcode() { return true; }
-    public function getSupport2DBarcode() { return false; }
-    public function getSupportFields()    { return 3; }
-    public function getSupportLogo()      { return true; }
-    public function getSupportTitle()     { return true; }
+    public function getSupportAssetTag()
+    {
+        return true; 
+    }
+    public function getSupport1DBarcode()
+    {
+        return true; 
+    }
+    public function getSupport2DBarcode()
+    {
+        return false; 
+    }
+    public function getSupportFields()
+    {
+        return 3; 
+    }
+    public function getSupportLogo()
+    {
+        return true; 
+    }
+    public function getSupportTitle()
+    {
+        return true; 
+    }
 
-    public function preparePDF($pdf) {}
+    public function preparePDF($pdf)
+    {
+    }
 
-    public function write($pdf, $record) {
+    public function write($pdf, $record)
+    {
         $pa = $this->getLabelPrintableArea();
 
         $usableWidth = $pa->w;

app/Models/Labels/Sheets/Avery/L7163.php

@@ -31,7 +31,8 @@ abstract class L7163 extends RectangleSheet
     private float $labelWidth;
     private float $labelHeight;
 
-    public function __construct() {
+    public function __construct()
+    {
         $paperSize = static::fromFormat(self::PAPER_FORMAT, self::PAPER_ORIENTATION, $this->getUnit(), 0);
         $this->pageWidth = $paperSize->width;
         $this->pageHeight = $paperSize->height;
@@ -48,24 +49,63 @@ abstract class L7163 extends RectangleSheet
         $this->labelHeight = Helper::convertUnit(self::LABEL_H, 'pt', $this->getUnit());
     }
 
-    public function getPageWidth()  { return $this->pageWidth; }
-    public function getPageHeight() { return $this->pageHeight; }
+    public function getPageWidth()
+    {
+        return $this->pageWidth; 
+    }
+    public function getPageHeight()
+    {
+        return $this->pageHeight; 
+    }
 
-    public function getPageMarginTop()    { return $this->pageMarginTop; }
-    public function getPageMarginBottom() { return $this->pageMarginTop; }
-    public function getPageMarginLeft()   { return $this->pageMarginLeft; }
-    public function getPageMarginRight()  { return $this->pageMarginLeft; }
+    public function getPageMarginTop()
+    {
+        return $this->pageMarginTop; 
+    }
+    public function getPageMarginBottom()
+    {
+        return $this->pageMarginTop; 
+    }
+    public function getPageMarginLeft()
+    {
+        return $this->pageMarginLeft; 
+    }
+    public function getPageMarginRight()
+    {
+        return $this->pageMarginLeft; 
+    }
 
-    public function getColumns() { return 2; }
-    public function getRows()    { return 7; }
+    public function getColumns()
+    {
+        return 2; 
+    }
+    public function getRows()
+    {
+        return 7; 
+    }
 
-    public function getLabelColumnSpacing() { return $this->columnSpacing; }
-    public function getLabelRowSpacing()    { return $this->rowSpacing; }
+    public function getLabelColumnSpacing()
+    {
+        return $this->columnSpacing; 
+    }
+    public function getLabelRowSpacing()
+    {
+        return $this->rowSpacing; 
+    }
 
-    public function getLabelWidth()  { return $this->labelWidth; }
-    public function getLabelHeight() { return $this->labelHeight; }
+    public function getLabelWidth()
+    {
+        return $this->labelWidth; 
+    }
+    public function getLabelHeight()
+    {
+        return $this->labelHeight; 
+    }
 
-    public function getLabelBorder() { return 0; }
+    public function getLabelBorder()
+    {
+        return 0; 
+    }
 }
 
 ?>

app/Models/Labels/Sheets/Avery/L7163_A.php

@@ -14,23 +14,59 @@ class L7163_A extends L7163
     private const FIELD_SIZE     =   4.80;
     private const FIELD_MARGIN   =   0.30;
 
-    public function getUnit() { return 'mm'; }
+    public function getUnit()
+    {
+        return 'mm'; 
+    }
 
-    public function getLabelMarginTop()    { return 1.0; }
-    public function getLabelMarginBottom() { return 1.0; }
-    public function getLabelMarginLeft()   { return 1.0; }
-    public function getLabelMarginRight()  { return 1.0; }
+    public function getLabelMarginTop()
+    {
+        return 1.0; 
+    }
+    public function getLabelMarginBottom()
+    {
+        return 1.0; 
+    }
+    public function getLabelMarginLeft()
+    {
+        return 1.0; 
+    }
+    public function getLabelMarginRight()
+    {
+        return 1.0; 
+    }
 
-    public function getSupportAssetTag()  { return true; }
-    public function getSupport1DBarcode() { return false; }
-    public function getSupport2DBarcode() { return true; }
-    public function getSupportFields()    { return 4; }
-    public function getSupportLogo()      { return false; }
-    public function getSupportTitle()     { return true; }
+    public function getSupportAssetTag()
+    {
+        return true; 
+    }
+    public function getSupport1DBarcode()
+    {
+        return false; 
+    }
+    public function getSupport2DBarcode()
+    {
+        return true; 
+    }
+    public function getSupportFields()
+    {
+        return 4; 
+    }
+    public function getSupportLogo()
+    {
+        return false; 
+    }
+    public function getSupportTitle()
+    {
+        return true; 
+    }
 
-    public function preparePDF($pdf) {}
+    public function preparePDF($pdf)
+    {
+    }
 
-    public function write($pdf, $record) {
+    public function write($pdf, $record)
+    {
         $pa = $this->getLabelPrintableArea();
 
         $usableWidth = $pa->w;

app/Models/Labels/Sheets/Avery/_3490.php

@@ -31,7 +31,8 @@ abstract class _3490 extends RectangleSheet
     private float $labelWidth;
     private float $labelHeight;
 
-    public function __construct() {
+    public function __construct()
+    {
         $paperSize = static::fromFormat(self::PAPER_FORMAT, self::PAPER_ORIENTATION, $this->getUnit(), 2);
         $this->pageWidth = $paperSize->width;
         $this->pageHeight = $paperSize->height;
@@ -48,24 +49,63 @@ abstract class _3490 extends RectangleSheet
         $this->labelHeight = Helper::convertUnit(self::LABEL_H, 'pt', $this->getUnit());
     }
 
-    public function getPageWidth()  { return $this->pageWidth; }
-    public function getPageHeight() { return $this->pageHeight; }
+    public function getPageWidth()
+    {
+        return $this->pageWidth; 
+    }
+    public function getPageHeight()
+    {
+        return $this->pageHeight; 
+    }
 
-    public function getPageMarginTop()    { return $this->pageMarginTop; }
-    public function getPageMarginBottom() { return $this->pageMarginTop; }
-    public function getPageMarginLeft()   { return $this->pageMarginLeft; }
-    public function getPageMarginRight()  { return $this->pageMarginLeft; }
+    public function getPageMarginTop()
+    {
+        return $this->pageMarginTop; 
+    }
+    public function getPageMarginBottom()
+    {
+        return $this->pageMarginTop; 
+    }
+    public function getPageMarginLeft()
+    {
+        return $this->pageMarginLeft; 
+    }
+    public function getPageMarginRight()
+    {
+        return $this->pageMarginLeft; 
+    }
 
-    public function getColumns() { return 3; }
-    public function getRows()    { return 10; }
+    public function getColumns()
+    {
+        return 3; 
+    }
+    public function getRows()
+    {
+        return 10; 
+    }
 
-    public function getLabelColumnSpacing() { return $this->columnSpacing; }
-    public function getLabelRowSpacing()    { return $this->rowSpacing; }
+    public function getLabelColumnSpacing()
+    {
+        return $this->columnSpacing; 
+    }
+    public function getLabelRowSpacing()
+    {
+        return $this->rowSpacing; 
+    }
 
-    public function getLabelWidth()  { return $this->labelWidth; }
-    public function getLabelHeight() { return $this->labelHeight; }
+    public function getLabelWidth()
+    {
+        return $this->labelWidth; 
+    }
+    public function getLabelHeight()
+    {
+        return $this->labelHeight; 
+    }
 
-    public function getLabelBorder() { return 0; }
+    public function getLabelBorder()
+    {
+        return 0; 
+    }
 }
 
 ?>

app/Models/Labels/Sheets/Avery/_3490_A.php

@@ -14,23 +14,59 @@ class _3490_A extends _3490
     private const FIELD_SIZE     =   0.150;
     private const FIELD_MARGIN   =   0.012;
 
-    public function getUnit() { return 'in'; }
+    public function getUnit()
+    {
+        return 'in'; 
+    }
 
-    public function getLabelMarginTop()    { return 0.06; }
-    public function getLabelMarginBottom() { return 0.06; }
-    public function getLabelMarginLeft()   { return 0.06; }
-    public function getLabelMarginRight()  { return 0.06; }
+    public function getLabelMarginTop()
+    {
+        return 0.06; 
+    }
+    public function getLabelMarginBottom()
+    {
+        return 0.06; 
+    }
+    public function getLabelMarginLeft()
+    {
+        return 0.06; 
+    }
+    public function getLabelMarginRight()
+    {
+        return 0.06; 
+    }
 
-    public function getSupportAssetTag()  { return false; }
-    public function getSupport1DBarcode() { return false; }
-    public function getSupport2DBarcode() { return true; }
-    public function getSupportFields()    { return 3; }
-    public function getSupportLogo()      { return false; }
-    public function getSupportTitle()     { return true; }
+    public function getSupportAssetTag()
+    {
+        return false; 
+    }
+    public function getSupport1DBarcode()
+    {
+        return false; 
+    }
+    public function getSupport2DBarcode()
+    {
+        return true; 
+    }
+    public function getSupportFields()
+    {
+        return 3; 
+    }
+    public function getSupportLogo()
+    {
+        return false; 
+    }
+    public function getSupportTitle()
+    {
+        return true; 
+    }
 
-    public function preparePDF($pdf) {}
+    public function preparePDF($pdf)
+    {
+    }
 
-    public function write($pdf, $record) {
+    public function write($pdf, $record)
+    {
         $pa = $this->getLabelPrintableArea();
 
         $currentX = $pa->x1;

app/Models/Labels/Sheets/Avery/_5267.php

@@ -31,7 +31,8 @@ abstract class _5267 extends RectangleSheet
     private float $labelWidth;
     private float $labelHeight;
 
-    public function __construct() {
+    public function __construct()
+    {
         $paperSize = static::fromFormat(self::PAPER_FORMAT, self::PAPER_ORIENTATION, $this->getUnit(), 2);
         $this->pageWidth = $paperSize->width;
         $this->pageHeight = $paperSize->height;
@@ -48,24 +49,63 @@ abstract class _5267 extends RectangleSheet
         $this->labelHeight = Helper::convertUnit(self::LABEL_H, 'pt', $this->getUnit());
     }
 
-    public function getPageWidth()  { return $this->pageWidth; }
-    public function getPageHeight() { return $this->pageHeight; }
+    public function getPageWidth()
+    {
+        return $this->pageWidth; 
+    }
+    public function getPageHeight()
+    {
+        return $this->pageHeight; 
+    }
 
-    public function getPageMarginTop()    { return $this->pageMarginTop; }
-    public function getPageMarginBottom() { return $this->pageMarginTop; }
-    public function getPageMarginLeft()   { return $this->pageMarginLeft; }
-    public function getPageMarginRight()  { return $this->pageMarginLeft; }
+    public function getPageMarginTop()
+    {
+        return $this->pageMarginTop; 
+    }
+    public function getPageMarginBottom()
+    {
+        return $this->pageMarginTop; 
+    }
+    public function getPageMarginLeft()
+    {
+        return $this->pageMarginLeft; 
+    }
+    public function getPageMarginRight()
+    {
+        return $this->pageMarginLeft; 
+    }
 
-    public function getColumns() { return 4; }
-    public function getRows()    { return 20; }
+    public function getColumns()
+    {
+        return 4; 
+    }
+    public function getRows()
+    {
+        return 20; 
+    }
 
-    public function getLabelColumnSpacing() { return $this->columnSpacing; }
-    public function getLabelRowSpacing()    { return $this->rowSpacing; }
+    public function getLabelColumnSpacing()
+    {
+        return $this->columnSpacing; 
+    }
+    public function getLabelRowSpacing()
+    {
+        return $this->rowSpacing; 
+    }
 
-    public function getLabelWidth()  { return $this->labelWidth; }
-    public function getLabelHeight() { return $this->labelHeight; }
+    public function getLabelWidth()
+    {
+        return $this->labelWidth; 
+    }
+    public function getLabelHeight()
+    {
+        return $this->labelHeight; 
+    }
 
-    public function getLabelBorder() { return 0; }
+    public function getLabelBorder()
+    {
+        return 0; 
+    }
 }
 
 ?>

app/Models/Labels/Sheets/Avery/_5267_A.php

@@ -12,23 +12,59 @@ class _5267_A extends _5267
     private const FIELD_SIZE     =   0.150;
     private const FIELD_MARGIN   =   0.012;
 
-    public function getUnit() { return 'in'; }
+    public function getUnit()
+    {
+        return 'in'; 
+    }
 
-    public function getLabelMarginTop()    { return 0.02; }
-    public function getLabelMarginBottom() { return 0.00; }
-    public function getLabelMarginLeft()   { return 0.04; }
-    public function getLabelMarginRight()  { return 0.04; }
+    public function getLabelMarginTop()
+    {
+        return 0.02; 
+    }
+    public function getLabelMarginBottom()
+    {
+        return 0.00; 
+    }
+    public function getLabelMarginLeft()
+    {
+        return 0.04; 
+    }
+    public function getLabelMarginRight()
+    {
+        return 0.04; 
+    }
 
-    public function getSupportAssetTag()  { return false; }
-    public function getSupport1DBarcode() { return true; }
-    public function getSupport2DBarcode() { return false; }
-    public function getSupportFields()    { return 1; }
-    public function getSupportLogo()      { return false; }
-    public function getSupportTitle()     { return true; }
+    public function getSupportAssetTag()
+    {
+        return false; 
+    }
+    public function getSupport1DBarcode()
+    {
+        return true; 
+    }
+    public function getSupport2DBarcode()
+    {
+        return false; 
+    }
+    public function getSupportFields()
+    {
+        return 1; 
+    }
+    public function getSupportLogo()
+    {
+        return false; 
+    }
+    public function getSupportTitle()
+    {
+        return true; 
+    }
 
-    public function preparePDF($pdf) {}
+    public function preparePDF($pdf)
+    {
+    }
 
-    public function write($pdf, $record) {
+    public function write($pdf, $record)
+    {
         $pa = $this->getLabelPrintableArea();
 
         if ($record->has('barcode1d')) {

app/Models/Labels/Sheets/Avery/_5520.php

@@ -31,7 +31,8 @@ abstract class _5520 extends RectangleSheet
     private float $labelWidth;
     private float $labelHeight;
 
-    public function __construct() {
+    public function __construct()
+    {
         $paperSize = static::fromFormat(self::PAPER_FORMAT, self::PAPER_ORIENTATION, $this->getUnit(), 2);
         $this->pageWidth = $paperSize->width;
         $this->pageHeight = $paperSize->height;
@@ -48,24 +49,63 @@ abstract class _5520 extends RectangleSheet
         $this->labelHeight = Helper::convertUnit(self::LABEL_H, 'pt', $this->getUnit());
     }
 
-    public function getPageWidth()  { return $this->pageWidth; }
-    public function getPageHeight() { return $this->pageHeight; }
+    public function getPageWidth()
+    {
+        return $this->pageWidth; 
+    }
+    public function getPageHeight()
+    {
+        return $this->pageHeight; 
+    }
 
-    public function getPageMarginTop()    { return $this->pageMarginTop; }
-    public function getPageMarginBottom() { return $this->pageMarginTop; }
-    public function getPageMarginLeft()   { return $this->pageMarginLeft; }
-    public function getPageMarginRight()  { return $this->pageMarginLeft; }
+    public function getPageMarginTop()
+    {
+        return $this->pageMarginTop; 
+    }
+    public function getPageMarginBottom()
+    {
+        return $this->pageMarginTop; 
+    }
+    public function getPageMarginLeft()
+    {
+        return $this->pageMarginLeft; 
+    }
+    public function getPageMarginRight()
+    {
+        return $this->pageMarginLeft; 
+    }
 
-    public function getColumns() { return 3; }
-    public function getRows()    { return 10; }
+    public function getColumns()
+    {
+        return 3; 
+    }
+    public function getRows()
+    {
+        return 10; 
+    }
 
-    public function getLabelColumnSpacing() { return $this->columnSpacing; }
-    public function getLabelRowSpacing()    { return $this->rowSpacing; }
+    public function getLabelColumnSpacing()
+    {
+        return $this->columnSpacing; 
+    }
+    public function getLabelRowSpacing()
+    {
+        return $this->rowSpacing; 
+    }
 
-    public function getLabelWidth()  { return $this->labelWidth; }
-    public function getLabelHeight() { return $this->labelHeight; }
+    public function getLabelWidth()
+    {
+        return $this->labelWidth; 
+    }
+    public function getLabelHeight()
+    {
+        return $this->labelHeight; 
+    }
 
-    public function getLabelBorder() { return 0; }
+    public function getLabelBorder()
+    {
+        return 0; 
+    }
 }
 
 ?>

app/Models/Labels/Sheets/Avery/_5520_A.php

@@ -14,23 +14,59 @@ class _5520_A extends _5520
     private const FIELD_SIZE     =   0.150;
     private const FIELD_MARGIN   =   0.012;
 
-    public function getUnit() { return 'in'; }
+    public function getUnit()
+    {
+        return 'in'; 
+    }
 
-    public function getLabelMarginTop()    { return 0.06; }
-    public function getLabelMarginBottom() { return 0.06; }
-    public function getLabelMarginLeft()   { return 0.06; }
-    public function getLabelMarginRight()  { return 0.06; }
+    public function getLabelMarginTop()
+    {
+        return 0.06; 
+    }
+    public function getLabelMarginBottom()
+    {
+        return 0.06; 
+    }
+    public function getLabelMarginLeft()
+    {
+        return 0.06; 
+    }
+    public function getLabelMarginRight()
+    {
+        return 0.06; 
+    }
 
-    public function getSupportAssetTag()  { return false; }
-    public function getSupport1DBarcode() { return false; }
-    public function getSupport2DBarcode() { return true; }
-    public function getSupportFields()    { return 3; }
-    public function getSupportLogo()      { return false; }
-    public function getSupportTitle()     { return true; }
+    public function getSupportAssetTag()
+    {
+        return false; 
+    }
+    public function getSupport1DBarcode()
+    {
+        return false; 
+    }
+    public function getSupport2DBarcode()
+    {
+        return true; 
+    }
+    public function getSupportFields()
+    {
+        return 3; 
+    }
+    public function getSupportLogo()
+    {
+        return false; 
+    }
+    public function getSupportTitle()
+    {
+        return true; 
+    }
 
-    public function preparePDF($pdf) {}
+    public function preparePDF($pdf)
+    {
+    }
 
-    public function write($pdf, $record) {
+    public function write($pdf, $record)
+    {
         $pa = $this->getLabelPrintableArea();
 
         $currentX = $pa->x1;

app/Models/Labels/Sheets/Avery/_5520_B.php

@@ -15,23 +15,59 @@ class _5520_B extends _5520
     private const FIELD_SIZE     =   0.150;
     private const FIELD_MARGIN   =   0.012;
 
-    public function getUnit() { return 'in'; }
+    public function getUnit()
+    {
+        return 'in'; 
+    }
 
-    public function getLabelMarginTop()    { return 0.06; }
-    public function getLabelMarginBottom() { return 0.06; }
-    public function getLabelMarginLeft()   { return 0.06; }
-    public function getLabelMarginRight()  { return 0.06; }
+    public function getLabelMarginTop()
+    {
+        return 0.06; 
+    }
+    public function getLabelMarginBottom()
+    {
+        return 0.06; 
+    }
+    public function getLabelMarginLeft()
+    {
+        return 0.06; 
+    }
+    public function getLabelMarginRight()
+    {
+        return 0.06; 
+    }
 
-    public function getSupportAssetTag()  { return false; }
-    public function getSupport1DBarcode() { return true; }
-    public function getSupport2DBarcode() { return false; }
-    public function getSupportFields()    { return 2; }
-    public function getSupportLogo()      { return false; }
-    public function getSupportTitle()     { return true; }
+    public function getSupportAssetTag()
+    {
+        return false; 
+    }
+    public function getSupport1DBarcode()
+    {
+        return true; 
+    }
+    public function getSupport2DBarcode()
+    {
+        return false; 
+    }
+    public function getSupportFields()
+    {
+        return 2; 
+    }
+    public function getSupportLogo()
+    {
+        return false; 
+    }
+    public function getSupportTitle()
+    {
+        return true; 
+    }
 
-    public function preparePDF($pdf) {}
+    public function preparePDF($pdf)
+    {
+    }
 
-    public function write($pdf, $record) {
+    public function write($pdf, $record)
+    {
         $pa = $this->getLabelPrintableArea();
 
         $currentX = $pa->x1;

app/Models/Labels/Tapes/Brother/TZe_12mm.php

@@ -11,9 +11,24 @@ abstract class TZe_12mm extends Label
     private const MARGIN_SIDES =  3.20;
     private const MARGIN_ENDS  =  3.20;
 
-    public function getHeight()       { return Helper::convertUnit(self::HEIGHT, 'mm', $this->getUnit()); }
-    public function getMarginTop()    { return Helper::convertUnit(self::MARGIN_SIDES, 'mm', $this->getUnit()); }
-    public function getMarginBottom() { return Helper::convertUnit(self::MARGIN_SIDES, 'mm', $this->getUnit());}
-    public function getMarginLeft()   { return Helper::convertUnit(self::MARGIN_ENDS, 'mm', $this->getUnit()); }
-    public function getMarginRight()  { return Helper::convertUnit(self::MARGIN_ENDS, 'mm', $this->getUnit()); }
+    public function getHeight()
+    {
+        return Helper::convertUnit(self::HEIGHT, 'mm', $this->getUnit()); 
+    }
+    public function getMarginTop()
+    {
+        return Helper::convertUnit(self::MARGIN_SIDES, 'mm', $this->getUnit()); 
+    }
+    public function getMarginBottom()
+    {
+        return Helper::convertUnit(self::MARGIN_SIDES, 'mm', $this->getUnit());
+    }
+    public function getMarginLeft()
+    {
+        return Helper::convertUnit(self::MARGIN_ENDS, 'mm', $this->getUnit()); 
+    }
+    public function getMarginRight()
+    {
+        return Helper::convertUnit(self::MARGIN_ENDS, 'mm', $this->getUnit()); 
+    }
 }

app/Models/Labels/Tapes/Brother/TZe_12mm_A.php

@@ -8,18 +8,45 @@ class TZe_12mm_A extends TZe_12mm
     private const BARCODE_MARGIN = 0.30;
     private const TEXT_SIZE_MOD  = 1.00;
 
-    public function getUnit()  { return 'mm'; }
-    public function getWidth() { return 50.0; }
-    public function getSupportAssetTag()  { return true; }
-    public function getSupport1DBarcode() { return true; }
-    public function getSupport2DBarcode() { return false; }
-    public function getSupportFields()    { return 1; }
-    public function getSupportLogo()      { return false; }
-    public function getSupportTitle()     { return false; }
+    public function getUnit()
+    {
+        return 'mm'; 
+    }
+    public function getWidth()
+    {
+        return 50.0; 
+    }
+    public function getSupportAssetTag()
+    {
+        return true; 
+    }
+    public function getSupport1DBarcode()
+    {
+        return true; 
+    }
+    public function getSupport2DBarcode()
+    {
+        return false; 
+    }
+    public function getSupportFields()
+    {
+        return 1; 
+    }
+    public function getSupportLogo()
+    {
+        return false; 
+    }
+    public function getSupportTitle()
+    {
+        return false; 
+    }
 
-    public function preparePDF($pdf) {}
+    public function preparePDF($pdf)
+    {
+    }
 
-    public function write($pdf, $record) {
+    public function write($pdf, $record)
+    {
         $pa = $this->getPrintableArea();
 
         if ($record->has('barcode1d')) {

app/Models/Labels/Tapes/Brother/TZe_18mm.php

@@ -11,9 +11,24 @@ abstract class TZe_18mm extends Label
     private const MARGIN_SIDES =  3.20;
     private const MARGIN_ENDS  =  3.20;
 
-    public function getHeight()       { return Helper::convertUnit(self::HEIGHT, 'mm', $this->getUnit()); }
-    public function getMarginTop()    { return Helper::convertUnit(self::MARGIN_SIDES, 'mm', $this->getUnit()); }
-    public function getMarginBottom() { return Helper::convertUnit(self::MARGIN_SIDES, 'mm', $this->getUnit());}
-    public function getMarginLeft()   { return Helper::convertUnit(self::MARGIN_ENDS, 'mm', $this->getUnit()); }
-    public function getMarginRight()  { return Helper::convertUnit(self::MARGIN_ENDS, 'mm', $this->getUnit()); }
+    public function getHeight()
+    {
+        return Helper::convertUnit(self::HEIGHT, 'mm', $this->getUnit()); 
+    }
+    public function getMarginTop()
+    {
+        return Helper::convertUnit(self::MARGIN_SIDES, 'mm', $this->getUnit()); 
+    }
+    public function getMarginBottom()
+    {
+        return Helper::convertUnit(self::MARGIN_SIDES, 'mm', $this->getUnit());
+    }
+    public function getMarginLeft()
+    {
+        return Helper::convertUnit(self::MARGIN_ENDS, 'mm', $this->getUnit()); 
+    }
+    public function getMarginRight()
+    {
+        return Helper::convertUnit(self::MARGIN_ENDS, 'mm', $this->getUnit()); 
+    }
 }

app/Models/Labels/Tapes/Brother/TZe_18mm_A.php

@@ -8,18 +8,45 @@ class TZe_18mm_A extends TZe_18mm
     private const BARCODE_MARGIN = 0.30;
     private const TEXT_SIZE_MOD  = 1.00;
 
-    public function getUnit()  { return 'mm'; }
-    public function getWidth() { return 50.0; }
-    public function getSupportAssetTag()  { return true; }
-    public function getSupport1DBarcode() { return true; }
-    public function getSupport2DBarcode() { return false; }
-    public function getSupportFields()    { return 1; }
-    public function getSupportLogo()      { return false; }
-    public function getSupportTitle()     { return false; }
+    public function getUnit()
+    {
+        return 'mm'; 
+    }
+    public function getWidth()
+    {
+        return 50.0; 
+    }
+    public function getSupportAssetTag()
+    {
+        return true; 
+    }
+    public function getSupport1DBarcode()
+    {
+        return true; 
+    }
+    public function getSupport2DBarcode()
+    {
+        return false; 
+    }
+    public function getSupportFields()
+    {
+        return 1; 
+    }
+    public function getSupportLogo()
+    {
+        return false; 
+    }
+    public function getSupportTitle()
+    {
+        return false; 
+    }
 
-    public function preparePDF($pdf) {}
+    public function preparePDF($pdf)
+    {
+    }
 
-    public function write($pdf, $record) {
+    public function write($pdf, $record)
+    {
         $pa = $this->getPrintableArea();
 
         if ($record->has('barcode1d')) {

app/Models/Labels/Tapes/Brother/TZe_24mm.php

@@ -11,9 +11,24 @@ abstract class TZe_24mm extends Label
     private const MARGIN_SIDES =  3.20;
     private const MARGIN_ENDS  =  3.20;
 
-    public function getHeight()       { return Helper::convertUnit(self::HEIGHT, 'mm', $this->getUnit()); }
-    public function getMarginTop()    { return Helper::convertUnit(self::MARGIN_SIDES, 'mm', $this->getUnit()); }
-    public function getMarginBottom() { return Helper::convertUnit(self::MARGIN_SIDES, 'mm', $this->getUnit());}
-    public function getMarginLeft()   { return Helper::convertUnit(self::MARGIN_ENDS, 'mm', $this->getUnit()); }
-    public function getMarginRight()  { return Helper::convertUnit(self::MARGIN_ENDS, 'mm', $this->getUnit()); }
+    public function getHeight()
+    {
+        return Helper::convertUnit(self::HEIGHT, 'mm', $this->getUnit()); 
+    }
+    public function getMarginTop()
+    {
+        return Helper::convertUnit(self::MARGIN_SIDES, 'mm', $this->getUnit()); 
+    }
+    public function getMarginBottom()
+    {
+        return Helper::convertUnit(self::MARGIN_SIDES, 'mm', $this->getUnit());
+    }
+    public function getMarginLeft()
+    {
+        return Helper::convertUnit(self::MARGIN_ENDS, 'mm', $this->getUnit()); 
+    }
+    public function getMarginRight()
+    {
+        return Helper::convertUnit(self::MARGIN_ENDS, 'mm', $this->getUnit()); 
+    }
 }

app/Models/Labels/Tapes/Brother/TZe_24mm_A.php

@@ -13,18 +13,45 @@ class TZe_24mm_A extends TZe_24mm
     private const FIELD_SIZE     =   3.20;
     private const FIELD_MARGIN   =   0.15;
 
-    public function getUnit()  { return 'mm'; }
-    public function getWidth() { return 65.0; }
-    public function getSupportAssetTag()  { return true; }
-    public function getSupport1DBarcode() { return false; }
-    public function getSupport2DBarcode() { return true; }
-    public function getSupportFields()    { return 3; }
-    public function getSupportLogo()      { return false; }
-    public function getSupportTitle()     { return true; }
+    public function getUnit()
+    {
+        return 'mm'; 
+    }
+    public function getWidth()
+    {
+        return 65.0; 
+    }
+    public function getSupportAssetTag()
+    {
+        return true; 
+    }
+    public function getSupport1DBarcode()
+    {
+        return false; 
+    }
+    public function getSupport2DBarcode()
+    {
+        return true; 
+    }
+    public function getSupportFields()
+    {
+        return 3; 
+    }
+    public function getSupportLogo()
+    {
+        return false; 
+    }
+    public function getSupportTitle()
+    {
+        return true; 
+    }
 
-    public function preparePDF($pdf) {}
+    public function preparePDF($pdf)
+    {
+    }
 
-    public function write($pdf, $record) {
+    public function write($pdf, $record)
+    {
         $pa = $this->getPrintableArea();
 
         $currentX = $pa->x1;

app/Models/Labels/Tapes/Brother/TZe_24mm_B.php

@@ -15,18 +15,45 @@ class TZe_24mm_B extends TZe_24mm
     private const FIELD_SIZE     =   3.20;
     private const FIELD_MARGIN   =   0.15;
 
-    public function getUnit()  { return 'mm'; }
-    public function getWidth() { return 73.0; }
-    public function getSupportAssetTag()  { return true; }
-    public function getSupport1DBarcode() { return false; }
-    public function getSupport2DBarcode() { return true; }
-    public function getSupportFields()    { return 3; }
-    public function getSupportLogo()      { return true; }
-    public function getSupportTitle()     { return true; }
+    public function getUnit()
+    {
+        return 'mm'; 
+    }
+    public function getWidth()
+    {
+        return 73.0; 
+    }
+    public function getSupportAssetTag()
+    {
+        return true; 
+    }
+    public function getSupport1DBarcode()
+    {
+        return false; 
+    }
+    public function getSupport2DBarcode()
+    {
+        return true; 
+    }
+    public function getSupportFields()
+    {
+        return 3; 
+    }
+    public function getSupportLogo()
+    {
+        return true; 
+    }
+    public function getSupportTitle()
+    {
+        return true; 
+    }
 
-    public function preparePDF($pdf) {}
+    public function preparePDF($pdf)
+    {
+    }
 
-    public function write($pdf, $record) {
+    public function write($pdf, $record)
+    {
         $pa = $this->getPrintableArea();
 
         $currentX = $pa->x1;

app/Models/Labels/Tapes/Brother/TZe_24mm_C.php

@@ -15,18 +15,45 @@ class TZe_24mm_C extends TZe_24mm
     private const FIELD_SIZE     =   3.20;
     private const FIELD_MARGIN   =   0.15;
 
-    public function getUnit()  { return 'mm'; }
-    public function getWidth() { return 34.0; }
-    public function getSupportAssetTag()  { return true; }
-    public function getSupport1DBarcode() { return false; }
-    public function getSupport2DBarcode() { return true; }
-    public function getSupportFields()    { return 0; }
-    public function getSupportLogo()      { return true; }
-    public function getSupportTitle()     { return false; }
+    public function getUnit()
+    {
+        return 'mm'; 
+    }
+    public function getWidth()
+    {
+        return 34.0; 
+    }
+    public function getSupportAssetTag()
+    {
+        return true; 
+    }
+    public function getSupport1DBarcode()
+    {
+        return false; 
+    }
+    public function getSupport2DBarcode()
+    {
+        return true; 
+    }
+    public function getSupportFields()
+    {
+        return 0; 
+    }
+    public function getSupportLogo()
+    {
+        return true; 
+    }
+    public function getSupportTitle()
+    {
+        return false; 
+    }
 
-    public function preparePDF($pdf) {}
+    public function preparePDF($pdf)
+    {
+    }
 
-    public function write($pdf, $record) {
+    public function write($pdf, $record)
+    {
         $pa = $this->getPrintableArea();
 
         $currentX = $pa->x1;

app/Models/Labels/Tapes/Brother/TZe_24mm_D.php

@@ -14,18 +14,45 @@ class TZe_24mm_D extends TZe_24mm
     private const FIELD_MARGIN   =   0.35;
     private const BARCODE1D_SIZE =   3.00;  // Size for the C128 barcode at bottom
 
-    public function getUnit()  { return 'mm'; }
-    public function getWidth() { return 65.0; }
-    public function getSupportAssetTag()  { return true; }
-    public function getSupport1DBarcode() { return true; }
-    public function getSupport2DBarcode() { return true; }
-    public function getSupportFields()    { return 3; }
-    public function getSupportLogo()      { return false; }
-    public function getSupportTitle()     { return true; }
+    public function getUnit()
+    {
+        return 'mm'; 
+    }
+    public function getWidth()
+    {
+        return 65.0; 
+    }
+    public function getSupportAssetTag()
+    {
+        return true; 
+    }
+    public function getSupport1DBarcode()
+    {
+        return true; 
+    }
+    public function getSupport2DBarcode()
+    {
+        return true; 
+    }
+    public function getSupportFields()
+    {
+        return 3; 
+    }
+    public function getSupportLogo()
+    {
+        return false; 
+    }
+    public function getSupportTitle()
+    {
+        return true; 
+    }
 
-    public function preparePDF($pdf) {}
+    public function preparePDF($pdf)
+    {
+    }
 
-    public function write($pdf, $record) {
+    public function write($pdf, $record)
+    {
         $pa = $this->getPrintableArea();
 
         $currentX = $pa->x1;

app/Models/Labels/Tapes/Brother/TZe_62mm_Landscape.php

@@ -14,10 +14,28 @@ abstract class TZe_62mm_Landscape extends Label
     private const MARGIN_SIDES =  1.50;
     private const MARGIN_ENDS  =  1.50;
 
-    public function getWidth()        { return Helper::convertUnit(self::WIDTH, 'mm', $this->getUnit()); }
-    public function getMarginTop()    { return Helper::convertUnit(self::MARGIN_SIDES, 'mm', $this->getUnit()); }
-    public function getMarginBottom() { return Helper::convertUnit(self::MARGIN_SIDES, 'mm', $this->getUnit());}
-    public function getMarginLeft()   { return Helper::convertUnit(self::MARGIN_ENDS, 'mm', $this->getUnit()); }
-    public function getMarginRight()  { return Helper::convertUnit(self::MARGIN_ENDS, 'mm', $this->getUnit()); }
-    public function getRotation()     { return 90; }
+    public function getWidth()
+    {
+        return Helper::convertUnit(self::WIDTH, 'mm', $this->getUnit()); 
+    }
+    public function getMarginTop()
+    {
+        return Helper::convertUnit(self::MARGIN_SIDES, 'mm', $this->getUnit()); 
+    }
+    public function getMarginBottom()
+    {
+        return Helper::convertUnit(self::MARGIN_SIDES, 'mm', $this->getUnit());
+    }
+    public function getMarginLeft()
+    {
+        return Helper::convertUnit(self::MARGIN_ENDS, 'mm', $this->getUnit()); 
+    }
+    public function getMarginRight()
+    {
+        return Helper::convertUnit(self::MARGIN_ENDS, 'mm', $this->getUnit()); 
+    }
+    public function getRotation()
+    {
+        return 90; 
+    }
 }

app/Models/Labels/Tapes/Brother/TZe_62mm_Landscape_A.php

@@ -4,14 +4,38 @@ namespace App\Models\Labels\Tapes\Brother;
 
 class TZe_62mm_Landscape_A extends TZe_62mm_Landscape
 {
-    public function getUnit()             { return 'mm'; }
-    public function getHeight()           { return 31.50; }
-    public function getSupportAssetTag()  { return true; }
-    public function getSupport1DBarcode() { return true; }
-    public function getSupport2DBarcode() { return true; }
-    public function getSupportFields()    { return 2; }
-    public function getSupportLogo()      { return true; }
-    public function getSupportTitle()     { return true; }
+    public function getUnit()
+    {
+        return 'mm'; 
+    }
+    public function getHeight()
+    {
+        return 31.50; 
+    }
+    public function getSupportAssetTag()
+    {
+        return true; 
+    }
+    public function getSupport1DBarcode()
+    {
+        return true; 
+    }
+    public function getSupport2DBarcode()
+    {
+        return true; 
+    }
+    public function getSupportFields()
+    {
+        return 2; 
+    }
+    public function getSupportLogo()
+    {
+        return true; 
+    }
+    public function getSupportTitle()
+    {
+        return true; 
+    }
 
     private const BARCODE1D_HEIGHT =   3.00;
     private const BARCODE1D_MARGIN =   3.00;
@@ -27,9 +51,12 @@ class TZe_62mm_Landscape_A extends TZe_62mm_Landscape
     private const FIELD_SIZE       =   3.00;
     private const FIELD_MARGIN     =   0.10;
 
-    public function preparePDF($pdf) {}
+    public function preparePDF($pdf)
+    {
+    }
 
-    public function write($pdf, $record) {
+    public function write($pdf, $record)
+    {
         $pa = $this->getPrintableArea();
 
         $currentX = $pa->x1;

app/Models/Labels/Tapes/Dymo/LabelWriter.php

@@ -11,9 +11,24 @@ abstract class LabelWriter extends Label
     private const MARGIN_SIDES =  0.1;
     private const MARGIN_ENDS  =  0.1;
 
-    public function getHeight()       { return Helper::convertUnit(self::HEIGHT, 'in', $this->getUnit()); }
-    public function getMarginTop()    { return Helper::convertUnit(self::MARGIN_SIDES, 'in', $this->getUnit()); }
-    public function getMarginBottom() { return Helper::convertUnit(self::MARGIN_SIDES, 'in', $this->getUnit());}
-    public function getMarginLeft()   { return Helper::convertUnit(self::MARGIN_ENDS, 'in', $this->getUnit()); }
-    public function getMarginRight()  { return Helper::convertUnit(self::MARGIN_ENDS, 'in', $this->getUnit()); }
+    public function getHeight()
+    {
+        return Helper::convertUnit(self::HEIGHT, 'in', $this->getUnit()); 
+    }
+    public function getMarginTop()
+    {
+        return Helper::convertUnit(self::MARGIN_SIDES, 'in', $this->getUnit()); 
+    }
+    public function getMarginBottom()
+    {
+        return Helper::convertUnit(self::MARGIN_SIDES, 'in', $this->getUnit());
+    }
+    public function getMarginLeft()
+    {
+        return Helper::convertUnit(self::MARGIN_ENDS, 'in', $this->getUnit()); 
+    }
+    public function getMarginRight()
+    {
+        return Helper::convertUnit(self::MARGIN_ENDS, 'in', $this->getUnit()); 
+    }
 }

app/Models/Labels/Tapes/Dymo/LabelWriter_1933081.php

@@ -14,19 +14,49 @@ class LabelWriter_1933081 extends LabelWriter
     private const FIELD_SIZE     =   2.80;
     private const FIELD_MARGIN   =   0.15;
 
-    public function getUnit()  { return 'mm'; }
-    public function getWidth() { return 89; }
-    public function getHeight() { return 25; }
-    public function getSupportAssetTag()  { return true; }
-    public function getSupport1DBarcode() { return true; }
-    public function getSupport2DBarcode() { return true; }
-    public function getSupportFields()    { return 5; }
-    public function getSupportLogo()      { return false; }
-    public function getSupportTitle()     { return true; }
+    public function getUnit()
+    {
+        return 'mm'; 
+    }
+    public function getWidth()
+    {
+        return 89; 
+    }
+    public function getHeight()
+    {
+        return 25; 
+    }
+    public function getSupportAssetTag()
+    {
+        return true; 
+    }
+    public function getSupport1DBarcode()
+    {
+        return true; 
+    }
+    public function getSupport2DBarcode()
+    {
+        return true; 
+    }
+    public function getSupportFields()
+    {
+        return 5; 
+    }
+    public function getSupportLogo()
+    {
+        return false; 
+    }
+    public function getSupportTitle()
+    {
+        return true; 
+    }
 
-    public function preparePDF($pdf) {}
+    public function preparePDF($pdf)
+    {
+    }
 
-    public function write($pdf, $record) {
+    public function write($pdf, $record)
+    {
         $pa = $this->getPrintableArea();
 
         $currentX = $pa->x1;

app/Models/Labels/Tapes/Dymo/LabelWriter_2112283.php

@@ -14,19 +14,49 @@ class LabelWriter_2112283 extends LabelWriter
     private const FIELD_SIZE     =   2.80;
     private const FIELD_MARGIN   =   0.15;
 
-    public function getUnit()  { return 'mm'; }
-    public function getWidth() { return 54; }
-    public function getHeight() { return 25; }
-    public function getSupportAssetTag()  { return true; }
-    public function getSupport1DBarcode() { return true; }
-    public function getSupport2DBarcode() { return true; }
-    public function getSupportFields()    { return 5; }
-    public function getSupportLogo()      { return false; }
-    public function getSupportTitle()     { return true; }
+    public function getUnit()
+    {
+        return 'mm'; 
+    }
+    public function getWidth()
+    {
+        return 54; 
+    }
+    public function getHeight()
+    {
+        return 25; 
+    }
+    public function getSupportAssetTag()
+    {
+        return true; 
+    }
+    public function getSupport1DBarcode()
+    {
+        return true; 
+    }
+    public function getSupport2DBarcode()
+    {
+        return true; 
+    }
+    public function getSupportFields()
+    {
+        return 5; 
+    }
+    public function getSupportLogo()
+    {
+        return false; 
+    }
+    public function getSupportTitle()
+    {
+        return true; 
+    }
 
-    public function preparePDF($pdf) {}
+    public function preparePDF($pdf)
+    {
+    }
 
-    public function write($pdf, $record) {
+    public function write($pdf, $record)
+    {
         $pa = $this->getPrintableArea();
 
         $currentX = $pa->x1;

app/Models/Labels/Tapes/Dymo/LabelWriter_30252.php

@@ -16,18 +16,45 @@ class LabelWriter_30252 extends LabelWriter
 
 
 
-    public function getUnit()  { return 'mm'; }
-    public function getWidth() { return 96.52; }
-    public function getSupportAssetTag()  { return true; }
-    public function getSupport1DBarcode() { return true; }
-    public function getSupport2DBarcode() { return true; }
-    public function getSupportFields()    { return 3; }
-    public function getSupportLogo()      { return false; }
-    public function getSupportTitle()     { return true; }
+    public function getUnit()
+    {
+        return 'mm'; 
+    }
+    public function getWidth()
+    {
+        return 96.52; 
+    }
+    public function getSupportAssetTag()
+    {
+        return true; 
+    }
+    public function getSupport1DBarcode()
+    {
+        return true; 
+    }
+    public function getSupport2DBarcode()
+    {
+        return true; 
+    }
+    public function getSupportFields()
+    {
+        return 3; 
+    }
+    public function getSupportLogo()
+    {
+        return false; 
+    }
+    public function getSupportTitle()
+    {
+        return true; 
+    }
 
-    public function preparePDF($pdf) {}
+    public function preparePDF($pdf)
+    {
+    }
 
-    public function write($pdf, $record) {
+    public function write($pdf, $record)
+    {
         $pa = $this->getPrintableArea();
 
         $currentX = $pa->x1;

app/Models/Labels/Tapes/Generic/Continuous_53mm.php

@@ -15,28 +15,30 @@ abstract class Continuous_53mm extends GenericTape
      * 
      * Assumes tape is continuous, set to false and specify 
      * $spacing in concrete classfor die-cut labels
-     * 
      *
-     * @param float $height Height of the label in mm (default 60mm)
-     * @param bool $continuous Whether the tape is continuous or pre-cut
-     * @param float $spacing Spacing between labels for non-continuous tapes (in mm)
+     * @param float $height     Height of the label in mm (default 60mm)
+     * @param bool  $continuous Whether the tape is continuous or pre-cut
+     * @param float $spacing    Spacing between labels for non-continuous tapes (in mm)
      */
-    public function __construct($height = 60.0, $continuous = true, $spacing = 0.0) {
+    public function __construct($height = 60.0, $continuous = true, $spacing = 0.0)
+    {
         parent::__construct(self::TAPE_WIDTH, $height, $continuous, $spacing);
         $this->tapeHeight = $height;
     }
     
-    public function getBarcodeRatio() {
+    public function getBarcodeRatio()
+    {
         return 0.9;  // Barcode should use 90% of available width
     }
     
     /**
      * Calculate the required height for the content
      *
-     * @param $record The record to calculate height for
+     * @param  $record The record to calculate height for
      * @return float The calculated height in mm
      */
-    protected function calculateRequiredHeight($record) {
+    protected function calculateRequiredHeight($record)
+    {
         $height = $this->marginTop + $this->marginBottom;
         
         // Add title height if present
@@ -45,8 +47,9 @@ abstract class Continuous_53mm extends GenericTape
         }
         
         // Add barcode height if present
-        if (($record->has('barcode2d') && $this->getSupport2DBarcode()) ||
-            ($record->has('barcode') && $this->getSupport1DBarcode())) {
+        if (($record->has('barcode2d') && $this->getSupport2DBarcode()) 
+            || ($record->has('barcode') && $this->getSupport1DBarcode())
+        ) {
             $pa = $this->getPrintableArea();
             $usableWidth = $pa->w;
             $barcodeSize = $usableWidth * $this->getBarcodeRatio();
@@ -71,31 +74,34 @@ abstract class Continuous_53mm extends GenericTape
     /**
      * Override the writeAll method to support dynamic page sizes for continuous tapes
      */
-    public function writeAll($pdf, $data) {
+    public function writeAll($pdf, $data)
+    {
         // Use auto-sizing for continuous tapes, fixed height for die-cut tapes
         if ($this->continuous) {
-            $data->each(function ($record, $index) use ($pdf) {
-                // Calculate the required height for this record
-                $requiredHeight = $this->calculateRequiredHeight($record);
+            $data->each(
+                function ($record, $index) use ($pdf) {
+                    // Calculate the required height for this record
+                    $requiredHeight = $this->calculateRequiredHeight($record);
                 
-                // Temporarily update the height property
-                $originalHeight = $this->height;
-                $this->height = $requiredHeight;
+                    // Temporarily update the height property
+                    $originalHeight = $this->height;
+                    $this->height = $requiredHeight;
                 
-                // Add a new page with the calculated dimensions
-                $pdf->AddPage(
-                    $this->getOrientation(),
-                    [$this->getWidth(), $requiredHeight],
-                    false, // Don't reset page number
-                    false  // Don't reset object ID
-                );
+                    // Add a new page with the calculated dimensions
+                    $pdf->AddPage(
+                        $this->getOrientation(),
+                        [$this->getWidth(), $requiredHeight],
+                        false, // Don't reset page number
+                        false  // Don't reset object ID
+                    );
                 
-                // Write the content
-                $this->write($pdf, $record);
+                    // Write the content
+                    $this->write($pdf, $record);
                 
-                // Restore the original height
-                $this->height = $originalHeight;
-            });
+                    // Restore the original height
+                    $this->height = $originalHeight;
+                }
+            );
         } else {
             // Use the default implementation for non-continuous (die-cut) tapes
             parent::writeAll($pdf, $data);

app/Models/Labels/Tapes/Generic/Continuous_53mm_A.php

@@ -5,19 +5,42 @@ namespace App\Models\Labels\Tapes\Generic;
 class Continuous_53mm_A extends Continuous_53mm
 {
     
-    public function getUnit() { return 'mm'; }
-    public function getSupportAssetTag() { return false; }
-    public function getSupport1DBarcode() { return true; }
-    public function getSupport2DBarcode() { return true; }
-    public function getSupportFields() { return 5; }
-    public function getSupportLogo() { return false; }
-    public function getSupportTitle() { return true; }
+    public function getUnit()
+    {
+        return 'mm'; 
+    }
+    public function getSupportAssetTag()
+    {
+        return false; 
+    }
+    public function getSupport1DBarcode()
+    {
+        return true; 
+    }
+    public function getSupport2DBarcode()
+    {
+        return true; 
+    }
+    public function getSupportFields()
+    {
+        return 5; 
+    }
+    public function getSupportLogo()
+    {
+        return false; 
+    }
+    public function getSupportTitle()
+    {
+        return true; 
+    }
     
-    public function preparePDF($pdf) {
+    public function preparePDF($pdf)
+    {
         $pdf->SetAutoPageBreak(false);
     }
     
-    public function write($pdf, $record) {
+    public function write($pdf, $record)
+    {
         $pa = $this->getPrintableArea();
         
         $currentX = $pa->x1;

app/Models/Labels/Tapes/Generic/Continuous_Landscape_0_59in.php

@@ -13,11 +13,12 @@ abstract class Continuous_Landscape_0_59in extends GenericTape
     protected float $minHeight = 1;
 
     /**
-     * @param float $length Length of the label in inches (default 2.36in which is 60mm)
-     * @param bool $continuous Whether the tape is continuous or pre-cut
-     * @param float $spacing Spacing between labels for non-continuous tapes (in inches)
+     * @param float $length     Length of the label in inches (default 2.36in which is 60mm)
+     * @param bool  $continuous Whether the tape is continuous or pre-cut
+     * @param float $spacing    Spacing between labels for non-continuous tapes (in inches)
      */
-    public function __construct($length = 0.6, $continuous = true, $spacing = 0.0) {
+    public function __construct($length = 0.6, $continuous = true, $spacing = 0.0)
+    {
         // Swap width and height for landscape orientation
         // The height becomes the width, and the length becomes the height
         parent::__construct($length, self::TAPE_WIDTH, $continuous, $spacing);
@@ -44,24 +45,27 @@ abstract class Continuous_Landscape_0_59in extends GenericTape
         $this->tagSize = $baseFontSize * 0.8;            // 80% of base font size
     }
     
-    public function getBarcodeRatio() {
+    public function getBarcodeRatio()
+    {
         return 1.0;  // Barcode should use 100% of available height
     }
     
     /**
      * Calculate the required length for the content
      *
-     * @param $record The record to calculate length for
+     * @param  $record The record to calculate length for
      * @return float The calculated length in inches
      */
-    protected function calculateRequiredLength($record) {
+    protected function calculateRequiredLength($record)
+    {
 
         // Calculate length needed for barcode and fields side by side
         $requiredLength = 0;
         
         // Add barcode length if present
-        if (($record->has('barcode2d') && $this->getSupport2DBarcode()) ||
-            ($record->has('barcode') && $this->getSupport1DBarcode())) {
+        if (($record->has('barcode2d') && $this->getSupport2DBarcode()) 
+            || ($record->has('barcode') && $this->getSupport1DBarcode())
+        ) {
             // Use full tape width for barcode size
             $barcodeSize = self::TAPE_WIDTH;
             $requiredLength += $barcodeSize + $this->barcodeMargin * 0.3; // Minimal margin
@@ -107,14 +111,15 @@ abstract class Continuous_Landscape_0_59in extends GenericTape
     /**
      * Calculate text width accurately using the PDF object
      *
-     * @param $pdf The PDF object
-     * @param string $text The text to measure
-     * @param string $font The font to use
-     * @param string $style The font style
-     * @param float $size The font size
+     * @param  $pdf   The PDF object
+     * @param  string $text  The text to measure
+     * @param  string $font  The font to use
+     * @param  string $style The font style
+     * @param  float  $size  The font size
      * @return float The calculated width
      */
-    protected function calculateTextWidth($pdf, $text, $font, $style, $size) {
+    protected function calculateTextWidth($pdf, $text, $font, $style, $size)
+    {
         $originalFont = $pdf->getFontFamily();
         $originalStyle = $pdf->getFontStyle();
         $originalSize = $pdf->getFontSizePt();
@@ -131,37 +136,40 @@ abstract class Continuous_Landscape_0_59in extends GenericTape
     /**
      * Override the writeAll method to support dynamic page sizes for continuous tapes
      */
-    public function writeAll($pdf, $data) {
+    public function writeAll($pdf, $data)
+    {
         // Use auto-sizing for continuous tapes, fixed height for die-cut tapes
         if ($this->continuous) {
-            $data->each(function ($record, $index) use ($pdf) {
-                // Calculate the required length by calling write with calculateOnly=true
-                $requiredLength = $this->write($pdf, $record);
+            $data->each(
+                function ($record, $index) use ($pdf) {
+                    // Calculate the required length by calling write with calculateOnly=true
+                    $requiredLength = $this->write($pdf, $record);
                 
-                // If write didn't return a length (old implementation), fall back to calculateRequiredLength
-                if ($requiredLength === null) {
-                    $requiredLength = $this->calculateRequiredLength($record);
+                    // If write didn't return a length (old implementation), fall back to calculateRequiredLength
+                    if ($requiredLength === null) {
+                        $requiredLength = $this->calculateRequiredLength($record);
+                    }
+                
+                    // Temporarily update the height property
+                    $originalHeight = $this->height;
+                    $this->height = self::TAPE_WIDTH; // Keep height fixed at tape width
+                
+                    // Add a new page with the calculated dimensions
+                    // Keep height fixed at TAPE_WIDTH, use calculated length for width
+                    $pdf->AddPage(
+                        $this->getOrientation(),
+                        [$requiredLength, self::TAPE_WIDTH],
+                        false, // Don't reset page number
+                        false  // Don't reset object ID
+                    );
+                
+                    // Write the content
+                    $this->write($pdf, $record);
+                
+                    // Restore the original height
+                    $this->height = $originalHeight;
                 }
-                
-                // Temporarily update the height property
-                $originalHeight = $this->height;
-                $this->height = self::TAPE_WIDTH; // Keep height fixed at tape width
-                
-                // Add a new page with the calculated dimensions
-                // Keep height fixed at TAPE_WIDTH, use calculated length for width
-                $pdf->AddPage(
-                    $this->getOrientation(),
-                    [$requiredLength, self::TAPE_WIDTH],
-                    false, // Don't reset page number
-                    false  // Don't reset object ID
-                );
-                
-                // Write the content
-                $this->write($pdf, $record);
-                
-                // Restore the original height
-                $this->height = $originalHeight;
-            });
+            );
         } else {
             // Use the default implementation for non-continuous (die-cut) tapes
             parent::writeAll($pdf, $data);

app/Models/Labels/Tapes/Generic/Continuous_Landscape_0_59in_A.php

@@ -4,19 +4,42 @@ namespace App\Models\Labels\Tapes\Generic;
 
 class Continuous_Landscape_0_59in_A extends Continuous_Landscape_0_59in
 {
-    public function getUnit() { return 'in'; }
-    public function getSupportAssetTag() { return false; }
-    public function getSupport1DBarcode() { return true; }
-    public function getSupport2DBarcode() { return true; }
-    public function getSupportFields() { return 2; }
-    public function getSupportLogo() { return false; }
-    public function getSupportTitle() { return false; }
+    public function getUnit()
+    {
+        return 'in'; 
+    }
+    public function getSupportAssetTag()
+    {
+        return false; 
+    }
+    public function getSupport1DBarcode()
+    {
+        return true; 
+    }
+    public function getSupport2DBarcode()
+    {
+        return true; 
+    }
+    public function getSupportFields()
+    {
+        return 2; 
+    }
+    public function getSupportLogo()
+    {
+        return false; 
+    }
+    public function getSupportTitle()
+    {
+        return false; 
+    }
 
-    public function preparePDF($pdf) {
+    public function preparePDF($pdf)
+    {
         $pdf->SetAutoPageBreak(false);
     }
 
-    public function write($pdf, $record, $calculateOnly = false) {
+    public function write($pdf, $record, $calculateOnly = false)
+    {
         $pa = $this->getPrintableArea();
         
         $currentX = $pa->x1;

app/Models/Labels/Tapes/Generic/GenericTape.php

@@ -35,12 +35,13 @@ abstract class GenericTape extends Label
     /**
      * Constructor for generic tape
      * 
-     * @param float $width Width of the tape in mm
-     * @param float $height Height of the label in mm (for continuous tapes, this is the default height)
-     * @param bool $continuous Whether the tape is continuous or pre-cut
-     * @param float $spacing Spacing between labels for non-continuous tapes (in mm)
+     * @param float $width      Width of the tape in mm
+     * @param float $height     Height of the label in mm (for continuous tapes, this is the default height)
+     * @param bool  $continuous Whether the tape is continuous or pre-cut
+     * @param float $spacing    Spacing between labels for non-continuous tapes (in mm)
      */
-    public function __construct(float $width, float $height, bool $continuous = true, float $spacing = 0.0) {
+    public function __construct(float $width, float $height, bool $continuous = true, float $spacing = 0.0)
+    {
         $this->width = $width;
         $this->height = $height;
         $this->continuous = $continuous;
@@ -70,17 +71,38 @@ abstract class GenericTape extends Label
     }
     
     // Unit of measurement
-    public function getUnit() { return 'mm'; }
+    public function getUnit()
+    {
+        return 'mm'; 
+    }
     
     // Label dimensions
-    public function getWidth() { return $this->width; }
-    public function getHeight() { return $this->height; }
+    public function getWidth()
+    {
+        return $this->width; 
+    }
+    public function getHeight()
+    {
+        return $this->height; 
+    }
     
     // Margins
-    public function getMarginTop() { return $this->marginTop; }
-    public function getMarginBottom() { return $this->marginBottom; }
-    public function getMarginLeft() { return $this->marginLeft; }
-    public function getMarginRight() { return $this->marginRight; }
+    public function getMarginTop()
+    {
+        return $this->marginTop; 
+    }
+    public function getMarginBottom()
+    {
+        return $this->marginBottom; 
+    }
+    public function getMarginLeft()
+    {
+        return $this->marginLeft; 
+    }
+    public function getMarginRight()
+    {
+        return $this->marginRight; 
+    }
 
     
     /**
@@ -88,7 +110,8 @@ abstract class GenericTape extends Label
      * 
      * @return bool
      */
-    public function isContinuous() {
+    public function isContinuous()
+    {
         return $this->continuous;
     }
     
@@ -97,7 +120,8 @@ abstract class GenericTape extends Label
      * 
      * @return float
      */
-    public function getSpacing() {
+    public function getSpacing()
+    {
         return $this->spacing;
     }
 }

app/Models/Labels/Tapes/Generic/Tape_53mm.php

@@ -12,11 +12,12 @@ abstract class Tape_53mm extends GenericTape
     /**
      * Constructor for 53mm tape
      *
-     * @param float $height Height of the label in mm (default 30mm)
-     * @param bool $continuous Whether the tape is continuous or pre-cut
-     * @param float $spacing Spacing between labels for non-continuous tapes (in mm)
+     * @param float $height     Height of the label in mm (default 30mm)
+     * @param bool  $continuous Whether the tape is continuous or pre-cut
+     * @param float $spacing    Spacing between labels for non-continuous tapes (in mm)
      */
-    public function __construct(float $height = 30.0, bool $continuous = true, float $spacing = 0.0) {
+    public function __construct(float $height = 30.0, bool $continuous = true, float $spacing = 0.0)
+    {
         parent::__construct(self::TAPE_WIDTH, $height, $continuous, $spacing);
         $this->tapeHeight = $height;
     }
@@ -26,7 +27,8 @@ abstract class Tape_53mm extends GenericTape
      *
      * @return float
      */
-    public function getBarcodeRatio() {
+    public function getBarcodeRatio()
+    {
         return 0.9;  // Barcode should use 90% of available width
     }
 }

app/Models/Labels/Tapes/Generic/Tape_53mm_A.php

@@ -5,23 +5,47 @@ namespace App\Models\Labels\Tapes\Generic;
 class Tape_53mm_A extends Tape_53mm
 {
 
-    public function __construct() {
+    public function __construct()
+    {
         parent::__construct(40.0, true, 0.0);
     }
     
-    public function getUnit() { return 'mm'; }
-    public function getSupportAssetTag() { return false; }
-    public function getSupport1DBarcode() { return false; }
-    public function getSupport2DBarcode() { return true; }
-    public function getSupportFields() { return 5; }
-    public function getSupportLogo() { return false; }
-    public function getSupportTitle() { return true; }
+    public function getUnit()
+    {
+        return 'mm'; 
+    }
+    public function getSupportAssetTag()
+    {
+        return false; 
+    }
+    public function getSupport1DBarcode()
+    {
+        return false; 
+    }
+    public function getSupport2DBarcode()
+    {
+        return true; 
+    }
+    public function getSupportFields()
+    {
+        return 5; 
+    }
+    public function getSupportLogo()
+    {
+        return false; 
+    }
+    public function getSupportTitle()
+    {
+        return true; 
+    }
     
-    public function preparePDF($pdf) {
+    public function preparePDF($pdf)
+    {
         $pdf->SetAutoPageBreak(false);
     }
     
-    public function write($pdf, $record) {
+    public function write($pdf, $record)
+    {
         $pa = $this->getPrintableArea();
         
         $currentX = $pa->x1;

app/Models/Ldap.php

@@ -31,7 +31,7 @@ class Ldap extends Model
      * Makes a connection to LDAP using the settings in Admin > Settings.
      *
      * @author [A. Gianotto] [<snipe@snipe.net>]
-     * @since [v3.0]
+     * @since  [v3.0]
      * @return connection
      */
     public static function connectToLdap()
@@ -81,10 +81,10 @@ class Ldap extends Model
      * Binds/authenticates the user to LDAP, and returns their attributes.
      *
      * @author [A. Gianotto] [<snipe@snipe.net>]
-     * @since [v3.0]
-     * @param $username
-     * @param $password
-     * @param bool|false $user
+     * @since  [v3.0]
+     * @param  $username
+     * @param  $password
+     * @param  bool|false $user
      * @return bool true    if the username and/or password provided are valid
      *              false   if the username and/or password provided are invalid
      *         array of ldap_attributes if $user is true
@@ -160,8 +160,8 @@ class Ldap extends Model
      * Here we also return a better error if the app key is donked.
      *
      * @author [A. Gianotto] [<snipe@snipe.net>]
-     * @since [v3.0]
-     * @param bool|false $user
+     * @since  [v3.0]
+     * @param  bool|false $user
      * @return bool true    if the username and/or password provided are valid
      *              false   if the username and/or password provided are invalid
      */
@@ -169,37 +169,37 @@ class Ldap extends Model
     {
         $ldap_username = Setting::getSettings()->ldap_uname;
 
-		if ( $ldap_username ) {
-			// Lets return some nicer messages for users who donked their app key, and disable LDAP
-			try {
-				$ldap_pass = Crypt::decrypt(Setting::getSettings()->ldap_pword);
-			} catch (Exception $e) {
-				throw new Exception('Your app key has changed! Could not decrypt LDAP password using your current app key, so LDAP authentication has been disabled. Login with a local account, update the LDAP password and re-enable it in Admin > Settings.');
-			}
+        if ($ldap_username ) {
+            // Lets return some nicer messages for users who donked their app key, and disable LDAP
+            try {
+                $ldap_pass = Crypt::decrypt(Setting::getSettings()->ldap_pword);
+            } catch (Exception $e) {
+                throw new Exception('Your app key has changed! Could not decrypt LDAP password using your current app key, so LDAP authentication has been disabled. Login with a local account, update the LDAP password and re-enable it in Admin > Settings.');
+            }
 
-			if (! $ldapbind = @ldap_bind($connection, $ldap_username, $ldap_pass)) {
-				throw new Exception('Could not bind to LDAP: '.ldap_error($connection));
-			}
-			// TODO - this just "falls off the end" but the function states that it should return true or false
-			// unfortunately, one of the use cases for this function is wrong and *needs* for that failure mode to fire
-			// so I don't want to fix this right now.
-			// this method MODIFIES STATE on the passed-in $connection and just returns true or false (or, in this case, undefined)
-			// at the next refactor, this should be appropriately modified to be more consistent.
-		} else {
-			// LDAP should also work with anonymous bind (no dn, no password available)
-			if (! $ldapbind = @ldap_bind($connection )) {
-				throw new Exception('Could not bind to LDAP: '.ldap_error($connection));
-			}
-		}
-	}
+            if (! $ldapbind = @ldap_bind($connection, $ldap_username, $ldap_pass)) {
+                throw new Exception('Could not bind to LDAP: '.ldap_error($connection));
+            }
+            // TODO - this just "falls off the end" but the function states that it should return true or false
+            // unfortunately, one of the use cases for this function is wrong and *needs* for that failure mode to fire
+            // so I don't want to fix this right now.
+            // this method MODIFIES STATE on the passed-in $connection and just returns true or false (or, in this case, undefined)
+            // at the next refactor, this should be appropriately modified to be more consistent.
+        } else {
+            // LDAP should also work with anonymous bind (no dn, no password available)
+            if (! $ldapbind = @ldap_bind($connection)) {
+                throw new Exception('Could not bind to LDAP: '.ldap_error($connection));
+            }
+        }
+    }
 
     /**
      * Parse and map LDAP attributes based on settings
      *
      * @author [A. Gianotto] [<snipe@snipe.net>]
-     * @since [v3.0]
+     * @since  [v3.0]
      *
-     * @param $ldapatttibutes
+     * @param  $ldapatttibutes
      * @return array|bool
      */
     public static function parseAndMapLdapAttributes($ldapattributes)
@@ -238,8 +238,8 @@ class Ldap extends Model
      * Create user from LDAP attributes
      *
      * @author [A. Gianotto] [<snipe@snipe.net>]
-     * @since [v3.0]
-     * @param $ldapatttibutes
+     * @since  [v3.0]
+     * @param  $ldapatttibutes
      * @return User | bool
      */
     public static function createUserFromLdap($ldapatttibutes, $password)
@@ -279,11 +279,11 @@ class Ldap extends Model
      * Searches LDAP
      *
      * @author [A. Gianotto] [<snipe@snipe.net>]
-     * @since [v3.0]
-     * @param $base_dn
-     * @param $count
-     * @param $filter
-     * @param $attributes
+     * @since  [v3.0]
+     * @param  $base_dn
+     * @param  $count
+     * @param  $filter
+     * @param  $attributes
      * @return array|bool
      */
     public static function findLdapUsers($base_dn = null, $count = -1, $filter = null, $attributes = [])
@@ -331,7 +331,7 @@ class Ldap extends Model
             $errmsg = null;
             $referrals = null;
             $controls = [];
-            ldap_parse_result($ldapconn, $search_results, $errcode , $matcheddn , $errmsg , $referrals, $controls);
+            ldap_parse_result($ldapconn, $search_results, $errcode, $matcheddn, $errmsg, $referrals, $controls);
             if (isset($controls[LDAP_CONTROL_PAGEDRESULTS]['value']['cookie'])) {
                 // You need to pass the cookie from the last call to the next one
                 $cookie = $controls[LDAP_CONTROL_PAGEDRESULTS]['value']['cookie'];

app/Models/License.php

@@ -123,32 +123,36 @@ class License extends Depreciable
      * Update seat counts when the license is updated
      *
      * @author A. Gianotto <snipe@snipe.net>
-     * @since [v3.0]
+     * @since  [v3.0]
      */
     public static function boot()
     {
         parent::boot();
         // We need to listen for created for the initial setup so that we have a license ID.
-        static::created(function ($license) {
-            $newSeatCount = $license->getAttributes()['seats'];
+        static::created(
+            function ($license) {
+                $newSeatCount = $license->getAttributes()['seats'];
 
-            return static::adjustSeatCount($license, 0, $newSeatCount);
-        });
+                return static::adjustSeatCount($license, 0, $newSeatCount);
+            }
+        );
         // However, we listen for updating to be able to prevent the edit if we cannot delete enough seats.
-        static::updating(function ($license) {
-            $newSeatCount = $license->getAttributes()['seats'];
-            //$oldSeatCount = isset($license->getOriginal()['seats']) ? $license->getOriginal()['seats'] : 0;
-            /*
-               That previous method *did* mostly work, but if you ever managed to get your $license->seats value out of whack
-               with your actual count of license_seats *records*, you would never manage to get back 'into whack'.
-               The below method actually grabs a count of existing license_seats records, so it will be more accurate.
-               This means that if your license_seats are out of whack, you can change the quantity and hit 'save' and it
-               will manage to 'true up' and make your counts line up correctly.
-            */
-            $oldSeatCount = $license->license_seats_count;
+        static::updating(
+            function ($license) {
+                $newSeatCount = $license->getAttributes()['seats'];
+                //$oldSeatCount = isset($license->getOriginal()['seats']) ? $license->getOriginal()['seats'] : 0;
+                /*
+                That previous method *did* mostly work, but if you ever managed to get your $license->seats value out of whack
+                with your actual count of license_seats *records*, you would never manage to get back 'into whack'.
+                The below method actually grabs a count of existing license_seats records, so it will be more accurate.
+                This means that if your license_seats are out of whack, you can change the quantity and hit 'save' and it
+                will manage to 'true up' and make your counts line up correctly.
+                */
+                $oldSeatCount = $license->license_seats_count;
 
-            return static::adjustSeatCount($license, $oldSeatCount, $newSeatCount);
-        });
+                return static::adjustSeatCount($license, $oldSeatCount, $newSeatCount);
+            }
+        );
     }
 
     public function prepareLimitChangeRule($parameters, $field)
@@ -163,7 +167,7 @@ class License extends Depreciable
      * Balance seat counts
      *
      * @author A. Gianotto <snipe@snipe.net>
-     * @since [v3.0]
+     * @since  [v3.0]
      * @return \Illuminate\Database\Eloquent\Relations\Relation
      */
     public static function adjustSeatCount($license, $oldSeats, $newSeats)
@@ -210,11 +214,15 @@ class License extends Depreciable
         }
         //Chunk and use DB transactions to prevent timeouts.
 
-        collect($licenseInsert)->chunk(1000)->each(function ($chunk) {
-            DB::transaction(function () use ($chunk) {
-                LicenseSeat::insert($chunk->toArray());
-            });
-        });
+        collect($licenseInsert)->chunk(1000)->each(
+            function ($chunk) {
+                DB::transaction(
+                    function () use ($chunk) {
+                        LicenseSeat::insert($chunk->toArray());
+                    }
+                );
+            }
+        );
 
         // On initial create, we shouldn't log the addition of seats.
         if ($license->id) {
@@ -235,7 +243,7 @@ class License extends Depreciable
      * Sets the attribute for whether or not the license is maintained
      *
      * @author A. Gianotto <snipe@snipe.net>
-     * @since [v1.0]
+     * @since  [v1.0]
      * @return mixed
      */
     public function setMaintainedAttribute($value)
@@ -247,7 +255,7 @@ class License extends Depreciable
      * Sets the reassignable attribute
      *
      * @author A. Gianotto <snipe@snipe.net>
-     * @since [v1.0]
+     * @since  [v1.0]
      * @return mixed
      */
     public function setReassignableAttribute($value)
@@ -259,7 +267,7 @@ class License extends Depreciable
      * Sets expiration date attribute
      *
      * @author A. Gianotto <snipe@snipe.net>
-     * @since [v1.0]
+     * @since  [v1.0]
      * @return mixed
      */
     public function setExpirationDateAttribute($value)
@@ -276,7 +284,7 @@ class License extends Depreciable
      * Sets termination date attribute
      *
      * @author A. Gianotto <snipe@snipe.net>
-     * @since [v2.0]
+     * @since  [v2.0]
      * @return mixed
      */
     public function setTerminationDateAttribute($value)
@@ -292,10 +300,11 @@ class License extends Depreciable
      * Sets free_seat_count attribute
      *
      * @author G. Martinez
-     * @since [v6.3]
+     * @since  [v6.3]
      * @return mixed
      */
-    public function getFreeSeatCountAttribute(){
+    public function getFreeSeatCountAttribute()
+    {
         return $this->attributes['free_seat_count'] = $this->remaincount();
     }
 
@@ -303,7 +312,7 @@ class License extends Depreciable
      * Establishes the license -> company relationship
      *
      * @author A. Gianotto <snipe@snipe.net>
-     * @since [v2.0]
+     * @since  [v2.0]
      * @return \Illuminate\Database\Eloquent\Relations\Relation
      */
     public function company()
@@ -315,7 +324,7 @@ class License extends Depreciable
      * Establishes the license -> category relationship
      *
      * @author A. Gianotto <snipe@snipe.net>
-     * @since [v4.4.0]
+     * @since  [v4.4.0]
      * @return \Illuminate\Database\Eloquent\Relations\Relation
      */
     public function category()
@@ -327,7 +336,7 @@ class License extends Depreciable
      * Establishes the license -> manufacturer relationship
      *
      * @author A. Gianotto <snipe@snipe.net>
-     * @since [v2.0]
+     * @since  [v2.0]
      * @return \Illuminate\Database\Eloquent\Relations\Relation
      */
     public function manufacturer()
@@ -339,7 +348,7 @@ class License extends Depreciable
      * Determine whether the user should be emailed on checkin/checkout
      *
      * @author A. Gianotto <snipe@snipe.net>
-     * @since [v2.0]
+     * @since  [v2.0]
      * @return bool
      */
     public function checkin_email()
@@ -354,7 +363,7 @@ class License extends Depreciable
      * Determine whether the user should be required to accept the license
      *
      * @author A. Gianotto <snipe@snipe.net>
-     * @since [v4.0]
+     * @since  [v4.0]
      * @return bool
      */
     public function requireAcceptance()
@@ -371,12 +380,12 @@ class License extends Depreciable
      * checks for a settings level EULA
      *
      * @author [A. Gianotto] [<snipe@snipe.net>]
-     * @since [v4.0]
+     * @since  [v4.0]
      * @return string | false
      */
     public function getEula()
     {
-        if ($this->category){
+        if ($this->category) {
             if ($this->category->eula_text) {
                 return Helper::parseEscapedMarkedown($this->category->eula_text);
             } elseif ($this->category->use_default_eula == '1') {
@@ -392,7 +401,7 @@ class License extends Depreciable
      * Establishes the license -> assigned user relationship
      *
      * @author A. Gianotto <snipe@snipe.net>
-     * @since [v2.0]
+     * @since  [v2.0]
      * @return \Illuminate\Database\Eloquent\Relations\Relation
      */
     public function assignedusers()
@@ -404,7 +413,7 @@ class License extends Depreciable
      * Establishes the license -> action logs relationship
      *
      * @author A. Gianotto <snipe@snipe.net>
-     * @since [v2.0]
+     * @since  [v2.0]
      * @return \Illuminate\Database\Eloquent\Relations\Relation
      */
     public function assetlog()
@@ -420,7 +429,7 @@ class License extends Depreciable
      * Establishes the license -> admin user relationship
      *
      * @author A. Gianotto <snipe@snipe.net>
-     * @since [v2.0]
+     * @since  [v2.0]
      * @return \Illuminate\Database\Eloquent\Relations\Relation
      */
     public function adminuser()
@@ -434,7 +443,7 @@ class License extends Depreciable
      * @todo this can probably be refactored at some point. We don't need counting methods.
      *
      * @author A. Gianotto <snipe@snipe.net>
-     * @since [v2.0]
+     * @since  [v2.0]
      * @return int
      */
     public static function assetcount()
@@ -450,7 +459,7 @@ class License extends Depreciable
      * @todo this can also probably be refactored at some point.
      *
      * @author A. Gianotto <snipe@snipe.net>
-     * @since [v2.0]
+     * @since  [v2.0]
      * @return \Illuminate\Database\Eloquent\Relations\Relation
      */
     public function totalSeatsByLicenseID()
@@ -467,7 +476,7 @@ class License extends Depreciable
      * Otherwise calling "count()" on each model results in n+1 sadness.
      *
      * @author A. Gianotto <snipe@snipe.net>
-     * @since [v2.0]
+     * @since  [v2.0]
      * @return \Illuminate\Database\Eloquent\Relations\Relation
      */
     public function licenseSeatsRelation()
@@ -479,7 +488,7 @@ class License extends Depreciable
      * Sets the license seat count attribute
      *
      * @author A. Gianotto <snipe@snipe.net>
-     * @since [v2.0]
+     * @since  [v2.0]
      * @return int
      */
     public function getLicenseSeatsCountAttribute()
@@ -495,7 +504,7 @@ class License extends Depreciable
      * Returns the number of total available seats across all licenses
      *
      * @author A. Gianotto <snipe@snipe.net>
-     * @since [v2.0]
+     * @since  [v2.0]
      * @return int
      */
     public static function availassetcount()
@@ -509,7 +518,7 @@ class License extends Depreciable
      * Returns the available seats remaining
      *
      * @author A. Gianotto <snipe@snipe.net>
-     * @since [v2.0]
+     * @since  [v2.0]
      * @return int
      */
 
@@ -517,7 +526,7 @@ class License extends Depreciable
      * Returns the number of total available seats for this license
      *
      * @author A. Gianotto <snipe@snipe.net>
-     * @since [v2.0]
+     * @since  [v2.0]
      * @return \Illuminate\Database\Eloquent\Relations\Relation
      */
     public function availCount()
@@ -532,7 +541,7 @@ class License extends Depreciable
      * Sets the available seats attribute
      *
      * @author A. Gianotto <snipe@snipe.net>
-     * @since [v3.0]
+     * @since  [v3.0]
      * @return mixed
      */
     public function getAvailSeatsCountAttribute()
@@ -548,22 +557,24 @@ class License extends Depreciable
      * Retuns the number of assigned seats for this asset
      *
      * @author A. Gianotto <snipe@snipe.net>
-     * @since [v3.0]
+     * @since  [v3.0]
      * @return \Illuminate\Database\Eloquent\Relations\Relation
      */
     public function assignedCount()
     {
-        return $this->licenseSeatsRelation()->where(function ($query) {
-            $query->whereNotNull('assigned_to')
-                ->orWhereNotNull('asset_id');
-        });
+        return $this->licenseSeatsRelation()->where(
+            function ($query) {
+                $query->whereNotNull('assigned_to')
+                    ->orWhereNotNull('asset_id');
+            }
+        );
     }
 
     /**
      * Sets the assigned seats attribute
      *
      * @author A. Gianotto <snipe@snipe.net>
-     * @since [v1.0]
+     * @since  [v1.0]
      * @return int
      */
     public function getAssignedSeatsCountAttribute()
@@ -579,7 +590,7 @@ class License extends Depreciable
      * Calculates the number of remaining seats
      *
      * @author A. Gianotto <snipe@snipe.net>
-     * @since [v1.0]
+     * @since  [v1.0]
      * @return int
      */
     public function remaincount()
@@ -595,7 +606,7 @@ class License extends Depreciable
      * Returns the total number of seats for this license
      *
      * @author A. Gianotto <snipe@snipe.net>
-     * @since [v1.0]
+     * @since  [v1.0]
      * @return int
      */
     public function totalcount()
@@ -611,7 +622,7 @@ class License extends Depreciable
      * Establishes the license -> seats relationship
      *
      * @author A. Gianotto <snipe@snipe.net>
-     * @since [v1.0]
+     * @since  [v1.0]
      * @return \Illuminate\Database\Eloquent\Relations\Relation
      */
     public function licenseseats()
@@ -623,7 +634,7 @@ class License extends Depreciable
      * Establishes the license -> supplier relationship
      *
      * @author A. Gianotto <snipe@snipe.net>
-     * @since [v1.0]
+     * @since  [v1.0]
      * @return \Illuminate\Database\Eloquent\Relations\Relation
      */
     public function supplier()
@@ -637,17 +648,19 @@ class License extends Depreciable
      * the API to populate next_seat
      *
      * @author A. Gianotto <snipe@snipe.net>
-     * @since [v3.0]
+     * @since  [v3.0]
      * @return mixed
      */
     public function freeSeat()
     {
         return  $this->licenseseats()
             ->whereNull('deleted_at')
-            ->where(function ($query) {
-                $query->whereNull('assigned_to')
-                    ->whereNull('asset_id');
-            })
+            ->where(
+                function ($query) {
+                    $query->whereNull('assigned_to')
+                        ->whereNull('asset_id');
+                }
+            )
             ->orderBy('id', 'asc')
             ->first();
     }
@@ -657,7 +670,7 @@ class License extends Depreciable
      * Establishes the license -> free seats relationship
      *
      * @author A. Gianotto <snipe@snipe.net>
-     * @since [v1.0]
+     * @since  [v1.0]
      * @return \Illuminate\Database\Eloquent\Relations\Relation
      */
     public function freeSeats()
@@ -671,7 +684,7 @@ class License extends Depreciable
      * @todo should refactor. I don't like get() in model methods
      *
      * @author A. Gianotto <snipe@snipe.net>
-     * @since [v1.0]
+     * @since  [v1.0]
      * @return \Illuminate\Database\Eloquent\Relations\Relation
      */
     public static function getExpiringLicenses($days = 60)
@@ -689,8 +702,8 @@ class License extends Depreciable
     /**
      * Query builder scope to order on manufacturer
      *
-     * @param  \Illuminate\Database\Query\Builder  $query  Query builder instance
-     * @param  string                              $order         Order
+     * @param \Illuminate\Database\Query\Builder $query Query builder instance
+     * @param string                             $order Order
      *
      * @return \Illuminate\Database\Query\Builder          Modified query builder
      */
@@ -703,8 +716,8 @@ class License extends Depreciable
     /**
      * Query builder scope to order on supplier
      *
-     * @param  \Illuminate\Database\Query\Builder  $query  Query builder instance
-     * @param  string                              $order         Order
+     * @param \Illuminate\Database\Query\Builder $query Query builder instance
+     * @param string                             $order Order
      *
      * @return \Illuminate\Database\Query\Builder          Modified query builder
      */
@@ -717,8 +730,8 @@ class License extends Depreciable
     /**
      * Query builder scope to order on company
      *
-     * @param  \Illuminate\Database\Query\Builder  $query  Query builder instance
-     * @param  text                              $order         Order
+     * @param \Illuminate\Database\Query\Builder $query Query builder instance
+     * @param text                               $order Order
      *
      * @return \Illuminate\Database\Query\Builder          Modified query builder
      */

app/Models/LicenseSeat.php

@@ -44,7 +44,7 @@ class LicenseSeat extends SnipeModel implements ICompanyableChild
      * Determine whether the user should be required to accept the license
      *
      * @author A. Gianotto <snipe@snipe.net>
-     * @since [v4.0]
+     * @since  [v4.0]
      * @return bool
      */
     public function requireAcceptance()
@@ -64,7 +64,7 @@ class LicenseSeat extends SnipeModel implements ICompanyableChild
      * Establishes the seat -> license relationship
      *
      * @author A. Gianotto <snipe@snipe.net>
-     * @since [v1.0]
+     * @since  [v1.0]
      * @return \Illuminate\Database\Eloquent\Relations\Relation
      */
     public function license()
@@ -76,7 +76,7 @@ class LicenseSeat extends SnipeModel implements ICompanyableChild
      * Establishes the seat -> assignee relationship
      *
      * @author A. Gianotto <snipe@snipe.net>
-     * @since [v1.0]
+     * @since  [v1.0]
      * @return \Illuminate\Database\Eloquent\Relations\Relation
      */
     public function user()
@@ -88,7 +88,7 @@ class LicenseSeat extends SnipeModel implements ICompanyableChild
      * Establishes the seat -> asset relationship
      *
      * @author A. Gianotto <snipe@snipe.net>
-     * @since [v4.0]
+     * @since  [v4.0]
      * @return \Illuminate\Database\Eloquent\Relations\Relation
      */
     public function asset()
@@ -101,7 +101,7 @@ class LicenseSeat extends SnipeModel implements ICompanyableChild
      * or asset its assigned to
      *
      * @author A. Gianotto <snipe@snipe.net>
-     * @since [v4.0]
+     * @since  [v4.0]
      * @return string
      */
     public function location()
@@ -118,8 +118,8 @@ class LicenseSeat extends SnipeModel implements ICompanyableChild
     /**
      * Query builder scope to order on department
      *
-     * @param  \Illuminate\Database\Query\Builder  $query  Query builder instance
-     * @param  text                              $order         Order
+     * @param \Illuminate\Database\Query\Builder $query Query builder instance
+     * @param text                               $order Order
      *
      * @return \Illuminate\Database\Query\Builder          Modified query builder
      */
@@ -130,4 +130,22 @@ class LicenseSeat extends SnipeModel implements ICompanyableChild
             ->whereNotNull('license_seats.assigned_to')
             ->orderBy('license_user_dept.name', $order);
     }
+
+
+    public function scopeByAssigned($query)
+    {
+
+        return $query->where(
+            function ($query) {
+                $query->whereNotNull('assigned_to')
+                    ->orWhere(
+                        function ($query) {
+                            $query->whereNotNull('asset_id');
+                        }
+                    );
+            }
+        );
+
+    }
+
 }

app/Models/Location.php

@@ -111,7 +111,7 @@ class Location extends SnipeModel
      * it can be deleted. It's tempting to load those here, but that increases the query load considerably.
      *
      * @author A. Gianotto <snipe@snipe.net>
-     * @since [v3.0]
+     * @since  [v3.0]
      * @return bool
      */
     public function isDeletable()
@@ -129,7 +129,7 @@ class Location extends SnipeModel
      * Establishes the user -> location relationship
      *
      * @author A. Gianotto <snipe@snipe.net>
-     * @since [v3.0]
+     * @since  [v3.0]
      * @return \Illuminate\Database\Eloquent\Relations\Relation
      */
     public function users()
@@ -152,17 +152,19 @@ class Location extends SnipeModel
      * Find assets with this location as their location_id
      *
      * @author A. Gianotto <snipe@snipe.net>
-     * @since [v3.0]
+     * @since  [v3.0]
      * @return \Illuminate\Database\Eloquent\Relations\Relation
      */
     public function assets()
     {
         return $this->hasMany(\App\Models\Asset::class, 'location_id')
-            ->whereHas('assetstatus', function ($query) {
-                $query->where('status_labels.deployable', '=', 1)
+            ->whereHas(
+                'assetstatus', function ($query) {
+                    $query->where('status_labels.deployable', '=', 1)
                         ->orWhere('status_labels.pending', '=', 1)
                         ->orWhere('status_labels.archived', '=', 0);
-            });
+                }
+            );
     }
 
 
@@ -170,7 +172,7 @@ class Location extends SnipeModel
      * Establishes the  asset -> rtd_location relationship
      *
      * @author A. Gianotto <snipe@snipe.net>
-     * @since [v3.0]
+     * @since  [v3.0]
      * @return \Illuminate\Database\Eloquent\Relations\Relation
      */
     public function rtd_assets()
@@ -190,7 +192,7 @@ class Location extends SnipeModel
      * Establishes the consumable -> location relationship
      *
      * @author A. Gianotto <snipe@snipe.net>
-     * @since [v3.0]
+     * @since  [v3.0]
      * @return \Illuminate\Database\Eloquent\Relations\Relation
      */
     public function consumables()
@@ -202,7 +204,7 @@ class Location extends SnipeModel
      * Establishes the component -> location relationship
      *
      * @author A. Gianotto <snipe@snipe.net>
-     * @since [v3.0]
+     * @since  [v3.0]
      * @return \Illuminate\Database\Eloquent\Relations\Relation
      */
     public function components()
@@ -214,7 +216,7 @@ class Location extends SnipeModel
      * Establishes the component -> accessory relationship
      *
      * @author A. Gianotto <snipe@snipe.net>
-     * @since [v3.0]
+     * @since  [v3.0]
      * @return \Illuminate\Database\Eloquent\Relations\Relation
      */
     public function accessories()
@@ -226,7 +228,7 @@ class Location extends SnipeModel
      * Find the parent of a location
      *
      * @author A. Gianotto <snipe@snipe.net>
-     * @since [v2.0]
+     * @since  [v2.0]
      * @return \Illuminate\Database\Eloquent\Relations\Relation
      */
     public function parent()
@@ -236,12 +238,12 @@ class Location extends SnipeModel
     }
 
     /**
-    * Establishes the locations -> company relationship
-    *
-    * @author [T. Regnery] [<tobias.regnery@gmail.com>]
-    * @since [v7.0]
-    * @return \Illuminate\Database\Eloquent\Relations\Relation
-    */
+     * Establishes the locations -> company relationship
+     *
+     * @author [T. Regnery] [<tobias.regnery@gmail.com>]
+     * @since  [v7.0]
+     * @return \Illuminate\Database\Eloquent\Relations\Relation
+     */
     public function company()
     {
         return $this->belongsTo(\App\Models\Company::class, 'company_id');
@@ -251,7 +253,7 @@ class Location extends SnipeModel
      * Find the manager of a location
      *
      * @author A. Gianotto <snipe@snipe.net>
-     * @since [v2.0]
+     * @since  [v2.0]
      * @return \Illuminate\Database\Eloquent\Relations\Relation
      */
     public function manager()
@@ -264,7 +266,7 @@ class Location extends SnipeModel
      * Find children of a location
      *
      * @author A. Gianotto <snipe@snipe.net>
-     * @since [v2.0]
+     * @since  [v2.0]
      * @return \Illuminate\Database\Eloquent\Relations\Relation
      */
     public function children()
@@ -277,7 +279,7 @@ class Location extends SnipeModel
      * Establishes the asset -> location assignment relationship
      *
      * @author A. Gianotto <snipe@snipe.net>
-     * @since [v3.0]
+     * @since  [v3.0]
      * @return \Illuminate\Database\Eloquent\Relations\Relation
      */
     public function assignedAssets()
@@ -289,7 +291,7 @@ class Location extends SnipeModel
      * Establishes the accessory -> location assignment relationship
      *
      * @author A. Gianotto <snipe@snipe.net>
-     * @since [v3.0]
+     * @since  [v3.0]
      * @return \Illuminate\Database\Eloquent\Relations\Relation
      */
     public function assignedAccessories()
@@ -306,8 +308,8 @@ class Location extends SnipeModel
     /**
      * Query builder scope to order on parent
      *
-     * @param  Illuminate\Database\Query\Builder  $query  Query builder instance
-     * @param  text                              $order       Order
+     * @param Illuminate\Database\Query\Builder $query Query builder instance
+     * @param text                              $order Order
      *
      * @return Illuminate\Database\Query\Builder          Modified query builder
      */
@@ -335,8 +337,8 @@ class Location extends SnipeModel
     /**
      * Query builder scope to order on parent
      *
-     * @param  Illuminate\Database\Query\Builder  $query  Query builder instance
-     * @param  text                              $order       Order
+     * @param Illuminate\Database\Query\Builder $query Query builder instance
+     * @param text                              $order Order
      *
      * @return Illuminate\Database\Query\Builder          Modified query builder
      */
@@ -349,8 +351,8 @@ class Location extends SnipeModel
     /**
      * Query builder scope to order on manager name
      *
-     * @param  \Illuminate\Database\Query\Builder  $query  Query builder instance
-     * @param  text                              $order       Order
+     * @param \Illuminate\Database\Query\Builder $query Query builder instance
+     * @param text                               $order Order
      *
      * @return \Illuminate\Database\Query\Builder          Modified query builder
      */
@@ -359,14 +361,14 @@ class Location extends SnipeModel
         return $query->leftJoin('users as location_user', 'locations.manager_id', '=', 'location_user.id')->orderBy('location_user.first_name', $order)->orderBy('location_user.last_name', $order);
     }
 
-   /**
-    * Query builder scope to order on company
-    *
-    * @param  \Illuminate\Database\Query\Builder  $query  Query builder instance
-    * @param  text                              $order       Order
-    *
-    * @return \Illuminate\Database\Query\Builder          Modified query builder
-    */
+    /**
+     * Query builder scope to order on company
+     *
+     * @param \Illuminate\Database\Query\Builder $query Query builder instance
+     * @param text                               $order Order
+     *
+     * @return \Illuminate\Database\Query\Builder          Modified query builder
+     */
     public function scopeOrderCompany($query, $order)
     {
         return $query->leftJoin('companies as company_sort', 'locations.company_id', '=', 'company_sort.id')->orderBy('company_sort.name', $order);

app/Models/Loggable.php

@@ -15,8 +15,8 @@ trait Loggable
     public ?bool $imported = false;
 
     /**
-     * @author  Daniel Meltzer <dmeltzer.devel@gmail.com>
-     * @since [v3.4]
+     * @author Daniel Meltzer <dmeltzer.devel@gmail.com>
+     * @since  [v3.4]
      * @return \App\Models\Actionlog
      */
     public function log()
@@ -30,8 +30,8 @@ trait Loggable
     }
 
     /**
-     * @author  Daniel Meltzer <dmeltzer.devel@gmail.com>
-     * @since [v3.4]
+     * @author Daniel Meltzer <dmeltzer.devel@gmail.com>
+     * @since  [v3.4]
      * @return \App\Models\Actionlog
      */
     public function logCheckout($note, $target, $action_date = null, $originalValues = [])
@@ -108,7 +108,7 @@ trait Loggable
             // NOTE - if the attribute exists in $originalValues, but *not* in ->getAttributes(), it isn't added to $changed
         }
 
-        if (!empty($changed)){
+        if (!empty($changed)) {
             $log->log_meta = json_encode($changed);
         }
 
@@ -135,8 +135,8 @@ trait Loggable
     }
 
     /**
-     * @author  Daniel Meltzer <dmeltzer.devel@gmail.com>
-     * @since [v3.4]
+     * @author Daniel Meltzer <dmeltzer.devel@gmail.com>
+     * @since  [v3.4]
      * @return \App\Models\Actionlog
      */
     public function logCheckin($target, $note, $action_date = null, $originalValues = [])
@@ -145,7 +145,7 @@ trait Loggable
 
         $fields_array = [];
 
-        if($target != null){
+        if($target != null) {
             $log->target_type = get_class($target);
             $log->target_id = $target->id;
 
@@ -205,7 +205,7 @@ trait Loggable
             }
         }
 
-        if (!empty($changed)){
+        if (!empty($changed)) {
             $log->log_meta = json_encode($changed);
         }
 
@@ -215,8 +215,8 @@ trait Loggable
     }
 
     /**
-     * @author  A. Gianotto <snipe@snipe.net>
-     * @since [v4.0]
+     * @author A. Gianotto <snipe@snipe.net>
+     * @since  [v4.0]
      * @return \App\Models\Actionlog
      */
     public function logAudit($note, $location_id, $filename = null, $originalValues = [])
@@ -249,7 +249,7 @@ trait Loggable
             }
         }
 
-        if (!empty($changed)){
+        if (!empty($changed)) {
             $log->log_meta = json_encode($changed);
         }
 
@@ -276,7 +276,7 @@ trait Loggable
             'location' => ($location) ? $location->name : '',
             'note' => $note,
         ];
-        if(Setting::getSettings()->webhook_selected === 'microsoft' && Str::contains(Setting::getSettings()->webhook_endpoint, 'workflows')){
+        if(Setting::getSettings()->webhook_selected === 'microsoft' && Str::contains(Setting::getSettings()->webhook_endpoint, 'workflows')) {
             $message = AuditNotification::toMicrosoftTeams($params);
             $notification = new TeamsNotification(Setting::getSettings()->webhook_endpoint);
             $notification->success()->sendMessage($message[0], $message[1]);
@@ -289,8 +289,8 @@ trait Loggable
     }
 
     /**
-     * @author  Daniel Meltzer <dmeltzer.devel@gmail.com>
-     * @since [v3.5]
+     * @author Daniel Meltzer <dmeltzer.devel@gmail.com>
+     * @since  [v3.5]
      * @return \App\Models\Actionlog
      */
     public function logCreate($note = null)
@@ -318,8 +318,8 @@ trait Loggable
     }
 
     /**
-     * @author  Daniel Meltzer <dmeltzer.devel@gmail.com>
-     * @since [v3.4]
+     * @author Daniel Meltzer <dmeltzer.devel@gmail.com>
+     * @since  [v3.4]
      * @return \App\Models\Actionlog
      */
     public function logUpload($filename, $note)
@@ -350,7 +350,7 @@ trait Loggable
      * Returns the latest acceptance ActionLog that contains a signature
      * from $user or null if there is none
      *
-     * @param User $user
+     * @param  User $user
      * @return null|Actionlog
      **/
     public function getLatestSignedAcceptance(User $user)

app/Models/PredefinedKit.php

@@ -11,8 +11,8 @@ use Watson\Validating\ValidatingTrait;
 /**
  * Model for predefined kits.
  *
- * @author [D. Minaev.] [<dmitriy.minaev.v@gmail.com>]
- * @version    v1.0
+ * @author  [D. Minaev.] [<dmitriy.minaev.v@gmail.com>]
+ * @version v1.0
  */
 class PredefinedKit extends SnipeModel
 {
@@ -39,8 +39,9 @@ class PredefinedKit extends SnipeModel
     /**
      * this rules use in edit an attached asset model form
      * see PredefinedKit::_makeRuleHelper function for details
-     * @param int $model_id
-     * @param bool $new = true if append a new element to kit
+     *
+     * @param int  $model_id
+     * @param bool $new      = true if append a new element to kit
      */
     public function makeModelRules($model_id, $new = false)
     {
@@ -50,8 +51,9 @@ class PredefinedKit extends SnipeModel
     /**
      * this rules use in edit an attached license form
      * see PredefinedKit::_makeRuleHelper function for details
-     * @param int $license_id
-     * @param bool $new = true if append a new element to kit
+     *
+     * @param int  $license_id
+     * @param bool $new        = true if append a new element to kit
      */
     public function makeLicenseRules($license_id, $new = false)
     {
@@ -61,8 +63,9 @@ class PredefinedKit extends SnipeModel
     /**
      * this rules use in edit an attached accessory form
      * see PredefinedKit::_makeRuleHelper function for details
-     * @param int $accessoriy_id
-     * @param bool $new = true if append a new element to kit
+     *
+     * @param int  $accessoriy_id
+     * @param bool $new           = true if append a new element to kit
      */
     public function makeAccessoryRules($accessory_id, $new = false)
     {
@@ -72,8 +75,9 @@ class PredefinedKit extends SnipeModel
     /**
      * this rules use in edit an attached consumable form
      * see PredefinedKit::_makeRuleHelper function for details
-     * @param int $consumable_id
-     * @param bool $new = true if append a new element to kit
+     *
+     * @param int  $consumable_id
+     * @param bool $new           = true if append a new element to kit
      */
     public function makeConsumableRules($consumable_id, $new = false)
     {
@@ -86,11 +90,12 @@ class PredefinedKit extends SnipeModel
      * uniqueness of the record in table for this kit
      * existence of record in table
      * and simple types check
-     * @param string $table element table name
-     * @param string $pivot_table kit+element table name
-     * @param string $pivot_elem_key element key name inside pivot table
-     * @param int $element_id
-     * @param bool $new = true if append a new element to kit
+     *
+     * @param  string $table          element table name
+     * @param  string $pivot_table    kit+element table name
+     * @param  string $pivot_elem_key element key name inside pivot table
+     * @param  int    $element_id
+     * @param  bool   $new            = true if append a new element to kit
      * @return array
      */
     protected function _makeRuleHelper($table, $pivot_table, $pivot_elem_key, $element_id, $new)
@@ -144,6 +149,7 @@ class PredefinedKit extends SnipeModel
 
     /**
      * Establishes the kits -> models relationship
+     *
      * @return \Illuminate\Database\Eloquent\Relations\Relation
      */
     public function models()
@@ -158,6 +164,7 @@ class PredefinedKit extends SnipeModel
 
     /**
      * Establishes the kits -> licenses relationship
+     *
      * @return \Illuminate\Database\Eloquent\Relations\Relation
      */
     public function licenses()
@@ -167,6 +174,7 @@ class PredefinedKit extends SnipeModel
 
     /**
      * Establishes the kits -> licenses relationship
+     *
      * @return \Illuminate\Database\Eloquent\Relations\Relation
      */
     public function consumables()
@@ -176,6 +184,7 @@ class PredefinedKit extends SnipeModel
 
     /**
      * Establishes the kits -> licenses relationship
+     *
      * @return \Illuminate\Database\Eloquent\Relations\Relation
      */
     public function accessories()

app/Models/Recipients/AdminRecipient.php

@@ -14,7 +14,8 @@ class AdminRecipient extends Recipient
         $this->email = trim($settings->admin_cc_email);
     }
     
-    public function getEmail(){
+    public function getEmail()
+    {
         return $this->email;
     }
 }

app/Models/ReportTemplate.php

@@ -39,16 +39,17 @@ class ReportTemplate extends Model
     protected static function booted()
     {
         // Scope to current user
-        static::addGlobalScope('current_user', function (Builder $builder) {
-            if (auth()->check()) {
-                $builder->where('created_by', auth()->id());
+        static::addGlobalScope(
+            'current_user', function (Builder $builder) {
+                if (auth()->check()) {
+                    $builder->where('created_by', auth()->id());
+                }
             }
-        });
+        );
     }
 
     /**
      * Establishes the report template -> creator relationship.
-     *
      */
     public function creator(): BelongsTo
     {
@@ -60,7 +61,6 @@ class ReportTemplate extends Model
      *
      * @param string $fieldName
      * @param string $fallbackValue The value to return if the report template is not saved yet.
-     *
      */
     public function checkmarkValue(string $fieldName, string $fallbackValue = '1'): string
     {
@@ -83,9 +83,8 @@ class ReportTemplate extends Model
      * Get the value of a radio field for the given field name.
      *
      * @param string $fieldName
-     * @param string $value The value to check against.
-     * @param bool $isDefault Whether the radio input being checked is the default.
-     *
+     * @param string $value     The value to check against.
+     * @param bool   $isDefault Whether the radio input being checked is the default.
      */
     public function radioValue(string $fieldName, string $value, bool $isDefault = false): bool
     {
@@ -109,11 +108,10 @@ class ReportTemplate extends Model
     /**
      * Get the value of a select field for the given field name.
      *
-     * @param string $fieldName
-     * @param string|null $model The Eloquent model to check against.
+     * @param string      $fieldName
+     * @param string|null $model     The Eloquent model to check against.
      *
      * @return mixed|null
-     *
      */
     public function selectValue(string $fieldName, string $model = null)
     {
@@ -146,11 +144,10 @@ class ReportTemplate extends Model
     /**
      * Get the values of a multi-select field for the given field name.
      *
-     * @param string $fieldName
-     * @param string|null $model The Eloquent model to check against.
+     * @param string      $fieldName
+     * @param string|null $model     The Eloquent model to check against.
      *
      * @return iterable
-     *
      */
     public function selectValues(string $fieldName, string $model = null): iterable
     {
@@ -178,8 +175,8 @@ class ReportTemplate extends Model
     /**
      * Get the value of a text field for the given field name.
      *
-     * @param  string  $fieldName
-     * @param  string|null  $fallbackValue
+     * @param string      $fieldName
+     * @param string|null $fallbackValue
      *
      * @return string
      */

app/Models/Requestable.php

@@ -21,9 +21,11 @@ trait Requestable
 
     public function scopeRequestedBy($query, User $user)
     {
-        return $query->whereHas('requests', function ($query) use ($user) {
-            $query->where('user_id', $user->id);
-        });
+        return $query->whereHas(
+            'requests', function ($query) use ($user) {
+                $query->where('user_id', $user->id);
+            }
+        );
     }
 
     public function request($qty = 1)
@@ -40,7 +42,7 @@ trait Requestable
 
     public function cancelRequest($user_id = null)
     {
-        if (!$user_id){
+        if (!$user_id) {
             $user_id = auth()->id();
         }
 

app/Models/SCIMUser.php

@@ -8,7 +8,8 @@ class SCIMUser extends User
 
     protected $throwValidationExceptions = true; // we want model-level validation to fully THROW, not just return false
 
-    public function __construct(array $attributes = []) {
+    public function __construct(array $attributes = [])
+    {
         $attributes['password'] = $this->noPassword();
         parent::__construct($attributes);
     }