Openbravo Issue Tracking System - Openbravo ERP
View Issue Details
0043180Openbravo ERPA. Platformpublic2020-02-10 19:322020-04-06 15:45
shuehner 
alostale 
normalminoralways
closedfixed 
5
 
3.0PR20Q1 
jarmendariz
Core
Production - Confirmed Stable
2018-10-30
3.0PR19Q1
https://gitlab.com/openbravo/product/dbsm/-/commit/789aa22ca2be4190e5f07a4770c2172e0e5c0552 [^]
No
0043180: Update.database message about local changes in application dictionary is missing table name
The update.database mechanism to check for local changes consists of 2 parts:
a.) structure changes
b.) changes in AD

For part b.) the code did show the table having changes also but is no longer doing so -> Regression.

Checking code of code in dbsm-main repo OBDataset.java method hasChanged:
  if (log != null) {
              log.warn("Change detected in table: " + table.getName());

So issue could be either:
- logger being null (even if seems to be set in OBDBSMOBUtil.java)
- logger being not configured to show by default.
1.) Have clean base (i.e. install.source or previous update.database ran)
2.) Cause local change in AD
update ad_window set updated=now() where ad_window_id = '100';
3.) Run update.database

4.) Check output missing info about table
     [java] 18844 [main] INFO org.openbravo.ddlutils.util.DBSMOBUtil - Checking if database structure was modified locally.
     [java] 19425 [main] INFO org.openbravo.ddlutils.util.DBSMOBUtil - Checking if data has changed in the application dictionary.
     [java] 19465 [main] ERROR org.openbravo.ddlutils.task.AlterDatabaseDataAll - Database has local changes. Update.database will not be done. You should export your changed modules before doing update.database, so that your Application Dictionary changes are preserved.
     [java] Database has local changes. Update.database not done.
No tags attached.
blocks defect 0043175 closed alostale Update.database message about local changes in application dictionary is missing table name 
Issue History
2020-02-11 10:39alostaleTypedefect => backport
2020-02-11 10:39alostaleTarget Version => 3.0PR20Q1
2020-02-18 10:13hgbotCheckin
2020-02-18 10:13hgbotNote Added: 0117899
2020-02-18 10:13hgbotStatusscheduled => resolved
2020-02-18 10:13hgbotResolutionopen => fixed
2020-02-18 10:13hgbotFixed in SCM revision => http://code.openbravo.com/erp/backports/3.0PR20Q1/rev/4ccf709168ed8cf1436a197058064ec550b98332 [^]
2020-02-18 10:13alostaleNote Added: 0117900
2020-02-19 09:44jarmendarizNote Added: 0117937
2020-02-19 09:44jarmendarizStatusresolved => closed
2020-04-06 11:53hgbotCheckin
2020-04-06 11:53hgbotNote Added: 0118962
2020-04-06 11:53hgbotStatusclosed => resolved
2020-04-06 11:53hgbotFixed in SCM revisionhttp://code.openbravo.com/erp/backports/3.0PR20Q1/rev/4ccf709168ed8cf1436a197058064ec550b98332 [^] => http://code.openbravo.com/erp/devel/pi/rev/2ecdf5dcd7b3908eaf7c2d88c6d31215c4320e8a [^]
2020-04-06 15:45cbernerNote Added: 0118987
2020-04-06 15:45cbernerStatusresolved => closed

Notes
(0117899)
hgbot   
2020-02-18 10:13   
Repository: erp/backports/3.0PR20Q1
Changeset: 4ccf709168ed8cf1436a197058064ec550b98332
Author: Asier Lostalé <asier.lostale <at> openbravo.com>
Date: Tue Feb 18 10:12:55 2020 +0100
URL: http://code.openbravo.com/erp/backports/3.0PR20Q1/rev/4ccf709168ed8cf1436a197058064ec550b98332 [^]

fixed BUG-43180: log local changes

  When updating database having local changes in AD, the tables with
  changes were not logged.

  While project to upgrade to log4j2 the API, old API passing old logger
  was kept but it become unusable as null logger was used.

---
M src-db/database/lib/dbsourcemanager.jar
---
(0117900)
alostale   
2020-02-18 10:13   
dbsm commit: https://gitlab.com/openbravo/product/dbsm/-/commit/aef1e32c82874f09bb7f4f2b9d31a3ac6965e3ed [^]
(0117937)
jarmendariz   
2020-02-19 09:44   
Reviewed
(0118962)
hgbot   
2020-04-06 11:53   
Repository: erp/devel/pi
Changeset: 2ecdf5dcd7b3908eaf7c2d88c6d31215c4320e8a
Author: Asier Lostalé <asier.lostale <at> openbravo.com>
Date: Tue Feb 18 10:12:55 2020 +0100
URL: http://code.openbravo.com/erp/devel/pi/rev/2ecdf5dcd7b3908eaf7c2d88c6d31215c4320e8a [^]

fixed BUG-43180: log local changes

  When updating database having local changes in AD, the tables with
  changes were not logged.

  While project to upgrade to log4j2 the API, old API passing old logger
  was kept but it become unusable as null logger was used.

---
M src-db/database/lib/dbsourcemanager.jar
---
(0118987)
cberner   
2020-04-06 15:45   
closed: wrongly reopened