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

fix for taxonomy and select #178

Merged
merged 2 commits into from
Dec 5, 2012
Merged

fix for taxonomy and select #178

merged 2 commits into from
Dec 5, 2012

Conversation

rolandinsh
Copy link
Contributor

No description provided.

@funkatron82
Copy link
Contributor

meta is already typecasted into an array in the meta function for each field.

@rolandinsh
Copy link
Contributor Author

with debug on ( posted also http://wordpress.org/support/topic/fix-for-taxonomyphp-on-line-160-and-selectphp-on-line-43?replies=1 )

[28-Nov-2012 11:24:22] PHP Warning: in_array() [<a href='function.in-array'>function.in-array</a>]: Wrong datatype for second argument in /var/.../wp-content/plugins/meta-box/inc/fields/select.php on line 43 [28-Nov-2012 11:24:22] PHP Warning: in_array() [<a href='function.in-array'>function.in-array</a>]: Wrong datatype for second argument in /var/.../wp-content/plugins/meta-box/inc/fields/taxonomy.php on line 160

@rilwis
Copy link
Member

rilwis commented Nov 28, 2012

@rolandinsh Thanks for your pull request. I have same thought as @funkedgeek. Can you please give an example when it happens?

@rolandinsh
Copy link
Contributor Author

copy-paste from /demo/ script for new webpage, with some customization and since select and taxonomy did not work, just turned on WP_DEBUG, and there it was (even without my customization).

Maybe incompitable with some plugin:

  • Yoast's SEO or GA,
  • Jetpack,
  • tinymce advanced

btw, using old method of including

https://gist.github.com/4161822

@rolandinsh
Copy link
Contributor Author

ok. look like this was/is just incompatibility of old and new meta-box code
PS I extended with "post type", not only taxonomy

rilwis added a commit that referenced this pull request Dec 5, 2012
fix for taxonomy and select
@rilwis rilwis merged commit 976867c into wpmetabox:master Dec 5, 2012
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants