Bug 81190 - "open with" downloads 10mb video and then checks if program is available, no cache used
Summary: "open with" downloads 10mb video and then checks if program is available, no ...
Status: RESOLVED FIXED
Alias: None
Product: konqueror
Classification: Applications
Component: general (show other bugs)
Version: unspecified
Platform: unspecified Linux
: NOR normal
Target Milestone: ---
Assignee: Konqueror Developers
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2004-05-09 12:52 UTC by soloturn99
Modified: 2008-11-05 02:08 UTC (History)
2 users (show)

See Also:
Latest Commit:
Version Fixed In:


Attachments
Possible fix for this bug, please test (694 bytes, patch)
2008-06-30 10:19 UTC, David Faure
Details

Note You need to log in before you can comment on or make changes to this bug.
Description soloturn99 2004-05-09 12:52:55 UTC
Version:           3.2.2 (using KDE 3.2.2,  (testing/unstable))
Compiler:          gcc version 3.3.3 (Debian 20040401)
OS:          Linux (i686) release 2.6.5-1-k7-smp

on http://www.go-mono.com/monologue/ i saw miguel de icazas note about us helicopter firing on wounded in iraq. if you click on the video link (http://www.hraunfjord.com/apache_killing.mpeg) and type "mplyer" in the open with box, it downloads it and THEN givs an error "no program".
trying again, this time with correctly typing "mplayer", it downloads it AGAIN and then playing it.

problems:
1. first download then check
2. download again on second click
3. no easy way to remember "mplayer"
   should play this.
Comment 1 soloturn99 2004-05-09 13:33:23 UTC
hmm ... report bug out of konqueror is strange, it lost versions ... and assistant did not display the version info before posting. opening help-bug report again and try to mark and past here also does not work, had to click "submit bug report" to get this for copy paste:

KDE Version
3.2.2 (KDE 3.2.2, (testing/unstable))


Application
Web Browser/File Manager


Operating System
Linux (i686) release 2.6.5-1-k7-smp


Compiler
gcc version 3.3.3 (Debian 20040401)
Comment 2 Florian Sievert 2006-08-30 18:06:19 UTC
He is right. It could save a lot of time, if konqueror would check first, if the wanted application really exist and then start the download. So caching is not really needed! This problem does not only affect video, but all kind of files.

Reproduce:
1. Open kde.org in konqueror
2. Right click on "home" in the menu.
3. Select Open With -> Other
4. Enter as application "kwritewrongspelled"

Result:
Konqeror starts to down the selected file, but nothing else happens, because he doesn't know this application.

Expecting:
Konqeror does check, if "kwritewrongspelled" is avaible. If it is not avaible he does warn the user, that the application is unknown and might be misspelled. If it does exist, Konqueror does start the download process and opens the document.

Using this example does demonstrate the problem. However, try imaging a user is download a "200MB" video file.
Comment 3 Daniel Hahler 2007-04-16 23:01:34 UTC
While trying to reproduce comment #2, I get NO ERROR at all here?! It works however when using an existing application, e.g. "kate".

(using Kubuntu Feisty)
Comment 4 David Faure 2008-06-30 10:19:19 UTC
Created attachment 25726 [details]
Possible fix for this bug, please test
Comment 5 Florian Sievert 2008-07-27 11:30:53 UTC
The patch does not fix the problem against the current trunk here. It is detecting at the beginning of the download that the program does not exist and complains about it. However, after the user clicks on "ok" konqueror does start the download. If the user cancels a very big download (iso file or something like that), he is complaing again that he does not find the program.

So basically the first part is working, but the program should leave the process immedentially without starting a download and the old check should be removed.

If this problem is fixed, it should also fix the problem described in #3. I can not exactly reproduce it, however it takes about 2 minutes, before the error message is displayed. Might be interesting to explore while it takes so long to display the message, however the problem itself should be fixed with this issue.

Thanks for your work Dave!
Comment 6 David Faure 2008-11-05 02:08:05 UTC
SVN commit 880260 by dfaure:

Check if the program typed in the open-with dialog exists, to avoid downloading a huge file for nothing (when making a typo).
Testcase: http://bugs.kde.org/show_bug.cgi?id=81190#c2
BUG: 81190


 M  +34 -14    kopenwithdialog.cpp  
 M  +2 -3      kopenwithdialog.h  


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