Drupal 7.26
Ctools 7.14
Applied patches:
cttols-n2195471-34.patch
ctools-n2195211-entity-from-field-access-callback-15.patch

Getting this error before and after the above patches were applied.
Notice: Undefined index: access callback in ctools_entity_from_field_context() (line 187 of /home/edelitst/public_html/sites/all/modules/ctools/plugins/relationships/entity_from_field.inc).

Thank you for any help.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

blackclover’s picture

Status: Active » Closed (fixed)

Updated panels, views, entity and file entity to the latest dev versions and that fixed it. Sorry not sure which one was the culprit. Cheers.

lex0r’s picture

Status: Closed (fixed) » Fixed

Correct me if I'm wrong but you can't set "Closed (fixed)" status on a ticket with the buggy version unless it is fixed in that version? You should have used "Fixed" and close it only with a new release.
Sincerely,
your boring!

lex0r’s picture

The patch for those of us who doesn't dare to use dev in production.

JayDarnell’s picture

I hate to reopen an issue but the patch in #3 above did not remove these error messages for me...

JayDarnell’s picture

Status: Fixed » Active

Drupal 7.26
Ctools 7.x-1.4
Panels 7.x-3.4

I'm not savvy enough to know what the cause of this is but the patch above did not fix the problem. Please let me know if there are other bits of information I can provide to help reproduce this.

JayDarnell’s picture

Status: Active » Fixed

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.

DSquaredB’s picture

Status: Closed (fixed) » Active

I have installed the latest dev version of ctools and am still getting this error message. Is there a fix that works?

sprite’s picture

I am also experiencing the following error message in ctools with:

Ctools 7.x-1.4
Panels 7.x-3.4

Undefined index: access callback in ctools_entity_from_field_context() (line 180 of /home/sprite/public_html/sites/all/modules/ctools/plugins/relationships/entity_from_field.inc).

The problem appears after setting a panel context to a node and then referencing an - image - field from the contextual node on the content of the panel.

When I temporarily reverted to Ctools 7.x-1.3 to check if the problem was in that module and not some other, the problem disappeared.

The problem also persists in the latest - 7.x-1.x-dev - 2014-Aug-22 - development version of the ctools module.

timaholt’s picture

Manually applying this patch worked for me but the patch itself doesn't apply cleanly to the latest head. I've re-rolled the patch.

timaholt’s picture

Status: Active » Needs review
timaholt’s picture

Version: 7.x-1.4 » 7.x-1.x-dev

Status: Needs review » Needs work

The last submitted patch, 10: ctools-undefined_access_callback-2209775-10.patch, failed testing.

reevo’s picture

Patch in #10 fails to apply. Attaching a reformatted version.

reevo’s picture

Status: Needs work » Needs review

dkane’s picture

Patch in #14 worked for me. Thanks!

segi’s picture

Status: Needs review » Reviewed & tested by the community

I tested the patch in #14 with ctools 7.x-1.7 and It seems to me the patch solved the problem. I tried to apply for current dev version and It was smooth so, I take it to to RTBC.

Anonymous’s picture

Patch #14 worked for me also. Thank you!

  • japerry committed 24d01a3 on authored by reevo
    Issue #2209775 by timaholt, reevo, lex0r: Undefined index: access...
japerry’s picture

Status: Reviewed & tested by the community » Fixed

Wow what a check. Looks good :-)

capfive’s picture

Patch #14 worked for me also, will upgrade to next version when it is out :) I "wouldn't dare upgrade to dev" :)

maximpodorov’s picture

Status: Fixed » Active

The last commit breaks already fixed issue #2195211: Warning after update: Missing argument 4 for entity_metadata_taxonomy_access (by reverting the part of the fix).

maximpodorov’s picture

Status: Active » Needs review
FileSize
967 bytes

The patch fixes the problem described in #23.

Eric115’s picture

Status: Needs review » Reviewed & tested by the community

Patch in #24 works as intended.

Eric115’s picture

Marking this issue as a child issue to Plan for CTools 7.x-1.11 release.

maximpodorov’s picture

I think new release with urgent fixes should follow. See https://www.drupal.org/node/2195211#comment-10247879 - the current release broke already fixed things.

adrien.felipe’s picture

Exact same issue here. Was generating the patch but @davidwhthomas did it already :)
For some reason 2 arguments from line 188 , $account, $to_entity)) { keep being deleted from the code, while actually leaving the variables unused on the previous code.

The patch from https://www.drupal.org/node/2195211#comment-10247879 fixes the issue.
I also vote RTBC

mglaman’s picture

+1 this break Drupal Commerce + Panels.

Eric_A’s picture

Eric_A’s picture

Anybody’s picture

Confirming the problem and the working patch. A fix is urgently required. Thanks a lot for your hard work!

NickDickinsonWilde’s picture

Works and be ideal to get this into the release stream ASAP.

dsnopek’s picture

Issue tags: +panopoly

We're now using this patch in Panopoly.

maxplus’s picture

Thanks,

patch #24 solved my issue (coming from issue https://www.drupal.org/node/2195211)

torgosPizza’s picture

Patch in #24 also worked for me, coming from a Drupal Commerce issue #2608754: commerce_entity_access incorrect access logic [fixed in Ctools 1.x-dev]. Would be nice to have this committed!

Xen’s picture

Confirmed that it removes the warning from https://www.drupal.org/node/2195211 .

Inspecting the patches, it's clear that #10 is a mis-reroll that cuts of the arguments added since the original patch was written.

#24 adds the missing arguments.

dobe’s picture

#24 worked for me.

sketman’s picture

#24 worked for me too, thanks.

MustangGB’s picture

  • japerry committed 1f2d43f on 7.x-1.x authored by maximpodorov
    Issue #2209775 by timaholt, maximpodorov, reevo, lex0r: Undefined index...
japerry’s picture

Status: Reviewed & tested by the community » Fixed

Patch 24 it is then! Looks good, committed.

maximpodorov’s picture

Thank you.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.