I get this error message since updating from -beta3 to -dev.

CommentFileSizeAuthor
#5 1403908-fix.patch649 bytesdawehner
#4 1403908-fix.patch649 bytesdawehner

Comments

sea4’s picture

sea4 commented

Hi, i am getting the same error since updating to the latest dev release.

pjcdawkins’s picture

pjcdawkins commented

Obviously a fire-fighting fix would be to add this before line 204 of views_plugin_display.inc:

        if (!isset($filter['group'])) {
          $filter['group'] = 0;
        }

but that doesn't address the cause (merely relieves watchdog a bit), and I don't understand what the element $filter['group'] is or does.

sea4’s picture

sea4 commented

@pjcdawkins: thanks .that would help for now. I too am not sure what the group element does.

dawehner’s picture

dawehner
Primary language German
commented
Version: 7.x-3.x-dev » 6.x-3.x-dev
Status: Active » Patch (to be ported)
StatusFileSize
new 1403908-fix.patch649 bytes

Even you can't believe it but 0 is not the right default value here ...
Here is the patch which got committed to 7.x-3.x

Some backport would be cool here as well.

dawehner’s picture

dawehner
Primary language German
commented
StatusFileSize
new 1403908-fix.patch649 bytes

Here is the patch, now for real :)

jackalope’s picture

jackalope commented

@dereine Thanks for that patch, it seems to be working for me against 7.x-3.x-dev!

klonos’s picture

klonos
he/him
Primary language English
Location 90% Melbourne, Australia - 10% Larissa, Greece
commented

Yep, #5 makes the error go away for me too in latest 7.x-3.x-dev. Thanx ;)

...RTBC then(?)

pjcdawkins’s picture

pjcdawkins commented

Of course it makes the error go away, it forcibly defines the otherwise undefined index... are you all sure that $filter['group'] shouldn't exist before this section? There should be at least some kind of inline comment explaining it.

chris matthews’s picture

chris matthews commented
Issue summary: View changes
Status: Patch (to be ported) » 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