fix(status-bar): 'Running query…' stays after a successful SELECT (plain row-count status line)
メンテナーはふだん 1 日以内に返信
まだ誰も着手していません。
評価
- 難易度
- 2/5
- 見積もり時間
- 1〜3時間
- 初心者へのやさしさ
- 79/100
調査の方向性
まず QueryaShellStatus.reportQueryResult と、処理中メッセージの動作に関するユニットテストを読みます。issueにはメソッド名と失敗する手順が記載されています。次に GenericSqlWorkspace の結果公開経路を調べ、単純な行数メッセージが null になる仕組みを確認します。単純な行数の結果で処理中メッセージが消え、スクリプトまたは影響を受けた行のステータステキストが表示されたままになり、指定されたユニットテストが通れば完了です。
索引モデルが issue の本文から書いたものです。
説明
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.
- 主要言語
- Dart
- スター
- 23
- フォーク
- 1
- 平均マージ
- 31分
- マージ済み PR(30日)
- 570
環境構築
このプロジェクトの開発コンテナを、あなたの GitHub アカウントでブラウザ上に起動します。
- Dockerfile・Docker Compose ファイルなし
- プルリクエストのテンプレートあり
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
QueryaHub/Querya-Desktop のほかの issue
-
enhancement ui
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
QueryaHub/Querya-Desktop#1689 ·
メンテナーはふだん 1 日以内に返信
-
ui(dialogs): make destructive query and DML preview dialogs adaptive and prevent table name overflowオープンbug frontend
難易度 2/5 1〜3時間 初心者へのやさしさ 82/100
QueryaHub/Querya-Desktop#1642 ·
メンテナーはふだん 1 日以内に返信
-
bug error-handling settings
難易度 1/5 1〜3時間 初心者へのやさしさ 78/100
QueryaHub/Querya-Desktop#1637 ·
メンテナーはふだん 1 日以内に返信
-
enhancement
難易度 1/5 1時間未満 初心者へのやさしさ 88/100
QueryaHub/Querya-Desktop#1636 ·
メンテナーはふだん 1 日以内に返信
-
enhancement marketplace
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
QueryaHub/Querya-Desktop#1635 ·
メンテナーはふだん 1 日以内に返信
QueryaHub/Querya-Desktop の issue をすべて見る
似ている issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 67/100
-
bug good first issue
難易度 2/5 1〜3時間 初心者へのやさしさ 82/100
basictech01/uniun#300 ·
メンテナーはふだん 1 日以内に返信
-
Listener that removes itself in its callback causes other listeners to miss updates対応中かも @RaphaelFakhri が今日担当しました。 オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 84/100
RevenueCat/purchases-flutter#1962 ·
メンテナーはふだん 1 日以内に返信
-
bug
難易度 2/5 1〜3時間 初心者へのやさしさ 67/100
メンテナーはふだん 5 日以内に返信
-
bug
難易度 2/5 1〜3時間 初心者へのやさしさ 70/100
jhomlala/betterplayer#1609 ·
メンテナーはふだん 1 日以内に返信