Fixed #19006 - update null location

.all-contributorsrc

@@ -4271,15 +4271,6 @@
       "contributions": [
         "code"
       ]
-    },
-    {
-      "login": "CybotTM",
-      "name": "Sebastian Mendel",
-      "avatar_url": "https://avatars.githubusercontent.com/u/326348?v=4",
-      "profile": "https://github.com/CybotTM",
-      "contributions": [
-        "code"
-      ]
     }
   ]
 }

.env.dev.docker

@@ -113,7 +113,7 @@ ENABLE_HSTS=false
 # --------------------------------------------
 CACHE_DRIVER=file
 SESSION_DRIVER=file
-QUEUE_CONNECTION=sync
+QUEUE_DRIVER=sync
 CACHE_PREFIX=snipeit
 
 # --------------------------------------------

.env.docker

@@ -37,7 +37,6 @@ MYSQL_ROOT_PASSWORD=changeme1234
 DB_PREFIX=null
 DB_DUMP_PATH='/usr/bin'
 DB_DUMP_SKIP_SSL=true
-DB_DUMP_SINGLE_TRANSACTION=false
 DB_CHARSET=utf8mb4
 DB_COLLATION=utf8mb4_unicode_ci
 
@@ -121,7 +120,7 @@ ENABLE_HSTS=false
 # --------------------------------------------
 CACHE_DRIVER=file
 SESSION_DRIVER=file
-QUEUE_CONNECTION=sync
+QUEUE_DRIVER=sync
 CACHE_PREFIX=snipeit
 
 # --------------------------------------------

.env.dusk.example

@@ -72,7 +72,7 @@ CORS_ALLOWED_ORIGINS="*"
 # --------------------------------------------
 CACHE_DRIVER=file
 SESSION_DRIVER=file
-QUEUE_CONNECTION=sync
+QUEUE_DRIVER=sync
 
 # --------------------------------------------
 # OPTIONAL: LOGIN THROTTLING

.env.example

@@ -32,7 +32,6 @@ DB_PASSWORD=null
 DB_PREFIX=null
 DB_DUMP_PATH='/usr/bin'
 DB_DUMP_SKIP_SSL=false
-DB_DUMP_SINGLE_TRANSACTION=false
 DB_CHARSET=utf8mb4
 DB_COLLATION=utf8mb4_unicode_ci
 DB_SANITIZE_BY_DEFAULT=false
@@ -134,7 +133,7 @@ BS_TABLE_DEEPLINK=true
 APP_TRUSTED_PROXIES=192.168.1.1,10.0.0.1
 ALLOW_IFRAMING=false
 REFERRER_POLICY=same-origin
-ENABLE_CSP=true
+ENABLE_CSP=false
 ADDITIONAL_CSP_URLS=null
 CORS_ALLOWED_ORIGINS=null
 ENABLE_HSTS=false
@@ -143,7 +142,7 @@ ENABLE_HSTS=false
 # OPTIONAL: CACHE SETTINGS
 # --------------------------------------------
 CACHE_DRIVER=file
-QUEUE_CONNECTION=sync
+QUEUE_DRIVER=sync
 CACHE_PREFIX=snipeit
 
 # --------------------------------------------
@@ -211,7 +210,6 @@ LOGIN_AUTOCOMPLETE=false
 RESET_PASSWORD_LINK_EXPIRES=15
 PASSWORD_CONFIRM_TIMEOUT=10800
 PASSWORD_RESET_MAX_ATTEMPTS_PER_MIN=50
-TWO_FACTOR_MAX_ATTEMPTS_PER_MIN=5
 INVITE_PASSWORD_LINK_EXPIRES=1500
 
 # --------------------------------------------

.github/workflows/ethicalcheck.yml

@@ -63,7 +63,7 @@ jobs:
           sarif-result-file: "ethicalcheck-results.sarif"
 
        - name: Upload sarif file to repository
-         uses: github/codeql-action/upload-sarif@v4
+         uses: github/codeql-action/upload-sarif@v3
          with:
           sarif_file: ./ethicalcheck-results.sarif
 

CONTRIBUTORS.md

@@ -69,7 +69,7 @@ Thanks goes to all of these wonderful people ([emoji key](https://github.com/ken
 | [<img src="https://avatars.githubusercontent.com/u/10965027?v=4" width="110px;"/><br /><sub>Ellie</sub>](https://leafedfox.xyz/)<br />[💻](https://github.com/snipe/snipe-it/commits?author=LeafedFox "Code") | [<img src="https://avatars.githubusercontent.com/u/20960555?v=4" width="110px;"/><br /><sub>GA Stamper</sub>](https://github.com/gastamper)<br />[💻](https://github.com/snipe/snipe-it/commits?author=gastamper "Code") | [<img src="https://avatars.githubusercontent.com/u/206553556?v=4" width="110px;"/><br /><sub>Guillaume Lefranc</sub>](https://github.com/gl-pup)<br />[💻](https://github.com/snipe/snipe-it/commits?author=gl-pup "Code") | [<img src="https://avatars.githubusercontent.com/u/733892?v=4" width="110px;"/><br /><sub>Hajo Möller</sub>](https://github.com/dasjoe)<br />[💻](https://github.com/snipe/snipe-it/commits?author=dasjoe "Code") | [<img src="https://avatars.githubusercontent.com/u/3420063?v=4" width="110px;"/><br /><sub>Istvan Basa</sub>](https://github.com/pottom)<br />[💻](https://github.com/snipe/snipe-it/commits?author=pottom "Code") | [<img src="https://avatars.githubusercontent.com/u/810824?v=4" width="110px;"/><br /><sub>JJ Asghar</sub>](https://jjasghar.github.io/)<br />[💻](https://github.com/snipe/snipe-it/commits?author=jjasghar "Code") | [<img src="https://avatars.githubusercontent.com/u/40404495?v=4" width="110px;"/><br /><sub>James E. Msenga</sub>](https://github.com/JemCdo)<br />[💻](https://github.com/snipe/snipe-it/commits?author=JemCdo "Code") |
 | [<img src="https://avatars.githubusercontent.com/u/6865786?v=4" width="110px;"/><br /><sub>Jan Felix Wiebe</sub>](https://github.com/jfwiebe)<br />[💻](https://github.com/snipe/snipe-it/commits?author=jfwiebe "Code") | [<img src="https://avatars.githubusercontent.com/u/43412008?v=4" width="110px;"/><br /><sub>Jo Drexl</sub>](https://www.nfon.com/)<br />[💻](https://github.com/snipe/snipe-it/commits?author=drexljo "Code") | [<img src="https://avatars.githubusercontent.com/u/4807843?v=4" width="110px;"/><br /><sub>Austin Sasko</sub>](https://github.com/austinsasko)<br />[💻](https://github.com/snipe/snipe-it/commits?author=austinsasko "Code") | [<img src="https://avatars.githubusercontent.com/u/4875039?v=4" width="110px;"/><br /><sub>Jasson</sub>](http://jassoncordones.github.io)<br />[💻](https://github.com/snipe/snipe-it/commits?author=JassonCordones "Code") | [<img src="https://avatars.githubusercontent.com/u/76069640?v=4" width="110px;"/><br /><sub>Okean</sub>](https://github.com/Tinyblargon)<br />[💻](https://github.com/snipe/snipe-it/commits?author=Tinyblargon "Code") | [<img src="https://avatars.githubusercontent.com/u/6515064?v=4" width="110px;"/><br /><sub>Alejandro Medrano</sub>](https://www.lst.tfo.upm.es/alejandro-medrano/)<br />[💻](https://github.com/snipe/snipe-it/commits?author=amedranogil "Code") | [<img src="https://avatars.githubusercontent.com/u/58696401?v=4" width="110px;"/><br /><sub>Lukas Kraic</sub>](https://github.com/lukaskraic)<br />[💻](https://github.com/snipe/snipe-it/commits?author=lukaskraic "Code") |
 | [<img src="https://avatars.githubusercontent.com/u/1571724?v=4" width="110px;"/><br /><sub>Герхард PICCORO Lenz McKAY </sub>](https://github-readme-stats.vercel.app/api?username=mckaygerhard)<br />[💻](https://github.com/snipe/snipe-it/commits?author=mckaygerhard "Code") | [<img src="https://avatars.githubusercontent.com/u/15015119?v=4" width="110px;"/><br /><sub>Johannes Pollitt</sub>](https://github.com/FlorestanII)<br />[💻](https://github.com/snipe/snipe-it/commits?author=FlorestanII "Code") | [<img src="https://avatars.githubusercontent.com/u/14185442?v=4" width="110px;"/><br /><sub>Michael Strobel</sub>](https://strobelm.de)<br />[💻](https://github.com/snipe/snipe-it/commits?author=strobelm "Code") | [<img src="https://avatars.githubusercontent.com/u/634790?v=4" width="110px;"/><br /><sub>Nicky West</sub>](http://nickwest.me)<br />[💻](https://github.com/snipe/snipe-it/commits?author=nickwest "Code") | [<img src="https://avatars.githubusercontent.com/u/1347327?v=4" width="110px;"/><br /><sub>akaspeh1</sub>](https://github.com/akaspeh1)<br />[💻](https://github.com/snipe/snipe-it/commits?author=akaspeh1 "Code") | [<img src="https://avatars.githubusercontent.com/u/2880129?v=4" width="110px;"/><br /><sub>Sebastian Marsching</sub>](http://sebastian.marsching.com/)<br />[💻](https://github.com/snipe/snipe-it/commits?author=smarsching "Code") | [<img src="https://avatars.githubusercontent.com/u/40658372?v=4" width="110px;"/><br /><sub>Mo</sub>](https://github.com/mohammad-ahmadi1)<br />[💻](https://github.com/snipe/snipe-it/commits?author=mohammad-ahmadi1 "Code") |
-| [<img src="https://avatars.githubusercontent.com/u/20994684?v=4" width="110px;"/><br /><sub>Owen V. Hayes</sub>](https://github.com/MarvelousAnything)<br />[💻](https://github.com/snipe/snipe-it/commits?author=MarvelousAnything "Code") | [<img src="https://avatars.githubusercontent.com/u/75509373?v=4" width="110px;"/><br /><sub>Peter Gallwas</sub>](https://www.husky.nz)<br />[💻](https://github.com/snipe/snipe-it/commits?author=Husky-Devel "Code") | [<img src="https://avatars.githubusercontent.com/u/326348?v=4" width="110px;"/><br /><sub>Sebastian Mendel</sub>](https://github.com/CybotTM)<br />[💻](https://github.com/snipe/snipe-it/commits?author=CybotTM "Code") |
+| [<img src="https://avatars.githubusercontent.com/u/20994684?v=4" width="110px;"/><br /><sub>Owen V. Hayes</sub>](https://github.com/MarvelousAnything)<br />[💻](https://github.com/snipe/snipe-it/commits?author=MarvelousAnything "Code") | [<img src="https://avatars.githubusercontent.com/u/75509373?v=4" width="110px;"/><br /><sub>Peter Gallwas</sub>](https://www.husky.nz)<br />[💻](https://github.com/snipe/snipe-it/commits?author=Husky-Devel "Code") |
 <!-- ALL-CONTRIBUTORS-LIST:END -->
 
 This project follows the [all-contributors](https://github.com/kentcdodds/all-contributors) specification. Contributions of any kind welcome!

Dockerfile.fpm-alpine

@@ -56,7 +56,6 @@ COPY --from=mlocati/php-extension-installer:2.1.15 /usr/bin/install-php-extensio
 RUN set -eux; \
     install-php-extensions \
         bcmath \
-        exif \
         gd \
         ldap \
         mysqli \

README.md

@@ -98,7 +98,6 @@ Since the release of the JSON REST API, several third-party developers have been
 - [InQRy (archived)](https://github.com/Microsoft/InQRy) by [@Microsoft](https://github.com/Microsoft)
 - [Marksman (archived)](https://github.com/Scope-IT/marksman) - A Windows agent for Snipe-IT
 - [Python Module (archived)](https://github.com/jbloomer/SnipeIT-PythonAPI) by [@jbloomer](https://github.com/jbloomer)
-[IT-Tools](https://github.com/chrisnox/Snipeit-it-tools) by @chrisnox - Browser bookmarklets for PDF handover/return protocols, digital signatures, label printing (Zebra ZD410), AirWatch MDM sync and Lansweeper CSV import.
 
 We also have a handful of [Google Apps scripts](https://github.com/grokability/google-apps-scripts-for-snipe-it) to help with various tasks.
 

app/Console/Commands/BulkDelete.php

@@ -1,989 +0,0 @@
-<?php
-
-namespace App\Console\Commands;
-
-use App\Events\CheckoutableCheckedIn;
-use App\Mail\BulkDeleteReportMail;
-use App\Models\Accessory;
-use App\Models\AccessoryCheckout;
-use App\Models\Actionlog;
-use App\Models\Asset;
-use App\Models\CheckoutAcceptance;
-use App\Models\Company;
-use App\Models\Component;
-use App\Models\Consumable;
-use App\Models\License;
-use App\Models\LicenseSeat;
-use App\Models\User;
-use Illuminate\Console\Command;
-use Illuminate\Database\Eloquent\Builder;
-use Illuminate\Support\Facades\DB;
-use Illuminate\Support\Facades\Log;
-use Illuminate\Support\Facades\Mail;
-use Illuminate\Support\Facades\Storage;
-use Symfony\Component\Console\Helper\ProgressBar;
-
-use function Laravel\Prompts\confirm;
-use function Laravel\Prompts\error;
-use function Laravel\Prompts\info;
-use function Laravel\Prompts\multisearch;
-use function Laravel\Prompts\multiselect;
-use function Laravel\Prompts\search;
-use function Laravel\Prompts\select;
-use function Laravel\Prompts\warning;
-
-class BulkDelete extends Command
-{
-    protected $signature = 'snipeit:checkin-delete-items';
-
-    protected $description = 'Interactively check in and/or delete items by company and type';
-
-    private const CHECKIN_NOTE = 'Checked in via bulk CLI operation';
-
-    private array $reportLines = [];
-
-    public function handle(): int
-    {
-        // Step 1: Dry run?
-        $dryRun = confirm(
-            label: 'Is this a dry run?',
-            default: true,
-            yes: 'Yes — preview only, no changes will be made',
-            no: 'No — LIVE RUN, changes WILL be made',
-        );
-
-        // Step 2: Who are you?
-        $adminId = search(
-            label: 'Who are you? Search by username, first or last name.',
-            placeholder: 'Type to search users...',
-            options: function (string $value): array {
-                if (strlen($value) < 1) {
-                    return [];
-                }
-
-                return User::where('activated', 1)
-                    ->whereNull('deleted_at')
-                    ->onlySuperAdmins()
-                    ->where(function ($query) use ($value) {
-                        $query->where('username', 'like', "%{$value}%")
-                            ->orWhere('first_name', 'like', "%{$value}%")
-                            ->orWhere('last_name', 'like', "%{$value}%")
-                            ->orWhereRaw("CONCAT(first_name, ' ', last_name) LIKE ?", ["%{$value}%"]);
-                    })
-                    ->get()
-                    ->mapWithKeys(fn (User $u) => [$u->id => "{$u->first_name} {$u->last_name} ({$u->username})"])
-                    ->toArray();
-            },
-            validate: fn (mixed $value) => ! $value ? 'A valid active user is required.' : null,
-        );
-
-        /** @var User $admin */
-        $admin = User::findOrFail((int) $adminId);
-
-        // Step 3: Which companies?
-        if (! Company::exists()) {
-            error('No companies found. Please create at least one company before using this command.');
-
-            return 1;
-        }
-
-        $selectedCompanyKeys = multisearch(
-            label: 'Which companies would you like to check in and delete items for?',
-            placeholder: 'Type to search companies...',
-            options: function (string $value): array {
-                $results = [];
-
-                if ($value === '' || str_contains('(no company / unassigned)', strtolower($value))) {
-                    $results['__null__'] = '(No Company / Unassigned)';
-                }
-
-                $query = Company::orderBy('name');
-                if ($value !== '') {
-                    $query->where('name', 'like', "%{$value}%");
-                }
-
-                $query->get()->each(function (Company $c) use (&$results) {
-                    $results[$c->id] = "{$c->name} (ID: {$c->id})";
-                });
-
-                return $results;
-            },
-            scroll: 10,
-            required: 'Please select at least one company.',
-            hint: 'If you\'re searching on several differently named companies, use the up-arrow to go back to the search box to search again. ',
-        );
-
-        $includeNullCompany = in_array('__null__', $selectedCompanyKeys);
-        $selectedCompanyIds = array_values(array_filter(
-            $selectedCompanyKeys,
-            fn ($k) => $k !== '__null__'
-        ));
-
-        $companyNamesById = Company::whereIn('id', $selectedCompanyIds)->pluck('name', 'id')->toArray();
-        $selectedCompanyNames = array_map(
-            fn ($id) => $id === '__null__' ? '(No Company)' : ($companyNamesById[$id] ?? "(ID: {$id})"),
-            $selectedCompanyKeys
-        );
-
-        // Step 4: Which item types?
-        $rawTypeSelection = multiselect(
-            label: 'What item types would you like to check in and delete?',
-            options: [
-                'all' => 'All Items (assets, licenses, accessories, components, consumables, users)',
-                'assets' => 'Assets',
-                'licenses' => 'Licenses',
-                'accessories' => 'Accessories',
-                'components' => 'Components',
-                'consumables' => 'Consumables',
-                'users' => 'Users',
-            ],
-            required: 'Please select at least one item type.',
-            hint: 'Select "All Items" to process every supported type.',
-        );
-
-        $allSubTypes = ['assets', 'licenses', 'accessories', 'components', 'consumables', 'users'];
-        $selectedTypes = in_array('all', $rawTypeSelection)
-            ? $allSubTypes
-            : array_values(array_intersect($allSubTypes, $rawTypeSelection));
-
-        // Compute and display counts now so the user can see what will be affected
-        $counts = $this->getCounts($selectedTypes, $selectedCompanyIds, $includeNullCompany);
-
-        $skipAdminUser = false;
-
-        $this->line('');
-        $this->line('  Items that would be affected:');
-        foreach ($counts as $type => $count) {
-            $this->line(sprintf('    %-14s %d', ucfirst($type).':', $count));
-        }
-
-        if (in_array('users', $selectedTypes)) {
-            $userInScope = $this->buildUserQuery($selectedCompanyIds, $includeNullCompany)
-                ->where('users.id', $admin->id)
-                ->exists();
-
-            if ($userInScope) {
-                $skipAdminUser = true;
-                $counts['users'] = max(0, ($counts['users'] ?? 0) - 1);
-                warning("  Your user ({$admin->username}) is within the selected scope and will be skipped during user deletion.");
-            }
-        }
-
-        $this->line('');
-
-        // Step 5: Hard delete, soft delete, or no delete?
-        $deleteType = select(
-            label: 'How should items be deleted?',
-            options: [
-                'soft' => 'Soft delete — items moved to trash (recoverable)',
-                'hard' => 'Hard delete — permanently removed (cannot be recovered)',
-                'none' => 'No delete — check in only, items remain in inventory',
-            ],
-            default: 'soft',
-        );
-
-        // Step 6: Send checkin notifications? (not applicable to users or consumables)
-        $notifiableTypes = array_intersect($selectedTypes, ['assets', 'licenses', 'accessories', 'components']);
-        $sendNotifications = false;
-
-        if (! empty($notifiableTypes)) {
-            $sendNotifications = confirm(
-                label: 'Should we send checkin notifications?',
-                default: true,
-                hint: 'Applies to: '.implode(', ', $notifiableTypes).'. Users and consumables are excluded.',
-            );
-        }
-
-        // Step 7: Clear related action_logs?
-        $clearLogs = confirm(
-            label: 'Should we clear related action logs?',
-            default: false,
-            hint: 'This removes all history for affected items, as if the data never existed.',
-        );
-
-        // Step 8: Delete associated files?
-        $deleteFiles = false;
-        if ($deleteType !== 'none') {
-            $deleteFiles = confirm(
-                label: 'Should we also delete associated image and upload files?',
-                default: $deleteType === 'hard',
-                hint: 'Permanently removes images, avatars, signatures, EULAs, and action log uploads from disk.',
-            );
-        }
-
-        // Step 9: Delete the companies themselves?
-        $deleteCompanyType = 'keep';
-        if (! empty($selectedCompanyIds)) {
-            $deleteCompanyType = select(
-                label: 'Should the selected companies also be deleted?',
-                options: [
-                    'keep' => 'Keep — do not delete the companies',
-                    'soft' => 'Soft delete — companies moved to trash (recoverable)',
-                    'hard' => 'Hard delete — permanently removed (cannot be recovered)',
-                ],
-                default: 'keep',
-            );
-        }
-
-        // Step 10: Backup first?
-        $doBackup = confirm(
-            label: 'Should we run a backup before proceeding?',
-            default: true,
-            hint: 'Strongly recommended. Saved as backup-before-bulk-delete-cli-[datetime].zip',
-        );
-
-        // Step 11: Summary + final confirmation
-        $this->line('');
-        $this->line('  ════════════════════════════════════════════════════');
-        $this->line('   SUMMARY OF ACTIONS');
-        $this->line('  ════════════════════════════════════════════════════');
-        $this->line("   Admin user:      {$admin->first_name} {$admin->last_name} ({$admin->username})");
-        $this->line('   Companies:       '.implode(', ', $selectedCompanyNames));
-        $this->line('   Item types:      '.implode(', ', $selectedTypes));
-        $this->line("   Delete mode:     {$deleteType}");
-        $this->line('   Notifications:   '.($sendNotifications ? 'Yes' : 'No'));
-        $this->line('   Clear logs:      '.($clearLogs ? 'Yes' : 'No'));
-        $this->line('   Delete files:    '.($deleteFiles ? 'Yes' : 'No'));
-        $this->line('   Delete companies: '.($deleteCompanyType === 'keep' ? 'No' : ucfirst($deleteCompanyType).' delete'));
-        $this->line('   Backup first:    '.($doBackup ? 'Yes' : 'No'));
-        $this->line('   Dry run:         '.($dryRun ? 'Yes' : 'No'));
-        $this->line('');
-        $this->line('   Items to be processed:');
-        foreach ($counts as $type => $count) {
-            $this->line(sprintf('     %-14s %d', ucfirst($type).':', $count));
-        }
-        if ($skipAdminUser) {
-            $this->line('   * Your user account will be skipped during user deletion.');
-        }
-        $this->line('  ════════════════════════════════════════════════════');
-        $this->line('');
-
-        // Step 10.5: Email report?
-        $sendEmailReport = false;
-        if ($admin->email) {
-            $sendEmailReport = confirm(
-                label: "Send an email report to {$admin->email}?",
-                default: false,
-                hint: 'A summary of all '.($dryRun ? 'would-be ' : '').'actions will be emailed to you.',
-            );
-        }
-
-        if (! $dryRun) {
-            $confirmed = confirm(
-                label: 'Are you sure you want to proceed? This cannot be undone.',
-                default: false,
-            );
-
-            if (! $confirmed) {
-                info('Aborted. No changes were made.');
-
-                return 0;
-            }
-        }
-
-        // Run backup if requested
-        if ($doBackup && ! $dryRun) {
-            $backupFilename = 'backup-before-bulk-delete-cli-'.now()->format('Y-m-d-H-i-s');
-            info("Running backup ({$backupFilename}.zip)...");
-            $result = $this->callSilently('snipeit:backup', ['--filename' => $backupFilename]);
-            if ($result === 0) {
-                info("Backup completed: {$backupFilename}.zip");
-            } else {
-                warning("Backup may have failed (exit code {$result}). Proceeding anyway.");
-            }
-        }
-
-        // Step 11: Execute with progress bar
-        $totalItems = array_sum($counts);
-        $bar = $this->output->createProgressBar($totalItems > 0 ? $totalItems : 1);
-        $bar->setFormat(' %current%/%max% [%bar%] %percent:3s%% %message%');
-        $bar->setMessage('Starting...');
-        $bar->start();
-
-        foreach ($selectedTypes as $type) {
-            match ($type) {
-                'assets' => $this->processAssets($selectedCompanyIds, $includeNullCompany, $sendNotifications, $admin, $dryRun, $deleteType, $clearLogs, $deleteFiles, $bar),
-                'licenses' => $this->processLicenses($selectedCompanyIds, $includeNullCompany, $sendNotifications, $admin, $dryRun, $deleteType, $clearLogs, $deleteFiles, $bar),
-                'accessories' => $this->processAccessories($selectedCompanyIds, $includeNullCompany, $sendNotifications, $admin, $dryRun, $deleteType, $clearLogs, $deleteFiles, $bar),
-                'components' => $this->processComponents($selectedCompanyIds, $includeNullCompany, $sendNotifications, $admin, $dryRun, $deleteType, $clearLogs, $deleteFiles, $bar),
-                'consumables' => $this->processConsumables($selectedCompanyIds, $includeNullCompany, $dryRun, $deleteType, $clearLogs, $deleteFiles, $bar),
-                'users' => $this->processUsers($selectedCompanyIds, $includeNullCompany, $admin, $skipAdminUser, $dryRun, $deleteType, $clearLogs, $deleteFiles, $bar),
-            };
-        }
-
-        $bar->setMessage('Done.');
-        $bar->finish();
-        $this->line('');
-        $this->line('');
-
-        // Delete companies if requested
-        if ($deleteCompanyType !== 'keep' && ! empty($selectedCompanyIds)) {
-            $companies = Company::whereIn('id', $selectedCompanyIds)->get();
-            foreach ($companies as $company) {
-                if ($dryRun) {
-                    $this->line("  [dry-run] Would {$deleteCompanyType}-delete company {$company->name}");
-                    $this->reportLines[] = "Would {$deleteCompanyType}-delete company {$company->name}";
-                } else {
-                    if ($deleteCompanyType === 'soft') {
-                        $company->delete();
-                    } else {
-                        $company->forceDelete();
-                    }
-                    // Remove any remaining pivot associations (e.g. the admin user who was
-                    // skipped during user processing but is still a member of this company)
-                    DB::table('company_user')->where('company_id', $company->id)->delete();
-                    $this->reportLines[] = ucfirst($deleteCompanyType)."-deleted company {$company->name}";
-                }
-            }
-        }
-
-        if ($dryRun) {
-            warning('Dry run complete — no changes were made.');
-        } else {
-            info('All actions completed successfully.');
-        }
-
-        if ($sendEmailReport && $admin->email) {
-            Mail::to($admin->email)->send(new BulkDeleteReportMail(
-                admin: $admin,
-                dryRun: $dryRun,
-                companyNames: $selectedCompanyNames,
-                selectedTypes: $selectedTypes,
-                deleteType: $deleteType,
-                reportLines: $this->reportLines,
-                runAt: now(),
-            ));
-            info("Report sent to {$admin->email}.");
-        }
-
-        return 0;
-    }
-
-    private function getCounts(array $types, array $companyIds, bool $includeNull): array
-    {
-        $counts = [];
-
-        if (in_array('assets', $types)) {
-            $counts['assets'] = $this->buildCompanyQuery(Asset::query(), $companyIds, $includeNull)->count();
-        }
-        if (in_array('licenses', $types)) {
-            $counts['licenses'] = $this->buildCompanyQuery(License::query(), $companyIds, $includeNull)->count();
-        }
-        if (in_array('accessories', $types)) {
-            $counts['accessories'] = $this->buildCompanyQuery(Accessory::query(), $companyIds, $includeNull)->count();
-        }
-        if (in_array('components', $types)) {
-            $counts['components'] = $this->buildCompanyQuery(Component::query(), $companyIds, $includeNull)->count();
-        }
-        if (in_array('consumables', $types)) {
-            $counts['consumables'] = $this->buildCompanyQuery(Consumable::query(), $companyIds, $includeNull)->count();
-        }
-        if (in_array('users', $types)) {
-            $counts['users'] = $this->buildUserQuery($companyIds, $includeNull)->count();
-        }
-
-        return $counts;
-    }
-
-    private function buildCompanyQuery(Builder $query, array $companyIds, bool $includeNull): Builder
-    {
-        return $query->where(function (Builder $q) use ($companyIds, $includeNull) {
-            if (! empty($companyIds)) {
-                $q->whereIn('company_id', $companyIds);
-            }
-            if ($includeNull) {
-                $method = ! empty($companyIds) ? 'orWhereNull' : 'whereNull';
-                $q->{$method}('company_id');
-            }
-        });
-    }
-
-    private function buildUserQuery(array $companyIds, bool $includeNull): Builder
-    {
-        return User::query()
-            ->where('activated', 1)
-            ->where(function (Builder $q) use ($companyIds, $includeNull) {
-                if (! empty($companyIds)) {
-                    $q->whereIn('company_id', $companyIds);
-                }
-                if ($includeNull) {
-                    $method = ! empty($companyIds) ? 'orWhereNull' : 'whereNull';
-                    $q->{$method}('company_id');
-                }
-            });
-    }
-
-    private function processAssets(
-        array $companyIds,
-        bool $includeNull,
-        bool $sendNotifications,
-        User $admin,
-        bool $dryRun,
-        string $deleteType,
-        bool $clearLogs,
-        bool $deleteFiles,
-        ProgressBar $bar,
-    ): void {
-        $assets = $this->buildCompanyQuery(Asset::query(), $companyIds, $includeNull)->get();
-
-        foreach ($assets as $asset) {
-            $bar->setMessage("Assets: {$asset->asset_tag}");
-
-            if ($asset->assignedTo) {
-                if ($dryRun) {
-                    $this->line("  [dry-run] Would check in asset {$asset->asset_tag} from {$asset->assignedTo->name}");
-                    $this->reportLines[] = "Would check in asset {$asset->asset_tag} (assigned to {$asset->assignedTo->name})";
-                } else {
-                    $target = $asset->assignedTo;
-                    $checkinAt = now()->format('Y-m-d H:i:s');
-                    $originalValues = $asset->getRawOriginal();
-
-                    if ($sendNotifications) {
-                        event(new CheckoutableCheckedIn($asset, $target, $admin, self::CHECKIN_NOTE, $checkinAt, $originalValues));
-                        DB::table('assets')->where('id', $asset->id)->update(['assigned_to' => null, 'assigned_type' => null]);
-                    } else {
-                        DB::table('assets')->where('id', $asset->id)->update(['assigned_to' => null, 'assigned_type' => null]);
-                        $asset->logCheckin($target, self::CHECKIN_NOTE, $checkinAt, $originalValues);
-                    }
-
-                    $this->reportLines[] = "Checked in asset {$asset->asset_tag} from {$target->name}";
-                    $asset->licenseseats()->update(['assigned_to' => null]);
-
-                    CheckoutAcceptance::where('checkoutable_type', Asset::class)
-                        ->where('checkoutable_id', $asset->id)
-                        ->whereNull('accepted_at')
-                        ->whereNull('declined_at')
-                        ->forceDelete();
-                }
-            }
-
-            if (! $dryRun) {
-                // Collect action log file paths before logs may be cleared
-                $actionLogPaths = $deleteFiles
-                    ? $asset->assetlog()->whereNotNull('filename')->get()
-                        ->map(fn (Actionlog $log) => $log->uploads_file_path())
-                        ->filter()
-                        ->values()
-                        ->toArray()
-                    : [];
-
-                // Delete checkout acceptance files, then hard-remove all acceptances
-                if ($deleteFiles) {
-                    CheckoutAcceptance::where('checkoutable_type', Asset::class)
-                        ->where('checkoutable_id', $asset->id)
-                        ->get()
-                        ->each(fn (CheckoutAcceptance $ca) => $this->deleteAcceptanceFiles($ca));
-                }
-                CheckoutAcceptance::where('checkoutable_type', Asset::class)
-                    ->where('checkoutable_id', $asset->id)
-                    ->forceDelete();
-
-                // Hard-delete-only cleanup: maintenance records, accessory checkouts to this
-                // asset, and any other assets that were assigned to this one
-                $maintenanceImages = [];
-                if ($deleteType === 'hard') {
-                    if ($deleteFiles) {
-                        $maintenanceImages = $asset->maintenances()
-                            ->whereNotNull('image')
-                            ->pluck('image')
-                            ->toArray();
-                    }
-                    $asset->maintenances()->forceDelete();
-                    AccessoryCheckout::where('assigned_to', $asset->id)
-                        ->where('assigned_type', Asset::class)
-                        ->delete();
-                    DB::table('assets')
-                        ->where('assigned_to', $asset->id)
-                        ->where('assigned_type', Asset::class)
-                        ->update(['assigned_to' => null, 'assigned_type' => null]);
-                }
-
-                match ($deleteType) {
-                    'soft' => $asset->delete(),
-                    'hard' => $asset->forceDelete(),
-                    default => null,
-                };
-
-                if ($deleteType !== 'none') {
-                    $this->reportLines[] = ucfirst($deleteType)."-deleted asset {$asset->asset_tag}";
-                }
-
-                if ($clearLogs) {
-                    $asset->assetlog()->forceDelete();
-                }
-
-                if ($deleteFiles) {
-                    if ($asset->image) {
-                        $this->deleteStorageFile('public', app('assets_upload_path').$asset->image);
-                    }
-                    foreach ($maintenanceImages as $img) {
-                        $this->deleteStorageFile('public', app('maintenances_upload_path').$img);
-                    }
-                    foreach ($actionLogPaths as $path) {
-                        $this->deleteStorageFile('local', $path);
-                    }
-                }
-            } elseif ($deleteType !== 'none') {
-                $this->line("  [dry-run] Would {$deleteType}-delete asset {$asset->asset_tag}");
-                $this->reportLines[] = "Would {$deleteType}-delete asset {$asset->asset_tag}";
-            }
-
-            $bar->advance();
-        }
-    }
-
-    private function processLicenses(
-        array $companyIds,
-        bool $includeNull,
-        bool $sendNotifications,
-        User $admin,
-        bool $dryRun,
-        string $deleteType,
-        bool $clearLogs,
-        bool $deleteFiles,
-        ProgressBar $bar,
-    ): void {
-        $licenses = $this->buildCompanyQuery(License::query(), $companyIds, $includeNull)->get();
-
-        foreach ($licenses as $license) {
-            $bar->setMessage("Licenses: {$license->name}");
-
-            $seats = LicenseSeat::where('license_id', $license->id)
-                ->where(fn ($q) => $q->whereNotNull('assigned_to')->orWhereNotNull('asset_id'))
-                ->get();
-
-            foreach ($seats as $seat) {
-                $target = $seat->assigned_to ? $seat->user : $seat->asset;
-
-                if ($dryRun) {
-                    $this->line("  [dry-run] Would check in license seat for {$license->name} from ".($target?->name ?? $target?->asset_tag ?? 'unknown'));
-                    $this->reportLines[] = "Would check in license seat for {$license->name} from ".($target?->name ?? $target?->asset_tag ?? 'unknown');
-                } else {
-                    $seat->assigned_to = null;
-                    $seat->asset_id = null;
-                    $seat->save();
-
-                    $this->reportLines[] = "Checked in license seat for {$license->name} from ".($target?->name ?? $target?->asset_tag ?? 'unknown');
-
-                    if ($target) {
-                        if ($sendNotifications) {
-                            event(new CheckoutableCheckedIn($seat, $target, $admin, self::CHECKIN_NOTE));
-                        } else {
-                            $seat->logCheckin($target, self::CHECKIN_NOTE);
-                        }
-                    }
-                }
-            }
-
-            if (! $dryRun) {
-                // Collect action log file paths before logs may be cleared
-                $actionLogPaths = $deleteFiles
-                    ? $license->assetlog()->whereNotNull('filename')->get()
-                        ->map(fn (Actionlog $log) => $log->uploads_file_path())
-                        ->filter()
-                        ->values()
-                        ->toArray()
-                    : [];
-
-                if ($deleteType === 'soft') {
-                    $license->licenseseats()->delete();
-                    $license->delete();
-                    $this->reportLines[] = "Soft-deleted license {$license->name}";
-                } elseif ($deleteType === 'hard') {
-                    $seatIds = $license->licenseseats()->pluck('id');
-                    if ($deleteFiles) {
-                        CheckoutAcceptance::where('checkoutable_type', LicenseSeat::class)
-                            ->whereIn('checkoutable_id', $seatIds)
-                            ->get()
-                            ->each(fn (CheckoutAcceptance $ca) => $this->deleteAcceptanceFiles($ca));
-                    }
-                    CheckoutAcceptance::where('checkoutable_type', LicenseSeat::class)
-                        ->whereIn('checkoutable_id', $seatIds)
-                        ->forceDelete();
-                    $license->licenseseats()->forceDelete();
-                    DB::table('kits_licenses')->where('license_id', $license->id)->delete();
-                    $license->forceDelete();
-                    $this->reportLines[] = "Hard-deleted license {$license->name}";
-                }
-
-                if ($clearLogs) {
-                    $license->assetlog()->forceDelete();
-                }
-
-                if ($deleteFiles) {
-                    foreach ($actionLogPaths as $path) {
-                        $this->deleteStorageFile('local', $path);
-                    }
-                }
-            } elseif ($deleteType !== 'none') {
-                $this->line("  [dry-run] Would {$deleteType}-delete license {$license->name}");
-                $this->reportLines[] = "Would {$deleteType}-delete license {$license->name}";
-            }
-
-            $bar->advance();
-        }
-    }
-
-    private function processAccessories(
-        array $companyIds,
-        bool $includeNull,
-        bool $sendNotifications,
-        User $admin,
-        bool $dryRun,
-        string $deleteType,
-        bool $clearLogs,
-        bool $deleteFiles,
-        ProgressBar $bar,
-    ): void {
-        $accessories = $this->buildCompanyQuery(Accessory::query(), $companyIds, $includeNull)->get();
-
-        foreach ($accessories as $accessory) {
-            $bar->setMessage("Accessories: {$accessory->name}");
-
-            $checkouts = AccessoryCheckout::where('accessory_id', $accessory->id)->get();
-
-            foreach ($checkouts as $checkout) {
-                $target = $checkout->assignedTo;
-
-                if ($dryRun) {
-                    $this->line("  [dry-run] Would check in accessory {$accessory->name} from ".($target?->name ?? 'unknown'));
-                    $this->reportLines[] = "Would check in accessory {$accessory->name} from ".($target?->name ?? 'unknown');
-                } else {
-                    $checkinAt = now()->format('Y-m-d H:i:s');
-                    $checkout->delete();
-
-                    $this->reportLines[] = "Checked in accessory {$accessory->name} from ".($target?->name ?? 'unknown');
-
-                    if ($target) {
-                        if ($sendNotifications) {
-                            event(new CheckoutableCheckedIn($accessory, $target, $admin, self::CHECKIN_NOTE, $checkinAt));
-                        } else {
-                            $accessory->logCheckin($target, self::CHECKIN_NOTE, $checkinAt);
-                        }
-                    }
-                }
-            }
-
-            if (! $dryRun) {
-                // Collect action log file paths before logs may be cleared
-                $actionLogPaths = $deleteFiles
-                    ? $accessory->assetlog()->whereNotNull('filename')->get()
-                        ->map(fn (Actionlog $log) => $log->uploads_file_path())
-                        ->filter()
-                        ->values()
-                        ->toArray()
-                    : [];
-
-                if ($clearLogs) {
-                    $accessory->assetlog()->forceDelete();
-                }
-
-                if ($deleteType === 'hard') {
-                    DB::table('kits_accessories')->where('accessory_id', $accessory->id)->delete();
-                }
-
-                match ($deleteType) {
-                    'soft' => $accessory->delete(),
-                    'hard' => $accessory->forceDelete(),
-                    default => null,
-                };
-
-                if ($deleteType !== 'none') {
-                    $this->reportLines[] = ucfirst($deleteType)."-deleted accessory {$accessory->name}";
-                }
-
-                if ($deleteFiles) {
-                    if ($accessory->image) {
-                        $this->deleteStorageFile('public', app('accessories_upload_path').$accessory->image);
-                    }
-                    foreach ($actionLogPaths as $path) {
-                        $this->deleteStorageFile('local', $path);
-                    }
-                }
-            } elseif ($deleteType !== 'none') {
-                $this->line("  [dry-run] Would {$deleteType}-delete accessory {$accessory->name}");
-                $this->reportLines[] = "Would {$deleteType}-delete accessory {$accessory->name}";
-            }
-
-            $bar->advance();
-        }
-    }
-
-    private function processComponents(
-        array $companyIds,
-        bool $includeNull,
-        bool $sendNotifications,
-        User $admin,
-        bool $dryRun,
-        string $deleteType,
-        bool $clearLogs,
-        bool $deleteFiles,
-        ProgressBar $bar,
-    ): void {
-        $components = $this->buildCompanyQuery(Component::query(), $companyIds, $includeNull)->get();
-
-        foreach ($components as $component) {
-            $bar->setMessage("Components: {$component->name}");
-
-            $assignments = DB::table('components_assets')
-                ->where('component_id', $component->id)
-                ->get();
-
-            foreach ($assignments as $assignment) {
-                $asset = Asset::find($assignment->asset_id);
-
-                if ($dryRun) {
-                    $this->line("  [dry-run] Would check in component {$component->name} from asset ".($asset?->asset_tag ?? 'unknown'));
-                    $this->reportLines[] = "Would check in component {$component->name} from asset ".($asset?->asset_tag ?? 'unknown');
-                } else {
-                    $checkinAt = now()->format('Y-m-d H:i:s');
-                    DB::table('components_assets')->where('id', $assignment->id)->delete();
-
-                    $this->reportLines[] = "Checked in component {$component->name} from asset ".($asset?->asset_tag ?? 'unknown');
-
-                    if ($asset) {
-                        if ($sendNotifications) {
-                            event(new CheckoutableCheckedIn($component, $asset, $admin, self::CHECKIN_NOTE, $checkinAt));
-                        } else {
-                            $component->logCheckin($asset, self::CHECKIN_NOTE, $checkinAt);
-                        }
-                    }
-                }
-            }
-
-            if (! $dryRun) {
-                // Collect action log file paths before logs may be cleared
-                $actionLogPaths = $deleteFiles
-                    ? $component->assetlog()->whereNotNull('filename')->get()
-                        ->map(fn (Actionlog $log) => $log->uploads_file_path())
-                        ->filter()
-                        ->values()
-                        ->toArray()
-                    : [];
-
-                if ($clearLogs) {
-                    $component->assetlog()->forceDelete();
-                }
-
-                match ($deleteType) {
-                    'soft' => $component->delete(),
-                    'hard' => $component->forceDelete(),
-                    default => null,
-                };
-
-                if ($deleteType !== 'none') {
-                    $this->reportLines[] = ucfirst($deleteType)."-deleted component {$component->name}";
-                }
-
-                if ($deleteFiles) {
-                    if ($component->image) {
-                        $this->deleteStorageFile('public', app('components_upload_path').$component->image);
-                    }
-                    foreach ($actionLogPaths as $path) {
-                        $this->deleteStorageFile('local', $path);
-                    }
-                }
-            } elseif ($deleteType !== 'none') {
-                $this->line("  [dry-run] Would {$deleteType}-delete component {$component->name}");
-                $this->reportLines[] = "Would {$deleteType}-delete component {$component->name}";
-            }
-
-            $bar->advance();
-        }
-    }
-
-    private function processConsumables(
-        array $companyIds,
-        bool $includeNull,
-        bool $dryRun,
-        string $deleteType,
-        bool $clearLogs,
-        bool $deleteFiles,
-        ProgressBar $bar,
-    ): void {
-        $consumables = $this->buildCompanyQuery(Consumable::query(), $companyIds, $includeNull)->get();
-
-        foreach ($consumables as $consumable) {
-            $bar->setMessage("Consumables: {$consumable->name}");
-
-            if (! $dryRun) {
-                // Collect action log file paths before logs may be cleared
-                $actionLogPaths = $deleteFiles
-                    ? $consumable->assetlog()->whereNotNull('filename')->get()
-                        ->map(fn (Actionlog $log) => $log->uploads_file_path())
-                        ->filter()
-                        ->values()
-                        ->toArray()
-                    : [];
-
-                if ($clearLogs) {
-                    $consumable->assetlog()->forceDelete();
-                }
-
-                if ($deleteType === 'hard') {
-                    DB::table('kits_consumables')->where('consumable_id', $consumable->id)->delete();
-                }
-
-                match ($deleteType) {
-                    'soft' => $consumable->delete(),
-                    'hard' => $consumable->forceDelete(),
-                    default => null,
-                };
-
-                if ($deleteType !== 'none') {
-                    $this->reportLines[] = ucfirst($deleteType)."-deleted consumable {$consumable->name}";
-                }
-
-                if ($deleteFiles) {
-                    if ($consumable->image) {
-                        $this->deleteStorageFile('public', app('consumables_upload_path').$consumable->image);
-                    }
-                    foreach ($actionLogPaths as $path) {
-                        $this->deleteStorageFile('local', $path);
-                    }
-                }
-            } elseif ($deleteType !== 'none') {
-                $this->line("  [dry-run] Would {$deleteType}-delete consumable {$consumable->name}");
-                $this->reportLines[] = "Would {$deleteType}-delete consumable {$consumable->name}";
-            }
-
-            $bar->advance();
-        }
-    }
-
-    private function processUsers(
-        array $companyIds,
-        bool $includeNull,
-        User $admin,
-        bool $skipAdminUser,
-        bool $dryRun,
-        string $deleteType,
-        bool $clearLogs,
-        bool $deleteFiles,
-        ProgressBar $bar,
-    ): void {
-        $users = $this->buildUserQuery($companyIds, $includeNull)->get();
-
-        foreach ($users as $user) {
-            if ($skipAdminUser && $user->id === $admin->id) {
-                continue;
-            }
-
-            $bar->setMessage("Users: {$user->username}");
-
-            // If real companies were selected, check whether this user also belongs to
-            // companies outside the selected scope. If so, only remove the selected-company
-            // associations and skip full deletion to avoid orphaning them from their other companies.
-            if (! empty($companyIds)) {
-                $allUserCompanyIds = array_unique(array_filter(array_merge(
-                    $user->companies()->pluck('companies.id')->toArray(),
-                    $user->company_id ? [$user->company_id] : [],
-                )));
-                $outsideCompanyIds = array_values(array_diff($allUserCompanyIds, $companyIds));
-
-                if (! empty($outsideCompanyIds)) {
-                    $outsideNames = Company::whereIn('id', $outsideCompanyIds)->pluck('name')->implode(', ');
-
-                    if ($dryRun) {
-                        $this->line("  [dry-run] Would partially disassociate user {$user->username} (also belongs to: {$outsideNames})");
-                        $this->reportLines[] = "Would partially disassociate user {$user->username} — also belongs to: {$outsideNames}";
-                    } else {
-                        $user->companies()->detach($companyIds);
-                        warning("  Skipped full deletion of {$user->username}: they also belong to {$outsideNames}. Removed selected company associations only.");
-                        $this->reportLines[] = "Partially disassociated user {$user->username} — also belongs to: {$outsideNames}. Full deletion skipped.";
-                    }
-
-                    $bar->advance();
-
-                    continue;
-                }
-            }
-
-            if (! $dryRun) {
-                // Collect file paths and acceptance records before deleting pivot data
-                $acceptancesToDelete = $deleteFiles
-                    ? CheckoutAcceptance::where('assigned_to_id', $user->id)->get()
-                    : collect();
-
-                $actionLogPaths = $deleteFiles
-                    ? Actionlog::where('item_type', User::class)
-                        ->where('item_id', $user->id)
-                        ->where('action_type', 'uploaded')
-                        ->whereNotNull('filename')
-                        ->get()
-                        ->map(fn (Actionlog $log) => $log->uploads_file_path())
-                        ->filter()
-                        ->values()
-                        ->toArray()
-                    : [];
-
-                // Clear pivot/assignment data that will orphan on deletion
-                LicenseSeat::where('assigned_to', $user->id)->update(['assigned_to' => null]);
-                AccessoryCheckout::where('assigned_to', $user->id)
-                    ->where('assigned_type', User::class)
-                    ->delete();
-                DB::table('consumables_users')->where('assigned_to', $user->id)->delete();
-                CheckoutAcceptance::where('assigned_to_id', $user->id)->forceDelete();
-                if ($deleteType === 'hard') {
-                    DB::table('company_user')->where('user_id', $user->id)->delete();
-                }
-
-                if ($clearLogs) {
-                    $user->userlog()->forceDelete();
-                }
-
-                match ($deleteType) {
-                    'soft' => $user->delete(),
-                    'hard' => $user->forceDelete(),
-                    default => null,
-                };
-
-                if ($deleteType !== 'none') {
-                    $this->reportLines[] = ucfirst($deleteType)."-deleted user {$user->username}";
-                }
-
-                if ($deleteFiles) {
-                    if ($user->avatar) {
-                        $this->deleteStorageFile('public', app('users_upload_path').$user->avatar);
-                    }
-                    $acceptancesToDelete->each(fn (CheckoutAcceptance $ca) => $this->deleteAcceptanceFiles($ca));
-                    foreach ($actionLogPaths as $path) {
-                        $this->deleteStorageFile('local', $path);
-                    }
-                }
-            } elseif ($deleteType !== 'none') {
-                $this->line("  [dry-run] Would {$deleteType}-delete user {$user->username}");
-                $this->reportLines[] = "Would {$deleteType}-delete user {$user->username}";
-            }
-
-            $bar->advance();
-        }
-    }
-
-    private function deleteStorageFile(string $disk, ?string $path): void
-    {
-        if (! $path) {
-            return;
-        }
-        try {
-            $storage = $disk === 'public'
-                ? Storage::disk('public')
-                : Storage::disk(config('filesystems.default'));
-            if ($storage->exists($path)) {
-                $storage->delete($path);
-            }
-        } catch (\Exception $e) {
-            Log::warning("Could not delete file {$path}: ".$e->getMessage());
-        }
-    }
-
-    private function deleteAcceptanceFiles(CheckoutAcceptance $acceptance): void
-    {
-        if ($acceptance->signature_filename) {
-            $this->deleteStorageFile('local', 'private_uploads/signatures/'.$acceptance->signature_filename);
-        }
-        if ($acceptance->stored_eula_file) {
-            $this->deleteStorageFile('local', 'private_uploads/eula-pdfs/'.$acceptance->stored_eula_file);
-        }
-    }
-}

app/Console/Commands/CheckinAndDeleteItems.php

@@ -1,308 +0,0 @@
-<?php
-
-namespace App\Console\Commands;
-
-use App\Events\CheckoutableCheckedIn;
-use App\Models\Accessory;
-use App\Models\AccessoryCheckout;
-use App\Models\Asset;
-use App\Models\CheckoutAcceptance;
-use App\Models\Component;
-use App\Models\License;
-use App\Models\LicenseSeat;
-use App\Models\User;
-use Illuminate\Console\Command;
-use Illuminate\Database\Eloquent\Builder;
-use Illuminate\Support\Facades\DB;
-
-class CheckinAndDeleteItems extends Command
-{
-    protected $signature = 'snipeit:checkin-delete-all
-        {--company-id= : Only process items belonging to this company ID}
-        {--admin-id= : ID of the user credited for the checkins (defaults to first superadmin)}
-        {--no-notifications : Suppress email and webhook notifications}
-        {--type=all : Comma-separated types to process: assets, licenses, accessories, components, or all}
-        {--note= : Note recorded on each checkin action log entry}
-        {--dry-run : Preview what would be processed without making any changes}
-        {--force : Skip the confirmation prompt}';
-
-    protected $description = 'Check in all assigned items and soft-delete them, optionally scoped to a company';
-
-    public function handle(): int
-    {
-        $companyId = $this->option('company-id');
-        $noNotifications = $this->option('no-notifications');
-        $dryRun = $this->option('dry-run');
-        $typeOption = $this->option('type') ?? 'all';
-        $note = $this->option('note') ?: 'Checked in and deleted via CLI';
-
-        $allTypes = ['assets', 'licenses', 'accessories', 'components'];
-        $typesToProcess = $typeOption === 'all'
-            ? $allTypes
-            : array_intersect(array_map('trim', explode(',', $typeOption)), $allTypes);
-
-        if (empty($typesToProcess)) {
-            $this->error('Invalid --type value. Use: assets, licenses, accessories, components, or all.');
-
-            return 1;
-        }
-
-        $admin = null;
-        if (! $dryRun && ! $noNotifications) {
-            if ($this->option('admin-id')) {
-                $admin = User::find($this->option('admin-id'));
-                if (! $admin) {
-                    $this->error('No user found with admin-id '.$this->option('admin-id').'.');
-
-                    return 1;
-                }
-            } else {
-                $admin = User::onlySuperAdmins()->first();
-            }
-
-            if (! $admin) {
-                $this->warn('No admin user found — notifications will be suppressed.');
-                $noNotifications = true;
-            }
-        }
-
-        $scopeMsg = $companyId ? "company ID {$companyId}" : 'all companies';
-        $typesMsg = implode(', ', $typesToProcess);
-
-        if ($dryRun) {
-            $this->warn('DRY RUN — no changes will be made.');
-        } elseif (! $this->option('force')) {
-            if (! $this->confirm("This will check in and soft-delete all [{$typesMsg}] for [{$scopeMsg}]. Continue?")) {
-                $this->info('Aborted.');
-
-                return 0;
-            }
-        }
-
-        if (in_array('assets', $typesToProcess)) {
-            $this->processAssets($companyId, $noNotifications, $note, $admin, $dryRun);
-        }
-
-        if (in_array('licenses', $typesToProcess)) {
-            $this->processLicenses($companyId, $noNotifications, $note, $admin, $dryRun);
-        }
-
-        if (in_array('accessories', $typesToProcess)) {
-            $this->processAccessories($companyId, $noNotifications, $note, $admin, $dryRun);
-        }
-
-        if (in_array('components', $typesToProcess)) {
-            $this->processComponents($companyId, $noNotifications, $note, $admin, $dryRun);
-        }
-
-        if ($dryRun) {
-            $this->warn('Dry run complete — no changes were made.');
-        }
-
-        return 0;
-    }
-
-    private function processAssets(?string $companyId, bool $noNotifications, string $note, ?User $admin, bool $dryRun): void
-    {
-        $query = Asset::query();
-        if ($companyId) {
-            $query->where('company_id', $companyId);
-        }
-
-        $assets = $query->get();
-        $checkedIn = 0;
-        $deleted = 0;
-
-        foreach ($assets as $asset) {
-            if ($asset->assignedTo) {
-                if ($dryRun) {
-                    $this->line('  Would check in asset: '.$asset->asset_tag.' (assigned to '.$asset->assignedTo->name.')');
-                } else {
-                    $target = $asset->assignedTo;
-                    $checkin_at = now()->format('Y-m-d H:i:s');
-                    $originalValues = $asset->getRawOriginal();
-
-                    if ($noNotifications) {
-                        DB::table('assets')->where('id', $asset->id)
-                            ->update(['assigned_to' => null, 'assigned_type' => null]);
-                        $asset->logCheckin($target, $note, $checkin_at, $originalValues);
-                    } else {
-                        // Fire event before clearing so the log captures the original state
-                        event(new CheckoutableCheckedIn($asset, $target, $admin, $note, $checkin_at, $originalValues));
-                        DB::table('assets')->where('id', $asset->id)
-                            ->update(['assigned_to' => null, 'assigned_type' => null]);
-                    }
-
-                    $asset->licenseseats()->update(['assigned_to' => null]);
-
-                    CheckoutAcceptance::pending()
-                        ->whereHasMorph('checkoutable', [Asset::class], fn (Builder $q) => $q->where('id', $asset->id))
-                        ->delete();
-                }
-
-                $checkedIn++;
-            }
-
-            if ($dryRun) {
-                $this->line('  Would delete asset: '.$asset->asset_tag);
-            } else {
-                $asset->delete();
-            }
-
-            $deleted++;
-        }
-
-        $action = $dryRun ? 'would be' : 'were';
-        $this->info("Assets: {$checkedIn} {$action} checked in, {$deleted} {$action} deleted.");
-    }
-
-    private function processLicenses(?string $companyId, bool $noNotifications, string $note, ?User $admin, bool $dryRun): void
-    {
-        $query = License::query();
-        if ($companyId) {
-            $query->where('company_id', $companyId);
-        }
-
-        $licenses = $query->get();
-        $seatsCheckedIn = 0;
-        $deleted = 0;
-
-        foreach ($licenses as $license) {
-            $seats = LicenseSeat::where('license_id', $license->id)
-                ->where(fn ($q) => $q->whereNotNull('assigned_to')->orWhereNotNull('asset_id'))
-                ->get();
-
-            foreach ($seats as $seat) {
-                $target = $seat->assigned_to ? $seat->user : $seat->asset;
-
-                if ($dryRun) {
-                    $this->line('  Would check in license seat for: '.$license->name.' (assigned to '.($target?->name ?? $target?->asset_tag ?? 'unknown').')');
-                } else {
-                    $seat->assigned_to = null;
-                    $seat->asset_id = null;
-                    $seat->save();
-
-                    if ($target) {
-                        if ($noNotifications) {
-                            $seat->logCheckin($target, $note);
-                        } else {
-                            event(new CheckoutableCheckedIn($seat, $target, $admin, $note));
-                        }
-                    }
-                }
-
-                $seatsCheckedIn++;
-            }
-
-            if ($dryRun) {
-                $this->line('  Would delete license: '.$license->name);
-            } else {
-                $license->licenseseats()->delete();
-                $license->delete();
-            }
-
-            $deleted++;
-        }
-
-        $action = $dryRun ? 'would be' : 'were';
-        $this->info("Licenses: {$seatsCheckedIn} seats {$action} checked in, {$deleted} licenses {$action} deleted.");
-    }
-
-    private function processAccessories(?string $companyId, bool $noNotifications, string $note, ?User $admin, bool $dryRun): void
-    {
-        $query = Accessory::query();
-        if ($companyId) {
-            $query->where('company_id', $companyId);
-        }
-
-        $accessories = $query->get();
-        $checkedIn = 0;
-        $deleted = 0;
-
-        foreach ($accessories as $accessory) {
-            $checkouts = AccessoryCheckout::where('accessory_id', $accessory->id)->get();
-
-            foreach ($checkouts as $checkout) {
-                $target = $checkout->assignedTo;
-
-                if ($dryRun) {
-                    $this->line('  Would check in accessory: '.$accessory->name.' (assigned to '.($target?->name ?? $target?->asset_tag ?? 'unknown').')');
-                } else {
-                    $checkin_at = now()->format('Y-m-d H:i:s');
-                    $checkout->delete();
-
-                    if ($target) {
-                        if ($noNotifications) {
-                            $accessory->logCheckin($target, $note, $checkin_at);
-                        } else {
-                            event(new CheckoutableCheckedIn($accessory, $target, $admin, $note, $checkin_at));
-                        }
-                    }
-                }
-
-                $checkedIn++;
-            }
-
-            if ($dryRun) {
-                $this->line('  Would delete accessory: '.$accessory->name);
-            } else {
-                $accessory->delete();
-            }
-
-            $deleted++;
-        }
-
-        $action = $dryRun ? 'would be' : 'were';
-        $this->info("Accessories: {$checkedIn} {$action} checked in, {$deleted} {$action} deleted.");
-    }
-
-    private function processComponents(?string $companyId, bool $noNotifications, string $note, ?User $admin, bool $dryRun): void
-    {
-        $query = Component::query();
-        if ($companyId) {
-            $query->where('company_id', $companyId);
-        }
-
-        $components = $query->get();
-        $checkedIn = 0;
-        $deleted = 0;
-
-        foreach ($components as $component) {
-            $assignments = DB::table('components_assets')
-                ->where('component_id', $component->id)
-                ->get();
-
-            foreach ($assignments as $assignment) {
-                $asset = Asset::find($assignment->asset_id);
-
-                if ($dryRun) {
-                    $this->line('  Would check in component: '.$component->name.' (assigned to '.($asset?->asset_tag ?? 'unknown').')');
-                } else {
-                    $checkin_at = now()->format('Y-m-d H:i:s');
-                    DB::table('components_assets')->where('id', $assignment->id)->delete();
-
-                    if ($asset) {
-                        if ($noNotifications) {
-                            $component->logCheckin($asset, $note, $checkin_at);
-                        } else {
-                            event(new CheckoutableCheckedIn($component, $asset, $admin, $note, $checkin_at));
-                        }
-                    }
-                }
-
-                $checkedIn++;
-            }
-
-            if ($dryRun) {
-                $this->line('  Would delete component: '.$component->name);
-            } else {
-                $component->delete();
-            }
-
-            $deleted++;
-        }
-
-        $action = $dryRun ? 'would be' : 'were';
-        $this->info("Components: {$checkedIn} {$action} checked in, {$deleted} {$action} deleted.");
-    }
-}

app/Console/Commands/CleanIncorrectCheckoutAcceptances.php

@@ -30,77 +30,41 @@ class CleanIncorrectCheckoutAcceptances extends Command
     {
         $deletions = 0;
         $skips = 0;
-        $total = CheckoutAcceptance::count();
 
-        $this->info("Processing {$total} checkout acceptances...");
-        $bar = $this->output->createProgressBar($total);
-        $bar->start();
+        // This walks *every* checkoutacceptance. That's gnarly. But necessary
+        $this->withProgressBar(CheckoutAcceptance::all(), function ($checkoutAcceptance) use (&$deletions, &$skips) {
+            $item = $checkoutAcceptance->checkoutable;
+            $checkout_to_id = $checkoutAcceptance->assigned_to_id;
+            if (is_null($item)) {
+                $this->info("'Checkoutable' Item is null, going to next record");
 
-        // Chunk to avoid loading the whole table into memory; eager-load checkoutable
-        // to eliminate the N+1 on that relationship.
-        CheckoutAcceptance::with('checkoutable')
-            ->chunkById(500, function ($chunk) use (&$deletions, &$skips, $bar) {
-                $idsToDelete = [];
+                return; // 'false' allegedly breaks execution entirely, so 'true' maybe doesn't? hrm. just straight return maybe?
+            }
+            if (get_class($item) == LicenseSeat::class) {
+                $item = $item->license;
+            }
+            foreach ($item->assetlog()->where('action_type', 'checkout')->get() as $assetlog) {
+                if ($assetlog->target_id == $checkout_to_id && $assetlog->target_type != User::class) {
+                    // We have a checkout-to an ID for a non-User, which matches to an ID in the checkout_acceptances table
 
-                foreach ($chunk as $checkoutAcceptance) {
-                    $item = $checkoutAcceptance->checkoutable;
-                    $checkout_to_id = $checkoutAcceptance->assigned_to_id;
-
-                    if (is_null($item)) {
-                        $skips++;
-                        $bar->advance();
-
-                        continue;
-                    }
-
-                    if (get_class($item) === LicenseSeat::class) {
-                        $item = $item->license;
-                        if (is_null($item)) {
-                            $skips++;
-                            $bar->advance();
-
-                            continue;
-                        }
-                    }
-
-                    if (is_null($checkoutAcceptance->created_at)) {
-                        $skips++;
-                        $bar->advance();
-
-                        continue;
-                    }
-
-                    // Push all filtering (including the ±5-second window) into the DB;
-                    // exists() returns as soon as one matching row is found rather than
-                    // fetching all checkout logs into PHP.
-                    $shouldDelete = $item->assetlog()
-                        ->where('action_type', 'checkout')
-                        ->where('target_id', $checkout_to_id)
-                        ->where('target_type', '!=', User::class)
-                        ->whereBetween('created_at', [
-                            $checkoutAcceptance->created_at->copy()->subSeconds(5),
-                            $checkoutAcceptance->created_at->copy()->addSeconds(5),
-                        ])
-                        ->exists();
-
-                    if ($shouldDelete) {
-                        $idsToDelete[] = $checkoutAcceptance->id;
+                    // now, let's compare the _times_ - are they close?
+                    // I'm picking `created_at` over `action_date` because I'm more interested in when the actionlogs
+                    // were _created_, not when they were alleged to have happened - those created_at times need to be within 'X' seconds of
+                    // each other (currently 5)
+                    if ($assetlog->created_at->diffInSeconds($checkoutAcceptance->created_at, true) <= 5) { // we're allowing for five _ish_ seconds of slop
                         $deletions++;
+                        $checkoutAcceptance->forceDelete(); // HARD delete this record; it should have never been
+
+                        return;
                     } else {
-                        $skips++;
+                        // $this->info("The two records are too far apart");
                     }
-
-                    $bar->advance();
+                } else {
+                    // $this->info("No match! checkout to id: " . $checkout_to_id." target_id: ".$assetlog->target_id." target_type: ".$assetlog->target_type);
                 }
-
-                // Bulk-delete the bad records in one query per chunk instead of one per row.
-                if (! empty($idsToDelete)) {
-                    CheckoutAcceptance::whereIn('id', $idsToDelete)->forceDelete();
-                }
-            });
-
-        $bar->finish();
-        $this->newLine();
-        $this->info("Final deletion count: {$deletions}, and skip count: {$skips}");
+            }
+            $skips++;
+        });
+        $this->error("Final deletion count: $deletions, and skip count: $skips");
     }
 }

app/Console/Commands/ResetDemoSettings.php

@@ -5,7 +5,6 @@ namespace App\Console\Commands;
 use App\Models\Setting;
 use App\Models\User;
 use Illuminate\Console\Command;
-use Illuminate\Support\Facades\Hash;
 
 class ResetDemoSettings extends Command
 {
@@ -48,7 +47,7 @@ class ResetDemoSettings extends Command
         $settings->auto_increment_assets = 1;
         $settings->logo = 'snipe-logo.png';
         $settings->alert_email = 'service@snipe-it.io';
-        $settings->login_note = "Use any of the following credentials to login to the demo:\n\n- `admin` / `password`\n- `assets` / `password`\n- `testuser` / `password`";
+        $settings->login_note = 'Use `admin` / `password` to login to the demo.';
         $settings->header_color = '#3c8dbc';
         $settings->link_dark_color = '#5fa4cc';
         $settings->link_light_color = '#296282;';
@@ -86,44 +85,6 @@ class ResetDemoSettings extends Command
             $user->save();
         }
 
-        $assetsUser = User::updateOrCreate(
-            ['username' => 'assets'],
-            [
-                'first_name' => 'Assets',
-                'last_name' => 'User',
-                'password' => Hash::make('password'),
-                'activated' => 1,
-            ]
-        );
-        $assetsUser->permissions = json_encode([
-            'assets.view' => 1,
-            'assets.create' => 1,
-            'assets.edit' => 1,
-            'assets.delete' => 1,
-            'assets.checkout' => 1,
-            'assets.checkin' => 1,
-            'assets.audit' => 1,
-            'assets.files' => 1,
-            'assets.view.requestable' => 1,
-            'assets.view.encrypted_custom_fields' => 1,
-        ]);
-        $assetsUser->save();
-
-        $testUser = User::updateOrCreate(
-            ['username' => 'testuser'],
-            [
-                'first_name' => 'Test',
-                'last_name' => 'User',
-                'password' => Hash::make('password'),
-                'activated' => 1,
-            ]
-        );
-        $testUser->permissions = json_encode([
-            'self.checkout_assets' => 1,
-            'assets.view.requestable' => 1,
-        ]);
-        $testUser->save();
-
         \Storage::disk('public')->put('snipe-logo.png', file_get_contents(public_path('img/demo/snipe-logo.png')));
         \Storage::disk('public')->put('snipe-logo-lg.png', file_get_contents(public_path('img/demo/snipe-logo-lg.png')));
 

app/Console/Commands/Version.php

@@ -4,9 +4,6 @@ namespace App\Console\Commands;
 
 use Illuminate\Console\Command;
 
-use function Laravel\Prompts\info;
-use function Laravel\Prompts\select;
-
 class Version extends Command
 {
     /**
@@ -14,7 +11,7 @@ class Version extends Command
      *
      * @var string
      */
-    protected $signature = 'version:update';
+    protected $signature = 'version:update {--branch=master} {--type=patch}';
 
     /**
      * The console command description.
@@ -40,40 +37,30 @@ class Version extends Command
      */
     public function handle()
     {
-        $use_branch = select(
-            label: 'Which branch?',
-            options: ['master', 'develop'],
-            default: 'develop',
-        );
-
-        $use_type = select(
-            label: 'Which release type?',
-            options: [
-                'hash' => 'Hash bump',
-                'patch' => 'Patch release',
-                'minor' => 'Minor release',
-                'major' => 'Major release',
-                'pre-patch' => 'Pre-patch release',
-                'pre-minor' => 'Pre-minor release',
-                'pre-major' => 'Pre-major release',
-            ],
-            default: 'hash',
-            scroll: 7,
-        );
-
+        $use_branch = $this->option('branch');
+        $use_type = $this->option('type');
         $git_branch = trim(shell_exec('git rev-parse --abbrev-ref HEAD'));
         $build_version = trim(shell_exec('git rev-list --count '.$use_branch));
         $versionFile = 'config/version.php';
         $full_hash_version = str_replace("\n", '', shell_exec('git describe master --tags'));
 
         $version = explode('-', $full_hash_version);
-        $app_version = $version[0];
+        $app_version = $current_app_version = $version[0];
         $hash_version = (array_key_exists('2', $version)) ? $version[2] : '';
         $prerelease_version = '';
 
-        if (array_key_exists('3', $version)) {
-            $prerelease_version = $version[1];
-            $hash_version = $version[3];
+        $this->line('Branch is: '.$use_branch);
+        $this->line('Type is: '.$use_type);
+        $this->line('Current version is: '.$full_hash_version);
+
+        if (count($version) == 3) {
+            $this->line('This does not look like an alpha/beta release.');
+        } else {
+            if (array_key_exists('3', $version)) {
+                $this->line('The current version looks like a beta release.');
+                $prerelease_version = $version[1];
+                $hash_version = $version[3];
+            }
         }
 
         $app_version_raw = explode('.', $app_version);
@@ -87,52 +74,54 @@ class Version extends Command
             $patch = $app_version_raw[2];
         }
 
-        if ($use_type === 'major') {
+        if ($use_type == 'major') {
             $app_version = 'v'.($maj + 1).".$min.$patch";
-        } elseif ($use_type === 'minor') {
+        } elseif ($use_type == 'minor') {
             $app_version = 'v'."$maj.".($min + 1).".$patch";
-        } elseif ($use_type === 'pre-patch') {
-            $app_version = 'v'."$maj.$min.".($patch + 1).'-pre';
-        } elseif ($use_type === 'pre-minor') {
-            $app_version = 'v'."$maj.".($min + 1).'.0-pre';
-        } elseif ($use_type === 'pre-major') {
-            $app_version = 'v'.($maj + 1).'.0.0-pre';
-        } elseif ($use_type === 'patch') {
+        } elseif ($use_type == 'pre') {
+            $pre_raw = str_replace('beta', '', $prerelease_version);
+            $pre_raw = str_replace('alpha', '', $pre_raw);
+            $pre_raw = str_ireplace('rc', '', $pre_raw);
+            $pre_raw = $pre_raw++;
+            $this->line('Setting the pre-release to '.$prerelease_version.'-'.$pre_raw);
+            $app_version = 'v'."$maj.".($min + 1).".$patch";
+        } elseif ($use_type == 'patch') {
             $app_version = 'v'."$maj.$min.".($patch + 1);
+            // If nothing is passed, leave the version as it is, just increment the build
+        } else {
+            $app_version = 'v'."$maj.$min.".$patch;
         }
 
-        if ($use_branch === 'develop' && ! str_ends_with($app_version, '-pre')) {
+        // Determine if this tag already exists, or if this prior to a release
+        $this->line('Running: git rev-parse master '.$current_app_version);
+        // $pre_release = trim(shell_exec('git rev-parse '.$use_branch.' '.$current_app_version.' 2>&1 1> /dev/null'));
+
+        if ($use_branch == 'develop') {
             $app_version = $app_version.'-pre';
         }
 
-        $full_hash_version = str_replace($version[0], $app_version, $full_hash_version);
         $full_app_version = $app_version.' - build '.$build_version.'-'.$hash_version;
 
-        $content = <<<PHP
-        <?php
+        $array = var_export(
+            [
+                'app_version' => $app_version,
+                'full_app_version' => $full_app_version,
+                'build_version' => $build_version,
+                'prerelease_version' => $prerelease_version,
+                'hash_version' => $hash_version,
+                'full_hash' => $full_hash_version,
+                'branch' => $git_branch, ],
+            true
+        );
 
-        return [
-            'app_version' => '$app_version',
-            'full_app_version' => '$full_app_version',
-            'build_version' => '$build_version',
-            'prerelease_version' => '$prerelease_version',
-            'hash_version' => '$hash_version',
-            'full_hash' => '$full_hash_version',
-            'branch' => '$git_branch',
-        ];
-
-        PHP;
+        // Construct our file content
+        $content = <<<CON
+<?php
+return $array;
+CON;
 
+        // And finally write the file and output the current version
         \File::put($versionFile, $content);
-        info('New version: '.$full_app_version.' ('.$git_branch.')');
-
-        info('Building JS/CSS assets...');
-        passthru('npm run prod', $exitCode);
-
-        if ($exitCode !== 0) {
-            $this->error('Asset build failed with exit code '.$exitCode);
-        } else {
-            info('Assets built successfully.');
-        }
+        $this->info('Setting NEW version: '.$full_app_version.' ('.$git_branch.')');
     }
 }

app/Enums/ActionType.php

@@ -31,9 +31,6 @@ enum ActionType: string
     case DeleteSeats = 'delete seats';
     case AddSeats = 'add seats';
 
-    // Maintenances
-    case MaintenanceComplete = 'completed';
-
     // File Uploads
     case Uploaded = 'uploaded';
     case UploadDeleted = 'upload deleted';

app/Exceptions/Handler.php

@@ -19,8 +19,6 @@ use Illuminate\Validation\ValidationException;
 use Intervention\Image\Exception\NotSupportedException;
 use JsonException;
 use League\OAuth2\Server\Exception\OAuthServerException;
-use Livewire\Exceptions\ComponentNotFoundException;
-use Livewire\Exceptions\PublicPropertyNotFoundException;
 use Symfony\Component\HttpKernel\Exception\HttpException;
 use Throwable;
 
@@ -43,8 +41,6 @@ class Handler extends ExceptionHandler
         JsonException::class,
         SCIMException::class, // these generally don't need to be reported
         InvalidFormatException::class,
-        PublicPropertyNotFoundException::class,
-        ComponentNotFoundException::class,
     ];
 
     /**
@@ -75,17 +71,6 @@ class Handler extends ExceptionHandler
     public function render($request, Throwable $e)
     {
 
-        // Livewire tried to set a property that doesn't exist (e.g. stale browser state sending a bare "0" as a property name)
-        if ($e instanceof PublicPropertyNotFoundException) {
-            return response()->json(['message' => $e->getMessage()], 422);
-        }
-
-        // A request named a Livewire component that doesn't exist in this app (e.g. bots probing
-        // for Filament endpoints). Return 404 so it doesn't surface as a 500.
-        if ($e instanceof ComponentNotFoundException) {
-            return response()->json(['message' => 'Component not found.'], 404);
-        }
-
         // CSRF token mismatch error
         if ($e instanceof TokenMismatchException) {
             return redirect()->back()->with('error', trans('general.token_expired'));

app/Helpers/Helper.php

@@ -14,7 +14,6 @@ use App\Models\License;
 use App\Models\Location;
 use App\Models\Setting;
 use App\Models\Statuslabel;
-use App\Models\User;
 use Carbon\Carbon;
 use Illuminate\Contracts\Encryption\DecryptException;
 use Illuminate\Http\RedirectResponse;
@@ -664,7 +663,7 @@ class Helper
      */
     public static function depreciationList()
     {
-        $depreciation_list = ['' => trans('admin/licenses/form.no_depreciation')] + Depreciation::orderBy('name', 'asc')
+        $depreciation_list = ['' => 'Do Not Depreciate'] + Depreciation::orderBy('name', 'asc')
             ->pluck('name', 'id')->toArray();
 
         return $depreciation_list;
@@ -1269,7 +1268,6 @@ class Helper
         $allowedExtensionMap = [
             // Images
             'jpg' => 'far fa-image',
-            'jfif' => 'far fa-image',
             'jpeg' => 'far fa-image',
             'gif' => 'far fa-image',
             'png' => 'far fa-image',
@@ -1598,17 +1596,7 @@ class Helper
         $checkout_to_type = session('checkout_to_type') ?? null;
         $checkedInFrom = session('checkedInFrom');
         $other_redirect = session('other_redirect');
-        $backUrl = str_replace(["\r", "\n"], '', session()->pull('url.intended', 'home'));
-
-        // Reject any stored back-URL that points off-site. redirect()->intended() performs
-        // no host validation, and url.intended can be written from the SAML RelayState POST
-        // parameter (SamlController), which an attacker-controlled IdP could set to an
-        // off-site URL.
-        $backHost = parse_url($backUrl, PHP_URL_HOST);
-        $appHost = parse_url(config('app.url'), PHP_URL_HOST);
-        if ($backHost && $backHost !== $appHost) {
-            $backUrl = route('home');
-        }
+        $backUrl = session()->pull('url.intended', 'home');
 
         // return to previous page
         if ($redirect_option == 'back') {
@@ -1701,8 +1689,6 @@ class Helper
             return [];
         }
 
-        $floater = (bool) Setting::getSettings()->null_company_is_floater;
-
         foreach ($locations as $location) {
             // in case of an update of a single location, use the newly requested company_id
             if ($new_company_id) {
@@ -1737,51 +1723,26 @@ class Helper
                 foreach ($keywords as $keyword) {
                     if ($relation == 'many') {
                         $items = $location->{$keyword}->all();
-                        // assignedAccessories returns AccessoryCheckout records (no company_id);
-                        // resolve each to its parent Accessory so the comparison is valid.
-                        if ($keyword === 'assignedAccessories') {
-                            $items = collect($items)->map(fn ($checkout) => $checkout->accessory)->filter()->values()->all();
-                        }
                     } else {
                         $items = collect([])->push($location->$keyword);
                     }
 
                     $count = 0;
                     foreach ($items as $item) {
-                        if (! $item) {
-                            continue;
-                        }
 
-                        // Users belong to companies via the many-to-many pivot (company_user).
-                        // canReceiveFromCompany() returns true only when the user's pivot
-                        // contains the location's company, so !canReceiveFromCompany() is
-                        // the correct mismatch signal.
-                        if ($item instanceof User) {
-                            $isMismatch = ! $item->canReceiveFromCompany((int) $location_company);
-                        } elseif ($item->company_id == $location_company) {
-                            $isMismatch = false;
-                        } elseif (is_null($item->company_id) || is_null($location_company)) {
-                            $isMismatch = ! $floater;
-                        } else {
-                            $isMismatch = true;
-                        }
-
-                        if ($isMismatch) {
-                            if ($item instanceof User) {
-                                $itemCompanyIds = $item->companies->pluck('id')->implode(', ');
-                                $itemCompanyNames = $item->companies->pluck('name')->implode(', ');
-                            } else {
-                                $itemCompanyIds = $item->company_id ?? null;
-                                $itemCompanyNames = $item->company->name ?? null;
-                            }
+                        if ($item && $item->company_id != $location_company) {
 
                             $mismatched[] = [
                                 class_basename(get_class($item)),
                                 $item->id,
                                 $item->name ?? $item->asset_tag ?? $item->serial ?? $item->username,
                                 $item->assigned_type ? str_replace('App\\Models\\', '', $item->assigned_type) : null,
-                                $itemCompanyIds,
-                                $itemCompanyNames,
+                                $item->company_id ?? null,
+                                $item->company->name ?? null,
+                                //                                    $item->defaultLoc->id ?? null,
+                                //                                    $item->defaultLoc->name ?? null,
+                                //                                    $item->defaultLoc->company->id ?? null,
+                                //                                    $item->defaultLoc->company->name ?? null,
                                 $item->location->name ?? null,
                                 $item->location->company->name ?? null,
                                 $location_company ?? null,
@@ -1895,43 +1856,4 @@ class Helper
         return 'App\\Models\\'.ucwords($model);
 
     }
-
-    /**
-     * Render a markdown-textarea value as HTML.
-     *
-     * Soft line breaks (single newlines) are rendered as <br> so that line
-     * breaks typed in the textarea are preserved in the output.
-     *
-     * When $inline is true, block-level elements are suppressed and hard
-     * breaks are pre-processed manually — used for the encrypted reveal span
-     * where block HTML cannot be placed inside a font-size-toggled <span>.
-     */
-    public static function renderMarkdown(?string $text, bool $inline = false): string
-    {
-        if (empty($text)) {
-            return '';
-        }
-
-        if ($inline) {
-            // Convert newlines to CommonMark hard breaks for inline rendering
-            $text = preg_replace('/(?<! {2})\n/', "  \n", $text);
-
-            return Str::inlineMarkdown($text, ['html_input' => 'escape', 'allow_unsafe_links' => false]);
-        }
-
-        $html = trim(Str::markdown($text, [
-            'html_input' => 'escape',
-            'allow_unsafe_links' => false,
-            'renderer' => ['soft_break' => "<br>\n"],
-        ]));
-
-        // If the entire output is a single <p> block, unwrap it so the content
-        // renders inline-ish without the <p> adding unwanted top spacing in the
-        // compact detail-view layout.
-        if (str_starts_with($html, '<p>') && str_ends_with($html, '</p>') && substr_count($html, '<p>') === 1) {
-            return substr($html, 3, -4);
-        }
-
-        return $html;
-    }
 }

app/Http/Controllers/Accessories/AccessoryCheckoutController.php

@@ -66,20 +66,6 @@ class AccessoryCheckoutController extends Controller
         $target = $this->determineCheckoutTarget();
         session()->put(['checkout_to_type' => $target]);
 
-        if (! $accessory->canCheckoutTo($target)) {
-            $targetType = match (class_basename($target)) {
-                'User' => trans('general.user'),
-                'Location' => trans('general.location'),
-                default => trans('general.asset'),
-            };
-
-            return redirect()->back()->with('error', trans('general.error_checkout_company_mismatch', [
-                'item' => trans('general.accessory').' "'.$accessory->name.'"',
-                'item_company' => $accessory->company?->name ?? trans('general.unassigned'),
-                'target' => $targetType.' "'.($target->name ?? $target->username ?? $target->id).'"',
-            ]));
-        }
-
         $accessory->checkout_qty = $request->input('checkout_qty', 1);
 
         for ($i = 0; $i < $accessory->checkout_qty; $i++) {

app/Http/Controllers/ActionlogController.php

@@ -4,6 +4,7 @@ namespace App\Http\Controllers;
 
 use App\Helpers\Helper;
 use App\Models\Actionlog;
+use App\Models\Asset;
 use Illuminate\Http\RedirectResponse;
 use Illuminate\Http\Response;
 use Illuminate\Support\Facades\Log;
@@ -16,9 +17,6 @@ class ActionlogController extends Controller
     {
         $filename = basename((string) $filename);
 
-        $actionlog = Actionlog::where('accept_signature', $filename)->with('item')->firstOrFail();
-        $this->authorize('view', $actionlog->item);
-
         // PHP doesn't let you handle file not found errors well with
         // file_get_contents, so we set the error reporting for just this class
         error_reporting(0);
@@ -31,6 +29,7 @@ class ActionlogController extends Controller
 
                 return redirect()->away(Storage::disk($disk)->temporaryUrl($file, now()->addMinutes(5)));
             default:
+                $this->authorize('view', Asset::class);
                 $file = config('app.private_uploads').'/signatures/'.$filename;
                 $filetype = Helper::checkUploadIsImage($file);
 

app/Http/Controllers/Api/AccessoriesController.php

@@ -107,7 +107,7 @@ class AccessoriesController extends Controller
         }
 
         // Make sure the offset and limit are actually integers and do not exceed system limits
-        $offset = ($request->input('offset') > $accessories->count()) ? $accessories->count() : app('api_offset_value');
+        $offset = ($request->input('offset') > $accessories->count()) ? $accessories->count() : abs($request->input('offset'));
         $limit = app('api_limit_value');
 
         $order = $request->input('order') === 'asc' ? 'asc' : 'desc';
@@ -234,10 +234,6 @@ class AccessoriesController extends Controller
         $total = $accessory_checkouts->count();
         $accessory_checkouts = $accessory_checkouts->skip($offset)->take($limit)->get();
 
-        $accessory_checkouts->loadMorph('assignedTo', [
-            User::class => ['companies'],
-        ]);
-
         return (new AccessoriesTransformer)->transformCheckedoutAccessory($accessory_checkouts, $total);
     }
 
@@ -307,7 +303,7 @@ class AccessoriesController extends Controller
         $this->authorize('checkout', $accessory);
         $target = $this->determineCheckoutTarget();
 
-        if ((Setting::getSettings()->full_multiple_companies_support == '1') && (! $target->companies()->where('companies.id', $accessory->company_id)->exists())) {
+        if ((Setting::getSettings()->full_multiple_companies_support == '1') && ($accessory->company_id !== $target->company_id)) {
             return response()->json(Helper::formatStandardApiResponse('error', null, trans('general.error_user_company')));
         }
 
@@ -405,7 +401,6 @@ class AccessoriesController extends Controller
      */
     public function selectlist(Request $request)
     {
-        $this->authorize('view.selectlists');
 
         $accessories = Accessory::select([
             'accessories.id',

app/Http/Controllers/Api/AssetModelsController.php

@@ -133,8 +133,7 @@ class AssetModelsController extends Controller
         }
 
         // Make sure the offset and limit are actually integers and do not exceed system limits
-        $total = $assetmodels->count();
-        $offset = ($request->input('offset') > $total) ? $total : app('api_offset_value');
+        $offset = ($request->input('offset') > $assetmodels->count()) ? $assetmodels->count() : abs($request->input('offset'));
         $limit = app('api_limit_value');
 
         $order = $request->input('order') === 'asc' ? 'asc' : 'desc';
@@ -158,6 +157,7 @@ class AssetModelsController extends Controller
                 break;
         }
 
+        $total = $assetmodels->count();
         $assetmodels = $assetmodels->skip($offset)->take($limit)->get();
 
         return (new AssetModelsTransformer)->transformAssetModels($assetmodels, $total);

app/Http/Controllers/Api/AssetsController.php

@@ -371,12 +371,6 @@ class AssetsController extends Controller
             $assets->where('assets.order_number', '=', strval($request->input('order_number')));
         }
 
-        foreach ($all_custom_fields as $field) {
-            if ($request->filled($field->db_column_name())) {
-                $assets->where($field->db_column_name(), '=', $request->input($field->db_column_name()));
-            }
-        }
-
         // This is kinda gross, but we need to do this because the Bootstrap Tables
         // API passes custom field ordering as custom_fields.fieldname, and we have to strip
         // that out to let the default sorter below order them correctly on the assets table.
@@ -596,7 +590,6 @@ class AssetsController extends Controller
      */
     public function selectlist(Request $request): array
     {
-        $this->authorize('view.selectlists');
 
         $assets = Asset::select([
             'assets.id',
@@ -609,20 +602,8 @@ class AssetsController extends Controller
         ])->with('model', 'status', 'assignedTo')
             ->NotArchived();
 
-        // When FMCS is enabled, automatically scope to companies the acting user belongs to.
-        // scopeCompanyables is a no-op for superusers and when FMCS is disabled.
-        $assets = Company::scopeCompanyables($assets);
-
-        // Allow further narrowing to a specific company passed via data-company-id on the select.
-        if ((Setting::getSettings()->full_multiple_companies_support == '1') && $request->filled('companyId')) {
-            $companyIds = array_values(array_filter(array_map('intval', explode(',', $request->input('companyId')))));
-            if (! empty($companyIds)) {
-                $assets->whereIn('assets.company_id', $companyIds);
-            }
-        }
-
-        if ($request->filled('excludeId')) {
-            $assets->where('assets.id', '!=', (int) $request->input('excludeId'));
+        if ((Setting::getSettings()->full_multiple_companies_support == '1') && ($request->filled('companyId'))) {
+            $assets->where('assets.company_id', $request->input('companyId'));
         }
 
         if ($request->filled('statusType') && $request->input('statusType') === 'RTD') {
@@ -913,7 +894,11 @@ class AssetsController extends Controller
 
     private function checkoutCompanyMismatchResponse(Asset $asset, User|Asset|Location $target): ?JsonResponse
     {
-        if (! $asset->canCheckoutTo($target)) {
+        if ((Setting::getSettings()->full_multiple_companies_support == '1')
+            && (! is_null($asset->company_id))
+            && (! is_null($target->company_id))
+            && ((int) $asset->company_id !== (int) $target->company_id)
+        ) {
             return response()->json(Helper::formatStandardApiResponse('error', null, trans('general.error_user_company')));
         }
 
@@ -1067,8 +1052,13 @@ class AssetsController extends Controller
         }
 
         // In FMCS mode, enforce explicit same-company target checks before mutating checkout state.
-        if ($mismatch = $this->checkoutCompanyMismatchResponse($asset, $target)) {
-            return $mismatch;
+        $targetCompanyId = data_get($target, 'company_id');
+        if ((Setting::getSettings()->full_multiple_companies_support == '1')
+            && (! is_null($asset->company_id))
+            && (! is_null($targetCompanyId))
+            && ((int) $asset->company_id !== (int) $targetCompanyId)
+        ) {
+            return response()->json(Helper::formatStandardApiResponse('error', $error_payload, trans('general.error_user_company')));
         }
 
         $checkout_at = request('checkout_at', date('Y-m-d H:i:s'));
@@ -1126,9 +1116,7 @@ class AssetsController extends Controller
         $asset->assignedTo()->disassociate($asset);
         $asset->accepted = null;
 
-        if ($request->input('clear_name') == '1') {
-            $asset->name = null;
-        } elseif ($request->has('name')) {
+        if ($request->has('name')) {
             $asset->name = $request->input('name');
         }
 
@@ -1271,10 +1259,6 @@ class AssetsController extends Controller
 
             $asset->last_audit_date = date('Y-m-d H:i:s');
 
-            if ($request->input('clear_name') == '1') {
-                $asset->name = null;
-            }
-
             // Set up the payload for re-display in the API response
             $payload = [
                 'id' => $asset->id,

app/Http/Controllers/Api/CompaniesController.php

@@ -9,7 +9,6 @@ use App\Http\Requests\ImageUploadRequest;
 use App\Http\Transformers\CompaniesTransformer;
 use App\Http\Transformers\SelectlistTransformer;
 use App\Models\Company;
-use App\Models\Setting;
 use Illuminate\Http\JsonResponse;
 use Illuminate\Http\Request;
 use Illuminate\Support\Facades\Storage;
@@ -207,16 +206,6 @@ class CompaniesController extends Controller
             'companies.tag_color',
         ]);
 
-        // When FMCS is enabled and the user is not a superuser, restrict the list to
-        // companies they belong to (primary company_id + pivot companies). This lets
-        // non-superusers select a company from their own set when creating assets, etc.
-        if (Setting::getSettings()->full_multiple_companies_support == '1' && ! auth()->user()->isSuperUser()) {
-            $userCompanyIds = auth()->user()->allCompanies()->pluck('id');
-            if ($userCompanyIds->isNotEmpty()) {
-                $companies->whereIn('companies.id', $userCompanyIds);
-            }
-        }
-
         if ($request->filled('search')) {
             $companies = $companies->where('companies.name', 'LIKE', '%'.$request->input('search').'%');
         }

app/Http/Controllers/Api/ConsumablesController.php

@@ -315,7 +315,7 @@ class ConsumablesController extends Controller
             return response()->json(Helper::formatStandardApiResponse('error', null, 'No user found'));
         }
 
-        if ((Setting::getSettings()->full_multiple_companies_support == '1') && (! $user->companies()->where('companies.id', $consumable->company_id)->exists())) {
+        if ((Setting::getSettings()->full_multiple_companies_support == '1') && ($consumable->company_id !== $user->company_id)) {
             return response()->json(Helper::formatStandardApiResponse('error', null, trans('general.error_user_company')));
         }
 
@@ -356,8 +356,6 @@ class ConsumablesController extends Controller
      */
     public function selectlist(Request $request): array
     {
-        $this->authorize('view.selectlists');
-
         $consumables = Consumable::select([
             'consumables.id',
             'consumables.name',

app/Http/Controllers/Api/LicenseSeatsController.php

@@ -27,7 +27,7 @@ class LicenseSeatsController extends Controller
         if ($license = License::find($licenseId)) {
             $this->authorize('view', $license);
 
-            $seats = LicenseSeat::with('license', 'user', 'asset', 'user.department', 'user.companies', 'asset.company')
+            $seats = LicenseSeat::with('license', 'user', 'asset', 'user.department', 'user.company', 'asset.company')
                 ->where('license_seats.license_id', $licenseId);
 
             if ($request->input('status') == 'available') {
@@ -132,110 +132,91 @@ class LicenseSeatsController extends Controller
 
         $this->authorize('checkout', License::class);
 
-        $errorResponse = null;
-        $updatedSeat = null;
+        $licenseSeat = LicenseSeat::with(['license', 'asset', 'user'])->find($seatId);
 
-        // Fetch the seat with a pessimistic lock inside a transaction so concurrent requests
-        // on the same seat serialise rather than racing to overwrite each other's assignment.
-        DB::transaction(function () use ($request, $licenseId, $seatId, $validated, &$errorResponse, &$updatedSeat): void {
-            $licenseSeat = LicenseSeat::with(['license', 'asset', 'user'])
-                ->lockForUpdate()
-                ->find($seatId);
+        if (! $licenseSeat) {
+            return response()->json(Helper::formatStandardApiResponse('error', null, 'Seat not found'));
+        }
 
-            if (! $licenseSeat) {
-                $errorResponse = response()->json(Helper::formatStandardApiResponse('error', null, 'Seat not found'));
+        $license = $licenseSeat->license;
+        if (! $license || $license->id != intval($licenseId)) {
+            return response()->json(Helper::formatStandardApiResponse('error', null, 'Seat does not belong to the specified license'));
+        }
 
-                return;
+        $targetUser = null;
+        if (! is_null($request->input('assigned_to'))) {
+            // Resolve unscoped target so we can return a clean cross-company error instead of a hidden-not-found.
+            $targetUser = User::withoutGlobalScopes()->find($request->input('assigned_to'));
+
+            if (! $targetUser) {
+                return response()->json(Helper::formatStandardApiResponse('error', null, 'Target not found'));
             }
 
-            $license = $licenseSeat->license;
-            if (! $license || $license->id != intval($licenseId)) {
-                $errorResponse = response()->json(Helper::formatStandardApiResponse('error', null, 'Seat does not belong to the specified license'));
+            if ((Setting::getSettings()->full_multiple_companies_support == '1') && ($license->company_id !== $targetUser->company_id)) {
+                return response()->json(Helper::formatStandardApiResponse('error', null, trans('general.error_user_company')));
+            }
+        }
 
-                return;
+        $targetAsset = null;
+        if (! is_null($request->input('asset_id'))) {
+            // Resolve unscoped target so FMCS company mismatch can be enforced explicitly.
+            $targetAsset = Asset::withoutGlobalScopes()->find($request->input('asset_id'));
+
+            if (! $targetAsset) {
+                return response()->json(Helper::formatStandardApiResponse('error', null, 'Target not found'));
             }
 
-            $targetUser = null;
-            if (! is_null($request->input('assigned_to'))) {
-                // Resolve unscoped target so we can return a clean cross-company error instead of a hidden-not-found.
-                $targetUser = User::withoutGlobalScopes()->find($request->input('assigned_to'));
-
-                if (! $targetUser) {
-                    $errorResponse = response()->json(Helper::formatStandardApiResponse('error', null, 'Target not found'));
-
-                    return;
-                }
-
-                if ((Setting::getSettings()->full_multiple_companies_support == '1') && (! $targetUser->companies()->where('companies.id', $license->company_id)->exists())) {
-                    $errorResponse = response()->json(Helper::formatStandardApiResponse('error', null, trans('general.error_user_company')));
-
-                    return;
-                }
+            if ((Setting::getSettings()->full_multiple_companies_support == '1') && ($license->company_id !== $targetAsset->company_id)) {
+                return response()->json(Helper::formatStandardApiResponse('error', null, trans('general.error_user_company')));
             }
+        }
 
-            $targetAsset = null;
-            if (! is_null($request->input('asset_id'))) {
-                // Resolve unscoped target so FMCS company mismatch can be enforced explicitly.
-                $targetAsset = Asset::withoutGlobalScopes()->find($request->input('asset_id'));
+        $oldUser = $licenseSeat->user;
+        $oldAsset = $licenseSeat->asset;
 
-                if (! $targetAsset) {
-                    $errorResponse = response()->json(Helper::formatStandardApiResponse('error', null, 'Target not found'));
+        // attempt to update the license seat
+        $licenseSeat->fill($validated);
 
-                    return;
-                }
+        // check if this update is a checkin operation
+        // 1. are relevant fields touched at all?
+        $assignmentTouched = $licenseSeat->isDirty('assigned_to') || $licenseSeat->isDirty('asset_id');
+        $anythingTouched = $licenseSeat->isDirty();
 
-                if ((Setting::getSettings()->full_multiple_companies_support == '1') && ($license->company_id !== $targetAsset->company_id)) {
-                    $errorResponse = response()->json(Helper::formatStandardApiResponse('error', null, trans('general.error_user_company')));
+        if (! $anythingTouched) {
+            return response()->json(
+                Helper::formatStandardApiResponse('success', $licenseSeat, trans('admin/licenses/message.update.success'))
+            );
+        }
+        if ($assignmentTouched && $licenseSeat->unreassignable_seat) {
+            return response()->json(Helper::formatStandardApiResponse('error', null, trans('admin/licenses/message.checkout.unavailable')));
+        }
 
-                    return;
-                }
-            }
-
-            $oldUser = $licenseSeat->user;
-            $oldAsset = $licenseSeat->asset;
-
-            $licenseSeat->fill($validated);
-
-            $assignmentTouched = $licenseSeat->isDirty('assigned_to') || $licenseSeat->isDirty('asset_id');
-            $anythingTouched = $licenseSeat->isDirty();
-
-            if (! $anythingTouched) {
-                $updatedSeat = $licenseSeat;
-
-                return;
-            }
-
-            if ($assignmentTouched && $licenseSeat->unreassignable_seat) {
-                $errorResponse = response()->json(Helper::formatStandardApiResponse('error', null, trans('admin/licenses/message.checkout.unavailable')));
-
-                return;
-            }
-
-            // Are the assignment fields cleared? If yes, this is a checkin operation.
-            $is_checkin = ($assignmentTouched && $licenseSeat->assigned_to === null && $licenseSeat->asset_id === null);
-
-            // The logging functions expect only one "target"; assets take precedence over users.
-            $target = null;
-            if ($licenseSeat->isDirty('assigned_to')) {
-                $target = $is_checkin ? $oldUser : $targetUser;
-            }
-            if ($licenseSeat->isDirty('asset_id')) {
-                $target = $is_checkin ? $oldAsset : $targetAsset;
-            }
-
-            if ($assignmentTouched && is_null($target)) {
-                // Both fields are null but one was provided — the related model is purged or bad data.
-                if (! is_null($request->input('asset_id')) || ! is_null($request->input('assigned_to'))) {
-                    $errorResponse = response()->json(Helper::formatStandardApiResponse('error', null, 'Target not found'));
-
-                    return;
-                }
+        // 2. are they cleared? if yes then this is a checkin operation
+        $is_checkin = ($assignmentTouched && $licenseSeat->assigned_to === null && $licenseSeat->asset_id === null);
+        $target = null;
+
+        // the logging functions expect only one "target". if both asset and user are present in the request,
+        // we simply let assets take precedence over users...
+        if ($licenseSeat->isDirty('assigned_to')) {
+            $target = $is_checkin ? $oldUser : $targetUser;
+        }
+
+        if ($licenseSeat->isDirty('asset_id')) {
+            $target = $is_checkin ? $oldAsset : $targetAsset;
+        }
+
+        if ($assignmentTouched && is_null($target)) {
+            // if both asset_id and assigned_to are null then we are "checking-in"
+            // a related model that does not exist (possible purged or bad data).
+            if (! is_null($request->input('asset_id')) || ! is_null($request->input('assigned_to'))) {
+                return response()->json(Helper::formatStandardApiResponse('error', null, 'Target not found'));
             }
+        }
 
+        // Keep seat updates and checkout/checkin logging atomic to prevent partial state changes.
+        $updated = DB::transaction(function () use ($licenseSeat, $assignmentTouched, $is_checkin, $target, $request): bool {
             if (! $licenseSeat->save()) {
-                $errorResponse = response()->json(Helper::formatStandardApiResponse('error', null, $licenseSeat->getErrors()));
-
-                return;
+                return false;
             }
 
             if ($assignmentTouched) {
@@ -244,29 +225,25 @@ class LicenseSeatsController extends Controller
                         $licenseSeat->unreassignable_seat = true;
 
                         if (! $licenseSeat->save()) {
-                            $errorResponse = response()->json(Helper::formatStandardApiResponse('error', null, $licenseSeat->getErrors()));
-
-                            return;
+                            return false;
                         }
                     }
 
+                    // todo: skip if target is null?
                     $licenseSeat->logCheckin($target, $licenseSeat->notes);
                 } else {
+                    // in this case, relevant fields are touched but it's not a checkin operation. so it must be a checkout operation.
                     $licenseSeat->logCheckout($request->input('notes'), $target);
                 }
             }
 
-            $updatedSeat = $licenseSeat;
+            return true;
         });
 
-        if ($errorResponse) {
-            return $errorResponse;
+        if ($updated) {
+            return response()->json(Helper::formatStandardApiResponse('success', $licenseSeat, trans('admin/licenses/message.update.success')));
         }
 
-        if ($updatedSeat) {
-            return response()->json(Helper::formatStandardApiResponse('success', $updatedSeat, trans('admin/licenses/message.update.success')));
-        }
-
-        return response()->json(Helper::formatStandardApiResponse('error', null, 'An unexpected error occurred'), 500);
+        return Helper::formatStandardApiResponse('error', null, $licenseSeat->getErrors());
     }
 }

app/Http/Controllers/Api/LicensesController.php

@@ -2,21 +2,15 @@
 
 namespace App\Http\Controllers\Api;
 
-use App\Events\CheckoutableCheckedIn;
-use App\Events\CheckoutableCheckedOut;
 use App\Helpers\Helper;
 use App\Http\Controllers\Controller;
 use App\Http\Requests\FilterRequest;
 use App\Http\Transformers\ActionlogsTransformer;
-use App\Http\Transformers\LicenseSeatsTransformer;
 use App\Http\Transformers\LicensesTransformer;
 use App\Http\Transformers\SelectlistTransformer;
-use App\Models\Asset;
 use App\Models\Company;
 use App\Models\License;
-use App\Models\LicenseSeat;
 use App\Models\Setting;
-use App\Models\User;
 use Illuminate\Http\JsonResponse;
 use Illuminate\Http\Request;
 use Illuminate\Support\Facades\DB;
@@ -267,167 +261,6 @@ class LicensesController extends Controller
         return response()->json(Helper::formatStandardApiResponse('error', null, trans('admin/licenses/message.assoc_users')));
     }
 
-    /**
-     * Checkout a license seat to a user or asset.
-     *
-     * Accepts an optional `seat_id`; if omitted the next available free seat is used.
-     * `target_type` must be "user" or "asset". Supply `assigned_to` for users or
-     * `asset_id` for assets.
-     *
-     * This will eventually use the same form request the UI uses, but we need to update the field names first.
-     *
-     * @param  int  $licenseId
-     */
-    public function checkout(Request $request, $licenseId): JsonResponse
-    {
-        $license = License::findOrFail($licenseId);
-        $this->authorize('checkout', $license);
-
-        $validated = $this->validate($request, [
-            'seat_id' => 'sometimes|integer|nullable',
-            'target_type' => 'required|in:user,asset',
-            'assigned_to' => 'required_if:target_type,user|integer|nullable',
-            'asset_id' => 'required_if:target_type,asset|integer|nullable',
-            'notes' => 'sometimes|string|nullable',
-        ]);
-
-        if ($license->isInactive()) {
-            return response()->json(Helper::formatStandardApiResponse('error', null, trans('admin/licenses/message.checkout.license_is_inactive')));
-        }
-
-        $errorResponse = null;
-        $updatedSeat = null;
-        $target = null;
-
-        DB::transaction(function () use ($license, $validated, &$errorResponse, &$updatedSeat, &$target): void {
-            $seatId = $validated['seat_id'] ?? null;
-
-            $licenseSeat = $seatId
-                ? LicenseSeat::where('id', $seatId)->where('license_id', $license->id)->lockForUpdate()->first()
-                : $license->freeSeat(lock: true);
-
-            if (! $licenseSeat) {
-                $errorResponse = response()->json(Helper::formatStandardApiResponse('error', null, trans('admin/licenses/message.checkout.not_enough_seats')));
-
-                return;
-            }
-
-            if ($licenseSeat->unreassignable_seat) {
-                $errorResponse = response()->json(Helper::formatStandardApiResponse('error', null, trans('admin/licenses/message.checkout.unavailable')));
-
-                return;
-            }
-
-            if ($validated['target_type'] === 'user') {
-                $target = User::withoutGlobalScopes()->whereNull('deleted_at')->find($validated['assigned_to'] ?? null);
-                if (! $target) {
-                    $errorResponse = response()->json(Helper::formatStandardApiResponse('error', null, trans('admin/licenses/message.user_does_not_exist')));
-
-                    return;
-                }
-
-                if (Company::isFullMultipleCompanySupportEnabled() && ! $target->companies()->where('companies.id', $license->company_id)->exists()) {
-                    $errorResponse = response()->json(Helper::formatStandardApiResponse('error', null, trans('general.error_user_company')));
-
-                    return;
-                }
-
-                $licenseSeat->assigned_to = $target->id;
-                $licenseSeat->asset_id = null;
-            } else {
-                $target = Asset::withoutGlobalScopes()->whereNull('deleted_at')->find($validated['asset_id'] ?? null);
-                if (! $target) {
-                    $errorResponse = response()->json(Helper::formatStandardApiResponse('error', null, trans('admin/licenses/message.asset_does_not_exist')));
-
-                    return;
-                }
-
-                if (Company::isFullMultipleCompanySupportEnabled() && $license->company_id && $license->company_id !== $target->company_id) {
-                    $errorResponse = response()->json(Helper::formatStandardApiResponse('error', null, trans('general.error_user_company')));
-
-                    return;
-                }
-
-                $licenseSeat->asset_id = $target->id;
-                $licenseSeat->assigned_to = null;
-
-                if ($target->checkedOutToUser()) {
-                    $licenseSeat->assigned_to = $target->assigned_to;
-                }
-            }
-
-            $licenseSeat->notes = $validated['notes'] ?? null;
-            $licenseSeat->created_by = auth()->id();
-
-            if (! $licenseSeat->save()) {
-                $errorResponse = response()->json(Helper::formatStandardApiResponse('error', null, $licenseSeat->getErrors()));
-
-                return;
-            }
-
-            event(new CheckoutableCheckedOut($licenseSeat, $target, auth()->user(), $validated['notes'] ?? null));
-            $updatedSeat = $licenseSeat->load('license', 'user', 'asset');
-        });
-
-        if ($errorResponse) {
-            return $errorResponse;
-        }
-
-        if ($updatedSeat) {
-            return response()->json(Helper::formatStandardApiResponse('success', (new LicenseSeatsTransformer)->transformLicenseSeat($updatedSeat), trans('admin/licenses/message.checkout.success')));
-        }
-
-        return response()->json(Helper::formatStandardApiResponse('error', null, 'An unexpected error occurred'), 500);
-    }
-
-    /**
-     * Checkin a license seat.
-     *
-     * `seat_id` is required to identify which seat to check back in.
-     *
-     * @param  int  $licenseId
-     */
-    public function checkin(Request $request, $licenseId): JsonResponse
-    {
-        $license = License::findOrFail($licenseId);
-        $this->authorize('checkin', $license);
-
-        $validated = $this->validate($request, [
-            'seat_id' => 'required|integer',
-            'notes' => 'sometimes|string|nullable',
-        ]);
-
-        $licenseSeat = LicenseSeat::where('id', $validated['seat_id'])
-            ->where('license_id', $license->id)
-            ->first();
-
-        if (! $licenseSeat) {
-            return response()->json(Helper::formatStandardApiResponse('error', null, trans('admin/licenses/message.not_found')));
-        }
-
-        if (is_null($licenseSeat->assigned_to) && is_null($licenseSeat->asset_id)) {
-            return response()->json(Helper::formatStandardApiResponse('error', null, trans('admin/licenses/message.checkin.error')));
-        }
-
-        $target = $licenseSeat->user ?? $licenseSeat->asset;
-
-        $licenseSeat->assigned_to = null;
-        $licenseSeat->asset_id = null;
-        $licenseSeat->notes = $validated['notes'] ?? null;
-
-        if (! $license->reassignable) {
-            $licenseSeat->unreassignable_seat = true;
-        }
-
-        if (! $licenseSeat->save()) {
-            return response()->json(Helper::formatStandardApiResponse('error', null, $licenseSeat->getErrors()));
-        }
-
-        event(new CheckoutableCheckedIn($licenseSeat, $target, auth()->user(), $licenseSeat->notes));
-
-        return response()->json(Helper::formatStandardApiResponse('success', (new LicenseSeatsTransformer)->transformLicenseSeat($licenseSeat->load('license', 'user', 'asset')), trans('admin/licenses/message.checkin.success')));
-    }
-
     /**
      * Gets a paginated collection for the select2 menus
      *
@@ -435,8 +268,6 @@ class LicensesController extends Controller
      */
     public function selectlist(Request $request): array
     {
-        $this->authorize('view.selectlists');
-
         $licenses = License::select([
             'licenses.id',
             'licenses.name',

app/Http/Controllers/Api/LocationsController.php

@@ -67,18 +67,7 @@ class LocationsController extends Controller
             'notes',
         ];
 
-        $locations = Location::with([
-            'parent',
-            'children',
-            'manager' => fn ($q) => $q->withCount([
-                'assets as assets_count',
-                'accessories as accessories_count',
-                'licenses as licenses_count',
-                'consumables as consumables_count',
-                'managesUsers as manages_users_count',
-                'managedLocations as manages_locations_count',
-            ]),
-        ])->select([
+        $locations = Location::with('parent', 'manager', 'children')->select([
             'locations.id',
             'locations.name',
             'locations.address',
@@ -114,9 +103,7 @@ class LocationsController extends Controller
             ->withCount('components as components_count')
             ->with('adminuser');
 
-        // scope_locations_fmcs is required for location-level company scoping (locations may not
-        // have company_id assigned unless the compatibility check has been completed in Settings).
-        // Without it, locations are visible to all authenticated users regardless of FMCS state.
+        // Only scope locations if the setting is enabled
         if (Setting::getSettings()->scope_locations_fmcs) {
             $locations = Company::scopeCompanyables($locations);
         }
@@ -170,6 +157,8 @@ class LocationsController extends Controller
             $locations->where('tag_color', '=', $request->input('locations.tag_color'));
         }
 
+        // Make sure the offset and limit are actually integers and do not exceed system limits
+        $offset = ($request->input('offset') > $locations->count()) ? $locations->count() : app('api_offset_value');
         $limit = app('api_limit_value');
 
         $order = $request->input('order') === 'asc' ? 'asc' : 'desc';
@@ -191,7 +180,6 @@ class LocationsController extends Controller
         }
 
         $total = $locations->count();
-        $offset = ($request->input('offset') > $total) ? $total : app('api_offset_value');
         $locations = $locations->skip($offset)->take($limit)->get();
 
         return (new LocationsTransformer)->transformLocations($locations, $total);
@@ -211,19 +199,12 @@ class LocationsController extends Controller
         $location->fill($request->all());
         $location = $request->handleImages($location);
 
+        // Only scope location if the setting is enabled
         if (Setting::getSettings()->scope_locations_fmcs) {
             $location->company_id = Company::getIdForCurrentUser($request->input('company_id'));
-        }
-
-        // Parent company check applies whenever FMCS is on, independent of scope_locations_fmcs.
-        if (Setting::getSettings()->full_multiple_companies_support) {
-            $parent = $location->parent_id ? Location::find($location->parent_id) : null;
-            if ($parent && $parent->company_id != $location->company_id) {
-                return response()->json(Helper::formatStandardApiResponse('error', null, trans('general.error_location_parent_company', [
-                    'parent' => $parent->name,
-                    'parent_company' => $parent->company?->name ?? trans('general.unassigned'),
-                    'location_company' => $location->company?->name ?? trans('general.unassigned'),
-                ])));
+            // check if parent is set and has a different company
+            if ($location->parent_id && Location::find($location->parent_id)->company_id != $location->company_id) {
+                response()->json(Helper::formatStandardApiResponse('error', null, 'different company than parent'));
             }
         }
 
@@ -246,19 +227,7 @@ class LocationsController extends Controller
     public function show($id): JsonResponse|array
     {
         $this->authorize('view', Location::class);
-        $location = Location::with([
-            'parent',
-            'children',
-            'company',
-            'manager' => fn ($q) => $q->withCount([
-                'assets as assets_count',
-                'accessories as accessories_count',
-                'licenses as licenses_count',
-                'consumables as consumables_count',
-                'managesUsers as manages_users_count',
-                'managedLocations as manages_locations_count',
-            ]),
-        ])
+        $location = Location::with('parent', 'manager', 'children', 'company')
             ->select([
                 'locations.id',
                 'locations.name',
@@ -310,36 +279,18 @@ class LocationsController extends Controller
         $location = $request->handleImages($location);
 
         if ($request->filled('company_id')) {
+            // Only scope location if the setting is enabled
             if (Setting::getSettings()->scope_locations_fmcs) {
                 $location->company_id = Company::getIdForCurrentUser($request->input('company_id'));
                 // check if there are related objects with different company
-                if ($mismatched = Helper::test_locations_fmcs(false, $id, $location->company_id)) {
-                    $first = $mismatched[0];
-
-                    return response()->json(Helper::formatStandardApiResponse('error', null, trans('general.error_location_scoped_items', [
-                        'item_type' => trans('general.'.strtolower($first[0])),
-                        'item_name' => $first[2],
-                        'item_company' => $first[5] ?? trans('general.unassigned'),
-                    ])));
+                if (Helper::test_locations_fmcs(false, $id, $location->company_id)) {
+                    return response()->json(Helper::formatStandardApiResponse('error', null, 'error scoped locations'));
                 }
             } else {
                 $location->company_id = $request->input('company_id');
             }
         }
 
-        // Parent company check applies whenever FMCS is on, independent of scope_locations_fmcs.
-        // Runs outside the company_id gate so a parent_id-only update is also validated.
-        if (Setting::getSettings()->full_multiple_companies_support) {
-            $parent = $location->parent_id ? Location::find($location->parent_id) : null;
-            if ($parent && $parent->company_id != $location->company_id) {
-                return response()->json(Helper::formatStandardApiResponse('error', null, trans('general.error_location_parent_company', [
-                    'parent' => $parent->name,
-                    'parent_company' => $parent->company?->name ?? trans('general.unassigned'),
-                    'location_company' => $location->company?->name ?? trans('general.unassigned'),
-                ])));
-            }
-        }
-
         if ($location->isValid()) {
 
             $location->save();
@@ -471,6 +422,11 @@ class LocationsController extends Controller
             'locations.tag_color',
         ]);
 
+        // Only scope locations if the setting is enabled
+        if (Setting::getSettings()->scope_locations_fmcs) {
+            $locations = Company::scopeCompanyables($locations);
+        }
+
         $page = 1;
         if ($request->filled('page')) {
             $page = $request->input('page');
@@ -480,10 +436,6 @@ class LocationsController extends Controller
             $locations = $locations->where('locations.name', 'LIKE', '%'.$request->input('search').'%');
         }
 
-        if ($request->filled('excludeId')) {
-            $locations->where('locations.id', '!=', (int) $request->input('excludeId'));
-        }
-
         $locations = $locations->orderBy('name', 'ASC')->get();
 
         $locations_with_children = [];

app/Http/Controllers/Api/MaintenanceTypesController.php

@@ -1,87 +0,0 @@
-<?php
-
-namespace App\Http\Controllers\Api;
-
-use App\Helpers\Helper;
-use App\Http\Controllers\Controller;
-use App\Http\Requests\FilterRequest;
-use App\Http\Transformers\MaintenanceTypesTransformer;
-use App\Models\MaintenanceType;
-use Illuminate\Http\JsonResponse;
-use Illuminate\Http\Request;
-
-class MaintenanceTypesController extends Controller
-{
-    public function index(FilterRequest $request): JsonResponse|array
-    {
-        $this->authorize('view', MaintenanceType::class);
-
-        $types = MaintenanceType::select(['id', 'name', 'created_at', 'updated_at', 'deleted_at']);
-
-        if ($request->input('deleted') == 'true') {
-            $types->onlyTrashed();
-        }
-
-        if ($request->filled('search')) {
-            $types->where('name', 'LIKE', '%'.$request->input('search').'%');
-        }
-
-        if ($request->filled('name')) {
-            $types->where('name', '=', $request->input('name'));
-        }
-
-        $offset = ($request->input('offset') > $types->count()) ? $types->count() : app('api_offset_value');
-        $limit = app('api_limit_value');
-        $order = $request->input('order') === 'asc' ? 'asc' : 'desc';
-        $sort = in_array($request->input('sort'), ['id', 'name', 'created_at', 'updated_at']) ? $request->input('sort') : 'name';
-
-        $total = $types->count();
-        $types = $types->orderBy($sort, $order)->skip($offset)->take($limit)->get();
-
-        return (new MaintenanceTypesTransformer)->transformMaintenanceTypes($types, $total);
-    }
-
-    public function show(MaintenanceType $maintenanceType): JsonResponse|array
-    {
-        $this->authorize('view', $maintenanceType);
-
-        return (new MaintenanceTypesTransformer)->transformMaintenanceType($maintenanceType);
-    }
-
-    public function store(Request $request): JsonResponse
-    {
-        $this->authorize('create', MaintenanceType::class);
-
-        $type = new MaintenanceType;
-        $type->name = $request->input('name');
-        $type->created_by = auth()->id();
-
-        if ($type->save()) {
-            return response()->json(Helper::formatStandardApiResponse('success', (new MaintenanceTypesTransformer)->transformMaintenanceType($type), trans('admin/maintenance_types/message.create.success')));
-        }
-
-        return response()->json(Helper::formatStandardApiResponse('error', null, $type->getErrors()));
-    }
-
-    public function update(Request $request, MaintenanceType $maintenanceType): JsonResponse
-    {
-        $this->authorize('update', $maintenanceType);
-
-        $maintenanceType->name = $request->input('name');
-
-        if ($maintenanceType->save()) {
-            return response()->json(Helper::formatStandardApiResponse('success', (new MaintenanceTypesTransformer)->transformMaintenanceType($maintenanceType), trans('admin/maintenance_types/message.update.success')));
-        }
-
-        return response()->json(Helper::formatStandardApiResponse('error', null, $maintenanceType->getErrors()));
-    }
-
-    public function destroy(MaintenanceType $maintenanceType): JsonResponse
-    {
-        $this->authorize('delete', $maintenanceType);
-
-        $maintenanceType->delete();
-
-        return response()->json(Helper::formatStandardApiResponse('success', null, trans('admin/maintenance_types/message.delete.success')));
-    }
-}

app/Http/Controllers/Api/MaintenancesController.php

@@ -2,19 +2,15 @@
 
 namespace App\Http\Controllers\Api;
 
-use App\Enums\ActionType;
 use App\Helpers\Helper;
 use App\Http\Controllers\Controller;
 use App\Http\Requests\FilterRequest;
 use App\Http\Requests\ImageUploadRequest;
 use App\Http\Transformers\ActionlogsTransformer;
 use App\Http\Transformers\MaintenancesTransformer;
-use App\Models\Actionlog;
 use App\Models\Asset;
 use App\Models\Company;
 use App\Models\Maintenance;
-use App\Models\Setting;
-use Illuminate\Database\Eloquent\Collection as EloquentCollection;
 use Illuminate\Http\JsonResponse;
 use Illuminate\Http\Request;
 
@@ -43,7 +39,7 @@ class MaintenancesController extends Controller
 
         $maintenances = Maintenance::select('maintenances.*')
             ->whereHas('asset')
-            ->with('asset', 'asset.model', 'asset.location', 'asset.defaultLoc', 'supplier', 'asset.company', 'asset.status', 'adminuser', 'asset.assignedTo', 'maintenanceType', 'responsibleParty', 'completedByUser');
+            ->with('asset', 'asset.model', 'asset.location', 'asset.defaultLoc', 'supplier', 'asset.company', 'asset.status', 'adminuser', 'asset.assignedTo');
 
         // This invokes the Searchable model trait scopeTextSearch and will handle input by search or by advanced search filter
         if ($request->filled('filter') || $request->filled('search')) {
@@ -66,53 +62,22 @@ class MaintenancesController extends Controller
             $maintenances->where('maintenances.url', '=', $request->input('url'));
         }
 
-        if ($request->filled('maintenance_type')) {
-            $maintenances->where('maintenance_type', '=', $request->input('maintenance_type'));
-        }
-
-        if ($request->filled('maintenance_type_id')) {
-            $maintenances->where('maintenance_type_id', '=', $request->input('maintenance_type_id'));
-        }
-
-        if ($request->filled('responsible_party_id')) {
-            $maintenances->where('responsible_party_id', '=', $request->input('responsible_party_id'));
-        }
-
-        if ($request->filled('completed')) {
-            if ($request->input('completed') === 'true') {
-                $maintenances->completed();
-            } else {
-                $maintenances->active();
-            }
-        }
-
-        if ($request->filled('upcoming_status')) {
-            $settings = Setting::getSettings();
-            switch ($request->input('upcoming_status')) {
-                case 'due':
-                    $maintenances->dueForCompletion($settings);
-                    break;
-                case 'overdue':
-                    $maintenances->overdueForCompletion();
-                    break;
-                case 'due-or-overdue':
-                    $maintenances->dueOrOverdueForCompletion($settings);
-                    break;
-            }
+        if ($request->filled('asset_maintenance_type')) {
+            $maintenances->where('asset_maintenance_type', '=', $request->input('asset_maintenance_type'));
         }
 
         // Make sure the offset and limit are actually integers and do not exceed system limits
-        $offset = ($request->input('offset') > $maintenances->count()) ? $maintenances->count() : app('api_offset_value');
+        $offset = ($request->input('offset') > $maintenances->count()) ? $maintenances->count() : abs($request->input('offset'));
         $limit = app('api_limit_value');
 
         $allowed_columns = [
             'id',
             'name',
             'asset_maintenance_time',
+            'asset_maintenance_type',
             'cost',
             'start_date',
             'completion_date',
-            'completed_at',
             'notes',
             'asset_tag',
             'asset_name',
@@ -124,7 +89,6 @@ class MaintenancesController extends Controller
             'status_label',
             'model',
             'model_number',
-            'maintenance_type',
         ];
 
         $order = $request->input('order') === 'asc' ? 'asc' : 'desc';
@@ -132,37 +96,31 @@ class MaintenancesController extends Controller
 
         switch ($sort) {
             case 'created_by':
-                $maintenances = $maintenances->orderByCreatedBy($order);
+                $maintenances = $maintenances->OrderByCreatedBy($order);
                 break;
             case 'supplier':
-                $maintenances = $maintenances->orderBySupplier($order);
+                $maintenances = $maintenances->OrderBySupplier($order);
                 break;
             case 'asset_tag':
-                $maintenances = $maintenances->orderByTag($order);
+                $maintenances = $maintenances->OrderByTag($order);
                 break;
             case 'asset_name':
-                $maintenances = $maintenances->orderByAssetName($order);
+                $maintenances = $maintenances->OrderByAssetName($order);
                 break;
             case 'model':
-                $maintenances = $maintenances->orderByAssetModelName($order);
+                $maintenances = $maintenances->OrderByAssetModelName($order);
                 break;
             case 'model_number':
-                $maintenances = $maintenances->orderByAssetModelNumber($order);
+                $maintenances = $maintenances->OrderByAssetModelNumber($order);
                 break;
             case 'serial':
-                $maintenances = $maintenances->orderByAssetSerial($order);
+                $maintenances = $maintenances->OrderByAssetSerial($order);
                 break;
             case 'location':
-                $maintenances = $maintenances->orderLocationName($order);
+                $maintenances = $maintenances->OrderLocationName($order);
                 break;
             case 'status_label':
-                $maintenances = $maintenances->orderStatusName($order);
-                break;
-            case 'maintenance_type':
-                $maintenances = $maintenances->orderByMaintenanceType($order);
-                break;
-            case 'completed_at':
-                $maintenances = $maintenances->orderByCompletedAt($order);
+                $maintenances = $maintenances->OrderStatusName($order);
                 break;
             default:
                 $maintenances = $maintenances->orderBy($sort, $order);
@@ -195,60 +153,19 @@ class MaintenancesController extends Controller
     {
         $this->authorize('update', Asset::class);
 
-        $isBulk = $request->has('asset_ids');
-        $assetIds = $isBulk
-            ? array_values(array_filter((array) $request->input('asset_ids')))
-            : [$request->input('asset_id')];
+        // create a new model instance
+        $maintenance = new Maintenance;
+        $maintenance->fill($request->all());
+        $maintenance->created_by = auth()->id();
+        $maintenance = $request->handleImages($maintenance);
+        // Was the asset maintenance created?
+        if ($maintenance->save()) {
+            return response()->json(Helper::formatStandardApiResponse('success', $maintenance, trans('admin/maintenances/message.create.success')));
 
-        $created = new EloquentCollection;
-        $errors = [];
-
-        foreach ($assetIds as $assetId) {
-            $asset = Asset::find($assetId);
-
-            if (! $asset) {
-                $errors[] = trans('general.item_not_found', ['item_type' => trans('general.asset'), 'id' => $assetId]);
-
-                continue;
-            }
-
-            if (! Company::isCurrentUserHasAccess($asset)) {
-                $errors[] = trans('general.action_permission_denied', ['item_type' => trans('general.asset'), 'id' => $assetId, 'action' => trans('general.create')]);
-
-                continue;
-            }
-
-            $maintenance = new Maintenance;
-            $maintenance->fill($request->except(['asset_id', 'asset_ids']));
-            $maintenance->asset_id = $assetId;
-            $maintenance->created_by = auth()->id();
-            $request->handleImages($maintenance);
-
-            if ($maintenance->save()) {
-                $created->push($maintenance->fresh());
-            } else {
-                $errors[] = $maintenance->getErrors();
-            }
         }
 
-        if ($isBulk) {
-            if ($created->isEmpty()) {
-                return response()->json(Helper::formatStandardApiResponse('error', null, count($errors) === 1 ? $errors[0] : $errors));
-            }
+        return response()->json(Helper::formatStandardApiResponse('error', null, $maintenance->getErrors()));
 
-            return response()->json(Helper::formatStandardApiResponse(
-                'success',
-                (new MaintenancesTransformer)->transformMaintenances($created, $created->count()),
-                trans('admin/maintenances/message.create.success')
-            ));
-        }
-
-        // Single asset_id path — backward compatible response shape
-        if ($created->isNotEmpty()) {
-            return response()->json(Helper::formatStandardApiResponse('success', $created->first(), trans('admin/maintenances/message.create.success')));
-        }
-
-        return response()->json(Helper::formatStandardApiResponse('error', null, ! empty($errors) ? $errors[0] : null));
     }
 
     /**
@@ -269,34 +186,18 @@ class MaintenancesController extends Controller
 
         if ($maintenance = Maintenance::with('asset')->find($id)) {
 
-            // The asset this maintenance is attached to is not valid or has been deleted
-            if (! $maintenance->asset) {
-                return response()->json(Helper::formatStandardApiResponse('error', null, trans('general.item_not_found', ['item_type' => trans('general.asset'), 'id' => $id])));
-            }
-
-            // Can this user manage the existing asset?
+            // Can this user manage this asset?
             if (! Company::isCurrentUserHasAccess($maintenance->asset)) {
                 return response()->json(Helper::formatStandardApiResponse('error', null, trans('general.action_permission_denied', ['item_type' => trans('admin/maintenances/general.maintenance'), 'id' => $id, 'action' => trans('general.edit')])));
             }
 
-            // If the request changes asset_id, verify the new asset is accessible
-            if ($request->filled('asset_id') && (int) $request->input('asset_id') !== $maintenance->asset_id) {
-                $newAsset = Asset::find($request->input('asset_id'));
-
-                if (! $newAsset) {
-                    return response()->json(Helper::formatStandardApiResponse('error', null, trans('general.item_not_found', ['item_type' => trans('general.asset'), 'id' => $request->input('asset_id')])));
-                }
-
-                if (! Company::isCurrentUserHasAccess($newAsset)) {
-                    return response()->json(Helper::formatStandardApiResponse('error', null, trans('general.action_permission_denied', ['item_type' => trans('general.asset'), 'id' => $request->input('asset_id'), 'action' => trans('general.edit')])), 403);
-                }
-
-                $maintenance->fill($request->except('asset_id'));
-                $maintenance->asset_id = $newAsset->id;
-            } else {
-                $maintenance->fill($request->except('asset_id'));
+            // The asset this miantenance is attached to is not valid or has been deleted
+            if (! $maintenance->asset) {
+                return response()->json(Helper::formatStandardApiResponse('error', null, trans('general.item_not_found', ['item_type' => trans('general.asset'), 'id' => $id])));
             }
 
+            $maintenance->fill($request->all());
+
             if ($maintenance->save()) {
                 return response()->json(Helper::formatStandardApiResponse('success', $maintenance, trans('admin/maintenances/message.edit.success')));
             }
@@ -355,35 +256,6 @@ class MaintenancesController extends Controller
 
     }
 
-    public function complete(Request $request, Maintenance $maintenance): JsonResponse
-    {
-        $this->authorize('update', Asset::class);
-
-        if (! Company::isCurrentUserHasAccess($maintenance->asset)) {
-            return response()->json(Helper::formatStandardApiResponse('error', null, trans('general.action_permission_denied', ['item_type' => trans('admin/maintenances/general.maintenance'), 'id' => $maintenance->id, 'action' => trans('admin/maintenances/form.mark_complete')])));
-        }
-
-        if ($maintenance->completed_at) {
-            return response()->json(Helper::formatStandardApiResponse('error', null, trans('admin/maintenances/form.already_complete')));
-        }
-
-        $maintenance->completed_at = now();
-        $maintenance->completed_by = auth()->id();
-        $maintenance->asset_maintenance_time = (int) $maintenance->created_at->diffInDays(now(), true);
-        $maintenance->saveQuietly();
-
-        $logAction = new Actionlog;
-        $logAction->item_type = Maintenance::class;
-        $logAction->item_id = $maintenance->id;
-        $logAction->target_type = Asset::class;
-        $logAction->target_id = $maintenance->asset_id;
-        $logAction->created_by = auth()->id();
-        $logAction->note = $request->input('note');
-        $logAction->logaction(ActionType::MaintenanceComplete);
-
-        return response()->json(Helper::formatStandardApiResponse('success', (new MaintenancesTransformer)->transformMaintenance($maintenance->fresh()), trans('admin/maintenances/message.complete.success')));
-    }
-
     public function history(Request $request, Maintenance $maintenance): JsonResponse|array
     {
         $this->authorize('history', $maintenance);
@@ -395,50 +267,4 @@ class MaintenancesController extends Controller
 
         return response()->json((new ActionlogsTransformer)->transformActionlogs($history, $total), 200, ['Content-Type' => 'application/json;charset=utf8'], JSON_UNESCAPED_UNICODE);
     }
-
-    public function notesIndex(Maintenance $maintenance): JsonResponse
-    {
-        $this->authorize('journal', $maintenance);
-
-        $notes = Actionlog::with('user:id,username')
-            ->where('item_type', Maintenance::class)
-            ->where('item_id', $maintenance->id)
-            ->where('action_type', 'note added')
-            ->orderBy('created_at', 'desc')
-            ->get(['id', 'created_at', 'note', 'created_by', 'item_id', 'item_type', 'action_type']);
-
-        $notesArray = $notes->map(fn ($note) => [
-            'id' => $note->id,
-            'created_at' => $note->created_at,
-            'note' => $note->note,
-            'created_by' => $note->created_by,
-            'username' => $note->user?->username,
-            'item_id' => $note->item_id,
-            'item_type' => $note->item_type,
-            'action_type' => $note->action_type,
-        ]);
-
-        return response()->json(Helper::formatStandardApiResponse('success', ['notes' => $notesArray, 'maintenance_id' => $maintenance->id]));
-    }
-
-    public function notesStore(Request $request, Maintenance $maintenance): JsonResponse
-    {
-        $this->authorize('update', $maintenance);
-
-        if (! $request->filled('note')) {
-            return response()->json(Helper::formatStandardApiResponse('error', null, trans('validation.required', ['attribute' => 'note'])), 422);
-        }
-
-        $logaction = new Actionlog;
-        $logaction->item_type = Maintenance::class;
-        $logaction->created_by = auth()->id();
-        $logaction->item_id = $maintenance->id;
-        $logaction->note = $request->input('note');
-
-        if ($logaction->logaction('note added')) {
-            return response()->json(Helper::formatStandardApiResponse('success', ['note' => $logaction->note, 'item_id' => $maintenance->id], trans('general.note_added')));
-        }
-
-        return response()->json(Helper::formatStandardApiResponse('error', null, 'Something went wrong'), 500);
-    }
 }

app/Http/Controllers/Api/PredefinedKitsController.php

@@ -6,9 +6,6 @@ use App\Helpers\Helper;
 use App\Http\Controllers\Controller;
 use App\Http\Transformers\PredefinedKitsTransformer;
 use App\Http\Transformers\SelectlistTransformer;
-use App\Models\Accessory;
-use App\Models\Consumable;
-use App\Models\License;
 use App\Models\PredefinedKit;
 use Illuminate\Http\JsonResponse;
 use Illuminate\Http\Request;
@@ -186,9 +183,6 @@ class PredefinedKitsController extends Controller
         }
 
         $license_id = $request->input('license');
-        $license = License::findOrFail($license_id);
-        $this->authorize('view', $license);
-
         $relation = $kit->licenses();
         if ($relation->find($license_id)) {
             return response()->json(Helper::formatStandardApiResponse('error', null, ['license' => trans('admin/kits/general.license_error')]));
@@ -335,9 +329,6 @@ class PredefinedKitsController extends Controller
         }
 
         $consumable_id = $request->input('consumable');
-        $consumable = Consumable::findOrFail($consumable_id);
-        $this->authorize('view', $consumable);
-
         $relation = $kit->consumables();
         if ($relation->find($consumable_id)) {
             return response()->json(Helper::formatStandardApiResponse('error', null, ['consumable' => trans('admin/kits/general.consumable_error')]));
@@ -411,9 +402,6 @@ class PredefinedKitsController extends Controller
         }
 
         $accessory_id = $request->input('accessory');
-        $accessory = Accessory::findOrFail($accessory_id);
-        $this->authorize('view', $accessory);
-
         $relation = $kit->accessories();
         if ($relation->find($accessory_id)) {
             return response()->json(Helper::formatStandardApiResponse('error', null, ['accessory' => trans('admin/kits/general.accessory_error')]));

app/Http/Controllers/Api/ReportsController.php

@@ -36,18 +36,18 @@ class ReportsController extends Controller
         // then they shouldn't be able to see the activity log for that item or target,
         // but if they have the general activity view permission,
         // then they can see all activity logs regardless of the item or target.
-        if ((! Gate::allows('activity.view')) && (($request->filled('target_type') && $request->filled('target_id')) || ($request->filled('item_type') && $request->filled('item_id')))) {
+        if ((! Gate::allows('activity.view')) && (($request->filled('target_type')) && ($request->filled('target_id'))) || (($request->filled('item_type')) && ($request->filled('item_id')))) {
 
             if (($request->filled('target_type')) && ($request->filled('target_id'))) {
-                $targetClass = Helper::normalizeFullModelName(request()->input('target_type'));
-                $target = $targetClass::withTrashed()->find(request()->input('target_id'));
-                $this->authorize('view', $target ?? $targetClass);
+                $target = Helper::normalizeFullModelName(request()->input('target_type'));
+                $target::find(request()->input('target_id'))?->withTrashed();
+                $this->authorize('view', $target);
             }
 
             if (($request->filled('item_type')) && ($request->filled('item_id'))) {
-                $itemClass = Helper::normalizeFullModelName(request()->input('item_type'));
-                $item = $itemClass::withTrashed()->find(request()->input('item_id'));
-                $this->authorize('view', $item ?? $itemClass);
+                $item = Helper::normalizeFullModelName(request()->input('item_type'));
+                $item::find(request()->input('item_id'))?->withTrashed();
+                $this->authorize('view', $item);
             }
 
         } else {

app/Http/Controllers/Api/UploadedFilesController.php

@@ -52,7 +52,7 @@ class UploadedFilesController extends Controller
         $uploads = self::$map_object_type[$object_type]::withTrashed()->find($id)->uploads()
             ->with('adminuser');
 
-        $offset = ($request->input('offset') > $uploads->count()) ? $uploads->count() : app('api_offset_value');
+        $offset = ($request->input('offset') > $uploads->count()) ? $uploads->count() : abs($request->input('offset'));
         $limit = app('api_limit_value');
         $order = $request->input('order') === 'asc' ? 'asc' : 'desc';
         $sort = in_array($request->input('sort'), $allowed_columns) ? $request->input('sort') : 'created_at';

app/Http/Controllers/Api/UsersController.php

@@ -22,7 +22,6 @@ use App\Models\Asset;
 use App\Models\Company;
 use App\Models\Consumable;
 use App\Models\License;
-use App\Models\Setting;
 use App\Models\User;
 use App\Notifications\CurrentInventory;
 use App\Notifications\WelcomeNotification;
@@ -52,6 +51,7 @@ class UsersController extends Controller
             'users.address',
             'users.avatar',
             'users.city',
+            'users.company_id',
             'users.country',
             'users.created_by',
             'users.created_at',
@@ -89,7 +89,7 @@ class UsersController extends Controller
         ])->with('manager')
             ->with('groups')
             ->with('userloc')
-            ->with('companies')
+            ->with('company')
             ->with('department')
             ->with('createdBy')
             ->withCount([
@@ -191,7 +191,7 @@ class UsersController extends Controller
         }
 
         if ($request->filled('company_id')) {
-            $users = $users->whereHas('companies', fn ($q) => $q->where('companies.id', $request->input('company_id')));
+            $users = $users->where('users.company_id', '=', $request->input('company_id'));
         }
 
         if ($request->filled('phone')) {
@@ -380,8 +380,6 @@ class UsersController extends Controller
      */
     public function selectlist(Request $request): array
     {
-        $this->authorize('view.selectlists');
-
         $users = User::select(
             [
                 'users.id',
@@ -396,22 +394,6 @@ class UsersController extends Controller
             ]
         )->where('show_in_list', '=', '1');
 
-        // When FMCS is enabled, automatically scope to companies the acting user belongs to.
-        // scopeCompanyables is a no-op for superusers and when FMCS is disabled.
-        $users = Company::scopeCompanyables($users, 'company_id', 'users');
-
-        // Allow further narrowing to a specific company passed via data-company-ids on the select.
-        if ((Setting::getSettings()->full_multiple_companies_support == '1') && $request->filled('companyId')) {
-            $companyIds = array_values(array_filter(array_map('intval', explode(',', $request->input('companyId')))));
-            if (! empty($companyIds)) {
-                $users = Company::scopeUsersByCompanyIds($users, $companyIds);
-            }
-        }
-
-        if ($request->filled('excludeId')) {
-            $users->where('users.id', '!=', (int) $request->input('excludeId'));
-        }
-
         if ($request->filled('search')) {
             $users = $users->where(function ($query) use ($request) {
                 $query->SimpleNameSearch($request->input('search'))
@@ -459,6 +441,7 @@ class UsersController extends Controller
         $authenticatedUser = auth()->user();
         $user = new User;
         $user->fill($request->all());
+        $user->company_id = Company::getIdForCurrentUser($request->input('company_id'));
         $user->created_by = auth()->id();
 
         if ($request->has('permissions')) {
@@ -503,12 +486,6 @@ class UsersController extends Controller
                 $user->groups()->sync($request->input('groups'));
             }
 
-            // Sync company memberships from company_ids[] or fall back to scalar company_id
-            $companyIds = array_filter(
-                (array) ($request->input('company_ids') ?? ($request->filled('company_id') ? [$request->input('company_id')] : []))
-            );
-            $user->syncCompaniesWithLogging(Company::getIdsForCurrentUser(array_map('intval', $companyIds)));
-
             return response()->json(Helper::formatStandardApiResponse('success', (new UsersTransformer)->transformUser($user), trans('admin/users/message.success.create')));
         }
 
@@ -598,6 +575,10 @@ class UsersController extends Controller
 
         }
 
+        if ($request->filled('company_id')) {
+            $user->company_id = Company::getIdForCurrentUser($request->input('company_id'));
+        }
+
         if ($user->id == $request->input('manager_id')) {
             return response()->json(Helper::formatStandardApiResponse('error', null, 'You cannot be your own manager'));
         }
@@ -626,18 +607,6 @@ class UsersController extends Controller
                 $user->groups()->sync($request->input('groups'));
             }
 
-            // company_ids (new format) = full replacement sync.
-            // Legacy company_id = add without removing other associations.
-            if ($request->has('company_ids')) {
-                $companyIds = array_filter(array_map('intval', (array) $request->input('company_ids')));
-                $user->syncCompaniesWithLogging(Company::getIdsForCurrentUser($companyIds));
-            } elseif ($request->filled('company_id')) {
-                $filtered = Company::getIdsForCurrentUser([(int) $request->input('company_id')]);
-                if (! empty($filtered)) {
-                    $user->companies()->syncWithoutDetaching($filtered);
-                }
-            }
-
             return response()->json(Helper::formatStandardApiResponse('success', (new UsersTransformer)->transformUser($user), trans('admin/users/message.success.update')));
         }
 

app/Http/Controllers/Assets/AssetCheckinController.php

@@ -84,7 +84,7 @@ class AssetCheckinController extends Controller
     public function store(AssetCheckinRequest $request, $assetId = null, $backto = null): RedirectResponse
     {
         // Check if the asset exists
-        if (is_null($asset = Asset::withTrashed()->find($assetId))) {
+        if (is_null($asset = Asset::find($assetId))) {
             // Redirect to the asset management page with error
             return redirect()->route('hardware.index')->with('error', trans('admin/hardware/message.does_not_exist'));
         }
@@ -133,17 +133,16 @@ class AssetCheckinController extends Controller
 
         $this->migrateLegacyLocations($asset);
 
-        $asset->location_id = $asset->rtd_location_id;
 
         if ($request->has('location_id')) {
             if ($request->filled('location_id')) {
-                Log::debug('NEW Location ID: '.$request->input('location_id'));
+                Log::debug('NEW Location ID: ' . $request->input('location_id'));
                 $asset->location_id = $request->input('location_id');
+
                 if ($request->input('update_default_location') == 0) {
                     $asset->rtd_location_id = $request->input('location_id');
                 }
             } else {
-                // Explicitly submitted as empty — clear the location
                 $asset->location_id = null;
             }
         }

app/Http/Controllers/Assets/AssetCheckoutController.php

@@ -9,6 +9,7 @@ use App\Http\Requests\AssetCheckoutRequest;
 use App\Http\Traits\CheckInOutTrait;
 use App\Models\Asset;
 use App\Models\CheckoutAcceptance;
+use App\Models\Setting;
 use App\Models\User;
 use Illuminate\Contracts\View\View;
 use Illuminate\Database\Eloquent\ModelNotFoundException;
@@ -118,18 +119,13 @@ class AssetCheckoutController extends Controller
             // Add any custom fields that should be included in the checkout
             $asset->customFieldsForCheckinCheckout('display_checkout');
 
-            if (! $asset->canCheckoutTo($target)) {
-                $targetType = match (class_basename($target)) {
-                    'User' => trans('general.user'),
-                    'Location' => trans('general.location'),
-                    default => trans('general.asset'),
-                };
+            $settings = Setting::getSettings();
 
-                return redirect()->route('hardware.checkout.create', $asset)->with('error', trans('general.error_checkout_company_mismatch', [
-                    'item' => trans('general.asset').' "'.$asset->display_name.'"',
-                    'item_company' => $asset->company?->name ?? trans('general.unassigned'),
-                    'target' => $targetType.' "'.($target->name ?? $target->username ?? $target->id).'"',
-                ]));
+            // We have to check whether $target->company_id is null here since locations don't have a company yet
+            if (($settings->full_multiple_companies_support) && ((! is_null($target->company_id)) && (! is_null($asset->company_id)))) {
+                if ($target->company_id != $asset->company_id) {
+                    return redirect()->route('hardware.checkout.create', $asset)->with('error', trans('general.error_user_company'));
+                }
             }
 
             session()->put([

app/Http/Controllers/Assets/AssetsController.php

@@ -358,7 +358,7 @@ class AssetsController extends Controller
 
             $qr_code = (object) [
                 'display' => $settings->qr_code == '1',
-                'url' => route('qr_code/common', ['object_type' => 'hardware', 'id' => $asset->id]),
+                'url' => route('qr_code/hardware', $asset),
             ];
 
             $total_maintenance_cost = $asset->maintenances?->sum('cost');
@@ -443,7 +443,7 @@ class AssetsController extends Controller
 
         if ($request->filled('image_delete')) {
             try {
-                unlink(public_path().'/uploads/assets/'.basename($asset->image));
+                unlink(public_path().'/uploads/assets/'.$asset->image);
                 $asset->image = '';
             } catch (\Exception $e) {
                 Log::info($e);
@@ -511,7 +511,7 @@ class AssetsController extends Controller
 
         // Validate required serial based on model setting
         if ($model && $model->require_serial === 1 && empty($serial[1])) {
-            return Helper::getRedirectOption($request, $asset->id, 'Assets')
+            return redirect()->to(Helper::getRedirectOption($request, $asset->id, 'Assets'))
                 ->with('warning', trans('admin/hardware/form.serial_required_post_model_update', [
                     'asset_model' => $model->name,
                 ]));
@@ -549,7 +549,7 @@ class AssetsController extends Controller
 
         if ($asset->image) {
             try {
-                Storage::disk('public')->delete('assets/'.basename($asset->image));
+                Storage::disk('public')->delete('assets'.'/'.$asset->image);
             } catch (\Exception $e) {
                 Log::debug($e);
             }
@@ -567,12 +567,11 @@ class AssetsController extends Controller
      *
      * @since [v3.0]
      */
-    public function getAssetBySerial(Request $request, $serial = null): RedirectResponse
+    public function getAssetBySerial(Request $request): RedirectResponse
     {
-        $serial = $serial ?: $request->input('serial');
         $topsearch = ($request->input('topsearch') == 'true');
 
-        if (! $asset = Asset::where('serial', '=', $serial)->first()) {
+        if (! $asset = Asset::where('serial', '=', $request->input('serial'))->first()) {
             return redirect()->route('hardware.index')->with('error', trans('admin/hardware/message.does_not_exist'));
         }
         $this->authorize('view', $asset);

app/Http/Controllers/Assets/BulkAssetsController.php

@@ -2,7 +2,6 @@
 
 namespace App\Http\Controllers\Assets;
 
-use App\Events\CheckoutableCheckedIn;
 use App\Events\CheckoutablesCheckedOutInBulk;
 use App\Helpers\Helper;
 use App\Http\Controllers\Controller;
@@ -10,17 +9,13 @@ use App\Http\Requests\AssetCheckoutRequest;
 use App\Http\Traits\CheckInOutTrait;
 use App\Models\Asset;
 use App\Models\AssetModel;
-use App\Models\CheckoutAcceptance;
 use App\Models\Company;
 use App\Models\CustomField;
-use App\Models\LicenseSeat;
 use App\Models\Setting;
 use App\Models\Statuslabel;
-use App\Models\User;
 use App\View\Label;
 use Carbon\Carbon;
 use Illuminate\Contracts\View\View;
-use Illuminate\Database\Eloquent\Builder;
 use Illuminate\Database\Eloquent\ModelNotFoundException;
 use Illuminate\Http\RedirectResponse;
 use Illuminate\Http\Request;
@@ -78,16 +73,6 @@ class BulkAssetsController extends Controller
             return redirect()->route('hardware.bulkcheckout.show');
         }
 
-        if ($request->input('bulk_actions') === 'checkin') {
-            $referer = request()->headers->get('referer');
-            if ($referer && parse_url($referer, PHP_URL_HOST) === parse_url(config('app.url'), PHP_URL_HOST)) {
-                redirect()->setIntendedUrl($referer);
-            }
-            $request->session()->flashInput(['selected_assets' => $asset_ids]);
-
-            return redirect()->route('hardware.bulkcheckin.show');
-        }
-
         if ($request->input('bulk_actions') === 'maintenance') {
             $request->session()->flashInput(['selected_assets' => $asset_ids]);
 
@@ -688,25 +673,18 @@ class BulkAssetsController extends Controller
                     ->with('error', trans('general.error_assets_already_checked_out'));
             }
 
-            // Prevent checking out assets across companies if FMCS enabled.
-            if (Setting::getSettings()->full_multiple_companies_support) {
-                $company_ids = $assets->pluck('company_id')->filter()->unique();
+            // Prevent checking out assets across companies if FMCS enabled
+            if (Setting::getSettings()->full_multiple_companies_support && $target->company_id) {
+                $company_ids = $assets->pluck('company_id')->unique();
 
-                if ($company_ids->isNotEmpty()) {
-                    if ($company_ids->count() > 1) {
-                        // Selected assets span multiple companies; bulk checkout can't satisfy all of them.
-                        $mismatch = true;
-                    } else {
-                        // All assets share the same company; let the model enforce the checkout rules.
-                        $mismatch = ! $assets->first()->canCheckoutTo($target);
-                    }
+                // if there is more than one unique company id or the singular company id does not match
+                // then the checkout is invalid
+                if ($company_ids->count() > 1 || $company_ids->first() != $target->company_id) {
+                    // re-add the asset ids so the assets select is re-populated
+                    $request->session()->flashInput(['selected_assets' => $asset_ids]);
 
-                    if ($mismatch) {
-                        $request->session()->flashInput(['selected_assets' => $asset_ids]);
-
-                        return redirect(route('hardware.bulkcheckout.show'))
-                            ->with('error', trans('general.error_user_company_multiple'));
-                    }
+                    return redirect(route('hardware.bulkcheckout.show'))
+                        ->with('error', trans('general.error_user_company_multiple'));
                 }
             }
 
@@ -781,112 +759,6 @@ class BulkAssetsController extends Controller
 
     }
 
-    /**
-     * Show Bulk Checkin Page
-     */
-    public function showCheckin(): View
-    {
-        $this->authorize('checkin', Asset::class);
-
-        $notAssigned = collect();
-
-        if (old('selected_assets') && is_array(old('selected_assets'))) {
-            $assets = Asset::withTrashed()->findMany(old('selected_assets'));
-
-            [$assigned, $notAssigned] = $assets->partition(function (Asset $asset) {
-                return $asset->assigned_to;
-            });
-
-            session()->flashInput(['selected_assets' => $assigned->pluck('id')->values()->toArray()]);
-        }
-
-        $do_not_change = ['' => trans('general.do_not_change')];
-        $status_label_list = $do_not_change + Helper::statusLabelList();
-
-        return view('hardware/bulk-checkin', [
-            'statusLabel_list' => $status_label_list,
-            'removed_assets' => $notAssigned,
-        ]);
-    }
-
-    /**
-     * Process Multiple Checkin Request
-     */
-    public function storeCheckin(Request $request): RedirectResponse
-    {
-        $this->authorize('checkin', Asset::class);
-
-        if (! is_array($request->input('selected_assets'))) {
-            return redirect()->route('hardware.bulkcheckin.show')->withInput()->with('error', trans('admin/hardware/message.multi-checkin.no_assets_selected'));
-        }
-
-        $asset_ids = array_filter($request->input('selected_assets'));
-
-        $assets = Asset::withTrashed()->findOrFail($asset_ids);
-
-        $checkin_at = date('Y-m-d H:i:s');
-        if ($request->filled('checkin_at') && $request->input('checkin_at') != date('Y-m-d')) {
-            $checkin_at = $request->input('checkin_at');
-        }
-
-        $errors = [];
-        $admin = auth()->user();
-
-        DB::transaction(function () use ($assets, $admin, $checkin_at, $request, &$errors) {
-            foreach ($assets as $asset) {
-                $this->authorize('checkin', $asset);
-
-                if (is_null($asset->assignedTo)) {
-                    continue;
-                }
-
-                $target = $asset->assignedTo;
-                $originalValues = $asset->getRawOriginal();
-
-                $asset->expected_checkin = null;
-                $asset->assignedTo()->disassociate($asset);
-                $asset->accepted = null;
-
-                if ($request->filled('status_id')) {
-                    $asset->status_id = $request->input('status_id');
-                }
-
-                $asset->location_id = $asset->rtd_location_id;
-                $asset->last_checkin = $checkin_at;
-
-                if ($request->boolean('checkin_licenses')) {
-                    $asset->licenseseats->each(function (LicenseSeat $seat) {
-                        $seat->update(['assigned_to' => null]);
-                    });
-                }
-
-                CheckoutAcceptance::pending()->whereHasMorph('checkoutable', [Asset::class], function (Builder $query) use ($asset) {
-                    $query->where('id', $asset->id);
-                })->get()->each->delete();
-
-                if ($asset->save()) {
-                    if ($request->boolean('checkin_child_assets')) {
-                        Asset::where('assigned_type', Asset::class)
-                            ->where('assigned_to', $asset->id)
-                            ->update(['location_id' => $asset->location_id]);
-                    }
-
-                    event(new CheckoutableCheckedIn($asset, $target, $admin, $request->input('note'), $checkin_at, $originalValues));
-                } else {
-                    $errors = array_merge_recursive($errors, $asset->getErrors()->toArray());
-                }
-            }
-        });
-
-        if (! $errors) {
-            return redirect()->intended(route('hardware.index'))->with('success', trans_choice('admin/hardware/message.multi-checkin.success', count($asset_ids)));
-        }
-
-        return redirect()->route('hardware.bulkcheckin.show')->withInput()
-            ->with('error', trans_choice('admin/hardware/message.multi-checkin.error', count($asset_ids)))
-            ->withErrors($errors);
-    }
-
     public function restore(Request $request): RedirectResponse
     {
         $this->authorize('update', Asset::class);

app/Http/Controllers/Auth/SamlController.php

@@ -75,7 +75,6 @@ class SamlController extends Controller
     {
         $auth = $this->saml->getAuth();
         $ssoUrl = $auth->login(session()->get('url.intended'), [], false, false, false, false);
-
         return redirect()->away($ssoUrl);
     }
 
@@ -96,7 +95,7 @@ class SamlController extends Controller
         $saml = $this->saml;
         $auth = $saml->getAuth();
         $saml_exception = false;
-        session()->put('url.intended', str_replace(["\r", "\n"], '', $request->post('RelayState')));
+        session()->put('url.intended', $request->post('RelayState'));
         try {
             $auth->processResponse();
         } catch (\Exception $e) {

app/Http/Controllers/Components/ComponentCheckinController.php

@@ -43,8 +43,7 @@ class ComponentCheckinController extends Controller
             }
             $this->authorize('checkin', $component);
 
-            return view('components/checkin', compact('component_assets', 'component', 'asset'))
-                ->with('snipe_component', $component);
+            return view('components/checkin', compact('component_assets', 'component', 'asset'));
         }
 
         return redirect()->route('components.index')->with('error', trans('admin/components/messages.not_found'));

app/Http/Controllers/Components/ComponentCheckoutController.php

@@ -7,6 +7,7 @@ use App\Helpers\Helper;
 use App\Http\Controllers\Controller;
 use App\Models\Asset;
 use App\Models\Component;
+use App\Models\Setting;
 use Illuminate\Auth\Access\AuthorizationException;
 use Illuminate\Contracts\View\View;
 use Illuminate\Http\RedirectResponse;
@@ -45,8 +46,7 @@ class ComponentCheckoutController extends Controller
                 }
 
                 // Return the checkout view
-                return view('components/checkout', compact('component'))
-                    ->with('snipe_component', $component);
+                return view('components/checkout', compact('component'));
             }
 
             // Invalid category
@@ -103,12 +103,8 @@ class ComponentCheckoutController extends Controller
         // Check if the asset exists
         $asset = Asset::find($request->input('asset_id'));
 
-        if (! $component->canCheckoutTo($asset)) {
-            return redirect()->route('components.checkout.show', $componentId)->with('error', trans('general.error_checkout_company_mismatch', [
-                'item' => trans('general.component').' "'.$component->name.'"',
-                'item_company' => $component->company?->name ?? trans('general.unassigned'),
-                'target' => trans('general.asset').' "'.$asset->display_name.'"',
-            ]));
+        if ((Setting::getSettings()->full_multiple_companies_support) && $component->company_id !== $asset->company_id) {
+            return redirect()->route('components.checkout.show', $componentId)->with('error', trans('general.error_user_company'));
         }
 
         $component->checkout_qty = $request->input('assigned_qty');

app/Http/Controllers/Consumables/ConsumableCheckoutController.php

@@ -96,14 +96,6 @@ class ConsumableCheckoutController extends Controller
             return redirect()->route('consumables.checkout.show', $consumable)->with('error', trans('admin/consumables/message.checkout.user_does_not_exist'))->withInput();
         }
 
-        if (! $consumable->canCheckoutTo($user)) {
-            return redirect()->back()->with('error', trans('general.error_checkout_company_mismatch', [
-                'item' => trans('general.consumable').' "'.$consumable->name.'"',
-                'item_company' => $consumable->company?->name ?? trans('general.unassigned'),
-                'target' => trans('general.user').' "'.$user->username.'"',
-            ]));
-        }
-
         // Update the consumable data
         $consumable->assigned_to = e($request->input('assigned_to'));
 

app/Http/Controllers/GoogleAuthController.php

@@ -54,7 +54,6 @@ class GoogleAuthController extends Controller
             Log::debug('Google user '.$socialUser->getEmail().' found in Snipe-IT');
             $user->update([
                 'avatar' => $socialUser->avatar,
-                'last_login' => \Carbon::now(),
             ]);
 
             Auth::login($user, true);

app/Http/Controllers/Kits/CheckoutKitController.php

@@ -53,8 +53,6 @@ class CheckoutKitController extends Controller
      */
     public function store(Request $request, $kit_id)
     {
-        $this->authorize('checkout', Asset::class);
-
         $user_id = e($request->input('user_id'));
         if (is_null($user = User::find($user_id))) {
             return redirect()->back()->with('error', trans('admin/users/message.user_not_found'));

app/Http/Controllers/Licenses/BulkLicensesController.php

@@ -17,7 +17,7 @@ class BulkLicensesController extends Controller
         $errors = [];
         $success_count = 0;
 
-        foreach ($request->input('ids', []) as $id) {
+        foreach ($request->ids as $id) {
             $license = License::find($id);
 
             if (is_null($license)) {

app/Http/Controllers/Licenses/LicenseCheckinController.php

@@ -13,7 +13,6 @@ use Illuminate\Auth\Access\AuthorizationException;
 use Illuminate\Contracts\View\View;
 use Illuminate\Http\RedirectResponse;
 use Illuminate\Http\Request;
-use Illuminate\Support\Facades\Gate;
 use Illuminate\Support\Facades\Log;
 use Illuminate\Support\Facades\Validator;
 
@@ -36,7 +35,7 @@ class LicenseCheckinController extends Controller
     {
         // Check if the asset exists
         $license = License::find($licenseSeat->license_id);
-        $this->authorize('checkin', $license);
+        $this->authorize('checkout', $license);
 
         return view('licenses/checkin', compact('licenseSeat'))->with('backto', $backTo);
     }
@@ -70,7 +69,7 @@ class LicenseCheckinController extends Controller
             return redirect()->route('licenses.index')->with('error', trans('admin/licenses/message.checkin.error'));
         }
 
-        $this->authorize('checkin', $license);
+        $this->authorize('checkout', $license);
 
         // Declare the rules for the form validation
         $rules = [
@@ -128,45 +127,10 @@ class LicenseCheckinController extends Controller
      * @see LicenseCheckinController::create() method that provides the form view
      * @since [v6.1.1]
      *
+     * @return RedirectResponse
+     *
      * @throws AuthorizationException
      */
-    public function bulkCheckinSelected(Request $request): RedirectResponse
-    {
-        $this->authorize('checkin', License::class);
-
-        $seatIds = $request->input('ids', []);
-
-        if (empty($seatIds)) {
-            return redirect()->back()->with('warning', trans('admin/licenses/general.bulk.checkin_selected.no_seats_selected'));
-        }
-
-        $seats = LicenseSeat::whereIn('id', $seatIds)
-            ->where(function ($query) {
-                $query->whereNotNull('assigned_to')->orWhereNotNull('asset_id');
-            })
-            ->with('license', 'user', 'asset')
-            ->get();
-
-        $count = 0;
-        foreach ($seats as $seat) {
-            if (! $seat->license || ! Gate::allows('checkin', $seat->license)) {
-                continue;
-            }
-            $target = $seat->user ?? $seat->asset;
-            $seat->assigned_to = null;
-            $seat->asset_id = null;
-            if (! $seat->license->reassignable) {
-                $seat->unreassignable_seat = true;
-            }
-            if ($seat->save()) {
-                event(new CheckoutableCheckedIn($seat, $target, auth()->user(), null));
-                $count++;
-            }
-        }
-
-        return redirect()->back()->with('success', trans_choice('admin/licenses/general.bulk.checkin_selected.success', $count, ['count' => $count]));
-    }
-
     public function bulkCheckin(Request $request, $licenseId)
     {
 

app/Http/Controllers/Licenses/LicenseCheckoutController.php

@@ -10,13 +10,11 @@ use App\Models\Asset;
 use App\Models\CheckoutAcceptance;
 use App\Models\License;
 use App\Models\LicenseSeat;
-use App\Models\Setting;
 use App\Models\User;
 use Illuminate\Auth\Access\AuthorizationException;
 use Illuminate\Contracts\View\View;
 use Illuminate\Http\Exceptions\HttpResponseException;
 use Illuminate\Http\RedirectResponse;
-use Illuminate\Support\Facades\DB;
 use Illuminate\Support\Facades\Log;
 
 class LicenseCheckoutController extends Controller
@@ -96,53 +94,23 @@ class LicenseCheckoutController extends Controller
             return redirect()->route('licenses.index')->with('error', trans('admin/licenses/message.checkout.license_is_inactive'));
         }
 
-        if (Setting::getSettings()->full_multiple_companies_support == '1') {
-            if ($request->filled('asset_id')) {
-                $fmcsTarget = Asset::find($request->input('asset_id'));
-                if ($fmcsTarget && ! $license->canCheckoutTo($fmcsTarget)) {
-                    return redirect()->route('licenses.index')->with('error', trans('general.error_checkout_company_mismatch', [
-                        'item' => trans('general.license').' "'.$license->name.'"',
-                        'item_company' => $license->company?->name ?? trans('general.unassigned'),
-                        'target' => trans('general.asset').' "'.$fmcsTarget->display_name.'"',
-                    ]));
-                }
-            } elseif ($request->filled('assigned_to')) {
-                $fmcsTarget = User::find($request->input('assigned_to'));
-                if ($fmcsTarget && ! $license->canCheckoutTo($fmcsTarget)) {
-                    return redirect()->route('licenses.index')->with('error', trans('general.error_checkout_company_mismatch', [
-                        'item' => trans('general.license').' "'.$license->name.'"',
-                        'item_company' => $license->company?->name ?? trans('general.unassigned'),
-                        'target' => trans('general.user').' "'.$fmcsTarget->username.'"',
-                    ]));
-                }
-            }
-        }
-
-        $licenseSeat = null;
-        $checkoutTarget = null;
-
-        DB::transaction(function () use ($request, $license, $seatId, &$licenseSeat, &$checkoutTarget): void {
-            $licenseSeat = $this->findLicenseSeatToCheckout($license, $seatId, lock: true);
-            $licenseSeat->created_by = auth()->id();
-            $licenseSeat->notes = $request->input('notes');
-
-            if ($request->filled('asset_id')) {
-                $checkoutTarget = $this->checkoutToAsset($licenseSeat);
-            } elseif ($request->filled('assigned_to')) {
-                $checkoutTarget = $this->checkoutToUser($licenseSeat);
-            }
-        });
+        $licenseSeat = $this->findLicenseSeatToCheckout($license, $seatId);
+        $licenseSeat->created_by = auth()->id();
+        $licenseSeat->notes = $request->input('notes');
 
         if ($request->filled('asset_id')) {
             session()->put(['checkout_to_type' => 'asset']);
+            $checkoutTarget = $this->checkoutToAsset($licenseSeat);
             $request->request->add(['assigned_asset' => $checkoutTarget->id]);
             session()->put([
                 'redirect_option' => $request->input('redirect_option'),
                 'checkout_to_type' => 'asset',
                 'sign_in_place' => $request->boolean('sign_in_place'),
             ]);
+
         } elseif ($request->filled('assigned_to')) {
             session()->put(['checkout_to_type' => 'user']);
+            $checkoutTarget = $this->checkoutToUser($licenseSeat);
             $request->request->add(['assigned_user' => $checkoutTarget->id]);
             session()->put([
                 'redirect_option' => $request->input('redirect_option'),
@@ -188,11 +156,9 @@ class LicenseCheckoutController extends Controller
         return redirect()->route('licenses.index')->with('error', trans('Something went wrong handling this checkout.'));
     }
 
-    protected function findLicenseSeatToCheckout($license, $seatId, bool $lock = false)
+    protected function findLicenseSeatToCheckout($license, $seatId)
     {
-        $licenseSeat = $seatId
-            ? LicenseSeat::where('id', $seatId)->when($lock, fn ($q) => $q->lockForUpdate())->first()
-            : $license->freeSeat(lock: $lock);
+        $licenseSeat = LicenseSeat::find($seatId) ?? $license->freeSeat();
 
         if (! $licenseSeat) {
             if ($seatId) {
@@ -263,10 +229,14 @@ class LicenseCheckoutController extends Controller
 
         Log::debug('Checking out '.$licenseId.' via bulk');
         $license = License::findOrFail($licenseId);
-        $this->authorize('checkout', $license);
+        $this->authorize('checkin', $license);
+        $avail_count = $license->getAvailSeatsCountAttribute();
 
-        if ($license->isInactive()) {
-            return redirect()->back()->with('error', trans('admin/licenses/message.checkout.license_is_inactive'));
+        $users = User::whereNull('deleted_at')->where('autoassign_licenses', '=', 1)->with('licenses')->get();
+        Log::debug($avail_count.' will be assigned');
+
+        if ($users->count() > $avail_count) {
+            Log::debug('You do not have enough free seats to complete this task, so we will check out as many as we can. ');
         }
 
         // If the license is valid, check that there is an available seat
@@ -274,19 +244,6 @@ class LicenseCheckoutController extends Controller
             return redirect()->back()->with('error', trans('admin/licenses/general.bulk.checkout_all.error_no_seats'));
         }
 
-        $avail_count = $license->getAvailSeatsCountAttribute();
-
-        $usersQuery = User::whereNull('deleted_at')->where('autoassign_licenses', '=', 1)->with('licenses');
-        if (Setting::getSettings()->full_multiple_companies_support && $license->company_id) {
-            $usersQuery->where('company_id', '=', $license->company_id);
-        }
-        $users = $usersQuery->get();
-        Log::debug($avail_count.' will be assigned');
-
-        if ($users->count() > $avail_count) {
-            Log::debug('You do not have enough free seats to complete this task, so we will check out as many as we can. ');
-        }
-
         $assigned_count = 0;
 
         foreach ($users as $user) {

app/Http/Controllers/LocationsController.php

@@ -89,24 +89,19 @@ class LocationsController extends Controller
         $location->fax = request('fax');
         $location->tag_color = $request->input('tag_color');
         $location->notes = $request->input('notes');
+        $location->company_id = Company::getIdForCurrentUser($request->input('company_id'));
+
+        // Only scope the location if the setting is enabled
         if (Setting::getSettings()->scope_locations_fmcs) {
             $location->company_id = Company::getIdForCurrentUser($request->input('company_id'));
+            // check if parent is set and has a different company
+            if ($location->parent_id && Location::find($location->parent_id)->company_id != $location->company_id) {
+                return redirect()->back()->withInput()->withInput()->with('error', 'different company than parent');
+            }
         } else {
             $location->company_id = $request->input('company_id');
         }
 
-        // Parent company check applies whenever FMCS is on, independent of scope_locations_fmcs.
-        if (Setting::getSettings()->full_multiple_companies_support) {
-            $parent = $location->parent_id ? Location::find($location->parent_id) : null;
-            if ($parent && $parent->company_id != $location->company_id) {
-                return redirect()->back()->withInput()->with('error', trans('general.error_location_parent_company', [
-                    'parent' => $parent->name,
-                    'parent_company' => $parent->company?->name ?? trans('general.unassigned'),
-                    'location_company' => $location->company?->name ?? trans('general.unassigned'),
-                ]));
-            }
-        }
-
         if ($request->has('use_cloned_image')) {
             $cloned_model_img = Location::select('image')->find($request->input('clone_image_from_id'));
             if ($cloned_model_img) {
@@ -176,34 +171,17 @@ class LocationsController extends Controller
         $location->tag_color = $request->input('tag_color');
         $location->notes = $request->input('notes');
 
+        // Only scope the location if the setting is enabled
         if (Setting::getSettings()->scope_locations_fmcs) {
             $location->company_id = Company::getIdForCurrentUser($request->input('company_id'));
             // check if there are related objects with different company
-            if ($mismatched = Helper::test_locations_fmcs(false, $location->id, $location->company_id)) {
-                $first = $mismatched[0];
-
-                return redirect()->back()->withInput()->with('error', trans('general.error_location_scoped_items', [
-                    'item_type' => trans('general.'.strtolower($first[0])),
-                    'item_name' => $first[2],
-                    'item_company' => $first[5] ?? trans('general.unassigned'),
-                ]));
+            if (Helper::test_locations_fmcs(false, $location->id, $location->company_id)) {
+                return redirect()->back()->withInput()->withInput()->with('error', 'error scoped locations');
             }
         } else {
             $location->company_id = $request->input('company_id');
         }
 
-        // Parent company check applies whenever FMCS is on, independent of scope_locations_fmcs.
-        if (Setting::getSettings()->full_multiple_companies_support) {
-            $parent = $location->parent_id ? Location::find($location->parent_id) : null;
-            if ($parent && $parent->company_id != $location->company_id) {
-                return redirect()->back()->withInput()->with('error', trans('general.error_location_parent_company', [
-                    'parent' => $parent->name,
-                    'parent_company' => $parent->company?->name ?? trans('general.unassigned'),
-                    'location_company' => $location->company?->name ?? trans('general.unassigned'),
-                ]));
-            }
-        }
-
         $location = $request->handleImages($location);
 
         if ($location->save()) {
@@ -299,7 +277,7 @@ class LocationsController extends Controller
                 ->with('assignedAssets', $location->assignedAssets)
                 ->with('accessories', $location->accessories)
                 ->with('assignedAccessories', $location->assignedAccessories)
-                ->with('users', $location->users()->with('companies')->get())
+                ->with('users', $location->users)
                 ->with('location', $location)
                 ->with('consumables', $location->consumables)
                 ->with('components', $location->components)
@@ -319,7 +297,7 @@ class LocationsController extends Controller
                 ->with('assignedAssets', $location->assignedAssets)
                 ->with('accessories', $location->accessories)
                 ->with('assignedAccessories', $location->assignedAccessories)
-                ->with('users', $location->users()->with('companies')->get())
+                ->with('users', $location->users)
                 ->with('location', $location)
                 ->with('consumables', $location->consumables)
                 ->with('components', $location->components)

app/Http/Controllers/MaintenanceTypesController.php

@@ -1,72 +0,0 @@
-<?php
-
-namespace App\Http\Controllers;
-
-use App\Models\MaintenanceType;
-use Illuminate\Contracts\View\View;
-use Illuminate\Http\RedirectResponse;
-use Illuminate\Http\Request;
-
-class MaintenanceTypesController extends Controller
-{
-    public function index(): View
-    {
-        $this->authorize('index', MaintenanceType::class);
-
-        return view('maintenance-types.index');
-    }
-
-    public function create(): View
-    {
-        $this->authorize('create', MaintenanceType::class);
-
-        return view('maintenance-types.edit')->with('item', new MaintenanceType);
-    }
-
-    public function store(Request $request): RedirectResponse
-    {
-        $this->authorize('create', MaintenanceType::class);
-
-        $type = new MaintenanceType;
-        $type->name = $request->input('name');
-        $type->created_by = auth()->id();
-
-        if ($type->save()) {
-            return redirect()->route('maintenance-types.index')
-                ->with('success', trans('admin/maintenance_types/message.create.success'));
-        }
-
-        return redirect()->back()->withInput()->withErrors($type->getErrors());
-    }
-
-    public function edit(MaintenanceType $maintenanceType): View
-    {
-        $this->authorize('update', $maintenanceType);
-
-        return view('maintenance-types.edit')->with('item', $maintenanceType);
-    }
-
-    public function update(Request $request, MaintenanceType $maintenanceType): RedirectResponse
-    {
-        $this->authorize('update', $maintenanceType);
-
-        $maintenanceType->name = $request->input('name');
-
-        if ($maintenanceType->save()) {
-            return redirect()->route('maintenance-types.index')
-                ->with('success', trans('admin/maintenance_types/message.update.success'));
-        }
-
-        return redirect()->back()->withInput()->withErrors($maintenanceType->getErrors());
-    }
-
-    public function destroy(MaintenanceType $maintenanceType): RedirectResponse
-    {
-        $this->authorize('delete', $maintenanceType);
-
-        $maintenanceType->delete();
-
-        return redirect()->route('maintenance-types.index')
-            ->with('success', trans('admin/maintenance_types/message.delete.success'));
-    }
-}

app/Http/Controllers/MaintenancesController.php

@@ -2,14 +2,11 @@
 
 namespace App\Http\Controllers;
 
-use App\Enums\ActionType;
 use App\Http\Requests\ImageUploadRequest;
 use App\Http\Requests\UploadFileRequest;
-use App\Models\Actionlog;
 use App\Models\Asset;
-use App\Models\Company;
 use App\Models\Maintenance;
-use App\Models\MaintenanceType;
+use Carbon\Carbon;
 use Illuminate\Contracts\View\View;
 use Illuminate\Http\RedirectResponse;
 use Illuminate\Http\Request;
@@ -60,7 +57,6 @@ class MaintenancesController extends Controller
 
         return view('maintenances/edit')
             ->with('maintenanceType', Maintenance::getImprovementOptions())
-            ->with('maintenanceTypes', MaintenanceType::orderBy('name')->get())
             ->with('asset', $asset)
             ->with('item', new Maintenance);
     }
@@ -86,10 +82,6 @@ class MaintenancesController extends Controller
         // Loop through the selected assets
         foreach ($assets as $asset) {
 
-            if (! Company::isCurrentUserHasAccess($asset)) {
-                continue;
-            }
-
             $maintenance = new Maintenance;
             $maintenance->supplier_id = $request->input('supplier_id');
             $maintenance->is_warranty = $request->input('is_warranty');
@@ -100,13 +92,20 @@ class MaintenancesController extends Controller
             // Save the asset maintenance data
             $maintenance->asset_id = $asset->id;
             $maintenance->asset_maintenance_type = $request->input('asset_maintenance_type');
-            $maintenance->maintenance_type_id = $request->input('maintenance_type_id');
             $maintenance->name = $request->input('name');
             $maintenance->start_date = $request->input('start_date');
             $maintenance->completion_date = $request->input('completion_date');
-            $maintenance->responsible_party_id = $request->input('responsible_party_id') ?: auth()->id();
             $maintenance->created_by = auth()->id();
 
+            if (($maintenance->completion_date !== null)
+                && ($maintenance->start_date !== '')
+                && ($maintenance->start_date !== '0000-00-00')
+            ) {
+                $startDate = Carbon::parse($maintenance->start_date);
+                $completionDate = Carbon::parse($maintenance->completion_date);
+                $maintenance->asset_maintenance_time = (int) $completionDate->diffInDays($startDate, true);
+            }
+
             $request->handleImages($maintenance);
 
             // Was the asset maintenance created?
@@ -142,7 +141,6 @@ class MaintenancesController extends Controller
             ->with('selected_assets', $maintenance->asset->pluck('id')->toArray())
             ->with('asset_ids', request()->input('asset_ids', []))
             ->with('maintenanceType', Maintenance::getImprovementOptions())
-            ->with('maintenanceTypes', MaintenanceType::orderBy('name')->get())
             ->with('item', $maintenance);
     }
 
@@ -171,12 +169,28 @@ class MaintenancesController extends Controller
         $maintenance->cost = $request->input('cost');
         $maintenance->notes = $request->input('notes');
         $maintenance->asset_maintenance_type = $request->input('asset_maintenance_type');
-        $maintenance->maintenance_type_id = $request->input('maintenance_type_id');
         $maintenance->name = $request->input('name');
         $maintenance->start_date = $request->input('start_date');
         $maintenance->completion_date = $request->input('completion_date');
-        $maintenance->responsible_party_id = $request->input('responsible_party_id');
         $maintenance->url = $request->input('url');
+
+        // Todo - put this in a getter/setter?
+        if (($maintenance->completion_date == null)) {
+            if (($maintenance->asset_maintenance_time !== 0)
+              || (! is_null($maintenance->asset_maintenance_time))
+            ) {
+                $maintenance->asset_maintenance_time = null;
+            }
+        }
+
+        if (($maintenance->completion_date !== null)
+          && ($maintenance->start_date !== '')
+          && ($maintenance->start_date !== '0000-00-00')
+        ) {
+            $startDate = Carbon::parse($maintenance->start_date);
+            $completionDate = Carbon::parse($maintenance->completion_date);
+            $maintenance->asset_maintenance_time = (int) $completionDate->diffInDays($startDate, true);
+        }
         $request->handleImages($maintenance);
 
         if ($maintenance->save()) {
@@ -239,36 +253,6 @@ class MaintenancesController extends Controller
         )->validate();
     }
 
-    /**
-     * Mark a maintenance record as complete, logging who completed it and when.
-     */
-    public function complete(Request $request, Maintenance $maintenance): RedirectResponse
-    {
-        $this->authorize('update', $maintenance->asset);
-
-        if ($maintenance->completed_at) {
-            return redirect()->back()
-                ->with('warning', trans('admin/maintenances/form.already_complete'));
-        }
-
-        $maintenance->completed_at = now();
-        $maintenance->completed_by = auth()->id();
-        $maintenance->asset_maintenance_time = (int) $maintenance->created_at->diffInDays(now(), true);
-        $maintenance->saveQuietly();
-
-        $logAction = new Actionlog;
-        $logAction->item_type = Maintenance::class;
-        $logAction->item_id = $maintenance->id;
-        $logAction->target_type = Asset::class;
-        $logAction->target_id = $maintenance->asset_id;
-        $logAction->created_by = auth()->id();
-        $logAction->note = $request->input('note');
-        $logAction->logaction(ActionType::MaintenanceComplete);
-
-        return redirect()->back()
-            ->with('success', trans('admin/maintenances/message.complete.success'));
-    }
-
     /**
      *  Delete an asset maintenance
      *

app/Http/Controllers/ModalController.php

@@ -30,7 +30,6 @@ class ModalController extends Controller
             'kit-consumable',
             'kit-accessory',
             'location',
-            'maintenance-type',
             'manufacturer',
             'model',
             'statuslabel',

app/Http/Controllers/NotesController.php

@@ -4,15 +4,13 @@ namespace App\Http\Controllers;
 
 use App\Models\Actionlog;
 use App\Models\Asset;
-use App\Models\Maintenance;
-use Illuminate\Http\RedirectResponse;
 use Illuminate\Http\Request;
 use Illuminate\Support\Facades\Auth;
 use Illuminate\Validation\Rule;
 
 class NotesController extends Controller
 {
-    public function store(Request $request): RedirectResponse
+    public function store(Request $request)
     {
         $this->authorize('update', Asset::class);
 
@@ -21,19 +19,13 @@ class NotesController extends Controller
             'note' => 'required|string|max:50000',
             'type' => [
                 'required',
-                Rule::in(['asset', 'maintenance']),
+                Rule::in(['asset']),
             ],
         ]);
 
-        if ($validated['type'] === 'maintenance') {
-            $item = Maintenance::findOrFail($validated['id']);
-            $this->authorize('update', $item->asset);
-            $redirect = redirect()->route('maintenances.show', $validated['id']);
-        } else {
-            $item = Asset::findOrFail($validated['id']);
-            $this->authorize('update', $item);
-            $redirect = redirect()->route('hardware.show', $validated['id']);
-        }
+        $item = Asset::findOrFail($validated['id']);
+
+        $this->authorize('update', $item);
 
         $logaction = new Actionlog;
         $logaction->item_id = $item->id;
@@ -42,6 +34,9 @@ class NotesController extends Controller
         $logaction->created_by = Auth::id();
         $logaction->logaction('note added');
 
-        return $redirect->withFragment('notes')->with('success', trans('general.note_added'));
+        return redirect()
+            ->route('hardware.show', $validated['id'])
+            ->withFragment('history')
+            ->with('success', trans('general.note_added'));
     }
 }

app/Http/Controllers/ProfileController.php

@@ -8,7 +8,6 @@ use App\Models\Asset;
 use App\Models\Setting;
 use App\Models\User;
 use App\Notifications\CurrentInventory;
-use App\Rules\CssColor;
 use Illuminate\Contracts\View\View;
 use Illuminate\Http\RedirectResponse;
 use Illuminate\Http\Request;
@@ -64,12 +63,6 @@ class ProfileController extends Controller
 
         $user->enable_sounds = $request->input('enable_sounds', false);
         $user->enable_confetti = $request->input('enable_confetti', false);
-        $request->validate([
-            'link_light_color' => ['nullable', new CssColor],
-            'link_dark_color' => ['nullable', new CssColor],
-            'nav_link_color' => ['nullable', new CssColor],
-        ]);
-
         $user->link_light_color = $request->input('link_light_color', '#296282');
         $user->link_dark_color = $request->input('link_dark_color', '#296282');
         $user->nav_link_color = $request->input('nav_link_color', '#FFFFFF');

app/Http/Controllers/QrCodeController.php

@@ -1,66 +0,0 @@
-<?php
-
-namespace App\Http\Controllers;
-
-use App\Helpers\Helper;
-use App\Models\Setting;
-use Com\Tecnick\Barcode\Barcode;
-use Illuminate\Http\Response;
-use Symfony\Component\HttpFoundation\BinaryFileResponse;
-
-class QrCodeController extends Controller
-{
-    public static $map_show_route = [
-        'accessories' => 'accessories.show',
-        'assets' => 'hardware.show',
-        'companies' => 'companies.show',
-        'components' => 'components.show',
-        'consumables' => 'consumables.show',
-        'hardware' => 'hardware.show',
-        'licenses' => 'licenses.show',
-        'locations' => 'locations.show',
-        'models' => 'models.show',
-        'users' => 'users.show',
-    ];
-
-    public function show($object_type, $id): Response|BinaryFileResponse|string|bool
-    {
-        $settings = Setting::getSettings();
-
-        if ($settings->label2_2d_type === 'none') {
-            return false;
-        }
-
-        if (! array_key_exists($object_type, self::$map_show_route)) {
-            return $object_type.' is not a valid type.';
-        }
-
-        $object = self::$map_object_type[$object_type]::withTrashed()->find($id);
-
-        if (! $object) {
-            return 'That item is invalid';
-        }
-
-        $this->authorize('view', $object);
-
-        $size = Helper::barcodeDimensions($settings->label2_2d_type);
-        $qr_file = public_path().'/uploads/barcodes/qr-'.str_slug($object_type).'-'.str_slug($id).'.png';
-
-        if (file_exists($qr_file)) {
-            return response()->file($qr_file, ['Content-type' => 'image/png']);
-        }
-
-        $barcode = new Barcode;
-        $barcode_obj = $barcode->getBarcodeObj(
-            $settings->label2_2d_type,
-            route(self::$map_show_route[$object_type], $id),
-            $size['height'],
-            $size['width'],
-            'black',
-            [-2, -2, -2, -2]
-        );
-        file_put_contents($qr_file, $barcode_obj->getPngData());
-
-        return response($barcode_obj->getPngData())->header('Content-type', 'image/png');
-    }
-}

app/Http/Controllers/ReportsController.php

@@ -36,6 +36,8 @@ use Illuminate\Support\Facades\Gate;
 use Illuminate\Support\Facades\Log;
 use Illuminate\Support\Facades\Mail;
 use League\Csv\EscapeFormula;
+use League\Csv\Reader;
+use League\Csv\Writer;
 use Symfony\Component\HttpFoundation\StreamedResponse;
 
 /**
@@ -103,46 +105,36 @@ class ReportsController extends Controller
      * @see ManufacturersController::getDatatable() method that generates the JSON response
      * @since [v1.0]
      */
-    public function exportAccessoryReport(): StreamedResponse
+    public function exportAccessoryReport(): Response
     {
         $this->authorize('reports.view');
+        $accessories = Accessory::orderBy('created_at', 'DESC')->get();
 
-        $response = new StreamedResponse(function () {
-            $handle = fopen('php://output', 'w');
+        $rows = [];
+        $header = [
+            trans('admin/accessories/table.title'),
+            trans('admin/accessories/general.accessory_category'),
+            trans('admin/accessories/general.total'),
+            trans('admin/accessories/general.remaining'),
+        ];
+        $header = array_map('trim', $header);
+        $rows[] = implode(', ', $header);
 
-            $header = [
-                trans('admin/accessories/table.title'),
-                trans('admin/accessories/general.accessory_category'),
-                trans('admin/accessories/general.total'),
-                trans('admin/accessories/general.remaining'),
-            ];
-            fputcsv($handle, $header);
+        // Row per accessory
+        foreach ($accessories as $accessory) {
+            $row = [];
+            $row[] = e($accessory->accessory_name);
+            $row[] = e($accessory->accessory_category);
+            $row[] = e($accessory->total);
+            $row[] = e($accessory->remaining);
 
-            $formatter = new EscapeFormula('`');
+            $rows[] = implode(',', $row);
+        }
 
-            Accessory::with('category')->orderBy('created_at', 'DESC')
-                ->chunk(500, function ($accessories) use ($handle, $formatter) {
-                    foreach ($accessories as $accessory) {
-                        $row = [
-                            $accessory->name,
-                            $accessory->category?->name,
-                            $accessory->qty,
-                            $accessory->numRemaining(),
-                        ];
-
-                        if (config('app.escape_formulas') === false) {
-                            fputcsv($handle, $row);
-                        } else {
-                            fputcsv($handle, $formatter->escapeRecord($row));
-                        }
-                    }
-                });
-
-            fclose($handle);
-        }, 200, [
-            'Content-Type' => 'text/csv',
-            'Content-Disposition' => 'attachment; filename="accessories-report-'.date('Y-m-d-his').'.csv"',
-        ]);
+        $csv = implode("\n", $rows);
+        $response = response()->make($csv, 200);
+        $response->header('Content-Type', 'text/csv');
+        $response->header('Content-disposition', 'attachment;filename=report.csv');
 
         return $response;
     }
@@ -171,80 +163,74 @@ class ReportsController extends Controller
      *
      * @since [v1.0]
      */
-    public function exportDeprecationReport(): StreamedResponse
+    public function exportDeprecationReport(): Response
     {
         $this->authorize('reports.view');
+        // Grab all the assets
+        $assets = Asset::with('model', 'assignedTo', 'status', 'defaultLoc', 'assetlog')
+            ->orderBy('created_at', 'DESC')->get();
 
-        $response = new StreamedResponse(function () {
-            $handle = fopen('php://output', 'w');
-            $formatter = new EscapeFormula('`');
+        $csv = Writer::createFromFileObject(new \SplTempFileObject);
+        $csv->setOutputBOM(Reader::BOM_UTF16_BE);
 
-            $header = [
-                trans('admin/hardware/table.asset_tag'),
-                trans('admin/hardware/table.title'),
-                trans('admin/hardware/table.serial'),
-                trans('admin/hardware/table.checkoutto'),
-                trans('admin/hardware/table.location'),
-                trans('admin/hardware/table.purchase_date'),
-                trans('admin/hardware/table.purchase_cost'),
-                trans('admin/hardware/table.book_value'),
-                trans('admin/hardware/table.diff'),
-            ];
-            fputcsv($handle, $header);
+        $rows = [];
 
-            Asset::with('model', 'assignedTo', 'status', 'defaultLoc', 'assetlog')
-                ->orderBy('created_at', 'DESC')
-                ->chunk(500, function ($assets) use ($handle, $formatter) {
-                    foreach ($assets as $asset) {
-                        $currency = $asset->location
-                            ? $asset->location->currency
-                            : Setting::getSettings()->default_currency;
+        // Create the header row
+        $header = [
+            trans('admin/hardware/table.asset_tag'),
+            trans('admin/hardware/table.title'),
+            trans('admin/hardware/table.serial'),
+            trans('admin/hardware/table.checkoutto'),
+            trans('admin/hardware/table.location'),
+            trans('admin/hardware/table.purchase_date'),
+            trans('admin/hardware/table.purchase_cost'),
+            trans('admin/hardware/table.book_value'),
+            trans('admin/hardware/table.diff'),
+        ];
 
-                        if ($target = $asset->assignedTo) {
-                            $assignedTo = $target->display_name;
-                        } else {
-                            $assignedTo = '';
-                        }
+        // we insert the CSV header
+        $csv->insertOne($header);
 
-                        if (($asset->assigned_to > 0) && ($location = $asset->location)) {
-                            if ($location->city) {
-                                $locationStr = $location->city.', '.$location->state;
-                            } elseif ($location->name) {
-                                $locationStr = $location->name;
-                            } else {
-                                $locationStr = '';
-                            }
-                        } else {
-                            $locationStr = '';
-                        }
+        // Create a row per asset
+        foreach ($assets as $asset) {
+            $row = [];
+            $row[] = e($asset->asset_tag);
+            $row[] = e($asset->name);
+            $row[] = e($asset->serial);
 
-                        $row = [
-                            $asset->asset_tag,
-                            $asset->name,
-                            $asset->serial,
-                            $assignedTo,
-                            $locationStr,
-                            Helper::getFormattedDateObject($asset->purchase_date, 'date', false),
-                            $currency.Helper::formatCurrencyOutput($asset->purchase_cost),
-                            $currency.Helper::formatCurrencyOutput($asset->getDepreciatedValue()),
-                            $currency.Helper::formatCurrencyOutput($asset->purchase_cost - $asset->getDepreciatedValue()),
-                        ];
+            if ($target = $asset->assignedTo) {
+                $row[] = e($target->display_name);
+            } else {
+                $row[] = ''; // Empty string if unassigned
+            }
 
-                        if (config('app.escape_formulas') === false) {
-                            fputcsv($handle, $row);
-                        } else {
-                            fputcsv($handle, $formatter->escapeRecord($row));
-                        }
-                    }
-                });
+            if (($asset->assigned_to > 0) && ($location = $asset->location)) {
+                if ($location->city) {
+                    $row[] = e($location->city).', '.e($location->state);
+                } elseif ($location->name) {
+                    $row[] = e($location->name);
+                } else {
+                    $row[] = '';
+                }
+            } else {
+                $row[] = '';  // Empty string if location is not set
+            }
 
-            fclose($handle);
-        }, 200, [
-            'Content-Type' => 'text/csv',
-            'Content-Disposition' => 'attachment; filename="depreciation-report-'.date('Y-m-d-his').'.csv"',
-        ]);
+            if ($asset->location) {
+                $currency = e($asset->location->currency);
+            } else {
+                $currency = e(Setting::getSettings()->default_currency);
+            }
 
-        return $response;
+            $row[] = Helper::getFormattedDateObject($asset->purchase_date, 'date', false);
+            $row[] = $currency.Helper::formatCurrencyOutput($asset->purchase_cost);
+            $row[] = $currency.Helper::formatCurrencyOutput($asset->getDepreciatedValue());
+            $row[] = $currency.Helper::formatCurrencyOutput(($asset->purchase_cost - $asset->getDepreciatedValue()));
+            $csv->insertOne($row);
+        }
+
+        $csv->output('depreciation-report-'.date('Y-m-d').'.csv');
+        exit;
     }
 
     /**
@@ -409,52 +395,45 @@ class ReportsController extends Controller
      *
      * @since [v1.0]
      */
-    public function exportLicenseReport(): StreamedResponse
+    public function exportLicenseReport(): Response
     {
         $this->authorize('reports.view');
+        $licenses = License::orderBy('created_at', 'DESC')->get();
 
-        $response = new StreamedResponse(function () {
-            $handle = fopen('php://output', 'w');
-            $formatter = new EscapeFormula('`');
+        $rows = [];
+        $header = [
+            trans('admin/licenses/table.title'),
+            trans('admin/licenses/table.serial'),
+            trans('admin/licenses/form.seats'),
+            trans('admin/licenses/form.remaining_seats'),
+            trans('admin/licenses/form.expiration'),
+            trans('general.purchase_date'),
+            trans('general.depreciation'),
+            trans('general.purchase_cost'),
+        ];
 
-            $header = [
-                trans('admin/licenses/table.title'),
-                trans('admin/licenses/table.serial'),
-                trans('admin/licenses/form.seats'),
-                trans('admin/licenses/form.remaining_seats'),
-                trans('admin/licenses/form.expiration'),
-                trans('general.purchase_date'),
-                trans('general.depreciation'),
-                trans('general.purchase_cost'),
-            ];
-            fputcsv($handle, $header);
+        $header = array_map('trim', $header);
+        $rows[] = implode(', ', $header);
 
-            License::orderBy('created_at', 'DESC')->chunk(500, function ($licenses) use ($handle, $formatter) {
-                foreach ($licenses as $license) {
-                    $row = [
-                        $license->name,
-                        $license->serial,
-                        $license->seats,
-                        $license->remaincount(),
-                        $license->expiration_date,
-                        $license->purchase_date,
-                        ($license->depreciation != '') ? $license->depreciation->name : '',
-                        Helper::formatCurrencyOutput($license->purchase_cost),
-                    ];
+        // Row per license
+        foreach ($licenses as $license) {
+            $row = [];
+            $row[] = e($license->name);
+            $row[] = e($license->serial);
+            $row[] = e($license->seats);
+            $row[] = $license->remaincount();
+            $row[] = $license->expiration_date;
+            $row[] = $license->purchase_date;
+            $row[] = ($license->depreciation != '') ? '' : e($license->depreciation->name);
+            $row[] = '"'.Helper::formatCurrencyOutput($license->purchase_cost).'"';
 
-                    if (config('app.escape_formulas') === false) {
-                        fputcsv($handle, $row);
-                    } else {
-                        fputcsv($handle, $formatter->escapeRecord($row));
-                    }
-                }
-            });
+            $rows[] = implode(',', $row);
+        }
 
-            fclose($handle);
-        }, 200, [
-            'Content-Type' => 'text/csv',
-            'Content-Disposition' => 'attachment; filename="licenses-report-'.date('Y-m-d-his').'.csv"',
-        ]);
+        $csv = implode("\n", $rows);
+        $response = response()->make($csv, 200);
+        $response->header('Content-Type', 'text/csv');
+        $response->header('Content-disposition', 'attachment;filename=report.csv');
 
         return $response;
     }
@@ -799,11 +778,12 @@ class ReportsController extends Controller
                 $checkout_start = Carbon::parse($request->input('checkout_date_start'))->startOfDay();
                 $checkout_end = Carbon::parse($request->input('checkout_date_end', now()))->endOfDay();
 
-                $actionlogassets = Actionlog::select('id')->where('action_type', '=', 'checkout')
-                    ->where('item_type', '=', Asset::class)
-                    ->whereBetween('action_date', [$checkout_start, $checkout_end]); // we are *not* doing ->get()...
+                $actionlogassets = Actionlog::where('action_type', '=', 'checkout')
+                    ->where('item_type', 'LIKE', '%Asset%')
+                    ->whereBetween('action_date', [$checkout_start, $checkout_end])
+                    ->pluck('item_id');
 
-                $assets->whereIn('id', $actionlogassets); // ...because this _should_ act as a 'subquery'
+                $assets->whereIn('assets.id', $actionlogassets);
             }
 
             if (($request->filled('checkin_date_start'))) {
@@ -1192,60 +1172,56 @@ class ReportsController extends Controller
      *
      * @version v1.0
      */
-    public function exportMaintenancesReport(): StreamedResponse
+    public function exportMaintenancesReport(): Response
     {
         $this->authorize('reports.view');
+        // Grab all the improvements
+        $Maintenances = Maintenance::with('asset', 'supplier')
+            ->orderBy('created_at', 'DESC')
+            ->get();
 
-        $response = new StreamedResponse(function () {
-            $handle = fopen('php://output', 'w');
-            $formatter = new EscapeFormula('`');
+        $rows = [];
 
-            $header = [
-                trans('admin/hardware/table.asset_tag'),
-                trans('admin/maintenances/table.asset_name'),
-                trans('general.supplier'),
-                trans('admin/maintenances/form.asset_maintenance_type'),
-                trans('admin/maintenances/form.title'),
-                trans('admin/maintenances/form.start_date'),
-                trans('admin/maintenances/form.completion_date'),
-                trans('admin/maintenances/form.asset_maintenance_time'),
-                trans('admin/maintenances/form.cost'),
-            ];
-            fputcsv($handle, $header);
+        $header = [
+            trans('admin/hardware/table.asset_tag'),
+            trans('admin/maintenances/table.asset_name'),
+            trans('general.supplier'),
+            trans('admin/maintenances/form.asset_maintenance_type'),
+            trans('admin/maintenances/form.title'),
+            trans('admin/maintenances/form.start_date'),
+            trans('admin/maintenances/form.completion_date'),
+            trans('admin/maintenances/form.asset_maintenance_time'),
+            trans('admin/maintenances/form.cost'),
+        ];
 
-            Maintenance::with('asset', 'supplier')
-                ->orderBy('created_at', 'DESC')
-                ->chunk(500, function ($maintenances) use ($handle, $formatter) {
-                    foreach ($maintenances as $maintenance) {
-                        $improvementTime = is_null($maintenance->asset_maintenance_time)
-                            ? (int) Carbon::now()->diffInDays(Carbon::parse($maintenance->start_date), true)
-                            : (int) $maintenance->asset_maintenance_time;
+        $header = array_map('trim', $header);
+        $rows[] = implode(',', $header);
 
-                        $row = [
-                            $maintenance->asset->asset_tag,
-                            $maintenance->asset->name,
-                            $maintenance->supplier->name,
-                            $maintenance->improvement_type,
-                            $maintenance->name,
-                            $maintenance->start_date,
-                            $maintenance->completion_date,
-                            $improvementTime,
-                            trans('general.currency').Helper::formatCurrencyOutput($maintenance->cost),
-                        ];
+        foreach ($Maintenances as $maintenance) {
+            $row = [];
+            $row[] = str_replace(',', '', e($maintenance->asset->asset_tag));
+            $row[] = str_replace(',', '', e($maintenance->asset->name));
+            $row[] = str_replace(',', '', e($maintenance->supplier->name));
+            $row[] = e($maintenance->improvement_type);
+            $row[] = e($maintenance->name);
+            $row[] = e($maintenance->start_date);
+            $row[] = e($maintenance->completion_date);
+            if (is_null($maintenance->asset_maintenance_time)) {
+                $improvementTime = (int) Carbon::now()
+                    ->diffInDays(Carbon::parse($maintenance->start_date), true);
+            } else {
+                $improvementTime = (int) $maintenance->asset_maintenance_time;
+            }
+            $row[] = $improvementTime;
+            $row[] = trans('general.currency').Helper::formatCurrencyOutput($maintenance->cost);
+            $rows[] = implode(',', $row);
+        }
 
-                        if (config('app.escape_formulas') === false) {
-                            fputcsv($handle, $row);
-                        } else {
-                            fputcsv($handle, $formatter->escapeRecord($row));
-                        }
-                    }
-                });
-
-            fclose($handle);
-        }, 200, [
-            'Content-Type' => 'text/csv',
-            'Content-Disposition' => 'attachment; filename="maintenances-report-'.date('Y-m-d-his').'.csv"',
-        ]);
+        // spit out a csv
+        $csv = implode("\n", $rows);
+        $response = response()->make($csv, 200);
+        $response->header('Content-Type', 'text/csv');
+        $response->header('Content-disposition', 'attachment;filename=report.csv');
 
         return $response;
     }
@@ -1324,11 +1300,6 @@ class ReportsController extends Controller
             // Redirect to the unaccepted items report page with error
             return redirect()->route('reports/unaccepted_assets')->with('error', trans('general.bad_data'));
         }
-
-        if (! $this->currentUserCanAccessAcceptance($acceptance)) {
-            return redirect()->route('reports/unaccepted_assets')->with('error', trans('general.insufficient_permissions'));
-        }
-
         $item = $acceptance->checkoutable;
         $assignee = $acceptance->assignedTo ?? $item->assignedTo ?? null;
         $email = $assignee?->email;
@@ -1363,33 +1334,6 @@ class ReportsController extends Controller
         return redirect()->route('reports/unaccepted_assets')->with('success', trans('admin/reports/general.reminder_sent'));
     }
 
-    private function currentUserCanAccessAcceptance(CheckoutAcceptance $acceptance): bool
-    {
-        if (Setting::getSettings()->full_multiple_companies_support != '1') {
-            return true;
-        }
-
-        $user = auth()->user();
-
-        if (! $user->company_id || $user->isSuperUser()) {
-            return true;
-        }
-
-        // Bypass Eloquent global scopes so cross-company items are still found
-        $checkoutableType = $acceptance->checkoutable_type;
-        $checkoutable = $checkoutableType::withoutGlobalScopes()->find($acceptance->checkoutable_id);
-
-        if ($checkoutable instanceof LicenseSeat) {
-            $itemCompanyId = License::withoutGlobalScopes()
-                ->where('id', $checkoutable->license_id)
-                ->value('company_id');
-        } else {
-            $itemCompanyId = $checkoutable?->company_id;
-        }
-
-        return $itemCompanyId === null || (int) $itemCompanyId === (int) $user->company_id;
-    }
-
     private function getCheckoutMailType(CheckoutAcceptance $acceptance, $logItem): Mailable
     {
         $lookup = [
@@ -1422,21 +1366,11 @@ class ReportsController extends Controller
     {
         $this->authorize('reports.view');
 
-        $acceptance = CheckoutAcceptance::pending()
-            ->with(['checkoutable' => function (MorphTo $morphTo) {
-                $morphTo->morphWith([LicenseSeat::class => ['license']]);
-            }])
-            ->find($acceptanceId);
-
-        if (! $acceptance) {
+        if (! $acceptance = CheckoutAcceptance::pending()->find($acceptanceId)) {
             // Redirect to the unaccepted assets report page with error
             return redirect()->route('reports/unaccepted_assets')->with('error', trans('general.bad_data'));
         }
 
-        if (! $this->currentUserCanAccessAcceptance($acceptance)) {
-            return redirect()->route('reports/unaccepted_assets')->with('error', trans('general.insufficient_permissions'));
-        }
-
         if ($acceptance->delete()) {
             return redirect()->route('reports/unaccepted_assets')->with('success', trans('admin/reports/general.acceptance_deleted'));
         } else {

app/Http/Controllers/SettingsController.php

@@ -19,7 +19,6 @@ use App\Models\Group;
 use App\Models\Setting;
 use App\Models\User;
 use App\Notifications\MailTest;
-use App\Rules\CssColor;
 use Illuminate\Contracts\View\View;
 use Illuminate\Http\JsonResponse;
 use Illuminate\Http\RedirectResponse;
@@ -93,12 +92,10 @@ class SettingsController extends Controller
         $old_locations_fmcs = $setting->scope_locations_fmcs;
         $setting->full_multiple_companies_support = $request->input('full_multiple_companies_support', '0');
         $setting->scope_locations_fmcs = $request->input('scope_locations_fmcs', '0');
-        $setting->null_company_is_floater = $request->input('null_company_is_floater', '0');
 
-        // These options make no sense without FullMultipleCompanySupport
+        // Backward compatibility for locations makes no sense without FullMultipleCompanySupport
         if (! $setting->full_multiple_companies_support) {
             $setting->scope_locations_fmcs = '0';
-            $setting->null_company_is_floater = '0';
         }
 
         // check for inconsistencies when activating scoped locations
@@ -192,13 +189,6 @@ class SettingsController extends Controller
                 $request->validate(['site_name' => 'required']);
             }
 
-            $request->validate([
-                'header_color' => ['nullable', new CssColor],
-                'link_light_color' => ['nullable', new CssColor],
-                'link_dark_color' => ['nullable', new CssColor],
-                'nav_link_color' => ['nullable', new CssColor],
-            ]);
-
             $setting->header_color = $request->input('header_color', '#3c8dbc');
             $setting->link_light_color = $request->input('link_light_color', '#296282');
             $setting->link_dark_color = $request->input('link_dark_color', '#5fa4cc');

app/Http/Controllers/SetupController.php

@@ -6,7 +6,6 @@ use App\Http\Requests\SetupUserRequest;
 use App\Models\Setting;
 use App\Models\User;
 use App\Notifications\FirstAdminNotification;
-use App\Rules\CssColor;
 use Illuminate\Contracts\View\View;
 use Illuminate\Http\RedirectResponse;
 use Illuminate\Http\Response;
@@ -167,12 +166,6 @@ class SetupController extends Controller
         $settings->alerts_enabled = 1;
         $settings->pwd_secure_min = 10;
         $settings->brand = 1;
-        $request->validate([
-            'link_light_color' => ['nullable', new CssColor],
-            'link_dark_color' => ['nullable', new CssColor],
-            'nav_link_color' => ['nullable', new CssColor],
-        ]);
-
         $settings->link_light_color = $request->input('link_light_color', '#296282');
         $settings->link_dark_color = $request->input('link_dark_color', '#296282');
         $settings->nav_link_color = $request->input('nav_link_color', '#FFFFFF');

app/Http/Controllers/UploadedFilesController.php

@@ -101,13 +101,11 @@ class UploadedFilesController extends Controller
         }
 
         if (request('inline') == 'true') {
-            $path = self::$map_storage_path[$object_type];
+            $headers = [
+                'Content-Disposition' => 'inline',
+            ];
 
-            if (! StorageHelper::allowSafeInline($path.$log->filename)) {
-                return StorageHelper::downloader($path.$log->filename);
-            }
-
-            return Storage::download($path.$log->filename, $log->filename, ['Content-Disposition' => 'inline']);
+            return Storage::download(self::$map_storage_path[$object_type].$log->filename, $log->filename, $headers);
         }
 
         return StorageHelper::downloader(self::$map_storage_path[$object_type].$log->filename);

app/Http/Controllers/Users/BulkUsersController.php

@@ -8,7 +8,6 @@ use App\Http\Controllers\Controller;
 use App\Models\Accessory;
 use App\Models\Actionlog;
 use App\Models\Asset;
-use App\Models\Company;
 use App\Models\ConsumableAssignment;
 use App\Models\Group;
 use App\Models\License;
@@ -169,21 +168,16 @@ class BulkUsersController extends Controller
 
         $this->conditionallyAddItem('location_id')
             ->conditionallyAddItem('department_id')
+            ->conditionallyAddItem('company_id')
             ->conditionallyAddItem('locale')
             ->conditionallyAddItem('remote')
+            ->conditionallyAddItem('ldap_import')
+            ->conditionallyAddItem('activated')
             ->conditionallyAddItem('display_name')
             ->conditionallyAddItem('start_date')
             ->conditionallyAddItem('end_date')
             ->conditionallyAddItem('city')
-            ->conditionallyAddItem('autoassign_licenses')
-            ->conditionallyAddItem('phone')
-            ->conditionallyAddItem('jobtitle')
-            ->conditionallyAddItem('address')
-            ->conditionallyAddItem('state')
-            ->conditionallyAddItem('country')
-            ->conditionallyAddItem('zip')
-            ->conditionallyAddItem('website')
-            ->conditionallyAddItem('notes');
+            ->conditionallyAddItem('autoassign_licenses');
 
         // If the manager_id is one of the users being updated, generate a warning.
         if (array_search($request->input('manager_id'), $user_raw_array)) {
@@ -208,7 +202,7 @@ class BulkUsersController extends Controller
             $this->update_array['manager_id'] = null;
         }
 
-        if ($request->input('null_company_ids') == '1') {
+        if ($request->input('null_company_id') == '1') {
             $this->update_array['company_id'] = null;
         }
 
@@ -228,46 +222,6 @@ class BulkUsersController extends Controller
             $this->update_array['display_name'] = null;
         }
 
-        if ($request->input('null_city') == '1') {
-            $this->update_array['city'] = null;
-        }
-
-        if ($request->input('null_phone') == '1') {
-            $this->update_array['phone'] = null;
-        }
-
-        if ($request->input('null_jobtitle') == '1') {
-            $this->update_array['jobtitle'] = null;
-        }
-
-        if ($request->input('null_employee_num') == '1') {
-            $this->update_array['employee_num'] = null;
-        }
-
-        if ($request->input('null_address') == '1') {
-            $this->update_array['address'] = null;
-        }
-
-        if ($request->input('null_state') == '1') {
-            $this->update_array['state'] = null;
-        }
-
-        if ($request->input('null_country') == '1') {
-            $this->update_array['country'] = null;
-        }
-
-        if ($request->input('null_zip') == '1') {
-            $this->update_array['zip'] = null;
-        }
-
-        if ($request->input('null_website') == '1') {
-            $this->update_array['website'] = null;
-        }
-
-        if ($request->input('null_notes') == '1') {
-            $this->update_array['notes'] = null;
-        }
-
         if (! $manager_conflict) {
             $this->conditionallyAddItem('manager_id');
         }
@@ -281,50 +235,11 @@ class BulkUsersController extends Controller
                 ->update(['location_id' => $this->update_array['location_id']]);
         }
 
-        // Handle company pivot sync separately from the mass update.
-        // company_ids[] comes from the multi-select; null_company_ids clears all memberships.
-        $bulkCompanyIds = array_filter(array_map('intval', (array) $request->input('company_ids', [])));
-        $clearCompanies = $request->input('null_company_ids') == '1';
+        // Only sync groups if groups were selected
+        if ($request->filled('groups')) {
 
-        if ($bulkCompanyIds || $clearCompanies) {
-            $allowedIds = Company::getIdsForCurrentUser($bulkCompanyIds);
-            // Also update the scalar company_id column for display/backward compat.
-            $scalarCompanyId = $allowedIds[0] ?? null;
-            User::whereIn('id', $user_raw_array)->where('id', '!=', auth()->id())
-                ->update(['company_id' => $scalarCompanyId]);
             foreach ($users as $user) {
-                if ($clearCompanies && ! auth()->user()->isSuperUser() && Company::isFullMultipleCompanySupportEnabled()) {
-                    // Non-superusers can only detach companies they belong to; sync([]) would
-                    // also wipe memberships for companies outside their scope.
-                    $user->companies()->detach(Company::getIdsForCurrentUser(
-                        $user->companies()->pluck('companies.id')->toArray()
-                    ));
-                } else {
-                    $user->companies()->sync($allowedIds);
-                }
-            }
-        }
-
-        // Fields that require canEditAuthFields (non-admins cannot touch admins/superusers,
-        // admins cannot touch superusers) must be applied per-user, not via mass update.
-        foreach ($users as $user) {
-            if (auth()->user()->can('canEditAuthFields', $user) && auth()->user()->can('editableOnDemo')) {
-                $authFieldUpdate = [];
-                if ($request->filled('activated')) {
-                    $authFieldUpdate['activated'] = $request->input('activated');
-                }
-                if ($request->filled('ldap_import')) {
-                    $authFieldUpdate['ldap_import'] = $request->input('ldap_import');
-                }
-                if ($request->filled('email')) {
-                    $authFieldUpdate['email'] = $request->input('email');
-                } elseif ($request->input('null_email') == '1') {
-                    $authFieldUpdate['email'] = null;
-                }
-                if (! empty($authFieldUpdate)) {
-                    $user->update($authFieldUpdate);
-                }
-                if ($request->filled('groups') && auth()->user()->isSuperUser()) {
+                if (auth()->user()->can('canEditAuthFields', $user) && auth()->user()->can('editableOnDemo')) {
                     $user->groups()->sync($request->input('groups'));
                 }
             }
@@ -395,31 +310,6 @@ class BulkUsersController extends Controller
             return redirect()->route('users.index')->with('error', 'No status selected');
         }
 
-        // Enforce per-item checkin permissions before touching anything (catches FMCS company scoping).
-        foreach ($assets as $asset) {
-            if (auth()->user()->cannot('checkin', $asset)) {
-                return redirect()->route('users.index')->with('error', trans('general.insufficient_permissions'));
-            }
-        }
-
-        $licenseModels = License::whereIn('id', $licenses->pluck('license_id')->unique())->get();
-        foreach ($licenseModels as $license) {
-            if (auth()->user()->cannot('checkin', $license)) {
-                return redirect()->route('users.index')->with('error', trans('general.insufficient_permissions'));
-            }
-        }
-
-        $accessoryModels = Accessory::whereIn('id', $accessoryUserRows->pluck('accessory_id')->unique())->get();
-        foreach ($accessoryModels as $accessory) {
-            if (auth()->user()->cannot('checkin', $accessory)) {
-                return redirect()->route('users.index')->with('error', trans('general.insufficient_permissions'));
-            }
-        }
-
-        if ($request->input('delete_user') == '1' && $users->isNotEmpty() && auth()->user()->cannot('delete', User::class)) {
-            return redirect()->route('users.index')->with('error', trans('general.insufficient_permissions'));
-        }
-
         $this->logItemCheckinAndDelete($assets, Asset::class);
         $this->logAccessoriesCheckin($accessoryUserRows);
         $this->logItemCheckinAndDelete($licenses, License::class);
@@ -508,7 +398,7 @@ class BulkUsersController extends Controller
      */
     public function merge(Request $request)
     {
-        $this->authorize('delete', User::class);
+        $this->authorize('update', User::class);
 
         if (config('app.lock_passwords')) {
             return redirect()->route('users.index')->with('error', trans('general.feature_disabled'));
@@ -526,17 +416,9 @@ class BulkUsersController extends Controller
         $users_to_merge = User::whereIn('id', $user_ids_to_merge)->with('assets', 'manager', 'userlog', 'licenses', 'consumables', 'accessories', 'managedLocations', 'uploads', 'acceptances')->get();
         $admin = User::find(auth()->id());
 
-        if (! auth()->user()->can('canEditAuthFields', $merge_into_user) || ! auth()->user()->can('editableOnDemo')) {
-            return redirect()->route('users.index')->with('error', trans('general.insufficient_permissions'));
-        }
-
         // Walk users
         foreach ($users_to_merge as $user_to_merge) {
 
-            if (! auth()->user()->can('canEditAuthFields', $user_to_merge) || ! auth()->user()->can('editableOnDemo')) {
-                return redirect()->route('users.index')->with('error', trans('general.insufficient_permissions'));
-            }
-
             foreach ($user_to_merge->assets as $asset) {
                 Log::debug('Updating asset: '.$asset->asset_tag.' to '.$merge_into_user->id);
                 $asset->assigned_to = $request->input('merge_into_id');
@@ -579,12 +461,6 @@ class BulkUsersController extends Controller
                 $managedLocation->save();
             }
 
-            // Carry over company pivot memberships from the merged user into the target.
-            $mergedCompanyIds = $user_to_merge->companies()->pluck('companies.id')->toArray();
-            if (! empty($mergedCompanyIds)) {
-                $merge_into_user->companies()->syncWithoutDetaching($mergedCompanyIds);
-            }
-
             $user_to_merge->delete();
 
             event(new UserMerged($user_to_merge, $merge_into_user, $admin));

app/Http/Controllers/Users/UsersController.php

@@ -10,14 +10,11 @@ use App\Http\Requests\DeleteUserRequest;
 use App\Http\Requests\ImageUploadRequest;
 use App\Http\Requests\SaveUserRequest;
 use App\Mail\UnacceptedAssetReminderMail;
-use App\Models\Accessory;
 use App\Models\Actionlog;
 use App\Models\Asset;
 use App\Models\CheckoutAcceptance;
 use App\Models\Company;
-use App\Models\Consumable;
 use App\Models\Group;
-use App\Models\License;
 use App\Models\Setting;
 use App\Models\User;
 use App\Notifications\CurrentInventory;
@@ -126,7 +123,7 @@ class UsersController extends Controller
         $user->mobile = $request->input('mobile');
         $user->location_id = $request->input('location_id', null);
         $user->department_id = $request->input('department_id', null);
-        $companyIds = array_filter(array_map('intval', (array) ($request->input('company_ids') ?? ($request->filled('company_id') ? [$request->input('company_id')] : []))));
+        $user->company_id = Company::getIdForUser($request->input('company_id', null));
         $user->manager_id = $request->input('manager_id', null);
         $user->notes = $request->input('notes');
         $user->address = $request->input('address', null);
@@ -156,7 +153,6 @@ class UsersController extends Controller
         }
 
         if ($user->save()) {
-            $user->syncCompaniesWithLogging(Company::getIdsForCurrentUser($companyIds));
 
             if (($user->activated == '1') && ($user->email != '') && ($request->input('send_welcome') == '1')) {
 
@@ -168,7 +164,7 @@ class UsersController extends Controller
 
             }
 
-            if (auth()->user()->isSuperUser() && auth()->user()->can('editableOnDemo')) {
+            if (auth()->user()->can('canEditAuthFields', $user) && auth()->user()->can('editableOnDemo')) {
                 $user->groups()->sync($request->input('groups'));
             }
 
@@ -279,7 +275,7 @@ class UsersController extends Controller
         $user->phone = $request->input('phone');
         $user->mobile = $request->input('mobile');
         $user->location_id = $request->input('location_id', null);
-        $companyIds = array_filter(array_map('intval', (array) ($request->input('company_ids') ?? ($request->filled('company_id') ? [$request->input('company_id')] : []))));
+        $user->company_id = Company::getIdForUser($request->input('company_id', null));
         $user->manager_id = $request->input('manager_id', null);
         $user->notes = $request->input('notes');
         $user->department_id = $request->input('department_id', null);
@@ -315,14 +311,12 @@ class UsersController extends Controller
                 $user->password = bcrypt($request->input('password'));
             }
 
-            if ($request->has('permission')) {
-                $user->permissions = json_encode(PreserveUnauthorizedPrivilegedPermissionsAction::run(
-                    requestedPermissions: NormalizePermissionsPayloadAction::run($request->input('permission')),
-                    authenticatedUser: $authenticatedUser,
-                    originalPermissions: $orig_permissions_array,
-                    targetUser: $user,
-                ));
-            }
+            $user->permissions = json_encode(PreserveUnauthorizedPrivilegedPermissionsAction::run(
+                requestedPermissions: NormalizePermissionsPayloadAction::run($request->input('permission')),
+                authenticatedUser: $authenticatedUser,
+                originalPermissions: $orig_permissions_array,
+                targetUser: $user,
+            ));
 
             // Only save groups if the user is a superuser
             if (auth()->user()->isSuperUser()) {
@@ -340,8 +334,6 @@ class UsersController extends Controller
         session()->put(['redirect_option' => $request->input('redirect_option')]);
 
         if ($user->save()) {
-            $user->syncCompaniesWithLogging(Company::getIdsForCurrentUser($companyIds));
-
             // Redirect to the user page
             return Helper::getRedirectOption($request, $user->id, 'Users')
                 ->with('success', trans('admin/users/message.success.update'));
@@ -486,7 +478,7 @@ class UsersController extends Controller
         $permissions = $request->input('permissions', []);
         app('request')->request->set('permissions', $permissions);
 
-        $user_to_clone = User::with('userloc', 'companies')->withTrashed()->find($user->id);
+        $user_to_clone = User::with('userloc')->withTrashed()->find($user->id);
         // Make sure they can view this particular user
         $this->authorize('view', $user_to_clone);
 
@@ -604,10 +596,10 @@ class UsersController extends Controller
                 'manager',
                 'groups',
                 'userloc',
-                'companies',
+                'company',
                 'createdBy'
             )->withCount(['managesUsers as manages_users_count', 'managedLocations as manages_locations_count'])
-                ->orderBy('created_at', 'DESC')
+            ->orderBy('created_at', 'DESC')
                 ->chunk(500, function ($users) use ($handle) {
 
                     $formatter = new EscapeFormula('`');
@@ -626,7 +618,7 @@ class UsersController extends Controller
                         // Add a new row with data
                         $values = [
                             $user->id,
-                            $user->companies->pluck('name')->implode('|'),
+                            ($user->company) ? $user->company->name : '',
                             $user->jobtitle,
                             $user->employee_num,
                             $user->first_name,
@@ -705,17 +697,9 @@ class UsersController extends Controller
     {
         $this->authorize('view', User::class);
 
-        $actor = auth()->user();
-        $canViewLicenses = $actor->can('view', License::class);
-        $canViewAccessories = $actor->can('view', Accessory::class);
-        $canViewConsumables = $actor->can('view', Consumable::class);
+        $user = User::withInventoryRelations($id)->first();
 
-        $user = User::withInventoryRelations($id, $canViewLicenses, $canViewAccessories, $canViewConsumables)->first();
-
-        $indirectItemsCount = $user?->assets?->flatMap->assignedAssets->count()
-            + $user?->assets?->flatMap->components->count()
-            + ($canViewLicenses ? $user?->assets?->flatMap->licenses->count() : 0)
-            + ($canViewAccessories ? $user?->assets?->flatMap->assignedAccessories->count() : 0);
+        $indirectItemsCount = $user?->assets?->flatMap->assignedAssets->count() + $user?->assets?->flatMap->components->count() + $user?->assets?->flatMap->licenses->count() + $user?->assets?->flatMap->assignedAccessories->count();
 
         if ($user) {
             $this->authorize('view', $user);

app/Http/Controllers/ViewAssetsController.php

@@ -19,7 +19,6 @@ use Illuminate\Contracts\View\View;
 use Illuminate\Http\RedirectResponse;
 use Illuminate\Http\Request;
 use Illuminate\Support\Collection;
-use Illuminate\Support\Facades\Log;
 
 /**
  * This controller handles all actions related to the ability for users
@@ -121,7 +120,6 @@ class ViewAssetsController extends Controller
             'consumables',
             'accessories',
             'licenses',
-            'companies',
         ])->find($selectedUserId);
 
         // If the user to view couldn't be found (shouldn't happen with proper logic), redirect with error
@@ -201,39 +199,21 @@ class ViewAssetsController extends Controller
 
         $settings = Setting::getSettings();
 
-        $is_admin = $user->isSuperUser() || $user->isAdmin();
-
-        if ($cancel_by_admin && ! $is_admin) {
-            return redirect()->back()->with('error', trans('general.insufficient_permissions'));
-        }
-
-        if (($item_request = $item->isRequestedBy($user)) || ($is_admin && $cancel_by_admin)) {
-            $item->cancelRequest($is_admin && $cancel_by_admin ? $requestingUser : null);
+        if (($item_request = $item->isRequestedBy($user)) || $cancel_by_admin) {
+            $item->cancelRequest($requestingUser);
             $data['item_quantity'] = ($item_request) ? $item_request->qty : 1;
             $logaction->logaction(ActionType::RequestCanceled);
 
             if (($settings->alert_email != '') && ($settings->alerts_enabled == '1') && (! config('app.lock_passwords'))) {
-                try {
-                    $settings->notify((new RequestAssetCancelation($data))->locale($settings->locale));
-                } catch (Exception $e) {
-                    Log::warning('Could not send request cancellation notification: '.$e->getMessage());
-                }
+                $settings->notify((new RequestAssetCancelation($data))->locale($settings->locale));
             }
 
             return redirect()->back()->with('success')->with('success', trans('admin/hardware/message.requests.canceled'));
         } else {
-            if ($fullItemType === Asset::class && is_null(Asset::RequestableAssets()->find($item->id))) {
-                return redirect()->back()->with('error', trans('admin/hardware/message.requests.error'));
-            }
-
             $item->request();
             if (($settings->alert_email != '') && ($settings->alerts_enabled == '1') && (! config('app.lock_passwords'))) {
                 $logaction->logaction('requested');
-                try {
-                    $settings->notify((new RequestAssetNotification($data))->locale($settings->locale));
-                } catch (Exception $e) {
-                    Log::warning('Could not send asset request notification: '.$e->getMessage());
-                }
+                $settings->notify((new RequestAssetNotification($data))->locale($settings->locale));
             }
 
             return redirect()->route('requestable-assets')->with('success')->with('success', trans('admin/hardware/message.requests.success'));

app/Http/Kernel.php

@@ -2,6 +2,7 @@
 
 namespace App\Http;
 
+use App\Http\Middleware\AssetCountForSidebar;
 use App\Http\Middleware\CheckColorSettings;
 use App\Http\Middleware\CheckForDebug;
 use App\Http\Middleware\CheckForSetup;
@@ -16,7 +17,6 @@ use App\Http\Middleware\PreventBackHistory;
 use App\Http\Middleware\RedirectIfAuthenticated;
 use App\Http\Middleware\SecurityHeaders;
 use App\Http\Middleware\SetAPIResponseHeaders;
-use App\Http\Middleware\SetPaginationDefaults;
 use App\Http\Middleware\TrimStrings;
 use App\Http\Middleware\TrustProxies;
 use App\Http\Middleware\VerifyCsrfToken;
@@ -74,6 +74,7 @@ class Kernel extends HttpKernel
             CheckUserIsActivated::class,
             CheckForTwoFactor::class,
             CreateFreshApiToken::class,
+            AssetCountForSidebar::class,
             CheckColorSettings::class,
             AuthenticateSession::class,
             SubstituteBindings::class,
@@ -83,7 +84,6 @@ class Kernel extends HttpKernel
             'auth:api',
             CheckLocale::class,
             LogAuthedUserHeader::class,
-            SetPaginationDefaults::class,
             SubstituteBindings::class,
         ],
 

app/Http/Middleware/AssetCountForSidebar.php

@@ -0,0 +1,119 @@
+<?php
+
+namespace App\Http\Middleware;
+
+use App\Models\Asset;
+use App\Models\Setting;
+use Closure;
+use Illuminate\Http\Request;
+use Illuminate\Support\Facades\Log;
+
+class AssetCountForSidebar
+{
+    /**
+     * Handle an incoming request.
+     *
+     * @param  Request  $request
+     * @return mixed
+     */
+    public function handle($request, Closure $next)
+    {
+        /**
+         * This needs to be set for the /setup process, since the tables might not exist yet
+         */
+        $total_assets = 0;
+        $total_due_for_checkin = 0;
+        $total_overdue_for_checkin = 0;
+        $total_due_for_audit = 0;
+        $total_overdue_for_audit = 0;
+
+        try {
+            $settings = Setting::getSettings();
+            view()->share('settings', $settings);
+        } catch (\Exception $e) {
+            Log::debug($e);
+        }
+
+        try {
+            $total_assets = Asset::AssetsForShow()->count();
+            view()->share('total_assets', $total_assets);
+        } catch (\Exception $e) {
+            Log::debug($e);
+        }
+
+        try {
+            $total_rtd_sidebar = Asset::RTD()->count();
+            view()->share('total_rtd_sidebar', $total_rtd_sidebar);
+        } catch (\Exception $e) {
+            Log::debug($e);
+        }
+
+        try {
+            $total_deployed_sidebar = Asset::Deployed()->count();
+            view()->share('total_deployed_sidebar', $total_deployed_sidebar);
+        } catch (\Exception $e) {
+            Log::debug($e);
+        }
+
+        try {
+            $total_archived_sidebar = Asset::Archived()->count();
+            view()->share('total_archived_sidebar', $total_archived_sidebar);
+        } catch (\Exception $e) {
+            Log::debug($e);
+        }
+
+        try {
+            $total_pending_sidebar = Asset::Pending()->count();
+            view()->share('total_pending_sidebar', $total_pending_sidebar);
+        } catch (\Exception $e) {
+            Log::debug($e);
+        }
+
+        try {
+            $total_undeployable_sidebar = Asset::Undeployable()->count();
+            view()->share('total_undeployable_sidebar', $total_undeployable_sidebar);
+        } catch (\Exception $e) {
+            Log::debug($e);
+        }
+
+        try {
+            $total_byod_sidebar = Asset::where('byod', '=', '1')->count();
+            view()->share('total_byod_sidebar', $total_byod_sidebar);
+        } catch (\Exception $e) {
+            Log::debug($e);
+        }
+
+        try {
+            $total_due_for_audit = Asset::DueForAudit($settings)->count();
+            view()->share('total_due_for_audit', $total_due_for_audit);
+        } catch (\Exception $e) {
+            Log::debug($e);
+        }
+
+        try {
+            $total_overdue_for_audit = Asset::OverdueForAudit()->count();
+            view()->share('total_overdue_for_audit', $total_overdue_for_audit);
+        } catch (\Exception $e) {
+            Log::debug($e);
+        }
+
+        try {
+            $total_due_for_checkin = Asset::DueForCheckin($settings)->count();
+            view()->share('total_due_for_checkin', $total_due_for_checkin);
+        } catch (\Exception $e) {
+            Log::debug($e);
+        }
+
+        try {
+            $total_overdue_for_checkin = Asset::OverdueForCheckin()->count();
+            view()->share('total_overdue_for_checkin', $total_overdue_for_checkin);
+        } catch (\Exception $e) {
+            Log::debug($e);
+        }
+
+        view()->share('total_due_and_overdue_for_checkin', ($total_due_for_checkin + $total_overdue_for_checkin));
+        view()->share('total_due_and_overdue_for_audit', ($total_due_for_audit + $total_overdue_for_audit));
+
+        return $next($request);
+    }
+}

app/Http/Middleware/SetPaginationDefaults.php

@@ -1,34 +0,0 @@
-<?php
-
-namespace App\Http\Middleware;
-
-use Closure;
-use Illuminate\Http\Request;
-
-class SetPaginationDefaults
-{
-    public function handle(Request $request, Closure $next)
-    {
-        $limit = config('app.max_results');
-        $intLimit = intval($request->input('limit'));
-
-        if (abs($intLimit) > 0 && $intLimit <= config('app.max_results')) {
-            $limit = abs($intLimit);
-        }
-
-        app()->instance('api_limit_value', $limit);
-
-        if ($request->filled('page') && ! $request->filled('offset')) {
-            $page = max(1, intval($request->input('page')));
-            $offset = ($page - 1) * $limit;
-        } else {
-            $offset = intval($request->input('offset'));
-            $page = $limit > 0 ? (int) floor($offset / $limit) + 1 : 1;
-        }
-
-        app()->instance('api_offset_value', $offset);
-        app()->instance('api_current_page', $page);
-
-        return $next($request);
-    }
-}

app/Http/Requests/FilterRequest.php

@@ -2,20 +2,9 @@
 
 namespace App\Http\Requests;
 
-use App\Models\Accessory;
-use App\Models\Asset;
-use App\Models\AssetModel;
-use App\Models\Component;
-use App\Models\Consumable;
-use App\Models\License;
-use App\Models\LicenseSeat;
-use App\Models\Location;
-use App\Models\Maintenance;
-use App\Models\User;
 use App\Rules\ValidJson;
 use Illuminate\Contracts\Validation\ValidationRule;
 use Illuminate\Foundation\Http\FormRequest;
-use Illuminate\Validation\Rule;
 
 class FilterRequest extends FormRequest
 {
@@ -34,37 +23,8 @@ class FilterRequest extends FormRequest
      */
     public function rules(): array
     {
-        $allowedTypes = [
-            'accessory',
-            Accessory::class,
-            'asset',
-            Asset::class,
-            'assetmodel',
-            'assetModel',
-            'AssetModel',
-            AssetModel::class,
-            'component',
-            Component::class,
-            'consumable',
-            Consumable::class,
-            'license',
-            License::class,
-            'licenseseat',
-            'licenseSeat',
-            'LicenseSeat',
-            LicenseSeat::class,
-            'location',
-            Location::class,
-            'maintenance',
-            Maintenance::class,
-            'user',
-            User::class,
-        ];
-
         return [
             'filter' => ['nullable', new ValidJson],
-            'item_type' => ['nullable', Rule::in($allowedTypes)],
-            'target_type' => ['nullable', Rule::in($allowedTypes)],
         ];
     }
 }

app/Http/Requests/ItemImportRequest.php

@@ -41,7 +41,7 @@ class ItemImportRequest extends FormRequest
         $classString = "App\\Importer\\{$class}Importer";
         $importer = new $classString($filename);
         $import->field_map = request('column-mappings');
-        $import->created_by = $import->created_by ?? auth()->id();
+        $import->created_by = auth()->id();
         $import->save();
         $fieldMappings = [];
 
@@ -51,7 +51,7 @@ class ItemImportRequest extends FormRequest
 
                 if (is_null($fieldValue)) {
                     $errorMessage = trans('validation.import_field_empty', ['fieldname' => $field]);
-                    $this->errorCallback($import, $field, [$field => [$errorMessage]]);
+                    $this->errorCallback($import, $field, [$field => $errorMessage]);
 
                     return $this->errors;
                 }

app/Http/Requests/SaveUserRequest.php

@@ -34,8 +34,6 @@ class SaveUserRequest extends FormRequest
             'department_id' => 'nullable|integer|exists:departments,id',
             'manager_id' => 'nullable|integer|exists:users,id',
             'company_id' => ['nullable', 'integer', 'exists:companies,id'],
-            'company_ids' => 'nullable|array',
-            'company_ids.*' => 'integer|exists:companies,id',
         ];
 
         switch ($this->method()) {
@@ -54,13 +52,13 @@ class SaveUserRequest extends FormRequest
                 $rules['first_name'] = 'required|string|min:1';
                 $rules['username'] = 'required_unless:ldap_import,1|string|min:1';
                 $rules['password'] = Setting::passwordComplexityRulesSaving('update').'|confirmed';
-                $rules['company_id'] = ['nullable', 'integer', 'exists:companies,id', new UserCannotSwitchCompaniesIfItemsAssigned];
+                $rules['company_id'] = [new UserCannotSwitchCompaniesIfItemsAssigned];
                 break;
 
                 // Save only what's passed
             case 'PATCH':
                 $rules['password'] = Setting::passwordComplexityRulesSaving('update');
-                $rules['company_id'] = ['nullable', 'integer', 'exists:companies,id', new UserCannotSwitchCompaniesIfItemsAssigned];
+                $rules['company_id'] = [new UserCannotSwitchCompaniesIfItemsAssigned];
                 break;
 
             default:

app/Http/Transformers/AccessoriesTransformer.php

@@ -26,7 +26,6 @@ class AccessoriesTransformer
             'id' => $accessory->id,
             'name' => e($accessory->name),
             'image' => ($accessory->image) ? Storage::disk('public')->url('accessories/'.e($accessory->image)) : null,
-            'qr_code_url' => route('qr_code/common', ['object_type' => 'accessories', 'id' => $accessory->id]),
             'company' => ($accessory->company) ? [
                 'id' => $accessory->company->id,
                 'name' => e($accessory->company->name),

app/Http/Transformers/ActionlogsTransformer.php

@@ -116,10 +116,10 @@ class ActionlogsTransformer
                                         $clean_meta[$fieldname]['old'] = '************';
                                         $clean_meta[$fieldname]['new'] = '************';
 
-                                        // Display the changes if the user has permission to view encrypted custom fields
-                                        if (Gate::allows('assets.view.encrypted_custom_fields')) {
-                                            $clean_meta[$fieldname]['old'] = ($enc_old) ? e(unserialize($enc_old, ['allowed_classes' => false])) : '';
-                                            $clean_meta[$fieldname]['new'] = ($enc_new) ? e(unserialize($enc_new, ['allowed_classes' => false])) : '';
+                                        // Display the changes if the user is an admin or superadmin
+                                        if (Gate::allows('admin')) {
+                                            $clean_meta[$fieldname]['old'] = ($enc_old) ? unserialize($enc_old, ['allowed_classes' => false]) : '';
+                                            $clean_meta[$fieldname]['new'] = ($enc_new) ? unserialize($enc_new, ['allowed_classes' => false]) : '';
                                         }
 
                                     }
@@ -293,28 +293,6 @@ class ActionlogsTransformer
             $clean_meta[trans('general.company')] = $clean_meta['company_id'];
             unset($clean_meta['company_id']);
         }
-
-        if (array_key_exists('companies', $clean_meta)) {
-            // clean_field() JSON-encodes array values into a string (e.g. "[14,15]").
-            // Decode them back to integer arrays before resolving names.
-            // Use withoutGlobalScopes so FMCS does not hide companies from the log viewer.
-            $resolveCompanyNames = function ($rawValue): string {
-                $ids = json_decode($rawValue, true);
-                if (empty($ids) || ! is_array($ids)) {
-                    return trans('general.unassigned');
-                }
-
-                return collect($ids)
-                    ->map(fn ($id) => Company::withoutGlobalScopes()->withTrashed()->find($id))
-                    ->map(fn ($c) => $c ? e($c->name) : trans('general.deleted'))
-                    ->join(', ');
-            };
-
-            $clean_meta['companies']['old'] = $resolveCompanyNames($clean_meta['companies']['old']);
-            $clean_meta['companies']['new'] = $resolveCompanyNames($clean_meta['companies']['new']);
-            $clean_meta[trans('general.companies')] = $clean_meta['companies'];
-            unset($clean_meta['companies']);
-        }
         if (array_key_exists('supplier_id', $clean_meta)) {
 
             $oldSupplier = $supplier->find($clean_meta['supplier_id']['old']);

app/Http/Transformers/AssetModelsTransformer.php

@@ -48,7 +48,6 @@ class AssetModelsTransformer
                 'tag_color' => ($assetmodel->manufacturer->tag_color) ? e($assetmodel->manufacturer->tag_color) : null,
             ] : null,
             'image' => ($assetmodel->image != '') ? Storage::disk('public')->url('models/'.e($assetmodel->image)) : null,
-            'qr_code_url' => route('qr_code/common', ['object_type' => 'models', 'id' => $assetmodel->id]),
             'model_number' => ($assetmodel->model_number ? e($assetmodel->model_number) : null),
             'min_amt' => ($assetmodel->min_amt) ? (int) $assetmodel->min_amt : null,
 

app/Http/Transformers/AssetsTransformer.php

@@ -98,7 +98,6 @@ class AssetsTransformer
                 'tag_color' => ($asset->defaultLoc->tag_color) ? e($asset->defaultLoc->tag_color) : null,
             ] : null,
             'image' => ($asset->getImageUrl()) ? $asset->getImageUrl() : null,
-            'qr_code_url' => route('qr_code/common', ['object_type' => 'hardware', 'id' => $asset->id]),
             'qr' => ($setting->qr_code == '1') ? Storage::disk('public')->url('barcodes/qr-'.str_slug($asset->asset_tag).'-'.str_slug($asset->id).'.png') : null,
             'alt_barcode' => ($setting->alt_barcode_enabled == '1') ? Storage::disk('public')->url('barcodes/'.str_slug($setting->alt_barcode).'-'.str_slug($asset->asset_tag).'.png') : null,
             'assigned_to' => $this->transformAssignedTo($asset),
@@ -145,7 +144,7 @@ class AssetsTransformer
 
                     $fields_array[$field->name] = [
                         'field' => e($field->db_column),
-                        'value' => ($field->element == 'markdown-textarea' && Gate::allows('assets.view.encrypted_custom_fields')) ? Helper::renderMarkdown($value) : e($value),
+                        'value' => e($value),
                         'field_format' => $field->format,
                         'element' => $field->element,
                     ];
@@ -159,7 +158,7 @@ class AssetsTransformer
 
                     $fields_array[$field->name] = [
                         'field' => e($field->db_column),
-                        'value' => ($field->element == 'markdown-textarea') ? Helper::renderMarkdown($value) : e($value),
+                        'value' => e($value),
                         'field_format' => $field->format,
                         'element' => $field->element,
                     ];
@@ -275,7 +274,7 @@ class AssetsTransformer
                         $value = Helper::getFormattedDateObject($value, 'date', false);
                     }
 
-                    $fields_array[$field->db_column] = ($field->element == 'markdown-textarea') ? Helper::renderMarkdown($value) : e($value);
+                    $fields_array[$field->db_column] = e($value);
                 }
 
                 $array['custom_fields'] = $fields_array;
@@ -389,9 +388,6 @@ class AssetsTransformer
         $permissions_array['available_actions'] = [
             'checkout' => false,
             'checkin' => Gate::allows('checkin', License::class),
-            'bulk_selectable' => [
-                'checkin' => Gate::allows('checkin', License::class),
-            ],
         ];
 
         $array += $permissions_array;

app/Http/Transformers/CategoriesTransformer.php

@@ -75,9 +75,6 @@ class CategoriesTransformer
             $permissions_array['available_actions'] = [
                 'update' => Gate::allows('update', Category::class),
                 'delete' => $category->isDeletable(),
-                'bulk_selectable' => [
-                    'delete' => $category->isDeletable(),
-                ],
             ];
 
             $array += $permissions_array;

app/Http/Transformers/CompaniesTransformer.php

@@ -30,7 +30,6 @@ class CompaniesTransformer
                 'fax' => ($company->fax != '') ? e($company->fax) : null,
                 'email' => ($company->email != '') ? e($company->email) : null,
                 'image' => ($company->image) ? Storage::disk('public')->url('companies/'.e($company->image)) : null,
-                'qr_code_url' => route('qr_code/common', ['object_type' => 'companies', 'id' => $company->id]),
                 'assets_count' => (int) $company->assets_count,
                 'licenses_count' => (int) $company->licenses_count,
                 'accessories_count' => (int) $company->accessories_count,

app/Http/Transformers/ComponentsTransformer.php

@@ -26,7 +26,6 @@ class ComponentsTransformer
             'id' => (int) $component->id,
             'name' => e($component->name),
             'image' => ($component->image) ? Storage::disk('public')->url('components/'.e($component->image)) : null,
-            'qr_code_url' => route('qr_code/common', ['object_type' => 'components', 'id' => $component->id]),
             'serial' => ($component->serial) ? e($component->serial) : null,
             'location' => ($component->location) ? [
                 'id' => (int) $component->location->id,

app/Http/Transformers/ConsumablesTransformer.php

@@ -25,7 +25,6 @@ class ConsumablesTransformer
             'id' => (int) $consumable->id,
             'name' => e($consumable->name),
             'image' => ($consumable->getImageUrl()) ? ($consumable->getImageUrl()) : null,
-            'qr_code_url' => route('qr_code/common', ['object_type' => 'consumables', 'id' => $consumable->id]),
             'category' => ($consumable->category) ? [
                 'id' => $consumable->category->id,
                 'name' => e($consumable->category->name),

app/Http/Transformers/LicenseSeatsTransformer.php

@@ -38,11 +38,13 @@ class LicenseSeatsTransformer
                             'tag_color' => $seat->user->department->tag_color ? e($seat->user->department->tag_color) : null,
 
                         ] : null,
-                'companies' => $seat->user->companies->map(fn ($c) => [
-                    'id' => (int) $c->id,
-                    'name' => e($c->name),
-                    'tag_color' => $c->tag_color ? e($c->tag_color) : null,
-                ])->values(),
+                'company' => ($seat->user->company) ?
+                    [
+                        'id' => (int) $seat->user->company->id,
+                        'name' => e($seat->user->company->name),
+                        'tag_color' => $seat->user->company->tag_color ? e($seat->user->company->tag_color) : null,
+
+                    ] : null,
                 'created_at' => Helper::getFormattedDateObject($seat->created_at, 'datetime'),
             ] : null,
             'assigned_asset' => ($seat->asset) ? [
@@ -68,9 +70,6 @@ class LicenseSeatsTransformer
             'clone' => Gate::allows('create', License::class),
             'update' => Gate::allows('update', License::class),
             'delete' => Gate::allows('delete', License::class),
-            'bulk_selectable' => [
-                'checkin' => Gate::allows('checkin', License::class) && ($seat->assigned_to || $seat->asset_id),
-            ],
         ];
 
         $array += $permissions_array;

app/Http/Transformers/LicensesTransformer.php

@@ -24,7 +24,6 @@ class LicensesTransformer
         $array = [
             'id' => (int) $license->id,
             'name' => e($license->name),
-            'qr_code_url' => route('qr_code/common', ['object_type' => 'licenses', 'id' => $license->id]),
             'company' => ($license->company) ? ['id' => (int) $license->company->id, 'name' => e($license->company->name)] : null,
             'manufacturer' => ($license->manufacturer) ? [
                 'id' => (int) $license->manufacturer->id,
@@ -67,6 +66,7 @@ class LicensesTransformer
             'created_at' => Helper::getFormattedDateObject($license->created_at, 'datetime'),
             'updated_at' => Helper::getFormattedDateObject($license->updated_at, 'datetime'),
             'deleted_at' => Helper::getFormattedDateObject($license->deleted_at, 'datetime'),
+            'user_can_checkout' => (bool) ($license->free_seats_count > 0),
             'disabled' => $license->isInactive(),
         ];
 
@@ -76,7 +76,6 @@ class LicensesTransformer
             'clone' => Gate::allows('create', License::class),
             'update' => Gate::allows('update', License::class),
             'delete' => $license->isDeletable(),
-            'user_can_checkout' => (bool) (($license->free_seats_count - License::unReassignableCount($license)) > 0),
             'bulk_selectable' => [
                 'delete' => $license->isDeletable(),
             ],

app/Http/Transformers/LocationsTransformer.php

@@ -39,7 +39,6 @@ class LocationsTransformer
                 'id' => (int) $location->id,
                 'name' => e($location->name),
                 'image' => ($location->image) ? Storage::disk('public')->url('locations/'.e($location->image)) : null,
-                'qr_code_url' => route('qr_code/common', ['object_type' => 'locations', 'id' => $location->id]),
                 'address' => ($location->address) ? e($location->address) : null,
                 'address2' => ($location->address2) ? e($location->address2) : null,
                 'city' => ($location->city) ? e($location->city) : null,

app/Http/Transformers/MaintenanceTypesTransformer.php

@@ -1,37 +0,0 @@
-<?php
-
-namespace App\Http\Transformers;
-
-use App\Helpers\Helper;
-use App\Models\MaintenanceType;
-use Illuminate\Database\Eloquent\Collection;
-use Illuminate\Support\Facades\Gate;
-
-class MaintenanceTypesTransformer
-{
-    public function transformMaintenanceTypes(Collection $types, int $total): array
-    {
-        $array = [];
-        foreach ($types as $type) {
-            $array[] = self::transformMaintenanceType($type);
-        }
-
-        return (new DatatablesTransformer)->transformDatatables($array, $total);
-    }
-
-    public function transformMaintenanceType(MaintenanceType $type): array
-    {
-        return [
-            'id' => (int) $type->id,
-            'name' => e($type->name),
-            'created_at' => Helper::getFormattedDateObject($type->created_at, 'datetime'),
-            'updated_at' => Helper::getFormattedDateObject($type->updated_at, 'datetime'),
-            'deleted_at' => Helper::getFormattedDateObject($type->deleted_at, 'datetime'),
-            'available_actions' => [
-                'update' => Gate::allows('update', $type),
-                'delete' => $type->isDeletable(),
-                'restore' => Gate::allows('delete', $type),
-            ],
-        ];
-    }
-}

app/Http/Transformers/MaintenancesTransformer.php

@@ -82,22 +82,6 @@ class MaintenancesTransformer
                 'id' => (int) $assetmaintenance->adminuser->id,
                 'name' => e($assetmaintenance->adminuser->display_name),
             ] : null,
-            'maintenance_type' => $assetmaintenance->maintenanceType
-                ? e($assetmaintenance->maintenanceType->name)
-                : null,
-            'responsible_party' => ($assetmaintenance->responsibleParty) ? [
-                'id' => (int) $assetmaintenance->responsibleParty->id,
-                'name' => e($assetmaintenance->responsibleParty->display_name),
-            ] : null,
-            'checked_out_to_at_creation' => $assetmaintenance->checked_out_to_id ? [
-                'id' => (int) $assetmaintenance->checked_out_to_id,
-                'type' => $assetmaintenance->checked_out_to_type,
-            ] : null,
-            'completed_at' => Helper::getFormattedDateObject($assetmaintenance->completed_at, 'datetime'),
-            'completed_by' => ($assetmaintenance->completedByUser) ? [
-                'id' => (int) $assetmaintenance->completedByUser->id,
-                'name' => e($assetmaintenance->completedByUser->display_name),
-            ] : null,
             'created_at' => Helper::getFormattedDateObject($assetmaintenance->created_at, 'datetime'),
             'updated_at' => Helper::getFormattedDateObject($assetmaintenance->updated_at, 'datetime'),
             'is_warranty' => (bool) $assetmaintenance->is_warranty,
@@ -107,7 +91,6 @@ class MaintenancesTransformer
         $permissions_array['available_actions'] = [
             'update' => (Gate::allows('update', Asset::class) && ((($assetmaintenance->asset) && $assetmaintenance->asset->deleted_at == ''))) ? true : false,
             'delete' => Gate::allows('delete', Asset::class),
-            'complete' => Gate::allows('update', Asset::class) && ! $assetmaintenance->completed_at,
         ];
 
         $array += $permissions_array;
@@ -145,23 +128,10 @@ class MaintenancesTransformer
             'supplier' => ($assetmaintenance->supplier) ? e($assetmaintenance->supplier?->name) : null,
             'url' => ($assetmaintenance->url) ? e($assetmaintenance->url) : null,
             'cost' => Helper::formatCurrencyOutput($assetmaintenance->cost),
-            'maintenance_type' => $assetmaintenance->maintenanceType
-                ? e($assetmaintenance->maintenanceType->name)
-                : null,
             'asset_maintenance_type' => e($assetmaintenance->asset_maintenance_type),
             'start_date' => Helper::getFormattedDateObject($assetmaintenance->start_date, 'date'),
             'asset_maintenance_time' => $assetmaintenance->asset_maintenance_time,
             'completion_date' => Helper::getFormattedDateObject($assetmaintenance->completion_date, 'date'),
-            'responsible_party' => ($assetmaintenance->responsibleParty) ? [
-                'id' => (int) $assetmaintenance->responsibleParty->id,
-                'name' => e($assetmaintenance->responsibleParty->display_name),
-            ] : null,
-            'checked_out_to_at_creation' => ($assetmaintenance->checkedOutTo) ? e($assetmaintenance->checkedOutTo->display_name) : null,
-            'completed_at' => Helper::getFormattedDateObject($assetmaintenance->completed_at, 'datetime'),
-            'completed_by' => ($assetmaintenance->completedByUser) ? [
-                'id' => (int) $assetmaintenance->completedByUser->id,
-                'name' => e($assetmaintenance->completedByUser->display_name),
-            ] : null,
             'created_by' => ($assetmaintenance->adminuser) ? e($assetmaintenance->adminuser->display_name) : null,
             'created_at' => Helper::getFormattedDateObject($assetmaintenance->created_at, 'datetime'),
             'updated_at' => Helper::getFormattedDateObject($assetmaintenance->updated_at, 'datetime'),

app/Http/Transformers/ManufacturersTransformer.php

@@ -52,9 +52,6 @@ class ManufacturersTransformer
                 'update' => (($manufacturer->deleted_at == '') && (Gate::allows('update', Manufacturer::class))),
                 'restore' => (($manufacturer->deleted_at != '') && (Gate::allows('create', Manufacturer::class))),
                 'delete' => $manufacturer->isDeletable(),
-                'bulk_selectable' => [
-                    'delete' => $manufacturer->isDeletable(),
-                ],
             ];
 
             $array += $permissions_array;

app/Http/Transformers/SuppliersTransformer.php

@@ -57,9 +57,6 @@ class SuppliersTransformer
             $permissions_array['available_actions'] = [
                 'update' => Gate::allows('update', Supplier::class),
                 'delete' => (Gate::allows('delete', Supplier::class) && ($supplier->isDeletable())),
-                'bulk_selectable' => [
-                    'delete' => (Gate::allows('delete', Supplier::class) && ($supplier->isDeletable())),
-                ],
             ];
 
             $array += $permissions_array;

app/Http/Transformers/UsersTransformer.php

@@ -21,6 +21,7 @@ class UsersTransformer
 
     public function transformUser(User $user)
     {
+
         $role = null;
         if ($user->isSuperUser()) {
             $role = 'superadmin';
@@ -30,7 +31,6 @@ class UsersTransformer
         $array = [
             'id' => (int) $user->id,
             'avatar' => e($user->present()->gravatar) ?? null,
-            'qr_code_url' => route('qr_code/common', ['object_type' => 'users', 'id' => $user->id]),
             'name' => e($user->getFullNameAttribute()) ?? null,
             'first_name' => e($user->first_name) ?? null,
             'last_name' => e($user->last_name) ?? null,
@@ -82,17 +82,11 @@ class UsersTransformer
             'consumables_count' => (int) $user->consumables_count,
             'manages_users_count' => (int) $user->manages_users_count,
             'manages_locations_count' => (int) $user->manages_locations_count,
-            // Legacy field — kept for backward API compatibility; use `companies` for multi-company support.
-            'company' => $user->companies->isNotEmpty() ? [
-                'id' => (int) $user->companies->first()->id,
-                'name' => e($user->companies->first()->name),
-                'tag_color' => ($user->companies->first()->tag_color) ? e($user->companies->first()->tag_color) : null,
+            'company' => ($user->company) ? [
+                'id' => (int) $user->company->id,
+                'name' => e($user->company->name),
+                'tag_color' => ($user->company->tag_color) ? e($user->company->tag_color) : null,
             ] : null,
-            'companies' => $user->companies->map(fn ($c) => [
-                'id' => (int) $c->id,
-                'name' => e($c->name),
-                'tag_color' => $c->tag_color ? e($c->tag_color) : null,
-            ])->values(),
             'created_by' => ($user->createdBy) ? [
                 'id' => (int) $user->createdBy->id,
                 'name' => e($user->createdBy->display_name),
@@ -150,11 +144,6 @@ class UsersTransformer
             'last_name' => e($user->last_name),
             'username' => e($user->username),
             'display_name' => e($user->display_name),
-            'companies' => $user->companies->map(fn ($c) => [
-                'id' => (int) $c->id,
-                'name' => e($c->name),
-                'tag_color' => $c->tag_color ? e($c->tag_color) : null,
-            ])->values(),
             'created_by' => $user->adminuser ? [
                 'id' => (int) $user->adminuser->id,
                 'name' => e($user->adminuser->present()->fullName),

app/Importer/AssetImporter.php

@@ -111,7 +111,7 @@ class AssetImporter extends ItemImporter
         }
 
         $this->item['notes'] = trim($this->findCsvMatch($row, 'asset_notes'));
-        $this->item['image'] = basename(trim($this->findCsvMatch($row, 'image')));
+        $this->item['image'] = trim($this->findCsvMatch($row, 'image'));
         $this->item['requestable'] = trim(($this->fetchHumanBoolean($this->findCsvMatch($row, 'requestable'))) == 1) ? '1' : 0;
         $asset->requestable = $this->item['requestable'];
         $this->item['warranty_months'] = intval(trim($this->findCsvMatch($row, 'warranty_months')));
@@ -214,25 +214,16 @@ class AssetImporter extends ItemImporter
             // -- the class that needs to use it (command importer or GUI importer inside the project).
             if (isset($target) && ($target !== false)) {
                 $asset = $asset->fresh();
+                $targetType = get_class($target);
+                $alreadyCheckedOutToTarget = ($asset->assigned_to == $target->id) && ($asset->assigned_type === $targetType);
 
-                if (! $asset->canCheckoutTo($target)) {
-                    $this->log(trans('general.error_checkout_company_mismatch', [
-                        'item' => trans('general.asset').' "'.$asset->display_name.'"',
-                        'item_company' => $asset->company?->name ?? trans('general.unassigned'),
-                        'target' => ($target->name ?? $target->username ?? $target->id),
-                    ]));
-                } else {
-                    $targetType = get_class($target);
-                    $alreadyCheckedOutToTarget = ($asset->assigned_to == $target->id) && ($asset->assigned_type === $targetType);
-
-                    // Skip duplicate checkout noise when update mode keeps the same assignment target.
-                    if (! $alreadyCheckedOutToTarget) {
-                        if (! is_null($asset->assigned_to)) {
-                            event(new CheckoutableCheckedIn($asset, $asset->assigned, auth()->user(), 'Checkin from CSV Importer', $checkin_date));
-                        }
-
-                        $asset->checkOut($target, $this->created_by, $checkout_date, null, 'Checkout from CSV Importer', $asset->name);
+                // Skip duplicate checkout noise when update mode keeps the same assignment target.
+                if (! $alreadyCheckedOutToTarget) {
+                    if (! is_null($asset->assigned_to)) {
+                        event(new CheckoutableCheckedIn($asset, $asset->assigned, auth()->user(), 'Checkin from CSV Importer', $checkin_date));
                     }
+
+                    $asset->checkOut($target, $this->created_by, $checkout_date, null, 'Checkout from CSV Importer', $asset->name);
                 }
             }
 

app/Importer/ComponentImporter.php

@@ -59,21 +59,13 @@ class ComponentImporter extends ItemImporter
 
             // If we have an asset tag, checkout to that asset.
             if (isset($this->item['asset_tag']) && ($asset = Asset::where('asset_tag', $this->item['asset_tag'])->first())) {
-                if (! $component->canCheckoutTo($asset)) {
-                    $this->log(trans('general.error_checkout_company_mismatch', [
-                        'item' => trans('general.component').' "'.$component->name.'"',
-                        'item_company' => $component->company?->name ?? trans('general.unassigned'),
-                        'target' => trans('general.asset').' "'.$asset->display_name.'"',
-                    ]));
-                } else {
-                    $component->assets()->attach($component->id, [
-                        'component_id' => $component->id,
-                        'created_by' => auth()->id(),
-                        'created_at' => date('Y-m-d H:i:s'),
-                        'assigned_qty' => 1, // Only assign the first one to the asset
-                        'asset_id' => $asset->id,
-                    ]);
-                }
+                $component->assets()->attach($component->id, [
+                    'component_id' => $component->id,
+                    'created_by' => auth()->id(),
+                    'created_at' => date('Y-m-d H:i:s'),
+                    'assigned_qty' => 1, // Only assign the first one to the asset
+                    'asset_id' => $asset->id,
+                ]);
             }
 
             return;

app/Importer/ItemImporter.php

@@ -82,7 +82,6 @@ class ItemImporter extends Importer
         $this->item['qty'] = $this->findCsvMatch($row, 'quantity');
         $this->item['requestable'] = $this->findCsvMatch($row, 'requestable');
         $this->item['created_by'] = auth()->id();
-        $this->item['asset_tag'] = $this->findCsvMatch($row, 'asset_tag');
         $this->item['serial'] = $this->findCsvMatch($row, 'serial');
         $this->item['item_no'] = trim($this->findCsvMatch($row, 'item_no'));
 

app/Importer/LicenseImporter.php

@@ -106,32 +106,16 @@ class LicenseImporter extends ItemImporter
                 }
 
                 if ($checkout_target) {
-                    if (! $license->canCheckoutTo($checkout_target)) {
-                        $this->log(trans('general.error_checkout_company_mismatch', [
-                            'item' => trans('general.license').' "'.$license->name.'"',
-                            'item_company' => $license->company?->name ?? trans('general.unassigned'),
-                            'target' => ($checkout_target->name ?? $checkout_target->username ?? $checkout_target->id),
-                        ]));
-                    } else {
-                        $targetLicense->assigned_to = $checkout_target->id;
-                        $targetLicense->created_by = auth()->id();
-                        if ($asset) {
-                            $targetLicense->asset_id = $asset->id;
-                        }
-                        $targetLicense->save();
-                    }
-                } elseif ($asset) {
-                    if (! $license->canCheckoutTo($asset)) {
-                        $this->log(trans('general.error_checkout_company_mismatch', [
-                            'item' => trans('general.license').' "'.$license->name.'"',
-                            'item_company' => $license->company?->name ?? trans('general.unassigned'),
-                            'target' => trans('general.asset').' "'.$asset->display_name.'"',
-                        ]));
-                    } else {
-                        $targetLicense->created_by = auth()->id();
+                    $targetLicense->assigned_to = $checkout_target->id;
+                    $targetLicense->created_by = auth()->id();
+                    if ($asset) {
                         $targetLicense->asset_id = $asset->id;
-                        $targetLicense->save();
                     }
+                    $targetLicense->save();
+                } elseif ($asset) {
+                    $targetLicense->created_by = auth()->id();
+                    $targetLicense->asset_id = $asset->id;
+                    $targetLicense->save();
                 }
             }
 

app/Importer/UserImporter.php

@@ -3,7 +3,6 @@
 namespace App\Importer;
 
 use App\Models\Asset;
-use App\Models\Company;
 use App\Models\Department;
 use App\Models\Setting;
 use App\Models\User;
@@ -36,31 +35,6 @@ class UserImporter extends ItemImporter
         $this->createUserIfNotExists($row);
     }
 
-    /**
-     * Parse a pipe-separated company column value into an array of company IDs,
-     * creating companies that do not yet exist. Returns an empty array when the
-     * raw value is blank (so callers can treat that as "don't change").
-     *
-     * @param  string  $raw  Raw cell value, e.g. "Acme Corp|Widget Inc"
-     * @return int[]
-     */
-    private function resolveCompanyIds(string $raw): array
-    {
-        if ($raw === '') {
-            return [];
-        }
-
-        $ids = [];
-        foreach (array_filter(array_map('trim', explode('|', $raw))) as $name) {
-            $id = $this->createOrFetchCompany($name);
-            if ($id) {
-                $ids[] = (int) $id;
-            }
-        }
-
-        return Company::getIdsForCurrentUser($ids);
-    }
-
     /**
      * Create a user if a duplicate does not exist.
      *
@@ -75,7 +49,7 @@ class UserImporter extends ItemImporter
         // Pull the records from the CSV to determine their values
         $this->item['id'] = trim($this->findCsvMatch($row, 'id'));
         $this->item['username'] = trim($this->findCsvMatch($row, 'username'));
-        $this->item['display_name'] = trim($this->findCsvMatch($row, 'display_name')) ?: null;
+        $this->item['display_name'] = trim($this->findCsvMatch($row, 'display_name'));
         $this->item['first_name'] = trim($this->findCsvMatch($row, 'first_name'));
         $this->item['last_name'] = trim($this->findCsvMatch($row, 'last_name'));
         $this->item['email'] = trim($this->findCsvMatch($row, 'email'));
@@ -106,13 +80,6 @@ class UserImporter extends ItemImporter
             $this->item['department_id'] = $this->createOrFetchDepartment($user_department);
         }
 
-        // Resolve pipe-separated company names (e.g. "Acme Corp|Widget Inc") into IDs.
-        // company_id is a legacy column — company membership is managed via the pivot.
-        // Unset whatever the parent set so it is not written to the DB.
-        $companyRaw = trim($this->findCsvMatch($row, 'company'));
-        $companyIds = $this->resolveCompanyIds($companyRaw);
-        unset($this->item['company_id']);
-
         if (is_null($this->item['username']) || $this->item['username'] == '') {
             $user_full_name = $this->item['first_name'].' '.$this->item['last_name'];
             $user_formatted_array = User::generateFormattedNameFromFullName($user_full_name, Setting::getSettings()->username_format);
@@ -137,13 +104,11 @@ class UserImporter extends ItemImporter
 
             $this->log('Updating User');
 
-            // CLI imports run unauthenticated and are fully trusted; only restrict web-initiated imports.
-            // Note: unset must target $this->item, not the model — sanitizeItemForUpdating() reads from $this->item.
-            if (Auth::check() && (! Auth::user()->hasAccess('users.edit') || ! Gate::allows('canEditAuthFields', $user))) {
-                unset($this->item['username']);
-                unset($this->item['email']);
-                unset($this->item['password']);
-                unset($this->item['activated']);
+            if (Auth::check() && (! Gate::allows('canEditAuthFields', $user))) {
+                unset($user->username);
+                unset($user->email);
+                unset($user->password);
+                unset($user->activated);
             }
 
             $user->update($this->sanitizeItemForUpdating($user));
@@ -151,11 +116,6 @@ class UserImporter extends ItemImporter
             // Why do we have to do this twice? Update should
             $user->save();
 
-            // Sync company pivot when companies were specified in this row.
-            if (! empty($companyIds)) {
-                $user->companies()->sync($companyIds);
-            }
-
             // Update the location of any assets checked out to this user
             Asset::where('assigned_type', User::class)
                 ->where('assigned_to', $user->id)
@@ -165,17 +125,6 @@ class UserImporter extends ItemImporter
             return;
         }
 
-        // With FMCS enabled, the scoped lookup above only sees users in the current user's companies.
-        // If the username exists in another company it would appear as "not found" and fall through
-        // to create — but usernames are unique system-wide, so we must skip instead.
-        if (Auth::check() && Company::isFullMultipleCompanySupportEnabled()) {
-            if (User::withoutGlobalScopes()->where('username', $this->item['username'])->exists()) {
-                $this->log('Skipping '.$this->item['username'].': username belongs to a user outside your company scope.');
-
-                return;
-            }
-        }
-
         // This needs to be applied after the update logic, otherwise we'll overwrite user passwords
         // Issue #5408
         $this->item['password'] = $this->tempPassword;
@@ -191,13 +140,6 @@ class UserImporter extends ItemImporter
         if ($user->save()) {
             $this->log('User '.$this->item['name'].' was created');
 
-            // Sync all resolved companies to the pivot. For single-company rows the
-            // User::created event already added company_id; sync() here is idempotent
-            // for that case and adds any additional companies for multi-company rows.
-            if (! empty($companyIds)) {
-                $user->companies()->sync($companyIds);
-            }
-
             if (($user->email) && ($user->activated == '1')) {
 
                 if ($this->send_welcome) {

app/Livewire/Importer.php

@@ -720,12 +720,6 @@ class Importer extends Component
         $this->message_type = 'danger';
     }
 
-    public function process(): void
-    {
-        $this->message = trans('general.token_expired');
-        $this->message_type = 'danger';
-    }
-
     public function clearMessage()
     {
         $this->message = null;

app/Livewire/OauthClients.php

@@ -6,7 +6,6 @@ use Illuminate\Support\Facades\DB;
 use Illuminate\Support\Facades\Log;
 use Laravel\Passport\Client;
 use Laravel\Passport\ClientRepository;
-use Laravel\Passport\Token;
 use Livewire\Component;
 
 class OauthClients extends Component
@@ -51,11 +50,11 @@ class OauthClients extends Component
                 ->get();
 
             if ($clients->isNotEmpty()) {
-                $tokenCountsByClientId = Token::query()
+                $tokenCountsByClientId = DB::table('oauth_access_tokens')
                     ->whereIn('client_id', $clients->pluck('id')->all())
-                    ->get(['client_id'])
+                    ->selectRaw('client_id, COUNT(*) as token_count')
                     ->groupBy('client_id')
-                    ->map->count();
+                    ->pluck('token_count', 'client_id');
 
                 $clients->each(function ($client) use ($tokenCountsByClientId): void {
                     $client->setAttribute('associated_token_count', (int) ($tokenCountsByClientId[$client->id] ?? 0));
@@ -65,28 +64,32 @@ class OauthClients extends Component
 
         $authorizedApplications = collect();
         if ($this->showAuthorizedApplications()) {
-            $authorizedApplications = Token::query()
-                ->where('revoked', false)
-                ->with([
-                    'client',
-                    'client.user' => fn ($q) => $q->withTrashed(),
+            $authorizedTokenSummary = DB::table('oauth_access_tokens as tokens')
+                ->where('tokens.revoked', false)
+                ->selectRaw('tokens.client_id')
+                ->selectRaw('MAX(tokens.scopes) as scopes')
+                ->selectRaw('MAX(tokens.created_at) as created_at')
+                ->selectRaw('MAX(tokens.expires_at) as expires_at')
+                ->groupBy('tokens.client_id');
+
+            $authorizedApplications = DB::table('oauth_clients as clients')
+                ->joinSub($authorizedTokenSummary, 'token_summary', function ($join) {
+                    $join->on('clients.id', '=', 'token_summary.client_id');
+                })
+                ->leftJoin('users as creators', 'clients.user_id', '=', 'creators.id')
+                ->select([
+                    'clients.id as client_id',
+                    'clients.name as client_name',
+                    'clients.user_id as client_owner_id',
+                    'creators.display_name as client_owner_display_name',
+                    'creators.username as client_owner_username',
+                    'creators.deleted_at as client_owner_deleted_at',
+                    'token_summary.scopes',
+                    'token_summary.created_at',
+                    'token_summary.expires_at',
                 ])
-                ->orderByDesc('created_at')
-                ->get()
-                ->unique('client_id')
-                ->filter(fn ($token) => $token->client !== null)
-                ->map(fn ($token) => (object) [
-                    'client_id' => $token->client_id,
-                    'client_name' => $token->client->name,
-                    'client_owner_id' => $token->client->user_id,
-                    'client_owner_display_name' => $token->client->user?->display_name,
-                    'client_owner_username' => $token->client->user?->username,
-                    'client_owner_deleted_at' => $token->client->user?->deleted_at,
-                    'scopes' => $token->scopes,
-                    'created_at' => $token->created_at,
-                    'expires_at' => $token->expires_at,
-                ])
-                ->values();
+                ->orderByDesc('token_summary.created_at')
+                ->get();
         }
 
         return view('livewire.oauth-clients', [

app/Mail/BulkDeleteReportMail.php

@@ -1,59 +0,0 @@
-<?php
-
-namespace App\Mail;
-
-use App\Models\User;
-use Carbon\Carbon;
-use Illuminate\Bus\Queueable;
-use Illuminate\Mail\Mailables\Address;
-use Illuminate\Mail\Mailables\Content;
-use Illuminate\Mail\Mailables\Envelope;
-use Illuminate\Queue\SerializesModels;
-
-class BulkDeleteReportMail extends BaseMailable
-{
-    use Queueable, SerializesModels;
-
-    public function __construct(
-        public readonly User $admin,
-        public readonly bool $dryRun,
-        public readonly array $companyNames,
-        public readonly array $selectedTypes,
-        public readonly string $deleteType,
-        public readonly array $reportLines,
-        public readonly Carbon $runAt,
-    ) {}
-
-    public function envelope(): Envelope
-    {
-        $subject = $this->dryRun
-            ? '[Dry Run] Bulk Check-in/Delete Report'
-            : 'Bulk Check-in/Delete Report';
-
-        return new Envelope(
-            from: new Address(config('mail.from.address'), config('mail.from.name')),
-            subject: $subject,
-        );
-    }
-
-    public function content(): Content
-    {
-        return new Content(
-            markdown: 'notifications.markdown.report-bulk-delete',
-            with: [
-                'admin' => $this->admin,
-                'dryRun' => $this->dryRun,
-                'companyNames' => $this->companyNames,
-                'selectedTypes' => $this->selectedTypes,
-                'deleteType' => $this->deleteType,
-                'reportLines' => $this->reportLines,
-                'runAt' => $this->runAt,
-            ],
-        );
-    }
-
-    public function attachments(): array
-    {
-        return [];
-    }
-}

app/Models/Accessory.php

@@ -80,7 +80,7 @@ class Accessory extends SnipeModel
         'name' => 'required|max:255',
         'qty' => 'nullable|integer|min:0',
         'category_id' => 'required|integer|exists:categories,id',
-        'company_id' => 'integer|nullable|exists:companies,id',
+        'company_id' => 'integer|nullable',
         'location_id' => 'exists:locations,id|nullable|fmcs_location',
         'min_amt' => 'integer|min:0|nullable',
         'purchase_cost' => 'numeric|nullable|gte:0|max:99999999999999999.99',

app/Models/AccessoryCheckout.php

@@ -146,8 +146,7 @@ class AccessoryCheckout extends Model
                     $search_str = '%'.$term.'%';
                     $query->where('first_name', 'like', $search_str)
                         ->orWhere('last_name', 'like', $search_str)
-                        ->orWhere('note', 'like', $search_str)
-                        ->orWhereHas('companies', fn ($q) => $q->where('companies.name', 'like', $search_str));
+                        ->orWhere('note', 'like', $search_str);
                 }
             }
         )->select('id');

app/Models/Asset.php

@@ -34,7 +34,7 @@ class Asset extends Depreciable
 {
     protected $presenter = AssetPresenter::class;
 
-    // protected $with = ['model', 'adminuser', 'location', 'company'];
+    protected $with = ['model', 'adminuser', 'location', 'company'];
 
     use CompanyableTrait;
     use HasFactory;
@@ -487,18 +487,16 @@ class Asset extends Depreciable
 
     public function availableForCheckIn()
     {
-        if ($this->assigned_to == '') {
-            return false;
-        }
 
-        // Deleted assets that are still checked out should always allow checkin
-        if ($this->deleted_at != '') {
+        // This asset is currently assigned to anyone and is not deleted...
+        if (($this->assigned_to != '') && ($this->status) && ($this->status->archived == '0')
+            && ($this->status->deployable == '1')
+        ) {
             return true;
+
         }
 
-        return $this->status
-            && ($this->status->archived == '0')
-            && ($this->status->deployable == '1');
+        return false;
     }
 
     /**
@@ -1482,10 +1480,13 @@ class Asset extends Depreciable
      */
     public function scopePending($query)
     {
-        // Pluck IDs then whereIn — do NOT replace with whereHas. whereHas generates a correlated EXISTS per row and causes severe slowdowns in withCount contexts.
-        $ids = Statuslabel::where('deployable', 0)->where('pending', 1)->where('archived', 0)->whereNull('deleted_at')->pluck('id');
-
-        return $query->whereIn('assets.status_id', $ids->isEmpty() ? [0] : $ids);
+        return $query->whereHas(
+            'status', function ($query) {
+                $query->where('deployable', '=', 0)
+                    ->where('pending', '=', 1)
+                    ->where('archived', '=', 0);
+            }
+        );
     }
 
     /**
@@ -1535,11 +1536,14 @@ class Asset extends Depreciable
      */
     public function scopeRTD($query)
     {
-        // Pluck IDs then whereIn — do NOT replace with whereHas. whereHas generates a correlated EXISTS per row and causes severe slowdowns in withCount contexts.
-        $ids = Statuslabel::where('deployable', 1)->where('pending', 0)->where('archived', 0)->whereNull('deleted_at')->pluck('id');
-
         return $query->whereNull('assets.assigned_to')
-            ->whereIn('assets.status_id', $ids->isEmpty() ? [0] : $ids);
+            ->whereHas(
+                'status', function ($query) {
+                    $query->where('deployable', '=', 1)
+                        ->where('pending', '=', 0)
+                        ->where('archived', '=', 0);
+                }
+            );
     }
 
     /**
@@ -1550,10 +1554,13 @@ class Asset extends Depreciable
      */
     public function scopeUndeployable($query)
     {
-        // Pluck IDs then whereIn — do NOT replace with whereHas. whereHas generates a correlated EXISTS per row and causes severe slowdowns in withCount contexts.
-        $ids = Statuslabel::where('deployable', 0)->where('pending', 0)->where('archived', 0)->whereNull('deleted_at')->pluck('id');
-
-        return $query->whereIn('assets.status_id', $ids->isEmpty() ? [0] : $ids);
+        return $query->whereHas(
+            'status', function ($query) {
+                $query->where('deployable', '=', 0)
+                    ->where('pending', '=', 0)
+                    ->where('archived', '=', 0);
+            }
+        );
     }
 
     /**
@@ -1564,10 +1571,11 @@ class Asset extends Depreciable
      */
     public function scopeNotArchived($query)
     {
-        // Pluck IDs then whereIn — do NOT replace with whereHas. whereHas generates a correlated EXISTS per row and causes severe slowdowns in withCount contexts.
-        $ids = Statuslabel::where('archived', 0)->whereNull('deleted_at')->pluck('id');
-
-        return $query->whereIn('assets.status_id', $ids->isEmpty() ? [0] : $ids);
+        return $query->whereHas(
+            'status', function ($query) {
+                $query->where('archived', '=', 0);
+            }
+        );
     }
 
     /**
@@ -1730,16 +1738,17 @@ class Asset extends Depreciable
      */
     public function scopeAssetsForShow($query)
     {
-        // Pluck IDs then whereIn — do NOT replace with whereHas. whereHas generates a correlated EXISTS per row and causes severe slowdowns in withCount contexts.
-        if (Setting::getSettings()->show_archived_in_list != 1) {
-            $validStatusIds = Statuslabel::where('archived', 0)
-                ->whereNull('deleted_at')
-                ->pluck('id');
 
-            return $query->whereIn('assets.status_id', $validStatusIds->isEmpty() ? [0] : $validStatusIds);
+        if (Setting::getSettings()->show_archived_in_list != 1) {
+            return $query->whereHas(
+                'status', function ($query) {
+                    $query->where('archived', '=', 0);
+                }
+            );
+        } else {
+            return $query;
         }
 
-        return $query;
     }
 
     /**
@@ -1750,10 +1759,13 @@ class Asset extends Depreciable
      */
     public function scopeArchived($query)
     {
-        // Pluck IDs then whereIn — do NOT replace with whereHas. whereHas generates a correlated EXISTS per row and causes severe slowdowns in withCount contexts.
-        $ids = Statuslabel::where('deployable', 0)->where('pending', 0)->where('archived', 1)->whereNull('deleted_at')->pluck('id');
-
-        return $query->whereIn('assets.status_id', $ids->isEmpty() ? [0] : $ids);
+        return $query->whereHas(
+            'status', function ($query) {
+                $query->where('deployable', '=', 0)
+                    ->where('pending', '=', 0)
+                    ->where('archived', '=', 1);
+            }
+        );
     }
 
     /**

app/Models/Builders/MaintenanceQueryBuilder.php

@@ -1,117 +0,0 @@
-<?php
-
-namespace App\Models\Builders;
-
-use App\Models\Setting;
-use Carbon\Carbon;
-use Illuminate\Database\Eloquent\Builder;
-
-class MaintenanceQueryBuilder extends Builder
-{
-    public function active(): static
-    {
-        return $this->whereNull('maintenances.completed_at');
-    }
-
-    public function completed(): static
-    {
-        return $this->whereNotNull('maintenances.completed_at');
-    }
-
-    public function dueForCompletion(Setting $settings): static
-    {
-        $interval = (int) ($settings->audit_warning_days ?? 0);
-        $today = Carbon::now();
-
-        return $this->whereNotNull('maintenances.completion_date')
-            ->whereNull('maintenances.completed_at')
-            ->whereBetween('maintenances.completion_date', [
-                $today->format('Y-m-d'),
-                $today->copy()->addDays($interval)->format('Y-m-d'),
-            ]);
-    }
-
-    public function overdueForCompletion(): static
-    {
-        return $this->whereNotNull('maintenances.completion_date')
-            ->whereNull('maintenances.completed_at')
-            ->where('maintenances.completion_date', '<', Carbon::now()->format('Y-m-d'));
-    }
-
-    public function dueOrOverdueForCompletion(Setting $settings): static
-    {
-        return $this->where(fn ($q) => $q->overdueForCompletion())
-            ->orWhere(fn ($q) => $q->dueForCompletion($settings));
-    }
-
-    public function orderBySupplier(string $order): static
-    {
-        return $this->leftJoin('suppliers as suppliers_maintenances', 'maintenances.supplier_id', '=', 'suppliers_maintenances.id')
-            ->orderBy('suppliers_maintenances.name', $order);
-    }
-
-    public function orderByTag(string $order): static
-    {
-        return $this->leftJoin('assets', 'maintenances.asset_id', '=', 'assets.id')
-            ->orderBy('assets.asset_tag', $order);
-    }
-
-    public function orderByAssetName(string $order): static
-    {
-        return $this->leftJoin('assets', 'maintenances.asset_id', '=', 'assets.id')
-            ->orderBy('assets.name', $order);
-    }
-
-    public function orderByAssetSerial(string $order): static
-    {
-        return $this->leftJoin('assets', 'maintenances.asset_id', '=', 'assets.id')
-            ->orderBy('assets.serial', $order);
-    }
-
-    public function orderStatusName(string $order): static
-    {
-        return $this->join('assets as maintained_asset', 'maintenances.asset_id', '=', 'maintained_asset.id')
-            ->leftJoin('status_labels as maintained_asset_status', 'maintained_asset_status.id', '=', 'maintained_asset.status_id')
-            ->orderBy('maintained_asset_status.name', $order);
-    }
-
-    public function orderLocationName(string $order): static
-    {
-        return $this->join('assets as maintained_asset', 'maintenances.asset_id', '=', 'maintained_asset.id')
-            ->leftJoin('locations as maintained_asset_location', 'maintained_asset_location.id', '=', 'maintained_asset.location_id')
-            ->orderBy('maintained_asset_location.name', $order);
-    }
-
-    public function orderByCreatedBy(string $order): static
-    {
-        return $this->leftJoin('users as admin_sort', 'maintenances.created_by', '=', 'admin_sort.id')
-            ->select('maintenances.*')
-            ->orderBy('admin_sort.first_name', $order)
-            ->orderBy('admin_sort.last_name', $order);
-    }
-
-    public function orderByAssetModelName(string $order): static
-    {
-        return $this->join('assets as maintained_asset', 'maintenances.asset_id', '=', 'maintained_asset.id')
-            ->leftJoin('models as maintained_asset_model', 'maintained_asset_model.id', '=', 'maintained_asset.model_id')
-            ->orderBy('maintained_asset_model.name', $order);
-    }
-
-    public function orderByAssetModelNumber(string $order): static
-    {
-        return $this->join('assets as maintained_asset', 'maintenances.asset_id', '=', 'maintained_asset.id')
-            ->leftJoin('models as maintained_asset_model', 'maintained_asset_model.id', '=', 'maintained_asset.model_id')
-            ->orderBy('maintained_asset_model.model_number', $order);
-    }
-
-    public function orderByMaintenanceType(string $order): static
-    {
-        return $this->leftJoin('maintenance_types as maintenance_type_sort', 'maintenances.maintenance_type_id', '=', 'maintenance_type_sort.id')
-            ->orderBy('maintenance_type_sort.name', $order);
-    }
-
-    public function orderByCompletedAt(string $order): static
-    {
-        return $this->orderBy('maintenances.completed_at', $order);
-    }
-}

app/Models/Company.php

@@ -11,11 +11,9 @@ use App\Presenters\Presentable;
 use Illuminate\Database\Eloquent\Factories\HasFactory;
 use Illuminate\Database\Eloquent\SoftDeletes;
 use Illuminate\Support\Facades\Auth;
-use Illuminate\Support\Facades\DB;
 use Illuminate\Support\Facades\Gate;
 use Illuminate\Support\Facades\Log;
 use Illuminate\Support\Facades\Schema;
-use Illuminate\Validation\ValidationException;
 use Watson\Validating\ValidatingTrait;
 
 /**
@@ -96,26 +94,7 @@ final class Company extends SnipeModel
         'notes',
     ];
 
-    /**
-     * Return the current user's company IDs by querying the pivot table directly.
-     *
-     * We deliberately bypass the Eloquent companies() relationship here because
-     * loading that relationship triggers CompanyableScope on the Company model,
-     * which calls this method again — infinite recursion.
-     */
-    private static function getCurrentUserCompanyIds(): array
-    {
-        if (! Auth::hasUser()) {
-            return [];
-        }
-
-        return DB::table('company_user')
-            ->where('user_id', auth()->id())
-            ->pluck('company_id')
-            ->toArray();
-    }
-
-    public static function isFullMultipleCompanySupportEnabled()
+    private static function isFullMultipleCompanySupportEnabled()
     {
         $settings = Setting::getSettings();
 
@@ -156,27 +135,11 @@ final class Company extends SnipeModel
             if ($current_user->isSuperUser()) {
                 return self::getIdFromInput($unescaped_input);
             } else {
-                $userCompanyIds = self::getCurrentUserCompanyIds();
-                $submittedId = (int) self::getIdFromInput($unescaped_input);
-
-                // Company membership is now determined entirely by the pivot (company_user table).
-                // If the submitted value is a company the user actually belongs to, honour it.
-                if ($submittedId && in_array($submittedId, $userCompanyIds)) {
-                    return $submittedId;
+                if ($current_user->company_id != null) {
+                    return $current_user->company_id;
+                } else {
+                    return null;
                 }
-
-                // A user with pivot memberships who submits a company they don't belong to is
-                // attempting cross-tenant assignment — reject outright rather than silently
-                // overriding or storing null.
-                if ($submittedId && ! empty($userCompanyIds)) {
-                    throw ValidationException::withMessages([
-                        'company_id' => [trans('validation.in', ['attribute' => 'company_id'])],
-                    ]);
-                }
-
-                // No company submitted (or user has no pivot memberships) — fall back to the
-                // user's single company if unambiguous, otherwise null.
-                return count($userCompanyIds) === 1 ? $userCompanyIds[0] : null;
             }
         }
     }
@@ -216,65 +179,20 @@ final class Company extends SnipeModel
         }
 
         if (auth()->user()) {
-            if (auth()->user()->isSuperUser()) {
-                return true;
+            // Log::warning('Companyable is '.$companyable);
+            $current_user_company_id = auth()->user()->company_id;
+            $companyable_company_id = $companyable->company_id;
+
+            // Set this to check companyable on company
+            if ($companyable instanceof Company) {
+                $companyable_company_id = $companyable->id;
             }
 
-            $userCompanyIds = self::getCurrentUserCompanyIds();
-
-            // Empty pivot = unrestricted only for true legacy "no-company" users
-            // (those whose scalar company_id is also null). Users who had their
-            // pivot cleared via the API retain their scalar company_id, so they
-            // do NOT qualify for this bypass.
-            if (empty($userCompanyIds) && is_null(auth()->user()->company_id)) {
-                return true;
-            }
-
-            // Users are scoped by pivot membership, not company_id, so check the pivot directly.
-            if ($companyable instanceof User) {
-                $companyableCompanyIds = DB::table('company_user')
-                    ->where('user_id', $companyable->id)
-                    ->pluck('company_id')
-                    ->toArray();
-
-                // A user with no pivot rows is a null-company user; no intersection is possible.
-                if (empty($companyableCompanyIds)) {
-                    return false;
-                }
-
-                return ! empty(array_intersect($userCompanyIds, $companyableCompanyIds));
-            }
-
-            $companyable_company_id = ($companyable instanceof Company)
-                ? $companyable->id
-                : $companyable->company_id;
-
-            return in_array($companyable_company_id, $userCompanyIds);
+            return ($current_user_company_id == null) || ($current_user_company_id == $companyable_company_id) || auth()->user()->isSuperUser();
         }
 
         return false;
-    }
 
-    /**
-     * Filter an array of requested company IDs to only those the current user
-     * belongs to. Superusers may assign any company; non-superusers are limited
-     * to their own pivot memberships when FMCS is enabled.
-     */
-    public static function getIdsForCurrentUser(array $requestedIds): array
-    {
-        if (! self::isFullMultipleCompanySupportEnabled()) {
-            return $requestedIds;
-        }
-
-        $current_user = auth()->user();
-
-        if ($current_user->isSuperUser()) {
-            return $requestedIds;
-        }
-
-        $allowedIds = self::getCurrentUserCompanyIds();
-
-        return array_values(array_intersect($requestedIds, $allowedIds));
     }
 
     public static function isCurrentUserAuthorized()
@@ -284,9 +202,8 @@ final class Company extends SnipeModel
 
     public static function canManageUsersCompanies()
     {
-        return ! self::isFullMultipleCompanySupportEnabled()
-            || auth()->user()->isSuperUser()
-            || ! empty(self::getCurrentUserCompanyIds());
+        return ! self::isFullMultipleCompanySupportEnabled() || auth()->user()->isSuperUser() ||
+                auth()->user()->company_id == null;
     }
 
     /**
@@ -325,7 +242,7 @@ final class Company extends SnipeModel
 
     public function users()
     {
-        return $this->belongsToMany(User::class, 'company_user');
+        return $this->hasMany(User::class, 'company_id');
     }
 
     public function assets()
@@ -387,100 +304,27 @@ final class Company extends SnipeModel
      */
     private static function scopeCompanyablesDirectly($query, $column = 'company_id', $table_name = null)
     {
-        $companyIds = self::getCurrentUserCompanyIds();
+
+        $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;
+        }
 
         // If we are scoping the companies table itself, look for the company.id
         if ($query->getModel()->getTable() == 'companies') {
-            if (empty($companyIds)) {
-                return $query->whereNull('companies.id');
-            }
-
-            return $query->whereIn('companies.id', $companyIds);
-        }
-
-        $floater = Setting::getSettings()->null_company_is_floater;
-
-        // Users are scoped by pivot membership (company_user), not by company_id column,
-        // since a user may belong to multiple companies and company_id alone is insufficient.
-        if ($query->getModel()->getTable() == 'users') {
-            if (empty($companyIds)) {
-                // Floater: actor has no company and is unrestricted — see everyone.
-                if ($floater) {
-                    return $query;
-                }
-
-                // No pivot memberships: mirror old null-company behavior — show only users
-                // who are also not in any company via the pivot.
-                return $query->whereNotIn('users.id', function ($sub) {
-                    $sub->select('user_id')->from('company_user');
-                });
-            }
-
-            // Floater: also include users with no company associations (they float). They all float down here, Georgie.).
-            if ($floater) {
-                return $query->where(function ($q) use ($companyIds) {
-                    $q->whereIn('users.id', function ($sub) use ($companyIds) {
-                        $sub->select('user_id')->from('company_user')->whereIn('company_id', $companyIds);
-                    })->orWhereDoesntHave('companies');
-                });
-            }
-
-            return $query->whereIn('users.id', function ($sub) use ($companyIds) {
-                $sub->select('user_id')->from('company_user')->whereIn('company_id', $companyIds);
-            });
+            return $query->where('companies.id', '=', $company_id);
         }
 
         // If the column exists in the table, use it to scope the query
-        if ($query && $query->getModel() && Schema::hasColumn($query->getModel()->getTable(), $column)) {
+        if ((($query) && ($query->getModel()) && (Schema::hasColumn($query->getModel()->getTable(), $column)))) {
+
+            // Dynamically get the table name if it's not passed in, based on the model we're querying against
             $table = ($table_name) ? $table_name.'.' : $query->getModel()->getTable().'.';
 
-            if (empty($companyIds)) {
-                // Floater: actor has no company and is unrestricted — see everything.
-                if ($floater) {
-                    return $query;
-                }
-
-                return $query->whereNull($table.$column);
-            }
-
-            // action_logs: a NULL company_id means the logged object (AssetModel, Company, etc.)
-            // has no company_id column of its own. Those are global objects, visible to all users,
-            // so their log entries should not be hidden by the company filter.
-            if ($query->getModel()->getTable() === 'action_logs') {
-                return $query->where(function ($q) use ($table, $column, $companyIds) {
-                    $q->whereIn($table.$column, $companyIds)
-                        ->orWhereNull($table.$column);
-                });
-            }
-
-            // Floater: null-company items are visible to users from any company.
-            if ($floater) {
-                return $query->where(function ($q) use ($table, $column, $companyIds) {
-                    $q->whereIn($table.$column, $companyIds)
-                        ->orWhereNull($table.$column);
-                });
-            }
-
-            return $query->whereIn($table.$column, $companyIds);
-        }
-    }
-
-    /**
-     * Scope a users query to those belonging to the given company IDs, respecting floater mode.
-     *
-     * Extracted from controller-level inline logic so the same rule is enforced consistently
-     * everywhere users are filtered by a specific set of company IDs (e.g. select2 dropdowns).
-     */
-    public static function scopeUsersByCompanyIds($query, array $companyIds): mixed
-    {
-        if (Setting::getSettings()->null_company_is_floater) {
-            return $query->where(function ($q) use ($companyIds) {
-                $q->whereHas('companies', fn ($q2) => $q2->whereIn('companies.id', $companyIds))
-                    ->orWhereDoesntHave('companies');
-            });
+            return $query->where($table.$column, '=', $company_id);
         }
 
-        return $query->whereHas('companies', fn ($q) => $q->whereIn('companies.id', $companyIds));
     }
 
     /**

app/Models/Consumable.php

@@ -48,7 +48,7 @@ class Consumable extends SnipeModel
         'name' => 'required|max:255',
         'qty' => 'required|integer|min:0|max:99999',
         'category_id' => 'required|integer',
-        'company_id' => 'integer|nullable|exists:companies,id',
+        'company_id' => 'integer|nullable',
         'location_id' => 'exists:locations,id|nullable|fmcs_location',
         'min_amt' => 'integer|min:0|max:99999|nullable',
         'purchase_cost' => 'numeric|nullable|gte:0|max:99999999999999999.99',

app/Models/CustomField.php

@@ -51,7 +51,7 @@ class CustomField extends Model
      */
     protected $rules = [
         'name' => 'required|unique:custom_fields',
-        'element' => 'required|in:text,listbox,textarea,markdown-textarea,checkbox,radio',
+        'element' => 'required|in:text,listbox,textarea,checkbox,radio',
         'field_encrypted' => 'nullable|boolean',
         'auto_add_to_fieldsets' => 'boolean',
         'show_in_listview' => 'boolean',

app/Models/Labels/FieldOption.php

@@ -3,7 +3,6 @@
 namespace App\Models\Labels;
 
 use App\Models\Asset;
-use App\Models\User;
 
 class FieldOption
 {
@@ -28,19 +27,14 @@ class FieldOption
         // assignedTo directly on the asset is a special case where
         // we want to avoid returning the property directly
         // and instead return the entity's presented name.
-        if (in_array($dataPath[0], ['assignedTo', 'displayName'])) {
-            $assigned = $asset->relationLoaded('assignedTo') ? $asset->assigned : $asset->assignedTo;
+        if ($dataPath[0] === 'assignedTo') {
+            if ($asset->relationLoaded('assignedTo')) {
+                // If the "assignedTo" relationship was eager loaded then the way to get the
+                // relationship changes from $asset->assignedTo to $asset->assigned.
+                return $asset->assigned ? $asset->assigned->display_name : null;
+            }
 
-            if (!$assigned) {
-                return null;
-            }
-            if ($dataPath[0] === 'displayName') {
-                return $assigned->getRawOriginal('display_name') ?? $assigned->display_name;
-            }
-            if ($assigned instanceof User) {
-                return $assigned->full_name;
-            }
-            return $assigned->name ?? $assigned->display_name ?? null;
+            return $asset->assignedTo ? $asset->assignedTo->display_name : null;
         }
 
         // Handle Laravel's stupid Carbon datetime casting

app/Models/Labels/Label.php

@@ -690,7 +690,7 @@ abstract class Label
         // Find one
         if ($name !== null) {
             return static::find()
-                ->first(
+                ->sole(
                     function ($label) use ($name) {
                         return $label->getName() == $name;
                     }

app/Models/License.php

@@ -59,7 +59,7 @@ class License extends Depreciable
         'license_name' => 'string|nullable|max:100',
         'notes' => 'string|nullable',
         'category_id' => 'required|exists:categories,id',
-        'company_id' => 'integer|nullable|exists:companies,id',
+        'company_id' => 'integer|nullable',
         'purchase_cost' => 'numeric|nullable|gte:0|max:99999999999999999.99',
         'purchase_date' => 'date_format:Y-m-d|nullable|max:10|required_with:depreciation_id',
         'expiration_date' => 'date_format:Y-m-d|nullable|max:10',
@@ -803,7 +803,7 @@ class License extends Depreciable
      *
      * @return mixed
      */
-    public function freeSeat(bool $lock = false)
+    public function freeSeat()
     {
         return $this->licenseseats()
             ->whereNull('deleted_at')
@@ -813,7 +813,6 @@ class License extends Depreciable
                     ->whereNull('asset_id');
             })
             ->orderBy('id', 'asc')
-            ->when($lock, fn ($q) => $q->lockForUpdate())
             ->first();
     }