Project

General

Profile

Bug #30643

ForemanVirtWhoConfigure::AuthSourceHiddenWithAuthentication is displayed on auth sources page

Added by Tomer Brisker 2 months ago. Updated about 1 month ago.

Status:
Closed
Priority:
Low
Assignee:
Category:
Authentication
Target version:
-
Difficulty:
Triaged:
No
Bugzilla link:
Fixed in Releases:
Found in Releases:

Description

Cloned from https://bugzilla.redhat.com/show_bug.cgi?id=1866515

Description of problem:

label for auth_source_type = ForemanVirtWhoConfigure::AuthSourceHiddenWithAuthentication is missing, along with a shorter box

Version-Release number of selected component (if applicable):
Red Hat Satellite 6.8 [snap 09]

How reproducible:
Always

Steps to Reproduce:
1. Administer --> Authentication sources

Actual results:

- Box, which redirects to auth_source_type = ForemanVirtWhoConfigure::AuthSourceHiddenWithAuthentication is missing

- Box/Button is shorter

Expected results:

- Appropriate Label and Box size
- Authentication sources , should be in capital letter. "S"

Associated revisions

Revision a4c0a160 (diff)
Added by Tomer Brisker about 2 months ago

Fixes #30643 - Allow inheriting auth source classes (#7905)

AuthSource subclasses may be inherited, leading scopes to return
unexpected results - for example, AuthSource.except_hidden returned auth
sources that inherited the AuthSourceHidden class. The scopes have been
modified to also take into account the descendants of the various types.

History

#1 Updated by The Foreman Bot 2 months ago

  • Assignee set to Tomer Brisker
  • Status changed from New to Ready For Testing
  • Pull request https://github.com/theforeman/foreman/pull/7905 added

#2 Updated by The Foreman Bot about 2 months ago

  • Fixed in Releases 2.3.0 added

#3 Updated by Tomer Brisker about 2 months ago

  • Subject changed from label for authentication source type ForemanVirtWhoConfigure::AuthSourceHiddenWithAuthentication is missing, along with a shorter box to ForemanVirtWhoConfigure::AuthSourceHiddenWithAuthentication is displayed on auth sources page

#4 Updated by Tomer Brisker about 2 months ago

  • Status changed from Ready For Testing to Closed

#5 Updated by The Foreman Bot about 1 month ago

  • Pull request https://github.com/theforeman/foreman/pull/7975 added

Also available in: Atom PDF