Jump to content

kittmaster

Members
  • Content count

    100
  • Joined

  • Last visited

  1. I'll take a closer look at this. Maybe include a trapable error fix and rebuild this section of the code to avoid this problem all together Chris
  2. Maybe not you, but someone was asking if it works, the link demostrates that it does. If you ran the proper sql's then the DB is not in error most likely a code update is the issue. No, Xmas and family trump all things that need doing. This is non-negotiable for me. Merry xmas all, I'll be back at it after the holiday Chris
  3. Yes go and check the site link that I provided above or on the last page. With XMAS on the horizon, most will be busy with the holidays. Check back with us next week and most likely it will be updated then. Chris
  4. Joe Already set up that way as suggested, I'm still diggin around. Xmas is killing me......LOL......got to get them gifts.......I'll be on this for the next few hours or until fix is found..........god willing..... Here is the code as it stands: global $order,$ot_coupon,$currency; $coupon_amount= tep_round($ot_coupon->pre_confirmation_check($order->info['subtotal']), $currencies->currencies[$currency]['decimal_places']); // $cc_id /* you will need to uncomment this if your tax order total module is AFTER shipping eg you have all of your tax, including tax from shipping module, in your tax total. if ($coupon_result['coupon_type']=='S') { //if not zero rated add vat to shipping $coupon_amount = tep_add_tax($coupon_amount, '17.5'); } */ $coupon_amount_out = $currencies->format($coupon_amount) . ' '; if ($coupon_result['coupon_minimum_order']>0) $coupon_amount_out .= 'on orders greater than ' . $currencies->format($coupon_result['coupon_minimum_order']) . ' and has been discounted from the final total.'; if (!tep_session_is_registered('cc_id')) tep_session_register('cc_id'); $cc_id = $coupon_result['coupon_id']; if ( strlen($cc_id)>0 && $coupon_amount==0 ) { $err_msg = ERROR_REDEEMED_AMOUNT . ERROR_REDEEMED_AMOUNT_ZERO; //<BR><STRONG>Please Note:</STRONG><BR>If the form fields are missing, just click on the payment option button below to view the form fields <I>again</I>; } else { $err_msg = ERROR_REDEEMED_AMOUNT . $coupon_amount_out; tep_redirect(tep_href_link(FILENAME_CHECKOUT_PAYMENT, 'payment_error='.$this->code.'&error=' . urlencode($err_msg), 'SSL')); } if ($HTTP_POST_VARS['submit_redeem_coupon_x']) tep_redirect(tep_href_link(FILENAME_CHECKOUT_PAYMENT, 'payment_error='.$this->code.'&error=' . urlencode(ERROR_NO_REDEEM_CODE), 'SSL')); }
  5. You guys may want to wait until we can get this sorted out. Right now we are in the process of correcting these issues. The code in the last few pages are for the people who have PHP skills who can apply this information to their sites to get it working correctly. The only operative word here is to have some patience while we get the final fixes in place Chris
  6. I'll look at that, I'll must have forgot the message redirector for that section since I've rewritten part of that section.
  7. I've also sent vouchers via email for testing for those willing to try it Chris
  8. Joe I think it would be a real value to add a case switch to that setup. If voucher value is > than 0, then display voucher value within table " Apply my $XXX.XX E-Gift Credit" > checkbox and I'm not sure but if no voucher is present, table is invisible completely to clean up the screen Chris
  9. Yep I forgot to mention the test code is 123 try that Chris
  10. Ok I've done some serious modifications for the redeem message area of this contrib. The current redeems are not (IMO) not fixed correctly...........sooooo.. Here is my site: http://www.gforceperformanceaudio.com/shoppingbeta/ the site is a beta site and you'll need to sign up for an account, the products are real, but obviously this is for testing only to see if the functionality of the redeeming system can be tested out. The more you beat it up the more we can iron out the reliability and find anything I've missed to be corrected. Let me know if it works for you and as expected. If it does and we are all in agreement I'll post the multiple changes to files tomorrow if I get feedback, if I hear nothing, then I'll assume its still not right and not update the contrib area. Its not just the language files, its the actual functional files that need amending. So give it a whirl, give me some feedback. And we'll go from here Chris
  11. yeah because the message redeem has been hacked so many times for both a non functioning module to double repeated error message with incorrect points, it keeps selecting the wrong messages for the right transaction. I'm working on it as we speak, but most likely won't have a final update until late tonight or tomorrow Chris
  12. Ok I'll be clear on how my site is set up so it will verify if you have the same setup as mine. My versions and fixes downloaded from the CCGV page are as follows: Credit Class and Gift Voucher 5.14 Baby Boy 22 Jul 2005 Since the last 2 upgrade haven't been added, here is an update for ya'll labeled as CCGV 5.14, no major updated, but 5.13 was getting a little outdated. This is the FULL PACKAGE and included the PayPal fix and brettsg's 4 fixes. I don't take no credit for this, just re-packaged and included all of the replaced files for everybody to have the updated version. Enjoy! CCGV Easier to Understand Payment Addon v1.2 aarondwyer 20 Nov 2005 Here we go again. This should nail it this time. I had missed a very small but vital piece of code in the readme.html. This has now been tested by others and works fully. Ignore v1.1 and v1.0 listed in this contribution area, just use this one. Aaron Redeem message fix scranmer 9 Nov 2005 This is a correction to the redeem message at checkout_payment.php which informs the customer when a voucher has been applied. It now also shows a warning if the voucher does not meet the criteria. eg order ?5 minimum order value ?20 message will show:- The coupon has been successfully applied for ***HOWEVER:No reducion available, please see the coupon restrictions*** order ?5 minimum order value ?0 5% discount message will show :- The coupon has been successfully applied for ?0.25 HTH Si. French fix + others for 5.14 shawkes 1 Aug 2005 I have translated the ot_gv.php to english and have included a couple of admin files that I found to have a problem. If you use a shared ssl cert and you have admin setup to use ssl for all links, the link in emails is not right. In the files I changed the variable referred to was HTTP_SERVER changing that to HTTP_CATALOG_SERVER fixes it for shared ssl users and should also work for others. Only the changed files are included in the zip file There are all kinds of other "fixes" but these are the files that are on my site with all my other modifications and therefore works as designed. I don't know why they keep the other crap on the there, they should have an archive the rest for other users and only keep what really works for the current setups, but I don't run this place so............ I'm seeing that it works for cooch, therefore I know that what I've done should be fine. I'd have to go back and verify seeing if your setup is identical to what we used here. Or you can just revert the simple change that you made and you'll be back where you started. If you revert does the site work as it should??? Send a link to your site so we can see what's going on Chris
  13. I finally found the fix for the issue with the voucher less than the value of the total order...........I'll add information to the contribution area http://www.oscommerce.com/community/contributions,282 BTW 5.15 is not a final fix, I found the install into the database when adding and removing the module to still be in french!!!!! I found this out by accident! I'm glad this fix is done, let me know how you guys make out, I've tested it six ways to sunday on a heavily modified site.........70+ contribs and seems to all be in order. let me know how you make out Chris
  14. The error is telling you what is wrong, it is looking for "ERROR_REDEEMED_AMOUNT_ZERO" which is stored in the catalog/includes/language/add_ccgvdc_english.php FILE, YOU must edit the file by adding this information by copying and pasting the following: define('ERROR_REDEEMED_AMOUNT_ZERO', 'a valid coupon number. HOWEVER: No reduction will be applied, please see the coupon restrictions that was sent within your offer email***'); at the very bottom of that file just before the ?> at the end of the file!!! This will solve your problem of the wrong error message showing up Hope this helps Chris
  15. If the shipping is deducted from the total amount as well it will subtract the gross total. If the total value falls below the value of the minimum that is set then it will flag that error. Try this. Create a product that is say $100 and whatever shipping. Then send a $25 voucher with a $1 minimum purchase requirement. Buy the product and the message should come up as "you've redeemed a $25 voucher from a minimum of the $1 blah blah blah. See if that is what happens and you'll know if it is wokring correctly. I found that if the voucher value is greater than the grand total but not inclusive to shipping that error shows up. I may look at this after I finish my current issue which I think I have a final solution that I'm testing now The email should have the restriction limits added when it is purchased or sent, or mass displayed somewhere but it isn't, I may modify that to fix this issue Chris
×