Redirect-Befehle in Klassenmodul

[ChristianRothe]

Im Klassenmodul /admin/includes/classes/categories.php finden sich in den Funktionen insert_product() und remove_category() Redirect-Anweisungen: xtc_redirect(...).

Beispiel:

Code: PHP  [Auswählen]

    //redirect by update button
    if(isset($products_data['prod_update'])) {
      xtc_redirect(xtc_href_link(FILENAME_CATEGORIES, 'cPath='.$_GET['cPath'].'&action=new_product&pID='.$products_id.$this->page_parameter));
    }
    xtc_redirect(xtc_href_link(FILENAME_CATEGORIES, xtc_get_path($dest_categories_id).'&pID='.$products_id.$this->page_parameter)); //web28 2012-04-14 new redirect
 

Meines Erachtens hat ein derartiger Redirect in einem Klassenmodul nichts verloren und gehört dort entfernt. Die Ablaufsteuerung sollte im aufrufenden Modul erfolgen, wo die Funktion aus dem Klassenmodul angesprochen und verwendet wird.

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

[baalze]

Wofür. Ich bezweifle sehr, dass man dann als Shopbetreiber mehr verkauft.

[ChristianRothe]

Wofür? Weil die aktuelle Umsetzung die Idee ad absurdum führt, Programmlogik in Klassen zu kapseln, damit sie an mehreren Stellen im Shop eingebunden und verwendet werden kann. Dies wird nämlich durch den Einbau von hardcodierten Redirects in die Funktionen in der Klasse sehr zuverlässig verhindert.

Das Argument "Mit sauberer Programmierung verkauf'ste ooch nich möhr..." ist natürlich ein Totschlagargument vom Feinsten. Gratulation. Damit kann man Pfusch jeglicher Couleur prima aus der Welt diskutieren.

[web28]

In 2.00 bereits entfernt. 

[Bonsai]

Das Argument "Mit sauberer Programmierung verkauf'ste ooch nich möhr..." ist natürlich ein Totschlagargument vom Feinsten. Gratulation. Damit kann man Pfusch jeglicher Couleur prima aus der Welt diskutieren.

   

[ChristianRothe]

In 2.00 bereits entfernt. 

2.00? Was war das nicht noch mal gleich?