As with the other functions in a PHP object, all of them must have the same signature.

From what I can see, you have to have a $group_by parameter in the query() function so one of them has the ability to avoid the use of the GROUP BY option in one situation.

In any event, this patch shows all the query() functions that need to be updated in order to fix Views 3.0.

Thank you.
Alexis Wilke

CommentFileSizeAuthor
#3 view-3.x-query_group_by.patch43.64 KBAlexisWilke
view-3.x-query_group_by.patch45.34 KBAlexisWilke

Comments

Status: Needs review » Needs work

The last submitted patch, view-3.x-query_group_by.patch, failed testing.

AlexisWilke’s picture

AlexisWilke commented

Reference: #653628: Declaration of views_handler_filter_many_to_one::init() should be compatible with that of views_handler_filter::init()

AlexisWilke’s picture

AlexisWilke commented
StatusFileSize
new view-3.x-query_group_by.patch43.64 KB

Massaged the patch so it would at least apply for testing.

AlexisWilke’s picture

AlexisWilke commented
Status: Needs work » Needs review

Change status 8-P

Status: Needs review » Needs work

The last submitted patch, view-3.x-query_group_by.patch, failed testing.

AlexisWilke’s picture

AlexisWilke commented
Status: Needs work » Active

Marking as active as there are no tests for this version and needs work may not be looked at.

chris matthews’s picture

chris matthews commented
Issue summary: View changes
Status: Active » Closed (outdated)

The Drupal 6 branch is no longer supported, please check with the D6LTS project if you need further support. For more information as to why this issue was closed, please see issue #3030347: Plan to clean process issue queue