Jump to content
Sign in to follow this  
ys4uno

about CCGV v5.20a

Recommended Posts

hello ,

 

i am very pretty new on oscommerce and have very basic knowledge about php, currently i try to run a oscommerce store, and tried to intall some useful addons .

 

i find CCGV and intalled on my oscommerce. but after installation , when i click " Check Out " button, on the payment page, it shows me the following messgae:

 

Parse error: syntax error, unexpected T_STRING in /home/aovieu/public_html/includes/classes/payment.php on line 91

 

 

any idea about how this happened ? and how to fix it ?

 

many thanks for the help

Share this post


Link to post
Share on other sites

here, i post the line between 82-97 ,in payment.php (include/class)

 

 

// Start - CREDIT CLASS Gift Voucher Contribution

// function javascript_validation() {

function javascript_validation($coversAll) {

//added the $coversAll to be able to pass whether or not the voucher will cover the whole

//price or not. If it does, then let checkout proceed when just it is passed.

$js = '';

if (is_array($this->modules)) {

if ($coversAll) {

$addThis='if (document.checkout_payment.cot_gv.checked) {

line 91=> payment_value='cot_gv';

} else ';

} else {

$addThis='';

}

// End - CREDIT CLASS Gift Voucher Contribution

Share this post


Link to post
Share on other sites
I compared yours with the original. I don't see the syntax error.

My older version has

 

payment_value=cot_gv; alert (\'hey yo\');


The Coopco Underwear Shop

 

If you live to be 100 years of age, that means you have lived for 36,525 days. Don't waste another, there aren't many left.

Share this post


Link to post
Share on other sites

CCGV5.19 bug fix2 19 Jan 2008

there is a erro in

 

CCGV5.19

 

in

 

catalogincludesclassespayment.php line 89

 

if ($coversAll) {

$addThis='if (document.checkout_payment.cot_gv.checked) {

payment_value=cot_gv; alert ('hey yo');

} else ';

} else {

$addThis='';

}

 

it will case not get payment value

 

change to

 

if ($coversAll) {

$addThis='if (document.checkout_payment.cot_gv.checked) {

payment_value='cot_gv';

} else ';

} else {

$addThis='';

}

 

If older versions work, maybe they will. Use whatever works.

Edited by BryceJr

Share this post


Link to post
Share on other sites
CCGV5.19 bug fix2 19 Jan 2008

Thanks, but mine works without the bug fix, version CCGV 5.14 and Jonyo mods.


The Coopco Underwear Shop

 

If you live to be 100 years of age, that means you have lived for 36,525 days. Don't waste another, there aren't many left.

Share this post


Link to post
Share on other sites
My older version has

 

payment_value=cot_gv; alert (\'hey yo\');

 

 

Maybe we need to escape the " ' " character. Why not try this adding a "\" in front of both " ' " on line number 91? :

 

if ($coversAll) {

$addThis='if (document.checkout_payment.cot_gv.checked) {

payment_value=\'cot_gv\';

} else ';

} else {

$addThis='';

}

Share this post


Link to post
Share on other sites

Create an account or sign in to comment

You need to be a member in order to leave a comment

Create an account

Sign up for a new account in our community. It's easy!

Register a new account

Sign in

Already have an account? Sign in here.

Sign In Now
Sign in to follow this  

×