This issue occured for me in version 7.x-3.6 it also exists in the newest version. I get a notice because the value that I want to show does not exist in the data array of all users.

I think it would be better if the render function checks if the value isset.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Johnny vd Laar’s picture

Status: Active » Needs review
FileSize
820 bytes

Here is a patch.

Status: Needs review » Needs work

The last submitted patch, views-serialized-check-value-isset.2120243-1.patch, failed testing.

Johnny vd Laar’s picture

Status: Needs work » Needs review
FileSize
692 bytes

oh

ericclaeren’s picture

Mr Johnny, I have a improvement for the patch, you checked a string and not the keys in the serialized array, so the problem still exists.

I have attached an updated patch on 7.x-3.7

ericclaeren’s picture

Anonymous’s picture

Patch #5 had no fallback if the if statement fails.

I've created a patch which checks the keys and returns a plain version or an empty string (which in case can be handled with the "no results behaviour" option).

francois o’s picture

Running into this issue: #6 works for me

ioanmar’s picture

#6 seems to do the job for me too.

Chris Matthews’s picture

Version: 7.x-3.6 » 7.x-3.x-dev

The 5 year old patch in #6 to views_handler_field_serialized.inc applied cleanly to the latest views 7.x-3.x-dev