Form errors from modal should be handled in modal

[andrewsben]

Don't know if this is done on purpose or not, but it is functional so I guess this would be a UX improvement thing...

If while launching an instance I don't include a required field, such as server name, the form redirects to a launch page instead of staying on the images page with the popup.

Steps:
Goto images page
select desired image and click launch.
hit submit without putting a server name
redirects to URL/nova/images_and_snapshots/IMAGE_ID/launch/

Expected / Desired:
Stay on the same page keeping the nice look and refreshing the popup dialogue.

[Dannina Anil Kumar (dannina-kumar)]

Hi,

I fixed this bug(943518).The changes are as follows.

In /horizon/horizon/horizon/dashboards/nova/templates/nova/images_and_snapshots/images/_launch.html file, we included one javascript that verifies whether server name field is empty or not.

If the field is empty, an error message is displayed stating that "Server Name field should not be empty". Textbox border will become "Red" color and background will become "Yellow".

Please let me know if anybody started working on this bug. If the work is not yet started on this bug, I will assign this bug to myself and check-in the code into repository.

The Java script is:

<script type="text/javascript">
function check()
{
var textBoxField = launch_image_form.id_name;
var errlabel=document.getElementById('error');
if(textBoxField.value == "")
{
        errlabel.innerHTML = "Error : Server Name field can't be empty.";
        launch_image_form.id_name.style.border = "solid 1px red";
        launch_image_form.id_name.style.background = "yellow"
        return false;
}
else
{
        return true;
}
}
</script>

Thanks and Regards,
Anil.

[Andy Chong (andycjw)]

this should be fixed in a generic way, e.g. a modal form handler in modal.js that submit form and replace the form in-place if the form submission failed, i would like to assign this to myself if emma is not working on it

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: master
Review: https://review.openstack.org/5312

[Gabriel Hurley (gabriel-hurley)]

Doing a full pass at modal form validation is out-of-scope for essex at this point.

[Gabriel Hurley (gabriel-hurley)]

After a good deal of discussion, several of the horizon core team have decided we will support a basic degree of form handling to refresh error'd forms into the modal for Essex.

Further development beyond that will be tracked by this blueprint: https://blueprints.launchpad.net/horizon/+spec/global-ajax-communication

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/5312
Committed: http://github.com/openstack/horizon/commit/c11cd9d1c5a4cfcc0120b5ab75564ef1d2df15f6
Submitter: Jenkins
Branch: master

commit c11cd9d1c5a4cfcc0120b5ab75564ef1d2df15f6
Author: Gabriel Hurley <email address hidden>
Date: Wed Mar 21 16:14:59 2012 -0700

    Implements AJAX form posting.

    This is somewhat of a hack for Essex, since the long-term solution
    is a reworking of the way AJAX is handled. But it solves problems
    in the interim and provides a significantly better experience.

    Thanks to Andy Chong for pushing forward with the initial attempts
    which lead to this patch.

    Fixes bug 943518.

    Change-Id: Ia65d926d3d406b07301e23b4c87de60c66ddec75