Skip to content
This repository was archived by the owner on Sep 24, 2018. It is now read-only.

Check taxonomies exist before trying to set properties #1354

Merged
merged 3 commits into from
Jul 23, 2015

Conversation

joehoyle
Copy link
Member

In the case where a develop has unregistered the post_tag / category taxonomy,
it currently produces PHP Notices.

In the case where a develop has unregistered the post_tag / category
taxonomy, it currently procudes PHP Notices.
@joehoyle joehoyle added this to the 2.0 Beta 3 milestone Jun 16, 2015
@joehoyle
Copy link
Member Author

@WP-API/amigos #reviewmerge

@danielbachhuber
Copy link
Member

In the case where a develop has unregistered the post_tag / category taxonomy,

I'll refrain from commenting about this ;)

@danielbachhuber
Copy link
Member

@WP-API/amigos anyone know why codecov fails?

@rmccue
Copy link
Member

rmccue commented Jun 17, 2015

@danielbachhuber Because there's no specific testing for this code, so it's not marked as covered. (codecov/patch is the coverage for just the changed lines)

@rmccue rmccue added Testing and removed Review labels Jun 17, 2015
@joehoyle joehoyle added Review and removed Testing labels Jul 12, 2015
@joehoyle
Copy link
Member Author

@WP-API/amigos #reviewmerge plz

rmccue added a commit that referenced this pull request Jul 23, 2015
Check taxonomies exist before trying to set properties
@rmccue rmccue merged commit 2cdbfbe into develop Jul 23, 2015
@rmccue rmccue deleted the taxonomies-not-registered branch July 23, 2015 10:56
@rmccue rmccue modified the milestones: 2.0 Beta 4, 2.0 Beta 3 Jul 23, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants