Needs review
Project:
Pay
Version:
6.x-1.1
Component:
Code
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Reporter:
Created:
25 May 2011 at 07:23 UTC
Updated:
26 May 2011 at 05:38 UTC
Jump to comment: Most recent file
It doesn't appear this module really supports multiple payment methods on the one form. Each payment methods form elements are displayed one after the other. This is going to be confusing to users as they will see lots of unrelated field, rather then just seeing what they need to fill in.
When you select the payment method it would be good to have the fields that are not required hidden.
| Comment | File | Size | Author |
|---|---|---|---|
| #2 | 0001-handle-multiple-payment-types-through-a-jquery-form-switcher.patch | 3.91 KB | univate |
| #1 | pay-1168116.patch | 2.44 KB | univate |
Comments
Comment #1
univate commentedThis is a simple patch to add some simple jquery to hide the form elements not required. It would be great to see something like this added.
One issue with this patch is it makes the assumption that there is only one pay form on the page. Also not sure where that closing
is missing from?
Comment #2
univate commentedUpdated patch.