feat(storage): rename set() into put()

The new api (put, get) is capable with AJS' $cacheFactory.

BREAKING CHANGE: This marks storage.set() as deprecated. In the
next major release v3, the old method `set()` will be dropped in favor
of `put()`.

Relates #772

src/service/storage-cookie.js

@@ -37,11 +37,28 @@ angular.module('pascalprecht.translate')
      * @description
      * Sets an item in cookieStorage by given name.
      *
+     * @deprecated use #put
+     *
      * @param {string} name Item name
      * @param {string} value Item value
      */
     set: function (name, value) {
       $cookieStore.put(name, value);
+    },
+
+    /**
+     * @ngdoc function
+     * @name pascalprecht.translate.$translateCookieStorage#put
+     * @methodOf pascalprecht.translate.$translateCookieStorage
+     *
+     * @description
+     * Sets an item in cookieStorage by given name.
+     *
+     * @param {string} name Item name
+     * @param {string} value Item value
+     */
+    put: function (name, value) {
+      $cookieStore.put(name, value);
     }
   };
 

src/service/storage-local.js

@@ -42,12 +42,29 @@ angular.module('pascalprecht.translate')
        * @description
        * Sets an item in localStorage by given name.
        *
+       * @deprecated use #put
+       *
        * @param {string} name Item name
        * @param {string} value Item value
        */
       set: function (name, value) {
         langKey=value;
         $window.localStorage.setItem(name, value);
+      },
+      /**
+       * @ngdoc function
+       * @name pascalprecht.translate.$translateLocalStorage#put
+       * @methodOf pascalprecht.translate.$translateLocalStorage
+       *
+       * @description
+       * Sets an item in localStorage by given name.
+       *
+       * @param {string} name Item name
+       * @param {string} value Item value
+       */
+      put: function (name, value) {
+        langKey=value;
+        $window.localStorage.setItem(name, value);
       }
     };
   }());

src/service/translate.js

@@ -903,7 +903,7 @@ angular.module('pascalprecht.translate').provider('$translate', ['$STORAGE_KEY',
         $rootScope.$emit('$translateChangeSuccess', {language: key});
 
         if ($storageFactory) {
-          Storage.set($translate.storageKey(), $uses);
+          Storage.put($translate.storageKey(), $uses);
         }
         // inform default interpolator
         defaultInterpolator.setLocale($uses);
@@ -977,8 +977,8 @@ angular.module('pascalprecht.translate').provider('$translate', ['$STORAGE_KEY',
       if ($storageFactory) {
         Storage = $injector.get($storageFactory);
 
-        if (!Storage.get || !Storage.set) {
-          throw new Error('Couldn\'t use storage \'' + $storageFactory + '\', missing get() or set() method!');
+        if (!Storage.get || !Storage.put) {
+          throw new Error('Couldn\'t use storage \'' + $storageFactory + '\', missing get() or put() method!');
         }
       }
 

src/translate.js

@@ -18,9 +18,9 @@ angular.module('pascalprecht.translate', ['ng'])
       if (angular.isString($translate.preferredLanguage())) {
         $translate.use($translate.preferredLanguage());
         // $translate.use() will also remember the language.
-        // So, we don't need to call storage.set() here.
+        // So, we don't need to call storage.put() here.
       } else {
-        storage.set(key, $translate.use());
+        storage.put(key, $translate.use());
       }
 
     } else {

test/unit/service/storage-cookie.spec.js

@@ -18,9 +18,10 @@ describe('pascalprecht.translate', function () {
       expect(typeof $translateCookieStorage).toBe('object');
     });
 
-    it('should have a set() and a get() method', function () {
+    it('should have a put() and a get() method', function () {
+      expect($translateCookieStorage.put).toBeDefined();
       expect($translateCookieStorage.get).toBeDefined();
-      expect($translateCookieStorage.set).toBeDefined();
+      expect($translateCookieStorage.set).toBeDefined(); // deprecated
     });
 
     describe('$translateCookieStorage#get', function () {
@@ -30,9 +31,10 @@ describe('pascalprecht.translate', function () {
       });
     });
 
-    describe('$translateCookieStorage#set', function () {
+    describe('$translateCookieStorage#put', function () {
       it('should be a function', function () {
-        expect(typeof $translateCookieStorage.set).toBe('function');
+        expect(typeof $translateCookieStorage.set).toBe('function'); // deprecated
+        expect(typeof $translateCookieStorage.put).toBe('function');
       });
     });
   });
@@ -82,7 +84,8 @@ describe('pascalprecht.translate', function () {
 
     it('should return registered storage instance if exists', function () {
       expect(typeof $translate.storage()).toBe('object');
-      expect($translate.storage().set).toBeDefined();
+      expect($translate.storage().set).toBeDefined(); // deprecated
+      expect($translate.storage().put).toBeDefined();
       expect($translate.storage().get).toBeDefined();
     });
   });

test/unit/service/storage-local.spec.js

@@ -18,9 +18,10 @@ describe('pascalprecht.translate', function () {
       expect(typeof $translateLocalStorage).toBe('object');
     });
 
-    it('should have a set() and a get() method', function () {
+    it('should have a put() and a get() method', function () {
+      expect($translateLocalStorage.put).toBeDefined();
       expect($translateLocalStorage.get).toBeDefined();
-      expect($translateLocalStorage.set).toBeDefined();
+      expect($translateLocalStorage.set).toBeDefined(); // deprecated
     });
 
     describe('$translateLocalStorage#get', function () {
@@ -30,9 +31,10 @@ describe('pascalprecht.translate', function () {
       });
     });
 
-    describe('$translateLocalStorage#set()', function () {
+    describe('$translateLocalStorage#put()', function () {
       it('should be a function', function () {
-        expect(typeof $translateLocalStorage.set).toBe('function');
+        expect(typeof $translateLocalStorage.set).toBe('function'); // deprecated
+        expect(typeof $translateLocalStorage.put).toBe('function');
       });
     });