pigretto - npm Package Compare versions

Comparing version 2.4.1 to 2.4.2

dist/es6/trap/executors/CallTrapExecutor.js

@@ -28,2 +28,3 @@ /*
 import { bindContextThisArg } from "../../polyfill/polyfill";
+import { isEmpty } from "js-utl";
 export default withFunctionTrapExecutor(class CallTrapExecutor extends TrapExecutor {
@@ -35,3 +36,3 @@ getTrapArgsArgumentsListIndex() {
   execute([target, property, receiver, propertyValue], before, around, after) {
-    if (typeof propertyValue === "function") {
+    if (typeof propertyValue === "function" && [before, around, after].some(advices => !isEmpty(advices))) {
       const superExecute = trapArgs => {
@@ -38,0 +39,0 @@ return super.execute(trapArgs, before, around, after);

package.json

 {
   "name": "pigretto",
-  "version": "2.4.1",
+  "version": "2.4.2",
   "description": "A JS helper to easily implement proxies, facades, lazy loading and monkey patching using an API with an AOP flavor.",
@@ -5,0 +5,0 @@ "keywords": [

src/trap/executors/CallTrapExecutor.js

@@ -29,2 +29,3 @@ /*
 import { bindContextThisArg } from "../../polyfill/polyfill";
+import { isEmpty } from "js-utl";
 
@@ -43,3 +44,6 @@ export default withFunctionTrapExecutor(
     ) {
-      if (typeof propertyValue === "function") {
+      if (
+        typeof propertyValue === "function" &&
+        [before, around, after].some(advices => !isEmpty(advices))
+      ) {
         const superExecute = trapArgs => {
@@ -46,0 +50,0 @@ return super.execute(trapArgs, before, around, after);

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

Total package byte prevSize

3082664

increased by0.04%

Lines of code

31496

increased by0.03%

No dependency changes