View Issue Details

Jump to NotesJump to History
IDProjectCategoryView StatusDate SubmittedLast Update
0002252Slicer4Core: CLI infrastructurepublic2012-06-22 08:362018-03-02 11:00
Reportermmodat Assigned Tomillerjv  
PrioritynormalSeverityfeatureReproducibilityN/A
Status closedResolutionwon't fix 
Product VersionSlicer 4.1.0 RC 
Target VersionSlicer 4.7.0Fixed in Version 
Summary0002252: Optional/required argument
Description

As it is, all arguments are optional in a sense that one can always press "Apply" even if no option have been provided (e.g. input images).
A tweak could be to include a optional/required flag in the XML file and then only enable the Apply button if all required arguments are filed.

TagsNo tags attached.

Activities

nicole

nicole

2012-08-20 08:19

administrator   ~0005577

optional/required is already implemented in XML, the only change is to do a check on enabling the Apply button only when all required arguments are set.
This will require monitoring all widgets for changes and updating the status of the Apply button and need to be as lightweight as possible (only on the currently visible GUI).
Related to a feature request wanting the Apply button to always be visible without scrolling.
millerjv

millerjv

2012-08-20 08:50

developer   ~0005581

Parameters with "flags" are considered optional by design and do not need to be specified. Parameters that are "indexed" are considered to be required. This follows the traditional command line style of parameters for flags were truly options.

In Slicer3, I believe you the "Apply" button would return with an error indicating that all the required parameters had been specified. I haven't looked to see if this carried over to Slicer4.
lassoan

lassoan

2017-09-27 12:27

developer   ~0015180

Since error messages are now displayed in the CLI module GUI, the user can see detailed explanation of what went wrong instead of just seeing a disabled button.

Issue History

Date Modified Username Field Change
2012-06-22 08:36 mmodat New Issue
2012-08-20 08:19 nicole Note Added: 0005577
2012-08-20 08:19 nicole Assigned To => millerjv
2012-08-20 08:19 nicole Status new => assigned
2012-08-20 08:19 nicole Target Version => Slicer 4.3.0
2012-08-20 08:19 nicole Severity tweak => feature
2012-08-20 08:50 millerjv Note Added: 0005581
2013-08-30 14:01 jcfr Target Version Slicer 4.3.0 => Slicer 4.4.0
2014-05-13 20:47 jcfr Target Version Slicer 4.4.0 => Slicer 4.5.0-1
2015-11-02 11:27 jcfr Target Version Slicer 4.5.0-1 => Slicer 4.6.0
2016-10-12 15:02 jcfr Target Version Slicer 4.6.0 => Slicer 4.7.0
2017-09-27 12:27 lassoan Status assigned => resolved
2017-09-27 12:27 lassoan Resolution open => won't fix
2017-09-27 12:27 lassoan Note Added: 0015180
2018-03-02 11:00 jcfr Status resolved => closed