Bug 1131204 - activation-key: red border doesn't appears on updating content-host limit with invalid value
Summary: activation-key: red border doesn't appears on updating content-host limit wit...
Keywords:
Status: CLOSED WONTFIX
Alias: None
Product: Red Hat Satellite
Classification: Red Hat
Component: WebUI
Version: 6.0.4
Hardware: Unspecified
OS: Unspecified
unspecified
low
Target Milestone: Unspecified
Assignee: Walden Raines
QA Contact: Katello QA List
URL: http://projects.theforeman.org/issues...
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2014-08-18 16:51 UTC by Sachin Ghai
Modified: 2017-01-04 20:22 UTC (History)
5 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2017-01-04 20:22:55 UTC
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)
red border doesn't appears on updating content-host limit with invalid value (52.82 KB, image/png)
2014-08-18 16:51 UTC, Sachin Ghai
no flags Details


Links
System ID Private Priority Status Summary Last Updated
Foreman Issue Tracker 7142 0 None None None 2016-04-22 14:53:30 UTC

Description Sachin Ghai 2014-08-18 16:51:36 UTC
Created attachment 927991 [details]
red border doesn't appears on updating content-host limit with invalid value

Description of problem:
Found a failure in automation where we are updating the content-host-limit of an existing activation-key with some invalid value (say with some string), and UI doesn't highlight the input box with red border and save button remains disabled.

Please note that this happens only while editing the activation-key. When we use invalid limit while creating activation-key, the red border appears. So the behaviour should be consistent with create activation-key. 

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

How reproducible:
always

Steps to Reproduce:
1. create a activation-key with default content-host-limit
2. update the limit with either string or -ve value
3.

Actual results:
UI doesn't highlight the input box with red border, when we update content-host-limit with some invalid value.

Expected results:
UI should highlight the input box with red border while updating the limit with some invalid value

Additional info:

Comment 2 Walden Raines 2014-08-18 19:24:26 UTC
Created redmine issue http://projects.theforeman.org/issues/7142 from this bug

Comment 3 Bryan Kearney 2015-08-25 20:04:13 UTC
Upstream bug component is WebUI

Comment 4 Bryan Kearney 2015-08-25 20:24:17 UTC
Upstream bug component is Activation Keys

Comment 5 Walden Raines 2015-09-15 18:47:39 UTC
(In reply to Sachin Ghai from comment #0)
> Expected results:
> UI should highlight the input box with red border while updating the limit
> with some invalid value

We currently don't do this anywhere else in the application.  Are we sure this is the expected behavior in this case?  If so, we need to make this change everywhere in the application.

Comment 6 Sachin Ghai 2015-10-15 07:50:37 UTC
Currently, when we create an activation-key or a hostgroup with invalid value of content-host limit then UI raises error as highlighted box with red border.

However, when we update the activation-key with invalid limit then UI doesn't raise any error instead make 'save' button inactive.

I believe behaviour should be consistent both while creating and updating the entity limit. Is there any specific reason for making it different while updating ?

Comment 8 Walden Raines 2015-10-21 17:25:27 UTC
(In reply to Sachin Ghai from comment #6)
> I believe behaviour should be consistent both while creating and updating
> the entity limit. Is there any specific reason for making it different while
> updating ?

So currently it is consistent with editing other entities, it's just not consistent with creating entities.  We can add a red border when editing activation key limits but we also need to do the same for other entities (host collections, etc).  I'm fine with doing this, just note that other entities need to be tested as well.

Comment 9 Bryan Kearney 2015-12-21 21:00:40 UTC
Upstream bug component is Uncategorized

Comment 10 Bryan Kearney 2016-02-25 21:01:25 UTC
Upstream bug assigned to walden

Comment 11 Bryan Kearney 2016-02-25 21:01:27 UTC
Upstream bug component is Uncategorized

Comment 12 Bryan Kearney 2016-04-13 20:01:19 UTC
Upstream bug component is Uncategorized

Comment 13 Bryan Kearney 2016-12-16 19:04:57 UTC
Upstream bug component is WebUI

Comment 14 Bryan Kearney 2017-01-04 20:22:55 UTC
This is an older bug, which is being tracked upstream. We will not be tracking this downstream. When the upstream issue is resolved, the next rebase will pull in the fix.


Note You need to log in before you can comment on or make changes to this bug.