`wp theme mod set` prints its failure message as `Success` and exits 0
Évaluation
- Difficulté
- 2/5
- Temps estimé
- 1-3 heures
- Accessibilité débutants
- 82/100
Piste de recherche
Lisez Theme_Mod_Command::set(), où, selon l’issue, la valeur qui ne correspond pas est détectée et le message d’échec est affiché. Vérifiez comment les échecs des commandes voisines signalent les erreurs et définissent le code de sortie ; reproduisez le cas avec le filtre décrit pre_set_theme_mod_background_color. Le travail est terminé lorsque la commande affiche Error: Could not update theme mod background_color. et se termine avec le code 1.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
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.
- Langage dominant
- Gherkin
- Étoiles
- 90
- Forks
- 87
- Merge moyen
- 2 j 7 h
- PR mergées (30 j)
- 3
Préparer son environnement
- Aucun Dockerfile ni fichier Docker Compose
- Propose un modèle de pull request
- Lire le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de wp-cli/extension-command
-
Download update packages concurrently before running the bulk upgraderPeut-être pris @austinginder l’a pris il y a 49 jours. Ouvertecommand:plugin-update command:theme-update
Difficulté 4/5 3-5 jours Accessibilité débutants 48/100
wp-cli/extension-command#555 · 2 commentaires ·
-
opcache invalidationPeut-être pris @swissspidy l’a pris il y a 265 jours. Ouvertecommand:plugin-update command:theme-update
wp-cli/extension-command#444 · 10 commentaires · 1 réaction · 2 personnes assignées ·
-
Add wp theme cache commandPeut-être pris @swissspidy l’a pris il y a 277 jours. Ouvertecommand:theme state:unconfirmed
wp-cli/extension-command#392 · 9 commentaires · 1 réaction · 2 personnes assignées ·
Toutes les issues de wp-cli/extension-command
Issues similaires
-
csp-probe needs efqdn to probe CSP domainsPeut-être pris @unknownhackerworld l’a pris il y a 1 jour. Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100
projectdiscovery/httpx#2725 ·
Les mainteneurs répondent en général sous 1 jour
-
enhancement good first issue
Difficulté 2/5 1-3 heures Accessibilité débutants 84/100
-
Difficulté 2/5 1-3 heures Accessibilité débutants 82/100
ktwu01/benchmark-radar#1167 ·
Les mainteneurs répondent en général sous 1 jour
-
linkml-validate --config without positional data files validates nothing and reports successOuverte
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
Les mainteneurs répondent en général sous 2 jours
-
domain/usage p3
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100