Manga::transform() - Code Metrics - Inspection of "Remove allowed failures for PHP 5.5 & 5.6" - timw4mail/HummingBirdAnimeClient - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Branch — develop (d8e67e)

by Timothy

created 2016-04-08 19:46 UTC

Manga::transform() A

↳ Parent: Manga

Complexity

Conditions	2
Paths	2

Size

Total Lines	14
Code Lines	8

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	1

Metric	Value
c	1
b	0
f	1
dl	0
loc	14
rs	9.4285
cc	2
eloc	8
nc	2
nop	1

<?php
/**
 * Hummingbird Anime Client
 *
 * An API client for Hummingbird to manage anime and manga watch lists
 *
 * @package     HummingbirdAnimeClient
 * @author      Timothy J. Warren
 * @copyright   Copyright (c) 2015 - 2016
 * @link        https://github.com/timw4mail/HummingBirdAnimeClient
 * @license     MIT
 */

namespace Aviat\AnimeClient\Model;

use GuzzleHttp\Cookie\Cookiejar;
use GuzzleHttp\Cookie\SetCookie;

use Aviat\Ion\Json;
use Aviat\AnimeClient\Model\API;
use Aviat\AnimeClient\Hummingbird\Transformer;
use Aviat\AnimeClient\Hummingbird\Enum\MangaReadingStatus;

/**
 * Model for handling requests dealing with the manga list
 */
class Manga extends API {

	const READING = 'Reading';
	const PLAN_TO_READ = 'Plan to Read';
	const DROPPED = 'Dropped';
	const ON_HOLD = 'On Hold';
	const COMPLETED = 'Completed';

	/**
	 * Map API constants to display constants
	 * @var array
	 */
	protected $const_map = [
		MangaReadingStatus::READING => self::READING,
		MangaReadingStatus::PLAN_TO_READ => self::PLAN_TO_READ,
		MangaReadingStatus::ON_HOLD => self::ON_HOLD,
		MangaReadingStatus::DROPPED => self::DROPPED,
		MangaReadingStatus::COMPLETED => self::COMPLETED
	];

	/**
	 * The base url for api requests
	 * @var string
	 */
	protected $base_url = "https://hummingbird.me/";

	/**
	 * Make an authenticated manga API call
	 *
	 * @param string $type - the HTTP verb
	 * @param string $url
	 * @param string|null $json
	 * @return array
	 */
	protected function _manga_api_call($type, $url, $json = NULL)
	{
		$token = $this->container->get('auth')
			->get_auth_token();

		// Set the token cookie, with the authentication token
		// from the auth class.
		$cookieJar = $this->cookieJar;
		$cookie_data = new SetCookie([
			'Name' => 'token',
			'Value' => $token,
			'Domain' => 'hummingbird.me'
		]);
		$cookieJar->setCookie($cookie_data);

		$config = [
			'cookies' => $cookieJar
		];

		if ( ! is_null($json))
		{
			$config['json'] = $json;
		}

		$result = $this->client->request(strtoupper($type), $url, $config);

		return [
			'statusCode' => $result->getStatusCode(),
			'body' => $result->getBody()
		];
	}

	/**
	 * Add a manga to the list
	 *
	 * @param array $data
	 */
	public function add($data)
	{
		$object = [
			'manga_library_entry' => [
				'status' => $data['status'],
				'manga_id' => $data['id']
			]
		];

		return $this->_manga_api_call('post', 'manga_library_entries', $object);
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
	}

	/**
	 * Update the selected manga
	 *
	 * @param array $data
	 * @return array
	 */
	public function update($data)
	{
		$id = $data['id'];

		return $this->_manga_api_call(
			'put',
			"manga_library_entries/{$id}",
			['manga_library_entry' => $data]
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
		);
	}

	/**
	 * Delete a manga entry
	 *
	 * @param  array $data
	 * @return array
	 */
	public function delete($data)
	{
		$id = $data['id'];

		return $this->_manga_api_call('delete', "manga_library_entries/{$id}");
	}

