Closed (fixed)
Project:
Broken Link
Version:
8.x-3.1
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
14 Sep 2020 at 14:08 UTC
Updated:
3 Feb 2023 at 07:09 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #2
hmdnawaz commentedComment #3
hmdnawaz commentedComment #4
hmdnawaz commentedThe query_string base field also uses the wrong length. We need to set the max length of the field instead of adding a length constraints.
Here is the new patch.
Comment #5
jcandan commentedRan into this error for the query_string base field after a Drupal core upgrade from 9.2.10 to 9.3.3. However, after upgrading to 9.2.11 I was not able to reproduce this error.
This seems like an arbitrary maximum. We tried bumping it to 20,000 (almost the 21,844 character maximum) with no success. I feel like this needs further assessment to identify just what is being stored here and why. For me, this may be related to Media as it happens on node save with featured image field.
Comment #8
unstatu commentedI have created a MR implementing the change that was in #4.
I have also implemented a hook_update in order to update the existing field definitions in case they were already installed.
Comment #9
unstatu commentedThe update database operation was failing if the table already contained data. Improved the patch to manage this situation
Comment #11
mahaveer003 commentedComment #12
mahaveer003 commentedThanks for your efforts merge change to branch (8.x-3.x)