aboutsummaryrefslogtreecommitdiffstats
path: root/include/classes/BigPipe
diff options
context:
space:
mode:
authorThomas Lange <code@nerdmind.de>2021-10-24 15:54:43 +0200
committerThomas Lange <code@nerdmind.de>2021-10-24 15:54:43 +0200
commitcbe7e31f68d4ca2bfcd53dffdb130f1f84be3343 (patch)
tree4e285e9a92116c11c2d72245e71af4422b65a961 /include/classes/BigPipe
parent6d69bab4d07cd502d78b565ee9eba3b1e9296931 (diff)
downloadbigpipe-cbe7e31f68d4ca2bfcd53dffdb130f1f84be3343.tar.gz
bigpipe-cbe7e31f68d4ca2bfcd53dffdb130f1f84be3343.tar.xz
bigpipe-cbe7e31f68d4ca2bfcd53dffdb130f1f84be3343.zip
Split up BigPipe "enabled" method into two methods
Split up the "enabled" method of the BigPipe class into two methods, one to check if the pipelining mode is currently enabled ("isEnabled") and another one to enable/disable the pipelining mode ("setEnabled"). This change prevents that a single method has two different jobs, which can be confusing for programmers who didn't wrote the code but only use them. Now there are two clearly named methods for exactly one job.
Diffstat (limited to 'include/classes/BigPipe')
-rw-r--r--include/classes/BigPipe/BigPipe.php19
-rw-r--r--include/classes/BigPipe/Pagelet.php4
2 files changed, 13 insertions, 10 deletions
diff --git a/include/classes/BigPipe/BigPipe.php b/include/classes/BigPipe/BigPipe.php
index 30e3f41..6162388 100644
--- a/include/classes/BigPipe/BigPipe.php
+++ b/include/classes/BigPipe/BigPipe.php
@@ -15,17 +15,20 @@ class BigPipe {
private static $pagelets = [];
#===============================================================================
- # Enable or disable the pipeline mode
+ # Check if pipelining mode is enabled
#===============================================================================
- public static function enabled($change = NULL) {
- if($change !== NULL) {
- self::$enabled = (bool) $change;
- }
-
+ public static function isEnabled(): bool {
return self::$enabled;
}
#===============================================================================
+ # Enable or disable the pipelining mode
+ #===============================================================================
+ public static function setEnabled(bool $enabled): void {
+ self::$enabled = $enabled;
+ }
+
+ #===============================================================================
# Insert pagelet into queue
#===============================================================================
public static function enqueue(Pagelet $Pagelet) {
@@ -63,7 +66,7 @@ class BigPipe {
if(!empty($pagelets_ordered)) {
$pagelets = call_user_func_array('array_merge', $pagelets_ordered);
- if(self::enabled()) {
+ if(self::isEnabled()) {
foreach($pagelets as $Pagelet) {
$Pagelet->flush();
}
@@ -90,7 +93,7 @@ class BigPipe {
}
}
- if(self::enabled()) {
+ if(self::isEnabled()) {
echo "<script>BigPipe.onLastPageletArrived();</script>\n";
}
}
diff --git a/include/classes/BigPipe/Pagelet.php b/include/classes/BigPipe/Pagelet.php
index 0358df6..bd50c4b 100644
--- a/include/classes/BigPipe/Pagelet.php
+++ b/include/classes/BigPipe/Pagelet.php
@@ -150,7 +150,7 @@ class Pagelet extends Item {
# Flush pagelet immediately
#===============================================================================
public function flush() {
- if(BigPipe::enabled()) {
+ if(BigPipe::isEnabled()) {
$pageletHTML = str_replace(["\r", "\n", "\t"], '', $this->getHTML());
$pageletHTML = str_replace('--', '&#45;&#45;', $pageletHTML);
@@ -169,7 +169,7 @@ class Pagelet extends Item {
#===============================================================================
public function __toString() {
$pageletHTML = "<{$this->tagName} id=\"{$this->getID()}\">";
- $pageletHTML .= !BigPipe::enabled() ? $this->getHTML() : $this->tagHTML;
+ $pageletHTML .= !BigPipe::isEnabled() ? $this->getHTML() : $this->tagHTML;
$pageletHTML .= "</{$this->tagName}>";
return $pageletHTML;