	/**
	 * Search for manga by name
	 *
	 * @param string $name
	 * @return array
	 */
	public function search($name)
	{
		$logger = $this->container->getLogger('default');

		$config = [
			'query' => [
				'scope' => 'manga',
				'depth' => 'full',
				'query' => $name
			]
		];

		$response = $this->get('search.json', $config);

		if ($response->getStatusCode() != 200)

		{
			$logger->warning("Non 200 response for search api call");
			$logger->warning($response->getBody());

			throw new RuntimeException($response->getEffectiveUrl());
		}

		return Json::decode($response->getBody(), TRUE);
	}

	/**
	 * Get the full set of anime lists
	 *
	 * @return array
	 */
	public function get_all_lists()
	{
		$data = $this->_get_list_from_api();

		foreach ($data as &$val)
		{
			$this->sort_by_name($val, 'manga');
		}

		return $data;
	}

	/**
	 * Get a category out of the full list
	 *
	 * @param string $status
	 * @return array
	 */
	public function get_list($status)
	{
		$data = $this->_get_list_from_api($status);
		$this->sort_by_name($data, 'manga');

		return $data;
	}

	/**
	 * Retrieve the list from the hummingbird api
	 *
	 * @param  string $status
	 * @return array
	 */
	protected function _get_list_from_api($status = "All")
	{
		$config = [
			'query' => [
				'user_id' => $this->config->get('hummingbird_username')
			],
			'allow_redirects' => FALSE
		];

		$response = $this->get('manga_library_entries', $config);
		$data = $this->transform($response);
		$output = $this->map_by_status($data);

		return (array_key_exists($status, $output))
			? $output[$status]
			: $output;
	}

	/**
	 * Check the status of the cache and return the appropriate response
	 *
	 * @param \GuzzleHttp\Message\Response $response
	 * @codeCoverageIgnore
	 * @return array
	 */
	private function transform($response)
	{
		// Bail out early if there isn't any manga data
		$api_data = Json::decode($response->getBody(), TRUE);
		if ( ! array_key_exists('manga', $api_data))
		{
			return [];
		}

		$zippered_data = $this->zipper_lists($api_data);
		$transformer = new Transformer\MangaListTransformer();
		$transformed_data = $transformer->transform_collection($zippered_data);
		return $transformed_data;
	}

	/**
	 * Map transformed anime data to be organized by reading status
	 *
	 * @param array $data
	 * @return array
	 */
	private function map_by_status($data)
	{
		$output = [
			self::READING => [],
			self::PLAN_TO_READ => [],
			self::ON_HOLD => [],
			self::DROPPED => [],
			self::COMPLETED => [],
		];

		foreach ($data as &$entry)
		{
			$entry['manga']['image'] = $this->get_cached_image(
				$entry['manga']['image'],
				$entry['manga']['slug'],
				'manga'
			);
			$key = $this->const_map[$entry['reading_status']];
			$output[$key][] = $entry;
		}

		return $output;
	}

	/**
	 * Combine the two manga lists into one
	 * @param  array $raw_data
	 * @return array
	 */
	private function zipper_lists($raw_data)
	{
		return (new Transformer\MangaListsZipper($raw_data))->transform();
	}
}
// End of MangaModel.php

