Merge pull request #257 from MalteGerth/patch-1

removed unused assignments in conditions
ZF-Commons · Jun 30, 2014 · 02a5593 · 02a5593
2 parents f6356a6 + 70b4d77
commit 02a5593
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 6 deletions.
diff --git a/src/ZfcRbac/View/Strategy/RedirectStrategy.php b/src/ZfcRbac/View/Strategy/RedirectStrategy.php
@@ -62,9 +62,9 @@ public function __construct(RedirectStrategyOptions $options, AuthenticationServ
     public function onError(MvcEvent $event)
     {
         // Do nothing if no error or if response is not HTTP response
-        if (!($exception = $event->getParam('exception') instanceof UnauthorizedExceptionInterface)
-            || ($result = $event->getResult() instanceof HttpResponse)
-            || !($response = $event->getResponse() instanceof HttpResponse)
+        if (!($event->getParam('exception') instanceof UnauthorizedExceptionInterface)
+            || ($event->getResult() instanceof HttpResponse)
+            || !($event->getResponse() instanceof HttpResponse)
         ) {
             return;
         }

diff --git a/src/ZfcRbac/View/Strategy/UnauthorizedStrategy.php b/src/ZfcRbac/View/Strategy/UnauthorizedStrategy.php
@@ -56,9 +56,9 @@ public function __construct(UnauthorizedStrategyOptions $options)
     public function onError(MvcEvent $event)
     {
         // Do nothing if no error or if response is not HTTP response
-        if (!($exception = $event->getParam('exception') instanceof UnauthorizedExceptionInterface)
-            || ($result = $event->getResult() instanceof HttpResponse)
-            || !($response = $event->getResponse() instanceof HttpResponse)
+        if (!($event->getParam('exception') instanceof UnauthorizedExceptionInterface)
+            || ($event->getResult() instanceof HttpResponse)
+            || !($event->getResponse() instanceof HttpResponse)
         ) {
             return;
         }