[Bug 27876] unnecessary things in mw.Uri.js

2011-11-17 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=27876

Neil Kandalgaonkar  changed:

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution||FIXED

--- Comment #5 from Neil Kandalgaonkar  2011-11-17 
22:04:04 UTC ---
Fixed, and it handles relative urls too now.

-- 
Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug.
You are on the CC list for the bug.

___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 27876] unnecessary things in mw.Uri.js

2011-03-08 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=27876

--- Comment #3 from Neil Kandalgaonkar  2011-03-08 
21:50:01 UTC ---
@mdale - an example? You mean URLs with '..' in the path or URLs which are just
paths, no protocol or host??

-- 
Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 27876] unnecessary things in mw.Uri.js

2011-03-08 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=27876

Krinkle  changed:

   What|Removed |Added

 Blocks||27730

-- 
Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 27876] unnecessary things in mw.Uri.js

2011-03-08 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=27876

--- Comment #2 from Michael Dale  2011-03-08 19:40:40 UTC ---
I would add that it would be nice if mw.Uri more gracefully handles relative
urls

-- 
Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 27876] unnecessary things in mw.Uri.js

2011-03-08 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=27876

--- Comment #1 from Neil Kandalgaonkar  2011-03-08 
18:10:16 UTC ---
Re: rawurlencode:

Had a look at mediawiki.util.js -- it's got a lot of miscellaneous stuff and
the dependencies pull in a lot of other jQuery modules. There are three methods
which are URI-related 
 - rawurlencode
 - wikiurlencode
 - getParamValue

Seems to me that the right thing is actually to:
- move mw.Uri.js to be a new module, mediaWiki.uri.js, ideally no dependencies
- refactor rawurlencode, and a few other such methods, into mediaWiki.Uri.js.
- refactor/rethink getParamValue, it is way better to do 
mw.Uri( currentUrl ).query[ 'name' ]

Also see bug #27730 which was discussing yet another method for url
manipulation

-- 
Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l