[Bug 25934] Optionally enable urldecode for external links

2012-09-28 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=25934

Bartosz Dziewoński matma@gmail.com changed:

   What|Removed |Added

 CC||matma@gmail.com
 Blocks|27292   |

--- Comment #10 from Bartosz Dziewoński matma@gmail.com 2012-09-28 
18:17:13 UTC ---
Removing bug 27292 as blocker, this has nothing to do with skins. (I don't
think there's a Parser tracking bug, at least I didn't find one.)

-- 
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 25934] Optionally enable urldecode for external links

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

Sumana Harihareswara suma...@panix.com changed:

   What|Removed |Added

   Keywords|need-review |reviewed
 CC||suma...@panix.com

--- Comment #9 from Sumana Harihareswara suma...@panix.com 2011-11-08 
14:03:14 UTC ---
Dmitriy, thank you for the patch.  I'm sorry it has taken so long for us to
respond to you!

Because it's been so long, when I tried to apply your patch, it didn't apply
cleanly to trunk.  But before you try to revise it so it applies, will you come
into the #mediawiki channel on FreeNode IRC and ask for a more thorough review
of your patch?  That way you won't waste time redoing work.  Thanks!

-- 
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 25934] Optionally enable urldecode for external links

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

Reedy s...@reedyboy.net changed:

   What|Removed |Added

 Blocks||27292
 Depends on|27292   |

-- 
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 25934] Optionally enable urldecode for external links

2011-02-24 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=25934

Dmitriy Sintsov c...@uniyar.ac.ru changed:

   What|Removed |Added

   Attachment #7818|0   |1
is obsolete||
   Attachment #7827|0   |1
is obsolete||

--- Comment #8 from Dmitriy Sintsov c...@uniyar.ac.ru 2011-02-24 11:13:25 UTC 
---
Created attachment 8203
  -- https://bugzilla.wikimedia.org/attachment.cgi?id=8203
Try to urldecode external links similar to the way Firefox does

Useful to improve readability of local UTF-8 encoded external links. Made with
1.17 branch.

-- 
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 25934] Optionally enable urldecode for external links

2011-02-24 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=25934

Bawolff bawolff...@gmail.com changed:

   What|Removed |Added

   Attachment #8203|0   |1
   is patch||
   Attachment #8203|application/octet-stream|text/plain
  mime type||

-- 
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 25934] Optionally enable urldecode for external links

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

--- Comment #7 from Brion Vibber br...@pobox.com 2011-02-12 06:26:31 UTC ---
Stumbled on this in bugzilla... I like the basic idea of the patch, but there's
a couple of issues which'll need to be worked out.

First, not all URLs with encoded characters are encoded in UTF-8... while we
like to hope that most of them are in this day and age, there's no guarantee.
Russian, Japanese, Chinese, etc sites may still use other national encodings,
especially on older links...

Reasonable behavior would at least need to check for UTF-8 validity to avoid
outputting garbage characters.


Second, there are lots of meaningful characters in URLs where the difference
between being encoded and not actually changes the URL; for instance Firefox
will show

http://en.wikipedia.org/wiki/What%27s_Eating_Gilbert_Grape%3F

as:

http://en.wikipedia.org/wiki/What's_Eating_Gilbert_Grape%3F

and not as:

http://en.wikipedia.org/wiki/What%27s_Eating_Gilbert_Grape?
which would actually point to [[What's Eating Gilbert Grape]] with an empty
query string on the end if you copy-pasted the text.


Third, if the above are resolved, I'd probably be pretty happy with *not*
adding the parameter on makeExternalLink() -- it's probably sane behavior for
the automatic formatting of the bare link display in pretty much all cases.

-- 
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 25934] Optionally enable urldecode for external links

2011-02-09 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=25934

Mark A. Hershberger m...@everybody.org changed:

   What|Removed |Added

 CC||m...@everybody.org
 Depends on||27292

-- 
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 25934] Optionally enable urldecode for external links

