hapi - npm Package Compare versions

Comparing version 0.15.2 to 0.15.3

lib/ext.js

@@ -35,2 +35,4 @@ // Load modules
 
+    var self = this;
+
     options = options || {};
@@ -40,22 +42,32 @@
 
-    this._events[event] = this._events[event] || []
+    // Validate rules
 
-    var ext = {
-        priority: this._events[event].length,
-        before: [].concat(options.before || []),
-        after: [].concat(options.after || []),
-        group: plugin || '?',
-        func: func
-    };
+    var before = [].concat(options.before || []);
+    var after = [].concat(options.after || []);
+    var group = plugin || '?';
 
-    // Validate rules
+    Utils.assert(before.indexOf(group) === -1, 'Plugin ext cannot come before itself (' + group + ')');
+    Utils.assert(before.indexOf('?') === -1, 'Plugin ext cannot come before unassociated exts');
+    Utils.assert(after.indexOf(group) === -1, 'Plugin ext cannot come after itself (' + group + ')');
+    Utils.assert(after.indexOf('?') === -1, 'Plugin ext cannot come after unassociated exts');
 
-    Utils.assert(ext.before.indexOf(ext.group) === -1, 'Plugin ext cannot come before itself (' + ext.plugin + ')');
-    Utils.assert(ext.before.indexOf('?') === -1, 'Plugin ext cannot come before unassociated exts');
-    Utils.assert(ext.after.indexOf(ext.group) === -1, 'Plugin ext cannot come after itself (' + ext.plugin + ')');
-    Utils.assert(ext.after.indexOf('?') === -1, 'Plugin ext cannot come after unassociated exts');
+    // Add functions
 
+    this._events[event] = this._events[event] || [];
+
+    ([].concat(func)).forEach(function (fn, i) {
+
+        var ext = {
+            priority: self._events[event].length,
+            before: before,
+            after: after,
+            group: group,
+            func: fn
+        };
+
+        self._events[event].push(ext);
+    });
+
     // Insert event
 
-    this._events[event].push(ext);
     var error = this.sort(event);
@@ -62,0 +74,0 @@ Utils.assert(!error, event + ' extension' + (plugin ? ' add by ' + plugin : '') + ' created a dependencies error');

lib/pack.js

@@ -0,0 +0,0 @@ // Load modules

lib/payload.js

@@ -0,0 +0,0 @@ // Load modules

lib/request.js

@@ -0,0 +0,0 @@ // Load modules

lib/response/error.js

@@ -0,0 +0,0 @@ // Load modules

lib/response/headers.js

@@ -0,0 +0,0 @@ // Load modules

lib/route.js

@@ -0,0 +0,0 @@ // Load modules

lib/server.js

@@ -0,0 +0,0 @@ // Load modules

package.json

@@ -5,3 +5,3 @@ {
   "homepage": "http://hapijs.com",
-  "version": "0.15.2",
+  "version": "0.15.3",
   "author": "Eran Hammer <[e-mail]> (http://hueniverse.com)",
@@ -8,0 +8,0 @@ "contributors": [

test/integration/composer.js

@@ -0,0 +0,0 @@ // Load modules

test/integration/ext.js

@@ -23,3 +23,3 @@ // Load modules
 
-    describe('onRequest', function (done) {
+    describe('#onRequest', function (done) {
 
@@ -43,2 +43,30 @@ it('replies with custom response', function (done) {
     });
+
+    describe('#ext', function () {
+
+        it('supports adding an array of ext methods', function (done) {
+
+            var server = new Hapi.Server();
+            server.ext('onPreHandler', [
+                function (request, next) {
+
+                    request.x = '1';
+                    next();
+                },
+                function (request, next) {
+
+                    request.x += '2';
+                    next();
+                }
+            ]);
+
+            server.route({ method: 'GET', path: '/', handler: function () { this.reply(this.x); } });
+
+            server.inject({ method: 'GET', url: '/' }, function (res) {
+
+                expect(res.result).to.equal('12');
+                done();
+            });
+        });
+    });
 });

test/integration/pack.js

@@ -0,0 +0,0 @@ // Load modules

test/integration/pack/--deps1/index.js

@@ -0,0 +0,0 @@ // Declare internals

test/integration/pack/--deps1/package.json

@@ -0,0 +0,0 @@ {

test/integration/pack/--deps2/index.js

@@ -0,0 +0,0 @@ // Declare internals

test/integration/pack/--deps2/package.json

@@ -0,0 +0,0 @@ {

test/integration/pack/--deps3/index.js

@@ -0,0 +0,0 @@ // Declare internals

test/integration/pack/--deps3/package.json

@@ -0,0 +0,0 @@ {

test/integration/pack/--test1/lib/index.js

@@ -0,0 +0,0 @@ // Declare internals

test/integration/pack/--test2/lib/index.js

@@ -0,0 +0,0 @@ // Declare internals

test/integration/pack/--views/lib/index.js

@@ -0,0 +0,0 @@ // Declare internals

test/integration/payload.js

@@ -0,0 +0,0 @@ // Load modules

test/integration/prerequisites.js

@@ -0,0 +0,0 @@ // Load modules

test/integration/proxy.js

@@ -0,0 +0,0 @@ // Load modules

test/integration/request.js

@@ -0,0 +0,0 @@ // Load modules

test/unit/ext.js

@@ -0,0 +0,0 @@ // Load modules

test/unit/views.js

@@ -0,0 +0,0 @@ // Load modules

New alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Fixed alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Improved metrics

Lines of code

13065

increased by0.21%

Worsened metrics

Total package byte prevSize

763649

decreased by-0.88%

No dependency changes