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

CRM-21258 support long display names. #11063

Merged
merged 1 commit into from
Oct 5, 2017

Conversation

eileenmcnaughton
Copy link
Contributor

@eileenmcnaughton eileenmcnaughton commented Oct 4, 2017

Overview

Gracefully truncate rather than hard-error if calculated field display_name field exceeds DB chars

Before

If you create a contact & the length of the calculated display name (e.g. first_name + " " + last_name) exceeds 128 chars you get a DB error

After

Display field & sort field are truncated rather than a hard error

Comments

This is pretty edge case & likely to be bad data but a db error on a calculated field being too long seems inappropriate


Gracefully truncate rather than hard-error if calculated field exceeds DB chars
@colemanw
Copy link
Member

colemanw commented Oct 5, 2017

Makes sense to me. Failure is unrelated.

@colemanw colemanw merged commit 5e39589 into civicrm:master Oct 5, 2017
@eileenmcnaughton eileenmcnaughton deleted the display_name branch October 5, 2017 23:33
wmfgerrit pushed a commit to wikimedia/wikimedia-fundraising-crm that referenced this pull request Oct 9, 2017
On testing I found that what we were setting was being ignored (otherwise the cleanups
would not have flowed through).

I have done an upstream PR to address length handling in core
civicrm/civicrm-core#11063

I don't think it's pressing for us to address long_name handling (ie.
we can suck that PR in now or wait for the next update) but
removing these 2 lines gets rid of the misleading impression that they
have an effect

Change-Id: I0be0a8883e19378019adf29f9b4cb76add4e812e
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants