In block_schema(), the 'block' table, the column 'delta' currently is a varchar(32).

It is quite easy to hit this limit. Especially if the block delta is composed from different parts for disambiguation.
E.g. mymodule-footer_product_categories (the block's module is "menupoly", not "mymodule")
Or #1512656: Block delta too long with some menus ("Better Jump Menus")
Or https://www.google.com/search?q=Data+too+long+for+column+'delta'+drupal+block&oq=Data+too+long+for+column+'delta'+drupal+block&sourceid=chrome&ie=UTF-8

Changing it from varchar(32) to varchar(64) would be an improvement already..

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

dcam’s picture

Issue tags: +Novice

I'm going to assume this doesn't apply to D8 since block information is no longer stored in the database. This should be a good Novice issue.

joshi.rohit100’s picture

Status: Active » Needs review
FileSize
450 bytes

Done 32 => 64

dcam’s picture

Status: Needs review » Needs work

My fault, I should have mentioned that this will need to have an update function.

joshi.rohit100’s picture

Status: Needs work » Needs review
FileSize
619 bytes
dcam’s picture

Status: Needs review » Needs work

Ok, once again I wasn't clear. Both of these changes are necessary. If you can give us a combined patch then I'll RTBC it. I'd do it, but I don't want to steal credit. I tested the combined changes and they're working correctly, on install and on update.

evilfurryone’s picture

Status: Needs work » Needs review
FileSize
900 bytes

I merged the two patches into one.

vishwac9’s picture

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

cilefen’s picture

One of them is a duplicate. It is probably this one, because the other one is older.

David_Rothstein’s picture

Status: Reviewed & tested by the community » Closed (duplicate)

That other issue looks further along since it also handles fixing an additional table that stores the block delta, so yeah, let's close this as a duplicate.