Bug 188212 - tinyurl script bug in KDE4 port
Summary: tinyurl script bug in KDE4 port
Status: RESOLVED FIXED
Alias: None
Product: konversation
Classification: Applications
Component: general (show other bugs)
Version: unspecified
Platform: Unlisted Binaries Linux
: NOR normal
Target Milestone: ---
Assignee: Konversation Developers
URL:
Keywords:
Depends on:
Blocks: 187307
  Show dependency treegraph
 
Reported: 2009-03-26 23:45 UTC by Mehmet Nur Olcay
Modified: 2009-04-05 04:02 UTC (History)
1 user (show)

See Also:
Latest Commit:
Version Fixed In:
Sentry Crash Report:


Attachments
patch for tinyurl to use api (872 bytes, patch)
2009-04-05 03:42 UTC, Travis McHenry
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Mehmet Nur Olcay 2009-03-26 23:45:15 UTC
Version:            (using KDE 4.2.1)
Compiler:          gcc 4.3.2 
OS:                Linux
Installed from:    Unspecified Linux

tinyurl does not working with complex urls.

i.e working for http://google.com

but for http://www.google.com.tr/search?hl=tr&q=konversation+kde4&sourceid=navclient-ff&rlz=1B5GGGL_trTR310TR310&ie=UTF-8 

it returns with this error: 

[D-Bus] Error: Unable run tinyurl script, please make sure you have curl or wget installed

looks like has some problems with special characters, could not figure out which ones.
Comment 1 Travis McHenry 2009-04-05 03:42:41 UTC
Created attachment 32603 [details]
patch for tinyurl to use api

This patch makes the script use tinyurl's api instead of screenscraping saving them bandwidth and also fixing this bug since the bug was a result of screenscraping messing up.
Comment 2 Eike Hein 2009-04-05 04:02:47 UTC
SVN commit 949353 by hein:

Make the tinyurl script less insane by using their API
instead of doing screen scraping, also fixing problems
with non-ASCII characters in the process. Thanks to
Travis for the patch.
BUG:188212


 M  +2 -2      tinyurl  


WebSVN link: http://websvn.kde.org/?view=rev&revision=949353