MODUL: OIL.js Cookie Consent Management

[lithoverlag]

Wenn ich das richtig verstehe wurde nur die tracking,php geändert oder?
Also um mir die Mühe zu sparen die ajax.php nochmals zu checken und Paypal Probleme erst gar nicht wieder neu entstehen zu lassen müsste es ja reichen die tracking.php zu ersetzen...?
Gruß
Litho

[Tomcraft]

Korrekt.

Grüße

Torsten

[snocer]

Jetzt driften wir hier vom eigentlichen Thema ab, aber ihr habt Recht und wir planen ab dem kommenden Release mit deutlich kürzeren Intervallen.
[...]

Muss einfach noch einmal Danke sagen. Das auslassen von 2.0.6.1 und gleich der Quantensprung auf die 2.0.7.0 und dazu das alles auch gut läuft, ohne dass es zu nennenswerten Problemen gekommen ist, inkl. der Unterstützung von PHP8.1 und vielem anderem. Einfach noch einmal Danke an Euch Team modified. Spende ist vorgemerkt und wird ausgelöst. Ich hoffe, dass sich auch viele weitere Nutzer der wirklich guten Shopsoftware durch Eure Leistungen angespornt fühlen ebenfalls mal wieder über eine Spende nach zu denken.

Meine Wünsche für Version 3.0.x.x werde ich mal in einem neuen Thread zusammenfassen.

[lithoverlag]

Moin,
ich war so frei und habe die ajax.php von Oil und dem aktuellsten PayPal Modul (2.0.6.0 rev 13500) gemerged.
die von Oil hat zwei Zeilen (Zeile 110) mehr Text ist aber ansonsten identisch.

Code: PHP  [Auswählen]

// log parse time
if (defined('STORE_PAGE_PARSE_TIME') && STORE_PAGE_PARSE_TIME == 'true') {
  $parse_time = number_format((microtime(true) - PAGE_PARSE_START_TIME), 3);
  if ($parse_time >= STORE_PAGE_PARSE_TIME_THRESHOLD) {
    if (is_file(DIR_FS_CATALOG.'includes/modified_cache.php')) {
      error_log(date(STORE_PARSE_DATE_TIME_FORMAT) . ' [' . $parse_time . 's] ' . getenv('REQUEST_URI') . "\n", 3, DIR_FS_LOG.'mod_parsetime_'. date('Y-m-d') .'.log');
    } else {
      error_log(strftime(STORE_PARSE_DATE_TIME_FORMAT) . ' [' . $parse_time . 's] ' . getenv('REQUEST_URI') . "\n", 3, DIR_FS_LOG.'mod_parsetime_'. date('Y-m-d') .'.log');
    }
  }
}

Diese Zeile ist bei Oil zusätzlich. (Zeilen 110-112):

Code: PHP  [Auswählen]

    if (is_file(DIR_FS_CATALOG.'includes/modified_cache.php')) {
      error_log(date(STORE_PARSE_DATE_TIME_FORMAT) . ' [' . $parse_time . 's] ' . getenv('REQUEST_URI') . "\n", 3, DIR_FS_LOG.'mod_parsetime_'. date('Y-m-d') .'.log');
    } else {

Da ich 5 Sprachen im Shop habe und beim Aufruf der ersten Sprache den Coockie Hinweis bestätige und dann die Sprachen wechsele bekomme ich nach dem dem 3 Wechsel wieder eine Cookie Aufforderung.

Ändere ich diese Zeile den Wert von 3 auf 5

Code: PHP  [Auswählen]

error_log(date(STORE_PARSE_DATE_TIME_FORMAT) . ' [' . $parse_time . 's] ' . getenv('REQUEST_URI') . "\n", 3, DIR_FS_LOG.'mod_parsetime_'. date('Y-m-d') .'.log');

dann kann ich alle Sprachen durchklicken ohne erneute Cookie Aufforderung.

1. Kann dies jemand mit mehr als 4 Sprachen im Shop dieses Problem nachvollziehen (erneute Cookie Bestätigung nach Wechsel auf die 4 Sprache) ?
2. Habe ich die richtige Zeile geändert? (Prinzip blindes Huhn und Korn??)
Gruß
Litho

[EDIT Tomcraft 24.05.2022: Code formatiert. Bitte kein BBCode in Code-Tags verwenden!]

[noRiddle (revilonetz)]

Du kannst ruhig die Datei aus dem OIL-Modul nehmen. Die beinhaltet an der von dir zitierten Stelle lediglich eine weitere Abfrage die sie mit der Shop-Version 2.0.7.0 kompatibel macht.

Deine Änderung mit der "3" ist nicht korrekt. Da geht es um den Typ in der Funktion error_log().  Das mußt du rückgängig machen.

Warum nach 3-maligem Sprachenwechsel das Cookie-Banner erneut auftaucht kann ich dir nicht sagen.
Wenn es nach deiner Änderung nicht mehr so war ist das Zufall, bzw. hat einen anderen Grund und der Schluß es läge an der von dir veränderten Code-Stelle ist ein Trugschluß.

Gruß,
noRiddle

[lithoverlag]

OK habe ich zurückgestellt und das Problem mit den Cookies scheint sich vorerst auch von selbst behoben zu haben.

[Tomcraft]

Die Module aus Beitrag 1 wurden nochmal überarbeitet. Eingeflossen sind die Korrekturen aus r14556, r14557 & r14558.

Grüße

Torsten

[Tomcraft]

Die Module aus Beitrag 1 wurden nochmal überarbeitet. Eingeflossen sind die Korrekturen aus r14628, r14635, r14665 & r14711.

Grüße

Torsten

[AllyG]

Hi,
ich habe das Modul jetzt aktualisiert und zum ersten Mal installiert. Ersetzt ein Modul, welches ich früher verwendet hatte (ist schon ordentlich deinstalliert) und ich finde es echt klasse!
Mein mod_warning_admin.log spuckte mir nach der Installation folgendes aus:

Code: [Auswählen]

[2022-09-14 13:20:30]    [warning]    [modified]    [pid:4107483]    WARNING found for URL: /#####/module_export.php?set=system&module=cookie_consent&action=install    {}    {}
[2022-09-14 13:20:30]    [warning]    [modified]    [pid:4107483]    1064 - You have an error in your SQL syntax; check the manual that corresponds to your MariaDB server version for the right syntax to use near '`cookie_consent_categories`' at line 1<br/><br/>SHOW TABLES LIKE `cookie_consent_categories` in File: /www/htdocs/w00c2d2f/adrenalin-fishing/modified-shop/inc/db_functions_mysqli.inc.php on Line: 153    {}    {}
[2022-09-14 13:20:30]    [warning]    [modified]    [pid:4107483]    Backtrace #0 - /www/htdocs/w00c2d2f/adrenalin-fishing/modified-shop/#####/includes/modules/system/cookie_consent.php called at Line 74    {}    {}
[2022-09-14 13:20:30]    [warning]    [modified]    [pid:4107483]    Backtrace #1 - /www/htdocs/w00c2d2f/adrenalin-fishing/modified-shop/#####/module_export.php called at Line 152    {}    {}
[2022-09-14 13:20:30]    [warning]    [modified]    [pid:4107483]    WARNING found for URL: /#####/module_export.php?set=system&module=cookie_consent&action=install    {}    {}
[2022-09-14 13:20:30]    [warning]    [modified]    [pid:4107483]    1064 - You have an error in your SQL syntax; check the manual that corresponds to your MariaDB server version for the right syntax to use near '`cookie_consent_cookies`' at line 1<br/><br/>SHOW TABLES LIKE `cookie_consent_cookies` in File: /www/htdocs/w00c2d2f/adrenalin-fishing/modified-shop/inc/db_functions_mysqli.inc.php on Line: 153    {}    {}
[2022-09-14 13:20:30]    [warning]    [modified]    [pid:4107483]    Backtrace #0 - /www/htdocs/w00c2d2f/adrenalin-fishing/modified-shop/######/includes/modules/system/cookie_consent.php called at Line 166    {}    {}
[2022-09-14 13:20:30]    [warning]    [modified]    [pid:4107483]    Backtrace #1 - /www/htdocs/w00c2d2f/adrenalin-fishing/modified-shop/######/module_export.php called at Line 152    {}    {}
Ich habe mal bei uns nachgesehen und wir scheinen auf MariaDB 10.5.16 zu laufen. Muss ich da jetzt aktiv werden oder sollte ich das einfach als Hinweis verstehen?

[Tomcraft]

Sind denn die Tabellen "cookie_consent_categories" & "cookie_consent_cookies" in der Datenbank vorhanden?

Grüße

Torsten

[AllyG]

Hallo Torsten,
hab ich eben geprüft und das Ergebnis ist JA.

[Karl1]

Hallo AllyG,
kann es sein, dass du eine veraltete Modulversion nutzt?

Der falsche SQL-Befehl soll aufgerufen werden in Zeile 74

[...]

Code: [Auswählen]

[...] Backtrace #0 - /www/htdocs/w00c2d2f/adrenalin-fishing/modified-shop/#####/includes/modules/system/cookie_consent.php called at Line 74    {}    {}
[...][...]

In allen neueren Modulversionen (seit 05/2022) ist der Aufruf in Zeile 75.

Gruß Karl

[AllyG]

Hallo Karl,
ich habe mir heute erst das Paket 1.34 für 2.0.6.0 hier aus dem Thread geladen und die Daten auf den FTP transferriert. Das Modul deinstalliert und wieder neu installiert.
Gruß Kay

[noRiddle (revilonetz)]

Dann hat dein FTP-Client die neuen Dateien nicht richtig hochgeladen.
Der alte Fehler mit den Backticks

Code: PHP  [Auswählen]

$check_query = xtc_db_query("SHOW TABLES LIKE `".TABLE_COOKIE_CONSENT_CATEGORIES."`");

ist längst behoben (und der Befehl steht in Zeile 75, wie Karl1  richtig bemerkt hat):

Code: PHP  [Auswählen]

$check_query = xtc_db_query("SHOW TABLES LIKE '".TABLE_COOKIE_CONSENT_CATEGORIES."'");

Außerdem spammst du das Forum zu mit deinen ganzen Warnings und Notices.
Das ist schon ziemlich dreist hier ohne Unterlass in mehreren Threads kostenlosen Privat-Support zu erfragen.

Gruß,
noRiddle

[Tomcraft]

[...]
Außerdem spammst du das Forum zu mit deinen ganzen Warnings und Notices.
Das ist schon ziemlich dreist hier ohne Unterlass in mehreren Threads kostenlosen Privat-Support zu erfragen.
[...]

In erster Linie ist es vor allem nicht zielführend immer gleich mehrere Module und Baustellen gleichzeitig zu eröffnen!
Schließe bitte immer erst eine Sache ab, bevor du ein neues Thema eröffnest!

Grüße

Torsten