ECB 1.0.8 Paypalproblem

[odiwan]

Hallo,

leider kann ich dem nichts hilfreiches entnehmen ausser das Version 1234 das Problem beheben wird. Wie kann ich den Fehler selbst beseitigen? Oder habe ich etwas übersehen?

Grüße
Odiwan

[Xantiva]

Die "Revision 1234" (die Nummer ist echt ), besagt, dass die Änderungen "irgendwann" zwischen der ECB - Shop Verison 1.0.9a und der Webs - Shop Version 1.0.10 gemacht wurde.
(siehe auch Versionsübersicht mit Revisionsnummern)

Hier sind die Änderungen der Version, ABER es ist nicht gesagt, dass die ECB-Shop v1.0.8 auch damit fehlerfrei läuft ...
(unified Diff Format)

Code: Alles auswählen

Index: paypal_checkout.php
===================================================================
--- paypal_checkout.php	(revision 1233)
+++ paypal_checkout.php	(revision 1234)
@@ -1,27 +1,29 @@
 <?php
-/**
- * Project: xt:Commerce - eCommerce Engine
- * @version $Id
- *
- * xt:Commerce - Shopsoftware
- * (c) 2003-2007 xt:Commerce (Winger/Zanier), http://www.xt-commerce.com
- *
- * xt:Commerce ist eine geschützte Handelsmarke und wird vertreten durch die xt:Commerce GmbH (Austria)
- * xt:Commerce is a protected trademark and represented by the xt:Commerce GmbH (Austria)
- *
- * @copyright Copyright 2003-2007 xt:Commerce (Winger/Zanier), www.xt-commerce.com
- * @copyright based on Copyright 2002-2003 osCommerce; www.oscommerce.com
- * @copyright Porttions Copyright 2003-2007 Zen Cart Development Team
- * @copyright Porttions Copyright 2004 DevosC.com
- * @license http://www.xt-commerce.com.com/license/2_0.txt GNU Public License V2.0
- *
- * For questions, help, comments, discussion, etc., please join the
- * xt:Commerce Support Forums at www.xt-commerce.com
- *
- * ab 15.08.2008 Teile vom Hamburger-Internetdienst geändert
- * Hamburger-Internetdienst Support Forums at www.forum.hamburger-internetdienst.de
- * Stand: 01.06.2009
-*/
+/** ----------------------------------------------------------------------------
+ **
+ *F includes/classes/paypal_checkout.php
+ **
+ ** WEB-Shop Software http://www.webs.de/
+ **
+ **  Paypal check functionality
+ **
+ ** ----------------------------------------------------------------------------
+ ** based on:
+ **  (c) 2002-2003 osCommerce; www.oscommerce.com
+ **  (c) 2003-2007 Zen Cart Development Team
+ **  (c) 2004 DevosC.com
+ **  (c) 2003-2007 XT-Commerce; http://xt-commerce.de/
+ **
+ ** modified by
+ **  Hamburger-Internetdienst Support Forums at www.forum.hamburger-internetdienst.de
+ **
+ ** Released under the GNU General Public License
+ **
+ ** @version $Id$
+ ** @copyright 2010 WEB-Shop Software http://www.webs.de/
+ **
+ ** ------------------------------------------------------------------------- */
+
 require_once(DIR_FS_INC . 'xtc_write_user_info.inc.php');
 define('PROXY_HOST', '127.0.0.1');
 define('PROXY_PORT', '808');
@@ -100,9 +102,9 @@
 				endif;
 			endif;
 		endif;
-		if(ereg('^#?([a-f]|[A-F]|[0-9]){3}(([a-f]|[A-F]|[0-9]){3})?$',PAYPAL_API_CO_BACK))
+		if(preg_match('/^#?[a-f0-9]{3}([a-f0-9]{3})?$/i', PAYPAL_API_CO_BACK))
 			$this->BackColor = PAYPAL_API_CO_BACK;
-		if(ereg('^#?([a-f]|[A-F]|[0-9]){3}(([a-f]|[A-F]|[0-9]){3})?$',PAYPAL_API_CO_BORD))
+		if(preg_match('/^#?[a-f0-9]{3}([a-f0-9]{3})?$/i', PAYPAL_API_CO_BORD))
 			$this->BorderColor = PAYPAL_API_CO_BORD;
 	}
 /*************************************************************/
@@ -235,15 +237,15 @@
 						"&HDRBORDERCOLOR=".$this->BorderColor.
 						"&HDRBACKCOLOR=".$this->BackColor.
 						"&CUSTOM=".''.
