Gesammelte Fehler in PayPal Modulen

[Torsten Riemer]

Bitte mal hier schauen:

• /admin/paypal_profile.php -> ​MODUL: PayPal PLUS Zahlungsmodul für modified eCommerce Shopsoftware

• /callback/paypal/paypalplus.php & /includes/external/paypal/classes/PayPalCommon.php -> ​MODUL: PayPal PLUS Zahlungsmodul für modified eCommerce Shopsoftware

• Fehlerhaftes Datum bei Banküberweisung: ​MODUL: PayPal PLUS Zahlungsmodul für modified eCommerce Shopsoftware

• Profile werden nicht gelöscht (Legt man einmal Profile an, dann lassen sich die Daten nicht mehr löschen. -> PAYPAL_STANDARD_PROFILE sowie Inhalt aus PAYPAL_'.strtoupper($this->code.'_'.$_SESSIONlanguage_code).'_PROFILE bleiben in Tabelle paypal_config und verursachen so Fehler (Logs vom 04.07.2016 per Mail an GTB)

• Opera hat Probleme mit <script type="application/javascript"> (Sollte ersetzt werden mit <script type="text/javascript"> in Dateien /callback/paypal/paypalplus.php & /callback/paypal/paypalpluslink.php), siehe dazu: ​MODUL: PayPal PLUS Zahlungsmodul für modified eCommerce Shopsoftware

[Gerhard Waldemair]

In 10051:

fix #924 & #933 - update paypal

[Gerhard Waldemair]

In 10052:

fix #933 - update paypal

[Gerhard Waldemair]

In 10053:

fix #933 - update paypal

[Gerhard Waldemair]

In 10054:

fix #933 - update paypal

[Gerhard Waldemair]

In 10055:

fix #933 - update paypal

[Torsten Riemer]

Es scheint trotz Update des Moduls weiterhin Probleme zu geben, siehe dazu:

• ​http://www.modified-shop.org/forum/index.php?topic=34597.msg322304#msg322304
• ​http://www.modified-shop.org/forum/index.php?topic=35315.msg322271#msg322271
• ​http://www.modified-shop.org/forum/index.php?topic=34597.msg322365#msg322365
• Das Feld "Angezeigter Name" bei den Profilen sollte mit einer Maximal-Länge versehen werden, siehe dazu: ​http://www.modified-shop.org/forum/index.php?topic=34597.msg322372#msg322372

[Gerhard Waldemair]

In 10061:

fix #933 - update paypal

[Gerhard Waldemair]

In 10062:

fix #933 - update paypal

[Gerhard Waldemair]

In 10063:

fix #933 - update paypal & update express checkout

[Gerhard Waldemair]

In 10064:

fix #933 - update paypal

[Torsten Riemer]

In 10065:

Minor addition on r10062 (fix #933 - update paypal)

[Gerhard Waldemair]

In 10066:

fix #933 - update paypal

[Torsten Riemer]

Es gibt weitere Fehlermeldungen:

• Die Versandkosten werden bei "paypalcart" nicht an PayPal übermittelt, siehe: ​MODUL: PayPal PLUS Zahlungsmodul für modified eCommerce Shopsoftware
• Es darf kein neuer access_token angefordert werden, solange der alte noch gültig ist, da dies sonst zu einer Sperrung führen kann, siehe dazu: ​Fehler bei PayPal Rest API (​Hier erneut vorgekommen)
• Bei der Installation zusätzlicher Sprachen gibt es einen Fehler 500 auf den Artikel-Seiten, wenn die Sprachdateien in "/includes/external/paypal/lang/" nicht auch bereitgestellt wurden.
• Seit r10092 gibt es eine Fehlermeldung im Adminbereich bei den Zahlungsmodulen:

  1146 - Table 'paypal_config' doesn't exist
  
  SELECT config_value FROM paypal_config WHERE config_key = 'PAYPAL_LOG_LEVEL'
  
  [MOD SQL Error]
  

  Ursache ist die Änderung in der Datei ​/includes/external/paypal/classes/PayPalPaymentBase.php.

[Gerhard Waldemair]

In 10106:

fix #933 - update paypal

[Torsten Riemer]

Ich öffne das Ticket nochmal, da die folgende Punkte wohl immer noch teilweise problematisch sind:

• Die Versandkosten werden bei "paypalcart" nicht an PayPal übermittelt, siehe: ​MODUL: PayPal PLUS Zahlungsmodul für modified eCommerce Shopsoftware -> Ein Teil des Problems lag wohl an einer falsch von mir angepassten Datei /includes/modules/payment/paypalcart.php die auf nicht vorhandene Dateien "/includes/shipping_action.php" & "/includes/shipping_block.php" zurückgreifen wollte. Auf der checkout_confirmation Seite erhalte ich aber nach Bestätigung der Versandart immer noch keine Versandkosten. -> Korrigiert in r10136, r10137 & r10138
• Es darf kein neuer access_token angefordert werden, solange der alte noch gültig ist, da dies sonst zu einer Sperrung führen kann, siehe dazu: ​Fehler bei PayPal Rest API (​Hier erneut vorgekommen) -> Korrigiert in r10139
• Neues Problem: Auf der checkout_confirmation Seite hat der Button "IMAGE_BUTTON_CONFIRM" für die Auswahl der Versandart nach Rückleitung von PayPal bei "paypalcart" keine Sprach-Konstante zugewiesen, was dazu führt, dass der Button im Responsive Template mit dem Text "IMAGE_BUTTON_CONFIRM" angezeigt wird. -> Korrigiert in r10132

Besonders das Changeset r10126 möchte ich nochmal zur Diskussion stellen. Ist es nicht sinnvoller die Datei im Ordner "/cache/" zu lassen und die Funktion clear_dir() um eine Ausnahme für die Dateien pp_auth_sandbox.cache & pp_auth_live.cache zu erweitern?

Nachtrag: In Absprache mit Gerhard habe ich r10126 in r10130 erst einmal wieder zurück gezogen.

[Torsten Riemer]

In 10132:

partly fix #933 - update paypal

[Gerhard Waldemair]

In 10136:

partly fix #933 - update paypal

[Gerhard Waldemair]

In 10137:

partly fix #933 - update paypal

[Gerhard Waldemair]

In 10138:

partly fix #933 - update paypal

[Gerhard Waldemair]

In 10139:

partly fix #933 - update paypal

[Torsten Riemer]

Bitte hier nochmal wegen dem access_token schauen: ​Fehler bei PayPal Rest API

[Ronald Parcinski]

Folgende Fehler wurde mir gemeldet:
---
Wenn man Paypal-Warenkorb klickt und nach Paypal wieder zum finalen Schritt in den Shop geleitet wird, dann wird dort ein richtiges Kundenkonto im Shop ("Neuer Kunde") angelegt. Wenn man den finalen Schritt bspw. nicht durchführt, sondern nochmal im Shop schaut und später im Warenkorb auf "zur Kasse" klickt, dann steht da immer nur "Die Paypal-Zahlung wurde abgebrochen". Der Kunde kann auch kein Kundenkonto anlegen, weil unter der Email-Adresse ja bereits eins besteht (ohne Kundennummer) und ein Passwort kennt er natürlich nicht, weil er nie eins vergeben hat.
---

[Torsten Riemer]

Bezüglich des access_tokens war uns in r10200 ein ​Fix für die Datei /includes/external/paypal/lib/PayPal/Auth/OAuthTokenCredential.php in ein Commit rein gerutscht, den wir evtl. wieder implementieren sollten.

[Torsten Riemer]

Ein weiterer Fehler, der im Trunk Demoshop nachvollziehbar ist:

Stellt man unter "PayPal Konfiguration" -> "Warenkorb" auf "Ja" und hat der Kunde einen Coupon-Code eingegeben, dann bekommt er kein PayPal PLUS mehr auf der checkout_payment Seite angezeigt.

[Gerhard Waldemair]

In 10209:

fix #933 - set PayPal auth cache lifetime to session expiry time

[Torsten Riemer]

Ist mit r10209 der Fix aus r10200 für die Datei ​/includes/external/paypal/lib/PayPal/Auth/OAuthTokenCredential.php wieder implementiert?

[Gerhard Waldemair]

ist damit gefixt auf Basis wie das SDK von PayPal aktualisiert wurde

[Torsten Riemer]

Offen sind noch folgende Fehler:

Replying to web28:

Folgende Fehler wurde mir gemeldet:
---
Wenn man Paypal-Warenkorb klickt und nach Paypal wieder zum finalen Schritt in den Shop geleitet wird, dann wird dort ein richtiges Kundenkonto im Shop ("Neuer Kunde") angelegt. Wenn man den finalen Schritt bspw. nicht durchführt, sondern nochmal im Shop schaut und später im Warenkorb auf "zur Kasse" klickt, dann steht da immer nur "Die Paypal-Zahlung wurde abgebrochen". Der Kunde kann auch kein Kundenkonto anlegen, weil unter der Email-Adresse ja bereits eins besteht (ohne Kundennummer) und ein Passwort kennt er natürlich nicht, weil er nie eins vergeben hat.
---

Replying to Tomcraft:

Ein weiterer Fehler, der im Trunk Demoshop nachvollziehbar ist:

Stellt man unter "PayPal Konfiguration" -> "Warenkorb" auf "Ja" und hat der Kunde einen Coupon-Code eingegeben, dann bekommt er kein PayPal PLUS mehr auf der checkout_payment Seite angezeigt.

[schreinermeiter]

Hallo
modified eCommerce Shopssoftware v1.06 rev 4642 SP4 dated: 2016-04-01
Datenbank Version: MOD_1.0.6.4

PayPal-PLUS wurde laut Anleitung eingebaut und zwischenzeitlich mehrmals überprüft.
Zahlungen über PayPal-Plus funktionieren bisher.

Folgender Fehler konnte bisher nicht behoben werden.
Bei der Auswahl einer Nicht PayPal PLUS Zahlungsart in der checkout_payment wird auf der checkout_confitmation Seite dennoch PayPal-Plus als Zahlungsart dargestellt.
Nach dem Klick auf KAUFEN wird dann kurzzeitig callback/paypal/paypalplus.php?checkout=true ausgeführt und die checkout_confitmation Seite mit ?conditions=true nochmals geladen. Dann wird die richtige Zahlungsart dargestellt und es muss nochmals auf KAUFEN geklickt werden.
Dies führt bei vielen Kunden, welche nicht mit PayPal-Plus zahlen möchten zu Verunsicherungen.

[Torsten Riemer]

Ich fasse hier nochmal zusammen und ergänze auch:

Offen sind noch folgende Fehler:

1.) Replying to web28:

Folgende Fehler wurde mir gemeldet:
---
Wenn man Paypal-Warenkorb klickt und nach Paypal wieder zum finalen Schritt in den Shop geleitet wird, dann wird dort ein richtiges Kundenkonto im Shop ("Neuer Kunde") angelegt. Wenn man den finalen Schritt bspw. nicht durchführt, sondern nochmal im Shop schaut und später im Warenkorb auf "zur Kasse" klickt, dann steht da immer nur "Die Paypal-Zahlung wurde abgebrochen". Der Kunde kann auch kein Kundenkonto anlegen, weil unter der Email-Adresse ja bereits eins besteht (ohne Kundennummer) und ein Passwort kennt er natürlich nicht, weil er nie eins vergeben hat.
---

2.) Replying to Tomcraft:

