Opened 4 years ago

Closed 4 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 4 years ago by hamish

  • Cc grass-dev@… added
  • Owner changed from grass-dev@… to hamish
  • Status changed from new to assigned

comment:2 Changed 4 years ago by hamish

  • Description modified (diff)

comment:3 Changed 4 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 4 years ago by hamish

all the python in trunk is ok

comment:5 Changed 4 years ago by hamish

6.5svn scripts updated in r44641.

comment:6 follow-up: Changed 4 years ago by neteler

Hamish, what about the 6.4 backport?

comment:7 in reply to: ↑ 6 Changed 4 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 4 years ago by hamish

  • Resolution set to fixed
  • Status changed from assigned to closed

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.