Project

General

Profile

Bug #31495

Javascript error on logout page "TypeError: Cannot read property 'icon' of null"

Added by Eric Helms 6 months ago. Updated 6 months ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
Web Interface
Target version:
-
Difficulty:
Triaged:
No
Bugzilla link:

Description

Full error:

react-dom.production.min.js:187 TypeError: Cannot read property 'icon' of null
    at Breadcrumb.js:33
    at Array.map (<anonymous>)
    at l (Breadcrumb.js:29)
    at Ra (react-dom.production.min.js:148)
    at Gs (react-dom.production.min.js:245)
    at Ds (react-dom.production.min.js:216)
    at Ns (react-dom.production.min.js:210)
    at Ms (react-dom.production.min.js:206)
    at react-dom.production.min.js:114

Associated revisions

Revision fb285ce9 (diff)
Added by Eric Helms 6 months ago

Fixes #31495: Set a title on the logout page

This prevents a javascript error by the Breadcrumb component and
adds some notion of what page the user is on.

Revision f53cc395 (diff)
Added by Tomer Brisker 6 months ago

Refs #31495 - Properly handle missing breadcrumb caption

If `caption` is null, deconstructing it will fail. Added a guard clause
to properly handle this case. This can occur when a page has no title
set.

Revision 1b7b7e95 (diff)
Added by Tomer Brisker 6 months ago

Refs #31495 - Remove unused title property

This property is not populated or used anywhere.

History

#1 Updated by The Foreman Bot 6 months ago

  • Assignee set to Eric Helms
  • Status changed from Assigned to Ready For Testing
  • Pull request https://github.com/theforeman/foreman/pull/8200 added

#2 Updated by The Foreman Bot 6 months ago

  • Fixed in Releases 2.4.0 added

#3 Updated by Eric Helms 6 months ago

  • Status changed from Ready For Testing to Closed

#4 Updated by Tomer Brisker 6 months ago

  • Fixed in Releases 2.2.3, 2.3.2 added

#5 Updated by The Foreman Bot 6 months ago

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

Also available in: Atom PDF