Merge branch '5.2' into 5.3

derrabus · derrabus · commit da9c0bea4997 · 2021-06-06T11:51:56.000+02:00
* 5.2:
  Fix incompatible implicit float-to-int conversions
  [Translation] Don't pass null to strtoupper()
  cs fix
  [String] Fix implicit float-to-int casts
  [DependencyInjection] Don't pass null to trim()
  Add return types to JsonSerializable implementations
  Fix Serializable deprecations triggered by token mocks
  Add missing security translations
diff --git a/Helper/ProgressBar.php b/Helper/ProgressBar.php
@@ -206,7 +206,7 @@ public function getProgressPercent(): float
 
     public function getBarOffset(): float
     {
-        return floor($this->max ? $this->percent * $this->barWidth : (null === $this->redrawFreq ? min(5, $this->barWidth / 15) * $this->writeCount : $this->step) % $this->barWidth);
+        return floor($this->max ? $this->percent * $this->barWidth : (null === $this->redrawFreq ? (int) (min(5, $this->barWidth / 15) * $this->writeCount) : $this->step) % $this->barWidth);
     }
 
     public function getEstimated(): float
@@ -280,7 +280,7 @@ public function setFormat(string $format)
     /**
      * Sets the redraw frequency.
      *
-     * @param int|float $freq The frequency in steps
+     * @param int|null $freq The frequency in steps
      */
     public function setRedrawFrequency(?int $freq)
     {
diff --git a/Helper/Table.php b/Helper/Table.php
@@ -442,7 +442,7 @@ private function renderRowSeparator(int $type = self::SEPARATOR_MID, string $tit
                 $formattedTitle = sprintf($titleFormat, Helper::substr($title, 0, $limit - $formatLength - 3).'...');
             }
 
-            $titleStart = ($markupLength - $titleLength) / 2;
+            $titleStart = intdiv($markupLength - $titleLength, 2);
             if (false === mb_detect_encoding($markup, null, true)) {
                 $markup = substr_replace($markup, $formattedTitle, $titleStart, $titleLength);
             } else {
diff --git a/Tests/Helper/ProgressBarTest.php b/Tests/Helper/ProgressBarTest.php
@@ -535,7 +535,7 @@ public function testRedrawFrequencyIsAtLeastOneIfZeroGiven()
     public function testRedrawFrequencyIsAtLeastOneIfSmallerOneGiven()
     {
         $bar = new ProgressBar($output = $this->getOutputStream(), 0, 0);
-        $bar->setRedrawFrequency(0.9);
+        $bar->setRedrawFrequency(0);
         $bar->start();
         $bar->advance();
 

Original file line number	Diff line number	Diff line change
`@@ -206,7 +206,7 @@ public function getProgressPercent(): float`
`206`	`206`
`207`	`207`	`public function getBarOffset(): float`
`208`	`208`	`{`
`209`		`- return floor($this->max ? $this->percent * $this->barWidth : (null === $this->redrawFreq ? min(5, $this->barWidth / 15) * $this->writeCount : $this->step) % $this->barWidth);`
	`209`	`+ return floor($this->max ? $this->percent * $this->barWidth : (null === $this->redrawFreq ? (int) (min(5, $this->barWidth / 15) * $this->writeCount) : $this->step) % $this->barWidth);`
`210`	`210`	`}`
`211`	`211`
`212`	`212`	`public function getEstimated(): float`
`@@ -280,7 +280,7 @@ public function setFormat(string $format)`
`280`	`280`	`/**`
`281`	`281`	`* Sets the redraw frequency.`
`282`	`282`	`*`
`283`		`- * @param int\|float $freq The frequency in steps`
	`283`	`+ * @param int\|null $freq The frequency in steps`
`284`	`284`	`*/`
`285`	`285`	`public function setRedrawFrequency(?int $freq)`
`286`	`286`	`{`
Original file line number	Diff line number	Diff line change
`@@ -442,7 +442,7 @@ private function renderRowSeparator(int $type = self::SEPARATOR_MID, string $tit`
`442`	`442`	`$formattedTitle = sprintf($titleFormat, Helper::substr($title, 0, $limit - $formatLength - 3).'...');`
`443`	`443`	`}`
`444`	`444`
`445`		`- $titleStart = ($markupLength - $titleLength) / 2;`
	`445`	`+ $titleStart = intdiv($markupLength - $titleLength, 2);`
`446`	`446`	`if (false === mb_detect_encoding($markup, null, true)) {`
`447`	`447`	`$markup = substr_replace($markup, $formattedTitle, $titleStart, $titleLength);`
`448`	`448`	`} else {`
Original file line number	Diff line number	Diff line change
`@@ -535,7 +535,7 @@ public function testRedrawFrequencyIsAtLeastOneIfZeroGiven()`
`535`	`535`	`public function testRedrawFrequencyIsAtLeastOneIfSmallerOneGiven()`
`536`	`536`	`{`
`537`	`537`	`$bar = new ProgressBar($output = $this->getOutputStream(), 0, 0);`
`538`		`- $bar->setRedrawFrequency(0.9);`
	`538`	`+ $bar->setRedrawFrequency(0);`
`539`	`539`	`$bar->start();`
`540`	`540`	`$bar->advance();`
`541`	`541`