Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[REF] [EXPORT] [TLA] Update handling of input fields so that the mapping format is accepted. #14792

Merged
merged 1 commit into from
Jul 11, 2019

Conversation

eileenmcnaughton
Copy link
Contributor

@eileenmcnaughton eileenmcnaughton commented Jul 11, 2019

Overview

Code cleanup on export - make it cleaner

Before

Confusing array format used

After

Sane array format used

Technical Details

Updates the export processor to accept the field mapping format from the civicrm_mapping_field table

The current situation is that a weird and wonderful format is used on the Mapping form quickform. This is converted to a sane format when saved to the civicrm_mapping_field table. However, the export processor currently expects the insane format - leading to complexity & lack of re-usablity.

This pr updates the workings of the export processor to use the sane format - as of this PR that are still received in the insane format but they are converted before they are actually used.

This sets us up to change the format received by the . function . & re-instate sanity (instate perhaps)

Comments

Although there is a lot of code in this PR it is very heavily tested. In fact every single change was made as part of stepping through a unit test because each changed line is specifically tested. We built up a lot of tests on this on the IM part of LeXIM over the last year

@civibot
Copy link

civibot bot commented Jul 11, 2019

(Standard links)

@civibot civibot bot added the master label Jul 11, 2019
@eileenmcnaughton eileenmcnaughton changed the title Update handling of input fields so that the mapping format is accepted. [REF] [EXPORT] [TROLLING COLEMAN] Update handling of input fields so that the mapping format is accepted. Jul 11, 2019
@eileenmcnaughton
Copy link
Contributor Author

test this please

1 similar comment
@eileenmcnaughton
Copy link
Contributor Author

test this please

At this point we are converting to the mapping format AFTEr the function receives the fields but
changing that is the next step
@eileenmcnaughton eileenmcnaughton changed the title [REF] [EXPORT] [TROLLING COLEMAN] Update handling of input fields so that the mapping format is accepted. [REF] [EXPORT] [TLA] Update handling of input fields so that the mapping format is accepted. Jul 11, 2019
@colemanw
Copy link
Member

Great step forward @eileenmcnaughton!
Merging this based on code review & passing tests.

@colemanw colemanw merged commit 686232e into civicrm:master Jul 11, 2019
@colemanw colemanw deleted the ex_x branch July 11, 2019 14:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants