[GRASS-dev] [GRASS GIS] #1723: G_OPT_DB_COLUMNS does not allow input of multiple column names

#1723: G_OPT_DB_COLUMNS does not allow input of multiple column names
-------------------------+--------------------------------------------------
Reporter: cmbarton | Owner: grass-dev@…
     Type: defect | Status: new
Priority: normal | Milestone: 6.4.3
Component: Default | Version: unspecified
Keywords: | Platform: Unspecified
      Cpu: Unspecified |
-------------------------+--------------------------------------------------
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

--
Ticket URL: <http://trac.osgeo.org/grass/ticket/1723&gt;
GRASS GIS <http://grass.osgeo.org>

#1723: G_OPT_DB_COLUMNS does not allow input of multiple column names
------------------------------+---------------------------------------------
Reporter: cmbarton | Owner: grass-dev@…
     Type: defect | Status: new
Priority: normal | Milestone: 6.4.3
Component: wxGUI | Version: unspecified
Keywords: multiple columns | Platform: Unspecified
      Cpu: Unspecified |
------------------------------+---------------------------------------------
Changes (by martinl):

* cc: martinl (added)
  * keywords: => multiple columns
  * component: Default => wxGUI

Comment:

Replying to [ticket:1723 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.

--
Ticket URL: <http://trac.osgeo.org/grass/ticket/1723#comment:1&gt;
GRASS GIS <http://grass.osgeo.org>

#1723: G_OPT_DB_COLUMNS does not allow input of multiple column names
------------------------------+---------------------------------------------
Reporter: cmbarton | Owner: grass-dev@…
     Type: defect | Status: new
Priority: normal | Milestone: 6.4.3
Component: wxGUI | Version: unspecified
Keywords: multiple columns | Platform: Unspecified
      Cpu: Unspecified |
------------------------------+---------------------------------------------

Comment(by martinl):

Replying to [comment:1 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.

--
Ticket URL: <https://trac.osgeo.org/grass/ticket/1723#comment:2&gt;
GRASS GIS <http://grass.osgeo.org>

#1723: G_OPT_DB_COLUMNS does not allow input of multiple column names
------------------------------+---------------------------------------------
Reporter: cmbarton | Owner: grass-dev@…
     Type: defect | Status: new
Priority: normal | Milestone: 6.4.3
Component: wxGUI | Version: unspecified
Keywords: multiple columns | Platform: Unspecified
      Cpu: Unspecified |
------------------------------+---------------------------------------------

Comment(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

--
Ticket URL: <https://trac.osgeo.org/grass/ticket/1723#comment:3&gt;
GRASS GIS <http://grass.osgeo.org>

#1723: G_OPT_DB_COLUMNS does not allow input of multiple column names
------------------------------+---------------------------------------------
Reporter: cmbarton | Owner: grass-dev@…
     Type: defect | Status: new
Priority: normal | Milestone: 6.4.3
Component: wxGUI | Version: unspecified
Keywords: multiple columns | Platform: Unspecified
      Cpu: Unspecified |
------------------------------+---------------------------------------------

Comment(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

--
Ticket URL: <https://trac.osgeo.org/grass/ticket/1723#comment:4&gt;
GRASS GIS <http://grass.osgeo.org>

#1723: G_OPT_DB_COLUMNS does not allow input of multiple column names
------------------------------+---------------------------------------------
Reporter: cmbarton | Owner: grass-dev@…
     Type: defect | Status: new
Priority: normal | Milestone: 6.4.3
Component: wxGUI | Version: unspecified
Keywords: multiple columns | Platform: Unspecified
      Cpu: Unspecified |
------------------------------+---------------------------------------------

Comment(by turek):

Please try r55891.
It should work now.

Stepan

--
Ticket URL: <https://trac.osgeo.org/grass/ticket/1723#comment:5&gt;
GRASS GIS <http://grass.osgeo.org>

#1723: G_OPT_DB_COLUMNS does not allow input of multiple column names
--------------------------+-------------------------------------------------
  Reporter: cmbarton | Owner: grass-dev@…
      Type: defect | Status: closed
  Priority: normal | Milestone: 6.4.3
Component: wxGUI | Version: unspecified
Resolution: fixed | Keywords: multiple columns
  Platform: Unspecified | Cpu: Unspecified
--------------------------+-------------------------------------------------
Changes (by cmbarton):

  * status: new => closed
  * resolution: => fixed

Comment:

Seems to be fixed now. Thanks. Closing.

--
Ticket URL: <https://trac.osgeo.org/grass/ticket/1723#comment:6&gt;
GRASS GIS <http://grass.osgeo.org>

#1723: G_OPT_DB_COLUMNS does not allow input of multiple column names
--------------------------+-------------------------------------------------
  Reporter: cmbarton | Owner: grass-dev@…
      Type: defect | Status: reopened
  Priority: normal | Milestone: 6.4.3
Component: wxGUI | Version: unspecified
Resolution: | Keywords: multiple columns
  Platform: Unspecified | Cpu: Unspecified
--------------------------+-------------------------------------------------
Changes (by cmbarton):

  * status: closed => reopened
  * resolution: fixed =>

Comment:

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

Michael

--
Ticket URL: <http://trac.osgeo.org/grass/ticket/1723#comment:7&gt;
GRASS GIS <http://grass.osgeo.org>

#1723: G_OPT_DB_COLUMNS does not allow input of multiple column names
--------------------------+-------------------------------------------------
  Reporter: cmbarton | Owner: grass-dev@…
      Type: defect | Status: reopened
  Priority: normal | Milestone: 6.4.3
Component: wxGUI | Version: unspecified
Resolution: | Keywords: multiple columns
  Platform: Unspecified | Cpu: Unspecified
--------------------------+-------------------------------------------------

Comment(by martinl):

Replying to [comment:7 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.

--
Ticket URL: <http://trac.osgeo.org/grass/ticket/1723#comment:8&gt;
GRASS GIS <http://grass.osgeo.org>

#1723: G_OPT_DB_COLUMNS does not allow input of multiple column names
--------------------------+-------------------------------------------------
  Reporter: cmbarton | Owner: grass-dev@…
      Type: defect | Status: reopened
  Priority: normal | Milestone: 6.4.3
Component: wxGUI | Version: unspecified
Resolution: | Keywords: multiple columns
  Platform: Unspecified | Cpu: Unspecified
--------------------------+-------------------------------------------------

Comment(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

--
Ticket URL: <http://trac.osgeo.org/grass/ticket/1723#comment:9&gt;
GRASS GIS <http://grass.osgeo.org>