2011-01-31 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=25934

Siebrand s.mazel...@xs4all.nl changed:

   What|Removed |Added

   Keywords||i18n
 CC||s.mazel...@xs4all.nl

-- 
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 25934] Optionally enable urldecode for external links

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

--- Comment #5 from Dmitriy c...@uniyar.ac.ru 2010-11-17 19:21:50 UTC ---
Created attachment 7827
  -- https://bugzilla.wikimedia.org/attachment.cgi?id=7827
urldecode of text for local utf8 characters, just as in major browser's address
line

improves the readability of such links a lot.

-- 
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 25934] Optionally enable urldecode for external links

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

--- Comment #6 from Dmitriy c...@uniyar.ac.ru 2010-11-17 19:22:03 UTC ---
New patch which should have better compatibility to the existing Linker / Skin
usage.

-- 
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 25934] Optionally enable urldecode for external links

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

Bawolff bawolff...@gmail.com changed:

   What|Removed |Added

   Keywords||need-review

-- 
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 25934] Optionally enable urldecode for external links

2010-11-15 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=25934

Bawolff bawolff...@gmail.com changed:

   What|Removed |Added

   Keywords||patch
 CC||bawolff...@gmail.com

--- Comment #1 from Bawolff bawolff...@gmail.com 2010-11-15 12:50:38 UTC ---
Just to clarify, this is for decoding the text part of the link, not the url in
the href?

The idea itself sounds sane if the user just writes a url in the wiki (at least
imho). However i don't think we'd want to url-decode something like:

[http://example.com some text for the link with %25 in it]

If the user specified the text for the link, we should assume they know what
they are doing and not decode it. (your patch would decode both).

-- 
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 25934] Optionally enable urldecode for external links

2010-11-15 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=25934

--- Comment #2 from Dmitriy c...@uniyar.ac.ru 2010-11-15 16:08:15 UTC ---
Firefox is performing entities decode for URL in address line. For example, try
to place the following link to wiki page (even without the patch), then open it
and look at address line:
http://ru.wikipedia.org/wiki/%D0%94%D1%80%D0%BE%D1%84%D0%B0
Series of hex codes %xx were replaced with Cyrillic chars, which are readable
to anyone who knows Cyrillic alphabet.

However, when you copy/paste such URL to text editor back from address line,
%xx will reappear again - so internally that is the same binary representation,
the decoding is preformed only for visualization.

Opera and Safari probably does this, too. IE8 - does not, haven't checked IE9,
yet.

[http://ru.wikipedia.org/wiki/%D0%94%D1%80%D0%BE%D1%84%D0%B0
%D0%94%D1%80%D0%BE%D1%84%D0%B0], the description is not decoded with my patch,
only URL. Quite opposite, however probably matches Firefox logic.

-- 
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 25934] Optionally enable urldecode for external links

2010-11-15 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=25934

Derk-Jan Hartman hart...@videolan.org changed:

   What|Removed |Added

 CC||hart...@videolan.org

--- Comment #3 from Derk-Jan Hartman hart...@videolan.org 2010-11-15 20:07:17 
UTC ---
I'm in favor of this idea, and the implementation is only for links in wikicode
without text. I think that with all the international versions we have, this
would be a welcome change for many.

-- 
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 25934] Optionally enable urldecode for external links

2010-11-15 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=25934

--- Comment #4 from Bawolff bawolff...@gmail.com 2010-11-16 01:47:04 UTC ---
Whoops. You're right this doesn't affect [http://example.com
%E0%B4%B3%E0%B5%8D%E2%80%8D] style links since the parser sets the $escape
argument for false for those kind of links. However it still seems a bit weird,
as if I do something like $sk-makeExternalLink( http://example.com/some_url;,
some text that is not a url, entered by the user, containing a %HH code ); in
an extension, the result would probably not be what is expected.

-- 
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