Project

General

Profile

Bug #32217

OS search on host list is broken

Added by Ewoud Kohl van Wijngaarden 6 months ago. Updated 6 months ago.

Status:
Closed
Priority:
Normal
Category:
Inventory
Target version:
-
Difficulty:
Triaged:
No
Bugzilla link:
Fixed in Releases:
Found in Releases:

Description

When on the host list you click on the OS it searches for it but can't find anything. This regression was introduced in #15406.


Related issues

Related to Foreman - Refactor #15406: Move puppet_environment overview field outside core's #overview_fields helper.Closed
Related to Foreman - Bug #17279: Host page Operating System should search title, not description. Closed2016-11-08

Associated revisions

Revision b8fa01d6 (diff)
Added by Ewoud Kohl van Wijngaarden 6 months ago

Fixes #32217 - Correct OS search on host list

377ff667bbe3c2df8cd0831d9fe60bb9b316d680 introduced a regression and
changed the OS search on the host list from os_title = "#{host.os.title}"
to os_description = #{host.os.desc}. This reverts commit
dc4cc33745c7fc6bff2314c271686aea2399d55d and also introduces a
regression when it comes to spaces. Without the quotes it can't find
anything.

Fixes: 377ff667bbe3c2df8cd0831d9fe60bb9b316d680

History

#1 Updated by Ewoud Kohl van Wijngaarden 6 months ago

  • Related to Refactor #15406: Move puppet_environment overview field outside core's #overview_fields helper. added

#2 Updated by Ewoud Kohl van Wijngaarden 6 months ago

  • Related to Bug #17279: Host page Operating System should search title, not description. added

#3 Updated by The Foreman Bot 6 months ago

  • Assignee set to Ewoud Kohl van Wijngaarden
  • Status changed from New to Ready For Testing
  • Pull request https://github.com/theforeman/foreman/pull/8417 added

#4 Updated by The Foreman Bot 6 months ago

  • Fixed in Releases 2.5.0 added

#5 Updated by Ewoud Kohl van Wijngaarden 6 months ago

  • Status changed from Ready For Testing to Closed

#6 Updated by Tomer Brisker 6 months ago

  • Fixed in Releases 2.3.4, 2.4.1 added

Also available in: Atom PDF