Ticket #1245 (closed defect: fixed)

Opened 4 years ago

Last modified 4 years ago

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@…
Platform: All CPU: All

Description (last modified by hamish) (diff)

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

  Changed 4 years ago by hamish

  • cc grass-dev@… added
  • owner changed from grass-dev@… to hamish
  • status changed from new to assigned

  Changed 4 years ago by hamish

  • description modified (diff)

  Changed 4 years ago by hamish

python scripts to check:

lib/python/vector.py

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

  Changed 4 years ago by hamish

all the python in trunk is ok

  Changed 4 years ago by hamish

6.5svn scripts updated in r44641.

follow-up: ↓ 7   Changed 4 years ago by neteler

Hamish, what about the 6.4 backport?

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

  Changed 4 years ago by hamish

  • status changed from assigned to closed
  • resolution set to fixed

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.