@@ -84,7 +84,7 @@ discard block |
||
84 | 84 | } |
85 | 85 | |
86 | 86 | /** |
87 | - * @param $regionCode |
|
87 | + * @param string $regionCode |
|
88 | 88 | * |
89 | 89 | * @throws \ErrorException |
90 | 90 | */ |
@@ -99,7 +99,7 @@ discard block |
||
99 | 99 | } |
100 | 100 | |
101 | 101 | /** |
102 | - * @param $url |
|
102 | + * @param string $url |
|
103 | 103 | * |
104 | 104 | * @return string |
105 | 105 | */ |
@@ -143,6 +143,9 @@ discard block |
||
143 | 143 | } |
144 | 144 | } |
145 | 145 | |
146 | + /** |
|
147 | + * @param \Psr\Http\Message\ResponseInterface|null $response |
|
148 | + */ |
|
146 | 149 | public function handleError($response) |
147 | 150 | { |
148 | 151 | $msg = $response->getBody(); |
@@ -173,7 +176,7 @@ discard block |
||
173 | 176 | * @param string $url |
174 | 177 | * @param array $query |
175 | 178 | * |
176 | - * @return mixed |
|
179 | + * @return QuiteSimpleXMLElement |
|
177 | 180 | */ |
178 | 181 | public function getXML($url, $query = []) |
179 | 182 | { |
@@ -61,7 +61,7 @@ |
||
61 | 61 | /** |
62 | 62 | * Get the services provided by the provider. |
63 | 63 | * |
64 | - * @return array |
|
64 | + * @return string[] |
|
65 | 65 | */ |
66 | 66 | public function provides() |
67 | 67 | { |
@@ -2,7 +2,6 @@ |
||
2 | 2 | |
3 | 3 | namespace Scriptotek\Alma\Providers; |
4 | 4 | |
5 | -use Illuminate\Foundation\Application; |
|
6 | 5 | use Illuminate\Support\ServiceProvider; |
7 | 6 | use Scriptotek\Alma\Client as AlmaClient; |
8 | 7 | use Scriptotek\Sru\Client as SruClient; |
@@ -2,7 +2,6 @@ |
||
2 | 2 | |
3 | 3 | namespace spec\Scriptotek\Alma\Bibs; |
4 | 4 | |
5 | -use Danmichaelo\QuiteSimpleXMLElement\QuiteSimpleXMLElement; |
|
6 | 5 | use PhpSpec\ObjectBehavior; |
7 | 6 | use Prophecy\Argument; |
8 | 7 | use Scriptotek\Alma\Bibs\Bib; |
@@ -42,6 +42,9 @@ |
||
42 | 42 | return $this->data; |
43 | 43 | } |
44 | 44 | |
45 | + /** |
|
46 | + * @param string $id_type |
|
47 | + */ |
|
45 | 48 | public function getIdOfType($id_type) |
46 | 49 | { |
47 | 50 | foreach ($this->user_identifier as $identifier) { |
@@ -37,6 +37,9 @@ discard block |
||
37 | 37 | } |
38 | 38 | } |
39 | 39 | |
40 | + /** |
|
41 | + * @param string $resumptionToken |
|
42 | + */ |
|
40 | 43 | protected function fetchRows($resumptionToken = null) |
41 | 44 | { |
42 | 45 | $results = $this->client->getXML('/analytics/reports', [ |
@@ -56,7 +59,7 @@ discard block |
||
56 | 59 | } |
57 | 60 | |
58 | 61 | /** |
59 | - * @param $results |
|
62 | + * @param \Danmichaelo\QuiteSimpleXMLElement\QuiteSimpleXMLElement $results |
|
60 | 63 | */ |
61 | 64 | protected function readResponseHeaders($results) |
62 | 65 | { |
@@ -75,7 +78,7 @@ discard block |
||
75 | 78 | } |
76 | 79 | |
77 | 80 | /** |
78 | - * @return \Generator|Row[] |
|
81 | + * @return \Generator |
|
79 | 82 | */ |
80 | 83 | public function getRows() |
81 | 84 | { |