Opened 7 years ago

Last modified 4 years ago

#1723 reopened defect

G_OPT_DB_COLUMNS does not allow input of multiple column names

Reported by: cmbarton Owned by: grass-dev@…
Priority: normal Milestone: 6.4.6
Component: wxGUI Version: unspecified
Keywords: multiple columns Cc: martinl
CPU: Unspecified Platform: Unspecified

Description

G_OPT_DB_COLUMNS (gis.h) is supposed to indicate multiple column names can be entered into a field. But only single column names are actually accepted. Check out v.out.ascii for an example. I'm not sure WHERE the actual problem lies--in gis.h, in g.parser, in forms.py (no reference to G_OPT_COLUMNS in the latter however), or somewhere else.

Wherever this is defined for the interface, G_OPT_M_DATATYPE could probably be fixed too. It is supposed to bring up a list of GRASS datatypes, but does nothing AFAICT.

Michael

Change History (10)

comment:1 in reply to:  description ; Changed 7 years ago by martinl

Cc: martinl added
Component: DefaultwxGUI
Keywords: multiple columns added

Replying to cmbarton:

G_OPT_DB_COLUMNS (gis.h) is supposed to indicate multiple column names can be entered into a field. But only single column names are actually accepted. Check out v.out.ascii for an example. I'm not sure WHERE the actual problem lies--in gis.h, in g.parser, in forms.py (no reference to G_OPT_COLUMNS in the latter however), or somewhere else.

fixed in 'turek' in r55651.

Wherever this is defined for the interface, G_OPT_M_DATATYPE could probably be fixed too. It is supposed to bring up a list of GRASS datatypes, but does nothing AFAICT.

This should be probably reported as separate ticket.

comment:2 in reply to:  1 Changed 7 years ago by martinl

Replying to martinl:

Wherever this is defined for the interface, G_OPT_M_DATATYPE could probably be fixed too. It is supposed to bring up a list of GRASS datatypes, but does nothing AFAICT.

This should be probably reported as separate ticket.

What is the problem with this element? Check for eg. g.list, check boxes seem to be fine.

comment:3 Changed 7 years ago by cmbarton

Maybe working now?

I'll be back to where I can recompile and test in a few days. Thanks for the update.

Michael

comment:4 Changed 7 years ago by cmbarton

I recompiled yesterday r55705. This still does not work. To test, try to output firestations from the nc_08 demo dataset to an ASCII file using v.out.ascii. The column tool shows an empty pulldown. The dropbox appears but it is blank.

Michael

comment:5 Changed 7 years ago by turek

Please try r55891. It should work now.

Stepan

comment:6 Changed 7 years ago by cmbarton

Resolution: fixed
Status: newclosed

Seems to be fixed now. Thanks. Closing.

comment:7 Changed 7 years ago by cmbarton

Resolution: fixed
Status: closedreopened

This fix needs to be ported to 6.4.3RC3. Still broken there.

Michael

comment:8 in reply to:  7 Changed 7 years ago by martinl

Replying to cmbarton:

This fix needs to be ported to 6.4.3RC3. Still broken there.

unfortunately the patch is not easy to apply in relbr64. The column widget is editable, so the user can add multiple columns manually.

comment:9 Changed 7 years ago by cmbarton

Maybe just put it back to a text box in relbr64? I guess either way is OK as long as the user can put in fields and it reads them all.

Michael

comment:10 Changed 4 years ago by neteler

Milestone: 6.4.36.4.6
Note: See TracTickets for help on using tickets.