I have run into a problem where my dst URI is longer than 128 characters, so i'm gonna modify this module and change it to at least 255 characters.
The request is basically to increase the default lenght of a URI in the database.

CommentFileSizeAuthor
#8 patch_commit_a0ab0db136a7.patch934 bytescyberschorsch
#6 patch_commit_a0ab0db136a7.patch933 bytescyberschorsch

Comments

Anonymous’s picture

Anonymous (not verified) commented

How can I do this too? Amazon affiliate links are way too long for this form. I've changed all instances of 128 to 255 in the module, but for some reason, the form still shows up with a max length of 128, how can I change this?

hass’s picture

hass commented
Title: Limited lenght of dst url » Extend the lenght of the "dst" url to 255
Version: 6.x-1.0 » master
Category: support » feature
Priority: Critical » Normal

Please provide a patch.

hass’s picture

hass commented
Version: master » 6.x-1.x-dev
Status: Active » Fixed

Should be fixed code wise. Can you test, please?

Status: Fixed » Closed (fixed)

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

aschiwi’s picture

aschiwi commented
Version: 6.x-1.x-dev » 7.x-1.1
Status: Closed (fixed) » Active

We stumbled upon the 255 limit and think it should be increased to 2048, which is link fields limit. Will post patch.

cyberschorsch’s picture

cyberschorsch
Primary language German
commented
Title: Extend the lenght of the "dst" url to 255 » Extend the lenght of the "dst" url to 2048
StatusFileSize
new patch_commit_a0ab0db136a7.patch933 bytes
cyberschorsch’s picture

cyberschorsch
Primary language German
commented
Status: Active » Needs review
cyberschorsch’s picture

cyberschorsch
Primary language German
commented
StatusFileSize
new patch_commit_a0ab0db136a7.patch934 bytes

Wrong return message, should be correct now.

aschiwi’s picture

aschiwi commented

works for us - and I think it makes sense to use the same limit that link module uses, right?

steffenr’s picture

steffenr
Primary language German
Location Bremen
commented
Status: Needs review » Reviewed & tested by the community

Just tested the patch in a site i currently work on - it works fine for me too - i'll set the status to rtbc..

SteffenR

jibus’s picture

jibus commented

Tested the patch ! Works for me ! (i use Postgre)

255 was too short for me ! Thanks !

ressa’s picture

ressa
he/him
commented

We also had this problem, extending the max. length of 'dst' field to 2048 fixed it. Ready to be commited?

This is the error message, if somebody else searches for it: PDOException: SQLSTATE[22001]: String data, right truncated: 1406 Data too long for column 'dst' at row 1: INSERT INTO {gotwo} (src, dst) VALUES (:db_insert_placeholder_0, :db_insert_placeholder_1); Array ( [:db_insert_placeholder_0] => ...

  • hass committed eed0e6d on 7.x-1.x authored by Cyberschorsch 
    Issue #370103 by Cyberschorsch: Extend the lenght of the "dst" url to...
hass’s picture

hass commented
Issue summary: View changes
Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

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