Merge pull request #46 from kuzzleio/KUZ372_removePutMapping

removed the dataCollection.putMapping method

dist/kuzzle.js

@@ -1760,29 +1760,6 @@ KuzzleDataCollection.prototype.publishMessage = function (document, options) {
   return this;
 };
 
-/**
- * Applies a new mapping to the data collection.
- * Note that you cannot delete an existing mapping, you can only add or update one.
- *
- * @param {object} mapping - mapping to apply
- * @param {object} [options] - optional arguments
- * @param {responseCallback} [cb] - Returns an instantiated KuzzleDataMapping object
- * @returns {*} this
- */
-KuzzleDataCollection.prototype.putMapping = function (mapping, options, cb) {
-  var dataMapping;
-
-  if (!cb && typeof options === 'function') {
-    cb = options;
-    options = null;
-  }
-
-  dataMapping = new KuzzleDataMapping(this, mapping);
-  dataMapping.apply(options, cb);
-
-  return this;
-};
-
 /**
  * Replace an existing document with a new one.
  *

package.json

@@ -1,6 +1,6 @@
 {
   "name": "kuzzle-sdk",
-  "version": "1.3.7",
+  "version": "1.3.8",
   "description": "Official Javascript SDK for Kuzzle",
   "author": "The Kuzzle Team <support@kuzzle.io>",
   "repository": {

src/kuzzleDataCollection.js

@@ -430,29 +430,6 @@ KuzzleDataCollection.prototype.publishMessage = function (document, options) {
   return this;
 };
 
-/**
- * Applies a new mapping to the data collection.
- * Note that you cannot delete an existing mapping, you can only add or update one.
- *
- * @param {object} mapping - mapping to apply
- * @param {object} [options] - optional arguments
- * @param {responseCallback} [cb] - Returns an instantiated KuzzleDataMapping object
- * @returns {*} this
- */
-KuzzleDataCollection.prototype.putMapping = function (mapping, options, cb) {
-  var dataMapping;
-
-  if (!cb && typeof options === 'function') {
-    cb = options;
-    options = null;
-  }
-
-  dataMapping = new KuzzleDataMapping(this, mapping);
-  dataMapping.apply(options, cb);
-
-  return this;
-};
-
 /**
  * Replace an existing document with a new one.
  *

test/kuzzleDataCollection/methods.test.js

@@ -696,73 +696,6 @@ describe('KuzzleDataCollection methods', function () {
     });
   });
 
-  describe('#putMapping', function () {
-    var
-      KuzzleDataCollectionProxyquired,
-      collection;
-
-    beforeEach(function () {
-      expectedQuery = {
-        index: 'bar',
-        collection: 'foo',
-        action: 'putMapping',
-        controller: 'admin',
-        body: {}
-      };
-
-      KuzzleDataCollectionProxyquired = proxyquire('../../src/kuzzleDataCollection', {
-        './kuzzleDataMapping': function (c, m) {
-          var dm = new KuzzleDataMapping(c, m);
-
-          dm.refresh = function (options, cb) {
-            cb(null, result);
-          };
-          return dm;
-        }
-      });
-
-      kuzzle = new Kuzzle('foo', {defaultIndex: 'bar'});
-      collection = new KuzzleDataCollectionProxyquired(kuzzle, expectedQuery.index, expectedQuery.collection);
-      kuzzle.query = queryStub;
-      emitted = false;
-      result = { result: {_source: { properties: {}}}};
-      error = null;
-    });
-
-    it('should instantiate a new KuzzleDataMapping object', function (done) {
-      var
-        options = { queuable: false };
-
-      expectedQuery.options = options;
-
-      should(collection.putMapping(result.result, options, function (err, res) {
-        should(err).be.null();
-        done();
-      })).be.exactly(collection);
-      should(emitted).be.true();
-    });
-
-    it('should handle the callback argument correctly', function () {
-      collection.putMapping(result.result, function () {});
-      should(emitted).be.true();
-
-      emitted = false;
-      collection.putMapping(result.result, {}, function () {});
-      should(emitted).be.true();
-    });
-
-    it('should call the callback with an error if one occurs', function (done) {
-      error = 'foobar';
-      this.timeout(50);
-
-      collection.putMapping({}, function (err, res) {
-        should(err).be.exactly('foobar');
-        should(res).be.undefined();
-        done();
-      });
-    });
-  });
-
   describe('#replaceDocument', function () {
     beforeEach(function () {
       kuzzle = new Kuzzle('foo', {defaultIndex: 'bar'});