Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Die Kennung der Skript-Container ist nicht eindeutig. #210

Closed
DanielWeitenauer opened this issue Dec 3, 2021 · 2 comments
Closed

Die Kennung der Skript-Container ist nicht eindeutig. #210

DanielWeitenauer opened this issue Dec 3, 2021 · 2 comments
Labels
enhancement New feature or request

Comments

@DanielWeitenauer
Copy link
Member

DanielWeitenauer commented Dec 3, 2021

Da data-uid sowohl für die Kennung der Checkboxen im Popup, als auch der Skript-Container verwendet wird, werden Skripte nicht korrekt initialisiert, wenn ein Cookie die gleich ID verwendet wie eine Cookiegruppe. In diesem Fall wird versucht, die Daten aus dem entsprechenden Input-Element und nicht aus dem passenden DIV zu lesen.
Ich hatte z.B. Gruppe und Cookie mit dem Schlüssel google-tag-manager versehen.

Die Elemente sollten hier über eindeutigere Attribute ausgelesen werden:

addScript(consent_managerBox.querySelector('[data-uid="' + uid + '"]'));

aeberhard added a commit that referenced this issue Dec 13, 2021
Siehe #210
Eigenes consent_manager_box.php-Fragment muss entsprechend angepasst werden.
@aeberhard
Copy link
Member

Hallo @DanielWeitenauer ! Kannst Du mal mit der aktuellen Github-Version testen!

@aeberhard aeberhard added the enhancement New feature or request label Dec 13, 2021
@aeberhard
Copy link
Member

Ich mache hier mal zu @DanielWeitenauer

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants