BETA MODUL: CustomersNotice - Hinweise für Kunden

[Michael.TB]

Habe mir nun nochmal die aktuelle Version v0.5 herunter geladen, denn zu meiner Version ist da einiges geändert. Allerdings hat das unterm Strich nun keinen Unterschied gemacht, sowie ich die "CustomersNoticeManager.class.php" in mein Verzeichnis kopiere habe ich eine weiße Seite.

im log steht folgendes:

Code: PHP  [Auswählen]

[2023-11-25 12:55:45]   [error] [modified]      [pid:32634]     ERROR found for URL: /  {}      {}
[2023-11-25 12:55:45]   [error] [modified]      [pid:32634]     Undefined constant "customers_email_address" in File: /is/htdocs/.../includes/external/customers_notice/classes/CustomersNoticeManager.class.php on Line: 28    {}      {}

Also auch hier wieder Line 28.

Der Inhalt der CustomersNoticeManager.class.php ist wie folgt:

Code: PHP  [Auswählen]

<?php
/**
 * Manage Customers notice
 *
 * @author    Timo Paul <mail@timopaul.biz>
 * @copyright (c) 2014, Timo Paul Dienstleistungen
 * @license   http://www.gnu.org/licenses/gpl-2.0.html
 *            GNU General Public License (GPL), Version 2.0
 */

class CustomersNoticeManager {
 
  public static function run() {
    global $smarty, $category_depth;
    $nullDate = '0000-00-00 00:00:00';

        //p3e BOF 20180305 Kunde ist Newsletterempfänger?
        $newletter = TRUE ;
        If ($_SESSION['customers_status']['customers_status_id'] != 1) {
                $account_query = xtc_db_query("select customers_id,
                                                                                                        customers_email_address
                                                                                          FROM ".TABLE_CUSTOMERS."
                                                                                         WHERE customers_id = '".(int) $_SESSION['customer_id']."'");
                $account_mail = xtc_db_fetch_array($account_query);
                $check_mail_query = xtc_db_query("SELECT customers_email_address,
                                                                                                   customers_id
                                                                                          FROM ".TABLE_NEWSLETTER_RECIPIENTS."
                                                                                         WHERE customers_email_address = '".$account_mail[customers_email_address]."'
                                                                                           AND mail_status = '1'
                                                                                   ");
                if (xtc_db_num_rows($check_mail_query) == 0) $newletter = FALSE ;
        }
        //p3e EOF 20180305 Kunde ist Newsletterempfänger?
       
    $cs = $_SESSION['customers_status']['customers_status_id'];
    $script = basename($_SERVER['SCRIPT_NAME']);
    $script = substr($script, 0, strripos($script, '.'));
    if ('index' == $script && xtc_not_null($category_depth) && 'top' != $category_depth) {
      $script = 'category';
    }
    if (preg_match('#^(account|address)_#', $script)) {
      $script = 'account';
    }
    if (preg_match('#^(checkout|checkout)_#', $script)) {
      $script = 'checkout';
    }
    $stmt = 'SELECT cn.*, cnd.title, cnd.description ' .
            'FROM ' . TABLE_CUSTOMERS_NOTICE . ' AS cn ' .
              'LEFT JOIN ' . TABLE_CUSTOMERS_NOTICE_DESCRIPTION . ' AS cnd ' .
                'ON cn.customers_notice_id = cnd.customers_notice_id ' .
                'AND cnd.languages_id = ' . $_SESSION['languages_id'] . ' ' .
            'WHERE cn.status = 1 ';
    if (($_SESSION['cs_popup'] == 'popup') OR ($newletter == TRUE )) $stmt .= ' AND cn.template <> "newsletter.html"'; // p3e 20180302 PopUps nur einmal pro Session und nur wenn kein Newsletterempfänger
    $stmt .=  'AND (cn.startdate = "' . $nullDate . '" OR cn.startdate < "' . date('Y-m-d H:i:s') . '") ' .
              'AND (cn.enddate = "' . $nullDate . '" OR cn.enddate > "' . date('Y-m-d H:i:s') . '") ' .
              'AND (cn.customers_status = "" OR cn.customers_status LIKE "' . $cs . '" OR cn.customers_status LIKE "' . $cs . ',%" OR cn.customers_status LIKE "%,' . $cs . '" OR cn.customers_status LIKE "%,' . $cs . ',%") ' .
              'AND (cn.pages = "" OR cn.pages LIKE "' . $script . '" OR cn.pages LIKE "' . $script . ',%" OR cn.pages LIKE "%,' . $script . '" OR cn.pages LIKE "%,' . $script . ',%") ' .
            'ORDER BY position';
    $query = xtc_db_query($stmt);
    $str = '';
    while ($row = xtc_db_fetch_array($query)) {
      $s = new Smarty();
      $s->assign('language', $_SESSION['language']);
      $s->assign('tpl_path', HTTP_SERVER.DIR_WS_CATALOG.'templates/'.CURRENT_TEMPLATE.'/');
      $s->caching = 0;
      foreach ($row as $k => $v) {
        $s->assign($k, $v);
      }
      $str .= $s->fetch(CURRENT_TEMPLATE . '/module/customers_notice/' . $row['template']);
      if ($row['template'] == 'newsletter.html') $_SESSION['cs_popup'] = 'popup'; // p3e 20180302 PopUps nur einmal pro Session
    }
    $smarty->assign('CUSTOMERS_NOTICE', $str);
  } // end of static method run()
 
} // end of class CustomersNoticeManager
?>

[awids]

Warum fehlen in dem Screenshot oben die Single Quotes bei $account_mail[customers_email_address]?
Müsste es nicht $account_mail['customers_email_address'] heißen?

 

[Tomcraft]

Habe mir nun nochmal die aktuelle Version V.05 herunter geladen, denn zu meiner Version ist da einiges geändert. Allerdings hat das unterm Strich nun keinen Unterschied gemacht, sowie ich die "CustomersNoticeManager.class.php" in mein Verzeichnis kopiere habe ich eine weiße Seite.
[...]

Die Version ist von 2019!
Wieso verwendest du nicht die aktuelle Version "nr_csn_reworked_0.2.4.zip"?

Grüße

Torsten

[Michael.TB]

Ich weiß es nicht mehr genau, die beiden Varianten gab es damals auch schon, haben beide funktioniert und ich meine die v05 hat mir irgendwie besser gefallen.

Ich habe nun die nr_csn_reworked_0.2.4 installiert. Ich habe allerdings auch noch nicht diese SQL Befehle gesendet, da ich mir nicht sicher bin welchen oder beide, für 0.2.4 steht da nichts direkt.

[...]
   Für von noRiddle erweiterte Version 0.2.2 diesen Befehl in phpMyAdmin absetzen, insofern eine Vorgänger-Version des Modules bereits installiert war:

Code: SQL  [Auswählen]

   ALTER TABLE customers_notice ADD countries mediumtext DEFAULT NULL;

   Für von noRiddle erweiterte Version 0.2.3 diesen Befehl in phpMyAdmin absetzen, insofern eine Vorgänger-Version des Modules bereits installiert war:

Code: SQL  [Auswählen]

   ALTER TABLE customers_notice ADD customers_id INT(11) DEFAULT NULL AFTER POSITION;

[...]

Nachtrag: nun scheint es zu funktionieren, allerdings nur auf der Hauptseite. Nicht im Warenkorb oder auf der Checkout Seite. Die Häkchen habe ich alle gesetzt.  Habe allerdinsg trotzdem noch keinen der o.g. SQL Befehle ausgeführt, evtl liegt es daran. Sollte ich das noch machen? Wenn ja, welchen?

Anbei noch ein Screenshot von der index.html . Habe den Code direkt unter den Slider eingebunden.

[EDIT Tomcraft 25.11.2023: Beitrag formatiert.]

[Tomcraft]

Die kannst du sicherheitshalber einfach ausführen. Sollte die Spalte bereits in der Tabelle sein, dann wird phpMyAdmin eh einen Fehler anzeigen und den SQL-Befehl nicht ausführen können.

Grüße

Torsten

[Michael.TB]

Ok, Befehle haben beide Fehlermeldung wegen doppelter Spalte ergeben, die hatte ich also schon.
Die Position in der "index.html" habe ich nun noch verändert und oberhalb des Sliders gesetzt, nun wird es auch auf den anderen Seiten angezeigt. Ich glaube nun funktioniert alles.

Vielen lieben Dank an alle für die Hilfe.

[noRiddle (revilonetz)]

@awids
Ist es nicht erquickend wie einfach ignoriert wird was man sagt ?, da kannst du auch 100 Experten-Sternchen haben.
Einer der Gründe (der zweitrangige) waum ich hier nichts (so gut wie nichts) mehr mache.

For the record:
In meiner Version hat es diesen Kommentar

Code: PHP  [Auswählen]

//added missing single quotes with $account_mail[customers_email_address] below, noRiddle

da ist es also gefixt.

Gruß,
noRiddle

[Tomcraft]

Ich würde nicht davon ausgehen, dass auch nur 10% hier aus dem Forum damit etwas anfangen können. Es sind keine Programmierer, sondern Shopbetreiber, die maximal einer Anleitung gemäß "Suche & Ersetze" folgen können.

Daher mein Hinweis das aktuelle Modul zu verwenden.

Grüße

Torsten

[awids]

@awids
Ist es nicht erquickend wie einfach ignoriert wird was man sagt ?, da kannst du auch 100 Experten-Sternchen haben.
Einer der Gründe (der zweitrangige) waum ich hier nichts (so gut wie nichts) mehr mache.
[...]

Tja, solche Leute landen auf meiner imaginären Blacklist und brauchen sich dann auch nicht wundern, wenn sie selbst künftig von mir ignoriert werden, obwohl eine Lösung bekannt ist. Meine Zeit ist mir selbst viel zu kostbar, als das ich sie darauf aufwenden würde, mich über sowas aufzuregen. War aus diesem Thema eigentlich auch schon raus und antworte jetzt nur, weil du mich direkt angesprochen hast.

[Tomcraft]

Was für ein Statement...

@Michael.TB
Ich möchte mich davon ausdrücklich distanzieren!
Es gibt hier im Forum auch diejenigen, die keine Programmierkenntnisse voraussetzen und helfen.

Grüße

Torsten

[bumerang24]

Hallo liebes Forum, hallo noRiddle,

mal ne unwissende Frage zu dem Tool, gibt es schon ein Update, bzw. ist was geplant für Version 3.x?

Danke schon mal für eine Info.

Grüße

Guido

[noRiddle (revilonetz)]

Ich habe vor einiger Zeit beschlossen nicht mehr an Modulen rumzufummeln die nicht von mir sind.
Das hat mehrere Gründe die ich mom. keine Lust habe aufzuführen.
Ich gebe im Höchstfall Tipps, schnüre jedoch keine neuen Pakete.

Abgesehen davon: Was geht denn nicht und wenn was nicht geht, wie lauten die Log-Einträge ?

Gruß,
noRiddle

[bumerang24]

Hallo noRiddle,

vielen Dank für den Hinweis und die Frage.

Es geht los bei der Installation:
Sauberer Shop, das Tool noch nie installiert, einfügen des DB Befehls:

Fehlermeldung:

Code: SQL  [Auswählen]

Fehler

SQL-Befehl:

-- new for version 0.2.2, 01-2022, noRiddle
ALTER TABLE customers_notice ADD countries mediumtext DEFAULT NULL

MySQL meldet: Dokumentation
#1060 - Doppelter Spaltenname: 'countries'

Nach kompletter Installation des Moduls unter Modified 3.0.2 ist auch Modul nicht unter Hilfsprogramme auffindbar.
Habe ich jetzt einen Denkfehler? oder...
Alles unter PHP 8.129

Übrigens,

tpl_modified_responsive wurde kopiert, die Kopie dann umbenannt in tpl_modified_nova

Das Nova findet auch Verwendung in meinen Testshop

Danke für einen Hinweis schon mal.

Grüße

Guido

[Q]

[...]
Es geht los bei der Installation:
Sauberer Shop, das Tool noch nie installiert, einfügen des DB Befehls:

Fehlermeldung:

Code: SQL  [Auswählen]

Fehler

SQL-Befehl:

-- new for version 0.2.2, 01-2022, noRiddle
ALTER TABLE customers_notice ADD countries mediumtext DEFAULT NULL

MySQL meldet: Dokumentation
#1060 - Doppelter Spaltenname: 'countries'

[...]

Dann hast du entweder doch schon einmal die SQL-Datei importiert, oder den Befehl

Code: SQL  [Auswählen]

ALTER TABLE customers_notice ADD countries mediumtext DEFAULT NULL

erneut ausgeführt.

[...]
Nach kompletter Installation des Moduls unter Modified 3.0.2 ist auch Modul nicht unter Hilfsprogramme auffindbar.
Habe ich jetzt einen Denkfehler? oder...
Alles unter PHP 8.129
[...]

Dazu:

[...]
Abgesehen davon: Was geht denn nicht und wenn was nicht geht, wie lauten die Log-Einträge ?
[...]

[...]
Übrigens,

tpl_modified_responsive wurde kopiert, die Kopie dann umbenannt in tpl_modified_nova

Das Nova findet auch Verwendung in meinen Testshop
[...]

Vermutlich kann ich damit nichts anfangen, weil ich das Modul nicht getestet habe. Aber die Aussage ist etwas schräg!?! Du nennst das responsive-Template in das nova-Template  um und nova findet AUCH Verwendung? Dann ist es aber nicht mehr das Nova, dass vom Team angeboten wird. Wieso macht man sowas?

[bumerang24]

Hallo Q,

Vielen Dank für den Versuch, aber der ist eher schlecht als recht.
Was Schräges habe ich nicht gemacht, das hast du mir schön in den Mund gelegt.
Aber wer das Modul nicht kennt, versteht natürlich auch nicht, was ich gemacht habe.

Im Modul ist ein Ordner Template, wie in so vielen Modulen.
Die Templates werden / wurden selten auf das neue Nova hochgezogen, aber ich brauche die Dateien auch für das verwendete Nova, sonst kann es ja nicht gehen.
Oder soll ich die Datei in das Responsive schreiben, und mich wundern das nichts geht?

Bin gespannt, ob du jetzt noch eine besser Idee hast.