Merge pull request #42 from ninject/revert-40-patch-1

scott-xu · web-flow · commit bf063b00c3a1 · 2017-10-08T15:33:08.000+08:00
Revert "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;`
`105`	`106`	`this.AfterInvoke(invocationClone);`
`106`	`107`	`this.AfterInvoke(invocationClone, t);`
`107`		`- invocationClone.ReturnValue = t.Result;`
`108`	`108`	`return (TResult)invocationClone.ReturnValue;`
`109`	`109`	`});`
`110`	`110`	`}`