Hi Ryan, Thanks for your

Posts: 67
Joined: 05/05/2008
Bug Finder

Hi Ryan,

Thanks for your reply.

If the field is empty the checkout does not validate. I would love to be using the 1.0 release, but this project has been in development for over 7 months, and was worked on by a number of developers who had no prior Drupal experience, which resulted in a considerable number of "hacks" throughout core and contributed modules. I did the majority of Ubercart development (well, most of other stuff, too), but am also guilty of doing large amounts of hacking to the module, as we have very specific requirements and at the time there were no contrib modules to provide the functionality we needed. We figured it would be quicker to add a few small hacks rather than writing a module for each modification (of which there are a *LOT*), but in hindsight it would have been much better to do everything "properly" via modules/theme overrides.

In summary, major hackage killed the upgrade path.

We were actually considering waiting for the 1.0 release, but since we had no guaranteed ETA we decided to use what we had access to at the time.

---

I just did the following:

- in uc_payflowpro.module, I removed the line <CVNum>' . $order->payment_details['cc_cvv'] . '</CVNum> from the XML
- in uc_credit.module, I
* commented out:

<?php
if (variable_get('uc_credit_bank_enabled', FALSE)) {
       
$review[] = array('title' => t('Issuing Bank'), 'data' => check_plain($arg1->payment_details['cc_bank']));
      }
?>

around line 360, which removes the CVV info from the review order page
* added style="display: none;" to the <tr> which displays the CVV field/info on the order page (around line 812)
* set '#default_value' to '000' for the actual CVV field around line 747

From first glance this seems to work - orders are being created properly. Will have to do actual "live" testing once we get some other things in place...

How to not sure CVV in RC4? By: kerunt (3 replies) Fri, 07/25/2008 - 14:22