I have installed wthis amazing module. However, I am having some issues when I want to show the popup message.
I have to languages, spanish and English. The default language is Spanish. The admin user has english language in the language settings. The rest of the users have Spanish language like default language.

I have configured the module and the permissions to show the popup to every role.

The popup only comes up for the admin user. The rest of the users can't see the popup.

I have checked the code and the css and js files are only loaded for the admin role.

Can anyone know how to solve this issue?

Thanks heaps
Javier

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Miszel’s picture

If you drop me a message/email with a your website url (with this module enabled) I can have a look. Otherwise I won't be able replicate this issue.

Miszel’s picture

Category: Bug report » Support request
javier.drupal.2012’s picture

Hi Marcin, is there some news about this issue?

Thanks
Javier

svenryen’s picture

Hi Javier!

I will fix your issue in the next version. I've taken on the co-maintainer seat of this module and I've run into the same issue.

The workaround is to set the Admin language to Spanish, then save your settings (including enabling the popup) and then switch back to English as admin language. If you look closely at the admin screen, you will see that you're editing the English version of the popup while the admin language is in English, and there's no option for accessing the Spanish version currently (plan to change this).

If you open up a clean browser window and clear your cookies, you should see the popup after following this workaround.

Cheers!

gargsuchi’s picture

Has this been addressed in the dev version yet?

svenryen’s picture

@gargsuchi, currently, you can edit one language at a time, so you'll need to follow the instructions in the workaround. Adding this requires a major rewrite of the admin interface, so I'm considering working on this in a v2.0 branch. Please try to follow the workaround for now :)

gargsuchi’s picture

Thanks for the update @svenryen

BAHbKA’s picture

FileSize
6.63 KB

Here is my workaround for multilanguage settings form.

BAHbKA’s picture

Status: Active » Needs review
BAHbKA’s picture

FileSize
7.63 KB

There was an issue with saving settings in previous patch.

luketsimmons’s picture

Status: Needs review » Reviewed & tested by the community

Good work @BAHbKA, it's very convenient you've done this since you posted this patch only a few hours ago! :-)

I've not reviewed the exact code for the patch, but I have applied it and tested the expected outcomes. With the patch I have been able to configure and display the patch for two sites where we have English for admin but French set as default for the frontend. I've had a few people test this to see if there are any obvious issues but all good so far.

Quick skim of the code in the patch, it looks good; there's a new line added for the comment "// Garland colors :)" on line 184 of eu_cookie_compliance.admin.inc, not important at all but just accidental extra formatting I guess? If supposed to be on new line, should it be on the preceding line?

Otherwise the other general tidying up you've done is excellent.

Thanks,
luke

clairedesbois@gmail.com’s picture

I have recreate the patch for the last version of dev for drupal 7 because it does not apply

clairedesbois@gmail.com’s picture

And an other version because the patch isn't compatible with the patch in the issue for the page cache https://www.drupal.org/node/2153799

doublejosh’s picture

Seems like the right place to follow doing i18n/language properly in this module is over here: https://www.drupal.org/node/2528272

  • svenryen committed bae1e9c on 7.x-1.x
    Issue #2166465 by BAHbKA, Calystod: Issue with language settings
    
svenryen’s picture

Status: Reviewed & tested by the community » Fixed
svenryen’s picture

Status: Fixed » Closed (fixed)