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

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

Assigned: Unassigned » dww
Issue tags: +drupal.org upgrade
dww’s picture

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

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
new685 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

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.