DOException: SQLSTATE[22001]: String data, right truncated: 1406 Data too long for column 'last_abort_function'.

This is the function name string that's too long: 'queue_fuse_scheduled_complete_listings' - 38 chars.

last_abort_function field is varchar(32), maybe it would be good to be 64?

Comments

chaps2’s picture

I second this. 64 should be enough...

gotheric’s picture

Assigned: Unassigned » gotheric
Status: Active » Needs work

Interesting.
I should put a check before the save to database and crop larger strings. This is to avoid the error always (even with strings > 64 chars)

Probably i'll update the column to 64 chars too.

I'll commit something soon, thanks for the issue.

regilero’s picture

Here is a 7.2.x-dev patch

regilero’s picture

Status: Needs work » Needs review
StatusFileSize
new1.9 KB

And this one should work with 7.x-2.1 last stable delivered version

xaris.tsimpouris’s picture

Shouldn't be 120 characters, as "name" column has and mostly includes function names?

adammalone’s picture

Assigned: gotheric » Unassigned
Issue summary: View changes
StatusFileSize
new1.68 KB
  • Patch in #4 does not apply
  • 64 chars is still not enough for some functions
    • fieldable_panels_panes_fieldable_panels_pane_content_type_edit_form_validate
    • panopoly_magic_form_ctools_entity_field_content_type_formatter_options_alter

I've rerolled and refactored the above patch to:

  • Increase the column limit to 128 characters
  • make smaller function names larger than 128
adammalone’s picture

Priority: Normal » Major

Setting this issue to major as it has the potential to stop cron from running on sites and it's a simple enough fix to get into the module. Also hiding patches that no longer apply to latest 7.x-2.x.

randell’s picture

Patch at #6 works as expected.

randell’s picture

Status: Needs review » Reviewed & tested by the community
wylbur’s picture

Applied this patch to our site, which was crashing because of a function name that was too long. This resolved our issue immediately. This is ready for deployment.

wylbur’s picture

Just to confirm, I applied the patch in comment #6.

  • kala4ek committed e90f9c2 on 7.x-2.x authored by typhonius
    Issue #1838956 by regilero, typhonius, modstore: DB Error: Data too long...
kala4ek’s picture

Committed to latest dev.

kala4ek’s picture

Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.