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.

CommentFileSizeAuthor
#4 feeds-config-columns-1715124-4.patch1.79 KBtheduke
#2 feeds-config-column-size-1715124-1.patch852 bytestheduke
#1 feeds-config-column-size-1715124-1.patch851 bytestheduke

Comments

theduke’s picture

theduke commented
Priority: Normal » Major
Status: Active » Needs review
StatusFileSize
new feeds-config-column-size-1715124-1.patch851 bytes

Patch.

Also changing priority to major since this completely breaks functionality once reaching a certain config size.

theduke’s picture

theduke commented
StatusFileSize
new feeds-config-column-size-1715124-1.patch852 bytes

Whoops, missing comma...

twistor’s picture

twistor commented
Status: Needs review » Needs work

hook_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?

theduke’s picture

theduke commented
Status: Needs work » Needs review
StatusFileSize
new feeds-config-columns-1715124-4.patch1.79 KB

I 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.

theduke’s picture

theduke commented

Are the changes ok for you twistor?

stylesuxx’s picture

stylesuxx commented

As stated (and closed as duplicate) here: http://drupal.org/node/1736438 I'd like to see this changed too :-)

theduke’s picture

theduke commented

Bumping...

twistor’s picture

twistor commented
Status: Needs review » Patch (to be ported)

Comitted to 7.x
http://drupalcode.org/project/feeds.git/commit/ceb98dd

twistor’s picture

twistor commented

How many mappings do you have?

stylesuxx’s picture

stylesuxx commented

About 320.

lyricnz’s picture

lyricnz commented

It seems db_change_field() didn't work for SQL server - #1828950: Update to alpha7 database update fails on MSSQL

megachriz’s picture

megachriz
Primary language Dutch
commented
Issue summary: View changes
Status: Patch (to be ported) » Fixed

This was fixed a long time ago.

megachriz’s picture

megachriz
Primary language Dutch
commented
Status: Fixed » Closed (fixed)