Thanks to visit codestin.com
Credit goes to github.com

Skip to content

[ProgressBar] Empty iterable throws Exception on "maximum number of steps is not set" #47259

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 6 commits into from
Closed

[ProgressBar] Empty iterable throws Exception on "maximum number of steps is not set" #47259

Changes from 1 commit
Commits
Show all changes
6 commits
Select commit Hold shift + click to select a range
010af2a
Fix: ProgressBar does not fail on emtpy Iterable
freimair Aug 12, 2022
50d6cd1
Fix: divide by Zero chance in ProgressBar
freimair Aug 12, 2022
eac82fc
Set max to int since it cannot be null
freimair Aug 23, 2022
9701863
Introduce isMaxKnown variable
freimair Aug 23, 2022
d0cc3bd
Ui work
freimair Aug 23, 2022
42a0884
Set maxKnown also in start function
freimair Oct 23, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Fix: divide by Zero chance in ProgressBar
  • Loading branch information
@freimair
freimair committed Aug 12, 2022
commit 50d6cd18c7fba0b7f619eb5808308e6bc5d74755
2 changes: 1 addition & 1 deletion src/Symfony/Component/Console/Helper/ProgressBar.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -209,7 +209,7 @@ public function getEstimated(): float

public function getRemaining(): float
{
if (!$this->step) {
if (!$this->step || $this->step === $this->startingStep) {
return 0;
}

Expand Down