0055238: getProcessIdForAudit for TerminalTypeUserConfigurationLoaderProcessor is returning a value higher than 32 and it fails

Openbravo Issue Tracking System - POS2

View Issue Details

Project

Category

View Status

Date Submitted

Last Update

0055238

POS2

POS

public

2024-03-20 13:06

2024-04-18 15:24

Reporter

ngarcia

Assigned To

SABARINATH P

Priority

urgent

Severity

major

Reproducibility

unable to reproduce

Status

closed

Resolution

fixed

Platform

OS Version

Product Version

Target Version

24Q1.2

Fixed in Version

24Q1.2

Merge Request Status

Review Assigned To

OBNetwork customer

Support ticket

Regression level

Regression date

Regression introduced in release

Regression introduced by commit

Triggers an Emergency Pack

Summary

0055238: getProcessIdForAudit for TerminalTypeUserConfigurationLoaderProcessor is returning a value higher than 32 and it fails

Description

getProcessIdForAudit for TerminalTypeUserConfigurationLoaderProcessor is returning a value higher than 32 and it fails.

The Data Import Entry related doesn't change to Error and it is always being processed

Steps To Reproduce

Set the SessionInfo to debug

Audit the following tables:

OBPOS_TerminalAccess
Organization
POSS_Terminal_Sessions

Change the configuration of the Terminal from Light mode to Black mode

Check a Data Import Entry of OBPOS2_UpdateTerminalTypeUserConfiguration type is created and it is not processed, it remains an Initial

The log shows:

DEBUG org.openbravo.database.SessionInfo - saving DB context info 4FBFAA553E184EEEA3A60CA1B7DE232D - null - IE - OBPOS2_UpdateTerminalTypeUserConfiguration

org.postgresql.util.PSQLException: ERROR: value too long for type character varying(32)
    at org.postgresql.core.v3.QueryExecutorImpl.receiveErrorResponse(QueryExecutorImpl.java:2676) ~[postgresql-42.5.0.jar:42.5.0]
    at org.postgresql.core.v3.QueryExecutorImpl.processResults(QueryExecutorImpl.java:2366) ~[postgresql-42.5.0.jar:42.5.0]
    at org.postgresql.core.v3.QueryExecutorImpl.execute(QueryExecutorImpl.java:356) ~[postgresql-42.5.0.jar:42.5.0]
    at org.postgresql.jdbc.PgStatement.executeInternal(PgStatement.java:496) ~[postgresql-42.5.0.jar:42.5.0]
    at org.postgresql.jdbc.PgStatement.execute(PgStatement.java:413) ~[postgresql-42.5.0.jar:42.5.0]
    at org.postgresql.jdbc.PgPreparedStatement.executeWithFlags(PgPreparedStatement.java:190) ~[postgresql-42.5.0.jar:42.5.0]
    at org.postgresql.jdbc.PgPreparedStatement.executeUpdate(PgPreparedStatement.java:152) ~[postgresql-42.5.0.jar:42.5.0]
    at jdk.internal.reflect.GeneratedMethodAccessor227.invoke(Unknown Source) ~[?:?]
    at jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43) ~[?:?]
    at java.lang.reflect.Method.invoke(Method.java:566) ~[?:?]
    at org.apache.tomcat.jdbc.pool.StatementFacade$StatementProxy.invoke(StatementFacade.java:114) ~[tomcat-jdbc.jar:?]
    at com.sun.proxy.$Proxy70.executeUpdate(Unknown Source) ~[?:?]
    at org.openbravo.database.SessionInfo.saveContextInfoIntoDB(SessionInfo.java:262) ~[openbravo-core.jar:?]
    at org.openbravo.dal.service.OBDal.flush(OBDal.java:263) ~[classes/:?]
    at org.openbravo.dal.core.SessionHandler.flushRemainingChanges(SessionHandler.java:700) ~[classes/:?]
    at org.openbravo.dal.core.SessionHandler.commitAndClose(SessionHandler.java:607) ~[classes/:?]
    at org.openbravo.dal.service.OBDal.commitAndClose(OBDal.java:228) ~[classes/:?]
    at org.openbravo.service.importprocess.ImportEntryManager.setImportEntryErrorIndependent(ImportEntryManager.java:550) ~[classes/:?]
    ... 8 more

Proposed Solution

In modules/org.openbravo.pos2/src/org/openbravo/pos2/importprocess/TerminalTypeUserConfigurationLoaderProcessor.java

change the return of the getProcessIdForAudit function to a value lower than 32 characters

Modify also the code in charge of setting the Data Import Entry to Error in this case

Additional Information