Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
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.
Comment | File | Size | Author |
---|---|---|---|
#24 | ctools-readd_access_callback_params-2209775-24.patch | 967 bytes | maximpodorov |
#14 | ctools-undefined_access_callback-2209775-14.patch | 922 bytes | reevo |
#10 | ctools-undefined_access_callback-2209775-10.patch | 1.24 KB | timaholt |
#3 | undefined_access_callback-2209775-3.patch | 984 bytes | lex0r |
Comments
Comment #1
blackclover CreditAttribution: blackclover commentedUpdated panels, views, entity and file entity to the latest dev versions and that fixed it. Sorry not sure which one was the culprit. Cheers.
Comment #2
lex0r CreditAttribution: lex0r commentedCorrect 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!
Comment #3
lex0r CreditAttribution: lex0r commentedThe patch for those of us who doesn't dare to use dev in production.
Comment #4
JayDarnellI hate to reopen an issue but the patch in #3 above did not remove these error messages for me...
Comment #5
JayDarnellDrupal 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.
Comment #6
JayDarnellComment #8
DSquaredB CreditAttribution: DSquaredB commentedI have installed the latest dev version of ctools and am still getting this error message. Is there a fix that works?
Comment #9
sprite CreditAttribution: sprite commentedI am also experiencing the following error message in ctools with:
Ctools 7.x-1.4
Panels 7.x-3.4
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.
Comment #10
timaholt CreditAttribution: timaholt commentedManually applying this patch worked for me but the patch itself doesn't apply cleanly to the latest head. I've re-rolled the patch.
Comment #11
timaholt CreditAttribution: timaholt commentedComment #12
timaholt CreditAttribution: timaholt commentedComment #14
reevo CreditAttribution: reevo commentedPatch in #10 fails to apply. Attaching a reformatted version.
Comment #15
reevo CreditAttribution: reevo commentedComment #17
dkane CreditAttribution: dkane commentedPatch in #14 worked for me. Thanks!
Comment #18
segi CreditAttribution: segi commentedI 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.
Comment #19
Anonymous (not verified) CreditAttribution: Anonymous commentedPatch #14 worked for me also. Thank you!
Comment #21
japerryWow what a check. Looks good :-)
Comment #22
capfive CreditAttribution: capfive commentedPatch #14 worked for me also, will upgrade to next version when it is out :) I "wouldn't dare upgrade to dev" :)
Comment #23
maximpodorov CreditAttribution: maximpodorov commentedThe 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).
Comment #24
maximpodorov CreditAttribution: maximpodorov commentedThe patch fixes the problem described in #23.
Comment #25
Eric115 CreditAttribution: Eric115 at PreviousNext commentedPatch in #24 works as intended.
Comment #26
Eric115 CreditAttribution: Eric115 at PreviousNext commentedMarking this issue as a child issue to Plan for CTools 7.x-1.11 release.
Comment #27
maximpodorov CreditAttribution: maximpodorov commentedI think new release with urgent fixes should follow. See https://www.drupal.org/node/2195211#comment-10247879 - the current release broke already fixed things.
Comment #28
adrien.felipe CreditAttribution: adrien.felipe commentedExact 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
Comment #29
mglaman+1 this break Drupal Commerce + Panels.
Comment #30
Eric_A CreditAttribution: Eric_A commentedComment #31
Eric_A CreditAttribution: Eric_A commentedComment #32
AnybodyConfirming the problem and the working patch. A fix is urgently required. Thanks a lot for your hard work!
Comment #33
NickDickinsonWildeWorks and be ideal to get this into the release stream ASAP.
Comment #34
dsnopekWe're now using this patch in Panopoly.
Comment #35
maxplus CreditAttribution: maxplus commentedThanks,
patch #24 solved my issue (coming from issue https://www.drupal.org/node/2195211)
Comment #36
torgosPizzaPatch 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!
Comment #37
Xen CreditAttribution: Xen at Reload commentedConfirmed 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.
Comment #38
dobe CreditAttribution: dobe as a volunteer commented#24 worked for me.
Comment #39
sketman CreditAttribution: sketman commented#24 worked for me too, thanks.
Comment #40
MustangGB CreditAttribution: MustangGB commentedComment #42
japerryPatch 24 it is then! Looks good, committed.
Comment #43
maximpodorov CreditAttribution: maximpodorov commentedThank you.