Notice: Undefined index: consent_storage_callback in eu_cookie_compliance_store_consent()

Hi!

Thanks for the report. I'll take a look.

It would help speed up the debugging if you could run drush vget eu_cookie_compliance and paste the result here.

Thanks. I'll take a look.

Do you by any chance have a multilingual web site? I can't reproduce directly.

I also have this problem

Notice: Undefined index: consent_storage_callback in eu_cookie_compliance_store_consent() (Zeile 858 von /****/modules/eu_cookie_compliance/eu_cookie_compliance.module).
I am runnig a multilingual website.

Same here, multilingual site and the same notice.

Ok let's have a look...

  // Get plugin.
  $consent_storage_plugin = ctools_get_plugins('eu_cookie_compliance', 'consent_storage', $consent_storage_method);
  $consent_storage_function = $consent_storage_plugin['consent_storage_callback'];
  $result = $consent_storage_function($type);

ctools_get_plugins('eu_cookie_compliance', 'consent_storage', $consent_storage_method);

doesn't return a proper value (method name). The reason might be outside, for example it would be interesting to hear which ctools versions the users are using here and what's returned instead.

Anyway I'd suggest to check by function_exists($consent_storage_function) that the given function exists and log error details if not. Furthermore I'm a bit afraid if it's secure to call a function returned by variable without limitations and prior checks?

Hello,

just a quick note:

it happened to one of our sites when upgrading from the version 7.x-1.31 of the module to the 1.32.

The prior istallation didn't have the consent_storage_method variable thus it was set to null.

The 1.32 configuration screen offers the choice with a radio button:
- Do not store (selected by default)
- basic storage

It appears that if the user doesn't intentionally save the configuration for *every* language of a multi language site, the variable is null so throwing the error when a user clicks on accept on the banner.

Note: I didn't inspect the code, I just reproduced the error. I set the variable first on a single language: no more errors in the log for users landing on that language, more errors for users landing on the other language (you can check this by manually going on the home page of a language on a new incognito/anonyous browser window). When I saved the the configuration on the second language, all errors disappeared.

Workaround: re-save the configuration after every upgrade for every language installed.

Here's a patch that should resolve the issue.

Test notes:
- Install the module first and then set up a Drupal 7 site with several languages.
- Do *not* save the configuration for the other language, leaving the variable for Consent Storage Method unconfigured.
- Visit the site on the other language and verify that the error doesn't happen.

Committed to 7.x dev branch. Thank you.