I think this is just a typo, but caused me a lot of worry until I realised what was going on -

In multifield_get_subfields(), $subfields[$result->bundle] is emptied as the results are looped over. As a consequence, only the last of multiple subfields is ever loaded for a bundle.

CommentFileSizeAuthor
#3 2190469-multifield-get-subfields-fixes.patch8.01 KBdave reid
#1 multifield-multiple_subfields_not_loaded-2190469-1.patch764 bytesben young

Comments

ben young’s picture

ben young commented
StatusFileSize
new multifield-multiple_subfields_not_loaded-2190469-1.patch764 bytes

Patch attached

drakmor’s picture

drakmor commented

Confirm. Thx for patch.

dave reid’s picture

dave reid
he/him
Primary language English
Location Nebraska USA
commented
Status: Active » Needs review
StatusFileSize
new 2190469-multifield-get-subfields-fixes.patch8.01 KB

Thanks for pointing this out! I noticed a couple of other small fixes, and ensured that the test coverage would account for this.

dave reid’s picture

dave reid
he/him
Primary language English
Location Nebraska USA
commented
Version: 7.x-1.0-unstable7 » 7.x-1.x-dev
dave reid’s picture

dave reid
he/him
Primary language English
Location Nebraska USA
commented
Status: Needs review » Fixed

Committed #3 to 7.x-1.x. Thanks everyone!
http://drupalcode.org/project/multifield.git/commit/33366da

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.