Status

()

RESOLVED INVALID
17 years ago
6 years ago

People

(Reporter: terry, Assigned: justdave)

Tracking

Details

Attachments

(1 attachment)

(Reporter)

Description

17 years ago
I was just stealing the code for url_decode for some other project, when I 
happened to notice a little mistake in the code.  If a URL contains more 
than one "+", it will fail to decode all of them into spaces.

I will attach the (extremely trivial) patch.

This bug has been there ever since Bugzilla was originally ported to perl...
(Reporter)

Comment 1

17 years ago
Created attachment 75530 [details] [diff] [review]
Patch to fix bug in url_decode
Comment on attachment 75530 [details] [diff] [review]
Patch to fix bug in url_decode

tr/// doesn't take a /g parameter.  This won't compile.  tr/// should get all
occurances by default.	If it's not, something else is broken.
Attachment #75530 - Flags: review-
(Reporter)

Comment 3

17 years ago
Never mind.  I am a bucket-head.
Status: NEW → RESOLVED
Last Resolved: 17 years ago
Resolution: --- → INVALID
QA Contact: matty_is_a_geek → default-qa
You need to log in before you can comment on or make changes to this bug.