hopefully the screenshot is good enough for an explanation

CommentFileSizeAuthor
#6 quickfix.patch401 bytesPaulMagrath
#2 sameerror.png27.69 KBtemp
views-grouping-table-row-even-assignment.gif2.86 KBgiorgosk

Comments

temp’s picture

temp commented

i have same error,
and this look like same error
http://drupal.org/node/622602

temp’s picture

temp commented
StatusFileSize
new sameerror.png27.69 KB
giorgosk’s picture

giorgosk
Location Chios, Greece 🇬🇷
commented

@temp
one screenshot should suffice its easy enough to understand

it might sound similar but patch from #622602: 2.6 to 2.7: Queries being created without ORDER BY does not solve this problem

PaulMagrath’s picture

PaulMagrath commented

I am also seeing this bug in Views 2.7 when grouping is applied.

PaulMagrath’s picture

PaulMagrath commented

Problem seems to be in theme/theme.inc in the template_preprocess_views_view_table() function as the $row_classes array is definitely confused about odd and even rows by the time it is passed to views-view-table.tpl.php.

PaulMagrath’s picture

PaulMagrath commented
Version: 6.x-2.7 » 6.x-2.x-dev
Status: Active » Needs review
StatusFileSize
new quickfix.patch401 bytes

Attached is a quick fix that works for me. This patch should be applied against a checkout of theme.inc from the 2.x branch in the CVS repository.

PaulMagrath’s picture

PaulMagrath commented

Patch above needs review. If you are experiencing this issue, please try the above patch and then comment here.

giorgosk’s picture

giorgosk
Location Chios, Greece 🇬🇷
commented
Status: Needs review » Reviewed & tested by the community

Patch has to be run from within views/theme

It applies cleanly and works as expected
Tested that it also does not affect the row odd/even assignment when no grouping is used and all works fine !!

its a trivial change and I think its safe to mark it as "reviewed and tested by the community"

temp’s picture

temp commented

i don't have errors with it too.

merlinofchaos’s picture

merlinofchaos commented
Status: Reviewed & tested by the community » Fixed

Committed to all branches. Note I changed $n to $count because I abhor one letter variable names.

Status: Fixed » Closed (fixed)

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