loopback-connector - npm Package Compare versions

Comparing version 1.1.1 to 1.2.0

CONTRIBUTING.md

@@ -1,25 +0,18 @@
-
 ### Contributing ###
 
-Thank you for your interest in `loopback`, an open source project
+Thank you for your interest in `loopback-connector`, an open source project
 administered by StrongLoop.
 
-Contributing to loopback is easy. In a few simple steps:
+Contributing to `loopback-connector` is easy. In a few simple steps:
 
-  * Ensure that your effort is aligned with the project’s roadmap by
+  * Ensure that your effort is aligned with the project's roadmap by
     talking to the maintainers, especially if you are going to spend a
-    lot of time on it. This project is currently maintained by
-    [@ritch](https://github.com/ritch), [@raymondfeng](https://github.com/raymondfeng),
-    and [@bajtos](https://github.com/bajtos). The preferred channel of communication
-    is [LoopBack Forum](https://groups.google.com/forum/#!forum/loopbackjs) or
-    [Github Issues](https://github.com/strongloop/loopback/issues).
+    lot of time on it.
 
   * Make something better or fix a bug.
 
-  * Adhere to code style outlined in the
+  * Adhere to code style outlined in the [Google C++ Style Guide][] and
     [Google Javascript Style Guide][].
 
-  * [Sign your patches](#signing-patches) to indicate that your are
-    making your contribution available under the terms of the
-    [Contributor License Agreement](#contributor-license-agreement).
+  * Sign the [Contributor License Agreement](https://cla.strongloop.com/strongloop/loopback-connector)
 
@@ -29,36 +22,2 @@ * Submit a pull request through Github.
 
-### Signing patches ###
-
-Like many open source projects, we need a contributor license agreement
-from you before we can merge in your changes.
-
-In summary, by submitting your code, you are granting us a right to use
-that code under the terms of this Agreement, including providing it to
-others. You are also certifying that you wrote it, and that you are
-allowed to license it to us. You are not giving up your copyright in
-your work. The license does not change your rights to use your own
-contributions for any other purpose.
-
-Contributor License Agreements are important because they define the
-chain of ownership of a piece of software. Some companies won't allow
-the use of free software without clear agreements around code ownership.
-That's why many open source projects collect similar agreements from
-contributors. The CLA here is based on the Apache CLA.
-
-To signify your agreement to these terms, add the following line to the
-bottom of your commit message. Use your real name and an actual e-mail
-address.
-
-```
-Signed-off-by: Random J Developer <[e-mail]>
-```
-
-Alternatively you can use the git command line to automatically add this
-line, as follows:
-
-```
-$ git commit -sm "Replace rainbows by unicorns"
-```
-
-
 ### Contributor License Agreement ###
@@ -192,5 +151,3 @@
 
-
+[Google C++ Style Guide]: https://google-styleguide.googlecode.com/svn/trunk/cppguide.xml
 [Google Javascript Style Guide]: https://google-styleguide.googlecode.com/svn/trunk/javascriptguide.xml
-[license]: LICENSE
-

lib/sql.js

@@ -374,18 +374,30 @@ var util = require('util');
   models = models || Object.keys(self._models);
-  async.each(models, function (model, callback) {
-    if (model in self._models) {
-      self.dropTable(model, function (err) {
+  if (models.length === 0) {
+    return process.nextTick(cb);
+  }
+
+  var invalidModels = models.filter(function(m) {
+      return !(m in self._models);
+    });
+  if (invalidModels.length) {
+    return process.nextTick(function() {
+      cb(new Error('Cannot migrate models not attached to this datasource: ' +
+        invalidModels.join(' ')));
+    });
+  }
+
+  async.each(models, function(model, done) {
+    self.dropTable(model, function(err) {
+      if (err) {
+        // TODO(bajtos) should we abort here and call cb(err)?
+        // The original code in juggler ignored the error completely
+        console.error(err);
+      }
+      self.createTable(model, function(err, result) {
         if (err) {
-          // TODO(bajtos) should we abort here and call cb(err)?
-          // The original code in juggler ignored the error completely
           console.error(err);
         }
-        self.createTable(model, function (err, result) {
-          if (err) {
-            console.error(err);
-          }
-          callback(err, result);
-        });
+        done(err, result);
       });
-    }
+    });
   }, cb);
@@ -392,0 +404,0 @@ };

package.json

 {
   "name": "loopback-connector",
-  "version": "1.1.1",
+  "version": "1.2.0",
   "description": "Building blocks for LoopBack connectors",
@@ -27,4 +27,6 @@ "keywords": [
   "devDependencies": {
+    "chai": "~1.9.2",
+    "loopback-datasource-juggler": "^2.0.0",
     "mocha": "^1.19.0"
   }
 }

No alert changes

Improved metrics

Total package byte prevSize

50357

increased by2.42%

Number of package files

13

increased by18.18%

Lines of code

645

increased by17.06%

Worsened metrics

Dev dependency count

3

increased by200%

No dependency changes