I couldn't enter the allowed tags string for the fckeditor. Patch bumps to 500 from 255.

CommentFileSizeAuthor
#8 allowed_tags_length_0_d5.patch907 bytesscor
#1 allowed_tags_length.patch695 bytesBioALIEN
allowed_tags_length.patch683 bytesSteve Dondley

Comments

BioALIEN’s picture

BioALIEN commented
Version: 6.2 » 7.x-dev
Assigned: Steve Dondley » BioALIEN
Status: Needs review » Reviewed & tested by the community
StatusFileSize
new allowed_tags_length.patch695 bytes

As discussed on IRC, setting maxlength to 500 doesn't really cut it. It should really be a textarea but for simplicity's sake and to ensure this gets committed quickly, I've increased it to 1024.

Setting issue to D7 then we'll work our way down to D6.

Attached is a patch that increases the maxlength from 255 to a reasonable limit of 1024.

BioALIEN’s picture

BioALIEN commented
Status: Reviewed & tested by the community » Needs review

Setting to CNR for redLED to contribute a little :)

davidlark’s picture

davidlark commented

I just changed this in filter.module ; I know I shouldn't hack core, but it seemed like the easy thing to do. I don't know if this is the place to make such a request, but could this simple change make its way into 6.4? Thanx

BioALIEN’s picture

BioALIEN commented
Status: Needs review » Reviewed & tested by the community

The world is disappointed with redLED.

Marking as RTBC. Please backport this to 6.x and 5.x as the same patch applies.

dries’s picture

dries commented
Status: Reviewed & tested by the community » Fixed

Committed to CVS HEAD and DRUPAL-6. Thanks! :)

BioALIEN’s picture

BioALIEN commented
Version: 7.x-dev » 5.x-dev
Status: Fixed » Reviewed & tested by the community

Thanks Dries, this needs fixing in D5.x also.

davidlark’s picture

davidlark commented

I agree with the 1024 limit, as I just counted the letters in the non-deprecated html4 tags, and it came to 507 including the delimiting characters. Thanks

scor’s picture

scor commented
StatusFileSize
new allowed_tags_length_0_d5.patch907 bytes

rerolling for DRUPAL-5.

drumm’s picture

drumm
he/him
Location NY, US
commented
Status: Reviewed & tested by the community » Fixed

Committed to 5.x.

Anonymous’s picture

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

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