Release(2.0.1): add support for _source filter (#133)

* task(Readme): readme file update

* fix(eslint): all lint issues, remove unused files

* fix(eslint): constructor, prototype issues for ESConnector class

* fix(lint): lint issues fixed in all.js and buildOrder.js

* feature(Client): all client APIs updated

* fix(replaceOrCreate): response fix

* task(README, examples): update README and example server

* Release: Version 2.0.0

* task(filter): added _source filter support for fields

* release(2.0.1): version update

Author: bharathkontham

lib/buildFilter.js

@@ -32,42 +32,22 @@ function buildFilter(modelName, idName, criteria = {}, size = null, offset = nul
       filter.from = criteria.offset;
     }
     if (criteria.fields) {
-      // { fields: {propertyName: <true|false>, propertyName: <true|false>, ... } }
-      // filter.body.fields = self.buildOrder(model, idName, criteria.fields);
-      // TODO: make it so
-      // http://www.elastic.co/guide/en/elasticsearch/reference/1.x/search-request-source-filtering.html
-      // http://www.elastic.co/guide/en/elasticsearch/reference/1.x/search-request-fields.html
-      /* POST /shakespeare/User/_search
-       {
-       '_source': {
-       'include': ['seq'],
-       'exclude': ['seq']
-       }
-       } */
-
-      /* @raymondfeng and @bajtos - I'm observing something super strange,
-       i haven't implemented the FIELDS filter for elasticsearch connector
-       but the test which should fail until I implement such a feature ... is actually passing!
-       ... did someone at some point of time implement an in-memory filter for FIELDS
-       in the underlying loopback-connector implementation? */
-
       // Elasticsearch _source filtering code
-      /* if (Array.isArray(criteria.fields) || typeof criteria.fields === 'string') {
+      if (Array.isArray(criteria.fields) || typeof criteria.fields === 'string') {
+        // eslint-disable-next-line no-underscore-dangle
         filter.body._source = criteria.fields;
       } else if (typeof criteria.fields === 'object' && Object.keys(criteria.fields).length > 0) {
-        filter.body._source.includes = _.map(_.pickBy(criteria.fields, function(v, k) {
-          return v === true;
-        }), function(v, k) { return k; });
-        filter.body._source.excludes = _.map(_.pickBy(criteria.fields, function(v, k) {
-          return v === false;
-        }), function(v, k) { return k; });
-      } */
+        // eslint-disable-next-line no-underscore-dangle
+        filter.body._source = {
+          includes: _.map(_.pickBy(criteria.fields, (v) => v === true), (v, k) => k),
+          excludes: _.map(_.pickBy(criteria.fields, (v) => v === false), (v, k) => k)
+        };
+      }
     }
     if (criteria.order) {
       log('ESConnector.prototype.buildFilter', 'will delegate sorting to buildOrder()');
       filter.body.sort = self.buildOrder(modelName, idName, criteria.order);
-    } else { // TODO: expensive~ish and no clear guidelines so turn it off?
-      // var idNames = this.idNames(model); // TODO: support for compound ids?
+    } else {
       // eslint-disable-next-line no-underscore-dangle
       const modelProperties = this._models[modelName].properties;
       if (idName === 'id' && modelProperties.id.generated) {

package.json

@@ -1,6 +1,6 @@
 {
   "name": "loopback-connector-esv6",
-  "version": "2.0.0",
+  "version": "2.0.1",
   "description": "LoopBack Connector for Elasticsearch 6.x and 7.x",
   "main": "index.js",
   "scripts": {