7 replies to this topic

[techs21]

I am having a problem with this add-on when a customer tries to pay for an order via paypal they get this error
Fatal error: Cannot redeclare cmp() (previously declared in /home2/techsonh/public_html/includes/modules/shipping/fedexwebservices.php:225) in /home2/techsonh/public_html/includes/modules/shipping/fedexwebservices.php on line 225

paypal and fedex said its not them so I need to see if I can get this fixed

any help appreciated thanks

[ckpepper02]

I need this answer also!

[web-project]

post the code of fedexwebservices.php on line 225, please?

[ckpepper02]

Line 225 starts here:

function cmp($a, $B) {
   if ($a['cost'] == $b['cost']) {
	 return 0;
   }
   return ($a['cost'] < $b['cost']) ? -1 : 1;
  }  
 
  usort($methods, 'cmp');

I tested and turns out that this error only occurs when using Paypal Express payment module with it. After the users selects shipping, they are sent to paypal to log in. This error shows up when paypal sends them back to checkout.

[web-project]

why:

function cmp($a, $B ) {

as it should be:

function cmp($a, $b ) {

you have the issue that the cmp() function is declared twice, as for the rest of code is fine, looking on example from the php.net and comparing to the following code:

<?php
function cmp($a, $b )
{
	if ($a == $b {
		return 0;
	}
	return ($a < $b ? -1 : 1;
}
$a = array(3, 2, 5, 6, 1);
usort($a, "cmp");
foreach ($a as $key => $value) {
	echo "$key: $value\n";
}
?>

http://php.net/manua...ction.usort.php

[ckpepper02]

I don't know why the code pasted in with a captial '$B', but it's a '$b' in the code as well.

[ckpepper02]

Still have not fixed the issue. But I did see that the cmp function was wrapped inside a foreach method. Tried moving it outside of that foreach and I got the same error on a different line. So no real progress here.

[jerrygarciuh]

This can be fixed like so:

if (!function_exists('cmp')) {
	function cmp($a, $B) {
		if ($a['cost'] == $b['cost']) {
			return 0;
		}
		return ($a['cost'] < $b['cost']) ? -1 : 1;
	}
}

Edited by jerrygarciuh, 28 March 2012, 17:49.