Simplify returns in generate-phpt

Author: carusogabriel

scripts/dev/generate-phpt/src/setup/gtOptionalSections.php

@@ -56,17 +56,11 @@ public function hasSkipif() {
   }
 
   public function hasSkipifKey() {
-    if($this->skipifKey != '') {
-      return true;
-    }
-    return false;
+    return $this->skipifKey != '';
   }
 
   public function hasSkipifExt() {
-    if($this->skipifExt != '') {
-      return true;
-    }
-    return false;
+    return $this->skipifExt != '';
   }
   public function hasIni() {
     return $this->optSections['ini'];

scripts/dev/generate-phpt/src/setup/preconditions/gtIfClassHasMethod.php

@@ -5,20 +5,14 @@
  *
  */
 class gtIfClassHasMethod extends gtPreCondition {
-  
+
   public function check( $clo) {
-    if($clo->hasOption('c')) {
-      if(!$clo->hasOption('m')) {
-        return false;
-      }
-      return  true;
-    }
-    return true;
+    return !$clo->hasOption('c') || $clo->hasOption('m');
   }
-  
+
   public function getMessage() {
     return gtText::get('methodNotSpecified');
   }
 
 }
-?>
+?>

scripts/dev/generate-phpt/src/setup/preconditions/gtIsSpecifiedFunctionOrMethod.php

@@ -5,17 +5,13 @@
  *
  */
 class gtIsSpecifiedFunctionOrMethod extends gtPreCondition {
-  
+
   public function check( $clo) {
-    if($clo->hasOption('f') || $clo->hasOption('m')) {
-    
-        return true;
-      }
-    return false;
+    return $clo->hasOption('f') || $clo->hasOption('m');
   }
-  
+
   public function getMessage() {
     return gtText::get('functionOrMethodNotSpecified');
   }
 }
-?>
+?>

scripts/dev/generate-phpt/src/setup/preconditions/gtIsSpecifiedTestType.php

@@ -5,17 +5,13 @@
  *
  */
 class gtIsSpecifiedTestType extends gtPreCondition {
-  
+
   public function check( $clo) {
-    if($clo->hasOption('b') || $clo->hasOption('e') || $clo->hasOption('v') ) {
-    
-        return true;
-      }
-    return false;
+    return $clo->hasOption('b') || $clo->hasOption('e') || $clo->hasOption('v');
   }
 
   public function getMessage() {
     return gtText::get('testTypeNotSpecified');
   }
 }
-?>
+?>

scripts/dev/generate-phpt/src/setup/preconditions/gtIsValidClass.php

@@ -9,10 +9,7 @@ class gtIsValidClass extends gtPreCondition {
   public function check( $clo) {
     if($clo->hasOption('c') ) {
       $className = $clo->getOption('c');
-      if( in_array( $className, get_declared_classes() ) ) {
-        return true;
-      }
-      return false;
+      return in_array( $className, get_declared_classes() );
     }
     return true;
   }
@@ -21,4 +18,4 @@ public function getMessage() {
     return gtText::get('unknownClass');
   }
 }
-?>
+?>

scripts/dev/generate-phpt/src/setup/preconditions/gtIsValidFunction.php

@@ -10,10 +10,7 @@ public function check( $clo) {
     if($clo->hasOption('f') ) {
       $function = $clo->getOption('f');
       $functions = get_defined_functions();
-      if( in_array( $function, $functions['internal'] ) ) {
-        return true;
-      }
-      return false;
+      return in_array( $function, $functions['internal'] );
     }
     return true;
   }
@@ -22,4 +19,4 @@ public function getMessage() {
     return gtText::get('unknownFunction');
   }
 }
-?>
+?>