Skip to content

Commit

Permalink
fix coding style
Browse files Browse the repository at this point in the history
  • Loading branch information
sgiehl committed Dec 31, 2019
1 parent c28aa5d commit a0c0635
Show file tree
Hide file tree
Showing 3 changed files with 16 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 @@ -410,7 +410,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 @@ -86,6 +86,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
}
}
8 changes: 7 additions & 1 deletion Parser/Device/AbstractDeviceParser.php
Original file line number Diff line number Diff line change
Expand Up @@ -751,7 +751,13 @@ 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 name '%s' needs to be in deviceBrands array. Tried parsing user agent: %s",
$brand,
$this->userAgent
)
); // @codeCoverageIgnore
}

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

0 comments on commit a0c0635

Please sign in to comment.