-						"&SHIPTONAME=".$sh_name.
-						"&SHIPTOSTREET=".$sh_street.
-						"&SHIPTOSTREET2=".$sh_street2.
-						"&SHIPTOCITY=".$sh_city.
-						"&SHIPTOZIP=".$sh_zip.
-						"&SHIPTOSTATE=".$sh_state.
-						"&SHIPTOCOUNTRYCODE=".$sh_countrycode.
-						"&SHIPTOCOUNTRYNAME=".$sh_countryname.
-						"&PHONENUM=".$sh_phonenum.
+						((ACCOUNT_STATE=='true') ? "&SHIPTONAME=".$sh_name.
+						 "&SHIPTOSTREET=".$sh_street.
+						 "&SHIPTOSTREET2=".$sh_street2.
+						 "&SHIPTOCITY=".$sh_city.
+						 "&SHIPTOZIP=".$sh_zip.
+						 '&SHIPTOSTATE='.$sh_state.
+						 "&SHIPTOCOUNTRYCODE=".$sh_countrycode.
+						 "&SHIPTOCOUNTRYNAME=".$sh_countryname.
+						 "&PHONENUM=".$sh_phonenum : NULL).
 						"&ALLOWNOTE=0".
 						"&ADDROVERRIDE=1";
 		// Artikel Details mitgeben
@@ -454,7 +456,10 @@
 		endif;
 		$sh_phonenum = urlencode($order->customer['telephone']);
 		$sh_zip = urlencode($order->delivery['postcode']);
-		$address = "&SHIPTONAME=".$sh_name."&SHIPTOSTREET=".$sh_street."&SHIPTOSTREET2=".$sh_street2."&SHIPTOCITY=".$sh_city."&SHIPTOZIP=".$sh_zip."&SHIPTOSTATE=".$sh_state."&SHIPTOCOUNTRYCODE=".$sh_countrycode."&SHIPTOCOUNTRYNAME=".$sh_countryname."&PHONENUM=".$sh_phonenum;
+		$address = '';
+		if (ACCOUNT_STATE == 'true') {
+		  $address = "&SHIPTONAME=".$sh_name."&SHIPTOSTREET=".$sh_street."&SHIPTOSTREET2=".$sh_street2."&SHIPTOCITY=".$sh_city."&SHIPTOZIP=".$sh_zip."&SHIPTOSTATE=".$sh_state."&SHIPTOCOUNTRYCODE=".$sh_countrycode."&SHIPTOCOUNTRYNAME=".$sh_countryname."&PHONENUM=".$sh_phonenum;
+		}
 		// String zusammenbauen
 		$nvpstr="&AMT=".$paymentAmount.
 						"&CURRENCYCODE=".$currencyCodeType.
@@ -547,7 +552,11 @@
 		endif;
 		$sh_phonenum = urlencode($order->customer['telephone']);
 		$sh_zip = urlencode($order->delivery['postcode']);
-		$address = "&SHIPTONAME=".$sh_name."&SHIPTOSTREET=".$sh_street."&SHIPTOSTREET2=".$sh_street2."&SHIPTOCITY=".$sh_city."&SHIPTOZIP=".$sh_zip."&SHIPTOSTATE=".$sh_state."&SHIPTOCOUNTRYCODE=".$sh_countrycode."&SHIPTOCOUNTRYNAME=".$sh_countryname."&PHONENUM=".$sh_phonenum;
+		$address = '';
+		if (ACCOUNT_STATE == 'true') {
+		  $address = "&SHIPTONAME=".$sh_name."&SHIPTOSTREET=".$sh_street."&SHIPTOSTREET2=".$sh_street2."&SHIPTOCITY=".$sh_city."&SHIPTOZIP=".$sh_zip."&SHIPTOSTATE=".$sh_state."&SHIPTOCOUNTRYCODE=".$sh_countrycode."&SHIPTOCOUNTRYNAME=".$sh_countryname."&PHONENUM=".$sh_phonenum;
+		}
+
 		// Versand Ende
 		$nvpstr='&TOKEN='.$token.
 						'&PAYERID='.$payerID.
@@ -688,7 +697,7 @@
 				$insert_id = xtc_db_insert_id($insert_query);
 				$insert_query = xtc_db_query("insert into " . TABLE_COUPON_EMAIL_TRACK . " (coupon_id, customer_id_sent, sent_firstname, emailed_to, date_sent) values ('" . $insert_id . "', '0', 'Admin', '" . $email_address . "', now() )");
 				$_SESSION['reshash']['SEND_GIFT'] = 'true';
-				$_SESSION['reshash']['GIFT_AMMOUNT'] = $xtPrice->xtcFormat(NEW_SIGNUP_GIFT_VOUCHER_AMOUNT, true);
+				$_SESSION['reshash']['GIFT_AMOUNT'] = $xtPrice->xtcFormat(NEW_SIGNUP_GIFT_VOUCHER_AMOUNT, true);
 				$_SESSION['reshash']['GIFT_CODE'] = $coupon_code;
 				$_SESSION['reshash']['GIFT_LINK'] = xtc_href_link(FILENAME_GV_REDEEM, 'gv_no=' . $coupon_code, 'NONSSL', false);
 			}
