Openbravo Issue Tracking System - Openbravo ERP
View Issue Details
0035490Openbravo ERPA. Platformpublic2017-03-12 13:132017-03-15 20:22
normalminorhave not tried
0035490: SystemService logValidationResults logs a ERROR line always if there are no errors (shows in export.database)
The 'Validating module' part of export.database calls to SystemService in erp sourcecode to do the validation + to log the result.

That logValidationResult method assembles a StringBuffer and logs it using log4j.error.

However if does not check if that String does contain anything so logs an 'empty' error all the time.

Note that this is minor for the moment as export.database does have bad log4j config so it does not show the log-level prefix at the moment. Fixing that manually show the problem clearly like this:

24623 [main] INFO org.openbravo.ddlutils.task.ExportDatabase - Validating Module...
25022 [main] ERROR org.openbravo.ddlutils.task.ExportDatabase -
Run export.database with any module in development and notice the extra empty line on Validating module.
With the debugging patch attached to this issue it will show the wrong log level clearly.
No tags attached.
related to defect 0035493 closed alostale missing log info when check consistency fails 
diff 35490_fix.diff (445) 2017-03-12 13:14
diff debugging-do-not-commit-fix-log4j.diff (941) 2017-03-12 13:15
Issue History
2017-03-12 13:13shuehnerNew Issue
2017-03-12 13:13shuehnerAssigned To => platform
2017-03-12 13:13shuehnerModules => Core
2017-03-12 13:13shuehnerTriggers an Emergency Pack => No
2017-03-12 13:14shuehnerFile Added: 35490_fix.diff
2017-03-12 13:15shuehnerFile Added: debugging-do-not-commit-fix-log4j.diff
2017-03-12 13:15shuehnerNote Added: 0094956
2017-03-13 07:49alostaleAssigned Toplatform => shuehner
2017-03-13 07:49hgbotCheckin
2017-03-13 07:49hgbotNote Added: 0094968
2017-03-13 07:49hgbotStatusnew => resolved
2017-03-13 07:49hgbotResolutionopen => fixed
2017-03-13 07:49hgbotFixed in SCM revision => [^]
2017-03-13 07:51alostaleReview Assigned To => alostale
2017-03-13 07:51alostaleNote Added: 0094969
2017-03-13 07:51alostaleStatusresolved => closed
2017-03-13 07:51alostaleFixed in Version => 3.0PR17Q2
2017-03-13 09:17alostaleRelationship addedrelated to 0035493
2017-03-15 20:22hudsonbotCheckin
2017-03-15 20:22hudsonbotNote Added: 0095293

2017-03-12 13:15   
Note: the debugging patch is for dbsm repo so to use it need to recompile dbsm and use that jar
2017-03-13 07:49   
Repository: erp/devel/pi
Changeset: 0dd550bbbef217729547c193505710672401aa65
Author: Stefan Hühner <stefan.huehner <at>>
Date: Mon Mar 13 07:48:48 2017 +0100
URL: [^]

fixed bug 35490: SystemService logs a ERROR line always

M src/org/openbravo/service/system/
2017-03-13 07:51   
2017-03-15 20:22   
A changeset related to this issue has been promoted main and to the
Central Repository, after passing a series of tests.

Promotion changeset: [^]
Maturity status: Test