Ein weiterer Fehler, der im Trunk Demoshop nachvollziehbar ist:

Stellt man unter "PayPal Konfiguration" -> "Warenkorb" auf "Ja" und hat der Kunde einen Coupon-Code eingegeben, dann bekommt er kein PayPal PLUS mehr auf der checkout_payment Seite angezeigt.

3.) Und hier bitte mal schauen wegen einem Problem mit dem Modul "pn_sofortueberweisung" in älteren Shopversionen: ​MODUL: PayPal PLUS Zahlungsmodul für modified eCommerce Shopsoftware
4.) Probleme in 1.06 rev 4642 SP4 mit weiteren Zahlungsmodulen, siehe ​hier, ​hier & ​hier.
5.) Erweiterung für Rechnungskauf, siehe: ​MODUL: PayPal PLUS Zahlungsmodul für modified eCommerce Shopsoftware
6.) Im Modulupdate für Shopversion 2.0.0.0 rev 9678 scheint ein Fehler zu sein. Die Rückzahlung funktioniert nicht mehr. Der Aufruf ist ​http://www.example.com/adminxyz/orders.php?oID=123456&action=custom&subaction=paypalaction
7.) /print_order.php erweitern um PayPal Zahlungsinformationen bei "Kauf auf Rechnung".

8.) Replying to schreinermeiter:

