Project

General

Profile

Actions

Bug #5161

closed

Default Content View should not be selectable if it cannot be added to a composite

Added by Walden Raines over 10 years ago. Updated over 6 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
Web UI
Target version:
Difficulty:
Triaged:
Yes
Fixed in Releases:
Found in Releases:

Description

(In reply to Corey Welton from comment #0)

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:

Actions #1

Updated by Walden Raines over 10 years ago

  • Status changed from Assigned to Ready For Testing
Actions #2

Updated by Walden Raines over 10 years ago

  • Category set to Web UI
Actions #3

Updated by Walden Raines over 10 years ago

  • Bugzilla link set to https://bugzilla.redhat.com/show_bug.cgi?id=1082712
Actions #4

Updated by Walden Raines over 10 years ago

  • Status changed from Ready For Testing to Closed
  • % Done changed from 0 to 100

Applied in changeset katello|commit:a072481e9e950e8dc5137220346ac3b10842dc56.

Actions #5

Updated by Mike McCune over 10 years ago

  • Status changed from Closed to Assigned

This appears to be still occurring in a more subtle way, the Default is no longer shown but it appears to be select-able:

1) go to the Content Views tab of a Composite View

2) Ensure you have multiple views available

3) Hit the Select All checkbox to select all views

4) Hit "Add"

5) note error:

"An error occurred updating the Content View: Cannot add default content view to composite content view "

Actions #6

Updated by Walden Raines over 10 years ago

  • Status changed from Assigned to Closed

The issue you are seeing here is #5181.

Actions #7

Updated by Eric Helms over 10 years ago

  • Target version set to 44
Actions #8

Updated by Eric Helms over 10 years ago

  • Triaged changed from No to Yes
Actions #9

Updated by Eric Helms over 10 years ago

  • Translation missing: en.field_release set to 13
Actions

Also available in: Atom PDF