matrix_field_formatter_view assumes items in correct order, which isn't always the case. If items are not in the correct order you'll get rows or columns missing from the output.

I've also identified a couple of issues with the logic which I've highlighted but don't have time to fix at the moment.

See attached patch.

CommentFileSizeAuthor
#1 matrix-2198389-1.patch1.22 KBintrafusion
matrix_field_formatter_view.patch1.34 KBAngry Dan
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

intrafusion’s picture

intrafusion
Primary language English
Location Edinburgh, UK
CreditAttribution: intrafusion commented
FileSize
matrix-2198389-1.patch1.22 KB

Thanks for your assistance, your patch has been re-rolled as it no longer applied cleanly.

intrafusion’s picture

intrafusion
Primary language English
Location Edinburgh, UK
CreditAttribution: intrafusion commented
Status: Needs review » Fixed

Patch OK, added to repo

  • intrafusion committed 2492f27 on 
    Issue #2198389 by Angry Dan, intrafusion: matrix_field_formatter_view...

Status: Fixed » Closed (fixed)

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