Remove the dead tree 'loadfail' handler in browser.js, which calls a non-existent tree.isSelected()
メンテナーはふだん 1 日以内に返信
まだ誰も着手していません。
評価
- 難易度
- 2/5
- 見積もり時間
- 1〜3時間
- 初心者へのやさしさ
- 72/100
- issue の種類
- リファクタリング
- 明瞭さ
- おおむね明確
- 活発さ
- 活発
- 技術スタック
- javascript
- 領域
- frontend
調査の方向性
web/pgadmin/browser/static/js/browser.js から始めます。onLoadFailNode 関数（1265〜1278 行付近）と、pgadmin-browser:tree:loadfail への登録（198 行付近）を見つけてください。web/pgadmin/static/js/tree/tree.js に isSelected メソッドが存在しないことを確認し、loadfail を発行しているものがないことを確かめてください。完了の条件は、使われていないハンドラーとその登録を削除するか、マスターパスワードのロジックを残す場合は self.tree.selected() を使って書き直すことであり、ブラウザがエラーなく読み込まれることです。
索引モデルが issue の本文から書いたものです。
説明
onLoadFailNode in web/pgadmin/browser/static/js/browser.js (around lines 1265-1278) calls self.tree.isSelected(_nodeData), but the current Tree class (web/pgadmin/static/js/tree/tree.js) has no isSelected method, so it would throw a TypeError if it ever ran.
It is never reached in practice: the handler is registered for pgadmin-browser:tree:loadfail (browser.js around line 198), and nothing emits a loadfail event since the move away from aciTree; load failures are now handled in tree_nodes.ts. There is no user-visible effect today, so this is a clean-up: remove the handler and its registration (or, if the master-password handling in it is still wanted, rewrite it using self.tree.selected() and emit the event from the load failure path).
- 主要言語
- Python
- スター
- 3.9k
- フォーク
- 908
- 平均マージ
- 1日 6時間
- マージ済み PR（30日）
- 38
環境構築
- Dockerfile または Docker Compose ファイルあり
- プルリクエストのテンプレートなし
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
pgadmin-org/pgadmin4 のほかの issue
-
Feature
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
pgadmin-org/pgadmin4#10426 · コメント 3 件 · リアクション 1 件 ·
メンテナーはふだん 1 日以内に返信
-
Bug
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
pgadmin-org/pgadmin4#10361 ·
メンテナーはふだん 1 日以内に返信
-
Feature
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
pgadmin-org/pgadmin4#10272 · コメント 5 件 ·
メンテナーはふだん 1 日以内に返信
-
Feature
難易度 2/5 1〜3時間 初心者へのやさしさ 76/100
pgadmin-org/pgadmin4#10157 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
Feature
難易度 3/5 1〜2日 初心者へのやさしさ 45/100
pgadmin-org/pgadmin4#10615 · コメント 2 件 ·
メンテナーはふだん 1 日以内に返信
pgadmin-org/pgadmin4 の issue をすべて見る
似ている issue
-
bot-triaged module: LrScheduler module: optimizer triaged
難易度 2/5 1〜3時間 初心者へのやさしさ 70/100
メンテナーはふだん 1 日以内に返信
-
documentation glm5 llm-model
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
vllm-project/vllm-ascend#19062 ·
メンテナーはふだん 1 日以内に返信
-
author-page correction
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
acl-org/acl-anthology#10496 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
Issue Type: Bug Report
難易度 2/5 1〜3時間 初心者へのやさしさ 82/100
celery/celery#10875 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
enhancement good first issue
難易度 2/5 1〜3時間 初心者へのやさしさ 76/100
SasanLabs/VulnerableApp#836 ·
メンテナーはふだん 1 日以内に返信