G_OPT_DB_COLUMNS does not allow input of multiple column names

[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.

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

[martinl]

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.

[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.

[cmbarton]

Maybe working now?

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

Michael

[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

[turek]

Please try r55891. It should work now.

Stepan

[cmbarton]

Seems to be fixed now. Thanks. Closing.

[cmbarton]

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

Michael

[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.

[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