- Issue created by @bluegeek9
- Merge request !44Make Relationship Form's Contact A and Contact B fields readonly when editing using Read-only Field Widget β (Open) created by Unnamed author
- πΊπΈUnited States jdleonard Austin, TX, USA
Created an MR that uses the Read-only Field Widget β module. Not sure I'm crazy about my approach.
-
bluegeek9 β
committed 09377816 on 1.0.x
Issue #3526507: Relationship: read-only contact fields
-
bluegeek9 β
committed 09377816 on 1.0.x
- πΊπΈUnited States bluegeek9
Your solution is more aesthetic.
I am hesitant to add a dependency unless necessary.
- πΊπΈUnited States jdleonard Austin, TX, USA
I agree. Should we enforce it with validation too (e.g. to prevent ECA actions from doing things they shouldn't)?
- πΊπΈUnited States bluegeek9
Form validation, or entity validation? We could add form validation
I am hesitant to add entity validation until we have a contact merger feature.
When merging two contacts, how do we update the entity reference fields? Do we use the entity api and save a new revision, or do we update the database tables, including revision tables, directly, and invalidate the contact tags? - πΊπΈUnited States jdleonard Austin, TX, USA
I would assume that entity validation would be the only way to prevent ECA actions from being naughty.
I see the challenge with merging. IIRC there's a way to bypass entity constraints. If that's the case (do you know?), I think entity validation is the way to go.
- πΊπΈUnited States bluegeek9
I think entity validation is the correct solution. I am learning toward direct database updates when merging contacts.
- πΊπΈUnited States jdleonard Austin, TX, USA
Why direct? Wouldn't that restrict CRM to SQL backends?