skobkin/point-tools
1
1
Fork 0
Code Issues 4 Pull requests 1 Releases Activity

Small quality fixes in AbstractApi.

Browse source
This commit is contained in:
Alexey Skobkin 2017-01-06 00:57:32 +03:00
parent 4e9c556320
commit a84c2dcd3b
1 changed files with 6 additions and 6 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

12
src/Skobkin/Bundle/PointToolsBundle/Service/AbstractApi.php
View file

@ -144,9 +144,9 @@ class AbstractApi
*
* @return string Base URL of client
*/
public function getBaseUrl()
public function getBaseUrl(): string
{
return $this->client->getBaseUrl();
return (string) $this->client->getBaseUrl();
}
/**
@ -156,7 +156,7 @@ class AbstractApi
* @param bool $useProtocol Do not change URL scheme (http/https) defined in $baseUrl
* @return $this
*/
public function setBaseUrl($baseUrl, $useProtocol = false)
public function setBaseUrl(string $baseUrl, bool $useProtocol = false): self
{
// Overriding protocol
if (!$useProtocol) {
@ -177,7 +177,7 @@ class AbstractApi
*
* @return bool
*/
public function isHttps()
public function isHttps(): bool
{
return $this->useHttps;
}
@ -187,7 +187,7 @@ class AbstractApi
*
* @return $this
*/
public function enableHttps()
public function enableHttps(): self
{
$this->useHttps = true;
$this->setBaseUrl($this->getBaseUrl());
@ -200,7 +200,7 @@ class AbstractApi
*
* @return $this
*/
public function disableHttps()
public function disableHttps(): self
{
$this->useHttps = false;
$this->setBaseUrl($this->getBaseUrl());