Openbravo Issue Tracking System - Retail Modules
View Issue Details
0021544Retail ModulesWeb POSpublic2012-09-04 12:472012-12-21 13:30
guilleaer 
guilleaer 
normalminorhave not tried
closedfixed 
5
 
RMP18 
No
0021544: duplicated js resources
Some resources are duplicated because they are included in "UID.js" and in separated files.

This files are loaded using script tags in index.html

js/components/terminal.js
js/model/terminal.js
js/components/commonbuttons.js
js/components/clock.js

And also included using component provider
Using web developers tools see that the model Terminal appears in a separated file and also in UID.js
Remove them from component provider.
No tags attached.
related to defect 0021453 closed guilleaer js resources are loaded, but they are still present in the webpage 
diff diffCompProvDuplicated.diff (1,297) 2012-12-13 09:45
https://issues.openbravo.com/file_download.php?file_id=5828&type=bug
Issue History
2012-09-04 12:47guilleaerNew Issue
2012-09-04 12:47guilleaerAssigned To => adrianromero
2012-09-04 12:54guilleaerFile Added: diffDuplicatedJs.diff
2012-09-13 17:25adrianromeroRelationship addedrelated to 0021453
2012-11-09 13:19alostaleStatusnew => scheduled
2012-11-09 13:19alostaleAssigned Toadrianromero => guilleaer
2012-11-09 13:19alostalefix_in_branch => pi
2012-12-13 09:45guilleaerFile Added: diffCompProvDuplicated.diff
2012-12-13 09:48guilleaerNote Added: 0054907
2012-12-13 10:33guilleaerFile Deleted: diffDuplicatedJs.diff
2012-12-14 13:28guilleaerNote Added: 0054947
2012-12-17 11:55hgbotCheckin
2012-12-17 11:55hgbotNote Added: 0054972
2012-12-17 11:55hgbotStatusscheduled => resolved
2012-12-17 11:55hgbotResolutionopen => fixed
2012-12-17 11:55hgbotFixed in SCM revision => http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/7a28b7fc102368747337b299132ad3516b1b6bc6 [^]
2012-12-18 13:03hgbotCheckin
2012-12-18 13:03hgbotNote Added: 0055050
2012-12-21 13:30marvintmStatusresolved => closed
2012-12-21 13:30marvintmFixed in Version => RMP18

Notes
(0054907)
guilleaer   
2012-12-13 09:48   
I've attached a file removing from the component provider the files which are duplicated.

I will test that all works without issues and then it will be pushed to be included in RMP18
(0054947)
guilleaer   
2012-12-14 13:28   
This script loader also can be removed:

model/terminal.js -> line 132

- $LAB.script('js/i18n.js').wait(function () {
+// $LAB.script('js/i18n.js').wait(function () {
.
.
.
- });
+// });
(0054972)
hgbot   
2012-12-17 11:55   
Repository: erp/pmods/org.openbravo.retail.posterminal
Changeset: 7a28b7fc102368747337b299132ad3516b1b6bc6
Author: Guillermo Alvarez de Eulate <guillermo.alvarez <at> openbravo.com>
Date: Mon Dec 17 11:54:40 2012 +0100
URL: http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/7a28b7fc102368747337b299132ad3516b1b6bc6 [^]

fixed issue 21544 -> remove duplicated js files

---
M src/org/openbravo/retail/posterminal/OBPOSComponentProvider.java
M web/org.openbravo.retail.posterminal/js/model/terminal.js
---
(0055050)
hgbot   
2012-12-18 13:03   
Repository: erp/pmods/org.openbravo.retail.posterminal
Changeset: d864c2f6898fd1e8ba1a24772134d3c294020f21
Author: Guillermo Alvarez de Eulate <guillermo.alvarez <at> openbravo.com>
Date: Tue Dec 18 13:02:35 2012 +0100
URL: http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/d864c2f6898fd1e8ba1a24772134d3c294020f21 [^]

related to issue 21544: Because labels are loaded after commonbuttons, getLabel actions must be done into initComponents

---
M web/org.openbravo.retail.posterminal/js/components/commonbuttons.js
---