Bug report #14703

Identify Results -> Save Link Fails

Added by Frank MR over 8 years ago. Updated almost 8 years ago.

Status:Closed
Priority:Normal
Assignee:Alessandro Pasotti
Category:Map Tools
Affected QGIS version:master_2 Regression?:No
Operating System: Easy fix?:No
Pull Request or Patch supplied:No Resolution:fixed/implemented
Crashes QGIS or corrupts data:No Copied to github as #:22664

Description

When using a OGC:WMS and selecting the (I) button, the WMS provider provides an HTML document with content that can be downloaded. When I click the "Save Link" to download content via <a href>...</a> links displayed in this page I get network timeout errors reported in QGIS. It does not pop-up a file save dialog to save content.

I've tried the links provided using Copy Link, and pasting it into a web browser and it works fine. Also, I tried this in other GIS tools and it works fine. So I believe this to be a bug that the "Save Link" function does not work in the "Identify Results" page.

qgis_issue_14703.qgs (5.1 KB) Giovanni Manghi, 2016-10-26 04:15 PM

Associated revisions

Revision bdc2e24d
Added by Alessandro Pasotti almost 8 years ago

File downloader for identify dialog hyperlinks

fixes #14703

Include C++ and Python tests

Revision a676bde4
Added by Alessandro Pasotti almost 8 years ago

[bugfix][backport] File downloader for identify dialog hyperlinks

fixes #14703

Include C++ and Python tests

( cherry-picked from commit bdc2e24 )

Try to convince Travis to behave like a normal mechanical being
Travis won: ported all test cases to Python
and disabled C++ companion test (still useful locally and
for debugging)

For the curious: QTemporaryFile is not working as expected

( cherry-picked from 57aa7fd )

Revision 4e96912c
Added by Alessandro Pasotti almost 8 years ago

[bugfix][forwardport] File downloader for identify dialog hyperlinks

fixes #14703

Include C++ and Python tests

Travis won: ported all test cases to Python
and disabled C++ companion test (still useful locally and
for debugging)

For the curious: QTemporaryFile is not working as expected

Moved to Qt5 new style signals

Disabled C++ test and connected cancel to progress

Make string comparison on SSL errors more robust

History

#1 Updated by Giovanni Manghi over 8 years ago

  • Priority changed from High to Normal

#2 Updated by Giovanni Manghi about 8 years ago

  • Status changed from Open to Feedback

Hi,
a few questions:

is the "(I) button" the identify tool?

what is the wms server, geoserver?

what is the platform where you use QGIS?

thanks!

#3 Updated by Giovanni Manghi about 8 years ago

  • File qgis_issue_14703.qgs added
  • Status changed from Feedback to Open
  • Affected QGIS version changed from 2.14.1 to master_2

Attached a project with a layer that allows to replicate the issue.

Clicking on links in the identify window causes a Timeout Network error in QGIS. Right click on links, click "copy link" and pasting into a browser works as expected.

Other GIS packages are working as expected.

It also true that there are other WMS services with a html getfeatureinfo were the links work as expected in QGIS identify window.

#4 Updated by Alessandro Pasotti about 8 years ago

  • Assignee set to Alessandro Pasotti

#5 Updated by Alessandro Pasotti about 8 years ago

  • Status changed from Open to Feedback

The problem here is that QGIS use QWebView to show content in the identify dialog. QWebView is not a full-featured web browser and it does not implement the download of content.

To fix this issue a system must be developed to download the files and choose the location where the downloads should be stored on the client machine.

#6 Updated by Alessandro Pasotti about 8 years ago

  • Category changed from Web Services clients/WMS to Map Tools

#7 Updated by Anonymous almost 8 years ago

  • Status changed from Feedback to Closed

#8 Updated by Alessandro Pasotti almost 8 years ago

Merged into 2.18, backporting to 2.14 and forward porting to master are in progress.

#9 Updated by Alessandro Pasotti almost 8 years ago

  • Status changed from Closed to Feedback

Backported to 2.14

#10 Updated by Anonymous almost 8 years ago

  • Status changed from Feedback to Closed

#11 Updated by Alessandro Pasotti almost 8 years ago

  • Resolution set to fixed/implemented

Also available in: Atom PDF