Bug #33049: Make name_column helper defensive with displaying statuses - Foreman

Bug #33049

Make name_column helper defensive with displaying statuses

Added by Tomer Brisker over 1 year ago. Updated over 1 year ago.

Status:

Closed

Priority:

Normal

Assignee:

Tomer Brisker

Category:

Inventory

Target version:

Difficulty:

Triaged:

Bugzilla link:

Pull request:

https://github.com/theforeman/foreman/pull/8675

Fixed in Releases:

2.5.3, 3.0.0

Found in Releases:

Red Hat JIRA:

Description

While this shouldn't occur, if for some reason a host status has no type column the name_column helper will cause the page to fail loading with

 
Backtrace for 'comparison of String with nil failed' error (ActionView::Template::Error): comparison of String with nil failed
 | /usr/share/foreman/app/helpers/hosts_helper.rb:97:in `sort_by'
 | /usr/share/foreman/app/helpers/hosts_helper.rb:97:in `name_column'
...

We can avoid this failure easily by being more defensive in our code.

Associated revisions

Revision 29aceca8 (diff)
Added by Tomer Brisker over 1 year ago

Fixes #33049 - Properly handle missing status type (#8675)

While this shouldn't occur, it appears in some edge cases a status might
be created with no type property. In that case we should not display the
status and not try to sort on it.