Openbravo Issue Tracking System - POS2
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0050318POS2Corepublic2022-09-23 09:252022-10-04 12:01
Reporteraferraz 
Assigned Toaferraz 
PrioritynormalSeveritymajorReproducibilityhave not tried
StatusclosedResolutionfixed 
PlatformOS5OS Version
Product Version 
Target VersionFixed in Version23Q1 
Merge Request Statusapproved
Review Assigned To
OBNetwork customerNo
Support ticket
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary0050318: Local terminal information is overriding the terminal information loaded from backoffice during the login process
DescriptionLocal terminal information is overriding the terminal information loaded from backoffice during the login process.

It should be the other way around as the information loaded from backoffice during the login process is more updated and should have more prevalence.
Steps To Reproduce- Implement a TerminalProperty extension and load any boolean terminal property.
- Create a java class implementing LoginDataProvider interface and load the same terminal property.

- Enable the terminal property in the backoffice and do a refresh masterdata from ReactPOS. Check the terminal property value is true -> this is correct.
- Disable the terminal property in the backoffice and do a login from ReactPOS without refreshing masterdata. Check the terminal property value is true -> this is not correct, it should be false.
Proposed Solution
Additional Information
TagsNo tags attached.
Relationships
blocks feature request 0050230 closed aferraz Retail Modules Implement open/close store process in ReactPOS 
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2022-09-23 09:25aferrazNew Issue
2022-09-23 09:25aferrazAssigned To => aferraz
2022-09-23 09:25aferrazOBNetwork customer => No
2022-09-23 09:25aferrazTriggers an Emergency Pack => No
2022-09-23 09:30hgbotMerge Request Status => open
2022-09-23 09:30hgbotNote Added: 0141319
2022-10-04 12:01hgbotMerge Request Statusopen => approved
2022-10-04 12:01hgbotResolutionopen => fixed
2022-10-04 12:01hgbotStatusnew => closed
2022-10-04 12:01hgbotNote Added: 0141629
2022-10-04 12:01hgbotFixed in Version => 23Q1
2022-10-04 12:01hgbotNote Added: 0141630
2022-10-21 12:56aferrazRelationship addedblocks 0050230

Notes
(0141319)
hgbot   
2022-09-23 09:30   
Merge Request created: https://gitlab.com/openbravo/product/pmods/org.openbravo.core2/-/merge_requests/853 [^]
(0141629)
hgbot   
2022-10-04 12:01   
Merge request merged: https://gitlab.com/openbravo/product/pmods/org.openbravo.core2/-/merge_requests/853 [^]
(0141630)
hgbot   
2022-10-04 12:01   
Directly closing issue as related merge request is already approved.

Repository: https://gitlab.com/openbravo/product/pmods/org.openbravo.core2 [^]
Changeset: 870d1f1dc04f80d0ed733249ba6ad9a91fa33366
Author: Álvaro Ferraz <alvaro.ferraz@openbravo.com>
Date: 04-10-2022 09:49:26
URL: https://gitlab.com/openbravo/product/pmods/org.openbravo.core2/-/commit/870d1f1dc04f80d0ed733249ba6ad9a91fa33366 [^]

Fixes ISSUE-50318: Local terminal information should not override the terminal information loaded from backoffice during the login process

---
M web-jspack/org.openbravo.core2/src/core/terminal/Terminal.js
M web-jspack/org.openbravo.core2/src/core/terminal/__test__/Terminal.test.js
---