send_order.php unnötigen Code entfernen

[FräuleinGarn]

Durch ein anderes Thema wo es um einen Fehler mit PayPal IPN ging, hab ich mal die neueste Version danach durchsucht.

​PayPal E-Mail: Überprüfen Sie den Server für sofortige Zahlungsbestätigungen

Dabei ist mir folgender Code aufgefallen

//BOF - web28 - 2010-06-11 - Send Order  by Admin Paypal IPN
        if(isset($send_by_admin)) { //DokuMan - 2010-09-18 - Undefined variable: send_by_admin
            require (DIR_FS_CATALOG_MODULES.'payment/paypal_ipn.php');
            include(DIR_FS_LANGUAGES.$order->info['language'].'/modules/payment/paypal_ipn.php');
            $payment_modules = new paypal_ipn;
        }
        //EOF - web28 - 2010-06-11 - Send Order  by Admin Paypal IPN

Allerdings gibt es keine Datei "includes/modules/payment/paypal_ipn.php" in 2.0.5.0, insofern müsste der Teil entweder überarbeitet oder entfernt werden.

[Torsten Riemer]

Der Code existiert bereits seit Shopversion 2.0.0.0 rev 9678 nicht mehr in der "send_order.php"!

[FräuleinGarn]

Sorry. Handelt sich um die Datei "includes/external/micropayment/send_order.php", wo es so noch in 2.0.5.0 drin steht.

[Torsten Riemer]

Es gibt sogar noch mehr Dateien, die "paypal.ipn" einbinden, aber die fragen alle ab, ob das Modul überhaupt vorhanden ist. Daher sehe ich das völlig unkritisch.

[FräuleinGarn]

Ok. Ging nur darum unnötigen Code zu entfernen. Aber die Datei ist wirklich nicht wichtig. Wenn vorher abgefragt wird, ob Modul vorhanden, dann kann es ja nicht zu Fehlermeldungen kommen. Hatte ich nur entdeckt weil wie im Link des ersten Beitrags der Nutzer eine Fehlermeldung von Paypal erhält wo

statt

​https://domain/callback/paypal/paypalipn.php

dieser Link drin steht.

​https://domain/callback/paypal/ipn.php

[Torsten Riemer]

Zu einem Fehler kann es nur dann kommen, wenn eine ganz alte Bestellung aufgerufen wird, die mittels "paypal_ipn.php" Zahlungsmodul getätigt wurde, aber selbst dann sollte das nur einen Eintrag im Log erzeugen.