Bug 1082712 - Default Content View should not be selectable if it cannot be added to a composite
Summary: Default Content View should not be selectable if it cannot be added to a comp...
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Red Hat Satellite
Classification: Red Hat
Component: WebUI
Version: Nightly
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: Unspecified
Assignee: Walden Raines
QA Contact: Corey Welton
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2014-03-31 16:54 UTC by Corey Welton
Modified: 2019-09-25 20:47 UTC (History)
2 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2014-07-02 14:04:52 UTC
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)

Description Corey Welton 2014-03-31 16:54:54 UTC
Description of problem:
By design, the default content view cannot be added to a composite.  However, it appears in the list when choosing views to be added to a composite.  

If we cannot add it, it seems needlessly error-prone to make it appear like it can be selected.  We should hide it or gray out the checkbox, perhaps with a tooltip noting that the default view cannot be added to any composites.

Version-Release number of selected component (if applicable):
* apr-util-ldap-1.3.9-3.el6_0.1.x86_64
* candlepin-0.9.7-1.el6_5.noarch
* candlepin-scl-1-5.el6_4.noarch
* candlepin-scl-quartz-2.1.5-5.el6_4.noarch
* candlepin-scl-rhino-1.7R3-1.el6_4.noarch
* candlepin-scl-runtime-1-5.el6_4.noarch
* candlepin-selinux-0.9.7-1.el6_5.noarch
* candlepin-tomcat6-0.9.7-1.el6_5.noarch
* elasticsearch-0.90.10-4.el6sat.noarch
* foreman-1.5.0.11-1.el6sat.noarch
* foreman-postgresql-1.5.0.11-1.el6sat.noarch
* foreman-proxy-1.5.4-1.el6sat.noarch
* foreman-selinux-1.5.0-0.develop.el6sat.noarch
* katello-1.5.0-19.el6sat.noarch
* katello-apache-1.0-1.noarch
* katello-ca-1.0-1.noarch
* katello-certs-tools-1.5.5-1.el6sat.noarch
* katello-installer-0.0.31-1.el6sat.noarch
* openldap-2.4.23-32.el6_4.1.x86_64
* openldap-devel-2.4.23-32.el6_4.1.x86_64
* pulp-katello-plugins-0.2-1.el6sat.noarch
* pulp-nodes-common-2.3.1-0.4.beta.el6sat.noarch
* pulp-nodes-parent-2.3.1-0.4.beta.el6sat.noarch
* pulp-puppet-plugins-2.3.1-0.4.beta.el6sat.noarch
* pulp-rpm-plugins-2.3.1-0.4.beta.el6sat.noarch
* pulp-selinux-2.3.1-0.4.beta.el6sat.noarch
* pulp-server-2.3.1-0.4.beta.el6sat.noarch
* python-ldap-2.3.10-1.el6.x86_64
* ruby193-rubygem-ldap_fluff-0.2.2-2.el6sat.noarch
* ruby193-rubygem-net-ldap-0.3.1-3.el6sat.noarch
* ruby193-rubygem-runcible-1.0.8-1.el6sat.noarch
* rubygem-hammer_cli-0.1.0-4.el6sat.noarch
* rubygem-hammer_cli_foreman-0.1.0-5.el6sat.noarch
* rubygem-hammer_cli_foreman_tasks-0.0.2-2.el6sat.noarch
* rubygem-hammer_cli_katello-0.0.3-11.el6sat.noarch

How reproducible:


Steps to Reproduce:
1.  Create some content views
2.  Attempt to create a composite content view; from the available content view options, select the default view alongside any others you might want to include.
3.  View resulting error mesg.

Actual results:
User is told that default view cannot be added to any composites.

Expected results:
Disable selection of, and/or hide this view if it cannot be added.

Additional info:

Comment 2 Walden Raines 2014-04-11 18:19:13 UTC
PR: https://github.com/Katello/katello/pull/3974

Comment 3 Walden Raines 2014-04-14 14:52:03 UTC
commit 06dfb2d54ac9f1b8aa7716e1123614958ee8df30
Merge: 04de2f4 a072481
Author: Walden Raines <walden>
Date:   Mon Apr 14 10:51:26 2014 -0400

    Merge pull request #3974 from waldenraines/bug-5161
    
    Fixes #5161, BZ1082712 - hide the default content view for composites.

commit a072481e9e950e8dc5137220346ac3b10842dc56
Author: Walden Raines <walden>
Date:   Fri Apr 11 14:17:55 2014 -0400

    Fixes #5161, BZ1082712 - hide the default content view for composites.

Comment 5 Mike McCune 2014-05-05 17:52:11 UTC
re-opened the upstream issue:

http://projects.theforeman.org/issues/5161

Comment 6 Walden Raines 2014-05-05 17:56:58 UTC
Re-closed upstream issue as a duplicate of http://projects.theforeman.org/issues/5181.

Comment 7 Corey Welton 2014-05-16 19:30:46 UTC
Verified in Satellite-6.0.3-RHEL-6-20140508.1

Comment 8 Bryan Kearney 2014-07-02 14:04:52 UTC
This was delivered with 6.0.3, which is the Satellite 6 Beta.


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