I wrote a test to test the glossary view for drupal7, but on drupal6 the test doesn't pass. Then i looked at the records, and wow, they weren't correct.

killes suggested to add SUM(1) instead of node.nid and wow this worked. I'm not sure whether this is the way to go.

There is added the patch and the test. I suggest to run first the test, to see that it is broken.

Comments

dawehner’s picture

Status: Needs review » Needs work

update status

The count(node.nid) is missing

dawehner’s picture

StatusFileSize
new495 bytes

There are really things which are easy to fix *sighs*

dawehner’s picture

Status: Needs work » Needs review

update status

dawehner’s picture

Status: Needs review » Fixed

i commited the patch, i hops thats ok

dawehner’s picture

Version: 6.x-3.x-dev » 7.x-3.x-dev
Status: Fixed » Patch (to be ported)

update :)

dawehner’s picture

Status: Patch (to be ported) » Fixed

fixed

Status: Fixed » Closed (fixed)

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