Synchronisation Artikel / richtiges Löschen der Artikel

[vowawi]

Hi,
bei der Webshop-Synchronisation (insbesondere Artikel) sind mir folgende Punkte aufgefallen:
1. Gelöschte Artikel (in JTL-Wawi) werden auf VM-Seite nicht gelöscht, sondern diese Produkte werden auf unpublished gesetzt.
2. Werden Produkte in virtuemart gelöscht, werden die Artikel bei JTL-Wawi nicht gelöscht.
3. Eine erneute Synchronisation legt diese Produkte (siehe 2.) nicht wieder an, da die Mapping-Tabelle (jtlwawi_martikel) die Zuordnung zu dem gelöschten Produkt noch enthält.

Für mich stellt die Wawi das führende System dar, somit stelle ich mal dies zur Diskussion:
zu 1. VM-Produkte sollten, wenn sie aus der Wawi entfernt wurden ebenso gelöscht werden.
zu 2. Das ist gut so, da die Änderungen im Shop keine Auswirkungen auf die Wawi haben sollten.
zu 3. Produkte sollten wieder über die Synchronisation angelegt werden, auch wenn sie mal im VM-Shop gelöscht wurden.

Was meint Ihr ?

Gruß Volker

 

[Guenter H.]

Re: Synchronisation Artikel

hi,

Hi,
bei der Webshop-Synchronisation (insbesondere Artikel) sind mir folgende Punkte aufgefallen:
1. Gelöschte Artikel (in JTL-Wawi) werden auf VM-Seite nicht gelöscht, sondern diese Produkte werden auf unpublished gesetzt.

das wird auch beim XTC- shop so gehandhabt. es gibt aber eine connector-anpassung, wo beim löschen von artikel in der wawi diese auch aus der shop-db gelöscht werden. connector anpassung XTC

2. Werden Produkte in virtuemart gelöscht, werden die Artikel bei JTL-Wawi nicht gelöscht.

änderungen an artikel sollten ausschließlich über die wawi durchgeführt werden, da es ansonsten zu problemen kommen kann. siehe zb. hier: löschen von artikel im shop-admin

3. Eine erneute Synchronisation legt diese Produkte (siehe 2.) nicht wieder an, da die Mapping-Tabelle (jtlwawi_martikel) die Zuordnung zu dem gelöschten Produkt noch enthält.

siehe punkt 2

grüße

günter

 

[vowawi]

Löschen von VM-Produkten statt unpublished

Hallo Günter,
vielen Dank für deine Hilfe sowie die hilfreichen Links.
Das Löschen von Produkten im VM- Shop werde ich somit organisatorisch sicherstellen, ist okay so.
Die Sache mit der Connector-Anpassung ist für mich ein bischen schwieriger, da ich noch zu wenig php-Kenntnisse habe. Nun habe ich mal folgenden Ansatz gewählt:
Artikel.php ab Zeile 116

		if ($product_id>0)
// Produkte nicht veröffentlichen, wenn in Wawi gelöscht
//			eS_execute_query("update ".DBPVM."product set product_publish=\"N\" where vendor_id=".VENDOR_ID." and product_id=".$product_id);
// Produkte löschen, wenn in Wawi gelöscht
/* Attribute löschen */
			eS_execute_query("delete from ".DBPVM."product_attribute_sku where product_id=".$product_id); 
/* Kategorien löschen */
			eS_execute_query("delete from ".DBPVM."product_category_xref where product_id=".$product_id);
/* Hersteller-Referenz löschen */
			eS_execute_query("delete from ".DBPVM."product_mf_xref where product_id=".$product_id);							
/* ProduktType-Relationen löschen */
			eS_execute_query("delete from ".DBPVM."product_product_type_xref where product_id=".$product_id);							
/* Produkt-Votes löschen */
			eS_execute_query("delete from ".DBPVM."product_votes where product_id=".$product_id);							/* Produkt-reviews löschen */
			eS_execute_query("delete from ".DBPVM."product_reviews where product_id=".$product_id);
/* Produkt-Relationen löschen */
			eS_execute_query("delete from ".DBPVM."product_relations where product_id=".$product_id);						
/* Produkt-Preise löschen */
			eS_execute_query("delete from ".DBPVM."product_price where product_id=".$product_id);
/* Produkt löschen */
			eS_execute_query("delete from ".DBPVM."product where vendor_id=".VENDOR_ID." and product_id=".$product_id); 
/* Mapping löschen */
			eS_execute_query("delete from jtlwawi_martikel where product_id=".$product_id); 			

 							
		$return = 0;

Den ursprünglichen Update habe ich auskommentiert und die DELETE-Anweisungen eingefügt, mit folgendem Ergebnis:
1. Artikel in Wawi löschen => Produkt in VM wird gelöscht

2. Artikel in Wawi inaktiv => Produkt in VM wird gelöscht. Okay, damit kann ich leben, obwohl es mir lieber wäre, wenn ich in der artikel.php das Artikel-Kennzeichen <inaktiv> zur Verfügung hätte. Dann könnte ich den Fall unterscheiden und das Produkt auf "unpublished" setzen. Fällt aber in die Rubrik nice-to-have

3. Inaktiven Artikel in Wawi wieder auf aktiv setzen => Produkt wird in VM angelegt, nur sind keine Produktbilder enthalten. Diese werden erst hochgeladen, wenn ich sie in der wawi nochmals zuordne. Muss ich da andes vorgehen ?

4. Was ich beim Löschen noch gar nicht behandelt habe (mangels php-Kenntnisse), ist das Entfernen der zugehörigen Image-Dateien aus dem Shop-Verzeichnis. Falls da noch jemand ein Tipp hätte.

