image create/edit image wizard allows name with white spaces only

Bug #1291414 reported by Benny Kopilov
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Dashboard (Horizon)
Won't Fix
Low
Unassigned

Bug Description

Description of problem:
When we create /modify image name , the user interface does not allow to set empty string
When user set single white space , the image name changed to image_id .

when name contains only what spaces it should be denied as empty string .

Version-Release number of selected component (if applicable):

How reproducible:
always

Steps to Reproduce:
create image name from dashboard .
change the name with edit option , set spaces only .
save and name changed to image_id .

Actual results:
the image name taken from image_id in database.

Expected results:
white spaces name should be rejected by ui .

Changed in horizon:
assignee: nobody → Juan Manuel Ollé (juan-m-olle)
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/80323

Changed in horizon:
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on horizon (master)

Change abandoned by Juan Manuel Ollé (<email address hidden>) on branch: master
Review: https://review.openstack.org/80323
Reason: This is wrong, overloading _clean_fields if a very bad idea, it depends on django internal implementation.

From an OO design point of view, extending CharField is the correct way to fix this issue.

Patch 6 is the correct way to fix this.

Thanks all of you for your time

Changed in horizon:
assignee: Juan Manuel Ollé (juan-m-olle) → nobody
status: In Progress → New
Changed in horizon:
status: New → Confirmed
importance: Undecided → Low
Gloria Gu (gloria-gu)
Changed in horizon:
assignee: nobody → Gloria Gu (gloria-gu)
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/115064

Changed in horizon:
status: Confirmed → In Progress
Gloria Gu (gloria-gu)
summary: - image create/edit image wizard name with white spaces only
+ image create/edit image wizard allows name with white spaces only
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on horizon (master)

Change abandoned by Gloria Gu (<email address hidden>) on branch: master
Review: https://review.openstack.org/115064
Reason: In the UI world, if it would be better prevent user entering empty spaces if they cause issues. It is a common practice. However openstack folks don't agree. It is a trivial issue so it is not worth to push it.

Gloria Gu (gloria-gu)
Changed in horizon:
assignee: Gloria Gu (gloria-gu) → nobody
Changed in horizon:
status: In Progress → Won't Fix
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.