diogo/symfony
Archived
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Fix ProgressHelper redraw when redrawFreq is greater than 1

Browse Source
This commit is contained in:
Giorgio Premi 2013-10-30 14:45:24 +01:00
parent 25160e39d2
commit 5eca1fb01a
2 changed files with 25 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
src/Symfony/Component/Console/Helper/ProgressHelper.php
View File

@ -235,8 +235,12 @@ class ProgressHelper extends Helper
$redraw = true;
}
$prevPeriod = intval($this->current / $this->redrawFreq);
$this->current += $step;
if ($redraw || 0 === $this->current % $this->redrawFreq) {
$currPeriod = intval($this->current / $this->redrawFreq);
if ($redraw || $prevPeriod !== $currPeriod || $this->max === $this->current) {
$this->display();
}
}
@ -265,8 +269,12 @@ class ProgressHelper extends Helper
$redraw = true;
}
$prevPeriod = intval($this->current / $this->redrawFreq);
$this->current = $current;
if ($redraw || 0 === $this->current % $this->redrawFreq) {
$currPeriod = intval($this->current / $this->redrawFreq);
if ($redraw || $prevPeriod !== $currPeriod || $this->max === $this->current) {
$this->display();
}
}

15
src/Symfony/Component/Console/Tests/Helper/ProgressHelperTest.php
View File

@ -136,6 +136,21 @@ class ProgressHelperTest extends \PHPUnit_Framework_TestCase
$progress->setCurrent(10);
}
public function testRedrawFrequency()
{
$progress = $this->getMock('Symfony\Component\Console\Helper\ProgressHelper', array('display'));
$progress->expects($this->exactly(4))
->method('display');
$progress->setRedrawFrequency(2);
$progress->start($output = $this->getOutputStream(), 6);
$progress->setCurrent(1);
$progress->advance(2);
$progress->advance(2);
$progress->advance(1);
}
public function testMultiByteSupport()
{
if (!function_exists('mb_strlen') || (false === $encoding = mb_detect_encoding('■'))) {