After launching the app from the command line, if a user selects no radar(s) and immediately clicks the Download/Execute button, sa.radar_list will be empty and thus the for loop is never entered. The results variable is not defined and the app will throw an error.
The fix should either be to (1) make it impossible for the user to click the Download/Execute button without selecting a radar, or (2) to check that len(sa.radar_list) > 0 above res = call_function(query_radar_files). (2) is probably a bit easier to implement.
tjturnage
commented
4 months ago
After launching the app from the command line, if a user selects no radar(s) and immediately clicks the Download/Execute button,
sa.radar_listwill be empty and thus the for loop is never entered. Theresultsvariable is not defined and the app will throw an error.The fix should either be to (1) make it impossible for the user to click the Download/Execute button without selecting a radar, or (2) to check that
len(sa.radar_list) > 0aboveres = call_function(query_radar_files). (2) is probably a bit easier to implement.