search

planetfederal / qgis-geogiglight-plugin

GNU General Public License v2.0
19 stars 9 forks source link

Python error when using the "export changes introduced by this version as new layer" functionality #93

Closed gioman closed 7 years ago

gioman commented 7 years ago

This is using the latest plugin code revision, GG server RC3 and BL Desktop 1.0.0

Traceback (most recent call last):
  File "C:/Users/boundless/.qgis2/python/plugins\geogig\gui\dialogs\historyviewer.py", line 122, in 
    exportDiffAction.triggered.connect(lambda: self.exportDiffs(item.commit))
  File "C:/Users/boundless/.qgis2/python/plugins\geogig\gui\dialogs\historyviewer.py", line 228, in exportDiffs
    addDiffLayer(self.repo, tree, commit)
  File "C:/Users/boundless/.qgis2/python/plugins\geogig\tools\layers.py", line 217, in addDiffLayer
    beforeGpkgfid = gpkgfidFromGeogigfid(beforeCursor, layername, geogigfid)
  File "C:/Users/boundless/.qgis2/python/plugins\geogig\tools\layers.py", line 152, in gpkgfidFromGeogigfid
    gpkgfid = int(cursor.fetchone()[0])
TypeError: 'NoneType' object has no attribute '__getitem__'
gioman commented 7 years ago

In tests this functionality is referred as "export diff as layer".

gioman commented 7 years ago

@volaya with the latest changes in the server and/or plugin this seems to have been fixed.

volaya commented 7 years ago

Awesome. I told the GG team, but got no reply on this. Good to see that they fixed it

On Tue, Dec 20, 2016 at 2:38 PM, Giovanni Manghi notifications@github.com wrote:

@volaya https://github.com/volaya with the latest changes in the server and/or plugin this seems to have been fixed.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/boundlessgeo/qgis-geogiglight-plugin/issues/93#issuecomment-268244933, or mute the thread https://github.com/notifications/unsubscribe-auth/ACARk--4I-MhXglguKLwr3TdNZYkFiXfks5rJ9pEgaJpZM4K2UW_ .

-- Victor Olaya Software Engineer | Boundless http://boundlessgeo.com/ volaya@boundlessgeo.com @boundlessgeo http://twitter.com/boundlessgeo/