Opened 5 years ago

Closed 5 years ago

#3736 closed defect (fixed)

i.fusion.hpf addon: tiny patch to fix suffix setting

Reported by: veroandreo Owned by: grass-dev@…
Priority: normal Milestone:
Component: Addons Version: unspecified
Keywords: i.fusion.hpf Cc: nik@…
CPU: Unspecified Platform: Unspecified

Description

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.

Attachments (1)

i.fusion.hpf_minor.diff (794 bytes ) - added by veroandreo 5 years ago.
small patch fixing suffix in i.fusion.hpf

Download all attachments as: .zip

Change History (3)

by veroandreo, 5 years ago

Attachment: i.fusion.hpf_minor.diff added

small patch fixing suffix in i.fusion.hpf

comment:1 by Nikos Alexandris, 5 years ago

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.

comment:2 by veroandreo, 5 years ago

Resolution: fixed
Status: newclosed

Committed and MR created, closing. Thanks Nikos

Note: See TracTickets for help on using tickets.