@@ -1224,6 +1233,7 @@
 					$request = stream_context_create($request_post);
 					$result= file_get_contents($this->IPN_URL, false, $request);
 				endif;
+				error_log($result);
 				if(strtoupper($result) == 'VERIFIED' or $result == '1') {
 					if($this->data['payment_status'] == 'Completed') {
 						if(PAYPAL_ORDER_STATUS_SUCCESS_ID > 0) {
@@ -1354,7 +1364,7 @@
 /*************************************************************/
 	function UTF8decode($string){
 		// Session vorhanden
-		// Stand: 29.04.2009
+ 		// Stand: 29.04.2009
 		if($this->detectUTF8($string))
 			$string=$this->mn_iconv('UTF-8', $_SESSION['language_charset'], $string);
 		return($string);
@@ -1413,4 +1423,4 @@
 		endif;
 	}
 }
-?>
\ No newline at end of file
+?>

Über Dein Feedback freuen sich aber die anderen dann hier

Ciao,
Mike

[bab]

Hi,

würde es reichen, wenn man die paypal_checkout.php einfach durch die neue Version ersetzt oder ist die nicht zu 1.08 kompatibel? Ich möchte ungerne einen funktionierenden Shop auf die neue Version aktualisieren, da ich nicht weiß, ob es dann Probleme geben wird ...

Gruß

[Doc Olson]

Vermutlich nicht.

Der Einbau des Paypal "Moduls" bedurfte der Änderung von dutzenden Dateien. Zwischen der 1.0.8 und dem Webs haben wir die Version des "Moduls" komplett aktualisiert.

[bab]

Das klingt wenig erfreulich ...

Ein Update auf eine aktuelle Webs Version von 1.08 bis PayPal funktioniert wird vermutlich nicht ohne Probleme wegen dem Template funktionieren?

Die Frage muss noch ergänzt werden - ist die Webs 1.08 = ECB 1.08 ? Das konnte ich in der Wiki nicht erkennen, da da hier nur die Versionsnummern stehen ...

[Doc Olson]

ist die Webs 1.08 = ECB 1.08 ? Das konnte ich in der Wiki nicht erkennen, da da hier nur die Versionsnummern stehen ...

Ja und nein. Es gibt keine Webs 1.0.8. Webs kam ab v 1.0.10. Aber im Grunde ist es egal. Das Produkt ist das selbe. von 1.0.9 zu 1.0.10 wurde nur aus ECB WEBS.

Anders gesagt: ECB gibt es nicht (mehr).

[bab]

JA, das es ECB so nicht mehr gibt ist ja logisch. Es ging ja nur um die Frage, ob die Versionen hier auch wieder vorne angefangen haben

Aber sehe ich das richtig, dass in 1.09 der Fehler behoben ist und ich ein Update auf 1.09 durchführen muss. Hierbei habe ich gesehen sind minimale Änderungen am Template nötig - der Aufwand sollte sich aber im Rahmen halten?

[yogi]

Hallo bab,

ein Update auf die Version 1.0.9a (1.0.9 gab es nur wenige Tage) reicht nicht um das Problem zu beheben.
Die Version 1.0.9a basiert auf der Revision 1196, der Fehler wurde jedoch erst in der Revision 1234 behoben, und somit nach der Veröffentlichung der V1.0.9a

Da die Datei includes/classes/paypal_checkout.php zwischen der V1.0.8 und der Revision 1234 nicht geändert wurde und Die Rev. 1234 der Datei in der V1.0.10 ist könntest Du versuchen Deinen Shop V1.0.8 mit der o.g. Datei aus der V1.0.10 laufen zu lassen - es sollte funktionieren da hier nur die Änderungen für den BT #290: ECB: 1.0.8 - PayPal Error bei USA Kunden, Error Number:10729

Natürlich solltest Du nach dieser Änderung das Paypal-Modul selbst mal austesten um die Funktionalität zu prüfen.

[bab]

Ich will nur kurz bescheid sagen, dass die PayPal Checkout aus der 1.10 mit der 1.08 ohne Probleme funktioniert. Es sollte nur darauf geachtet werden, dass die PayPal Farben unter Partner nicht mit einer # anfangen. Sonst gibt es folgenden Fehler:

Ack: Failure
Correlation ID: 82641904acad5
Version:57.0
Error Number:10438
Error Short Message: Transaction refused because of an invalid argument. See additional error messages for details.
Error Long Message: cpp-header-border-color value exceeds maximum allowable length.
Error Number:10439
Error Short Message: Transaction refused because of an invalid argument. See additional error messages for details.
Error Long Message: cpp-header-back-color value exceeds maximum allowable length.

[yogi]

damit markiere ich das Thema mal als erledigt,

das Bei den Farben kein Hash-Zeichen vor dem Farbwert stehen soll ist ja auch im Text entsprechend angegeben