"disable scripts" in script manager needs update for 5.6
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Aptitud para principiantes
- 52/100
Línea de trabajo
Comienza en tools/script_manager.lua, en el callback del botón disable_scripts; después inspecciona la preferencia para deshabilitar los scripts de Lua y la ruta de carga de luarc incluida en darktable 5.6. Reproduce el comportamiento con los scripts incluidos habilitados y determina si se debe eliminar el botón o hacer que alterne la preferencia. Se considera terminado cuando deshabilitar los scripts mediante el módulo de scripts impide realmente que se carguen los scripts incluidos, y el camino para volver a habilitarlos sigue siendo claro.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Description
The disable scripts button in the scripts module renames <config_dir>/luarc to luarc.disabled (the disable_scripts button callback in tools/script_manager.lua). This worked before the scripts were bundled, when darktable used <config_dir>/luarc as the Lua entry point.
Since 5.6, darktable loads the bundled luarc, which calls require "tools/script_manager" regardless of config_dir/luarc existing or not. Renaming config_dir/luarc no longer disables the scripts. It only prevents the user's own config_dir/luarc from being re-run. The bundled scripts still load and start.
Steps to reproduce
use darktable 5.6 with bundled scripts.
scripts module / action: install/update scripts / enable "disable scripts" button / click disable scripts.
observe luarc being renamed in the config dir
Restart darktable.
Expected: the Lua scripts are disabled.
Observed: bundled scripts still load and run.
Possible fixes
The current way to disable the scripts is the disable Lua scripts preference.
a) have the button toggle the disable_scripts preference (with the checkbox as the re-enable path)
b) remove the button in favour of the preference
I'd favour the latter - then enabling/disabling lua scripts is reduced to the preferences.
- Lenguaje dominante
- Lua
- Estrellas
- 219
- Forks
- 142
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Guía de contribución
No hay ninguna guía de contribución indexada para este repositorio
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 darktable-org/lua-scripts
-
dtutils.string: build_substitute_list errors when a metadata field (e.g. rights) has been removed Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
darktable-org/lua-scripts#716 · 2 comentarios ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 63/100
darktable-org/lua-scripts#706 ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
darktable-org/lua-scripts#627 · 2 comentarios ·
-
bug
Dificultad 1/5 Menos de una hora Aptitud para principiantes 65/100
darktable-org/lua-scripts#618 · 3 comentarios ·
-
Dificultad 3/5 1-2 días Aptitud para principiantes 52/100
darktable-org/lua-scripts#710 · 3 comentarios ·
Todos los issues de darktable-org/lua-scripts
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
LandSandBoat/server#11579 ·
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 90/100
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
ArchiveTeam/sinavideo-grab#7 · 3 reacciones ·
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
mailcow/mailcow-dockerized#7480 ·