Openbravo Issue Tracking System - Openbravo Localizations
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0049318Openbravo LocalizationsLocalization Portugalpublic2022-05-13 12:252022-06-01 10:43
Reporterucarrion 
Assigned Toigor_trebol 
PriorityhighSeveritymajorReproducibilityalways
StatusclosedResolutionfixed 
PlatformOS5OS Version
Product Version 
Target VersionFixed in Version 
Merge Request Statusapproved
Regression date
Regression introduced by commit
Regression level
Support ticket
OBNetwork customerGold
Regression introduced in release
Summary0049318: EWI generated because of ad_user organizational level
DescriptionIf I do a return in a store B of a sale done in store A with a user related to organization store B, I get the EWI below:

org.openbravo.base.exception.OBException: Couldn't enable triggers:
...
Caused by: org.openbravo.base.exception.OBSecurityException: Entity NC XXXXXXXX/XXXXXX- 12-05-2022 - -69.95 (Invoice) with organization STORE B references an entity USER B through its property Invoice.saftStatusupdatedby but this referenced entity belongs to an organization STORE A which is not part of the natural tree of STORE B.

Attached, the complete EWI log for the case raised in Norauto.
Steps To Reproduce-Precondition: to be able to use touchpoints from two different PT-configured stores.

-Do a simplified invoice (simple sale) into the store A
-Open the touchpoint in store B using a USER whose organization is STORE B (not higher as legal entity, * ...)
-Do a return in store B of the invoice generated in step one
-Check the 'Errors While Importing POS data' window in the backoffice
Proposed Solution
Additional Information
TagsNOR
Relationships
Attached Filestxt 49318_errorLog.txt (3,508) 2022-05-13 12:26
https://issues.openbravo.com/file_download.php?file_id=17021&type=bug
Issue History
Date ModifiedUsernameFieldChange
2022-05-13 12:25ucarrionNew Issue
2022-05-13 12:25ucarrionAssigned To => psanjuan
2022-05-13 12:25ucarrionOBNetwork customer => Gold
2022-05-13 12:25ucarrionTag Attached: NOR
2022-05-13 12:26ucarrionFile Added: 49318_errorLog.txt
2022-05-30 13:27aferrazAssigned Topsanjuan => igor_trebol
2022-06-01 08:56hgbotMerge Request Status => open
2022-06-01 08:56hgbotNote Added: 0137911
2022-06-01 09:02hgbotNote Added: 0137912
2022-06-01 09:52hgbotNote Added: 0137916
2022-06-01 09:52hgbotNote Added: 0137917
2022-06-01 10:11hgbotNote Added: 0137918
2022-06-01 10:12hgbotNote Added: 0137919
2022-06-01 10:26hgbotNote Added: 0137920
2022-06-01 10:34hgbotNote Added: 0137922
2022-06-01 10:40hgbotMerge Request Statusopen => approved
2022-06-01 10:40hgbotNote Added: 0137923
2022-06-01 10:40hgbotResolutionopen => fixed
2022-06-01 10:40hgbotStatusnew => closed
2022-06-01 10:40hgbotNote Added: 0137924
2022-06-01 10:40hgbotNote Added: 0137925
2022-06-01 10:40hgbotNote Added: 0137926
2022-06-01 10:41aferrazNote Deleted: 0137911
2022-06-01 10:41aferrazNote Deleted: 0137917
2022-06-01 10:41aferrazNote Deleted: 0137912
2022-06-01 10:41aferrazNote Deleted: 0137916
2022-06-01 10:41aferrazNote Deleted: 0137920
2022-06-01 10:41aferrazNote Deleted: 0137918
2022-06-01 10:43hgbotNote Added: 0137927
2022-06-01 10:43hgbotNote Added: 0137928

