Skip to content

Commit

Permalink
Merge pull request #536 from creative-commoners/pulls/2/returntypewil…
Browse files Browse the repository at this point in the history
…lchange

API Removed ReturnTypeWillChange annotation
  • Loading branch information
GuySartorelli authored Jan 23, 2023
2 parents 5f25688 + 9a1e094 commit 8372705
Showing 1 changed file with 5 additions and 10 deletions.
15 changes: 5 additions & 10 deletions src/Storage/DefaultAssetNameGenerator.php
Original file line number Diff line number Diff line change
Expand Up @@ -120,8 +120,7 @@ protected function getPrefix()
return Config::inst()->get(__CLASS__, 'version_prefix');
}

#[\ReturnTypeWillChange]
public function current()
public function current(): string
{
$version = $this->version;

Expand All @@ -146,26 +145,22 @@ public function current()
return $filename;
}

#[\ReturnTypeWillChange]
public function key()
public function key(): int
{
return $this->version - $this->first;
}

#[\ReturnTypeWillChange]
public function next()
public function next(): void
{
$this->version++;
}

#[\ReturnTypeWillChange]
public function rewind()
public function rewind(): void
{
$this->version = $this->first;
}

#[\ReturnTypeWillChange]
public function valid()
public function valid(): bool
{
return $this->version < $this->max + $this->first;
}
Expand Down

0 comments on commit 8372705

Please sign in to comment.