Issues in Events.php - New Issues - Inspection of "Merge pull request #331 from Bakual/ComposerUpdate" - joomla-projects/com_localise - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — develop ( 06e328...3bdb15 )

unknown

created 2018-03-29 14:23 UTC

vendor/joomla/github/src/Package/Issues/Events.php (1 issue)

Labels

Duplication 1

Severity

Informational 1

Upgrade to new PHP Analysis Engine

These results are based on our legacy PHP analysis, consider migrating to our new PHP analysis engine instead. Learn more

<?php
/**
 * Part of the Joomla Framework Github Package
 *
 * @copyright  Copyright (C) 2005 - 2018 Open Source Matters, Inc. All rights reserved.
 * @license    GNU General Public License version 2 or later; see LICENSE
 */

namespace Joomla\Github\Package\Issues;

use Joomla\Github\AbstractPackage;

/**
 * GitHub API Issues Events class for the Joomla Framework.
 *
 * Records various events that occur around an Issue or Pull Request.
 * This is useful both for display on issue/pull request information pages and also
 * to determine who should be notified of comments.
 *
 * @link   https://developer.github.com/v3/issues/events/
 *
 * @since  1.0
 */
class Events extends AbstractPackage
{
	/**
	 * List events for an issue.
	 *
	 * @param   string   $owner        The name of the owner of the GitHub repository.
	 * @param   string   $repo         The name of the GitHub repository.
	 * @param   integer  $issueNumber  The issue number.
	 * @param   integer  $page         The page number from which to get items.
	 * @param   integer  $limit        The number of items on a page.
	 *
	 * @return object
	 */
	public function getList($owner, $repo, $issueNumber, $page = 0, $limit = 0)

	{
		// Build the request path.
		$path = '/repos/' . $owner . '/' . $repo . '/issues/' . (int) $issueNumber . '/events';

		// Send the request.
		return $this->processResponse(
			$this->client->get($this->fetchUrl($path, $page, $limit))
		);
	}

	/**
	 * List events for a repository.
	 *
	 * @param   string   $owner    The name of the owner of the GitHub repository.
	 * @param   string   $repo     The name of the GitHub repository.
	 * @param   integer  $issueId  The issue number.
	 * @param   integer  $page     The page number from which to get items.
	 * @param   integer  $limit    The number of items on a page.
	 *
	 * @return object
	 */
	public function getListRepository($owner, $repo, $issueId, $page = 0, $limit = 0)
	{
		// Build the request path.
		$path = '/repos/' . $owner . '/' . $repo . '/issues/' . (int) $issueId . '/comments';

		// Send the request.
		return $this->processResponse(
			$this->client->get($this->fetchUrl($path, $page, $limit))
		);
	}

	/**
	 * Get a single event.
	 *
	 * @param   string   $owner  The name of the owner of the GitHub repository.
	 * @param   string   $repo   The name of the GitHub repository.
	 * @param   integer  $id     The event number.
	 *
	 * @return object
	 */
	public function get($owner, $repo, $id)
	{
		// Build the request path.
		$path = '/repos/' . $owner . '/' . $repo . '/issues/events/' . (int) $id;

		// Send the request.
		return $this->processResponse(
			$this->client->get($this->fetchUrl($path))
		);
	}
}


GitHub Access Token became invalid

Push — develop ( 06e328...3bdb15 )

vendor/joomla/github/src/Package/Issues/Events.php (1 issue)

Labels

Severity

Introduced By

Upgrade to new PHP Analysis Engine

1		<?php
2		/**
3		* Part of the Joomla Framework Github Package
4		*
5		* @copyright Copyright (C) 2005 - 2018 Open Source Matters, Inc. All rights reserved.
6		* @license GNU General Public License version 2 or later; see LICENSE
7		*/
8
9		namespace Joomla\Github\Package\Issues;
10
11		use Joomla\Github\AbstractPackage;
12
13		/**
14		* GitHub API Issues Events class for the Joomla Framework.
15		*
16		* Records various events that occur around an Issue or Pull Request.
17		* This is useful both for display on issue/pull request information pages and also
18		* to determine who should be notified of comments.
19		*
20		* @link https://developer.github.com/v3/issues/events/
21		*
22		* @since 1.0
23		*/
24		class Events extends AbstractPackage
25		{
26		/**
27		* List events for an issue.
28		*
29		* @param string $owner The name of the owner of the GitHub repository.
30		* @param string $repo The name of the GitHub repository.
31		* @param integer $issueNumber The issue number.
32		* @param integer $page The page number from which to get items.
33		* @param integer $limit The number of items on a page.
34		*
35		* @return object
36		*/
37	View Code Duplication	public function getList($owner, $repo, $issueNumber, $page = 0, $limit = 0)
		0 ignored issues – show Duplication introduced 2018-03-29 06:21 UTC by Report Bug Copy Issue Report Show Similar Issues like this This method seems to be duplicated in 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...
38		{
39		// Build the request path.
40		$path = '/repos/' . $owner . '/' . $repo . '/issues/' . (int) $issueNumber . '/events';
41
42		// Send the request.
43		return $this->processResponse(
44		$this->client->get($this->fetchUrl($path, $page, $limit))
45		);
46		}
47
48		/**
49		* List events for a repository.
50		*
51		* @param string $owner The name of the owner of the GitHub repository.
52		* @param string $repo The name of the GitHub repository.
53		* @param integer $issueId The issue number.
54		* @param integer $page The page number from which to get items.
55		* @param integer $limit The number of items on a page.
56		*
57		* @return object
58		*/
59		public function getListRepository($owner, $repo, $issueId, $page = 0, $limit = 0)
60		{
61		// Build the request path.
62		$path = '/repos/' . $owner . '/' . $repo . '/issues/' . (int) $issueId . '/comments';
63
64		// Send the request.
65		return $this->processResponse(
66		$this->client->get($this->fetchUrl($path, $page, $limit))
67		);
68		}
69
70		/**
71		* Get a single event.
72		*
73		* @param string $owner The name of the owner of the GitHub repository.
74		* @param string $repo The name of the GitHub repository.
75		* @param integer $id The event number.
76		*
77		* @return object
78		*/
79		public function get($owner, $repo, $id)
80		{
81		// Build the request path.
82		$path = '/repos/' . $owner . '/' . $repo . '/issues/events/' . (int) $id;
83
84		// Send the request.
85		return $this->processResponse(
86		$this->client->get($this->fetchUrl($path))
87		);
88		}
89		}
90

joomla-projects / com_localise

GitHub Access Token became invalid

Push — develop ( 06e328...3bdb15 )

vendor/joomla/github/src/Package/Issues/Events.php (1 issue)

Labels

Severity

Introduced By

Upgrade to new PHP Analysis Engine

Duplication Side-by-Side

Filter issues like