Fixed bugs and updated version to 1.1.7

Fixed bug in Logger in ServiceWorker context.
Fixed bug with the pouchDBPersistenceStore.
Fixed vary header support.

Author: kentarokinebuchi

JSDOC.md

@@ -1,4 +1,4 @@
-# offline-persistence-toolkit 1.1.6 #
+# offline-persistence-toolkit 1.1.7 #
 
 ## Introduction ##
 

README.md

@@ -1,4 +1,4 @@
-# offline-persistence-toolkit 1.1.6 #
+# offline-persistence-toolkit 1.1.7 #
 
 offline-persistence-toolkit is a client-side JavaScript library that provides caching and offline support at the HTTP request layer. This support is transparent to the user and is done through the Fetch API and an XHR adapter. HTTP requests made while the client device is offline are captured for replay when connection to the server is restored. Additional capabilities include a persistent storage layer, synchronization manager, binary data support and various configuration APIs for customizing the default behavior. This framework can be used in both ServiceWorker and non-ServiceWorker contexts within web and hybrid mobile apps.
 
@@ -58,16 +58,16 @@ If your app uses [RequireJS](http://www.requirejs.org/ "RequireJS"), update the
 ```javascript
   requirejs.config({
     paths: {
-      'persist' : 'js/libs/persist/v1.1.6/min'
+      'persist' : 'js/libs/persist/v1.1.7/min'
 
       // Other path mappings here
  }
 ```
-For Oracle JET apps, also open `appDir/src/js/main-release-paths.json` and add the `'persist' : 'js/libs/persist/v1.1.6/min'` entry to the list of paths.
+For Oracle JET apps, also open `appDir/src/js/main-release-paths.json` and add the `'persist' : 'js/libs/persist/v1.1.7/min'` entry to the list of paths.
 
 You can choose the name of the paths prefix. That is, you can use a different value to the ‘persist’ value shown in the examples.
 
-It is recommended to add the version number as a convention in your application build step such as `'persist' : 'js/libs/persist/v1.1.6/min'`.
+It is recommended to add the version number as a convention in your application build step such as `'persist' : 'js/libs/persist/v1.1.7/min'`.
 
 Versions of the toolkit are also available on CDN under the latest JET release. e.g. 
 
@@ -91,7 +91,7 @@ And again, if you are using RequireJS, you will need to map paths for these pack
     paths: {
       'pouchdb': 'js/libs/pouchdb-6.3.4',
       'pouchfind': 'js/libs/pouchdb.find',
-      'persist' : 'js/libs/persist/v1.1.6/min'
+      'persist' : 'js/libs/persist/v1.1.7/min'
 
       // Other path mappings here
  }

USAGE.md

@@ -1,4 +1,4 @@
-# offline-persistence-toolkit 1.1.6 #
+# offline-persistence-toolkit 1.1.7 #
 
 # Introduction #
 

docs/index.html

@@ -43,7 +43,7 @@ <h3> </h3>
 
 
     <section>
-        <article><h1>offline-persistence-toolkit 1.1.6</h1><h2>Introduction</h2><p>This document provides a reference to the offline-persistence-toolkit's JavaScript API. Links to JSDoc for each class can be found to the right.</p>
+        <article><h1>offline-persistence-toolkit 1.1.7</h1><h2>Introduction</h2><p>This document provides a reference to the offline-persistence-toolkit's JavaScript API. Links to JSDoc for each class can be found to the right.</p>
 <p>The toolkit's <a href="https://github.com/oracle/offline-persistence-toolkit/" title="README">README</a> file provides an overview of the toolkit’s capabilities. Having read that document, you can find more information about how to implement a range of common and advanced use cases with the toolkit in the
 <a href="https://github.com/oracle/offline-persistence-toolkit/blob/master/USAGE.md" title="Usage guide">Usage guide</a>.</p>
 <h2>License</h2><p>Copyright (c) 2017 Oracle and/or its affiliates The Universal Permissive License (UPL), Version 1.0.</p></article>

src/impl/OfflineCache.js

@@ -264,6 +264,7 @@ define(["./defaultCacheHandler", "./logger"], function (cacheHandler, logger) {
       var varyArray = varyValue.split(',');
       for (var index = 0; index < varyArray.length; index++) {
         var varyHeaderName = varyArray[index].toLowerCase();
+        varyHeaderName = varyHeaderName.trim();
         var requestVaryHeaderValue = requestHeaders.get(varyHeaderName);
         var cachedRequestVaryHeaderValue = cacheRequestHeaders[varyHeaderName];
         logger.log("Offline Persistence Toolkit OfflineCache: HTTP Vary header name: " + varyHeaderName);

src/impl/defaultCacheHandler.js

@@ -189,6 +189,7 @@ define(['../persistenceUtils', '../persistenceStoreManager', './logger'],
             var varyFields = varyValue.split(',');
             for (var index = 0; index < varyFields.length; index++) {
               var varyField = varyFields[index];
+              varyField = varyField.trim();
               var varyValue = requestHeaders ? requestHeaders.get(varyField) : 'undefined';
               key += varyField + '=' + varyValue;
             }

src/impl/logger.js

@@ -232,6 +232,8 @@ define([], function () {
       writer = self.option("writer");
     } else if (typeof window !== 'undefined' && window.console !== undefined) {
       writer = window.console;
+    } else if (typeof console !== 'undefined') {
+      writer = console;
     }
     return writer;
   };

src/impl/pouchDBPersistenceStore.js

@@ -182,16 +182,21 @@ define(["../PersistenceStore", "../impl/storageUtils", "pouchdb", "./logger"],
       // get all rows and use our own find logic
       return self._db.allDocs({include_docs: true}).then(function (result) {
         if (result && result.rows && result.rows.length) {
-          var satisfiedRows = result.rows.filter(function(row) {
-            if (storageUtils.satisfy(findExpression.selector, row.doc)) {
-              return true;
-            }
-            return false;
+          var fixedDocPromises = result.rows.map(function(row) {
+            return self._fixValue(row.doc);
           });
-          var promises = satisfiedRows.map(function(row) {
-            return self._findResultCallback(modifiedFind.fields).bind(self)(row.doc);
+          return Promise.all(fixedDocPromises).then(function(docs) {
+            var satisfiedRows = result.rows.filter(function(row) {
+              if (storageUtils.satisfy(findExpression.selector, row.doc)) {
+                return true;
+              }
+              return false;
+            });
+            var promises = satisfiedRows.map(function(row) {
+              return self._findResultCallback(modifiedFind.fields).bind(self)(row.doc);
+            });
+            return Promise.all(promises);
           });
-          return Promise.all(promises);
         } else {
           return Promise.resolve([]);
         }

test/caches/cachestest.js

@@ -185,7 +185,7 @@ define(['persist/persistenceStoreManager', 'persist/localPersistenceStoreFactory
       response: new Response('response body for all departments compressed with gzip random', {
                       headers: new Headers({
                         'Content-Encoding': 'gzip random',
-                        Vary: 'test-accept-encoding'
+                        Vary: 'test-accept-encoding, test-accept-encoding1'
                       })
                     })
     }];

test/storage/storagetest.js

@@ -971,6 +971,20 @@ define(['persist/persistenceStoreManager', 'persist/localPersistenceStoreFactory
           }
           return testStore.find(findExpression);
         }
+      }).then(function (valueFound) {
+        valueFound = valueFound[0].value;
+        if (!valueFound || !valueFound.logo ||
+            blobFromStringSize !== valueFound.logo.size) {
+          assert.ok(false, 'first row cannot be found.');
+          return Promise.reject({message: 'cannot find the inserted value.'});
+        } else {
+          assert.ok(true, 'first row is found.');
+          var findExpression = {
+            selector: {key: {$eq: 'testBinaryKey1'}},
+            fields: ['value.logo']
+          }
+          return testStore.find(findExpression);
+        }
       }).then(function (valueFound) {
         valueFound = valueFound[0].value;
         if (!valueFound || !valueFound.logo ||