fix(status-bar): 'Running query…' stays after a successful SELECT (plain row-count status line)
Mantenedores costumam responder em até 1 dia
Ninguém assumiu esta issue ainda.
Avaliação
- Dificuldade
- 2/5
- Tempo estimado
- 1-3 horas
- Facilidade para iniciantes
- 79/100
Direção de pesquisa
Comece lendo QueryaShellStatus.reportQueryResult e os testes unitários do comportamento da mensagem de atividade; a issue nomeia o método e fornece a sequência que falha. Em seguida, inspecione o caminho de publicação dos resultados de GenericSqlWorkspace para confirmar como uma mensagem simples de contagem de linhas se torna null. O trabalho estará concluído quando resultados com contagem simples de linhas apagarem a mensagem de atividade, o texto de status do script ou das linhas afetadas continuar visível e o teste unitário especificado passar.
Escrita pelo modelo de indexação a partir do texto da issue.
Descrição
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.
- Linguagem predominante
- Dart
- Estrelas
- 23
- Forks
- 1
- Merge médio
- 31min
- PRs com merge (30d)
- 570
Preparar o ambiente
Inicia o contêiner de desenvolvimento do projeto no navegador, com a sua própria conta do GitHub.
- Sem Dockerfile nem arquivo Docker Compose
- Tem um modelo de pull request
- Ler o guia de contribuição
Primeiros passos
- Leia a issue inteira e depois o guia de contribuição do projeto.
- Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
- Faça um fork do repositório e trabalhe em uma branch.
- Abra um pull request que referencie o número da issue.
Mais de QueryaHub/Querya-Desktop
-
enhancement ui
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 78/100
QueryaHub/Querya-Desktop#1689 ·
Mantenedores costumam responder em até 1 dia
-
ui(dialogs): make destructive query and DML preview dialogs adaptive and prevent table name overflowAbertabug frontend
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 82/100
QueryaHub/Querya-Desktop#1642 ·
Mantenedores costumam responder em até 1 dia
-
bug error-handling settings
Dificuldade 1/5 1-3 horas Facilidade para iniciantes 78/100
QueryaHub/Querya-Desktop#1637 ·
Mantenedores costumam responder em até 1 dia
-
enhancement
Dificuldade 1/5 Menos de uma hora Facilidade para iniciantes 88/100
QueryaHub/Querya-Desktop#1636 ·
Mantenedores costumam responder em até 1 dia
-
enhancement marketplace
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 78/100
QueryaHub/Querya-Desktop#1635 ·
Mantenedores costumam responder em até 1 dia
Todas as issues de QueryaHub/Querya-Desktop
Issues semelhantes
-
bug
Dificuldade 1/5 Menos de uma hora Facilidade para iniciantes 65/100
chriskelly/wish-together#36 ·
Mantenedores costumam responder em até 1 dia
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 72/100
AppFlowy-IO/AppFlowy#9075 ·
-
Paste asks for iOS paste permission because the default context menu ignores SystemContextMenuTalvez já em andamento @dorklein assumiu hoje. Aberta
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 72/100
maheshj01/searchfield#267 ·
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 67/100
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 72/100
hajisensai/Fushi#2340 · 2 comentários ·
Mantenedores costumam responder em até 1 dia