BUG: `ExecutablePluginV2` is not set up correctly and FL uses `ExecutablePlugin` (v1) instead

Cerrado Apto para principiantes
#4,697 0 comentarios 0 reacciones 0 asignados Ver en GitHub

Los mantenedores suelen responder en 4 días

Ya se ha fusionado un pull request relacionado.

  • #4710 de @DavidGBrett — fusionado

Evaluación

Dificultad
1/5
Tiempo estimado
Menos de una hora
Aptitud para principiantes
90/100
Tipo de issue
Error
Claridad
Bien especificado
Estado de actividad
Activo
Stack tecnológico
csharp
Área
backend

Línea de trabajo

Abre Flow.Launcher.Core/Plugin/PluginsLoader.cs alrededor de la línea 144 e inspecciona la función enlazada en el issue, que construye un ExecutablePlugin cuando lo que se pretende es ExecutablePluginV2; el issue también cita como referencia el commit que rompe la compatibilidad b50db58. Compáralo con la forma en que el código del loader circundante instancia otros tipos de plugins V2, haz la única sustitución y confirma con alguna de las pruebas existentes de carga de plugins o compilando la solution.

Escrito por el modelo de indexación a partir del texto del issue.

Descripción

bug
Checks
  • I have checked that this issue has not already been reported.

  • I am using the latest version of Flow Launcher.

  • I am using the prerelease version of Flow Launcher.

Problem Description

Wrong code: https://github.com/Flow-Launcher/Flow.Launcher/blob/dev/Flow.Launcher.Core/Plugin/PluginsLoader.cs#L144-L156

In the linked function it should be ExecutablePluginV2 but instead is ExecutablePlugin (v1).

I guess it was broken here: https://github.com/Flow-Launcher/Flow.Launcher/commit/b50db58673d19756dda1de4d8d453f8c5e535504#diff-ae0e091b837a327253f4d3191e6a4b770e0115738ec5d2eed73b5011ebfd54edR166

Confirmed by jjw24 on discord here: https://discord.com/channels/727828229250875472/727828229250875476/1557145225921826939

To Reproduce
  1. check the code i linked above
Screenshots

No response

Flow Launcher Version

2.1.4

Windows Build Number

irrelevant

Error Log

irrelevant

Lenguaje dominante
C#
Estrellas
16k
Forks
673
Merge medio
3 d 2 h
PR fusionados (30 d)
8

Preparar el entorno

Este proyecto no incluye contenedor de desarrollo, Dockerfile ni guía de contribución, así que la configuración corre por tu cuenta: empieza por su README y consulta nuestra guía para la primera contribución para los pasos generales.

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Más de Flow-Launcher/Flow.Launcher

Todos los issues de Flow-Launcher/Flow.Launcher

Issues similares

Más issues de C#

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.