Views doesn't link correctly to files with ampersand in their name. That's because the URL is escaped twice.

I'm attaching a patch. I removed the call to check_url(), because it's done in l().

CommentFileSizeAuthor
views_upload.inc_.diff1.22 KBmooffie
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

mikepetersonwi’s picture

mikepetersonwi CreditAttribution: mikepetersonwi commented

I tried applying the patch and received the following:

patching file views_upload.inc
patch unexpectedly ends in middle of line
Hunk #2 succeeded at 194 with fuzz 1.

I'm a little new to patching and am not sure if it's me or the patch file.

Please advise,

Mikep

mikepetersonwi’s picture

mikepetersonwi CreditAttribution: mikepetersonwi commented

I manually made the changes to the views_upload file and still get the same result. Any suggestions?

Mikep

newbuntu’s picture

newbuntu CreditAttribution: newbuntu commented
Title: Ampersand escaped twice in links to files » can't use Ampersand in views menu title

can't use Ampersand in views menu title. May be it's related.

If I enter "A&B" in views "menu title", it shows up "A&B" in the tapped menu. I tried a few combinations to no avail.

I tested regular menu item like 'a&b' (not associated with any views), it worked correctly.

Is there anything I can do (hack) to make it show up correctly?

sun’s picture

sun
Primary language German
Location Karlsruhe
CreditAttribution: sun commented
Title: can't use Ampersand in views menu title » Ampersand escaped twice in links to files

Please do not hi-jack existing issues.

esmerel’s picture

esmerel CreditAttribution: esmerel commented
Status: Needs review » Closed (won't fix)

At this time, only security fixes will be made to the 5.x version of Views.