Closed (fixed)
Project:
Payment
Version:
8.x-2.x-dev
Component:
Code
Priority:
Critical
Category:
Task
Assigned:
Unassigned
Reporter:
Created:
22 Jul 2013 at 18:56 UTC
Updated:
22 Oct 2013 at 20:50 UTC
Jump to comment: Most recent file
Port the payment form field and add it to Payment, as the different plugins will be so small and independent that it hardly makes any sense to keep them in a separate module anymore.
| Comment | File | Size | Author |
|---|---|---|---|
| #11 | payment_2047805_11.patch | 56.25 KB | xano |
| #10 | payment_2047805_10.patch | 66.78 KB | xano |
| #9 | payment_2047805_9.patch | 60.91 KB | xano |
| #8 | payment_2047805_8.patch | 49.97 KB | xano |
| #6 | payment_2047805_6.patch | 48.86 KB | xano |
Comments
Comment #1
xanoDepends on #2047919: Add a payment add form controller, and #1896416: Use contexts as payment bundles so the context can store an entity reference to the entity the form is attached to.
Comment #2
xanoComment #3
xanoField types are now plugins. See https://drupal.org/node/2064123.
Comment #4
xanoComment #5
xanoThis patch keeps the field in a separate module, because a configurable entity reference field is use to link payments to the entities they were made from. We may be able to make this dependency optional and merge the field into the main module anyway.
Comment #6
xanoOne problem is that every field requires one payment currency and an undefined number of line items. Field storage cannot handle this. Two possible solutions are:
Comment #7
xanoComment #8
xanoThe field type and widget plugins work and are tested. The formatter and payment context plugins need work, and so will the entity form controller that is added for the formatter to use.
Let's fix #2089375: Remove payment method plugin payment operations and #2089355: Store payments' payment method configuration before we continue with this patch, so we won't have to rework form execution later on anymore.
Comment #9
xanoThe three field plugins are done and tested. The payment type plugin seems to work as well, but needs more testing.
The biggest problem now is that payment type derivatives contain a colon, which is not allowed in filenames. See #2035345: Reconsider whether to use ':' as separator for derivative plugins.
Comment #10
xanoWorking on the upgrade path.
Comment #11
xanoBecause of #2035345: Reconsider whether to use ':' as separator for derivative plugins and #2107713: Remove the upgrade path, the upgrade path and entity references to the source entities were dropped.