Skip to content
This repository has been archived by the owner on Jun 4, 2020. It is now read-only.

Spatialite checkout needs to be upgraded to work perfectly on higher version of QGIS ex. QGIS 2.14 #56

Closed
Sameme opened this issue Mar 30, 2017 · 6 comments

Comments

@Sameme
Copy link

Sameme commented Mar 30, 2017

Using the spatialiet checkout of the plugin only works perfectly with QGIS 2.8, once you start using it with higher QGIS version, it starts coming up with various errors and issues.

Find error screenshots below
picture1
picture2
picture3

@vmora
Copy link
Contributor

vmora commented Mar 31, 2017

Can you tell me the version of gdal (ogr2ogr), the version of spatialite and the version of qgis you are using ?

@Sameme
Copy link
Author

Sameme commented Mar 31, 2017

I used QGIS 2.14.12, gdal/ogr version 2.1.3 and spatialite Version 4.3.0.

@gignacnic
Copy link
Contributor

I supposed this kind of test will be easier to do when the Travis-CI QA will be fully integrated in the process?

@vmora
Copy link
Contributor

vmora commented Mar 31, 2017

@gignacnic I have made some progresses on the travis front too, the problem now is that the pyspatialite module cannot be found (although it's installed) see #57 for details of travis failures.

Also the units tests can now be run from the qgis python console:

from qgis_versioning.test import test
test()

@gignacnic
Copy link
Contributor

gignacnic commented Apr 2, 2017

First, one problem wass related to checkout without selecting any features in the table in QGIS before checking out. Once you select any element in the table in QGIS before doing a checkout it works as expected with QGIS 2.18 (or other version). There were also a problem with OGC_FID field which needed to be lower instead of upper (maybe related to ogr2ogr version update). So I fix this and now it works for QGIS 2.18 that is executing a SpatiaLite checkout to fix these two problems: #60

gignacnic added a commit to gignacnic/qgis-versioning that referenced this issue Apr 2, 2017
@vmora
Copy link
Contributor

vmora commented Apr 7, 2017

@Sameme can you confirm it's fixed by #57 (the last zip I sent you) such that I can merge and close ?

@Sameme Sameme closed this as completed Apr 7, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants