Opened 12 years ago
Last modified 8 years ago
#1723 reopened defect
G_OPT_DB_COLUMNS does not allow input of multiple column names
Reported by: | cmbarton | Owned by: | |
---|---|---|---|
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)
follow-up: 2 comment:1 by , 12 years ago
Cc: | added |
---|---|
Component: | Default → wxGUI |
Keywords: | multiple columns added |
comment:2 by , 12 years ago
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 by , 12 years ago
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 by , 12 years ago
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:6 by , 12 years ago
Resolution: | → fixed |
---|---|
Status: | new → closed |
Seems to be fixed now. Thanks. Closing.
follow-up: 8 comment:7 by , 11 years ago
Resolution: | fixed |
---|---|
Status: | closed → reopened |
This fix needs to be ported to 6.4.3RC3. Still broken there.
Michael
comment:8 by , 11 years ago
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 by , 11 years ago
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 by , 8 years ago
Milestone: | 6.4.3 → 6.4.6 |
---|
Replying to cmbarton:
fixed in 'turek' in r55651.
This should be probably reported as separate ticket.