Kleiner Bug beim Artikel duplizieren

[bluepepper]

Hallo Liebe Gemeide,

wie mir gerade auffällt ist beim duplizieren von Artikeln ein Bug(chen) drin.
Betrifft v1.06 rev 4642 (frühere kann ich nicht sagen)

Beim betrachten der Bestseller tauchte ein neuer Artikel ganz oben auf.
Ursache ist, das beim duplizieren eines Artikels auch die products_ordered mit dupliziert werden.

sucht in admin/includes/classes/categories.php in der function duplicate_product nach

Code: PHP  [Auswählen]

$sql_data_array['products_status'] = $products_status;

und fügt danach folgendes ein.

Code: PHP  [Auswählen]

$sql_data_array['products_ordered'] = '0';

dann stimmen die Bestseller auch.

BTW:
Hab ich auch noch in der admin/includes/modules/categories_view.php folgende ergänzung drin,
die die Anzahl der verkäufe anzeigt. ist ganz hilfreich.

sucht nach

Code: PHP  [Auswählen]

$select_str = "SELECT p.products_tax_class_id,
                                      p.products_sort,
                                      p.products_id,
                                      pd.products_name,
                                      p.products_quantity,
                                      p.products_image,
                                      p.products_model,
                                      p.products_price,
                                      p.products_discount_allowed,
                                      p.products_date_added,
                                      p.products_last_modified,
                                      p.products_date_available,
                                      p.products_status,
                                      p.products_startpage,
                                      p.products_startpage_sort
                                 FROM " . TABLE_PRODUCTS . " p
                            LEFT JOIN " . TABLE_PRODUCTS_DESCRIPTION . " pd ON p.products_id = pd.products_id AND pd.language_id = '" . (int)$_SESSION['languages_id'] . "'
                                      " . $add_join . $add_where ."
                             ORDER BY " . $prodsort;

und ersetzt mit

Code: PHP  [Auswählen]

$select_str = "SELECT p.products_tax_class_id,
                                      p.products_sort,
                                      p.products_id,
                                      pd.products_name,
                                      p.products_quantity,
                                      p.products_ordered,
                                      p.products_image,
                                      p.products_model,
                                      p.products_price,
                                      p.products_discount_allowed,
                                      p.products_date_added,
                                      p.products_last_modified,
                                      p.products_date_available,
                                      p.products_status,
                                      p.products_startpage,
                                      p.products_startpage_sort
                                 FROM " . TABLE_PRODUCTS . " p
                            LEFT JOIN " . TABLE_PRODUCTS_DESCRIPTION . " pd ON p.products_id = pd.products_id AND pd.language_id = '" . (int)$_SESSION['languages_id'] . "'
                                      " . $add_join . $add_where ."
                             ORDER BY " . $prodsort;

sucht nach

Code: PHP  [Auswählen]

// END IN-SOLUTION

davor einfügen

Code: PHP  [Auswählen]

$contents[] = array('text' => '<div style="padding-left: 30px;">Verkauft:&nbsp;' . $pInfo->products_ordered . '</div>');

Gurß Martin


Linkback: https://www.modified-shop.org/forum/index.php?topic=29533.0

[Tomcraft]

Ich habe das in Ticket #216 festgehalten.

Grüße

Torsten

[colibri]

Hallo zusammen,

beim Duplizieren von Artikeln taucht noch ein weiterer Fehler auf: offenbar wird bei Original und Duplikat jeweils das Einstellungsdatum vertauscht. Für diesen Test habe ich mal im Demoshop den Test-Artikel 002 dupliziert und das Duplikat dann mit der neuen Artikel-Nummer 1234556789 versehen.

Nun sollte ja der zweite Artikel neuere Einstellungsdaten aufweisen - ist aber leider nicht so. Dies kann anhand der beigefügten Screenshots aus dem Demoshop nachvollzogen werden:

auswahl-original-artikel.jpg -> zeigt rechts unter "Informationen", dass der Artikel angeblich am 08.04.2014 hinzugefügt wurde, obwohl es sich um dem original Artikel des Demoshops handelt (Artikel-Nr. 002)

auswahl-duplizierter-artikel.jpg -> zeigt rechts unter "Informationen", dass der Artikel anscheinend schon am 14.06.2099 hinzugefügt wurde (obwohl es der neu duplizierte Artikel mit der Artikel-Nummer 1234556789 ist)

