Anonymous | Login
Project:
RSS
  
News | My View | View Issues | Roadmap | Summary

View Issue DetailsJump to Notes ] Issue History ] Print ]
ID
0049791
TypeCategorySeverityReproducibilityDate SubmittedLast Update
defect[Retail Modules] Retail APImajoralways2022-07-12 16:092022-09-29 18:29
ReporteraxelmercadoView Statuspublic 
Assigned ToAugustoMauch 
PriorityhighResolutionfixedFixed in Version
StatusclosedFix in branchFixed in SCM revision
ProjectionnoneETAnoneTarget VersionRR21Q1.1
OSAnyDatabaseAnyJava version
OS VersionDatabase versionAnt version
Product VersionRR21Q1.1SCM revision 
Review Assigned To
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary

0049791: updates through ImportService are not reflected as audit information

DescriptionWhen updating a product through the ImportService/PriceList API, the product changes correctly but the "Updated" field is not changed, so the changes are not reflected in the POS, which generates a problem when updating prices in the stores.
Steps To ReproduceVideo with the steps: https://www.awesomescreenshot.com/video/9924375?key=4daaa332e4d204c2381395db3208dca2 [^]

Attach JSON.
TagsNo tags attached.
Attached Files? file icon product.json [^] (690 bytes) 2022-07-12 16:09

- Relationships Relation Graph ] Dependency Graph ]
related to feature request 0039404 closedAugustoMauch Modules Allow via preference updating the "updated" audit column when importing records 
has duplicate defect 0049846 closedradhakrishnan Retail Modules API issue - When updating the price through an API, the audit fields are not updated 

-  Notes
(0139349)
alostale (manager)
2022-07-15 11:16

This is controlled with the global "Update 'updated' audit field on import" preference that can be set to "Y" at System level (tomcat restart is required afterward).

I'm internally raising the topic to understand why we have this preference at all and in which cases it might make sense not to update audit info.
(0140986)
hgbot (developer)
2022-09-12 00:45

Merge Request created: https://gitlab.com/openbravo/product/pmods/org.openbravo.service.external.integration/-/merge_requests/26 [^]
(0141518)
hgbot (developer)
2022-09-29 18:28

Merge request merged: https://gitlab.com/openbravo/product/pmods/org.openbravo.service.external.integration/-/merge_requests/26 [^]
(0141519)
hgbot (developer)
2022-09-29 18:28

Directly closing issue as related merge request is already approved.

Repository: https://gitlab.com/openbravo/product/pmods/org.openbravo.service.external.integration [^]
Changeset: 40e944d61ea2e3a75c4a80dcf8c2b43a893a6ddd
Author: Augusto Mauch <augusto.mauch@openbravo.com>
Date: 29-09-2022 18:15:11
URL: https://gitlab.com/openbravo/product/pmods/org.openbravo.service.external.integration/-/commit/40e944d61ea2e3a75c4a80dcf8c2b43a893a6ddd [^]

Fixes ISSUE-49791: Update audit fields by default, allow to disable it at entity mapping devel

We were not updating the "updated" audit field by default, and we were only allowing it to disable it at global preference level,
so either it was disabled for all entities or it was for none. Now that preferencec has been disabled, and we use now a new field
at entity mapping level to determine if the audit field should be updated.

That field is excluded from the AD dataset, because different clients might have different needs

---
A src-db/database/sourcedata/AD_DATASET_COLUMN.xml
M src-db/database/model/tables/OBEI_ENTITY_MAPPING.xml
M src-db/database/sourcedata/AD_COLUMN.xml
M src-db/database/sourcedata/AD_ELEMENT.xml
M src-db/database/sourcedata/AD_FIELD.xml
M src-db/database/sourcedata/AD_REF_LIST.xml
M src/org/openbravo/service/external/integration/edl/BaseImportItemProcessor.java
---
(0141520)
caristu (manager)
2022-09-29 18:29

Updated documentation: http://wiki.openbravo.com/wiki/Modules:External_Integration_Infrastructure/UserGuide#Entity_Mappings [^]

- Issue History
Date Modified Username Field Change
2022-07-12 16:09 axelmercado New Issue
2022-07-12 16:09 axelmercado Assigned To => Triage Platform Conn
2022-07-12 16:09 axelmercado File Added: product.json
2022-07-12 16:09 axelmercado Triggers an Emergency Pack => No
2022-07-12 16:17 Practics Issue Monitored: Practics
2022-07-15 11:12 alostale Summary Error ImportService/Pricelist => updates through ImportService are not reflected as audit information
2022-07-15 11:12 alostale Relationship added related to 0039404
2022-07-15 11:16 alostale Note Added: 0139349
2022-07-15 11:21 alostale Status new => acknowledged
2022-08-03 08:03 alostale Relationship added has duplicate 0049846
2022-08-11 08:35 adrianromero Assigned To Triage Platform Conn => caristu
2022-08-19 16:01 jose_duque Issue cloned 0050016
2022-08-26 13:25 AugustoMauch Assigned To caristu => AugustoMauch
2022-09-12 00:45 hgbot Note Added: 0140986
2022-09-29 18:28 hgbot Note Added: 0141518
2022-09-29 18:28 hgbot Resolution open => fixed
2022-09-29 18:28 hgbot Status acknowledged => closed
2022-09-29 18:28 hgbot Note Added: 0141519
2022-09-29 18:29 caristu Note Added: 0141520


Copyright © 2000 - 2009 MantisBT Group
Powered by Mantis Bugtracker