Status
()
People
(Reporter: terry, Assigned: justdave)
Tracking
Details
Attachments
(1 attachment)
|
896 bytes,
patch
|
justdave
:
review-
|
Details | Diff | Splinter Review |
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
|
(Assignee) | |
Comment 2•17 years ago
|
||
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
|
||
Updated•6 years ago
|
||
QA Contact: matty_is_a_geek → default-qa
You need to log in
before you can comment on or make changes to this bug.
Description
•