Hallo
modified eCommerce Shopssoftware v1.06 rev 4642 SP4 dated: 2016-04-01
Datenbank Version: MOD_1.0.6.4

PayPal-PLUS wurde laut Anleitung eingebaut und zwischenzeitlich mehrmals überprüft.
Zahlungen über PayPal-Plus funktionieren bisher.

Folgender Fehler konnte bisher nicht behoben werden.
Bei der Auswahl einer Nicht PayPal PLUS Zahlungsart in der checkout_payment wird auf der checkout_confitmation Seite dennoch PayPal-Plus als Zahlungsart dargestellt.
Nach dem Klick auf KAUFEN wird dann kurzzeitig callback/paypal/paypalplus.php?checkout=true ausgeführt und die checkout_confitmation Seite mit ?conditions=true nochmals geladen. Dann wird die richtige Zahlungsart dargestellt und es muss nochmals auf KAUFEN geklickt werden.
Dies führt bei vielen Kunden, welche nicht mit PayPal-Plus zahlen möchten zu Verunsicherungen.

9.) Das Modul "PayPal Warenkorb (paypalcart)" sollte dringend wieder erweitert werden um die Übertragung der vorläufigen Versandkosten, so wie es beim alten PayPal Modul der Fall war. Die entsprechenden Sprachkonstanten dafür sind immer noch vorhanden ("/lang/german/german.php"):

