Fatal error when using aggregation and a field is not attached to all entities in the view

Oh wow this totally makes sense, things can be quite easy somtimes.
First i thought this is related to http://drupal.org/node/1417520 but it's not.

+++ b/modules/field/views_handler_field_field.incundefined
@@ -770,7 +770,7 @@ class views_handler_field_field extends views_handler_field {
+    if (empty($values->_field_data[$this->field_alias]) || empty($values->_field_data[$this->field_alias]['entity']) || !isset($values->_field_data[$this->field_alias]['entity']->{$this->definition['field_name']})) {

A comment would be really helpful here. What kind of "edge" cases can appear. At least for the one you added it's actual a real logical and reproducable reason.

This fatal error is still happening.

Same here: For me it happens when I try to aggregate on fields which do not have value.

I've been struggling with this about a year ago and eventually managed to work around this bug by adding default value of 0 (zero) to all fields participating in the view.

Now however I cannot do this due to complexity with 'Field Collections' and other content, so I would really need help with fixing this.

Thanks,

Pimmy

I narrowed it down to a view that always gave the following ajax pop up error:

An AJAX HTTP error occurred. HTTP Result Code: 200 Debugging information follows.
Path: xxxx/admini/structure/views/view/general/preview/ block/ajax
StatusText: OK
ResponseText: Fatal error: Cannot access empty property in "path to module \ modules\ fielthfield.attach.inc on line 325"

So I removed and redid the view and it solved the problem.

patch makes the error disappear, but aggregation still fails to work for columns containing empty results.
See example: http://drupal.org/node/1451894

redoing the view makes no difference.

Same problem with a commerce-kickstart installation.
Glad to know a little more about the problem now.

For me it happens when I try to aggregate on fields which do not have value.

Seems to be the same here. Need to review this.

I have this error disabled aggregation in views (D7.19)

"Fatal error: Cannot access empty property in field.attach.inc on line 325"

and that when I refresh other site or add content:

Notice: Undefined index: w _field_invoke_multiple() (linia 310 z ... /modules/field/field.attach.inc).

patch not help :/

These errors pop up when printing field and displays the node does not contain the field.
This code in the page.tpl.php makes these errors:

$g_field_name = field_get_items('node', $node, 'field_name');
$v_field_name = field_view_value('node', $node, 'field_name', $g_field_name[ 0 ]);

print render($v_field_name);

but this will avoid page--node-type-with-field_name.tpl.php

sorry for my English;)

Not that it is specific to this module, but back to David_Rothstein's point. This is not a bug inherent to views, we experience this on a production site with the commerce module when trying to pass information to another entity that is using the same field as an existing field (another field instance).

I am also getting the same issue, redoing the view didn't fix it for me either.

Has anyone had any joy?

Thanks
Gem

I worked out what my issue was!

I didn't set the filter criteria to the content type so it was bringing back all content types. Obviously this is going to cause issues because it's bring back text as well as numbers...

So I just disabled Aggregation added in the filter criteria to filter only that content type, enabled aggregation and it worked.

Hopefully this will help someone else.

Gem

@David_Rothstein : Your path works well and solves this issue. Great work David_Rothstein!

Based on #12 and my own experience I am closing this issue.

@David_Rothstein : Yeah I forgot that and closed this ticket, but thanks again for your work!

Worked for me. Thanks!

@David_Rothstein : Worked for me. Thanks!

Thanks @David_Rothstein

Patch in original pors worked for me.

Problem occurred on MERCI module https://drupal.org/project/MERCI White Screen of Dead ( WSOD )
Error in apache log:
PHP Fatal error: Cannot access empty property in /var/www/[DOMAINNAME]/html/modules/field/field.attach.inc on line 324

Drupal core: 7.23
Views: 7.x-3.7

I really wonder why this patch isn't still committed in the Views module.

Added a line of documentation for myself and committed the initial patch.

Drupal 8 looks way different so this don't have to be ported.

This is not a Views bug. It's a bug in _field_invoke_get_instances() that doesn't check if the field exists in the bundle, so the results is sometimes array(NULL), which is obviously incorrect. Adding an array_filter() fixes the problem, in the right place, not in Views.

return array_filter($instances);

but that's core, so that's never gonna change...

Oh wow I completely missed that. My bad. I created a core bug that I'll now close.

I have the same error, but I can't solve it with David_Rothstein patch.
Any other solution?
I'm using Drupal 7.21