https://bugzilla.wikimedia.org/show_bug.cgi?id=27876

           Summary: unnecessary things in mw.Uri.js
           Product: MediaWiki extensions
           Version: any
          Platform: All
        OS/Version: All
            Status: NEW
          Severity: enhancement
          Priority: Normal
         Component: UploadWizard
        AssignedTo: [email protected]
        ReportedBy: [email protected]
                CC: [email protected]


Michael Dale has argued me into thinking that the following things about this
class are overkill.

- Breaking down userinfo into user & password. Our primary use case is HTTP and
URI passwords are actually illegal there anyway. Removing this will save
several internal methods and tests when serializing the URI.

- The uri.clone() method, which is just sugar for another new() call

- having its own encode function -- there is such a function already in
mediawiki.util.js, called rawurlencode , should standardize on that.

- should investigate whether jQuery.param() can replace the path serialization
code (not sure if we can do that and also have the slightly custom encoding &
decoding that mediawiki seems to like).

Arguably this bug is a request for "de-enhancement". But we need more people
who think like that ;)

-- 
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
[email protected]
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l

Reply via email to