This patch sets the maxlength to 2000 up from the drupal default of 120

CommentFileSizeAuthor
media_internet_maxlength.patch516 bytesjames.elliott

Comments

Status: Needs review » Needs work

The last submitted patch, media_internet_maxlength.patch, failed testing.

JacobSingh’s picture

JacobSingh commented
Status: Needs work » Reviewed & tested by the community
james.elliott’s picture

james.elliott commented

Committed

james.elliott’s picture

james.elliott commented
Status: Reviewed & tested by the community » Fixed
effulgentsia’s picture

effulgentsia commented

+1 that 120 was too short. If we're going to raise it 2000 anyway, let's raise it 2083. I committed that with the following code comment:

    // There is no standard specifying a maximum length for a URL. Internet
    // Explorer supports upto 2083 (http://support.microsoft.com/kb/208427), so
    // we assume publicly available media URLs are within this limit.
    '#maxlength' => 2083,

Status: Fixed » Closed (fixed)

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