horizon fails to add security group rule if hidden field has invalid value

Bug #1237128 reported by Russell Sim
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Dashboard (Horizon)
Fix Released
Low
Russell Sim

Bug Description

Hey,

I found that when you for example fill in the port field by accident
with invalid data. Then change the port_or_range selector to 'port range'
and fill in valid data in the to and from fields, the failed validation of the
port field will prevent you from successfully submitting the form.

The error message is also hidden from the user because the field that
failed to validate isn't visible.

This same problem exists when you fill in invalid port and or port_range
then change to icmp. Or insert an invalid cidr then change to the
Security Group selector.

Thanks,

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to horizon (master)

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

Changed in horizon:
assignee: nobody → Russell Sim (russell-sim)
status: New → In Progress
Kieran Spear (kspear)
Changed in horizon:
importance: Undecided → Low
milestone: none → icehouse-1
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to horizon (master)

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

commit fa3a039d0585c8c995504c09657070af4025305f
Author: Russell Sim <email address hidden>
Date: Wed Oct 9 12:54:41 2013 +1100

    Ignore irrelevant secgroup form field errors

    When cleaning data from in the security group rule form, remove any
    associated error messages that are not related to the actual fields
    being saved. If we are setting a value derived from another field, also
    delete the error message associated with that field.

    Change-Id: I1eb4835ea2368b62374f230c6293062a2340980d
    Closes-Bug:1237128

Changed in horizon:
status: In Progress → Fix Committed
Thierry Carrez (ttx)
Changed in horizon:
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in horizon:
milestone: icehouse-1 → 2014.1
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.