0045564: Service workers infrastructure precaches CSS files that are then never used

Description

Currently there is code inside the Service workers/AppCacheManifest infrastructure that adds CSS files belonging to external modules automatically to the Manifest as separate requests:

MobileCoreApplicationCacheComponent.java: method getcssFileList()
cache-manifest-ftl <#list data.cssFileList as cssFile>
${cssFile}
</#list>

This causes the Service Worker to request and store those files separately. However, the WebPOS application then never formally requests them, as their content is already present inside the main StyleSheetResources request.

Therefore, we are wasting quite a bit of bandwidth and triggering many unnecessary requests.

Steps To Reproduce

Go to livebuilds and access the modules instance. Check in developer tools and see that many css requests exist separately, coming from the Service Worker.

Then reload the page (F5), and realise that those css files are never actually requested by the application.

Proposed Solution

Delete both the getcssFileList method inside MobileCoreApplicationCacheComponent.java and the corresponding CSS section inside cache-manifest-ftl

Tags

No tags attached.

Attached Files

Relationships [ Relation Graph ] [ Dependency Graph ]

blocks

defect

0045561

closed

ranjith_qualiantech_com

Service workers infrastructure precaches CSS files that are then never used

Notes
(0125050) hgbot (developer) 2020-12-23 04:22	Repository: https://gitlab.com/openbravo/product/pmods/org.openbravo.mobile.core [^] Changeset: 60380ef19618e97934ee52f4778b1eaac44216a7 Author: Ranjith S R <ranjith@qualiantech.com> Date: 2020-12-23T08:52:32+05:30 URL: https://gitlab.com/openbravo/product/pmods/org.openbravo.mobile.core/-/commit/60380ef19618e97934ee52f4778b1eaac44216a7 [^] Fixed ISSUE-45564: Removed CSS Componenet in Cache Component --- M src/org/openbravo/mobile/core/BaseCoreAppCacheResourceProvider.java M src/org/openbravo/mobile/core/MobileCoreApplicationCacheComponent.java M src/org/openbravo/mobile/core/templates/cache-manifest.ftl ---

Issue History
Date Modified	Username	Field	Change
2020-12-14 11:16	shuehner	Type	defect => backport
2020-12-14 11:16	shuehner	Target Version	=> RR20Q4.1
2020-12-23 03:48	ranjith_qualiantech_com	Assigned To	Retail => ranjith_qualiantech_com
2020-12-23 04:22	hgbot	Resolution	open => fixed
2020-12-23 04:22	hgbot	Status	scheduled => resolved
2020-12-23 04:22	hgbot	Fixed in Version	=> RR20Q4.1
2020-12-23 04:22	hgbot	Note Added: 0125050
2020-12-28 07:27	marvintm	Review Assigned To	=> marvintm
2020-12-28 07:27	marvintm	Status	resolved => closed