Page MenuHomePhabricator

Move staging functions into helper classes
Closed, ResolvedPublic2 Estimated Story Points

Description

This is a small cleanup, to make staging functions more reusable between gateways and to reduce business logic clutter.

Event Timeline

Change 277698 had a related patch set uploaded (by Awight):
[WIP] Prototype encapsulation for staging functions

https://gerrit.wikimedia.org/r/277698

Change 277698 merged by jenkins-bot:
Encapsulate staging functions

https://gerrit.wikimedia.org/r/277698

Change 277705 had a related patch set uploaded (by Awight):
Encapsulate AstroPay staging methods

https://gerrit.wikimedia.org/r/277705

Change 277845 had a related patch set uploaded (by Awight):
Generalize data transformations

https://gerrit.wikimedia.org/r/277845

DStrine set the point value for this task to 2.Mar 16 2016, 10:44 PM

Change 277936 had a related patch set uploaded (by Awight):
Encapsulate Ingenico staging functions

https://gerrit.wikimedia.org/r/277936

Change 277979 had a related patch set uploaded (by Awight):
Clean up, add accessors

https://gerrit.wikimedia.org/r/277979

Change 277978 had a related patch set uploaded (by Awight):
Put all extra transaction keys in submethod meta

https://gerrit.wikimedia.org/r/277978

Change 277981 had a related patch set uploaded (by Awight):
Split ContributionTrackingPlusUnique into a reusable thing

https://gerrit.wikimedia.org/r/277981

Change 277980 had a related patch set uploaded (by Awight):
Make BrazilianFiscalNumber staging reusable

https://gerrit.wikimedia.org/r/277980

Change 277983 had a related patch set uploaded (by Awight):
Finish encapsulating Ingenico staging logic

https://gerrit.wikimedia.org/r/277983

Change 277982 had a related patch set uploaded (by Awight):
Split out language staging

https://gerrit.wikimedia.org/r/277982

Change 277985 had a related patch set uploaded (by Awight):
Encapsulate PayPal staging logic

https://gerrit.wikimedia.org/r/277985

Change 277978 merged by jenkins-bot:
Put all extra transaction keys in submethod meta

https://gerrit.wikimedia.org/r/277978

Change 277979 merged by jenkins-bot:
Clean up, add accessors

https://gerrit.wikimedia.org/r/277979

awight added a subscriber: Ejegg.

I believe @Ejegg was looking at the Adyen staging functions, so I'm leaving those alone for now. The patches here are safe to merge individually.

Change 277705 merged by jenkins-bot:
Encapsulate AstroPay staging methods

https://gerrit.wikimedia.org/r/277705

Change 277845 merged by jenkins-bot:
Generalize data transformations

https://gerrit.wikimedia.org/r/277845

Change 277936 merged by jenkins-bot:
Encapsulate Ingenico staging functions

https://gerrit.wikimedia.org/r/277936

Change 277980 abandoned by Awight:
Make BrazilianFiscalNumber staging reusable

https://gerrit.wikimedia.org/r/277980

Change 277981 abandoned by Awight:
Split ContributionTrackingPlusUnique into a reusable thing

https://gerrit.wikimedia.org/r/277981

Change 277982 abandoned by Awight:
Split out language staging

https://gerrit.wikimedia.org/r/277982

Change 277983 merged by jenkins-bot:
Finish encapsulating Ingenico staging logic

https://gerrit.wikimedia.org/r/277983

Change 277985 merged by jenkins-bot:
Encapsulate PayPal staging logic

https://gerrit.wikimedia.org/r/277985

@Ejegg
I'm happy to pick up the Adyen staging work whereever you leave it...