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.

Sign up for GitHub

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

Jump to bottom

[REF] [Export] More export Structure arrays to processor #14812

Merged
merged 2 commits into from
Jul 12, 2019
Merged

[REF] [Export] More export Structure arrays to processor #14812

merged 2 commits into from
Jul 12, 2019

Conversation

eileenmcnaughton
Copy link
Contributor

@eileenmcnaughton eileenmcnaughton commented Jul 12, 2019 •
edited
Loading

Overview

Move the function to get the export structure arrays to the processor class

Before

Function is static on BAO_Export

After

Function moved to ExportProcessor

Technical Details

Part of larger cleanup

Comments

@civibot
Copy link

civibot bot commented Jul 12, 2019

(Standard links)

@civibot civibot bot added the master label Jul 12, 2019
@eileenmcnaughton eileenmcnaughton changed the title More export Structure arrays to processor [REF] [Export] More export Structure arrays to processor Jul 12, 2019
@colemanw
Copy link
Member

Merging based on code review & passing tests.

@colemanw colemanw merged commit e38990f into civicrm:master Jul 12, 2019
@colemanw colemanw deleted the ee branch July 12, 2019 12:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
master
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@eileenmcnaughton @colemanw