define('PAYPAL_EXP_VORL','Vorläufige Versandkosten');
define('PAYPAL_EXP_VERS','6.90');

10.) Die durch den API-Call hervorgerufene Verzögerung beim Aufruf einer Bestellung, die per PayPal bezahlt wurde geht doch sicherlich zu beseitigen, indem die Anfrage erst nachträglich per AJAX ausgeführt wird, oder nicht?
11.) Bei Benutzung von paypalcart.php werden auf der checkout_confirmation Seite die Fehlermeldungen bei nicht gesetzten Häkchen zu weit unten angezeigt. Überall sonst im Shop geben wir die Fehler oben auf der Seite aus.

[Gerhard Waldemair]

In 10339:

partly fix #933

[Torsten Riemer]

Damit bleibt nur noch:

1.) Replying to web28:

Folgende Fehler wurde mir gemeldet:
---
Wenn man Paypal-Warenkorb klickt und nach Paypal wieder zum finalen Schritt in den Shop geleitet wird, dann wird dort ein richtiges Kundenkonto im Shop ("Neuer Kunde") angelegt. Wenn man den finalen Schritt bspw. nicht durchführt, sondern nochmal im Shop schaut und später im Warenkorb auf "zur Kasse" klickt, dann steht da immer nur "Die Paypal-Zahlung wurde abgebrochen". Der Kunde kann auch kein Kundenkonto anlegen, weil unter der Email-Adresse ja bereits eins besteht (ohne Kundennummer) und ein Passwort kennt er natürlich nicht, weil er nie eins vergeben hat.
---

2.) Replying to Tomcraft:

Ein weiterer Fehler, der im Trunk Demoshop nachvollziehbar ist:

Stellt man unter "PayPal Konfiguration" -> "Warenkorb" auf "Ja" und hat der Kunde einen Coupon-Code eingegeben, dann bekommt er kein PayPal PLUS mehr auf der checkout_payment Seite angezeigt.