Danke und Gruß
Volker

 

[Thomas Lisson]

Hallo Volker,

hast Du doch ganz gut hinbekommen.

Artikel in Wawi inaktiv => Produkt in VM wird gelöscht. Okay, damit kann ich leben, obwohl es mir lieber wäre, wenn ich in der artikel.php das Artikel-Kennzeichen <inaktiv> zur Verfügung hätte.

Beide Ereignisse lösen dasselbe aus, nämlich action=3. Du könntest schummeln und etwa die Mindestbestellmenge für die Deaktivierung des Artikels nutzen. Die Mindestbestellmenge hat nämlich keine Funktion im Shop, wird aber übertragen.

Also zB: Sobald Mindestbestellmenge = 5, dann Produkt auf inaktiv setzen...

4. Was ich beim Löschen noch gar nicht behandelt habe (mangels php-Kenntnisse), ist das Entfernen der zugehörigen Image-Dateien aus dem Shop-Verzeichnis. Falls da noch jemand ein Tipp hätte.

Ob es so wichtig ist, ist eine andere Sache. Platz auf dem Server kostet nichts mehr.

so löschst Du die Bilder auf der Platte:

if (file_exists(IMAGEPATH."product/".$product_id.".jpg"))
  unlink (IMAGEPATH."product/".$product_id.".jpg");
//vorschaubild löschen
if (file_exists(IMAGEPATH."product/".$product_id."_klein.jpg"))
  unlink (IMAGEPATH."product/".$product_id."_klein.jpg");

 

[vowawi]

Vielen Dank

Hallo Thomas,

vielen Dank für die nützlichen Tipps, die Bilder werden nun gelöscht, den Trick mit der Mindestbestellmenge kann ich so nutzen. Nach wie vor wäre es mir lieber, wenn ich den Fall eindeutig unterscheiden könnte, z.B. über das Feld inaktiv oder über ein eigenes Ereignis (action=?). Vielleicht in einem späteren Release .

Jetzt bräuchte ich nur noch'n Tipp für den 3. Punkt:

3. Inaktiven Artikel in Wawi wieder auf aktiv setzen => Produkt wird in VM angelegt, nur sind keine Produktbilder enthalten. Diese werden erst hochgeladen, wenn ich sie in der wawi nochmals zuordne. Muss ich da andes vorgehen ?

Gruß Volker

 

[Thomas Lisson]

Hi,

3. Inaktiven Artikel in Wawi wieder auf aktiv setzen => Produkt wird in VM angelegt, nur sind keine Produktbilder enthalten. Diese werden erst hochgeladen, wenn ich sie in der wawi nochmals zuordne. Muss ich da andes vorgehen ?

Nein, das ist derzeit der einzige Weg...

 

[vowawi]

date of creation / ~ modification

Schade, aber das Wörtchen <derzeit> lässt ja hoffen ... .

Weitere Anregungen / Verbesserungsvorschläge meinerseits:

Die DB-Felder der product-Tabelle
cdate ... date of creation (Timestamp)
mdate ... date of modification (Timestamp)
werden im Moment bei der Synchronisation nicht befüllt.
Dadurch wird nichts angezeigt bzw. sind solche Abfragen, wie z.B. "neueste Artikel der letzten 10 Tage" nicht möglich.

In der Artikel.php habe ich folgende Änderungen gemacht:
1. Variable eingefügt:
$timestamp = time();

2. DB-Update (ab Zeile 61) sieht nun so aus:

eS_execute_query("update ".DBPVM."product set 
									product_sku=\"".$artikel->cArtNr."\",
									product_s_desc=\"".$artikel->cKurzBeschreibung."\",
									product_desc=\"".$artikel->cBeschreibung."\",
									product_publish=\"Y\",
									product_weight=\"".$artikel->fGewicht."\",
									product_weight_uom=\"kg\",
									product_in_stock=\"".$artikel->nLagerbestand."\",
									product_availability=\"".$einstellungen->liefertermin."\",
									product_special=\"".$artikel->cTopArtikel."\",
									product_name=\"".$artikel->cName."\",
									product_tax_id=\"".$tax_rate_id."\",
									product_unit=\"".$artikel->cEinheit."\",
									mdate=".$timestamp."									
								where
									vendor_id=".VENDOR_ID." and
									product_id=".$product_id);

3. Der DB-Insert (ab Zeile 93) so:

eS_execute_query("insert into ".DBPVM."product (product_sku, product_s_desc, product_desc, product_publish, product_weight, 
product_weight_uom,product_in_stock, product_availability, product_special, product_name, product_tax_id, product_unit, 
vendor_id, cdate, mdate) values 
(\"".$artikel->cArtNr."\",\"".$artikel->cKurzBeschreibung."\",\"".$artikel->cBeschreibung."\",\"Y\",\"
".$artikel->fGewicht."\",\"kg\",\"".$artikel->nLagerbestand."\",\"".$einstellungen->liefertermin."\",\"
".$artikel->cTopArtikel."\",\"".$artikel->cName."\",\"".$tax_rate_id."\",\"".$artikel->cEinheit."\"
,".VENDOR_ID.",".$timestamp.",".$timestamp.")");

Meine Tests waren erfolgreich, vielleicht könnt Ihr mal eine Qualitätssicherung vornehmen bzw. den Code übernehmen.

Grüße Volker

 

[Thomas Lisson]

Danke Dir Volker

 

[Andreas Grambow]

Das Löschen habe ich in eine vorläufige datei eingebaut. weitere Änderungen folgen. UST ist auch angepasst.

Die Link zur geänderten datei findet Ihr hier