Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
I have an import with quite a lot of fields mapped.
When i try to add an additional mapping field I get the error "Data truncated. Data too long for column..."
Solution is to set the config column size to big.
Attaching patch in first comment.
Comment | File | Size | Author |
---|---|---|---|
#4 | feeds-config-columns-1715124-4.patch | 1.79 KB | theduke |
#2 | feeds-config-column-size-1715124-1.patch | 852 bytes | theduke |
#1 | feeds-config-column-size-1715124-1.patch | 851 bytes | theduke |
Comments
Comment #1
theduke CreditAttribution: theduke commentedPatch.
Also changing priority to major since this completely breaks functionality once reaching a certain config size.
Comment #2
theduke CreditAttribution: theduke commentedWhoops, missing comma...
Comment #3
twistor CreditAttribution: twistor commentedhook_schema needs to be updated as well. Also, while we're at it, let's change from text to blob. Would it make more sense to split the configs into three separate columns? How many mappings do you have?
Comment #4
theduke CreditAttribution: theduke commentedI don't think it's necessary to split into 3 columns.
MySQL longblob/longtext columns can hold 4GB of data.
That should suffice.
I'm attaching a patch that changes both config fields to blob.
It's probably not necessary to change the source config to big,
but let's make them consistent.
Comment #5
theduke CreditAttribution: theduke commentedAre the changes ok for you twistor?
Comment #6
stylesuxx CreditAttribution: stylesuxx commentedAs stated (and closed as duplicate) here: http://drupal.org/node/1736438 I'd like to see this changed too :-)
Comment #7
theduke CreditAttribution: theduke commentedBumping...
Comment #8
twistor CreditAttribution: twistor commentedComitted to 7.x
http://drupalcode.org/project/feeds.git/commit/ceb98dd
Comment #9
twistor CreditAttribution: twistor commentedHow many mappings do you have?
Comment #10
stylesuxx CreditAttribution: stylesuxx commentedAbout 320.
Comment #11
lyricnz CreditAttribution: lyricnz commentedIt seems db_change_field() didn't work for SQL server - #1828950: Update to alpha7 database update fails on MSSQL
Comment #12
MegaChriz CreditAttribution: MegaChriz as a volunteer commentedThis was fixed a long time ago.
Comment #13
MegaChriz CreditAttribution: MegaChriz as a volunteer commented