Remove the dead tree 'loadfail' handler in browser.js, which calls a non-existent tree.isSelected()
Los mantenedores suelen responder en 1 día
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 72/100
- Tipo de issue
- Refactorización
- Claridad
- Bastante claro
- Estado de actividad
- Activo
- Stack tecnológico
- javascript
- Área
- frontend
Línea de trabajo
Empieza en web/pgadmin/browser/static/js/browser.js: localiza la función onLoadFailNode (alrededor de las líneas 1265-1278) y su registro en pgadmin-browser:tree:loadfail (alrededor de la línea 198). Comprueba en web/pgadmin/static/js/tree/tree.js que no existe ningún método isSelected y confirma que nada emite loadfail. Se considera terminado cuando el manejador muerto y su registro se eliminan, o se reescriben con self.tree.selected() si se conserva la lógica de la contraseña maestra, y el navegador se carga sin errores.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
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).
- Lenguaje dominante
- Python
- Estrellas
- 3.9k
- Forks
- 908
- Merge medio
- 1 d 6 h
- PR fusionados (30 d)
- 38
Preparar el entorno
- Incluye un Dockerfile o un archivo de Docker Compose
- Sin plantilla de pull request
- Leer la guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de pgadmin-org/pgadmin4
-
Feature
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
pgadmin-org/pgadmin4#10426 · 3 comentarios · 1 reacción ·
Los mantenedores suelen responder en 1 día
-
Bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
pgadmin-org/pgadmin4#10361 ·
Los mantenedores suelen responder en 1 día
-
Feature
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
pgadmin-org/pgadmin4#10272 · 5 comentarios ·
Los mantenedores suelen responder en 1 día
-
Feature
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
pgadmin-org/pgadmin4#10157 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
Feature
Dificultad 3/5 1-2 días Aptitud para principiantes 45/100
pgadmin-org/pgadmin4#10615 · 2 comentarios ·
Los mantenedores suelen responder en 1 día
Todos los issues de pgadmin-org/pgadmin4
Issues similares
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 88/100
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
aws-samples/sample-per-model-bedrock#261 ·
Los mantenedores suelen responder en 1 día
-
[UI/Accessibility] Low contrast text visibility in country selection dropdown on settings pageAbierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 65/100
pwncollege/dojo#1188 ·
Los mantenedores suelen responder en 1 día
-
bug launcher packaging
Dificultad 1/5 Menos de una hora Aptitud para principiantes 88/100
tenstorrent/tt-studio#1426 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
python-attrs/cattrs#796 ·
Los mantenedores suelen responder en 1 día