Openbravo Issue Tracking System - Retail Modules
View Issue Details
0045224Retail ModulesWeb POSpublic2020-10-14 13:272020-10-20 08:43
jarmendariz 
jarmendariz 
urgentminorhave not tried
closedfixed 
5
 
RR21Q1 
No
0045224: Remove SynchronizationBuffer listeners from Business Logic js files
SyncIconManagement is a SynchronizationBuffer listener that updates the sync icon in the WebPOS. This logic is not needed in newer applications, so it makes no sense to export this file. Same should be applied to other listeners.
See above
Move SyncIconManagement.js, CheckSourcesVersionListener.js and SyncPollingListener.js out of app/ folder and update the Component Provider.
No tags attached.
Issue History
2020-10-14 13:27jarmendarizNew Issue
2020-10-14 13:27jarmendarizAssigned To => jarmendariz
2020-10-14 13:27jarmendarizTriggers an Emergency Pack => No
2020-10-14 13:28jarmendarizStatusnew => acknowledged
2020-10-14 13:28jarmendarizStatusacknowledged => scheduled
2020-10-14 13:32hgbotNote Added: 0123679
2020-10-16 13:59hgbotResolutionopen => fixed
2020-10-16 13:59hgbotStatusscheduled => closed
2020-10-16 13:59hgbotNote Added: 0123747
2020-10-16 13:59hgbotFixed in Version => RR21Q1
2020-10-16 13:59hgbotNote Added: 0123748
2020-10-20 08:32jarmendarizNote Added: 0123788
2020-10-20 08:32jarmendarizStatusclosed => new
2020-10-20 08:32jarmendarizResolutionfixed => open
2020-10-20 08:32jarmendarizFixed in VersionRR21Q1 =>
2020-10-20 08:32jarmendarizStatusnew => scheduled
2020-10-20 08:34jarmendarizSummaryRemove SyncIconManagement file from Business Logic js files => Remove SynchronizationBuffer listeners from Business Logic js files
2020-10-20 08:34jarmendarizDescription Updatedbug_revision_view_page.php?rev_id=21808#r21808
2020-10-20 08:34jarmendarizProposed Solution updated
2020-10-20 08:41hgbotNote Added: 0123789
2020-10-20 08:43hgbotResolutionopen => fixed
2020-10-20 08:43hgbotStatusscheduled => closed
2020-10-20 08:43hgbotNote Added: 0123790
2020-10-20 08:43hgbotFixed in Version => RR21Q1
2020-10-20 08:43hgbotNote Added: 0123791

Notes
(0123679)
hgbot   
2020-10-14 13:32   
Merge Request created: https://gitlab.com/openbravo/product/pmods/org.openbravo.mobile.core/-/merge_requests/83 [^]
(0123747)
hgbot   
2020-10-16 13:59   
Merge request merged: https://gitlab.com/openbravo/product/pmods/org.openbravo.mobile.core/-/merge_requests/83 [^]
(0123748)
hgbot   
2020-10-16 13:59   
Directly closing issue as related merge request is already approved.

Repository: https://gitlab.com/openbravo/product/pmods/org.openbravo.mobile.core [^]
Changeset: 7eef8125e1f3e27d29cc513462aacada7b1814af
Author: Javier Armendáriz <javier.armendariz@openbravo.com>
Date: 2020-10-16T00:22:26+02:00
URL: https://gitlab.com/openbravo/product/pmods/org.openbravo.mobile.core/-/commit/7eef8125e1f3e27d29cc513462aacada7b1814af [^]

Fixed ISSUE-45224: Moved SyncIconManagement file out of new business logic files

---
A web/org.openbravo.mobile.core/source/utils/SyncIconManagement.js
M src/org/openbravo/mobile/core/MobileCoreComponentProvider.java
---
(0123788)
jarmendariz   
2020-10-20 08:32   
SynchronizationBuffer has other listeners (polling, sources version) that should be moved away from BL as well
(0123789)
hgbot   
2020-10-20 08:41   
Merge Request created: https://gitlab.com/openbravo/product/pmods/org.openbravo.mobile.core/-/merge_requests/87 [^]
(0123790)
hgbot   
2020-10-20 08:43   
Merge request merged: https://gitlab.com/openbravo/product/pmods/org.openbravo.mobile.core/-/merge_requests/87 [^]
(0123791)
hgbot   
2020-10-20 08:43   
Directly closing issue as related merge request is already approved.

Repository: https://gitlab.com/openbravo/product/pmods/org.openbravo.mobile.core [^]
Changeset: c95a5a92b6cf4766e049ee5c01fb10974ad0e15f
Author: Javier Armendáriz <javier.armendariz@openbravo.com>
Date: 2020-10-20T08:39:36+02:00
URL: https://gitlab.com/openbravo/product/pmods/org.openbravo.mobile.core/-/commit/c95a5a92b6cf4766e049ee5c01fb10974ad0e15f [^]

Fixed ISSUE-45224: Removed all remaining SyncBuffer listeners from business logic

SynchronizationBuffer listeners (polling, source version) contains logic only applicable in the old POS application,
so it should not be in the business logic files.

---
A web/org.openbravo.mobile.core/source/utils/CheckSourcesVersionListener.js
A web/org.openbravo.mobile.core/source/utils/SyncPollingListener.js
M src/org/openbravo/mobile/core/MobileCoreComponentProvider.java
---