[GRASS-dev] [GRASS GIS] #3736: i.fusion.hpf addon: tiny patch to fix suffix setting

#3736: i.fusion.hpf addon: tiny patch to fix suffix setting
--------------------------+-------------------------
Reporter: veroandreo | Owner: grass-dev@…
     Type: defect | Status: new
Priority: normal | Milestone:
Component: Addons | Version: unspecified
Keywords: i.fusion.hpf | CPU: Unspecified
Platform: Unspecified |
--------------------------+-------------------------
When setting the suffix parameter in G7A:i.fusion.hpf one still gets a dot
(.) just before the desired suffix.

IMHO, a suffix option should prevail and no symbols should be inserted in
between mapname and suffix. Otherwise one only realizes the command
behaviour when the next command in the script does not find the output map
of i.fusion.hpf.

So, I removed the dot and set the default suffix to .hpf (see super tiny
patch attached). In this way the suffix that the user sets is fully
respected. Can I commit this small modification? I do not want to break
others' scripts.

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

#3736: i.fusion.hpf addon: tiny patch to fix suffix setting
--------------------------+--------------------------
  Reporter: veroandreo | Owner: grass-dev@…
      Type: defect | Status: new
  Priority: normal | Milestone:
Component: Addons | Version: unspecified
Resolution: | Keywords: i.fusion.hpf
       CPU: Unspecified | Platform: Unspecified
--------------------------+--------------------------
Changes (by veroandreo):

* Attachment "i.fusion.hpf_minor.diff" added.

small patch fixing suffix in i.fusion.hpf

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

#3736: i.fusion.hpf addon: tiny patch to fix suffix setting
--------------------------+--------------------------
  Reporter: veroandreo | Owner: grass-dev@…
      Type: defect | Status: new
  Priority: normal | Milestone:
Component: Addons | Version: unspecified
Resolution: | Keywords: i.fusion.hpf
       CPU: Unspecified | Platform: Unspecified
--------------------------+--------------------------

Comment (by Nikos Alexandris):

Please, go ahead. Would you also create an MR for
https://gitlab.com/NikosAlexandris/i.fusion.hpf? Don't bother if you have
no time. Thank you Vero.

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

#3736: i.fusion.hpf addon: tiny patch to fix suffix setting
--------------------------+--------------------------
  Reporter: veroandreo | Owner: grass-dev@…
      Type: defect | Status: closed
  Priority: normal | Milestone:
Component: Addons | Version: unspecified
Resolution: fixed | Keywords: i.fusion.hpf
       CPU: Unspecified | Platform: Unspecified
--------------------------+--------------------------
Changes (by veroandreo):

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

Comment:

Committed and MR created, closing. Thanks Nikos

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