`wp theme mod set` prints its failure message as `Success` and exits 0
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 82/100
Línea de trabajo
Lee Theme_Mod_Command::set(), donde, según el issue, se detecta el valor que no coincide y se imprime el mensaje de error. Comprueba cómo los errores de comandos cercanos informan de los errores y establecen el estado de salida; reproduce el caso con el filtro descrito pre_set_theme_mod_background_color. Se considera terminado cuando el comando imprime Error: Could not update theme mod background_color. y termina con el código 1.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
When wp theme mod set cannot save the value, it prints the failure through WP_CLI::success():
$ wp theme mod set background_color 123456
Success: Could not update theme mod background_color.
$ echo $?
0
To reproduce, add a pre_set_theme_mod_background_color filter that returns a different value (for example from a mu-plugin). Theme_Mod_Command::set() reads the mod back, sees it does not match, and calls WP_CLI::success( "Could not update theme mod {$mod}." ). Scripts that check the exit code treat the failure as a success.
Expected: Error: Could not update theme mod background_color. with exit code 1.
- Lenguaje dominante
- Gherkin
- Estrellas
- 90
- Forks
- 87
- Merge medio
- 2 d 7 h
- PR fusionados (30 d)
- 3
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Tiene una 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 wp-cli/extension-command
-
Download update packages concurrently before running the bulk upgraderPosiblemente ocupada @austinginder la tomó hace 49 días. Abiertocommand:plugin-update command:theme-update
Dificultad 4/5 3-5 días Aptitud para principiantes 48/100
wp-cli/extension-command#555 · 2 comentarios ·
-
opcache invalidationPosiblemente ocupada @swissspidy la tomó hace 265 días. Abiertocommand:plugin-update command:theme-update
wp-cli/extension-command#444 · 10 comentarios · 1 reacción · 2 asignados ·
-
Add wp theme cache commandPosiblemente ocupada @swissspidy la tomó hace 277 días. Abiertocommand:theme state:unconfirmed
wp-cli/extension-command#392 · 9 comentarios · 1 reacción · 2 asignados ·
Todos los issues de wp-cli/extension-command
Issues similares
-
csp-probe needs efqdn to probe CSP domainsPosiblemente ocupada @unknownhackerworld la tomó hace 1 día. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
projectdiscovery/httpx#2725 ·
Los mantenedores suelen responder en 1 día
-
enhancement good first issue
Dificultad 2/5 1-3 horas Aptitud para principiantes 84/100
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
ktwu01/benchmark-radar#1167 ·
Los mantenedores suelen responder en 1 día
-
linkml-validate --config without positional data files validates nothing and reports successAbierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
Los mantenedores suelen responder en 2 días
-
domain/usage p3
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100