1		<?php
2		/**
3		* Hummingbird Anime Client
4		*
5		* An API client for Hummingbird to manage anime and manga watch lists
6		*
7		* @package HummingbirdAnimeClient
8		* @author Timothy J. Warren
9		* @copyright Copyright (c) 2015 - 2016
10		* @link https://github.com/timw4mail/HummingBirdAnimeClient
11		* @license MIT
12		*/
13
14		namespace Aviat\AnimeClient\Model;
15
16		use GuzzleHttp\Cookie\Cookiejar;
17		use GuzzleHttp\Cookie\SetCookie;
18
19		use Aviat\Ion\Json;
20		use Aviat\AnimeClient\Model\API;
21		use Aviat\AnimeClient\Hummingbird\Transformer;
22		use Aviat\AnimeClient\Hummingbird\Enum\MangaReadingStatus;
23
24		/**
25		* Model for handling requests dealing with the manga list
26		*/
27		class Manga extends API {
28
29		const READING = 'Reading';
30		const PLAN_TO_READ = 'Plan to Read';
31		const DROPPED = 'Dropped';
32		const ON_HOLD = 'On Hold';
33		const COMPLETED = 'Completed';
34
35		/**
36		* Map API constants to display constants
37		* @var array
38		*/
39		protected $const_map = [
40		MangaReadingStatus::READING => self::READING,
41		MangaReadingStatus::PLAN_TO_READ => self::PLAN_TO_READ,
42		MangaReadingStatus::ON_HOLD => self::ON_HOLD,
43		MangaReadingStatus::DROPPED => self::DROPPED,
44		MangaReadingStatus::COMPLETED => self::COMPLETED
45		];
46
47		/**
48		* The base url for api requests
49		* @var string
50		*/
51		protected $base_url = "https://hummingbird.me/";
52
53		/**
54		* Make an authenticated manga API call
55		*
56		* @param string $type - the HTTP verb
57		* @param string $url
58		* @param string\|null $json
59		* @return array
60		*/
61		protected function _manga_api_call($type, $url, $json = NULL)
62		{
63		$token = $this->container->get('auth')
64		->get_auth_token();
65
66		// Set the token cookie, with the authentication token
67		// from the auth class.
68		$cookieJar = $this->cookieJar;
69		$cookie_data = new SetCookie([
70		'Name' => 'token',
71		'Value' => $token,
72		'Domain' => 'hummingbird.me'
73		]);
74		$cookieJar->setCookie($cookie_data);
75
76		$config = [
77		'cookies' => $cookieJar
78		];
79
80		if ( ! is_null($json))
81		{
82		$config['json'] = $json;
83		}
84
85		$result = $this->client->request(strtoupper($type), $url, $config);
86
87		return [
88		'statusCode' => $result->getStatusCode(),
89		'body' => $result->getBody()
90		];
91		}
92
93		/**
94		* Add a manga to the list
95		*
96		* @param array $data
97		*/
98		public function add($data)
99		{
100		$object = [
101		'manga_library_entry' => [
102		'status' => $data['status'],
103		'manga_id' => $data['id']
104		]
105		];
106
107		return $this->_manga_api_call('post', 'manga_library_entries', $object);
		0 ignored issues – show Documentation introduced 2016-02-10 01:15 UTC by Report Bug Copy Issue Report `$object` is of type `array<string,array<strin...\"manga_id\":\"?\"}>"}>`, but the function expects a `string\|null`. It seems like the type of the argument is not accepted by the function/method which you are calling. In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug. We suggest to add an explicit type cast like in the following example: function acceptsInteger($int) { } $x = '123'; // string "123" // Instead of acceptsInteger($x); // we recommend to use acceptsInteger((integer) $x); Loading history...
108		}
109
110		/**
111		* Update the selected manga
112		*
113		* @param array $data
114		* @return array
115		*/
116		public function update($data)
117		{
118		$id = $data['id'];
119
120		return $this->_manga_api_call(
121		'put',
122		"manga_library_entries/{$id}",
123		['manga_library_entry' => $data]
		0 ignored issues – show Documentation introduced 2016-02-10 01:15 UTC by Report Bug Copy Issue Report `array('manga_library_entry' => $data)` is of type `array<string,array,{"man...ibrary_entry":"array"}>`, but the function expects a `string\|null`. It seems like the type of the argument is not accepted by the function/method which you are calling. In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug. We suggest to add an explicit type cast like in the following example: function acceptsInteger($int) { } $x = '123'; // string "123" // Instead of acceptsInteger($x); // we recommend to use acceptsInteger((integer) $x); Loading history...
124		);
125		}
126
127		/**
128		* Delete a manga entry
129		*
130		* @param array $data
131		* @return array
132		*/
133		public function delete($data)
134		{
135		$id = $data['id'];
136
137		return $this->_manga_api_call('delete', "manga_library_entries/{$id}");
138		}
139
140		/**
141		* Search for manga by name
142		*
143		* @param string $name
144		* @return array
145		*/
146		public function search($name)
147		{
148		$logger = $this->container->getLogger('default');
149
150		$config = [
151		'query' => [
152		'scope' => 'manga',
153		'depth' => 'full',
154		'query' => $name
155		]
156		];
157
158		$response = $this->get('search.json', $config);
159
160	View Code Duplication	if ($response->getStatusCode() != 200)
		0 ignored issues – show Duplication introduced 2016-02-02 16:37 UTC by Report Bug Copy Issue Report This code seems to be duplicated across your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
161		{
162		$logger->warning("Non 200 response for search api call");
163		$logger->warning($response->getBody());
164
165		throw new RuntimeException($response->getEffectiveUrl());
166		}
167
168		return Json::decode($response->getBody(), TRUE);
169		}
170
171		/**
172		* Get the full set of anime lists
173		*
174		* @return array
175		*/
176		public function get_all_lists()
177		{
178		$data = $this->_get_list_from_api();
179
180		foreach ($data as &$val)
181		{
182		$this->sort_by_name($val, 'manga');
183		}
184
185		return $data;
186		}
187
188		/**
189		* Get a category out of the full list
190		*
191		* @param string $status
192		* @return array
193		*/
194		public function get_list($status)
195		{
196		$data = $this->_get_list_from_api($status);
197		$this->sort_by_name($data, 'manga');
198
199		return $data;
200		}
201
202		/**
203		* Retrieve the list from the hummingbird api
204		*
205		* @param string $status
206		* @return array
207		*/
208		protected function _get_list_from_api($status = "All")
209		{
210		$config = [
211		'query' => [
212		'user_id' => $this->config->get('hummingbird_username')
213		],
214		'allow_redirects' => FALSE
215		];
216
217		$response = $this->get('manga_library_entries', $config);
218		$data = $this->transform($response);
219		$output = $this->map_by_status($data);
220
221		return (array_key_exists($status, $output))
222		? $output[$status]
223		: $output;
224		}
225
226		/**
227		* Check the status of the cache and return the appropriate response
228		*
229		* @param \GuzzleHttp\Message\Response $response
230		* @codeCoverageIgnore
231		* @return array
232		*/
233		private function transform($response)
234		{
235		// Bail out early if there isn't any manga data
236		$api_data = Json::decode($response->getBody(), TRUE);
237		if ( ! array_key_exists('manga', $api_data))
238		{
239		return [];
240		}
241
242		$zippered_data = $this->zipper_lists($api_data);
243		$transformer = new Transformer\MangaListTransformer();
244		$transformed_data = $transformer->transform_collection($zippered_data);
245		return $transformed_data;
246		}
247
248		/**
249		* Map transformed anime data to be organized by reading status
250		*
251		* @param array $data
252		* @return array
253		*/
254		private function map_by_status($data)
255		{
256		$output = [
257		self::READING => [],
258		self::PLAN_TO_READ => [],
259		self::ON_HOLD => [],
260		self::DROPPED => [],
261		self::COMPLETED => [],
262		];
263
264		foreach ($data as &$entry)
265		{
266		$entry['manga']['image'] = $this->get_cached_image(
267		$entry['manga']['image'],
268		$entry['manga']['slug'],
269		'manga'
270		);
271		$key = $this->const_map[$entry['reading_status']];
272		$output[$key][] = $entry;
273		}
274
275		return $output;
276		}
277
278		/**
279		* Combine the two manga lists into one
280		* @param array $raw_data
281		* @return array
282		*/
283		private function zipper_lists($raw_data)
284		{
285		return (new Transformer\MangaListsZipper($raw_data))->transform();
286		}
287		}
288		// End of MangaModel.php

timw4mail / HummingBirdAnimeClient

Branch — develop (d8e67e)

Manga::transform() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like