Notes
(0137919)
hgbot   
2022-06-01 10:12   
Merge Request created: https://gitlab.com/openbravo/product/pmods/com.practicsbs.saftpt/-/merge_requests/34 [^]
(0137922)
hgbot   
2022-06-01 10:34   
Merge Request created: https://gitlab.com/openbravo/product/pmods/com.practicsbs.saftpt.retail/-/merge_requests/49 [^]
(0137923)
hgbot   
2022-06-01 10:40   
Merge request merged: https://gitlab.com/openbravo/product/pmods/com.practicsbs.saftpt/-/merge_requests/34 [^]
(0137924)
hgbot   
2022-06-01 10:40   
Directly closing issue as related merge request is already approved.

Repository: https://gitlab.com/openbravo/product/pmods/com.practicsbs.saftpt [^]
Changeset: 389856b26ed8839ea41ddb84784facf0545fd679
Author: Igor Trebol <igor.trebol@openbravo.com>
Date: 01-06-2022 10:13:31
URL: https://gitlab.com/openbravo/product/pmods/com.practicsbs.saftpt/-/commit/389856b26ed8839ea41ddb84784facf0545fd679 [^]

Fixes ISSUE-49318: set setSaftStatusupdatedby so it allows cross store returns

---
M src-db/database/sourcedata/AD_COLUMN.xml
---
(0137925)
hgbot   
2022-06-01 10:40   
Directly closing issue as related merge request is already approved.

Repository: https://gitlab.com/openbravo/product/pmods/com.practicsbs.saftpt.retail [^]
Changeset: 9072d6c5237ae220ebcced917a5d0cdb7b308110
Author: Igor Trebol <igor.trebol@openbravo.com>
Date: 01-06-2022 10:34:24
URL: https://gitlab.com/openbravo/product/pmods/com.practicsbs.saftpt.retail/-/commit/9072d6c5237ae220ebcced917a5d0cdb7b308110 [^]

Fixes ISSUE-49318: set setSaftStatusupdatedby so it allows cross store returns

---
M src/com/practicsbs/saftpt/retail/hook/SaftrOrderLoaderHook.java
---
(0137926)
hgbot   
2022-06-01 10:40   
Merge request merged: https://gitlab.com/openbravo/product/pmods/com.practicsbs.saftpt.retail/-/merge_requests/49 [^]
(0137927)
hgbot   
2022-06-01 10:43   
Directly closing issue as related merge request is already approved.

Repository: https://gitlab.com/openbravo/product/pmods/com.practicsbs.saftpt.retail [^]
Changeset: 5fea17dd316bc67988852277565488255d0ac115
Author: Igor Trebol <igor.trebol@openbravo.com>
Date: 01-06-2022 08:43:16
URL: https://gitlab.com/openbravo/product/pmods/com.practicsbs.saftpt.retail/-/commit/5fea17dd316bc67988852277565488255d0ac115 [^]

Fixes ISSUE-49318: set setSaftStatusupdatedby so it allows cross store returns


(cherry picked from commit 9072d6c5237ae220ebcced917a5d0cdb7b308110)
---
M src/com/practicsbs/saftpt/retail/hook/SaftrOrderLoaderHook.java
---
(0137928)
hgbot   
2022-06-01 10:43   
Directly closing issue as related merge request is already approved.

Repository: https://gitlab.com/openbravo/product/pmods/com.practicsbs.saftpt [^]
Changeset: 206da6a281adfb79f9ba39c8420db7792727072c
Author: Igor Trebol <igor.trebol@openbravo.com>
Date: 01-06-2022 08:43:48
URL: https://gitlab.com/openbravo/product/pmods/com.practicsbs.saftpt/-/commit/206da6a281adfb79f9ba39c8420db7792727072c [^]

Fixes ISSUE-49318: set setSaftStatusupdatedby so it allows cross store returns


(cherry picked from commit 389856b26ed8839ea41ddb84784facf0545fd679)
---
M src-db/database/sourcedata/AD_COLUMN.xml
---