From f2d944b1171f7cbca2ef7256a41da7628d39d670 Mon Sep 17 00:00:00 2001
From: Jacek Pyziak <jacek.pyziak@project-pro.pl>
Date: Mon, 6 Jul 2026 22:17:16 +0200
Subject: [PATCH] Operacje: nowrap dla daty/kwoty, ucinanie opisu i
 stronicowanie 10/str

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
---
 app/Controllers/Operations.php | 10 ++++++-
 app/Models/OperationModel.php  | 31 +++++++++++++++++++---
 app/Views/operations/index.php | 48 ++++++++++++++++++++++++++++------
 3 files changed, 76 insertions(+), 13 deletions(-)

diff --git a/app/Controllers/Operations.php b/app/Controllers/Operations.php
index 98bd8e2..500f765 100644
--- a/app/Controllers/Operations.php
+++ b/app/Controllers/Operations.php
@@ -21,12 +21,20 @@ class Operations extends BaseController
             'type'        => $this->request->getGet('type'),
         ];
 
+        $perPage = 10;
+        $total   = $this->model()->countFiltered($f);
+        $pages   = max(1, (int) ceil($total / $perPage));
+        $page    = min($pages, max(1, (int) $this->request->getGet('page')));
+
         return view('operations/index', [
             'title'      => 'Operacje',
-            'operations' => $this->model()->filtered($f),
+            'operations' => $this->model()->filtered($f, $perPage, $page),
             'totals'     => $this->model()->totals($f),
             'categories' => model(CategoryModel::class)->tree(),
             'filter'     => $f,
+            'page'       => $page,
+            'pages'      => $pages,
+            'total'      => $total,
         ]);
     }
 
diff --git a/app/Models/OperationModel.php b/app/Models/OperationModel.php
index 99d755b..0e260be 100644
--- a/app/Models/OperationModel.php
+++ b/app/Models/OperationModel.php
@@ -10,7 +10,7 @@ class OperationModel extends Model
     protected $primaryKey    = 'id';
     protected $returnType    = 'array';
     protected $useTimestamps = true;
