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

dev/core#2835 - Activity export broken in 5.41 #21456

Merged
merged 1 commit into from
Sep 13, 2021

Conversation

demeritcowboy
Copy link
Contributor

@demeritcowboy demeritcowboy commented Sep 13, 2021

Overview

https://lab.civicrm.org/dev/core/-/issues/2835

Before

  1. Find Activities
  2. Select a couple and from the dropdown choose export.
  3. You're now on a different screen, not the export screen - which screen seems to depend where you were before.
  4. Warning: Illegal offset type in CRM_Activity_StateMachine_Search->__construct() (line 41 of ...\CRM\Activity\StateMachine\Search.php).

After

Works

Technical Details

This seems intended to be a cast to array not a nested array:

20eb1cf#diff-0118f13b1d17a95ba175eca5956f83bcf00341a38d4fa9741f275ff44e8252b3R160

Comments

Before the patch the unit test fails for EXPORT, EMAIL, and BATCH.

@civibot
Copy link

civibot bot commented Sep 13, 2021

(Standard links)

@civibot civibot bot added the 5.42 label Sep 13, 2021
@eileenmcnaughton
Copy link
Contributor

thanks @demeritcowboy

@eileenmcnaughton eileenmcnaughton merged commit cc3be6c into civicrm:5.42 Sep 13, 2021
@demeritcowboy demeritcowboy deleted the activity-export542 branch September 14, 2021 00:58
@demeritcowboy
Copy link
Contributor Author

Thanks. I guess I'll put up a 5.41 version in case there's a 5.41 release.

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.

2 participants