The maxlength for the embed url/markup field is fixed at 1024. This is too short for some providers (e.g. spikedhumor). As a result, markup is truncated.

This should increased to something much larger. The MySQL column used to store this value is a LONGTEXT, so it's pretty big....

Comments

aaron’s picture

aaron commented
Priority: Normal » Critical

thanks. i think this has been fixed, but not sure. bumping to make sure.

aaron’s picture

aaron commented
Status: Active » Fixed

changed to 2048.

Anonymous’s picture

Anonymous (not verified) commented
Status: Fixed » Closed (fixed)

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

drupalhooked’s picture

drupalhooked commented
Status: Closed (fixed) » Needs review

Hello Aaron,

Thanks for this 2048 length fix. But it is a bit short for the provider that I am embedding audio from.
Actually the problem arise when I try to to embed an audio from Internet Archive. Drupal notifies me the following error:
Audio from other site cannot be longer than 2048 characters but is currently 2474 characters long.

Is there any way to increase this length so that lengthy embed code could also be saved?

Many thanks in Advance.

Gaurav,

drupalhooked’s picture

drupalhooked commented
Title: Embed url/markup field fixed at 1024 (too short for some providers) » Embed url/markup field fixed at 1048 (too short for some providers)
drupalhooked’s picture

drupalhooked commented
Title: Embed url/markup field fixed at 1048 (too short for some providers) » Embed url/markup field fixed at 2048 (too short for some providers)
aaron’s picture

aaron commented
Version: 5.x-1.x-dev » 6.x-1.x-dev
Priority: Critical » Normal
Status: Needs review » Fixed

OK, changed this to 4096 now.

drupalhooked’s picture

drupalhooked commented

Hi Aaron, Thank you very much for this fix. Now to get it working should I re-download/re-install this module?

Status: Fixed » Closed (fixed)

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