Remove the dead tree 'loadfail' handler in browser.js, which calls a non-existent tree.isSelected()
I maintainer di solito rispondono entro 1 giorno
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 2/5
- Tempo stimato
- 1-3 ore
- Idoneità per principianti
- 72/100
- Tipo di issue
- Refactoring
- Chiarezza
- Abbastanza chiara
- Stato di attività
- Attiva
- Stack tecnologico
- javascript
- Ambito
- frontend
Direzione di ricerca
Inizia da web/pgadmin/browser/static/js/browser.js: trova la funzione onLoadFailNode (intorno alle righe 1265-1278) e la sua registrazione su pgadmin-browser:tree:loadfail (intorno alla riga 198). Verifica in web/pgadmin/static/js/tree/tree.js che non esista alcun metodo isSelected e conferma che nulla emetta loadfail. Il lavoro è completo quando il gestore inutilizzato e la sua registrazione vengono rimossi, oppure riscritti con self.tree.selected() se la logica della master password viene mantenuta, e il browser si carica senza errori.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
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).
- Lingua principale
- Python
- Stelle
- 3.9k
- Fork
- 908
- Merge medio
- 1g 6h
- PR unite (30g)
- 38
Preparare l'ambiente
- Include un Dockerfile o un file Docker Compose
- Nessun modello di pull request
- Leggi la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di pgadmin-org/pgadmin4
-
Feature
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
pgadmin-org/pgadmin4#10426 · 3 commenti · 1 reazione ·
I maintainer di solito rispondono entro 1 giorno
-
Bug
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
pgadmin-org/pgadmin4#10361 ·
I maintainer di solito rispondono entro 1 giorno
-
Bug
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
pgadmin-org/pgadmin4#10272 · 5 commenti ·
I maintainer di solito rispondono entro 1 giorno
-
Feature
Difficoltà 2/5 1-3 ore Idoneità per principianti 76/100
pgadmin-org/pgadmin4#10157 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
-
Feature
Difficoltà 3/5 1-2 giorni Idoneità per principianti 45/100
pgadmin-org/pgadmin4#10615 ·
I maintainer di solito rispondono entro 1 giorno
Tutte le issue di pgadmin-org/pgadmin4
Issue simili
-
Questions crashes on a real SQuAD2 no-answer predictionForse già presa @HuaTNA l’ha presa oggi. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
I maintainer di solito rispondono entro 1 giorno
-
GST is read as Gulf Standard Time (UTC+4) instead of Guam Standard Time (UTC+10)Forse già presa @yang00333 l’ha presa oggi. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
DOI-USGS/dataretrieval-python#454 ·
I maintainer di solito rispondono entro 4 giorni
-
area:core-platform effort:S priority:P3 status:in-review type:test
Difficoltà 2/5 Meno di un'ora Idoneità per principianti 80/100
randomparity/kdive#3246 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
-
Requantification library converts m/z to mass with the hydrogen-atom mass instead of the proton massApertabug
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 85/100
nf-core/metaboigniter#130 ·
I maintainer di solito rispondono entro 1 giorno
-
netbox status: needs owner type: documentation
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 75/100
netbox-community/netbox#23412 · 1 reazione ·
I maintainer di solito rispondono entro 1 giorno