Speichern/Aktualisieren eines Artikels führt zum aktivieren des Artikels

[akustikstoff]

Speichert man einen inaktiven Artikel wird dieser auf "Aktiv" gesetzt. Dies geschieht scheinbar nur mit Artikeln die sich in der Kategorie "Top" befinden.

[Torsten Riemer]

Das Verhalten kann ich im ​Stable-Demoshop nachvollziehen, vielen Dank für die Meldung.
Es muss dafür bei der Artikelanlage "Artikelstatus" auf "Deaktiviert" und "Auf Startseite zeigen" auf "Ja" gestellt werden um das nachvollziehen zu können.

[Gerhard Waldemair]

In 14689:

fix #2287 - status after editing startpage products

[noRiddle]

Im Thread ​Artikel deaktivieren | Antort #7 habe ich dazu noch etwas bemerkt.
Wenn es sich verhält wie Torsten einen Post höher schrieb, daß nämlich die zwei Vorkommen von set_product_status() entfernt wurden, frage ich mich halt dies:
"
[...] verstehe ich nicht warum man die Funktionen einfach entfernt, anstatt der Funktion link_product() einen weiteren Parameter zu geben der bestimmt ob set_product_status() ausgeführt werden soll.
Das Entfernen der Funktion aus der Methode move_product() ist doch außerdem überflüssig ?
"

Gruß,
noRiddle

[Torsten Riemer]

Wozu? Was willst du damit bezwecken?
Da ist keine Entfernung überflüssig, sonst würde ein inaktiver Artikel nach dem Verschieben aktiv werden.
Der Fix ist nach unserer Ansicht richtig und vollkommen ausreichend.

[noRiddle]

Nun, ich denke, daß die Funktion vorher nicht ohne Grund eingefügt wurde. Dann hat man seine Meinung darüber wohl geändert.

Gruß,
noRiddle

[Torsten Riemer]

Nein, die Funktion link_product() ist zur Verlinkung von Artikeln gedacht, set_product_status() ändert den Status eines Artikels.
Wir sehen keine Notwendigkeit der Funktion link_product() einen weiteren Parameter mitzugeben, mit dem man den Status von Artikeln ändern kann.

Die Funktion so zu benutzen in den Funktionen link_product() & move_product macht doch keinen Sinn:

$this->set_product_status($src_products_id, 1);

Das würde wie gesagt dazu führen, dass Artikel automatisch aktiviert werden, was man ggf. nicht möchte.
Wenn überhaupt, dann so:

$this->set_product_status($src_products_id, $products_status);

Aber auch das ist einfach nicht nötig.