Opened 7 years ago

Closed 7 years ago

#1245 closed defect (fixed)

v.db.connect parsing

Reported by: hamish Owned by: hamish
Priority: major Milestone: 6.4.1
Component: Vector Version: svn-releasebranch64
Keywords: v.db.connect, scripts Cc: grass-dev@…
CPU: All Platform: All

Description (last modified by hamish)

Hi,

just a placeholder bug report for the records; as reported on the mailing list.

sqlite+v.in.ogr adds a table name to the table number reported by v.db.connect -p,-g as "number/name". So if parsing for the table number we need to use the -l flag as well to suppress the name.

the following shell scripts are affected, need to check the python scripts/lib for this too.

v.colors (fixed)
d.vect.thematic
v.db.addcol
v.db.addtable
v.db.dropcol
v.db.droptable
v.db.join
v.db.renamecol
v.db.update
v.dissolve
v.out.gpsbabel
v.rast.stats
v.report

Hamish

Change History (8)

comment:1 Changed 7 years ago by hamish

Cc: grass-dev@… added
Owner: changed from grass-dev@… to hamish
Status: newassigned

comment:2 Changed 7 years ago by hamish

Description: modified (diff)

comment:3 Changed 7 years ago by hamish

python scripts to check:

lib/python/vector.py

v.db.addtable
v.db.droptable
v.db.reconnect.all

comment:4 Changed 7 years ago by hamish

all the python in trunk is ok

comment:5 Changed 7 years ago by hamish

6.5svn scripts updated in r44641.

comment:6 Changed 7 years ago by neteler

Hamish, what about the 6.4 backport?

comment:7 in reply to:  6 Changed 7 years ago by hamish

Replying to neteler:

Hamish, what about the 6.4 backport?

as it was not a simple incisive change, before backporting I thought to leave it in 6.5svn for a while in case there were any typos.

aka I haven't tested the changes to see if they still work. other than that, no reason not to backport. (not being lazy with backports; I try not to backport anything I haven't tested. so just being lazy with testing :)

Hamish

comment:8 Changed 7 years ago by hamish

Resolution: fixed
Status: assignedclosed

backported to relbr64 in r45260.

possibly broken (poorly tested) is less bad than known broken (sqlite ops in 6.4.0).

Hamish

Note: See TracTickets for help on using tickets.