Bug 188212

Summary:	tinyurl script bug in KDE4 port
Product:	[Applications] konversation	Reporter:	Mehmet Nur Olcay <mnurolcay>
Component:	general	Assignee:	Konversation Developers <konversation-devel>
Status:	RESOLVED FIXED
Severity:	normal	CC:	hein
Priority:	NOR
Version:	unspecified
Target Milestone:	---
Platform:	Unlisted Binaries
OS:	Linux
Latest Commit:		Version Fixed In:
Sentry Crash Report:
Bug Depends on:
Bug Blocks:	187307
Attachments:	patch for tinyurl to use api

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