Project

General

Profile

Bug #32389

Incorrect search query is formed while selecting location/organization of the host

Added by Ondřej Ezr 5 months ago. Updated 5 months ago.

Status:
Closed
Priority:
Low
Category:
Inventory
Target version:
-

Description

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

Description of problem:

If you have Organization or Location with 2 or more words like (Default Organization or Default Location). GoTo Hosts -> All Hosts -> Select/Click a host to see details. On the left-hand detail box, click Organization name (Default Organization) or Location name (Default Location).

This will take you to the Hosts -> All Hosts page with search query but the query is incorrect so you will see no hosts.

query formed is as below
~~
organization = Default Organization
~
~

Version-Release number of selected component (if applicable):
Foreman 1.24

How reproducible:
Always

Steps to Reproduce:
1. Select the host in "Default Organization"
2. Click the host to see detail page
3. Click on the Default Organization

Actual results:
Wrong search query formed as a result no hosts is shown.
~~
organization = Default Organization
~
~

Expected results:
Search query should be
~~
organization = "Default Organization" ----> In Quotes
~
~

Additional info:
This works fine in Satellite 6.6

Associated revisions

Revision 7fabf78d (diff)
Added by Adi Abramovitch 5 months ago

Fixes #32389 - Quote taxonomies in search query (#8490)

Fixes bug introduced by 377ff66, where we forgot to quote search query strings in links on host details page.

History

#1 Updated by Adi Abramovitch 5 months ago

  • Assignee set to Adi Abramovitch
  • Subject changed from Incorrect search query is formed while selecting location/organization of the host to Incorrect search query is formed while selecting location/organization of the host

#2 Updated by The Foreman Bot 5 months ago

  • Status changed from New to Ready For Testing
  • Pull request https://github.com/theforeman/foreman/pull/8490 added

#3 Updated by Ewoud Kohl van Wijngaarden 5 months ago

  • Triaged changed from No to Yes
  • Found in Releases 1.24.0 added

#4 Updated by The Foreman Bot 5 months ago

  • Fixed in Releases 2.5.0 added

#5 Updated by Anonymous 5 months ago

  • Status changed from Ready For Testing to Closed

#6 Updated by The Foreman Bot 5 months ago

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

#7 Updated by The Foreman Bot 5 months ago

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

#8 Updated by Tomer Brisker 5 months ago

  • Fixed in Releases 2.4.1 added

Also available in: Atom PDF