Skip to content
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

Add support for setting terms in batch #7

Open
bobbingwide opened this issue Mar 9, 2017 · 1 comment
Open

Add support for setting terms in batch #7

bobbingwide opened this issue Mar 9, 2017 · 1 comment
Assignees
Labels

Comments

@bobbingwide
Copy link
Owner

The logic to create letter taxonomy terms should work when the code is being run in batch.
This is needed when parsing the source code in oik-shortcodes for WP-a2z.org and oik-plugins.

Expected output

Letter taxonomy terms set for posts created or updated in oik-batch

Actual output

Letter taxonomy terms were not being set.

Explanation

The lazy logic was not being loaded since $_REQUEST['tax_input'] was not set.

Solution

Check for Command Line Interface.
In the lazy loaded code cater for $_REQUEST['tax_input'] not being set.

@bobbingwide
Copy link
Owner Author

It appears that the check for command line interface works better if you call php_sapi_name() rather than test the value of the PHP_SAPI constant. See bobbingwide/oik-batch/issues/19

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant