Wrong Null Check in Dashboard Controller

Description

Steps to reproduce:

  • create a status line, for any message (e.g. Test), sql: SELECT 1

  • create a Dashboard Content for the status line

  • navigate to Dashboard Content Translation

  • paste any text to the HTML field and save

  • delete the HTML field (should be empty) and save

After this in our DB the HTML column contains an empty string ('') instead of null, this adds an empty <span> element to the status line in the dashboard in the given language.

Although I could not reproduce it in iDempiere, generally in these cases Util.isEmpty should be used instead of != null check, I think it’s a trivial fix.

Here is an example how it looks on a status line and on a chart:

Environment

None

Attachments

1
  • 09 Apr 2024, 07:00 AM

Activity

Fixed

Details

Assignee

Reporter

Fix versions

Priority

Created April 9, 2024 at 7:00 AM
Updated June 1, 2024 at 3:18 PM
Resolved April 9, 2024 at 1:13 PM