[Bug] GUI generated by Simple Python Editor reopens instead of executing the command if default settings are used

[klimesm]

Describe the bug

When using the Simple Python Editor to create a custom GRASS module with a GUI interface, there's an issue when running the module without changing any of the default values in the GUI. Instead of executing the command, the GUI simply reopens, and the command is not run. If any parameter is changed, the module runs correctly.

To reproduce

1. Create a module using Simple Python Editor.
2. Leave all parameters in their default state.
3. Run the module via GUI.
4. the GUI reopens instead of executing the command.

Expected behavior

The module should execute with the default parameters, without requiring the user to change any values.

[petrasovaa]

Could you provide an example when this happens?

[klimesm]

Yes, for clarification, it happens only if none of the parameters are required and left unchanged at the same time. If some of the parameters are required, it works normally.

#!/usr/bin/env python3

#

##############################################################################

#

# MODULE:       model

#

# AUTHOR(S):    martin

#

# PURPOSE:      Script generated by wxGUI Graphical Modeler.

#

# DATE:         Wed Apr  3 22:01:42 2024

#

##############################################################################



# %module

# % description: Script generated by wxGUI Graphical Modeler.

# %end

# %option

# % key: resolution

# % description: Target resolution

# % required: no

# % type: integer

# % answer: 5

# %end



import sys



from grass.script import parser



def cleanup():

    pass



def collect_files(path):

    pass



def main(options, flags):

    pass



def process_product(name):

    pass



if __name__ == "__main__":

    options, flags = parser()

    sys.exit(main(options, flags))

[pesekon2]

The issue is much worse. The same happens if you run the script provided by @klimesm from gconsole. And with the --ui option on, it happens also for required parameters.

I should have a fix though. I'll just test it a bit more and open a PR.