Update taxonomy term query methods to utilize latest API changes

[akmur]

Hello,
great job on this package. I am trying to find a way to filter out a category from my posts collection, is it feasible? I tried using something like -1 (where 1 is the id of the category), similarly to what is done in WordPress custom loops, but it won't work.

Any idea is appreciated, thanks!

[kadamwhite]

Hey @akmur, thanks for the question. I just returned from holiday and I'm getting things back in order, I'll dig in to what would be necessary to accomplish what you want in the next couple days; apologies for the delay.

[akmur]

Actually I just retried and it would seem like it does work, I don't know why it was not working before, probably some mistake of mine.

I was able to get this to work

wp.posts().category(-67).then(function( data ) { res.render('posts', { posts: data }); });

So I'd say it's already implemented, maybe you could just doublecheck it :)

Thanks a lot

[kadamwhite]

@akmur after digging into this, there's a lot of inconsistency over how the cat filter parameter is handled by the API; and internally, we weren't always accurate about how we tracked method inputs. if your test somehow ended up trying to pass category() a string, like '-67' instead of -67, it probably would not have worked. I've got a branch that takes care of that, and intend to raise the overall category inconsistency question with the API leads.

[kadamwhite]

Since it sounds like you're sorted, I'm taking over this issue to account for updating the taxonomy filtering logic to reflect the latest and greatest in the REST API betas

[akmur]

Yes, I think i'm kind of sorted :) Many thanks!