fix(status-bar): 'Running query…' stays after a successful SELECT (plain row-count status line)
Les mainteneurs répondent en général sous 1 jour
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 2/5
- Temps estimé
- 1-3 heures
- Accessibilité débutants
- 79/100
Piste de recherche
Commencez par lire QueryaShellStatus.reportQueryResult et les tests unitaires du comportement du message d’activité ; l’issue nomme la méthode et fournit la séquence qui échoue. Examinez ensuite le chemin de publication des résultats de GenericSqlWorkspace pour confirmer comment un simple message de nombre de lignes devient null. Le travail est terminé lorsque les résultats avec un simple nombre de lignes effacent le message d’activité, que le texte d’état du script ou des lignes affectées reste visible et que le test unitaire spécifié réussit.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
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.
- Langage dominant
- Dart
- Étoiles
- 23
- Forks
- 1
- Merge moyen
- 31 min
- PR mergées (30 j)
- 570
Préparer son environnement
Lance le conteneur de développement du projet dans votre navigateur, avec votre propre compte GitHub.
- Aucun Dockerfile ni fichier Docker Compose
- Propose un modèle de pull request
- Lire le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de QueryaHub/Querya-Desktop
-
enhancement ui
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
QueryaHub/Querya-Desktop#1689 ·
Les mainteneurs répondent en général sous 1 jour
-
ui(dialogs): make destructive query and DML preview dialogs adaptive and prevent table name overflowOuvertebug frontend
Difficulté 2/5 1-3 heures Accessibilité débutants 82/100
QueryaHub/Querya-Desktop#1642 ·
Les mainteneurs répondent en général sous 1 jour
-
bug error-handling settings
Difficulté 1/5 1-3 heures Accessibilité débutants 78/100
QueryaHub/Querya-Desktop#1637 ·
Les mainteneurs répondent en général sous 1 jour
-
enhancement
Difficulté 1/5 Moins d'une heure Accessibilité débutants 88/100
QueryaHub/Querya-Desktop#1636 ·
Les mainteneurs répondent en général sous 1 jour
-
enhancement marketplace
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
QueryaHub/Querya-Desktop#1635 ·
Les mainteneurs répondent en général sous 1 jour
Toutes les issues de QueryaHub/Querya-Desktop
Issues similaires
-
[Bug] Web Guide has broken linksOuverte
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100
AppFlowy-IO/AppFlowy#9075 ·
-
Paste asks for iOS paste permission because the default context menu ignores SystemContextMenuPeut-être pris @dorklein l’a pris aujourd’hui. Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100
maheshj01/searchfield#267 ·
-
Difficulté 2/5 1-3 heures Accessibilité débutants 67/100
-
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100
avdept/JellyBoxPlayer#375 ·
Les mainteneurs répondent en général sous 1 jour
-
bug good first issue
Difficulté 2/5 1-3 heures Accessibilité débutants 82/100
basictech01/uniun#300 ·
Les mainteneurs répondent en général sous 1 jour