-    protected $allowedFields = ['date', 'amount', 'type', 'category_id', 'description'];
+    protected $allowedFields = ['date', 'amount', 'type', 'category_id', 'receipt_id', 'description'];
 
     protected $validationRules = [
         'date'        => 'required|valid_date[Y-m-d]',
@@ -23,14 +23,39 @@ class OperationModel extends Model
     /**
      * Lista operacji z nazwa kategorii, z filtrami.
      * $f: from, to, category_id, type (wszystkie opcjonalne).
+     * Gdy $perPage > 0, zwraca tylko jedna strone (LIMIT/OFFSET).
      */
-    public function filtered(array $f): array
+    public function filtered(array $f, int $perPage = 0, int $page = 1): array
     {
         $b = $this->db->table('operations o')
             ->select('o.*, c.name AS category_name, p.name AS parent_name')
             ->join('categories c', 'c.id = o.category_id', 'left')
             ->join('categories p', 'p.id = c.parent_id', 'left');
 
+        $this->applyFilters($b, $f);
+
+        $b->orderBy('o.date', 'DESC')->orderBy('o.id', 'DESC');
+
+        if ($perPage > 0) {
+            $b->limit($perPage, max(0, ($page - 1) * $perPage));
+        }
+
+        return $b->get()->getResultArray();
+    }
+
+    /**
+     * Liczba operacji pasujacych do filtrow (do stronicowania).
+     */
+    public function countFiltered(array $f): int
+    {
+        $b = $this->db->table('operations o');
+        $this->applyFilters($b, $f);
+
+        return $b->countAllResults();
+    }
+
+    private function applyFilters($b, array $f): void
+    {
         if (! empty($f['from'])) {
             $b->where('o.date >=', $f['from']);
         }
@@ -43,8 +68,6 @@ class OperationModel extends Model
         if (! empty($f['type'])) {
             $b->where('o.type', $f['type']);
         }
-
-        return $b->orderBy('o.date', 'DESC')->orderBy('o.id', 'DESC')->get()->getResultArray();
     }
 
     /**
diff --git a/app/Views/operations/index.php b/app/Views/operations/index.php
index b45b37f..e426488 100644
--- a/app/Views/operations/index.php
+++ b/app/Views/operations/index.php
@@ -2,6 +2,9 @@
 
 <?= $this->section('content') ?>
 <?php $fmt = static fn ($v) => number_format((float) $v, 2, ',', ' ') . ' zł'; ?>
+<style>
+    .col-desc { max-width: 320px; overflow: hidden; text-overflow: ellipsis; white-space: nowrap; }
+</style>
 
 <div class="d-flex justify-content-between align-items-center mb-3">
     <h1 class="h4 mb-0">Operacje</h1>
@@ -65,10 +68,10 @@
         <table class="table table-hover mb-0 align-middle">
             <thead class="table-light">
                 <tr>
-                    <th>Data</th>
-                    <th>Kategoria</th>
+                    <th class="text-nowrap">Data</th>
+                    <th class="text-nowrap">Kategoria</th>
                     <th>Opis</th>
-                    <th class="text-end">Kwota</th>
+                    <th class="text-end text-nowrap">Kwota</th>
                     <th class="text-end">Akcje</th>
                 </tr>
             </thead>
@@ -77,8 +80,8 @@
                 <tr><td colspan="5" class="text-center text-muted py-4">Brak operacji dla wybranych filtrów.</td></tr>
             <?php else: foreach ($operations as $o): ?>
                 <tr>
-                    <td><?= esc($o['date']) ?></td>
-                    <td>
+                    <td class="text-nowrap"><?= esc($o['date']) ?></td>
+                    <td class="text-nowrap">
                         <?php
                             if (! empty($o['category_name'])) {
                                 echo esc(! empty($o['parent_name'])
@@ -89,11 +92,11 @@
                             }
                         ?>
                     </td>
-                    <td class="text-muted"><?= esc($o['description'] ?? '') ?></td>
-                    <td class="text-end <?= $o['type'] === 'income' ? 'amount-income' : 'amount-expense' ?>">
+                    <td class="text-muted col-desc" title="<?= esc($o['description'] ?? '') ?>"><?= esc($o['description'] ?? '') ?></td>
+                    <td class="text-end text-nowrap <?= $o['type'] === 'income' ? 'amount-income' : 'amount-expense' ?>">
                         <?= $o['type'] === 'income' ? '+' : '−' ?><?= $fmt($o['amount']) ?>
                     </td>
-                    <td class="text-end">
+                    <td class="text-end text-nowrap">
                         <a href="<?= site_url('operations/' . $o['id'] . '/edit') ?>" class="btn btn-sm btn-outline-secondary">Edytuj</a>
                         <a href="<?= site_url('operations/' . $o['id'] . '/delete') ?>" class="btn btn-sm btn-outline-danger"
                            onclick="return confirm('Usunąć operację?')">Usuń</a>
@@ -103,5 +106,34 @@
             </tbody>
         </table>
     </div>
+    <?php if ($pages > 1): ?>
+        <?php
+            $qs = array_filter([
+                'from'        => $filter['from'] ?? '',
+                'to'          => $filter['to'] ?? '',
+                'category_id' => $filter['category_id'] ?? '',
+                'type'        => $filter['type'] ?? '',
+            ], static fn ($v) => $v !== '' && $v !== null);
+            $link = static fn (int $p) => site_url('operations') . '?' . http_build_query($qs + ['page' => $p]);
+        ?>
+        <div class="card-footer d-flex justify-content-between align-items-center flex-wrap gap-2">
+            <span class="text-muted small">Strona <?= $page ?> z <?= $pages ?> (<?= $total ?> operacji)</span>
+            <nav>
+                <ul class="pagination pagination-sm mb-0">
+                    <li class="page-item <?= $page <= 1 ? 'disabled' : '' ?>">
+                        <a class="page-link" href="<?= $link(max(1, $page - 1)) ?>">‹</a>
+                    </li>
+                    <?php for ($p = 1; $p <= $pages; $p++): ?>
+                        <li class="page-item <?= $p === $page ? 'active' : '' ?>">
+                            <a class="page-link" href="<?= $link($p) ?>"><?= $p ?></a>
+                        </li>
+                    <?php endfor; ?>
+                    <li class="page-item <?= $page >= $pages ? 'disabled' : '' ?>">
+                        <a class="page-link" href="<?= $link(min($pages, $page + 1)) ?>">›</a>
+                    </li>
+                </ul>
+            </nav>
+        </div>
+    <?php endif; ?>
 </div>
 <?= $this->endSection() ?>