colinsheppard/insalmo-instream-gui — Issue #10: write*Report special treatment?
Author: colinsheppard
Created: 2011-12-17 00:05:13+00:00
State: open

So after thinking it through, I'm questioning whether it makes sense to treat all of the write*Report parameters differently than every other parameter. Namely, in order to make them optional from the GUI perspective, we need to rewrite code in the following places:

-during read of Model.Setup, add conditional logic to not complain if these parameters are missing
-during write to Model.Setup, do something similar
-extra interface buttons / wizard will be needed to give the user the opportunity to add/remove these parameters from the Model.Setup pane
-extra code will need to be added in several places to treat these parameters differently when checking the whole project for errors/warnings

The alternative to these changes would be to simply not touch the GUI code and have the following occur if the user opens a project that doesn't have these parameters in Model.Setup:

-a window appears saying ~ "these parameters are missing, they will be automatically added to Model.Setup with blank values" (this already occurs, no extra work needed)
-the user will need to add "Yes" or "No" to those values (I could instead use a default value instead of blank so that these would all default to "NO")