Skip to content

Commit

Permalink
fix coding style
Browse files Browse the repository at this point in the history
  • Loading branch information
sgiehl committed Sep 27, 2019
1 parent daa2146 commit 65c9c99
Show file tree
Hide file tree
Showing 3 changed files with 17 additions and 3 deletions.
6 changes: 5 additions & 1 deletion Parser/Client/Browser.php
Original file line number Diff line number Diff line change
Expand Up @@ -357,7 +357,11 @@ public function parse(): ?array
}

// This Exception should never be thrown. If so a defined browser name is missing in $availableBrowsers
throw new \Exception(sprintf('Detected browser name "%s" was not found in $availableBrowsers. Tried to parse user agent: %s', $name, $this->userAgent)); // @codeCoverageIgnore
throw new \Exception(sprintf(
'Detected browser name "%s" was not found in $availableBrowsers. Tried to parse user agent: %s',
$name,
$this->userAgent
)); // @codeCoverageIgnore
}

/**
Expand Down
5 changes: 4 additions & 1 deletion Parser/Client/Browser/Engine.php
Original file line number Diff line number Diff line change
Expand Up @@ -85,6 +85,9 @@ public function parse(): ?array
}

// This Exception should never be thrown. If so a defined browser name is missing in $availableEngines
throw new \Exception(sprintf('Detected browser engine was not found in $availableEngines. Tried to parse user agent: %s', $this->userAgent)); // @codeCoverageIgnore
throw new \Exception(sprintf(
'Detected browser engine was not found in $availableEngines. Tried to parse user agent: %s',
$this->userAgent
)); // @codeCoverageIgnore
}
}
9 changes: 8 additions & 1 deletion Parser/Device/AbstractDeviceParser.php
Original file line number Diff line number Diff line change
Expand Up @@ -717,7 +717,14 @@ public function parse(): ?array

if (false === $brandId) {
// This Exception should never be thrown. If so a defined brand name is missing in $deviceBrands
throw new \Exception(sprintf("The brand with name '%s' should be listed in the deviceBrands array. Tried to parse user agent: %s", $brand, $this->userAgent)); // @codeCoverageIgnore
throw new \Exception(
sprintf(
"The brand with name '%s' should be listed in the deviceBrands array. "
. 'Tried to parse user agent: %s',
$brand,
$this->userAgent
)
); // @codeCoverageIgnore
}

$this->brand = (string) $brandId;
Expand Down

0 comments on commit 65c9c99

Please sign in to comment.