Closed (fixed)
Project:
Commerce DPS
Version:
7.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Task
Assigned:
Unassigned
Reporter:
Created:
13 May 2014 at 21:38 UTC
Updated:
30 Mar 2015 at 04:04 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
xurizaemonComment #2
heathstannardPatch for creation of commerce_dps module.
Only includes an updated shared image & available currencies but could include extra functions in the future such as generating merchant references - although I think for now it's a good idea to leave these in the sub-modules as each may wish to implement a different merchant ref.
When this module is in place we can look at updating the sub-modules to depend on this module and use the included functions/files.
Comment #3
garethhallnz commentedI was thinking we should write a DPS payment class with an interface that each submodule implements. That way when D8 comes along we only need to deal with minimal routing and hook changes.
That way pxpay, pxfusion, pxpost and account2account can all implement their own classes and extend the dps interface.
Comment #4
xurizaemonI like that concept, Gareth. For now I'm just going to add a .inc based on Heath's submission, we don't even need a module to depend on since we can just refer to the file.
Comment #5
xurizaemonWhoops half my comment ended up in the description.
Heath I might create a separate issue for the new paymentexpress.png you've added here.
Comment #7
xurizaemonFixed. Heath the .png change didn't come in as part of this - do you want to submit as a separate isssue if it's still something needs to happen?