Ability to change Worldpay Test and Live URLs

matason - October 22, 2009 - 08:50
Project:UC Worldpay
Version:6.x-1.0-beta1
Component:Code
Category:feature request
Priority:normal
Assigned:Unassigned
Status:closed
Description

The URLs for Worldpay test and live environments have changed, although the old URLs work fine, this patch gives the user the ability to change them via the existing Worldpay settings form setting them by default to the new URLs.

I've also included a couple of minor fixes:

  • Added "core = 6.x" to uc_worldpay.info
  • Removed t() from hook_menu() title value - it's not needed in D6

I can supply separate patches if necessary.

#1

matason - October 22, 2009 - 08:52

Patch attached.

AttachmentSize
uc_worldpay-611440.patch 2.86 KB

#2

matason - October 22, 2009 - 09:34

I'd split "name" and "description" in the .info file with "core", no bigee but this patch is preferable for readability etc :)

AttachmentSize
uc_worldpay-611440.patch 2.86 KB

#3

matason - October 22, 2009 - 09:53

I put the URLs into their own fieldset, okay, I'll stop tinkering now...

AttachmentSize
uc_worldpay-611440.patch 2.95 KB

#4

alexpott - October 28, 2009 - 17:10

+1 for this patch.

The old worldpay url's have just burned me! They don't always work fine. Intermittently some users experience a problem and get a message that their session has timed out. Been on the phone to worldpay and this is a known issue. The live url should be https://secure.wp3.rbsworldpay.com/wcc/purchase (at the moment)

#5

matason - October 28, 2009 - 17:15
Status:needs review» reviewed & tested by the community

Alex, thanks for the information, I'll get this patch committed this evening.

#6

matason - October 28, 2009 - 23:06
Status:reviewed & tested by the community» fixed

Committed, thanks alexpott for testing - http://drupal.org/cvs?commit=281078

#7

System Message - November 11, 2009 - 23:10
Status:fixed» closed

Automatically closed -- issue fixed for 2 weeks with no activity.

 
 

Drupal is a registered trademark of Dries Buytaert.