1139414 - Replace Bugzilla::Util::url_quote() by URI::Escape::uri_escape_utf8()

Status: NEW

(Bugzilla :: Bugzilla-General, enhancement)

Description

[Frédéric Buclin]

Since Bugzilla 3.4, the URI module is a required module. So we no longer need our own Bugzilla::Util::url_quote() subroutine which is simply a clone of URI::Escape::uri_escape_utf8(). You use it as follow:

use URI::Escape;

my $escaped_string = uri_escape_utf8($string);


URI 1.55 and newer follows RFC 3986 by default, which is what we want, see bug 970283.

Comment 1

[Sneh Vyas]

I would like to work on this as my first bug. I was trying to search for the call sites of the function in question using Mozilla Cross reference but could not find any results. Could someone please provide some pointers on how to begin?

Comment 2

[Frédéric Buclin]

The source code is here: http://git.mozilla.org/?p=bugzilla/bugzilla.git;a=tree [github]. To work on this bug, I suggest you install Bugzilla: http://bugzilla.readthedocs.org/en/latest/installing/index.html

Comment 3

[Akshay Jain]

Can I work on this bug or somebody already doing it?

Comment 4

[Andre Klapper]

"Assigned To" says "Nobody", hence working on this task is welcome :)

Comment 5

[Akshay Jain]

Hello Andre, any tips to help me get started?
I am newbie so I would appreciate it. Thanks.

Comment 6

[Andre Klapper]

@Akshay: Sure, happy to help! Do you have a specific question? :)
(Also see http://catb.org/~esr/faqs/smart-questions.html#explicit - thanks!)

Comment 7

[Pradeep Gangwar [:pradeepgangwar]]

Is this bug solved or i can work on this?

Comment 8

[Firefox Bug Husbandry Bot]

Removing good-first-bug keyword because team does not have bandwidth to mentor at the moment.