Bug 1085380

Summary: Dialog is not highlighted if VM cannot be created before clicking to "Show Advanced Options"
Product: Red Hat Enterprise Virtualization Manager Reporter: Petr Spacek <pspacek>
Component: ovirt-engine-userportalAssignee: Tomas Jelinek <tjelinek>
Status: CLOSED ERRATA QA Contact: Lukas Svaty <lsvaty>
Severity: medium Docs Contact:
Priority: unspecified    
Version: 3.3.0CC: ecohen, gklein, iheim, juwu, mavital, michal.skrivanek, ofrenkel, rbalakri, Rhev-m-bugs, tgeft, yeylon
Target Milestone: ---   
Target Release: 3.5.0   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard: virt
Fixed In Version: vt2.2 Doc Type: Bug Fix
Doc Text:
Previously, when trying to save a virtual machine editing window in the User Portal with validation errors, users cannot see the error message and would try to save it with a failed attempt. The error message was only available if the user clicked on 'Show Advanced Options'. With this update, users can now see the advanced options and the error message when attempting to save with validation errors.
Story Points: ---
Clone Of: Environment:
Last Closed: 2015-02-11 18:00:31 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On:    
Bug Blocks: 1142923, 1156165    
Attachments:
Description Flags
advanced options dialog with highlighted error
none
dialog without advanced options - error highlighting is missing here none

Description Petr Spacek 2014-04-08 13:25:09 UTC
Description of problem:
Dialog is not highlighted if VM cannot be created before clicking to "Show Advanced Options". See steps to reproduce.

Version-Release number of selected component (if applicable):
3.3.1-0.48.el6ev

How reproducible:
100 %

Steps to Reproduce:
1. Log-in into user portal with an user not allowed to create VM from a template
2. Open "new VM" dialog
3. Select VM template
4. Fill-in VM name
5. Click to Ok buton

Actual results:
VM is not created, no error message is displayed, no part of dialog is highlighted. See screenshot simple.png. The error is hidden under "Show Advanced Options" button. See screenshot advanced-options.png.


Expected results:
A problematic part of the dialog is highlighted. View is switched to Advanced Options when necessary.

Comment 1 Petr Spacek 2014-04-08 13:25:59 UTC
Created attachment 884071 [details]
advanced options dialog with highlighted error

Comment 2 Petr Spacek 2014-04-08 13:26:50 UTC
Created attachment 884072 [details]
dialog without advanced options - error highlighting is missing here

Comment 3 Omer Frenkel 2014-04-09 07:41:47 UTC
Tomas, is it possible to jump to place of error in the dialog when clicking ok?

Comment 4 Tomas Jelinek 2014-04-10 06:57:51 UTC
@Omer - well, it is not that simple - where shell I jump if there where errors on more places? Especially if some are visible in basic context and some in advanced only? Also, the user could be a bit confused if after clicking ok he would end up in a totally different part of the same dialog.

A simpler solution would be to do this:
- you have the basic options shown only and have a validation error
- click ok
- if all the errors are only on fields which are visible in basic mode only, than do nothing
- if at least one error is visible in advanced mode, than show the advanced options (so the validation error place will be visible)

Would this be acceptable?

Comment 5 Tomas Jelinek 2014-04-10 07:09:05 UTC
*** Bug 1008426 has been marked as a duplicate of this bug. ***

Comment 6 Michal Skrivanek 2014-08-14 08:38:14 UTC
see https://bugzilla.redhat.com/show_bug.cgi?id=1128440#c9 for some further thoughts

Comment 8 Lukas Svaty 2014-10-06 13:09:04 UTC
verified in vt4

Comment 10 errata-xmlrpc 2015-02-11 18:00:31 UTC
Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory, and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

https://rhn.redhat.com/errata/RHSA-2015-0158.html