BETA MODUL: Merkzettel oder Merkliste

[WeXsler]

wenn ich das noch ergänzen darf. 1.06er Shops, die auf charset utf-8 / Kollation utf8_general_ci ein- bzw. umgestellt wurden, müßten vermutlich andere Datenbankbefehle aus der Anleitung "AnleitungV1.1_xwishlist_XTC-M_106.txt" erhalten. Für den Kenner versteht sich das vermutlich von selbst. Ist das so wie ich es unten gemacht habe korrekt? Oder muß es utf8 statt utf-8 heißen? Bin mir nicht sicher worin der Unterschied besteht.

Code: PHP  [Auswählen]

CREATE TABLE `customers_wishlist` (
  `customers_basket_id` int(11) NOT NULL auto_increment,
  `customers_id` int(11) NOT NULL default '0',
  `products_id` tinytext collate utf8_general_ci NOT NULL,
  `customers_basket_quantity` int(2) NOT NULL default '0',
  `final_price` decimal(15,4) NOT NULL default '0.0000',
  `customers_basket_date_added` varchar(8) collate utf8_general_ci default NULL,
  PRIMARY KEY  (`customers_basket_id`)
) ENGINE=MyISAM DEFAULT CHARSET=utf-8 COLLATE=utf8_general_ci AUTO_INCREMENT=1 ;

Code: PHP  [Auswählen]

CREATE TABLE `customers_wishlist_attributes` (
  `customers_basket_attributes_id` int(11) NOT NULL auto_increment,
  `customers_id` int(11) NOT NULL default '0',
  `products_id` tinytext collate utf8_general_ci NOT NULL,
  `products_options_id` int(11) NOT NULL default '0',
  `products_options_value_id` int(11) NOT NULL default '0',
  PRIMARY KEY  (`customers_basket_attributes_id`)
) ENGINE=MyISAM DEFAULT CHARSET=utf-8 COLLATE=utf8_general_ci AUTO_INCREMENT=1 ;

[WeXsler]

Hallo, tolles und wichtiges Modul! Läuft auch prima. Kann es sein, dass die Grundpreisanzeige (VPE) vergessen wurde? Oder hab ich vielleicht einen Fehler beim Einbauen gemacht? Wie bekomme ich den Grundpreis bei meinen grundpreispflichtigen Produkten im Merkzettel angezeigt?

[WeXsler]

... und wenn ich schon mal dabei bin - da ist ein Warenkorbbutton. Der muß ja auch sein. Mit dieser Möglichkeit ein Produkt in den Warenkorb zu legen gehen meines Wissens auch noch andere Pflichtinfos außer dem Grundpreis einher nämlich u.a. die Steuerinfo (inkl. MwSt.) und Versandkosteninfo (Link: zzgl. Versand). Hat da jemand vielleicht schon eine Lösung?

[WeXsler]

wenn ich das richtig sehe, ist das an sich wirklich schöne Modul für den Produktiveinsatz nicht ohne Bauchschmerzen einsetzbar. Es fehlen schlicht einige Pflichtinfos sh. Vorpost. Hat den gar keiner eine Idee wie man das da hinein bekommt? Ich bin jetzt den dritten Tag drüber wenigstens die VPE angezeigt zu bekommen aber die will sich einfach nicht zeigen 

Was ich u.a. und im letzte Versuch analog zur Anleitung "Anzeige der VPE im Warenkorb" von hier https://www.modified-shop.org/forum/index.php?topic=30545.new#new versucht habe:

In includes/modules/order_details_wishlist.php
suche

Code: PHP  [Auswählen]

  $module_content[$i]=array(
  'PRODUCTS_NAME'              => $products[$i]['name'].$mark_stock,
  'PRODUCTS_QTY'               => xtc_draw_input_field('cart_quantity[]', $products[$i]['quantity'], 'size="2"') . xtc_draw_hidden_field('products_id[]', $products[$i]['id']),
  'PRODUCTS_MODEL'             => $products[$i]['model'],
  'PRODUCTS_TAX'               => number_format($products[$i]['tax'], TAX_DECIMAL_PLACES),
  'PRODUCTS_IMAGE'             => $image,
  'IMAGE_ALT'                  => $products[$i]['name'],
  'BOX_DELETE'                 => xtc_draw_checkbox_field('cart_delete[]', $products[$i]['id']),
  'DELETE_ICON'                => '<a href="'. xtc_href_link(FILENAME_SHOPPING_CART, 'action=update_product_cart&products_id='.$products[$i]['id']). '">'.  xtc_image('templates/'.CURRENT_TEMPLATE.'/img/button_cart_delete.jpg', 'l&ouml;schen'). '</a>',

und füge danach ein:

Code: PHP  [Auswählen]

'PRODUCTS_VPE' => $products[$i]['vpe'],

Im templates/xtc5/module/wish_list_order_details.html
suche

Code: PHP  [Auswählen]

<td valign="top" class="main"><strong><a href="{$module_data.PRODUCTS_LINK}">{$module_data.PRODUCTS_NAME}</a></strong>{$module_data.PRODUCTS_SINGLE_PRICE}<br />

und füge danach ein

Code: PHP  [Auswählen]

<class="singleprice">{if $module_data.PRODUCTS_VPE}({$module_data.PRODUCTS_VPE}){/if}<br />

Ergebnis: Eine leere Leerzeile unterhalb des Pruduktlinks und Einzelpreis ... . Ich habe fertig - was fehlt da bloß? Das müßte doch theoretisch analog zur Anzeige der VPE im Warenkorb zu machen sein. Ist es aber offensichtlich nicht. Vielleicht kann das Problem jemand lösen. Ich gehe einfach mal ganz naiv davon aus, dass das für alle Anwender der Merkeliste hilfreich wäre ... .

[WeXsler]

zu Hülf! Hat den sonst niemand der Anwender ein Problem damit, dass zentrale Pflichtinfos (Grundpreis, Steuerinfo, Versandkostenhinweis) in dem Modul fehlen bzw. nicht angezeigt werden? Ich möchte das Modul wirklich gerne nutzen, aber mir ist schon ganz schlecht vor Sorge, dass das Leute auf den Plan ruft, die anderes im Sinn haben, als etwas zu bestellen ... .  Bin für jede Hilfe dankbar, habe keine Idee mehr wie ich das reinbekommen soll.

[Bonsai]

Ich habe mich nicht genau damit befasst, was Du da bastelst, aber eventuell hilft es Dir die richtige Frage zu lesen
Wo wird diese Variable denn befüllt?

Code: PHP  [Auswählen]

$products[$i]['vpe']

Da muss irgendein Aufruf für eine SQL Abfrage sein. Eventuell wird auch die Klasse products verwendet.

[WeXsler]

Hallo Bonsai,

gebastelt wurde ja im Grunde schon. Das Modul Merkzettel dürfte schon eine Weile bei vielen im Einsatz sein.

Dieses im Grunde wirklich brauchbare Modul hat aber den schlagenden Nachteil, dass die genannten Pflichtinfos nicht mit angezeigt werden! Das sollte alle betreffen, die dieses Modul im Einsatz haben unabhängig davon ob sie auch grundpreispflichtige Artikel anbieten, denn es fehlt ja dann trotzdem immer noch der Hinweis auf die MwSt. und die Versandkosten am Artikel! Wenn meine rechtskundlichen Bemühungen der letzten Jahre mich nicht völlig in die Irre führen, gehören diese Infos da hin. Alles andere kann teuer werden.

Darum bin ich ob der fehlenden Resonanz etwas ratlos.

Danke für Deinen Hinweis! So ähnlich habe ich das auch schon vermutet. Da wo die Variablen abgefragt werden sollen, ist vermutlich entweder gar kein entsprechender code vorhanden oder er ist falsch.

Bei dem Thema bin ich allerdings wirklich draußen das übersteigt derzeit noch bei weitem meine prog-Kenntnisse

Ich möchte auch keine Extrawurst gebraten bekommen in dem jemand Wissendes das nur für mich alleine  vorkaut.

Wenn niemand eine Lösung für die Allgemeinheit, die diese womöglich nicht braucht, anbieten mag, werde ich das über kurz oder lang natürlich beauftragen.

[Bonsai]

Zeig mal die codestelle an der das Array $products gefüllt wird! Die sollte man finden können. Dann habe ich eventuell eine Idee.

[WeXsler]

siehst Du, damit kann ich schon nicht wirklich etwas anfangen

codestelle an der das Array $products gefüllt wird??

[Ceciro]

Vielleicht sollte man sich davon verabschieden, an jeder Stelle des Shops alle Funktionalitäten anzubieten.

Die Aufgabe eines Merkzettels ist doch, die interessanten Artikel auf einer Liste bereit zu halten, damit der Kunde abseits des Warenkorbes  eine Gedankenstütze / Arbeitspapier hat.

Deshalb habe ich die Ausgabe des Moduls gestrippt und biete "nur" das Produktbild, die Produktbezeichnung und die Möglichkeit des Entfernens an. Über die Produktbezeichnung wird die Artikeldetailseite aufgerufen.

Kein Preis, keine MwSt., keine Versandkosten, kein Warenkorbbutton und gut ist. Dadurch wird auch gleichzeitig die unschöne Erfahrung vermieden, dass ein Artikel von der Merkliste im Warenkorb mit dem Vermerk "nicht lieferbar" erscheint.

Manchmal ist weniger halt mehr.

Gruß Cicero

[Bonsai]

@Cicero: volle Zustimmung! Weniger ist manchmal mehr

@WeXsler

Irgendwo muss stehen

Code: PHP  [Auswählen]

$products[$i] =

eventuell auch sowas:

Code: PHP  [Auswählen]

$products[] =

oder sowas:

Code: PHP  [Auswählen]

$products[]['name'] =

Auf alle Fälle steht irgendwo $products links von einem =
Die Stelle brauchen wir, dann kann geholfen werden.

[WeXsler]

grundsätzlich kann man da auch nur zustimmen. Weniger ist manchmal mehr. Unbedingt. Die Grenzen was zu viel oder zu wenig ist, sind wohl aber doch fließend und wenn ich hier die Platzhirsche in den diversen Branchen anschaue gibt es da, was Merkzettel bzw. auch Wunschliste angeht, schon hochkomfortable Teile ... . Die Möglichkeit ein Produkt aus einer Merkliste in den Warenkorb zu legen gehört da immer dazu. Die Pflichtinfos werden in der Regel da auch nicht "vergessen".

Letztlich halte ich die Möglichkeit ein Produkt direkt aus der Merkliste in den Warenkorb zu legen einfach für kundenfreundlich und die Merkliste funktioniert hier ja auch prima. Ich würde mir lediglich noch als feature wünschen, dass ein Kunde sich mehrere Merkzettel anlegen kann. Aber das hat Zeit. Erstmal die Pflichtinfos reinpfriemeln, dann kann ich die Merkeliste wieder freischalten

Nun also zu den Dateien in denen $products links von einem = stehen. Hmm, das sind aber eigentlich alle drei php Dateien und dann auch noch gleich an mehreren Stellen ... .

Magst Du Dir die drei php`s nicht mal im Ganzen anschauen Bonsai? Das gibt doch mehr Sinn.

Ich habe das Package hier nochmal mit angehangen.

[Bonsai]

Ich hab keine Zeit mich da komplett durchzuwühlen und mich da reinzudenken. Ich mach mal einen Schuss ins blaue, eventuell ist es gleich richtig:
wish_list.php ab Zeile 259:

Code: PHP  [Auswählen]

  function get_products() {
    global $xtPrice;
    if (!is_array($this->contents)) return false;
    $products_array = array();
    reset($this->contents);
    while (list($products_id, ) = each($this->contents)) {
      $products_query = xtc_db_query("select p.products_id, p.products_status, pd.products_name,p.products_image, p.products_model, p.products_price, p.products_discount_allowed, p.products_weight, p.products_tax_class_id from " . TABLE_PRODUCTS . " p, " . TABLE_PRODUCTS_DESCRIPTION . " pd where p.products_id='" . xtc_get_prid($products_id) . "' and  p.products_status = '1' and pd.products_id = p.products_id and pd.language_id = '" . $_SESSION['languages_id'] . "'");
      if ($products = xtc_db_fetch_array($products_query)) {
        $prid = $products['products_id'];
        $products_price = $xtPrice->xtcGetPrice($products['products_id'],
        $format=false,
        $this->contents[$products_id]['qty'],
        $products['products_tax_class_id'],
        $products['products_price']);
        $products_array[] = array('id' => $products_id,
                'name'         => $products['products_name'],
        'products_status'         => $products['products_status'],
        'model'        => $products['products_model'],
        'image'        => $products['products_image'],
        'price'        => $products_price+$this->attributes_price($products_id),
        'quantity'     => $this->contents[$products_id]['qty'],
        'weight'       => $products['products_weight'],
        'final_price'  => ($products_price+ $this->attributes_price($products_id)),
        'tax_class_id' => $products['products_tax_class_id'],
        'attributes'   => $this->contents[$products_id]['attributes']);
      }
    }
    return $products_array;
  }
 

Du müsstest den Datenbankquery anpassen

Code: PHP  [Auswählen]

      $products_query = xtc_db_query("select p.products_id, p.products_status, pd.products_name,p.products_image, p.products_model, p.products_price, p.products_discount_allowed, p.products_weight, p.products_tax_class_id from " . TABLE_PRODUCTS . " p, " . TABLE_PRODUCTS_DESCRIPTION . " pd where p.products_id='" . xtc_get_prid($products_id) . "' and  p.products_status = '1' and pd.products_id = p.products_id and pd.language_id = '" . $_SESSION['languages_id'] . "'");

also da muss noch ein p.products_vpe oder pd.products_vpe rein (ich weiß jetzt nicht auswendig in welcher der beiden Tabellen das ist, schau nach!)

und ein paar Zeilen tiefer unter

Code: PHP  [Auswählen]

        $products_array[] = array('id' => $products_id,
                'name'         => $products['products_name'],
        'products_status'         => $products['products_status'],
        'model'        => $products['products_model'],
 

die Zeile:

Code: PHP  [Auswählen]

'vpe'        => $products['products_vpe'],

mit

Code: PHP  [Auswählen]

return $products_array;

wird das ganze zurückgegeben an was auch immer, das die liste aus der DB holen will und Du hast somit eine

Code: PHP  [Auswählen]

$products[<irgendwas>]['vpe']

, die Du ja brauchst.

[WeXsler]

danke Dir! Ich hab`s versucht, aber da mache ich wohl was falsch.

Was ich getan habe und NICHT funktioniert:

in der Z. 263
/includes/classes/wish_list.php
den code ergänzt sowohl einmal mit p.products_vpe und der zweite Versuch mit pd.products_vpe. Letzteres erbrachte eine Fehlermeldung, dass pd.products_vpe nicht bekannt sei.

Code: PHP  [Auswählen]

while (list($products_id, ) = each($this->contents)) {
      $products_query = xtc_db_query("select p.products_id, pd.products_name, p.products_image, p.products_model, p.products_price, p.products_vpe, p.products_discount_allowed, p.products_weight, p.products_tax_class_id from " . TABLE_PRODUCTS . " p, " . TABLE_PRODUCTS_DESCRIPTION . " pd where p.products_id='" . xtc_get_prid($products_id) . "' and pd.products_id = p.products_id and pd.language_id = '" . $_SESSION['languages_id'] . "'");
      if ($products = xtc_db_fetch_array($products_query)) {

in der Z. 48
/includes/modules/order_details_wishlist.php
habe ich in Z. 59 den code mit

Code: PHP  [Auswählen]

'vpe'                        => $products['products_vpe'],

ergänzt - s. code unten.

Code: PHP  [Auswählen]

$module_content[$i]=array(
  'PRODUCTS_NAME'              => $products[$i]['name'].$mark_stock,
  'PRODUCTS_QTY'               => xtc_draw_input_field('cart_quantity[]', $products[$i]['quantity'], 'size="2"') . xtc_draw_hidden_field('products_id[]', $products[$i]['id']),
  'PRODUCTS_MODEL'             => $products[$i]['model'],
  'PRODUCTS_TAX'               => number_format($products[$i]['tax'], TAX_DECIMAL_PLACES),
  'PRODUCTS_IMAGE'             => $image,
  'IMAGE_ALT'                  => $products[$i]['name'],
  'BOX_DELETE'                 => xtc_draw_checkbox_field('cart_delete[]', $products[$i]['id']),
  'DELETE_ICON'                => '<a href="'. xtc_href_link(FILENAME_SHOPPING_CART, 'action=update_product_cart&products_id='.$products[$i]['id']). '">'.  xtc_image('templates/'.CURRENT_TEMPLATE.'/img/button_cart_delete.jpg', 'l&ouml;schen'). '</a>',
  'PRODUCTS_LINK'              => xtc_href_link(FILENAME_PRODUCT_INFO, 'products_id=' . $products[$i]['id']),
  'PRODUCTS_PRICE'             => $xtPrice->xtcFormat($products[$i]['price']*$products[$i]['quantity'],true),
  'vpe'                        => $products['products_vpe'],
  'PRODUCTS_SINGLE_PRICE'      => $xtPrice->xtcFormat($products[$i]['price'],true),
  'PRODUCTS_SHORT_DESCRIPTION' => strip_tags(xtc_get_short_description($products[$i]['id'])),
  'ATTRIBUTES' => '',
  'BUY_NOW' => '<br /><a href="' . xtc_href_link(basename($PHP_SELF), xtc_get_all_get_params(array('action')) . 'action=buy_now&BUYproducts_id=' . $products[$i]['id'], 'NONSSL') . '">' . xtc_image_button('button_buy_now_small.gif', TEXT_BUY . TEXT_NOW)
  );

in der Z. 15
/templates/xtc5/module/wish_list_order_details.html
habe ich in Z. 18

Code: PHP  [Auswählen]

{if $module_data.PRODUCTS_VPE}({$module_data.PRODUCTS_VPE}){/if}<br />

ergänzt. Siehe code unten.

Code: PHP  [Auswählen]

<td align="left" valign="top" class="main">{if $module_data.PRODUCTS_IMAGE!=''}<img src="{$module_data.PRODUCTS_IMAGE}" width="60" height="60" border="0">{/if}</td>
    <td align="left" valign="top" class="main"><div align="center">{$module_data.PRODUCTS_QTY}</div></td>
    <td valign="top" class="main"><strong><a href="{$module_data.PRODUCTS_LINK}">{$module_data.PRODUCTS_NAME}</a></strong>{$module_data.PRODUCTS_SINGLE_PRICE}<br />
        {if $module_data.PRODUCTS_VPE}({$module_data.PRODUCTS_VPE}){/if}<br />
      {if $module_data.ATTRIBUTES!=''}

Ergebnis: Merkzettel wird angezeigt wie zuvor. Keine Grundpreisanzeige, keine Anzeige inkl. Mwst. zzgl. Versand. Also kein Effekt

[Bonsai]

Ja sorry, da fehlte ein Schritt, bist aber schon richtig!

Beim ersten Schritt (SQL) stimmt alles
Direkt unter dem SQL kommt das:

Code: PHP  [Auswählen]

if ($products = xtc_db_fetch_array($products_query)) {

Hier kommt der zweite Schritt (Datenbank => products array)

Code: PHP  [Auswählen]

    'vpe'        => $products['products_vpe'],

Und der dritte Schritt (Products_array => smarty modul array) in dem Block

Code: PHP  [Auswählen]

$module_content[$i]=array(

das rein:

Code: PHP  [Auswählen]

  'PRODUCTS_VPE'                  => $products[$i]['vpe'],

Dann sollte Schritt 4, das auslesen im Smarty funktionieren:

Code: PHP  [Auswählen]

{if $module_data.PRODUCTS_VPE}{$module_data.PRODUCTS_VPE}{/if}<br />

Achtung! Die runden Klammern bei dir im Code müssen da nicht hin!

Was mir jetzt erst aufgefallen ist, (hatte mit den VPE bisher kaum was zu tun) Du brauchst noch mehr!
Das hättest du selbst feststellen können, wenn Du dir die products Tabelle angesehen hättest.

Was in PRODUCTS_VPE drinsteht, wenn es funktioniert wie beschrieben ist eine Zahl, die Nummer der VPE
In der Tabelle products stehen diese 3 Spalten:
products_vpe
products_vpe_status
products_vpe_value
In der Tabelle products_vpe stehen diese 2 Spalten:
products_vpe_id
language_id
products_vpe_name

Was Du rausbekommst mit dem Code oben ist erstmal nur die ID der VPE. Den Wert, und den Status brauchst du auch! Das funktioniert genauso!
Komplizierter wird dann aus der ID (z.B. 1) den Text rauszubekommen. (z.B. Meter oder Liter, Waschladung etc.)

Der SQL Query braucht noch einen JOIN um die dritte Tabelle auch mitzulesen, oder (MURKS) Du baust eine zweite Abfrage. Ich würde sagen, komm erst mal soweit, dass die Zahl angezeigt wird, dann machen wir den nächsten Schritt. Da müsstest Du auch mal jemand anders fragen. Mit JOINs stehe ich auf dem Kriegsfuß. Ich müsste das ganze durch ausprobieren herausfinden. Ich vermute mal, das gäbe einen INNER JOIN