@forelse ($this->replyQuickActions as $qa)
@empty
@@ -439,6 +451,7 @@
MONITOR CZASU PRACY
- Czas w zgłoszeniu:
+ {{-- Guarded call: even with the interval/listener cleanup above, a
+ refreshTicketData() response already in flight when the operator
+ navigates away can still land afterward and get morphed against
+ a scope that's mid-teardown — typeof-guarding here means that
+ race can no longer throw, whatever exactly triggers it. --}}
+ Czas w zgłoszeniu:
edit
@if ($ticket->isClosed())
diff --git a/src/tests/Feature/AdminLogsViewerTest.php b/src/tests/Feature/AdminLogsViewerTest.php
new file mode 100644
index 0000000..7493b21
--- /dev/null
+++ b/src/tests/Feature/AdminLogsViewerTest.php
@@ -0,0 +1,78 @@
+create(['name' => 'Admin', 'email' => 'admin-logs@example.com', 'roles' => ['admin']]);
+}
+
+beforeEach(function () {
+ $this->logPath = storage_path('logs/pest-admin-logs-test.log');
+ file_put_contents($this->logPath, implode("\n", [
+ '[2026-08-05 08:00:00] production.INFO: normalny wpis o połączeniu',
+ '[2026-08-05 08:01:00] production.ERROR: coś się nie udało',
+ '#0 stack trace line belonging to the ERROR entry above',
+ '[2026-08-05 08:02:00] production.WARNING: uwaga na przyszłość',
+ ])."\n");
+});
+
+afterEach(function () {
+ @unlink($this->logPath);
+});
+
+test('admin sees the log file list and its tail content', function () {
+ $admin = adminUserForLogsTest();
+
+ Livewire::actingAs($admin)->test(Logs::class)
+ ->set('selectedFile', 'pest-admin-logs-test.log')
+ ->assertSee('pest-admin-logs-test.log')
+ ->assertSee('normalny wpis o połączeniu')
+ ->assertSee('coś się nie udało')
+ ->assertSee('stack trace line belonging to the ERROR entry above');
+});
+
+test('multi-line entries stay grouped and level filter narrows to matching entries', function () {
+ $admin = adminUserForLogsTest();
+
+ $component = Livewire::actingAs($admin)->test(Logs::class)
+ ->set('selectedFile', 'pest-admin-logs-test.log');
+
+ expect($component->get('entries'))->toHaveCount(3);
+
+ $component->set('levelFilter', 'ERROR');
+ $entries = $component->get('entries');
+
+ expect($entries)->toHaveCount(1)
+ ->and($entries[0]['text'])->toContain('coś się nie udało')
+ ->and($entries[0]['text'])->toContain('stack trace line belonging to the ERROR entry above');
+});
+
+test('search filters entries by substring', function () {
+ $admin = adminUserForLogsTest();
+
+ $component = Livewire::actingAs($admin)->test(Logs::class)
+ ->set('selectedFile', 'pest-admin-logs-test.log')
+ ->set('search', 'uwaga');
+
+ expect($component->get('entries'))->toHaveCount(1)
+ ->and($component->get('entries')[0]['text'])->toContain('uwaga na przyszłość');
+});
+
+test('selecting an unknown file name is ignored, preventing path traversal via the public property', function () {
+ $admin = adminUserForLogsTest();
+
+ $component = Livewire::actingAs($admin)->test(Logs::class)
+ ->set('selectedFile', 'pest-admin-logs-test.log')
+ ->call('selectFile', '../../.env');
+
+ expect($component->get('selectedFile'))->toBe('pest-admin-logs-test.log');
+});
+
+test('non-admin cannot open the admin panel logs tab', function () {
+ $operator = User::query()->create(['name' => 'Op', 'email' => 'op-logs@example.com', 'roles' => ['operator']]);
+
+ $this->actingAs($operator)->get('/admin')->assertForbidden();
+});
diff --git a/src/tests/Feature/StatsCategoryBreakdownTest.php b/src/tests/Feature/StatsCategoryBreakdownTest.php
new file mode 100644
index 0000000..4ec069d
--- /dev/null
+++ b/src/tests/Feature/StatsCategoryBreakdownTest.php
@@ -0,0 +1,59 @@
+create(['name' => 'Admin', 'email' => 'stats-admin-cat@example.com', 'roles' => ['admin']]);
+
+ $it = Category::query()->create(['name' => 'IT-Pomoc']);
+ $vpn = $it->subcategories()->create(['name' => 'VPN']);
+ $delegacje = Category::query()->create(['name' => 'Delegacje']);
+
+ makeTicket(['number' => '7001', 'subcategory_id' => $vpn->id]);
+ makeTicket(['number' => '7002', 'subcategory_id' => $vpn->id]);
+ // Routed to a whole category with no subcategory (e.g. an IMAP mailbox
+ // routed to "całą kategorię") — previously invisible to byCategory().
+ makeTicket(['number' => '7003', 'category_id' => $delegacje->id]);
+ makeTicket(['number' => '7004', 'category_id' => $delegacje->id]);
+ makeTicket(['number' => '7005', 'category_id' => $delegacje->id]);
+
+ $rows = Livewire::actingAs($admin)->test(Stats::class)->instance()->byCategory;
+
+ expect($rows->firstWhere('label', 'IT-Pomoc'))->toBe(['label' => 'IT-Pomoc', 'count' => 2])
+ ->and($rows->firstWhere('label', 'Delegacje'))->toBe(['label' => 'Delegacje', 'count' => 3])
+ ->and($rows->count())->toBe(2);
+});
+
+test('byCategory sums both bare-category and subcategorized tickets into the same category total', function () {
+ seedStatusesAndPriorities();
+ $admin = User::query()->create(['name' => 'Admin', 'email' => 'stats-admin-cat-2@example.com', 'roles' => ['admin']]);
+
+ $it = Category::query()->create(['name' => 'IT-Pomoc']);
+ $vpn = $it->subcategories()->create(['name' => 'VPN']);
+
+ makeTicket(['number' => '7101', 'subcategory_id' => $vpn->id]);
+ makeTicket(['number' => '7102', 'category_id' => $it->id]);
+
+ $rows = Livewire::actingAs($admin)->test(Stats::class)->instance()->byCategory;
+
+ expect($rows->firstWhere('label', 'IT-Pomoc'))->toBe(['label' => 'IT-Pomoc', 'count' => 2]);
+});
+
+test('filterCategory includes tickets routed to a bare category with no subcategory', function () {
+ seedStatusesAndPriorities();
+ $admin = User::query()->create(['name' => 'Admin', 'email' => 'stats-admin-cat-3@example.com', 'roles' => ['admin']]);
+
+ $delegacje = Category::query()->create(['name' => 'Delegacje']);
+ makeTicket(['number' => '7201', 'category_id' => $delegacje->id]);
+ makeTicket(['number' => '7202']);
+
+ $kpis = Livewire::actingAs($admin)->test(Stats::class)
+ ->set('filterCategory', (string) $delegacje->id)
+ ->instance()->kpis;
+
+ expect($kpis['total'])->toBe(1);
+});
diff --git a/src/tests/Feature/StatsSubcategoryBreakdownTest.php b/src/tests/Feature/StatsSubcategoryBreakdownTest.php
index ffc593a..3bfb58e 100644
--- a/src/tests/Feature/StatsSubcategoryBreakdownTest.php
+++ b/src/tests/Feature/StatsSubcategoryBreakdownTest.php
@@ -28,3 +28,22 @@ test('bySubcategory groups tickets per subcategory, labeled "Category / Subcateg
->and($rows->firstWhere('label', 'Zamówienia / Sprzęt'))->toBe(['label' => 'Zamówienia / Sprzęt', 'count' => 1])
->and($rows->count())->toBe(3);
});
+
+test('bySubcategory gives tickets routed to a bare category (no subcategory) their own "(bez podkategorii)" row', function () {
+ seedStatusesAndPriorities();
+ $admin = User::query()->create(['name' => 'Admin', 'email' => 'stats-admin-sub-2@example.com', 'roles' => ['admin']]);
+
+ $it = Category::query()->create(['name' => 'IT-Pomoc']);
+ $vpn = $it->subcategories()->create(['name' => 'VPN']);
+ $delegacje = Category::query()->create(['name' => 'Delegacje']);
+
+ makeTicket(['number' => '4101', 'subcategory_id' => $vpn->id]);
+ makeTicket(['number' => '4102', 'category_id' => $delegacje->id]);
+ makeTicket(['number' => '4103', 'category_id' => $delegacje->id]);
+
+ $rows = Livewire::actingAs($admin)->test(Stats::class)->instance()->bySubcategory;
+
+ expect($rows->firstWhere('label', 'IT-Pomoc / VPN'))->toBe(['label' => 'IT-Pomoc / VPN', 'count' => 1])
+ ->and($rows->firstWhere('label', 'Delegacje (bez podkategorii)'))->toBe(['label' => 'Delegacje (bez podkategorii)', 'count' => 2])
+ ->and($rows->count())->toBe(2);
+});
diff --git a/src/tests/Feature/TicketBusinessRulesTest.php b/src/tests/Feature/TicketBusinessRulesTest.php
index 67c6eee..b71acb4 100644
--- a/src/tests/Feature/TicketBusinessRulesTest.php
+++ b/src/tests/Feature/TicketBusinessRulesTest.php
@@ -41,6 +41,18 @@ test('next ticket number is one above the current max', function () {
expect(Ticket::nextNumber())->toBe('1002');
});
+test('next ticket number compares numerically, not lexicographically, across differing digit counts', function () {
+ seedStatusesAndPriorities();
+
+ // A plain string MAX()/ORDER BY would rank '999' above '1000' (lexicographic
+ // "9" > "1"), which is exactly the bug nextNumber()'s length-first ordering
+ // guards against — see Ticket::nextNumber().
+ makeTicket(['number' => '999']);
+ makeTicket(['number' => '1000']);
+
+ expect(Ticket::nextNumber())->toBe('1001');
+});
+
test('sla info reports overdue once the resolution deadline has passed', function () {
seedStatusesAndPriorities();
diff --git a/src/tests/Feature/TicketFieldValuesSyncTest.php b/src/tests/Feature/TicketFieldValuesSyncTest.php
new file mode 100644
index 0000000..e39b8cb
--- /dev/null
+++ b/src/tests/Feature/TicketFieldValuesSyncTest.php
@@ -0,0 +1,50 @@
+create(['name' => 'IT-Pomoc']);
+ $subcategory = $category->subcategories()->create(['name' => 'VPN']);
+ $field = CustomField::query()->create(['label' => 'Teamviewer-ID', 'type' => 'text', 'required' => false, 'sort_order' => 1]);
+ $field->subcategories()->attach($subcategory->id, ['position' => 1]);
+
+ $ticket = makeTicket(['subcategory_id' => $subcategory->id, 'custom_fields' => [$field->id => '123-456']]);
+
+ expect($ticket->fieldValues)->toHaveCount(1);
+ expect($ticket->fieldValues->first()->custom_field_id)->toBe($field->id);
+ expect($ticket->fieldValues->first()->value)->toBe('123-456');
+});
+
+test('updating custom_fields keeps ticket_field_values in sync — added, changed and removed entries', function () {
+ seedStatusesAndPriorities();
+ $category = Category::query()->create(['name' => 'IT-Pomoc']);
+ $subcategory = $category->subcategories()->create(['name' => 'VPN']);
+ $fieldA = CustomField::query()->create(['label' => 'A', 'type' => 'text', 'required' => false, 'sort_order' => 1]);
+ $fieldB = CustomField::query()->create(['label' => 'B', 'type' => 'text', 'required' => false, 'sort_order' => 2]);
+ $fieldA->subcategories()->attach($subcategory->id, ['position' => 1]);
+ $fieldB->subcategories()->attach($subcategory->id, ['position' => 2]);
+
+ $ticket = makeTicket([
+ 'subcategory_id' => $subcategory->id,
+ 'custom_fields' => [$fieldA->id => 'first', $fieldB->id => 'second'],
+ ]);
+
+ $ticket->update(['custom_fields' => [$fieldA->id => 'changed']]);
+
+ expect($ticket->fieldValues()->count())->toBe(1);
+ expect($ticket->fieldValues()->first()->value)->toBe('changed');
+});
+
+test('a blank custom field value is not synced into ticket_field_values', function () {
+ seedStatusesAndPriorities();
+ $category = Category::query()->create(['name' => 'IT-Pomoc']);
+ $subcategory = $category->subcategories()->create(['name' => 'VPN']);
+ $field = CustomField::query()->create(['label' => 'A', 'type' => 'text', 'required' => false, 'sort_order' => 1]);
+ $field->subcategories()->attach($subcategory->id, ['position' => 1]);
+
+ $ticket = makeTicket(['subcategory_id' => $subcategory->id, 'custom_fields' => [$field->id => '']]);
+
+ expect($ticket->fieldValues()->count())->toBe(0);
+});
diff --git a/src/tests/Feature/TicketSourceValidationTest.php b/src/tests/Feature/TicketSourceValidationTest.php
new file mode 100644
index 0000000..c7cc19a
--- /dev/null
+++ b/src/tests/Feature/TicketSourceValidationTest.php
@@ -0,0 +1,38 @@
+ uniqid(), 'source' => $source]);
+ expect($ticket->source)->toBe($source);
+ }
+});
+
+test('creating a ticket with an unrecognized source throws instead of silently storing it', function () {
+ seedStatusesAndPriorities();
+
+ expect(fn () => makeTicket(['source' => 'totally-made-up']))
+ ->toThrow(InvalidArgumentException::class);
+});
+
+test('a ticket message accepts a null source (implicit web) and the known email source', function () {
+ seedStatusesAndPriorities();
+ $ticket = makeTicket();
+
+ $webMessage = $ticket->messages()->create(['author_name' => 'A', 'body' => 'web reply']);
+ $mailMessage = $ticket->messages()->create(['author_name' => 'B', 'body' => 'mail reply', 'source' => 'email']);
+
+ expect($webMessage->source)->toBeNull()
+ ->and($mailMessage->source)->toBe('email');
+});
+
+test('a ticket message with an unrecognized source throws instead of silently storing it', function () {
+ seedStatusesAndPriorities();
+ $ticket = makeTicket();
+
+ expect(fn () => $ticket->messages()->create(['author_name' => 'A', 'body' => 'x', 'source' => 'totally-made-up']))
+ ->toThrow(InvalidArgumentException::class);
+});
diff --git a/wiki/admin/README.md b/wiki/admin/README.md
index 879b2b0..144339c 100644
--- a/wiki/admin/README.md
+++ b/wiki/admin/README.md
@@ -353,6 +353,17 @@ razem, zamiast być rozrzucone po różnych zakładkach.
- **Prompt systemowy podsumowania** — edytowalne pole tekstowe z gotową
wartością domyślną i przyciskiem **„Resetuj”**.
+## Logi
+
+Zakładka **„Logi”** pokazuje zawartość plików `storage/logs/*.log` (aplikacja,
+IMAP, automatyzacja AI, import z Heska, itd.) bez potrzeby dostępu do
+kontenera przez SSH/shell. Lista plików po lewej pokazuje rozmiar i datę
+ostatniej modyfikacji (najnowsze na górze) — kliknięcie ładuje zawartość.
+Filtry: poziom (ERROR/WARNING/INFO/...), wyszukiwanie tekstowe i liczba
+pokazywanych wpisów (100–3000, pokazuje ostatnie N pasujących), plus
+opcjonalne automatyczne odświeżanie co 5 s. Widok jest tylko do odczytu i
+dostępny wyłącznie dla administratorów.
+
## API
Panel `/admin/api-docs` udostępnia interaktywną dokumentację (Swagger) REST API
diff --git a/wiki/operator/README.md b/wiki/operator/README.md
index 0a14942..331eac5 100644
--- a/wiki/operator/README.md
+++ b/wiki/operator/README.md
@@ -47,7 +47,9 @@ otworzyłeś, najnowsze na górze.
e-mailu dowolnego konta (nie tylko klientów), pokazuje liczbę jego zgłoszeń i
prowadzi od razu do kolejki przefiltrowanej do tego klienta.
-**Filtry** nad tabelą: status, priorytet, kategoria, wyszukiwanie po numerze/
+**Filtry** nad tabelą: status, priorytet, kategoria (opcja **„Bez kategorii”**
+pokazuje zgłoszenia bez przypisanej kategorii i podkategorii — np. z poczty
+IMAP, która nie trafiła w żadną), wyszukiwanie po numerze/
temacie/kliencie/treści zgłoszenia i odpowiedzi w wątku. **Kolumny** można dowolnie
włączać/wyłączać przyciskiem „Kolumny” (numer, temat, klient, kategoria,
podkategoria, priorytet, status, SLA, przypisany, zespół, utworzono — kilka z