[zendframework/zendframework#5364] Simplify conditional

- we know we have a boolean; remove true === test - combine if statement from body with else parent
zendframework · Jan 3, 2014 · accee7c · accee7c
1 parent b437ca2
commit accee7c
Showing 1 changed file with 3 additions and 5 deletions.
diff --git a/src/Hydrator/ClassMethods.php b/src/Hydrator/ClassMethods.php
@@ -81,12 +81,10 @@ public function setUnderscoreSeparatedKeys($underscoreSeparatedKeys)
     {
         $this->underscoreSeparatedKeys = (bool) $underscoreSeparatedKeys;
 
-        if (true === $this->underscoreSeparatedKeys) {
+        if ($this->underscoreSeparatedKeys) {
             $this->setNamingStrategy(new UnderscoreNamingStrategy);
-        } else {
-            if ($this->getNamingStrategy() instanceof UnderscoreNamingStrategy) {
-                $this->removeNamingStrategy();
-            }
+        } elseif ($this->getNamingStrategy() instanceof UnderscoreNamingStrategy) {
+            $this->removeNamingStrategy();
         }
 
         return $this;