Quantcast
Channel: Bugs Latest Topics
Viewing all articles
Browse latest Browse all 760

Insufficient check for .manifest file version

$
0
0

I'm currently working on a project that has both a live and development version, utilizing offline storage and .manifest files for all of my Scene objects. Everything works as expected until I run the old project after having just run the new one in Chrome, specifically. The latest versions of the .scene files are always loaded from the cache, despite the older version number in their .manifest files. I did some digging, and found this:

Database.prototype._loadVersionFromDBAsync = function (url, callback, updateInDBCallback) {
...
    transaction.oncomplete = function (event) {
        if (version) {
        // If the version in the JSON file is > than the version in DB
            if (_this.manifestVersionFound > version.data) {
                _this.mustUpdateRessources = true;

Notice that the version number comparison operator is >, not !=. Shouldn't the comparison be != to cover any changes to the .manifest file version, regardless of its age? I've changed this on my end, and all is well now.


Viewing all articles
Browse latest Browse all 760

Trending Articles



<script src="https://jsc.adskeeper.com/r/s/rssing.com.1596347.js" async> </script>