Opened 3 years ago

Closed 4 weeks ago

#3558 closed defect (fixed)

wx.metadata: several issues in syntax

Reported by: sbl Owned by: grass-dev@…
Priority: normal Milestone: 7.8.3
Component: Addons Version: 7.4.0
Keywords: wx.metadata, g.gui.metadata, r.iso.info Cc:
CPU: All Platform: All

Description

wx.metadata has a couple of issues mostly due to changes in underlying libraries, I guess. The attached diffs fix some of them, but remaining issues amongst others require wxGUI programming skills, which I do not have unfortunately.

After changes applied editing metadata still throws:

Traceback (most recent call last):
  File "/home/stefan/.grass7/addons/scri
pts/g.gui.metadata", line 506, in onEdit
    ok = self.editMapMetadata()
  File "/home/NINA.NO/stefan.blumentrath/.grass7/addons/scri
pts/g.gui.metadata", line 822, in editMapMetadata
    if self.chckProfileSelection('spatial'):
  File "/home/NINA.NO/stefan.blumentrath/.grass7/addons/scri
pts/g.gui.metadata", line 790, in chckProfileSelection
    text = self.MdDataCatalogPanelLeft.GetItemText(parent)
  File "/usr/lib/python2.7/dist-
packages/wx-3.0-gtk2/wx/_controls.py", line 5255, in
GetItemText
    return _controls_.TreeCtrl_GetItemText(*args, **kwargs)
TypeError: in method 'TreeCtrl_GetItemText', expected
argument 2 of type 'wxTreeItemId const &'

r.info.iso gives the following warning message:

/usr/lib/python2.7/dist-packages/owslib/iso.py:111:
FutureWarning: the .identification and
.serviceidentification properties will merge into
.identification being a list of properties.  This is
currently implemented in .identificationinfo.  Please see
https://github.com/geopython/OWSLib/issues/38 for more
information
  FutureWarning)
/usr/lib/python2.7/dist-packages/owslib/iso.py:451:
FutureWarning: The .keywords and .keywords2 properties will
merge into the .keywords property in the future, with
.keywords becoming a list of MD_Keywords instances. This is
currently implemented in .keywords2. Please see
https://github.com/geopython/OWSLib/issues/301 for more
information
  FutureWarning)
{'status': 'succeded', 'errors': [], 'num_of_errors': '0'}

Attachments (2)

mdgrass_py.diff (2.3 KB) - added by sbl 3 years ago.
g7.4 findfile syntax, correct mdutils module call
g_gui_metadata_py.diff (1.8 KB) - added by sbl 3 years ago.
Adjust parts of the syntax to wxpython 3 requirements

Download all attachments as: .zip

Change History (5)

Changed 3 years ago by sbl

Attachment: mdgrass_py.diff added

g7.4 findfile syntax, correct mdutils module call

Changed 3 years ago by sbl

Attachment: g_gui_metadata_py.diff added

Adjust parts of the syntax to wxpython 3 requirements

comment:1 Changed 2 years ago by martinl

Remove Milestone from Addons bugreports.

comment:2 Changed 17 months ago by neteler

Milestone: 7.8.3

comment:3 Changed 4 weeks ago by sbl

Resolution: fixed
Status: newclosed

There have been various fixes to wx.metadata:

https://github.com/OSGeo/grass-addons/pulls?q=is%3Apr+wx.metadata

Closing the ticket.

Note: See TracTickets for help on using tickets.