search

FriendsOfREDAXO / consent_manager

Stellt ein Opt-In Cookie Banner zur Verfügung
MIT License
57 stars 15 forks source link

Undefined array key "majorVersion" #294

Closed tbaddade closed 1 year ago

tbaddade commented 1 year ago

Description / Beschreibung

Undefined array key "majorVersion" Datei: src/addons/consent_manager/lib/consent_manager_frontend.php:31

Affected versions / Verwendete Versionen Version: 4.0.3 … scheint in der GH Version genauso zu sein

aeberhard commented 1 year ago

Thomas @tbaddade was steht denn in der redaxo\data\addons\consent_manager\config.json ?

tbaddade commented 1 year ago

Thomas @tbaddade was steht denn in der redaxo\data\addons\consent_manager\config.json ?

Bildschirmfoto 2023-05-09 um 13 48 05

Ich vermute es liegt daran, dass nach einem Deploy die json Datei noch nicht existiert und erst später wieder vom AddOn angelegt wird.

aeberhard commented 1 year ago

Ja die wird bei install/update angelegt

aeberhard commented 1 year ago

@tbaddade könnt ihr beim deploy auch eine Klasse aufrufen? consent_manager_cache::forceWrite();

tbaddade commented 1 year ago

könnt ihr beim deploy auch eine Klasse aufrufen?

könnten wir. Wäre es nicht sinnvoller, wenn das AddOn es selbst regelt.

Das Problem scheint hier zu sein https://github.com/FriendsOfREDAXO/consent_manager/blob/main/lib/consent_manager_cache.php#L194

Da holst du den Cache und prüfst hier auf null https://github.com/FriendsOfREDAXO/consent_manager/blob/main/lib/consent_manager_frontend.php#L31

getCache liefert aber gar kein null sondern ein leeres [] als default zurück.