0020819: NullPointerException is thrown if a time is null

Openbravo Issue Tracking System - Openbravo ERP

View Issue Details

Project

Category

View Status

Date Submitted

Last Update

0020819

Openbravo ERP

A. Platform

public

2012-06-20 16:57

2012-06-27 05:28

Reporter

AugustoMauch

Assigned To

AugustoMauch

Priority

immediate

Severity

major

Reproducibility

sometimes

Status

closed

Resolution

fixed

Platform

OS Version

Product Version

Target Version

3.0MP13

Fixed in Version

3.0MP13

Merge Request Status

Review Assigned To

OBNetwork customer

Web browser

Modules

Core

Support ticket

Regression level

Pre packaging ( pi )

Regression date

2012-06-12

Regression introduced in release

Regression introduced by commit

https://code.openbravo.com/erp/devel/pi/rev/976c8398def2e80de470ab55db222555d88566d8 [^]

Triggers an Emergency Pack

Summary

0020819: NullPointerException is thrown if a time is null

Description

In the Process Request window, is a process has a time with value null, the getFieldProperties of TimeUIDefinition throws the following exception:

pro595dbc4c 1872929 [http-8880-4] ERROR org.openbravo.base.exception.OBException - Couldn't get data for column Finishes_Time
java.lang.NullPointerException
    at org.openbravo.client.kernel.reference.TimeUIDefinition.getFieldProperties(TimeUIDefinition.java:89)
    at org.openbravo.client.application.window.FormInitializationComponent.computeColumnValues(FormInitializationComponent.java:570)
    at org.openbravo.client.application.window.FormInitializationComponent.execute(FormInitializationComponent.java:227)
    at org.openbravo.client.kernel.BaseActionHandler.execute(BaseActionHandler.java:71)
    at org.openbravo.client.kernel.KernelServlet.processActionRequest(KernelServlet.java:193)

Steps To Reproduce

- Open the Process Request window
- Open in form view a process whose Finish Time is null.
- Return to grid view. The exception will be thrown

Proposed Solution

Replace the columnValue.isEmpty() with StringUtils.isEmpty(columnValue). That way, the function will return true for a null columnValue, instead of throwing a NullPointerException.

Additional Information