participant status condition form gives an error

When trying to add a participant status condition the form doesn't show. An error is thrown that says 'entity participant does not implement status condition'

mentioned in merge request !59 (merged)

@Joost Can you look at !55 (merged) - I think that fixes at least a part of this.

@Joost I think I fixed it last week here: 8472860a

@jaapjansma there were 2 problems with the participant status comparisson. One of them was that the list of possible statusses wasn't displayed on the form, and your commit fixes that. The other one is that form didn't get displayed in because of the missing condition for participant status here: https://lab.civicrm.org/extensions/civirules/-/blob/master/CRM/CivirulesConditions/Form/Status.php#L62

@mattwire That's indeed part of a solution. But the renaming of the participant status class from CRM_CivirulesConditions_Participant_ParticipantStatus to CRM_CivirulesConditions_Participant_status will be a neccessity to make the generic status comparisson work with participant statuses as well. That's a large part of my merge request.

I can rebase my merge request as soon as your code gets merged in, that way the generic status comparisson is fixed for the entities you fixed as well as for participant statuses.

Great @Joost Can you test and comment on !55 (merged) so that @jaapjansma is confident to merge?

I have merged !55 (merged)

I have rebased my merge request

Hey @Joost the conditions are stored in https://lab.civicrm.org/extensions/civirules/-/blob/master/sql/conditions.json#L62

And then in the upgrader you can use a function like this: CRM_Civirules_Utils_Upgrader::insertConditionsFromJson($this->extensionDir . DIRECTORY_SEPARATOR . 'sql/conditions.json');

Could you change your code to that? Thanks anyway for all your work!

@jaapjansma I changed that to match what you said.

closed via merge request !59 (merged)

mentioned in commit 599e355f

Thanks!