Das Ganze vererbt sich weiter nach unten, wenn der duplizierte Artikel dann nochmals dupliziert wird ... (siehe dritter-duplizierter-artikel.jpg) ...

Das hat z.B. zur Folge, dass mittels Duplizierung erstellte, neue Artikel nicht via products_new.php angezeigt werden.

Aus meiner Sicht ein Bug, zu  dem ich leider auch keine weiteren Infos/Hilfestellungen/Lösungen über die Forensuche nach "duplizieren" finden konnte.

Schöne Grüße,

colibri

[noRiddle (revilonetz)]

Du hast die Artikel schlicht vertauscht (beim Vergeben der Artikelnummer).
Der Duplizierte, also der neue durch Duplizieren generierte Artikel, ist der mit dem neuen Datum.
Das erkennt man an der roten "Ampel" bei TOP, das ist immer der neue Artikel wenn der alte in TOP gelistet war.

Das habe ich jetzt nicht nachgeprüft sondern anhand deiner Screenshots gesehen/gedeutet.
Kannst du das nochmal genauer verifizieren ?
Teste es mal mit einem Artikel der Attribute hat und kopiere diese nicht mit (Haken unten raus).
Dann siehst du eindeutig welcher der neue Artikel ist, nämlich der ohne Attribute.

Gruß,
noRiddle

[colibri]

Hi noRiddle,

nein, da ist nichts vertauscht - wie auch schon schrob und anhand der Screenshots nachvollziehbar ist:

Der original Artikel hat zwar immer noch seine Artikel-Nummer 002, ist aber lt. Screenshot am 08.04.2014 hinzugefügt worden. Während der duplizierte Artikel angeblich am 14.06.2009 hinzugefügt wurde - also somit älter ist ist als der originale Artikel.

Bis auf das händische Ändern der Artikel-Nr. (zu Unterscheidungszwecken) ist da nix von "mir" gemacht bzw. "vertauscht" worden.

Schöne Grüße,

Marc

[Ceciro]

Hallo Marc,

nicht die Artikelnummer gibt an, welcher Artikel zuerst da war, sondern die ID in der Browserzeile. Der duplizierte Artikel hat ja zunächst die selbe Artikelnummer, wie der Originale, aber eine andere, höhere ID.

Auf den Bildern hat der Artikel 002 die ID59 und der Neue 123456789 die ID2.

Darauf muss man penibel achten, weil der Shop mal den alten, und mal den neuen Artikel oben in der Liste anzeigt.

Gruß Cicero

[colibri]

Hallo Cicero,

danke für den Hinweis. Ich hatte mich daneben auch darauf verlassen, dass das angekreuzte "Seitenaufruf nach dem Kopieren/Verlinken" die Eingabemaske des duplizierten Artikels öffnet. Es wurde allerdings jedes Mal der originale Artikel zum Bearbeiten aufgerufen, so dass ich diesem dann die neue Artikel-Nr. verpasst habe ... nun ja.

Aber ok, damit wäre das für mich geklärt. Danke für die Hinweise.

Schöne Grüße,

Marc

[noRiddle (revilonetz)]

Sagte ich doch, vertauscht
Cicero hat's aber besser erklärt.

Gruß,
noRiddle

[piru]

Ich hatte mich daneben auch darauf verlassen, dass das angekreuzte "Seitenaufruf nach dem Kopieren/Verlinken" die Eingabemaske des duplizierten Artikels öffnet. Es wurde allerdings jedes Mal der originale Artikel zum Bearbeiten aufgerufen, so dass ich diesem dann die neue Artikel-Nr. verpasst habe ... nun ja.

So kannst du das ändern: Artikel duplizieren - Details des alten Artikels werden verändert

Gruß piru

[Tomcraft]

Ich habe das in Ticket #216 festgehalten.
[...]

Der Fehler wurde bereits korrigiert, siehe Ticket. Die Lösung ist aber eine andere als hier in Beitrag 1 beschrieben.

Grüße

Torsten

[kaisa]

Gute Hinweise, habe ich aber leider erst spät entdeckt.

Wie kann ich nun herausfinden, wo oft ein Produkt wirklich verkauft wurde und die Datenbank korrigieren? Habe ca. 200 Produkte.