Skip to content

Commit da7cd12

Browse files
authored
Merge pull request #159 from AlessandroMinoccheri/semplify_returns
simplified return statement
2 parents e3b258a + c436e84 commit da7cd12

File tree

1 file changed

+16
-9
lines changed

1 file changed

+16
-9
lines changed

src/Pusher.php

Lines changed: 16 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -384,6 +384,7 @@ public static function array_implode($glue, $separator, $array)
384384
if (!is_array($array)) {
385385
return $array;
386386
}
387+
387388
$string = array();
388389
foreach ($array as $key => $val) {
389390
if (is_array($val)) {
@@ -451,11 +452,13 @@ public function trigger($channels, $event, $data, $socket_id = null, $debug = fa
451452

452453
if ($response['status'] === 200 && $debug === false) {
453454
return true;
454-
} elseif ($debug === true || $this->settings['debug'] === true) {
455+
}
456+
457+
if ($debug === true || $this->settings['debug'] === true) {
455458
return $response;
456-
} else {
457-
return false;
458459
}
460+
461+
return false;
459462
}
460463

461464
/**
@@ -498,11 +501,13 @@ public function triggerBatch($batch = array(), $debug = false, $already_encoded
498501

499502
if ($response['status'] === 200 && $debug === false) {
500503
return true;
501-
} elseif ($debug === true || $this->settings['debug'] === true) {
504+
}
505+
506+
if ($debug === true || $this->settings['debug'] === true) {
502507
return $response;
503-
} else {
504-
return false;
505508
}
509+
510+
return false;
506511
}
507512

508513
/**
@@ -661,10 +666,12 @@ public function notify($interests, $data = array(), $debug = false)
661666

662667
if ($response['status'] === 202 && $debug === false) {
663668
return true;
664-
} elseif ($debug === true || $this->settings['debug'] === true) {
669+
}
670+
671+
if ($debug === true || $this->settings['debug'] === true) {
665672
return $response;
666-
} else {
667-
return false;
668673
}
674+
675+
return false;
669676
}
670677
}

0 commit comments

Comments
 (0)