[GRASS-dev] Re: [GRASS GIS] #635: v.kernel and r.surf.fractal overwriting

[GRASS GIS]

#635: v.kernel and r.surf.fractal overwriting
---------------------+------------------------------------------------------
  Reporter:  pcav    |       Owner:  grass-dev at lists.osgeo.org
      Type:  defect  |      Status:  new                      
  Priority:  normal  |   Milestone:  6.4.0                    
 Component:  Vector  |     Version:  svn-develbranch6         
Resolution:          |    Keywords:  v.kernel                 
  Platform:  All     |         Cpu:  All                      
---------------------+------------------------------------------------------
Changes (by hamish):

  * keywords:  => v.kernel
  * platform:  Unspecified => All
  * version:  unspecified => svn-develbranch6
  * component:  default => Vector
  * cpu:  Unspecified => All

Comment:

 Replying to [ticket:635 pcav]:
 > Several GRASS modules in QGIS (I found v.kernel and
 > r.surf.fractal) don't check if a layer exist before
 > overwriting it.

 the problem is that their option description miss the gisprompt field
 which says if it will be and old or new map. For new maps the parser does
 the overwrite check.

 AFAICT Martin fixed this for r.surf.fractal in December by changing the
 custom option definition to using a standard defined output one, but for
 v.kernel it is still missing.

 the problem with v.kernel is that the output option is used somehow for
 both raster and vector maps. And since the gisprompt definition can only
 be set to one or the other it is not set at all. In this case the module
 should include its own 'already exists' check and check for --overwrite to
 decide if that will be a fatal error or just a warning.


 Hamish

-- 
Ticket URL: <https://trac.osgeo.org/grass/ticket/635#comment:1>
GRASS GIS <http://grass.osgeo.org>