diff --git a/public/api/openapi.php b/public/api/openapi.php index 713629c..6788e39 100644 --- a/public/api/openapi.php +++ b/public/api/openapi.php @@ -374,6 +374,7 @@ $spec = [ ['name' => 'severity', 'in' => 'query', 'schema' => ['type' => 'string'], 'description' => 'Mehrere kommagetrennt'], ['name' => 'target_agent', 'in' => 'query', 'schema' => ['type' => 'string']], ['name' => 'unclaimed_only', 'in' => 'query', 'schema' => ['type' => 'boolean']], + ['name' => 'search', 'in' => 'query', 'schema' => ['type' => 'string'], 'description' => 'Volltext ueber Titel, Beschreibung, Fehlermeldung, Tags, Dateipfad. Eine Zahl findet zusaetzlich das Item mit dieser Nummer; mit vorangestellter Raute (#288) ausschliesslich dieses eine.'], ['name' => 'updated_since', 'in' => 'query', 'schema' => ['type' => 'string', 'format' => 'date-time'], 'description' => 'Delta-Abfrage fuer Polling'], ['name' => 'order', 'in' => 'query', 'schema' => ['type' => 'string', 'enum' => ['newest', 'oldest', 'updated', 'severity', 'occurrences']]], ['name' => 'limit', 'in' => 'query', 'schema' => ['type' => 'integer', 'default' => 100, 'maximum' => 500]], diff --git a/public/docs/bugtracker.md b/public/docs/bugtracker.md index 2bcdc8d..2abe74b 100644 --- a/public/docs/bugtracker.md +++ b/public/docs/bugtracker.md @@ -275,7 +275,7 @@ curl "https://dc.mhdf.de/api/bugtracker/v1/manage?action=list&project_slug=myapp | `status`, `severity` | Mehrere Werte kommagetrennt | | `type`, `environment`, `project_slug` | Einzelwert oder `all` | | `target_agent`, `claimed_by`, `push_id` | Exakte Übereinstimmung | -| `search` | Volltext über Titel, Beschreibung, Fehlermeldung, Tags, Dateipfad | +| `search` | Volltext über Titel, Beschreibung, Fehlermeldung, Tags, Dateipfad. Eine Zahl findet zusätzlich das Item mit dieser **Nummer**, `#288` ausschließlich dieses eine | | `unclaimed_only` | `true` — nur Items, die kein Agent bearbeitet | | `updated_since` | ISO-8601 — **Delta-Abfrage für effizientes Polling** | | `order` | `newest`, `oldest`, `updated`, `severity`, `occurrences` | diff --git a/public/docs/changelog.json b/public/docs/changelog.json index 594b449..57fa8b8 100644 --- a/public/docs/changelog.json +++ b/public/docs/changelog.json @@ -43,13 +43,25 @@ "area": "webui", "breaking": false, "title": "Filter und Formulare werfen nicht mehr aufs Dashboard zurueck", - "text": "Welche Ansicht sichtbar ist, hing allein am Anker der URL. Fuer POST-Formulare gab es dafuer laengst eine Loesung (_return und dc_redirect), die Filter fielen aber durch beide Netze: Es sind GET-Formulare, und ein GET-Formular verwirft beim Absenden den Anker seiner action; ausserdem loesten sie ueber form.submit() aus, was gar kein submit-Ereignis erzeugt. Nach jeder Filteraenderung stand man wieder im Dashboard und musste zurueckklicken - beim Filtern nach mehreren Parametern jedes Mal. Jetzt haengt der zentrale Zuhoerer die offene Ansicht bei GET als Parameter 'view' an den Query-String, ausgeloest wird ueber requestSubmit(), und beim Laden wird die Ansicht aus 'view' wiederhergestellt, wenn kein Anker da ist." + "text": "Welche Ansicht sichtbar ist, hing allein am Anker der URL - und die Ansichtsrettung, die das auffangen sollte, griff aus zwei Gruenden nicht.\n\nErstens fragte sie dokumentweit nach der offenen Unteransicht. Im Markup tragen aber ACHT Unteransichten die Klasse \"active\": jedes Modul hat seine eigene Startansicht, und switchMainTab() raeumt nur innerhalb des Moduls auf, das es oeffnet. Zurueck kam deshalb immer die erste in Dokumentreihenfolge, unabhaengig davon, was gerade sichtbar war. Dass die Rettung trotzdem zu funktionieren schien, lag an den wenigen Formularen, die ihren Anker fest in der action stehen haben. Gesucht wird jetzt erst das aktive Modul und darin die Unteransicht.\n\nZweitens fielen die Filter durch das Netz: Es sind GET-Formulare, und ein GET-Formular verwirft beim Absenden den Anker seiner action; ausserdem loesten sie ueber form.submit() aus, was gar kein submit-Ereignis erzeugt. Der zentrale Zuhoerer haengt die offene Ansicht bei GET jetzt als Parameter \"view\" an den Query-String, ausgeloest wird ueber requestSubmit(), und beim Laden wird die Ansicht daraus wiederhergestellt. Die drei Filterformulare tragen das Feld zusaetzlich fest im Markup - dort ist ohne Zweifel bekannt, wo sie stehen, und das haengt an keiner DOM-Abfrage mehr." }, { "area": "api", "breaking": false, "title": "/api/bugtracker/v1/roadmap ist jetzt auch ohne .php erreichbar", "text": "Jeder andere Endpunkt kommt ohne Dateiendung aus, dieser nicht: Die Route fehlte in der .htaccess, der Aufruf lief in ein 404, und die OpenAPI-Beschreibung fuehrte deshalb als einzigen Pfad einen mit .php. Wer dem Muster der uebrigen API folgte, lief auf. Die Route ist nachgetragen, die Beschreibung nennt jetzt den Pfad ohne Endung. Der alte Pfad mit .php funktioniert unveraendert weiter - er faellt auf die allgemeine Regel fuer /api/ zurueck." + }, + { + "area": "webui", + "breaking": false, + "title": "Die Suche findet Eintraege ueber ihre Nummer", + "text": "Die Nummer ist das, was ueberall zitiert wird - in Commits, in Kommentaren, im Changelog - und war ausgerechnet ueber die Suche nicht auffindbar: Sie steht in keiner der durchsuchten Spalten. Wer \"288\" eingab, bekam nichts und musste die Liste durchblaettern. Jetzt sucht die Nummer mit, im Bugtracker wie im Fehler-Stream und ebenso ueber den Parameter search der API. Zwei Schreibweisen, zwei Bedeutungen: \"288\" findet das Item mit dieser Nummer UND alles, was die Zahl im Text stehen hat; \"#288\" ist eindeutig gemeint und liefert ausschliesslich dieses eine Item." + }, + { + "area": "api", + "breaking": false, + "title": "Der Parameter search ist jetzt beschrieben", + "text": "action=list kennt search seit jeher, in der OpenAPI-Beschreibung stand er nicht. Nachgetragen, mitsamt dem Verhalten bei Zahlen." } ] }, diff --git a/public/index.php b/public/index.php index 34e0aeb..d819ecf 100644 --- a/public/index.php +++ b/public/index.php @@ -4018,6 +4018,13 @@ chmod +x update-agent
+ + + style="width:190px; padding:0.3rem 0.6rem; font-size:0.8rem;" placeholder="🔍 Suchen oder #288" + title="Volltext ueber Titel, Beschreibung, Fehlermeldung, Tags und Dateipfad. Eine Zahl findet zusaetzlich das Item mit dieser Nummer; mit Raute (#288) nur dieses eine."> @@ -5026,6 +5041,13 @@ chmod +x update-agent + + + style="width:190px; padding:0.3rem 0.6rem; font-size:0.8rem;" placeholder="🔍 Suchen oder #288" + title="Volltext ueber Titel, Beschreibung, Fehlermeldung, Tags und Dateipfad. Eine Zahl findet zusaetzlich das Item mit dieser Nummer; mit Raute (#288) nur dieses eine."> Zurücksetzen
@@ -6260,9 +6283,24 @@ SYSTEM * Absenden aus einem onchange ueberhaupt ein submit-Ereignis entsteht. */ function dcCurrentView() { - const activeSub = document.querySelector('.subtab-content.active'); + // Erst das Modul, dann die Unteransicht DARIN. Die Reihenfolge ist + // der Kern der Sache: Im Markup tragen ACHT Unteransichten die + // Klasse "active" - jedes Modul hat seine eigene Startansicht, und + // switchMainTab() raeumt nur innerhalb des Moduls auf, das es + // oeffnet. Ein dokumentweites querySelector('.subtab-content.active') + // liefert deshalb immer die erste in Dokumentreihenfolge + // (sub-projects-list), voellig unabhaengig davon, was gerade + // sichtbar ist. Genau daran ist die Ansichtsrettung fuer POST seit + // jeher gescheitert - und der Anschein, sie funktioniere, kam von + // den wenigen Formularen, die ihren Anker fest in der action + // stehen haben. const activeTab = document.querySelector('.main-content > .tab-content.active'); - return activeSub ? activeSub.id : (activeTab ? activeTab.id : ''); + if (!activeTab) { + return ''; + } + + const activeSub = activeTab.querySelector('.subtab-content.active'); + return activeSub ? activeSub.id : activeTab.id; } /** @@ -6995,6 +7033,15 @@ SYSTEM const moduleName = moduleOfSubtab(hash); if (!moduleName) { + // Unbekannte Unteransicht - dann wenigstens das Modul, wenn es + // sich aus dem Namen ergibt. Wortlos im Dashboard zu landen ist + // die schlechteste aller Antworten: Der Nutzer sieht nicht, + // dass etwas schiefging, sondern nur, dass er woanders ist. + const rest = hash.substring(4); + const guess = Object.keys(subnavItems).find(m => rest.startsWith(m + '-')); + if (guess) { + switchMainTab(guess, null, null, false); + } return; } diff --git a/src/Modules/Bugtracker/BugRepo.php b/src/Modules/Bugtracker/BugRepo.php index bc863ed..19e58ff 100644 --- a/src/Modules/Bugtracker/BugRepo.php +++ b/src/Modules/Bugtracker/BugRepo.php @@ -1121,16 +1121,43 @@ final class BugRepo // Volltextsuche. Jede Spalte bekommt einen eigenen Platzhalter - // derselbe Name mehrfach ist bei nativen Prepared Statements unzulaessig. + // + // Die Nummer sucht mit. Sie ist das, was ueberall zitiert wird - in + // Commits, in Kommentaren, im Changelog -, und war ausgerechnet ueber + // die Suche nicht auffindbar: Sie steht in keiner der durchsuchten + // Spalten. Wer "288" eingab, bekam nichts und musste die Liste + // durchblaettern. + // + // Zwei Schreibweisen, zwei Bedeutungen: + // #288 - eindeutig gemeint ist dieses eine Item. Nur die ID. + // 288 - kann auch im Text vorkommen. ID ODER Volltext. $search = $filters['search'] ?? null; if (is_string($search) && trim($search) !== '') { - $needle = '%' . trim($search) . '%'; - $columns = ['title', 'description', 'error_message', 'push_id', 'target_agent', 'tags', 'file_path']; + $roh = trim($search); + $text = ltrim($roh, '#'); + + // Die Laengengrenze haelt die Umwandlung nach int im Rahmen; eine + // 30-stellige Zahl waere ohnehin keine Item-Nummer. + $istNummer = $text !== '' && strlen($text) <= 18 && ctype_digit($text); + $nurNummer = $istNummer && $roh !== $text; + $parts = []; - foreach ($columns as $i => $column) { - $ph = ':search_' . $i; - $parts[] = $column . ' LIKE ' . $ph; - $params[$ph] = $needle; + + if ($istNummer) { + $parts[] = 'id = :search_id'; + $params[':search_id'] = (int)$text; } + + if (!$nurNummer) { + $needle = '%' . ($text !== '' ? $text : $roh) . '%'; + $columns = ['title', 'description', 'error_message', 'push_id', 'target_agent', 'tags', 'file_path']; + foreach ($columns as $i => $column) { + $ph = ':search_' . $i; + $parts[] = $column . ' LIKE ' . $ph; + $params[$ph] = $needle; + } + } + $where[] = '(' . implode(' OR ', $parts) . ')'; } diff --git a/tests/e2e/suite_bugtracker.py b/tests/e2e/suite_bugtracker.py index d855428..2af4d78 100644 --- a/tests/e2e/suite_bugtracker.py +++ b/tests/e2e/suite_bugtracker.py @@ -184,6 +184,47 @@ def _(ctx): erwarte(ctx['bug_id'] in ids, "Item #%s nicht in der Liste (%s)" % (ctx['bug_id'], ids[:10])) +@test('bugtracker', 'Suche nach der Nummer findet das Item') +def _(ctx): + """ + Die Nummer ist das, was ueberall zitiert wird - und war ausgerechnet + ueber die Suche nicht auffindbar: Sie steht in keiner der durchsuchten + Spalten. Wer sie eingab, bekam nichts. + """ + r = ctx.client.get('/api/bugtracker/v1/manage', + query={'action': 'list', 'project_slug': PROJEKT, + 'search': str(ctx['bug_id']), 'limit': 50}) + erwarte_erfolg(r) + ids = [i.get('id') for i in _liste(r, 'items', 'bugs', 'data') if isinstance(i, dict)] + erwarte(ctx['bug_id'] in ids, + "Suche nach %s findet das Item nicht (%s)" % (ctx['bug_id'], ids[:10])) + + +@test('bugtracker', 'Suche mit Raute liefert genau dieses Item') +def _(ctx): + """ + Mit Raute ist die Absicht eindeutig - dann darf kein Treffer dabei sein, + der die Zahl nur im Text stehen hat. + """ + r = ctx.client.get('/api/bugtracker/v1/manage', + query={'action': 'list', 'project_slug': PROJEKT, + 'search': '#%s' % ctx['bug_id'], 'limit': 50}) + erwarte_erfolg(r) + ids = [i.get('id') for i in _liste(r, 'items', 'bugs', 'data') if isinstance(i, dict)] + erwarte(ids == [ctx['bug_id']], + "Erwartet genau [%s], bekommen %s" % (ctx['bug_id'], ids)) + + +@test('bugtracker', 'Suche nach einer fremden Nummer findet nichts') +def _(ctx): + r = ctx.client.get('/api/bugtracker/v1/manage', + query={'action': 'list', 'project_slug': PROJEKT, + 'search': '#999999', 'limit': 50}) + erwarte_erfolg(r) + ids = [i.get('id') for i in _liste(r, 'items', 'bugs', 'data') if isinstance(i, dict)] + erwarte(ids == [], "Unerwartete Treffer: %s" % ids[:10]) + + @test('bugtracker', 'Einzelabruf liefert das Item') def _(ctx): r = ctx.client.get('/api/bugtracker/v1/manage',