Openbravo Issue Tracking System - Openbravo ERP
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0036829Openbravo ERPA. Platformpublic2017-09-11 17:592018-02-22 18:19
Reporterdmiguelez 
Assigned Todmiguelez 
PrioritynormalSeverityminorReproducibilityalways
StatusclosedResolutionfixed 
PlatformOS5OS Version
Product Version 
Target VersionFixed in Version3.0PR18Q2 
Merge Request Status
Review Assigned Toalostale
OBNetwork customer
Web browser
ModulesCore
Support ticket
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary0036829: In Monitor Tab on Process Request Window, it should be possible to see a large Log
DescriptionIn Monitor Tab on Process Request Window, it should be possible to see a Log larger than 4000 characters as it is working now.
Steps To ReproduceThere is no process in Core as of today that can produce a Log this large.
However, this can be accomplished by some processes in Advanced Warehouse Operations depending on the configuration.

It is easier to understand it by looking into the Code.
In ProcessMonitor.java Class, the log sent when updating the Process Run data is trucated to 4000 characters:

final String executionLog = bundle.getLog().length() >= 4000 ? bundle.getLog().substring(0, 3999) : bundle.getLog();

This was fixed in a previous commit, but for the Process Runner Class, and should be fixed here as well:
https://code.openbravo.com/erp/devel/pi/rev/e81caae0b006 [^]
Proposed SolutionThere is a file attached with the proposed fix:
* Remove artificial constraint in the Code
* Remove displayLenght of Log field
* Changed field length in Log columns to the max numer allowed
Additional Information
TagsNo tags attached.
Relationships
related to defect 0015739 closed alostale In Oracle, I get an error when disabling Heartbeat 
related to defect 0015115 closed alostale Disabling heartbeat is not working under some circunstances in 3.0 RC2 
Attached Filesdiff fix.diff (3,345) 2017-09-11 17:59
https://issues.openbravo.com/file_download.php?file_id=11055&type=bug
Issue History
Date ModifiedUsernameFieldChange
2017-09-11 17:59dmiguelezNew Issue
2017-09-11 17:59dmiguelezAssigned To => platform
2017-09-11 17:59dmiguelezFile Added: fix.diff
2017-09-11 17:59dmiguelezModules => Core
2017-09-11 17:59dmiguelezTriggers an Emergency Pack => No
2017-09-11 17:59dmiguelezAssigned Toplatform => alostale
2018-02-09 10:58alostaleRelationship addedrelated to 0015739
2018-02-09 10:58alostaleRelationship addedrelated to 0015115
2018-02-09 11:35alostaleAssigned Toalostale => dmiguelez
2018-02-09 11:35alostaleReview Assigned To => alostale
2018-02-09 11:36hgbotCheckin
2018-02-09 11:36hgbotNote Added: 0102299
2018-02-09 11:36hgbotStatusnew => resolved
2018-02-09 11:36hgbotResolutionopen => fixed
2018-02-09 11:36hgbotFixed in SCM revision => http://code.openbravo.com/erp/devel/pi/rev/dcb9af89d341ecda193b725f0ba8f84176f57868 [^]
2018-02-09 11:36alostaleNote Added: 0102300
2018-02-09 11:36alostaleStatusresolved => closed
2018-02-09 11:36alostaleFixed in Version => 3.0PR18Q2
2018-02-22 18:19hudsonbotCheckin
2018-02-22 18:19hudsonbotNote Added: 0102741

Notes
(0102299)
hgbot   
2018-02-09 11:36   
Repository: erp/devel/pi
Changeset: dcb9af89d341ecda193b725f0ba8f84176f57868
Author: David Miguelez <david.miguelez <at> openbravo.com>
Date: Fri Feb 09 11:32:12 2018 +0100
URL: http://code.openbravo.com/erp/devel/pi/rev/dcb9af89d341ecda193b725f0ba8f84176f57868 [^]

fixed bug 36829: can't display long process execution logs

  Even in database is defined as an arbitary lenght text, logs were truncated
  to 4K characters. This artificial restriction has been removed, so that logs
  can now be longer.

---
M src-db/database/sourcedata/AD_COLUMN.xml
M src-db/database/sourcedata/AD_FIELD.xml
M src/org/openbravo/scheduling/ProcessMonitor.java
---
(0102300)
alostale   
2018-02-09 11:36   
reviewed + tested
(0102741)
hudsonbot   
2018-02-22 18:19   
A changeset related to this issue has been promoted main and to the
Central Repository, after passing a series of tests.

Promotion changeset: https://code.openbravo.com/erp/devel/main/rev/980a6ad5bbf5 [^]
Maturity status: Test