[ 
https://issues.apache.org/jira/browse/CLOUDSTACK-1435?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13615159#comment-13615159
 ] 

Pranav Saxena commented on CLOUDSTACK-1435:
-------------------------------------------

Mike , 

These kind of  validation checks on the data are done at the API layer and not 
at the UI layer . So when you entered the erroneous data and clicked OK , then 
the API calls were triggered  , the logic at the API layer realized that that 
something was wrong with the cluster data and hence threw an exception which 
was rendered by the UI due to which you happened to see that message "Failed to 
add storage pool" . I agree , this message could be improved in the backend API 
code . But what you are suggesting here is not possible with the current UI 
implementation . The kind of checks which UI can perform deal with the format 
of the data or it's not an empty field or something of this sort . 

I agree that this forces the user to go back and re-enter data but at this 
stage the user has to bear this pain .

Marking it as won't fix !

Thanks for raising the issue though .
                
> Error on dialog leads to having to re-enter data
> ------------------------------------------------
>
>                 Key: CLOUDSTACK-1435
>                 URL: https://issues.apache.org/jira/browse/CLOUDSTACK-1435
>             Project: CloudStack
>          Issue Type: Bug
>      Security Level: Public(Anyone can view this level - this is the 
> default.) 
>          Components: UI
>    Affects Versions: 4.0.0, 4.0.1, 4.0.2, 4.1.0, 4.2.0
>         Environment: Chrome on Mac OS X 10.8.2
>            Reporter: Mike Tutkowski
>            Assignee: Pranav Saxena
>            Priority: Minor
>             Fix For: 4.2.0
>
>
> I went into the "Add Primary Storage" dialog.
> I left the default selection for Zone, Pod, and Cluster (although it was my 
> mistake to not change the Cluster).
> I entered a name, protocol, etc.
> I clicked the OK button.
> The "Add Primary Storage" dialog was closed and an error message was 
> displayed.
> Some problems here:
> 1) The error text was fairly useless ("Failed to add storage pool").  There 
> was no indication of what I entered incorrectly.
> 2) I don't think we intend on using the term "storage pool" in the GUI.  We 
> probably should have said "primary storage."
> 3) To fix the issue, I had to re-open the "Add Primary Storage" dialog and 
> fill in all of the info again (correcting the Cluster).  It would have been 
> way nicer if this error message would have appeared on top of the "Add 
> Primary Storage" dialog.
> I believe this kind of a problem is prevalent in the GUI...not just a problem 
> with this one scenario.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

Reply via email to