Skip to content

Fix #125 set contact_id for membership edit trigger. Also fix inconsistencies with capitals/lowercase of entity name

mattwire requested to merge mattwire/civirules:membershipcontactid into master

Fix #125 (closed) set contact_id for membership edit trigger. Also fix inconsistencies with capitals/lowercase of entity name.

@maynardsmith Please can you test and confirm if it fixes your problem?

The issue is that the Membership Edit trigger does not always pass the contact_id so we have to look it up. I have done this at the earliest point where the entityData is set and also made sure that other properties are set correctly.

I have also fixed inconsistencies with the entity Data where some places in the code expect a case sensitive version, others convert to lowercase and others convert to "ucwords". As CiviCRM uses "CamelCase" eg. Contact, EntityTag I think it is sensible to follow that and we don't have to keep converting the case.

Merge request reports