Openbravo Issue Tracking System - Retail Modules
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0035853Retail ModulesWeb POSpublic2017-04-27 08:422017-06-12 09:14
Reportercaristu 
Assigned Tojorge-garcia 
PriorityhighSeverityminorReproducibilityalways
StatusclosedResolutionfixed 
PlatformOS5OS Version
Product Version 
Target VersionFixed in VersionRR17Q3 
Merge Request Status
Review Assigned Tomarvintm
OBNetwork customer
Support ticket
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary0035853: Use the StaticResourceProvider in mobile to cache static resources
DescriptionIt would be nice if the caching of the static resources, done by MobileStaticResourceComponent class would make use of the API provided by the new StaticResourceProvider class.

This way this logic would be centralized.
Steps To ReproduceIn description
Proposed SolutionMobileStaticResourceComponent should extend core's StaticResourceComponent
Additional Information
TagsNo tags attached.
Relationships
related to feature request 0035852 closed caristu Openbravo ERP Add a mechanism to cache the static resources 
related to defect 0035972 closed jorge-garcia Retail Modules API change approve required fro Static Resources 
Attached Filesdiff 35853.diff (5,050) 2017-05-09 13:06
https://issues.openbravo.com/file_download.php?file_id=10742&type=bug
diff 35853_fix.diff (11,119) 2017-05-11 09:23
https://issues.openbravo.com/file_download.php?file_id=10746&type=bug
Issue History
Date ModifiedUsernameFieldChange
2017-04-27 08:42caristuNew Issue
2017-04-27 08:42caristuAssigned To => Retail
2017-04-27 08:42caristuTriggers an Emergency Pack => No
2017-04-27 08:42caristuRelationship addedrelated to 0035852
2017-04-27 09:04caristuFile Added: MobileStaticResourceComponent.diff
2017-04-27 09:04caristuProposed Solution updated
2017-05-09 10:46caristuFile Deleted: MobileStaticResourceComponent.diff
2017-05-09 10:47caristuProposed Solution updated
2017-05-09 13:06caristuFile Added: 35853.diff
2017-05-09 13:07caristuNote Added: 0096489
2017-05-10 15:02jorge-garciaStatusnew => scheduled
2017-05-10 15:02jorge-garciaAssigned ToRetail => jorge-garcia
2017-05-11 09:21jorge-garciaRelationship addedrelated to 0035972
2017-05-11 09:23jorge-garciaFile Added: 35853_fix.diff
2017-05-11 12:34hgbotCheckin
2017-05-11 12:34hgbotNote Added: 0096525
2017-05-11 12:37hgbotCheckin
2017-05-11 12:37hgbotNote Added: 0096526
2017-05-11 12:37hgbotStatusscheduled => resolved
2017-05-11 12:37hgbotResolutionopen => fixed
2017-05-11 12:37hgbotFixed in SCM revision => http://code.openbravo.com/erp/pmods/org.openbravo.mobile.core/rev/2edc5b3909053d4c94f0f3627d834257fd18f286 [^]
2017-05-11 12:38hgbotCheckin
2017-05-11 12:38hgbotNote Added: 0096527
2017-05-29 17:43hudsonbotCheckin
2017-05-29 17:43hudsonbotNote Added: 0096958
2017-06-12 09:14marvintmReview Assigned To => marvintm
2017-06-12 09:14marvintmStatusresolved => closed
2017-06-12 09:14marvintmFixed in Version => RR17Q3

Notes
(0096489)
caristu   
2017-05-09 13:07   
Attached a file with a patch that implements the support of overriding the result generated by the StaticResourceComponent.
(0096525)
hgbot   
2017-05-11 12:34   
Repository: erp/devel/pi
Changeset: 665cccf00c19fd8bf8fc35957482384ead2de1c9
Author: Carlos Aristu <carlos.aristu <at> openbravo.com>
Date: Thu May 11 12:33:20 2017 +0200
URL: http://code.openbravo.com/erp/devel/pi/rev/665cccf00c19fd8bf8fc35957482384ead2de1c9 [^]

related to issue 35853: ability to override StaticResourceComponent response

---
M modules/org.openbravo.client.kernel/src/org/openbravo/client/kernel/StaticResourceComponent.java
---
(0096526)
hgbot   
2017-05-11 12:37   
Repository: erp/pmods/org.openbravo.mobile.core
Changeset: 2edc5b3909053d4c94f0f3627d834257fd18f286
Author: Jorge Garcia <jorge.garcia <at> openbravo.com>
Date: Wed May 10 13:13:46 2017 +0200
URL: http://code.openbravo.com/erp/pmods/org.openbravo.mobile.core/rev/2edc5b3909053d4c94f0f3627d834257fd18f286 [^]

Fixed issue 35853: Use the StaticResourceProvider in mobile to cache static
resourcesFixed issue

Adapt infrastructure for static resources in mobile applications.

---
M src/org/openbravo/mobile/core/MobileCoreApplicationCacheComponent.java
M src/org/openbravo/mobile/core/MobileStaticResourceComponent.java
M src/org/openbravo/mobile/core/login/MobileCoreLoginHandler.java
M src/org/openbravo/mobile/core/process/JSONProcessSimple.java
---
(0096527)
hgbot   
2017-05-11 12:38   
Repository: tools/automation/pi-mobile
Changeset: 4a83a4ac01e3531c23707a615e3e0cc7c6dea638
Author: Jorge Garcia <jorge.garcia <at> openbravo.com>
Date: Wed May 10 17:11:04 2017 +0200
URL: http://code.openbravo.com/tools/automation/pi-mobile/rev/4a83a4ac01e3531c23707a615e3e0cc7c6dea638 [^]

Related to issue 35853: Use the StaticResourceProvider in mobile to cache static
resourcesFixed issue

Due to these changes, this test doesn't make sense.

---
R src-test/org/openbravo/test/mobile/retail/pack/selenium/tests/system/I27936_SuffixOfGeneratedJavascript.java
---
(0096958)
hudsonbot   
2017-05-29 17:43   
A changeset related to this issue has been promoted main and to the
Central Repository, after passing a series of tests.

Promotion changeset: https://code.openbravo.com/erp/devel/main/rev/1ee70113bdc4 [^]
Maturity status: Test