`wp theme mod set` prints its failure message as `Success` and exits 0
Bewertung
- Schwierigkeit
- 2/5
- Geschätzter Aufwand
- 1-3 Stunden
- Anfängerfreundlichkeit
- 82/100
Rechercherichtung
Lies Theme_Mod_Command::set(), wo laut Issue der nicht übereinstimmende Wert erkannt und die Fehlermeldung ausgegeben wird. Prüfe, wie benachbarte Befehlsfehler Fehler melden und den Exit-Status setzen; reproduziere das Verhalten mit dem beschriebenen Filter pre_set_theme_mod_background_color. Fertig ist es, wenn der Befehl Error: Could not update theme mod background_color. ausgibt und mit Code 1 beendet wird.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Beschreibung
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.
- Vorherrschende Sprache
- Gherkin
- Sterne
- 90
- Forks
- 87
- Ø Merge
- 2 T. 7 Std.
- Gemergte PRs (30 T.)
- 3
Entwicklungsumgebung
- Kein Dockerfile und keine Docker-Compose-Datei
- Hat eine Pull-Request-Vorlage
- Beitragsleitfaden lesen
Erste Schritte
- Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
- Forken Sie das Repository und arbeiten Sie in einem Branch.
- Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.
Mehr aus wp-cli/extension-command
-
Download update packages concurrently before running the bulk upgraderEvtl. vergeben @austinginder hat das vor 49 Tagen übernommen. Offencommand:plugin-update command:theme-update
Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 48/100
wp-cli/extension-command#555 · 2 Kommentare ·
-
opcache invalidationEvtl. vergeben @swissspidy hat das vor 265 Tagen übernommen. Offencommand:plugin-update command:theme-update
wp-cli/extension-command#444 · 10 Kommentare · 1 Reaktion · 2 zugewiesene Personen ·
-
Add wp theme cache commandEvtl. vergeben @swissspidy hat das vor 277 Tagen übernommen. Offencommand:theme state:unconfirmed
wp-cli/extension-command#392 · 9 Kommentare · 1 Reaktion · 2 zugewiesene Personen ·
Alle Issues in wp-cli/extension-command
Ähnliche Issues
-
bug good first issue priority: low
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 78/100
TMHSDigital/Free-Game-Dev-Assets#250 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 80/100
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 76/100
fish-shell/fish-shell#13084 · 1 Kommentar ·
Maintainer antworten meist innerhalb von 2 Tagen
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 78/100
http-party/http-server#982 ·
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 86/100
hardikverma22/sandfit#20 ·
Maintainer antworten meist innerhalb von 1 Tag