Skip to content
This repository has been archived by the owner on Nov 3, 2021. It is now read-only.

Commit

Permalink
Merge pull request #13515 from acperez/bug-936028
Browse files Browse the repository at this point in the history
Bug 936028 - Remove appcache in hosted single variant apps to avoid wrong installation without internet connection. r=yurenju
  • Loading branch information
Albert committed Nov 19, 2013
2 parents fe38a91 + f8a48be commit 82bc089
Showing 1 changed file with 10 additions and 0 deletions.
10 changes: 10 additions & 0 deletions build/variant.js
Expand Up @@ -98,6 +98,11 @@ function prepareHostedApp(manifestUrl, manifest, appPath, appId, origin,
'origin': origin || getOrigin(manifest['launch_path'])
};

// Bug 936028 - appcache needs to be removed from manifest
if (manifest.appcache_path) {
delete manifest.appcache_path;
}

// write manifest
utils.writeContentToFile(utils.joinPath(appPath, 'manifest.webapp'),
JSON.stringify(manifest));
Expand Down Expand Up @@ -171,6 +176,11 @@ function processApp(appId, manifestUrl, origin, installOrigin, profilePath,
var fileJSON = utils.readJSONFromPath(utils.joinPath(appPath,
manifestFile));

// Bug 936028 - Comparation has to be done removing appcache
if (json.appcache_path) {
delete json.appcache_path;
}

if (JSON.stringify(json) === JSON.stringify(fileJSON)) {
copyAppPathToProfile();
return;
Expand Down

0 comments on commit 82bc089

Please sign in to comment.