fix(status-bar): 'Running query…' stays after a successful SELECT (plain row-count status line)
Maintainer antworten meist innerhalb von 1 Tag
Dieses Issue hat noch niemand übernommen.
Bewertung
- Schwierigkeit
- 2/5
- Geschätzter Aufwand
- 1-3 Stunden
- Anfängerfreundlichkeit
- 79/100
Rechercherichtung
Lies zuerst QueryaShellStatus.reportQueryResult und die Unit-Tests für das Verhalten der Meldung bei laufender Abfrage; im Issue werden die Methode genannt und die fehlschlagende Abfolge angegeben. Untersuche dann den Ergebnisveröffentlichungspfad von GenericSqlWorkspace, um zu bestätigen, wie eine einfache Zeilenanzahlmeldung zu null wird. Fertig ist die Änderung, wenn Ergebnisse mit einfacher Zeilenanzahl die Meldung bei laufender Abfrage löschen, Statustext zu Skripten oder betroffenen Zeilen sichtbar bleibt und der angegebene Unit-Test besteht.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Beschreibung
Problem
After a successful SELECT the status bar keeps showing "Running query…" until the next query. It is stuck, not just late: seen 4 s after the grid had been shown.
Cause
GenericSqlWorkspace publishes the result with message: _shellMessage(session.statusLine), and _shellMessage deliberately returns null for a plain "N row(s)." status line (the counts are already on the right of the bar). QueryaShellStatus.reportQueryResult then does:
if (message != null) _statusMessage = message; // null: keeps the old "Running query…"
_messageIsBusy = false; // ...and now it is no longer dropped by endBusy()
So the busy message set by beginBusy survives and is re-labelled as a normal message.
Reproduction
Demo Playground -> SQL -> SELECT * FROM orders LIMIT 200; -> F5. The grid shows 12 rows, the bar shows Running query…, 1 ms, 12 rows, 8 cols.
Fix
In reportQueryResult, take the message as given (_statusMessage = message;), or drop the busy message when message == null (if (_messageIsBusy) _statusMessage = null before the assignment). Add a unit test: beginBusy(message: 'Running query…') then reportQueryResult(message: null) leaves statusMessage == null.
Acceptance
- Plain-count results leave no message in the bar.
- Results with a script / affected-rows status still show their text.
- Unit test for the sequence above.
Related: #714 (live status-bar metrics), #1481.
- Vorherrschende Sprache
- Dart
- Sterne
- 23
- Forks
- 1
- Ø Merge
- 32 Min.
- Gemergte PRs (30 T.)
- 537
Entwicklungsumgebung
Startet den Dev-Container des Projekts im Browser, mit Ihrem eigenen GitHub-Konto.
- Kein Dockerfile und keine Docker-Compose-Datei
- Hat eine Pull-Request-Vorlage
- Beitragsleitfaden lesen
Erste Schritte
- Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
- Forken Sie das Repository und arbeiten Sie in einem Branch.
- Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.
Mehr aus QueryaHub/Querya-Desktop
-
enhancement ui
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 78/100
QueryaHub/Querya-Desktop#1689 ·
Maintainer antworten meist innerhalb von 1 Tag
-
ui(dialogs): make destructive query and DML preview dialogs adaptive and prevent table name overflowOffenbug frontend
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 82/100
QueryaHub/Querya-Desktop#1642 ·
Maintainer antworten meist innerhalb von 1 Tag
-
bug error-handling settings
Schwierigkeit 1/5 1-3 Stunden Anfängerfreundlichkeit 78/100
QueryaHub/Querya-Desktop#1637 ·
Maintainer antworten meist innerhalb von 1 Tag
-
enhancement
Schwierigkeit 1/5 Unter einer Stunde Anfängerfreundlichkeit 88/100
QueryaHub/Querya-Desktop#1636 ·
Maintainer antworten meist innerhalb von 1 Tag
-
enhancement marketplace
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 78/100
QueryaHub/Querya-Desktop#1635 ·
Maintainer antworten meist innerhalb von 1 Tag
Alle Issues in QueryaHub/Querya-Desktop
Ähnliche Issues
-
bug
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 70/100
jhomlala/betterplayer#1609 ·
Maintainer antworten meist innerhalb von 1 Tag
-
[FR] 缓存键放置在外面Offenenhancement
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 72/100
bggRGjQaUbCoE/PiliPlus#3310 ·
Maintainer antworten meist innerhalb von 1 Tag
-
package:code_builder type-bug
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 90/100
Maintainer antworten meist innerhalb von 1 Tag
-
type:feature
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 68/100
Maintainer antworten meist innerhalb von 1 Tag
-
[material_ui] CalendarDatePicker day numbers are cut off with large text scale on narrow screensOffen
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 78/100
Maintainer antworten meist innerhalb von 1 Tag