While trying to create a taxonomy term, I ran into an issue with a wrong bundle check that let it fail.

CommentFileSizeAuthor
#3 restws_create_fix.patch1.08 KBfago
restws_create_fix.patch1.09 KBfago
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

fago’s picture

fago
Primary language German
Location Vienna
CreditAttribution: fago commented
Status: Active » Needs review

Status: Needs review » Needs work

The last submitted patch, restws_create_fix.patch, failed testing.

fago’s picture

fago
Primary language German
Location Vienna
CreditAttribution: fago commented
Status: Needs work » Needs review
FileSize
restws_create_fix.patch1.08 KB

good test case :-)
Looks like the entity api function initaliztes the bundle to an empty string, so we've to check it using empty(). Updated patch attached.

klausi’s picture

klausi
he/him||they/them
Primary language German
Location 🇦🇹 Vienna
CreditAttribution: klausi commented
Status: Needs review » Needs work

Would be cool to get that taxonomy term creation as test case?

klausi’s picture

klausi
he/him||they/them
Primary language German
Location 🇦🇹 Vienna
CreditAttribution: klausi commented
Issue tags: +D7 stable release blocker

Tagging