SearchKit - Support conditional links

[colemanw]

Overview

Advanced feature to conditionally show/hide links in a links/buttons/menu column, based on user permissions or row values.

Before

Links would display unconditionally.

After

Links can be hidden based on user permissions (e.g. "access CiviCase") or according to a value (e.g. is_active = TRUE).

[civibot]

(Standard links)

• If this is your first pull-request for CiviCRM, please browse CONTRIBUTING.md for information about the development and testing processes.
• If you are reviewing this pull-request, you may wish to consult the test sites and the Review Standards (long template, short template).

[eileenmcnaughton]

@colemanw in trying to test this PR I hit a possible recent regression - going to this page

http://core-22557-5o32p.test-3.civicrm.org:8001/civicrm/acl/entityrole?action=update&id=2

And attempting to save I get this

My understanding is that the CRM_Core_BAO_Managed::on_hook_civicrm_post function should only be called on the Managed entity - but here it is responding to the entity role

[colemanw]

@eileenmcnaughton it always responds to the hook (because that's the only way to implement a hook) but then decides to act based on the entity name. Why on earth is entity = null in that event? That's the problem.

[eileenmcnaughton]

@colemanw yeah that is fixed now - so I can try again to replicate this

[eileenmcnaughton]

@colemanw did you make more changes? I just tested this - it took me a bit to figure out what it was doing but once it did I added some docs

https://lab.civicrm.org/documentation/docs/user-en/-/merge_requests/548

The only 'issue' (more like a feature gap) I noticed is that for tokens I can use fields not in the display but for the conditional links I can't. What I tried to do was add a link on the email field to the url

civicrm/activity/email/add?action=add&reset=1&selectedChild=activity&atype=3&email_id=[Contact_Email_contact_id_01.id]

but only if the on_hold field - which is not in the display = 0

Also side-issue - that url if kind of a hidden url (since normally cid rather than email id is used) but I think it would be good to define as a link in the email entity - we do define non-crud links?

[colemanw]

Thanks for the review @eileenmcnaughton. The change I made was to take your code style suggestions.
Yes I kept the code simpler and avoided making myself crazy by only supporting fields in the SELECT clause in these conditions.