small fixes for QR code and labels generator

Author: Flatroy

README.md

@@ -95,7 +95,9 @@ Still some part are in WIP status: I need to add queue worker to docker-compose.
   - [x] Add spotlight. Click: CTRL + K or CMD + K or CTRL + / or CMD + /
   - [x] Add Label Generator
   - [ ] Add REST API
-
+  - [ ] Add backup  [via spatie package](https://github.com/shuvroroy/filament-spatie-laravel-backu)
+  - [ ] add https://github.com/CodeWithDennis/filament-select-tree
+ 
 
 ## Contributing
 

app/Filament/Pages/LabelsGenerator.php

@@ -76,11 +76,23 @@ public function form(Form $form): Form
                     ->label('Label Height (inches)')
                     ->numeric()
                     ->required()
-                    ->default($this->cardHeight),
+                    ->default($this->cardHeight)
+                    ->rules(['required', 'numeric', 'min:0.1', function($attribute, $value, $fail) {
+                        $availableHeight = $this->pageHeight - $this->pageTopPadding - $this->pageBottomPadding;
+                        if ($value > $availableHeight) {
+                            $fail("The card height must not exceed the available page height.");
+                        }
+                    }]),
                 TextInput::make('cardWidth')
                     ->label('Label Width (inches)')
                     ->numeric()
                     ->required()
+                    ->rules(['required', 'numeric', 'min:0.1', function($attribute, $value, $fail) {
+                        $availableWidth = $this->pageWidth - $this->pageLeftPadding - $this->pageRightPadding;
+                        if ($value > $availableWidth) {
+                            $fail("The card width must not exceed the available page width.");
+                        }
+                    }])
                     ->default($this->cardWidth),
                 TextInput::make('pageWidth')
                     ->label('Page Width (inches)')
@@ -124,7 +136,7 @@ public function form(Form $form): Form
             ])->columns(2);
     }
 
-    public function generatePages($showNotification = true)
+    public function generatePages($showNotification = true): void
     {
         $this->validate();
         $this->init();
@@ -181,11 +193,10 @@ private function calculateGridData()
             ];
         }
 
-        $cols = floor($availablePageWidth / $this->cardWidth);
-        $rows = floor($availablePageHeight / $this->cardHeight);
-        $gapX = ($availablePageWidth - $cols * $this->cardWidth) / ($cols - 1);
-        $gapY = ($this->pageHeight - $rows * $this->cardHeight) / ($rows - 1);
-
+        $cols = max(1, floor($availablePageWidth / $this->cardWidth));
+        $rows = max(1, floor($availablePageHeight / $this->cardHeight));
+        $gapX = $cols > 1 ? ($availablePageWidth - $cols * $this->cardWidth) / ($cols - 1) : 0;
+        $gapY = $rows > 1 ? ($availablePageHeight - $rows * $this->cardHeight) / ($rows - 1) : 0;
         return [
             'cols' => $cols,
             'rows' => $rows,

routes/web.php

@@ -15,24 +15,27 @@
     Route::get('/dashboard', function () {
         return redirect()->to('/app');
     })->name('dashboard');
+
+
+    Route::get('/a/{asset_id}', function (string $asset_id) {
+        $team_id = auth()?->user()?->currentTeam()?->getChild()?->id ?: auth()->user()->ownedTeams()->first()->id;
+
+        // todo: put this logic in one place
+        $asset_id = ltrim($asset_id, '0');
+        $asset_id = ltrim($asset_id, '-');
+
+        $asset = \App\Models\Item::where('asset_id', $asset_id)
+            ->where('team_id', $team_id)
+            ->first();
+        if (! $asset) {
+            // create new empty one with this asset id
+            $asset = new \App\Models\Item();
+            $asset->asset_id = $asset_id;
+            $asset->team_id = $team_id;
+            $asset->save();
+        }
+
+        return redirect()->to(ItemResource::getUrl('edit', ['record' => $asset, 'tenant' => $team_id]));
+    })->name('asset');
 });
 
-Route::get('/a/{asset_id}', function (string $asset_id) {
-    $team_id = auth()?->user()?->currentTeam()?->getChild()?->id;
-    // todo: put this logic in one place
-    $asset_id = ltrim($asset_id, '0');
-    $asset_id = ltrim($asset_id, '-');
-
-    $asset = \App\Models\Item::where('asset_id', $asset_id)
-        ->where('team_id', $team_id)
-        ->first();
-    if (! $asset) {
-        // create new empty one with this asset id
-        $asset = new \App\Models\Item();
-        $asset->asset_id = $asset_id;
-        $asset->team_id = $team_id;
-        $asset->save();
-    }
-
-    return redirect()->to(ItemResource::getUrl('edit', ['record' => $asset, 'tenant' => $team_id]));
-})->name('asset');