In theme_upload_attachments :

The l function already performs the check_url and check_plain checks.
Currently, we have broken links for attachments whose filename contains 'special chars' such as single quotes.

CommentFileSizeAuthor
#1 upload.module_39.patch958 bytesyched
upload.module_38.patch968 bytesyched
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

yched’s picture

yched CreditAttribution: yched commented
Status: Active » Reviewed & tested by the community
FileSize
upload.module_39.patch958 bytes

this might be worth backporting ? here is the patch for 4.7.

I think this is RTBC, BTW...

beginner’s picture

beginner CreditAttribution: beginner commented

patch doesn't apply because of non Unix standard line endings.

Wouldn't this patch also fix http://drupal.org/node/70742 ?

Dries’s picture

Dries CreditAttribution: Dries commented
Status: Reviewed & tested by the community » Fixed

Committed to CVS HEAD.

yched’s picture

yched CreditAttribution: yched commented
Status: Fixed » Reviewed & tested by the community

reopening just as a reminder for a 4.7 backport ?

killes@www.drop.org’s picture

killes@www.drop.org CreditAttribution: killes@www.drop.org commented
Status: Reviewed & tested by the community » Fixed

backported

Anonymous’s picture

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