ys4uno 0 Posted October 16, 2008 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
BryceJr 20 Posted October 17, 2008 Post line 85-97 of payment.php (includes/classes) Share this post Link to post Share on other sites
ys4uno 0 Posted October 17, 2008 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
ys4uno 0 Posted October 17, 2008 anyone can give some help :D :rolleyes: Share this post Link to post Share on other sites
ys4uno 0 Posted October 18, 2008 need some help §§§ Share this post Link to post Share on other sites
BryceJr 20 Posted October 19, 2008 I compared yours with the original. I don't see the syntax error. Share this post Link to post Share on other sites
Coopco 8 Posted October 19, 2008 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
BryceJr 20 Posted October 19, 2008 (edited) 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 October 19, 2008 by BryceJr Share this post Link to post Share on other sites
Coopco 8 Posted October 19, 2008 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
tswern 0 Posted November 8, 2008 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