Bug 24434

Summary:	konqueroe save image option doesn't save some images already loaded and displayed.
Product:	[Unmaintained] kio	Reporter:	taupter
Component:	http	Assignee:	Unassigned bugs <unassigned-bugs-null>
Status:	CLOSED FIXED
Severity:	normal	CC:	giopirtskhalaishvili
Priority:	NOR
Version First Reported In:	unspecified
Target Milestone:	---
Platform:	Debian testing
OS:	Linux
Latest Commit:		Version Fixed In:
Sentry Crash Report:

Description taupter 2001-04-20 09:39:00 UTC

(*** This bug was imported into bugs.kde.org ***)

Package:           konqueror
Version:           KDE 2.1.1 
Severity:          normal
Installed from:    Debian Packages
Compiler:          gcc 2.95.3
OS:                Linux
OS/Compiler notes: Standard stable Debian with X4.02 and KDE apt lines.

Konqueror fails to save an image from a given page (http://www.visi.com/~tburt/wallpaper/speedofthought.html).
The page loads fine and is rendered as we should expect but when I right-click on the image and select the "Save image as" option it tries to download the image and fails showing a popup window with the message "The file or directory http://www.visi.com/~tburt/wallpaper/speedofthought.jpg doesn't exist". The strange is the fact that the image is already loaded to my computer as I can see it on the Konqueror window.
In my opinion the image should be saved from the already cached image instead of trying to download it.

(Submitted via bugs.kde.org)

Comment 1 Waldo Bastian 2001-09-30 23:26:58 UTC

On Sunday 30 September 2001 03:16 pm Waldo Bastian wrote:
> BR31341 (http://bugs.kde.org/db/31/31341.html) is an interesting bugreport.
> The reason the drag&drop doesn't work seems to be that the actual download
> is handled by kdesktop who gets the url when it accepts the drop. However
> in order to download the image successfully the referrer should be set
> accordingly (I assume) and kdesktop has no way of doing this since it does
> not have this information.

BR24434 might very well similar to this. The difference is that the "Save 
Image As" code is handled by khtml itself which makes it easier to solve:

khtml_ext.cpp:456

      if(!saved)
      {
        /*KIO::Job *job = */ KIO::copy( url destURL );
        // TODO connect job result to display errors
      }

In this case we should be able to pass MetaData to the copy-job.

Cheers
Waldo
-- 
bastian@kde.org | SuSE Labs KDE Developer | bastian@suse.com

Comment 2 johnj 2002-06-04 18:33:48 UTC

<HTML>
<HEAD>
<TITLE>wrmailer</TITLE>
<META HTTP-EQUIV=3D"Content-Type" CONTENT=3D"text/html; charset=3Diso-8859-=
1">
</HEAD>
<BODY BGCOLOR=3D#FFFFFF LEFTMARGIN=3D0 TOPMARGIN=3D0 MARGINWIDTH=3D0 MARGIN=
HEIGHT=3D0>
<br>
<table cellpadding=3D0 cellspacing=3D0 border=3D0 height=3D18 width=3D600 a=
lign=3D"center">
  <tr>
    <td height=3D"35" width=3D"600" colspan=3D"2"> <font color=3D"#000000" =
size=3D"1" face=3D"Arial">This
      message is an advertisement. We will continue to bring you valuable p=
ermission
      based messages on the products and services that interest you most un=
less
      you wish to <a href=3D"http://www.cramz.net/help.html">decline</a>. W=
e process
      all requests immediately. Brought to you by world Reach Corporation. =
Copyright
      2000 2001 2002 all rights reserved.</font><font size=3D"1"> </font>=
</td>
  </tr>
</table>
<br>
<TABLE WIDTH=3D560 BORDER=3D0 CELLPADDING=3D0 CELLSPACING=3D0 align=3D"cent=
er">
  <TR>
<TD>
<IMG src=3D"http://www.cramz.net/images/wrmailer_01.gif" WIDTH=3D325 HEI=
GHT=3D100 ALT=3D""></TD>
<TD>
<IMG src=3D"http://www.cramz.net/images/wrmailer_02.gif" WIDTH=3D235 HEI=
GHT=3D100 ALT=3D""></TD>
</TR>
<TR>
<TD>
<IMG src=3D"http://www.cramz.net/images/wrmailer_03.gif" WIDTH=3D325 HEI=
GHT=3D142 ALT=3D""></TD>
<TD>
<IMG src=3D"http://www.cramz.net/images/wrmailer_04.gif" WIDTH=3D235 HEI=
GHT=3D142 ALT=3D""></TD>
</TR>
<TR>
<TD>
<IMG src=3D"http://www.cramz.net/images/wrmailer_05.gif" WIDTH=3D325 HEI=
GHT=3D113 ALT=3D""></TD>

    <TD> <a href=3D"http://www.cramz.net/form.html"><IMG src=3D"http://wwwc=
ramz.net/images/wrmailer_06.gif" WIDTH=3D235 HEIGHT=3D113 ALT=3D"" border=
=3D"0"></a></TD>
</TR>
<TR>
<TD>
<IMG src=3D"http://www.cramz.net/images/wrmailer_07.gif" WIDTH=3D325 HEI=
GHT=3D45 ALT=3D""></TD>
<TD>
<IMG src=3D"http://www.cramz.net/images/wrmailer_08.gif" WIDTH=3D235 HEI=
GHT=3D45 ALT=3D""></TD>
</TR>
</TABLE>
<div align=3D"center"><br>
  <img src=3D"http://www.cramz.net/mailer.gif" width=3D"1" height=3D"1"><br>
  <br>
</div>
<table cellpadding=3D0 cellspacing=3D0 border=3D0 height=3D18 width=3D599 a=
lign=3D"center">
  <tr>
    <td height=3D"20" width=3D"599" colspan=3D"2">
      <div align=3D"center"><a href=3D"http://www.cramz.net/form.html"><img=
 src=3D"http://www.cramz.net/footer.gif" width=3D"576" height=3D"89" border=
=3D"0"></a>
      </div>
    </td>
  </tr>
  <tr>
    <td height=3D"20" width=3D"599" colspan=3D"2"><font color=3D"#000000" s=
ize=3D"1" face=3D"Arial">This
      message is an advertisement. We will continue to bring you valuable p=
ermission
      based messages on the products and services that interest you most un=
less
      you wish to <a href=3D"http://www.cramz.net/help.html">decline</a>. W=
e process
      all requests immediately. Brought to you by world Reach Corporation. =
Copyright
      2000 2001 2002 all rights reserved.</font><font size=3D"1"> </font>=
</td>
  </tr>
</table>
<p>&nbsp;</p></BODY>
</HTML>