Code revised with reactor

Applied rules:
 * ChangeOrIfContinueToMultiContinueRector

Author: DanProgs

firstLogin.php

@@ -190,9 +190,12 @@ function notifyGlobalAdmins(&$dbHandler, &$userObj)
         $dest = [];
         $validator = new Zend_Validate_EmailAddress();
         foreach ($mail['to'] as $mm) {
-            $ema = trim($mm);
-            if ($ema == '' || ! $validator->isValid($ema)) {
-                continue;
+            $ema = trim($mm);
+            if ($ema == '') {
+                continue;
+            }
+            if (! $validator->isValid($ema)) {
+                continue;
             }
             $dest[] = $ema;
         }

lib/functions/plugin_api.php

@@ -413,9 +413,12 @@ function plugin_find_all()
 
     if ($t_dir = opendir($t_plugin_path)) {
         while (($t_file = readdir($t_dir)) !== false) {
-            if ('.' == $t_file || '..' == $t_file) {
-                continue;
-            }
+            if ('.' == $t_file) {
+                continue;
+            }
+            if ('..' == $t_file) {
+                continue;
+            }
             if (is_dir($t_plugin_path . $t_file)) {
                 $t_plugin = plugin_register($t_file, true);
 
@@ -609,9 +612,12 @@ function get_all_available_plugins($existing_plugins)
     // Find all plugins that are newly available (And not already registered)
     if ($t_dir = opendir(TL_PLUGIN_PATH)) {
         while (($t_file = readdir($t_dir)) !== false) {
-            if ('.' == $t_file || '..' == $t_file) {
-                continue;
-            }
+            if ('.' == $t_file) {
+                continue;
+            }
+            if ('..' == $t_file) {
+                continue;
+            }
             if (! in_array($t_file, $registered_plugin_names) &&
                 is_dir(TL_PLUGIN_PATH . $t_file) && plugin_include($t_file)) {
                 $t_classname = $t_file . 'Plugin';

lib/functions/print.inc.php

@@ -828,9 +828,12 @@ function renderTestSpecTreeForPrinting(&$db, &$node, &$options, $env, $context,
         $childNodes = $node['childNodes'];
         $children_qty = count($childNodes);
         for ($idx = 0; $idx < $children_qty; $idx ++) {
-            $current = $childNodes[$idx];
-            if (is_null($current) || $current == REMOVEME) {
-                continue;
+            $current = $childNodes[$idx];
+            if (is_null($current)) {
+                continue;
+            }
+            if ($current == REMOVEME) {
+                continue;
             }
 
             if (isset($current['node_type_id']) &&

lib/functions/treeMenu.inc.php

@@ -957,10 +957,12 @@ function renderExecTreeNode($level, &$node, &$tcase_node, $hash_id_descr,
         // Can not assign anymore to intermediate variables.
         $nodes_qty = count($node['childNodes']);
         for ($idx = 0; $idx < $nodes_qty; $idx ++) {
-            if (is_null($node['childNodes'][$idx]) ||
-                $node['childNodes'][$idx] == REMOVEME) {
-                continue;
-            }
+            if (is_null($node['childNodes'][$idx])) {
+                continue;
+            }
+            if ($node['childNodes'][$idx] == REMOVEME) {
+                continue;
+            }
             renderExecTreeNode($level + 1, $node['childNodes'][$idx],
                 $tcase_node, $hash_id_descr, $linkto, $testCasePrefix, $opt);
         }

lib/plan/planAddTC.php

@@ -839,10 +839,12 @@ function sendMailToTesters(&$dbHandler, &$tcaseMgr, &$guiObj, &$argsObj,
                 $guiObj->testPlanName;
             foreach ($tester_set as $user_id => $value) {
                 $userObj = $userData[$user_id];
-                $email['to_address'] = trim($userObj->emailAddress);
-                if ($email['to_address'] == '' ||
-                    ! $validator->isValid($email['to_address'])) {
-                    continue;
+                $email['to_address'] = trim($userObj->emailAddress);
+                if ($email['to_address'] == '') {
+                    continue;
+                }
+                if (! $validator->isValid($email['to_address'])) {
+                    continue;
                 }
 
                 $email['body'] = $body_first_lines;

lib/plan/tc_exec_assignment.php

@@ -529,10 +529,12 @@ function sendMailToTesters(&$dbHandler, &$tcaseMgr, &$guiObj, &$argsObj,
                 }
 
                 $userObj = $guiObj->all_users[$user_id];
-                $email['to_address'] = trim($userObj->emailAddress);
-                if ($email['to_address'] == '' ||
-                    ! $validator->isValid($email['to_address'])) {
-                    continue;
+                $email['to_address'] = trim($userObj->emailAddress);
+                if ($email['to_address'] == '') {
+                    continue;
+                }
+                if (! $validator->isValid($email['to_address'])) {
+                    continue;
                 }
 
                 $email['body'] = $body_header;