New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix `var_export` return in `WP_CLI::print_value` #4926

Merged
merged 1 commit into from Aug 25, 2018

Conversation

2 participants
@soulseekah
Copy link
Contributor

soulseekah commented Aug 24, 2018

By default var_export will dump its output to stdout. Set the return
parameter to true to save it into the $value variable for further
processing as expected.

Fix `var_export` return in `WP_CLI::print_value`
By default `var_export` will dump its output to stdout. Set the return
parameter to true to save it into the `$value` variable for further
processing as expected.
@schlessera

This comment has been minimized.

Copy link
Member

schlessera commented Aug 25, 2018

Thanks for the pull-request, @soulseekah !

@schlessera schlessera added this to the 2.1.0 milestone Aug 25, 2018

@schlessera schlessera merged commit 2d27c16 into wp-cli:master Aug 25, 2018

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment