GeoNode / QGISGeoNodePlugin

A QGIS plugin that provides integration with GeoNode
https://geonode.org/QGISGeoNodePlugin/
GNU General Public License v3.0
12 stars 16 forks source link

Loading of GeoNode ressources not possible #268

Closed gannebamm closed 1 month ago

gannebamm commented 8 months ago

Expected Behavior:

If I connect to a stable GeoNode version with the QGIS plugin, I will get a list of resources and options to load them (wms/wfs). By clicking wfs, the layer is loaded as a QGIS layer.

Actual Behaviour:

The list gets created, but clicking any of the options (wms/wfs) will result in an error: grafik

Additional Information:

The stacktrace lists an error with _parse_dataset_detail function and raw_dataset.get("metadata_author", {}).get("username", "")

2024-03-06T12:54:02     WARNING    Traceback (most recent call last):
              File "C:\Users/-----/AppData/Roaming/QGIS/QGIS3\profiles\default/python/plugins\qgis_geonode\apiclient\geonode_v3.py", line 159, in handle_dataset_detail
              dataset = self._parse_dataset_detail(
              File "C:\Users/-----/AppData/Roaming/QGIS/QGIS3\profiles\default/python/plugins\qgis_geonode\apiclient\geonode_v3.py", line 405, in _parse_dataset_detail
              metadata_author=raw_dataset.get("metadata_author", {}).get("username", ""),
             AttributeError: 'list' object has no attribute 'get'
cedricroy commented 5 months ago

Hi Florian, Have you solved this issue? I have the same issue.

gannebamm commented 5 months ago

Not yet. We are waiting for funding.

gannebamm commented 4 months ago

Update: We will start working on this soon

cedricroy commented 3 months ago

great ! thanks for the information

vanecat commented 2 months ago

yes please! would love a fix!

ahmadzfaiz commented 2 months ago

I added compatibility for GeoNode version 4.2.x here. You can use this version while waiting for the official release.

ridoo commented 1 month ago

@giohappy who responsible for this being merged?

Gpetrak commented 1 month ago

@ridoo the PR was merged.