Is there a compatibility issue with e-commerce-cvs and PHP 4.4.1? I am getting this error when I go to admin/modules. Using today's Drupal-CVS and E-commerce-CVS...

Parse error: parse error, unexpected '=', expecting ')' in /home/sdga/public_html/modules/ecommerce/shipping/shipping.module on line 346

I am trying to fix it. I have commented out the code. It starts to work but I get other problems so I figured that is not the best way to fix the problem. Excuse me if I am not using these forums correctly - I am new. If I find a fix I'll post it back. It must have something to do with my PHP version - if nobody else is having the problem...

Here is line 346 - 363 of shipping.module.

function shipping_call_active_shipping_modules(&$txn,$op,&$args3 = NULL ,&$args4 = NULL) {
foreach(variable_get("ec_active_shipping_modules", array()) as $module => $module_settings) {
$f = $module. "_shippingapi";
if (function_exists($f)) {
//If we are called with $txn = 'check', call the modules with $txn = the modules name
//This is normally done to see if the modules support the $op portion of the shippingapi
if($txn == 'check') {
$results[$module] = $f($module, $op);
}
else {
//I don't know that we really need to set the $module here, but it might be usefull in the future
//When we want to know which modules returned which results, or which modules returned true.
$results[$module] = $f($txn, $op);
}
}
}
return $results;
}

Thank you very much.

Comments

gordon’s picture

Status: Active » Closed (duplicate)
WithoutaDoubt’s picture

Does update mean the CVS download at http://drupal.org/project/Modules/cvs under e-commerce is updated right away? Or do I wait a while? (daily)

sime’s picture

If there has been a very recent change you can ensure that you get the latest copy by going to cvs:
http://cvs.drupal.org/viewcvs/drupal/contributions/modules/ecommerce/

WithoutaDoubt’s picture

Thank you very much. The link you gave me I am able to get individual files. Can I download an entire directory? In this case I just need the one module, but in the future, I may want to download an entire directory from the cvs.
Thanks.