When using views field grouping, the tpl.php gets called once for each group and the row index continues to increment, but in template_preprocess_views_flipped_table() the row index starts from 0 each time the function gets called. This patch uses the original field key when setting the flipped rows to prevent undefined offset errors in views-flipped-table.tpl.php at lines 38 and 53.

CommentFileSizeAuthor
#6 flipping_rows_undefined_index-2277369-6.patch536 bytesproofoftom
#1 flipping_rows_undefined_index-2277369-1.patch536 bytesproofoftom

Comments

proofoftom’s picture

proofoftom commented
StatusFileSize
new flipping_rows_undefined_index-2277369-1.patch536 bytes

Patch for 7.x-1.x-dev

proofoftom’s picture

proofoftom commented
Status: Active » Needs review
proofoftom’s picture

proofoftom commented

Marked #2279151: PHP undefined offset errors in views-flipped-table.tpl.php at lines 38 and 53 as duplicate of this issue. It explains the symptoms of this issue.

proofoftom’s picture

proofoftom commented
Title: Undefined Index: Flipping the rows currently causes row index to get lost » Undefined offset errors in views-flipped-table.tpl.php at lines 38 and 53
Issue summary: View changes
andileco’s picture

andileco commented

If you are modifying version 7.x-1.0, you will want to modify lines 25 and 27.

proofoftom’s picture

proofoftom commented
StatusFileSize
new flipping_rows_undefined_index-2277369-6.patch536 bytes

Patch for 7.x-1.0

naught101’s picture

naught101 commented

Thanks Tommy Cox, patch in commit 0ce83f1. Should be out in 7.x-1.x soon.

naught101’s picture

naught101 commented
Status: Needs review » Fixed

  • Commit 0ce83f1 on 7.x-1.x authored by Tommy Cox: 
    Issue #2277369 by Tommy Cox: Fixed Undefined offset errors in views-...

Status: Fixed » Closed (fixed)

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