Visit the /project page and any project type with a space in it has an unencoded space in the URL.

CommentFileSizeAuthor
#4 375349-4.project_plus_not_urlencode.patch685 bytesdww

Comments

webernet’s picture

webernet commented
Title: Project yype URLs can contain spaces » Project type URLs can contain spaces
dww’s picture

dww
we/he/they
commented
Assigned: Unassigned » dww
Issue tags: +drupal.org upgrade
dww’s picture

dww
we/he/they
commented
Category: bug » task
Status: Active » Postponed (maintainer needs more info)

Hrm, I'm confused.

A) The spaces are all urlencoded when I look on d6.d.o. e.g. http://d6.drupal.org/project/installation%20profiles

B) Nothing at all changed in the project code for this between D5 and D6. AFAICT, the change must be due to something deep inside url() that changed in core.

tempted to mark this "by design", but I'd like to get more info first...

dww’s picture

dww
we/he/they
commented
Title: Project type URLs can contain spaces » Keep '+' in the project type URLs that contain spaces
Priority: Normal » Minor
Status: Postponed (maintainer needs more info) » Needs review
StatusFileSize
new 375349-4.project_plus_not_urlencode.patch685 bytes

After further inspection, the URLs are properly encoded. The confusing thing is that FF3 apparently decodes and renders the link with the raw space (both in the status bar on hover, and in the main URL area when you click on it).

Anyway, due to (B), the only way I can think of to avoid this behavior is to just munge the link we get back from l() to replace %20 with +. Seems like a hack, but it'll maintain the existing behavior...

dww’s picture

dww
we/he/they
commented
Status: Needs review » Fixed

Committed to HEAD.

Status: Fixed » Closed (fixed)
Issue tags: -drupal.org upgrade

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

Issue tags: +drupal.org upgrade

Restoring issue tags, see #2125755: System messages removed all issue tags during D7 upgrade.