Opened 6 years ago

Closed 6 years ago

Last modified 6 years ago

#3536 closed defect (fixed)

python library db_table_in_vector function

Reported by: lucadelu Owned by: grass-dev@…
Priority: normal Milestone: 7.4.1
Component: Python Version: unspecified
Keywords: db_table_in_vector, db.droptable, pygrass table Cc:
CPU: All Platform: All

Description

The function db_table_in_vector in db.py is looking for the vector of all the mapset. I think this is a problem specially when it is used in db.droptable.

Actually db_table_in_vector is used only in db.droptable and pygrass vector/table.py.

Attached I propose to add as default to look in actual mapset.

Attachments (1)

db_table_in_vector.diff​ (939 bytes ) - added by lucadelu 6 years ago.
proposed patch for db_table_in_vector

Download all attachments as: .zip

Change History (5)

by lucadelu, 6 years ago

Attachment: db_table_in_vector.diff​ added

proposed patch for db_table_in_vector

comment:1 by lucadelu, 6 years ago

Can I submit this patch?

in reply to:  1 comment:2 by mmetz, 6 years ago

Replying to lucadelu:

Can I submit this patch?

When changing it, could you replace 'vect' with 'vector' in list_strings()? 

vects = list_strings('vector', mapset=mapset)

See manual of g.list

Thanks!

comment:3 by lucadelu, 6 years ago

Resolution: → fixed
Status: new → closed

In 72740:

db.py: added option in db_table_in_vector to check by default only vectors in current mapset. fixed #3536

comment:4 by neteler, 6 years ago

Is this a backport candidate?

Note: See TracTickets for help on using tickets.