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

dev/core#2645 Proposed fix for exporting preferred_mail_format field #20626

Merged
merged 1 commit into from
Jun 21, 2021

Conversation

akosgarai
Copy link
Contributor

Overview

A patch for CORE-2645
Exporting in localized civicrm crashes due to long translation for preferred_mail_format varchar(8) field.

Before

Export of all contact data with at least one contact with preferred_mail_format 'Both' in Hungary l10n crashes with
Data too long for column 'preferred_mail_format' exception.

After

The export works again, the csv is downloadable.

Technical Details

The preferred_mail_format is varchar(8) by default, but it is not long enough for the Hungarian translation, so that it has been changed to varchar(16), that is enough long for the translated string.

@civicrm-builder
Copy link

Can one of the admins verify this patch?

@civibot
Copy link

civibot bot commented Jun 16, 2021

(Standard links)

@civibot civibot bot added the master label Jun 16, 2021
@eileenmcnaughton
Copy link
Contributor

test this please

@seamuslee001
Copy link
Contributor

Jenkins add to whitelist

@yashodha
Copy link
Contributor

@akosgarai Can you please squash the commits?

A localized value for this field does not fit into the original varchar(8)

dev/core#2645 Update testcases
@akosgarai
Copy link
Contributor Author

The commits have been sqashed @yashodha

@eileenmcnaughton eileenmcnaughton merged commit 40df867 into civicrm:master Jun 21, 2021
@yashodha
Copy link
Contributor

@akosgarai Thanks!

kcristiano added a commit to tadpolecc/civicrm that referenced this pull request Nov 4, 2021
kcristiano added a commit to tadpolecc/civicrm that referenced this pull request Nov 4, 2021
kcristiano added a commit to tadpolecc/civicrm that referenced this pull request Nov 4, 2021
kcristiano added a commit to tadpolecc/civicrm that referenced this pull request Nov 4, 2021
kcristiano added a commit to tadpolecc/civicrm that referenced this pull request Nov 15, 2021
kcristiano added a commit to tadpolecc/civicrm that referenced this pull request Nov 15, 2021
kcristiano added a commit to tadpolecc/civicrm that referenced this pull request Dec 4, 2021
kcristiano added a commit to tadpolecc/civicrm that referenced this pull request Jan 7, 2022
kcristiano added a commit to tadpolecc/civicrm that referenced this pull request Jan 7, 2022
kcristiano added a commit to tadpolecc/civicrm that referenced this pull request Jan 12, 2022
kcristiano added a commit to tadpolecc/civicrm that referenced this pull request Jan 12, 2022
kcristiano added a commit to tadpolecc/civicrm that referenced this pull request Jan 29, 2022
kcristiano added a commit to tadpolecc/civicrm that referenced this pull request Jan 29, 2022
kcristiano added a commit to tadpolecc/civicrm that referenced this pull request Feb 4, 2022
kcristiano added a commit to tadpolecc/civicrm that referenced this pull request Feb 4, 2022
kcristiano added a commit to tadpolecc/civicrm that referenced this pull request Feb 4, 2022
kcristiano added a commit to tadpolecc/civicrm that referenced this pull request Feb 4, 2022
kcristiano added a commit to tadpolecc/civicrm that referenced this pull request Feb 9, 2022
kcristiano added a commit to tadpolecc/civicrm that referenced this pull request Feb 9, 2022
kcristiano added a commit to tadpolecc/civicrm that referenced this pull request Aug 4, 2022
kcristiano added a commit to tadpolecc/civicrm that referenced this pull request Aug 9, 2022
kcristiano added a commit to tadpolecc/civicrm that referenced this pull request Aug 13, 2022
kcristiano added a commit to tadpolecc/civicrm that referenced this pull request Aug 31, 2022
kcristiano added a commit to tadpolecc/civicrm that referenced this pull request Sep 8, 2022
kcristiano added a commit to tadpolecc/civicrm that referenced this pull request Oct 9, 2022
kcristiano added a commit to tadpolecc/civicrm that referenced this pull request Oct 28, 2022
kcristiano added a commit to tadpolecc/civicrm that referenced this pull request Nov 3, 2022
kcristiano added a commit to tadpolecc/civicrm that referenced this pull request Nov 8, 2022
kcristiano added a commit to tadpolecc/civicrm that referenced this pull request Nov 18, 2022
kcristiano added a commit to tadpolecc/civicrm that referenced this pull request Dec 9, 2022
kcristiano added a commit to tadpolecc/civicrm that referenced this pull request Dec 21, 2022
kcristiano added a commit to tadpolecc/civicrm that referenced this pull request Jan 5, 2023
kcristiano added a commit to tadpolecc/civicrm that referenced this pull request Jan 5, 2023
kcristiano added a commit to tadpolecc/civicrm that referenced this pull request Jan 12, 2023
kcristiano added a commit to tadpolecc/civicrm that referenced this pull request Jan 26, 2023
kcristiano added a commit to tadpolecc/civicrm that referenced this pull request Jan 31, 2023
kcristiano added a commit to tadpolecc/civicrm that referenced this pull request Feb 2, 2023
kcristiano added a commit to tadpolecc/civicrm that referenced this pull request Feb 16, 2023
kcristiano added a commit to tadpolecc/civicrm that referenced this pull request Mar 2, 2023
kcristiano added a commit to tadpolecc/civicrm that referenced this pull request Mar 6, 2023
kcristiano added a commit to tadpolecc/civicrm that referenced this pull request Mar 11, 2023
kcristiano added a commit to tadpolecc/civicrm that referenced this pull request Mar 16, 2023
kcristiano added a commit to tadpolecc/civicrm that referenced this pull request Mar 28, 2023
kcristiano added a commit to tadpolecc/civicrm that referenced this pull request Apr 6, 2023
kcristiano added a commit to tadpolecc/civicrm that referenced this pull request May 4, 2023
kcristiano added a commit to tadpolecc/civicrm that referenced this pull request May 6, 2023
kcristiano added a commit to tadpolecc/civicrm that referenced this pull request May 16, 2023
kcristiano added a commit to tadpolecc/civicrm that referenced this pull request May 20, 2023
kcristiano added a commit to tadpolecc/civicrm that referenced this pull request May 24, 2023
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.

5 participants