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

Fix import activity & add tests #23707

Merged
merged 4 commits into from
Jun 7, 2022

Conversation

eileenmcnaughton
Copy link
Contributor

@eileenmcnaughton eileenmcnaughton commented Jun 7, 2022

Extends #23703 to do all the same things for activity import

@civibot
Copy link

civibot bot commented Jun 7, 2022

(Standard links)

@civibot civibot bot added the master label Jun 7, 2022
@eileenmcnaughton eileenmcnaughton force-pushed the import_activity branch 2 times, most recently from fbb846c to af8366b Compare June 7, 2022 08:26
@aydun
Copy link
Contributor

aydun commented Jun 7, 2022

On first screen of Activity Import, the help icon produces a pop-up with no information.

Warnings on second page:

    Warning: Undefined array key "savedMappingName" in include() (line 8 of /home/jenkins/bknix-dfl/build/core-23703-3lxzt/web/sites/default/files/civicrm/templates_c/en_US/%%2E/2E3/2E3CBBFD%%MapTable.tpl.php).
    Warning: Undefined array key 2 in include() (line 77 of /home/jenkins/bknix-dfl/build/core-23703-3lxzt/web/sites/default/files/civicrm/templates_c/en_US/%%2E/2E3/2E3CBBFD%%MapTable.tpl.php).
    Warning: Trying to access array offset on value of type null in include() (line 77 of /home/jenkins/bknix-dfl/build/core-23703-3lxzt/web/sites/default/files/civicrm/templates_c/en_US/%%2E/2E3/2E3CBBFD%%MapTable.tpl.php).
    Warning: Undefined array key 2 in include() (line 77 of /home/jenkins/bknix-dfl/build/core-23703-3lxzt/web/sites/default/files/civicrm/templates_c/en_US/%%2E/2E3/2E3CBBFD%%MapTable.tpl.php).
    Warning: Trying to access array offset on value of type null in include() (line 77 of /home/jenkins/bknix-dfl/build/core-23703-3lxzt/web/sites/default/files/civicrm/templates_c/en_US/%%2E/2E3/2E3CBBFD%%MapTable.tpl.php).
    Warning: Undefined array key 2 in include() (line 77 of /home/jenkins/bknix-dfl/build/core-23703-3lxzt/web/sites/default/files/civicrm/templates_c/en_US/%%2E/2E3/2E3CBBFD%%MapTable.tpl.php).
    Warning: Trying to access array offset on value of type null in include() (line 77 of /home/jenkins/bknix-dfl/build/core-23703-3lxzt/web/sites/default/files/civicrm/templates_c/en_US/%%2E/2E3/2E3CBBFD%%MapTable.tpl.php).
    Warning: Undefined array key "savedMappingName" in include() (line 83 of /home/jenkins/bknix-dfl/build/core-23703-3lxzt/web/sites/default/files/civicrm/templates_c/en_US/%%2E/2E3/2E3CBBFD%%MapTable.tpl.php).
    Warning: Undefined array key "mappingDetailsError" in include() (line 107 of /home/jenkins/bknix-dfl/build/core-23703-3lxzt/web/sites/default/files/civicrm/templates_c/en_US/%%2E/2E3/2E3CBBFD%%MapTable.tpl.php).
    Warning: Undefined array key "highlightedRelFields" in include() (line 14 of /home/jenkins/bknix-dfl/build/core-23703-3lxzt/web/sites/default/files/civicrm/templates_c/en_US/%%31/319/31965542%%highLightImport.tpl.php).

@aydun
Copy link
Contributor

aydun commented Jun 7, 2022

After hitting Continue on step 2 (Match fields):

Error: Call to undefined method CRM_Activity_Import_Parser_Activity::setFieldMetadata() in CRM_Import_Parser->getMappedFieldLabel() (line 1773 of /home/jenkins/bknix-dfl/build/core-23703-3lxzt/web/sites/all/modules/civicrm/CRM/Import/Parser.php).

@aydun
Copy link
Contributor

aydun commented Jun 7, 2022

When using a saved mapping, I get extra empty dropdowns:
Screenshot-8

@eileenmcnaughton
Copy link
Contributor Author

I just rebased this over the fixes on participant - and got through the whole flow

image

@colemanw
Copy link
Member

colemanw commented Jun 7, 2022

Let's get this merged so we can test them all together

@eileenmcnaughton
Copy link
Contributor Author

@aydun can you re-test once merged & we will address any issues - it's actively broken without this

@eileenmcnaughton eileenmcnaughton merged commit 2b4ed18 into civicrm:master Jun 7, 2022
@eileenmcnaughton eileenmcnaughton deleted the import_activity branch June 7, 2022 23:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants