Openbravo Issue Tracking System - Modules
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0040674ModulesExternal Integration Infrastructurepublic2019-04-24 17:092019-05-10 15:14
ReporterAugustoMauch 
Assigned ToAugustoMauch 
PrioritynormalSeverityminorReproducibilityhave not tried
StatusclosedResolutionfixed 
PlatformOS5OS Version
Product Version 
Target VersionFixed in Version 
Merge Request Status
Regression date
Regression introduced by commit
Regression level
Review Assigned Tocaristu
Support ticket
OBNetwork customer
Regression introduced in release
Summary0040674: Include context organization in SBO, so that it can be recovered during mapping and export steps
DescriptionWhen an integration process is executed, the organization of the Process Request will be set in the context. When an EDL Request Line is executed, the original context organization is lost.

We have been requested to include the original context organization in the SBO object, so that it is available when the SBO is exported.

An API like this would be useful:

sbo.getContextOrganizationId()
Steps To Reproduce-
Proposed Solution
Additional Information
TagsNo tags attached.
Relationships
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2019-04-24 17:09AugustoMauchNew Issue
2019-04-24 17:09AugustoMauchAssigned To => AugustoMauch
2019-04-24 17:11AugustoMauchDescription Updatedbug_revision_view_page.php?rev_id=18667#r18667
2019-04-24 17:14hgbotCheckin
2019-04-24 17:14hgbotNote Added: 0111290
2019-04-24 17:14hgbotStatusnew => resolved
2019-04-24 17:14hgbotResolutionopen => fixed
2019-04-24 17:14hgbotFixed in SCM revision => http://code.openbravo.com/erp/pmods/org.openbravo.service.external.integration/rev/e606b55b6bc966747bccb7fc5819e27aa1b8a418 [^]
2019-04-24 17:15AugustoMauchReview Assigned To => caristu
2019-05-06 09:27hgbotCheckin
2019-05-06 09:27hgbotNote Added: 0111487
2019-05-09 15:37hgbotCheckin
2019-05-09 15:37hgbotNote Added: 0111603
2019-05-09 15:37hgbotFixed in SCM revisionhttp://code.openbravo.com/erp/pmods/org.openbravo.service.external.integration/rev/e606b55b6bc966747bccb7fc5819e27aa1b8a418 [^] => http://code.openbravo.com/erp/pmods/org.openbravo.service.external.integration/rev/688a076854912b14f7f8a7274ac88cd86b97aaf4 [^]
2019-05-10 13:47hgbotCheckin
2019-05-10 13:47hgbotNote Added: 0111623
2019-05-10 15:14caristuNote Added: 0111624
2019-05-10 15:14caristuStatusresolved => closed
2019-05-10 15:14caristuCategorySAP Connector => External Integration Infrastructure

Notes
(0111290)
hgbot   
2019-04-24 17:14   
Repository: erp/pmods/org.openbravo.service.external.integration
Changeset: e606b55b6bc966747bccb7fc5819e27aa1b8a418
Author: Augusto Mauch <augusto.mauch <at> openbravo.com>
Date: Fri Apr 05 12:45:51 2019 +0200
URL: http://code.openbravo.com/erp/pmods/org.openbravo.service.external.integration/rev/e606b55b6bc966747bccb7fc5819e27aa1b8a418 [^]

Fixes issue 40674: Original context organization is now included in SBO exporter

The original context organization is now available in the SynchronizableBusinessObjectExporter class,
via the getContextOrganizationId method. It is also available in the SynchronizableBusinessObjectImporter class.

The new method is called getContextOrganizationId()

---
M src/org/openbravo/service/external/integration/edl/BaseExportItemProcessor.java
M src/org/openbravo/service/external/integration/edl/BaseImportItemProcessor.java
M src/org/openbravo/service/external/integration/edl/EdlConstants.java
M src/org/openbravo/service/external/integration/export/SynchronizableBusinessObjectExporter.java
M src/org/openbravo/service/external/integration/importer/SynchronizableBusinessObjectImporter.java
M src/org/openbravo/service/external/integration/process/EdlRequestBuilder.java
---
(0111487)
hgbot   
2019-05-06 09:27   
Repository: erp/pmods/org.openbravo.service.external.integration
Changeset: 2d7ea460e728fc46519e811e23906eec00e1b055
Author: Carlos Aristu <carlos.aristu <at> openbravo.com>
Date: Mon May 06 09:24:35 2019 +0200
URL: http://code.openbravo.com/erp/pmods/org.openbravo.service.external.integration/rev/2d7ea460e728fc46519e811e23906eec00e1b055 [^]

related to issue 40674: minor code improvements

  - Can set the context organization id for both the sbo and sboimporter at the same point
  - Do not need to use "this"

---
M src/org/openbravo/service/external/integration/edl/BaseImportItemProcessor.java
M src/org/openbravo/service/external/integration/export/SynchronizableBusinessObjectExporter.java
---
(0111603)
hgbot   
2019-05-09 15:37   
Repository: erp/pmods/org.openbravo.service.external.integration
Changeset: 688a076854912b14f7f8a7274ac88cd86b97aaf4
Author: Augusto Mauch <augusto.mauch <at> openbravo.com>
Date: Thu May 09 15:36:58 2019 +0200
URL: http://code.openbravo.com/erp/pmods/org.openbravo.service.external.integration/rev/688a076854912b14f7f8a7274ac88cd86b97aaf4 [^]

Fixes issue 40674: getContextOrganizationId method does not need to be public

The purpose of this feature request was to make available the original context organization in the
instances of SynchronizableBusinessObject. There is no need to expose the contextOrganizationId in
the SynchronizableBusinessObjectExporter or in the SynchronizableBusinessObjectImporter.

---
M src/org/openbravo/service/external/integration/edl/BaseImportItemProcessor.java
M src/org/openbravo/service/external/integration/export/SynchronizableBusinessObjectExporter.java
M src/org/openbravo/service/external/integration/importer/SynchronizableBusinessObjectImporter.java
---
(0111623)
hgbot   
2019-05-10 13:47   
Repository: erp/pmods/org.openbravo.service.external.integration
Changeset: 6ab1e42dcd0db6c9a1fad09dc49d6ef9caf0860a
Author: Augusto Mauch <augusto.mauch <at> openbravo.com>
Date: Fri May 10 13:47:08 2019 +0200
URL: http://code.openbravo.com/erp/pmods/org.openbravo.service.external.integration/rev/6ab1e42dcd0db6c9a1fad09dc49d6ef9caf0860a [^]

Related with issue 40674: Removes uneeded static import

---
M src/org/openbravo/service/external/integration/edl/BaseImportItemProcessor.java
---
(0111624)
caristu   
2019-05-10 15:14   
Reviewed