3.) Und hier bitte mal schauen wegen einem Problem mit dem Modul "pn_sofortueberweisung" in älteren Shopversionen: ​MODUL: PayPal PLUS Zahlungsmodul für modified eCommerce Shopsoftware
4.) Probleme in 1.06 rev 4642 SP4 mit weiteren Zahlungsmodulen, siehe ​hier, ​hier & ​hier.
5.) Erweiterung für Rechnungskauf, siehe: ​MODUL: PayPal PLUS Zahlungsmodul für modified eCommerce Shopsoftware
6.) Im Modulupdate für Shopversion 2.0.0.0 rev 9678 scheint ein Fehler zu sein. Die Rückzahlung funktioniert nicht mehr. Der Aufruf ist ​http://www.example.com/adminxyz/orders.php?oID=123456&action=custom&subaction=paypalaction
7.) /print_order.php erweitern um PayPal Zahlungsinformationen bei "Kauf auf Rechnung".

8.) Replying to schreinermeiter:

Hallo
modified eCommerce Shopssoftware v1.06 rev 4642 SP4 dated: 2016-04-01
Datenbank Version: MOD_1.0.6.4

PayPal-PLUS wurde laut Anleitung eingebaut und zwischenzeitlich mehrmals überprüft.
Zahlungen über PayPal-Plus funktionieren bisher.

Folgender Fehler konnte bisher nicht behoben werden.
Bei der Auswahl einer Nicht PayPal PLUS Zahlungsart in der checkout_payment wird auf der checkout_confitmation Seite dennoch PayPal-Plus als Zahlungsart dargestellt.
Nach dem Klick auf KAUFEN wird dann kurzzeitig callback/paypal/paypalplus.php?checkout=true ausgeführt und die checkout_confitmation Seite mit ?conditions=true nochmals geladen. Dann wird die richtige Zahlungsart dargestellt und es muss nochmals auf KAUFEN geklickt werden.
Dies führt bei vielen Kunden, welche nicht mit PayPal-Plus zahlen möchten zu Verunsicherungen.

9.) Das Modul "PayPal Warenkorb (paypalcart)" sollte dringend wieder erweitert werden um die Übertragung der vorläufigen Versandkosten, so wie es beim alten PayPal Modul der Fall war. Die entsprechenden Sprachkonstanten dafür sind immer noch vorhanden ("/lang/german/german.php"):

define('PAYPAL_EXP_VORL','Vorläufige Versandkosten');
define('PAYPAL_EXP_VERS','6.90');

10.) Die durch den API-Call hervorgerufene Verzögerung beim Aufruf einer Bestellung, die per PayPal bezahlt wurde geht doch sicherlich zu beseitigen, indem die Anfrage erst nachträglich per AJAX ausgeführt wird, oder nicht?

[Andreas Guder]

Hallo, wie ich feststellen musste, bricht bei Verwendung des PayPalPlus-Warenkorbs der Redirect von PayPal zum Shop ab. Natürlich nur wenn die Ausgabe der Fehlermeldungen im Shop aktiv ist.

Ursache für den Abbruch ist eine PHP-Warning, welche von der Datei includes/classes/order.php geworfen wird. Sowohl in PayPal-Plus der 1.06er-Versionen, als auch der 2.0er Versionen.
Die Ursache der Fehlermeldung liegt in dieser Datei:
/callback/paypal/paypalcart.php
Hier wird der Session-Wert "shipping" als String definiert.

"$_SESSION['shipping'] = '';"

In der includes/classes/order.php wird dann jedoch versucht, einen Array-Wert zu bekommen:

$this->info = array('order_status' => DEFAULT_ORDERS_STATUS_ID,
                    ...
                    'shipping_method' => isset($_SESSION['shipping']) ? $_SESSION['shipping']['title'] : '',
                    ...

Ob noch andere Effekte auftreten, habe ich nicht geprüft.

[Gerhard Waldemair]

In 10343:

partly fix #933, #1032

[Gerhard Waldemair]

In 10344:

partly fix #933

[Gerhard Waldemair]

In 10346:

partly fix #933

[Gerhard Waldemair]

In 10347:

partly fix #933

[Gerhard Waldemair]

In 10349:

partly fix #933

[Gerhard Waldemair]

In 10350:

partly fix #933

[Gerhard Waldemair]

In 10351:

partly fix #933

[Gerhard Waldemair]

In 10352:

partly fix #933

[Gerhard Waldemair]

In 10353:

partly fix #933