Repository navigation
wp theme list show multiple status #224
Description
I am creating an audit script and noticed that you cannot use more than one status with the wp theme list command.
Here is an example on a live site:
wp theme list
Output
+----------------+--------+--------+---------+
| name | status | update | version |
+----------------+--------+--------+---------+
| woodmart-child | active | none | 1.0.0 |
| woodmart | parent | none | 4.4.0 |
+----------------+--------+--------+---------+
I can filter by one status
wp theme list --status=active
Output
+----------------+--------+--------+---------+
| name | status | update | version |
+----------------+--------+--------+---------+
| woodmart-child | active | none | 1.0.0 |
+----------------+--------+--------+---------+
But if I try to get both the active and parent status in one command it fails
wp theme list --status=active,parent
Output
+------+--------+--------+---------+
| name | status | update | version |
+------+--------+--------+---------+
+------+--------+--------+---------+
Activity
swissspidy commentedon Nov 10, 2023
Sounds like it might be looking for a status called
active,parentand not actually doing any splitting to filter by both.So this would be more of an enhancement. I'd be open to accept a PR that covers all relevant commands and also clearly documents the behavior. Would also be good to check other commands as well to ensure they all behave similarly.
thelovekesh commentedon Nov 10, 2023
@swissspidy IMO It's hard to guess whether the user wants to pass comma-separated strings as multiple strings or the string itself contains the comma. In this case, we can explore the strategy to pass a flag multiple times in case multiple values are intended for a flag.
Example:
danielbachhuber commentedon Nov 11, 2023
Related #159
test: add comma-separated --status test for wp theme list (closes #224)
Allow multiple values for field filter options in `wp plugin list` an…