I have Fatal error "EntityMalformedException - Propriété de bundle manquante sur l'entité de type node." on custom pane preview, if I'm using a "Entity: Field Value" access plugin

It's because $context->data is null, i don't have node context and pane is custom (not node view)

CommentFileSizeAuthor
#3 ctools-7.x-1.3-fix-fatal-while-preview-1903866-3.patch873 bytesbpresles
#1 ctools-7.x-1.2-fix-fatal-while-preview-1903866-1.patch1.03 KBagalitsyn

Comments

agalitsyn’s picture

agalitsyn commented
Status: Active » Needs review
StatusFileSize
new ctools-7.x-1.2-fix-fatal-while-preview-1903866-1.patch1.03 KB

Here is patch

Status: Needs review » Needs work

The last submitted patch, ctools-7.x-1.2-fix-fatal-while-preview-1903866-1.patch, failed testing.

bpresles’s picture

bpresles commented
Version: 7.x-1.2 » 7.x-1.3
Status: Needs work » Needs review
StatusFileSize
new ctools-7.x-1.3-fix-fatal-while-preview-1903866-3.patch873 bytes

I generated a new patch that should pass validation

maximpodorov’s picture

maximpodorov commented

Here #1630820: entity_field_value is completely broken is the patch which fixes Entity field value plugin.

tripper54’s picture

tripper54 commented
Issue summary: View changes
Status: Needs review » Reviewed & tested by the community

Patch in #3 applies cleanly and fixes the problem. Thanks!

jm_drupal’s picture

jm_drupal commented

Patch in #3 fixes my problem too. I was getting "EntityMalformedException" error when i try to access any views 'page display' only. "Views block display" was working ok. This patch in #3 fixes my problem. Thanks.

nandt queued 3: ctools-7.x-1.3-fix-fatal-while-preview-1903866-3.patch for re-testing.

japerry’s picture

japerry
Location KOMK
commented
Status: Reviewed & tested by the community » Closed (duplicate)

Marking as duplicate, as this is fixed in #1630820

Specifically in the diff: 

function ctools_entity_field_value_ctools_access_check($conf, $context, $plugin) {
  if (!isset($context->data)) {
  if ((!is_object($context)) || (empty($context->data))) {

is now there before this function in #3 which means we don't need to perform this check.