Merge pull request #40 from zach-snell/patch-1

scott-xu · web-flow · commit 4350ad0f2fda · 2017-09-28T07:35:46.000+08:00
Issue: #39, AsyncIntercept Exceptions
diff --git a/src/Ninject.Extensions.Interception/AsyncInterceptor.cs b/src/Ninject.Extensions.Interception/AsyncInterceptor.cs
@@ -102,9 +102,9 @@ private void InterceptTaskWithResult<TResult>(IInvocation invocation)
                     }).Unwrap()
                 .ContinueWith(t =>
                         {
-                            invocationClone.ReturnValue = t.Result;
                             this.AfterInvoke(invocationClone);
                             this.AfterInvoke(invocationClone, t);
+                            invocationClone.ReturnValue = t.Result;
                             return (TResult)invocationClone.ReturnValue;
                         });
         }

Original file line number	Diff line number	Diff line change
`@@ -102,9 +102,9 @@ private void InterceptTaskWithResult<TResult>(IInvocation invocation)`
`102`	`102`	`}).Unwrap()`
`103`	`103`	`.ContinueWith(t =>`
`104`	`104`	`{`
`105`		`- invocationClone.ReturnValue = t.Result;`
`106`	`105`	`this.AfterInvoke(invocationClone);`
`107`	`106`	`this.AfterInvoke(invocationClone, t);`
	`107`	`+ invocationClone.ReturnValue = t.Result;`
`108`	`108`	`return (TResult)invocationClone.ReturnValue;`
`109`	`109`	`});`
`110`	`110`	`}`