Members - Code Metrics - Inspection of "Merge pull request #322 from Bakual/Composer" - joomla-projects/com_localise - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — develop ( b5e3c2...7799bc )

unknown

created 2017-12-21 09:01 UTC

Members A

↳ Parent: Project

Complexity

Total Complexity

Size/Duplication

Total Lines	347
Duplicated Lines	28.53 %

Coupling/Cohesion

Components	1
Dependencies	3

Importance

Changes

Metric	Value
dl	99
loc	347
rs	10
c	0
b	0
f	0
wmc	22
lcom	1
cbo	3

13 Methods

Rating	Name	Duplication	Size	Complexity
A	getList()	23	23	3
B	check()	30	30	4
A	remove()	0	10	1
A	getListPublic()	9	9	1
B	checkPublic()	24	24	3
A	publicize()	0	10	1
A	conceal()	0	10	1
A	getMembership()	0	7	1
A	updateMembership()	0	18	2
A	removeMembership()	0	10	1
A	listMemberships()	0	7	1
A	listOrganizationMembership()	0	7	1
A	editOrganizationMembership()	13	13	2

How to fix Duplicated Code

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

namespace Joomla\Github\Package\Orgs;

use Joomla\Github\AbstractPackage;

/**
 * GitHub API Orgs Members class for the Joomla Framework.
 *
 * @documentation http://developer.github.com/v3/orgs/members/
 *
 * @since  1.0
 */
class Members extends AbstractPackage
{
	/**
	 * Members list.
	 *
	 * List all users who are members of an organization.
	 * A member is a user that belongs to at least 1 team in the organization.
	 * If the authenticated user is also a member of this organization then
	 * both concealed and public members will be returned.
	 * If the requester is not a member of the organization the query will be
	 * redirected to the public members list.
	 *
	 * @param   string  $org  The name of the organization.
	 *
	 * @throws \UnexpectedValueException
	 * @since  1.0
	 *
	 * @return boolean|mixed
	 */
	public function getList($org)

	{
		// Build the request path.
		$path = '/orgs/' . $org . '/members';

		$response = $this->client->get($this->fetchUrl($path));

		switch ($response->code)
		{
			case 302 :
				// Requester is not an organization member.
				return false;
				break;
switch ($x) {
    case 1:
        return 'foo';
        break; // This break is not necessary and can be left off.
}

			case 200 :
				return json_decode($response->body);
				break;
switch ($x) {
    case 1:
        return 'foo';
        break; // This break is not necessary and can be left off.
}

			default :
				throw new \UnexpectedValueException('Unexpected response code: ' . $response->code);
				break;
function fx() {
    try {
        doSomething();
        return true;
    }
    catch (\Exception $e) {
        return false;
    }

    return false;
}
		}
	}

	/**
	 * Check membership.
	 *
	 * Check if a user is, publicly or privately, a member of the organization.
	 *
	 * @param   string  $org   The name of the organization.
	 * @param   string  $user  The name of the user.
	 *
	 * @throws \UnexpectedValueException
	 * @since  1.0
	 *
	 * @return boolean
	 */
	public function check($org, $user)

	{
		// Build the request path.
		$path = '/orgs/' . $org . '/members/' . $user;

		$response = $this->client->get($this->fetchUrl($path));

		switch ($response->code)
		{
			case 204 :
				// Requester is an organization member and user is a member.
				return true;
				break;
switch ($x) {
    case 1:
        return 'foo';
        break; // This break is not necessary and can be left off.
}

			case 404 :
				// Requester is an organization member and user is not a member.
				// Requester is not an organization member and is inquiring about themselves.
				return false;
				break;
switch ($x) {
    case 1:
        return 'foo';
        break; // This break is not necessary and can be left off.
}

			case 302 :
				// Requester is not an organization member.
				return false;
				break;
switch ($x) {
    case 1:
        return 'foo';
        break; // This break is not necessary and can be left off.
}

			default :
				throw new \UnexpectedValueException('Unexpected response code: ' . $response->code);
				break;
function fx() {
    try {
        doSomething();
        return true;
    }
    catch (\Exception $e) {
        return false;
    }

    return false;
}
		}
	}

	/**
	 * Add a member.
	 *
	 * To add someone as a member to an org, you must add them to a team.
	 */

	/**
	 * Remove a member.
	 *
	 * Removing a user from this list will remove them from all teams and they will no longer have
	 * any access to the organization’s repositories.
	 *
	 * @param   string  $org   The name of the organization.
	 * @param   string  $user  The name of the user.
	 *
	 * @since  1.0
	 *
	 * @return object
	 */
	public function remove($org, $user)
	{
		// Build the request path.
		$path = '/orgs/' . $org . '/members/' . $user;

		return $this->processResponse(
			$this->client->delete($this->fetchUrl($path)),
			204
		);
	}

	/**
	 * Public members list.
	 *
	 * Members of an organization can choose to have their membership publicized or not.
	 *
	 * @param   string  $org  The name of the organization.
	 *
	 * @since  1.0
	 *
	 * @return object
	 */
	public function getListPublic($org)

	{
		// Build the request path.
		$path = '/orgs/' . $org . '/public_members';

		return $this->processResponse(
			$this->client->get($this->fetchUrl($path))
		);
	}

	/**
	 * Check public membership.
	 *
	 * @param   string  $org   The name of the organization.
	 * @param   string  $user  The name of the user.
	 *
	 * @throws \UnexpectedValueException
	 * @since  1.0
	 *
	 * @return boolean
	 */
	public function checkPublic($org, $user)

	{
		// Build the request path.
		$path = '/orgs/' . $org . '/public_members/' . $user;

		$response = $this->client->get($this->fetchUrl($path));

		switch ($response->code)
		{
			case 204 :
				// Response if user is a public member.
				return true;
				break;
switch ($x) {
    case 1:
        return 'foo';
        break; // This break is not necessary and can be left off.
}

			case 404 :
				// Response if user is not a public member.
				return false;
				break;
switch ($x) {
    case 1:
        return 'foo';
        break; // This break is not necessary and can be left off.
}

			default :
				throw new \UnexpectedValueException('Unexpected response code: ' . $response->code);
				break;
function fx() {
    try {
        doSomething();
        return true;
    }
    catch (\Exception $e) {
        return false;
    }

    return false;
}
		}
	}

	/**
	 * Publicize a user’s membership.
	 *
	 * @param   string  $org   The name of the organization.
	 * @param   string  $user  The name of the user.
	 *
	 * @since  1.0
	 *
	 * @return object
	 */
	public function publicize($org, $user)
	{
		// Build the request path.
		$path = '/orgs/' . $org . '/public_members/' . $user;

		return $this->processResponse(
			$this->client->put($this->fetchUrl($path), ''),
			204
		);
	}

	/**
	 * Conceal a user’s membership.
	 *
	 * @param   string  $org   The name of the organization.
	 * @param   string  $user  The name of the user.
	 *
	 * @since  1.0
	 *
	 * @return object
	 */
	public function conceal($org, $user)
	{
		// Build the request path.
		$path = '/orgs/' . $org . '/public_members/' . $user;

		return $this->processResponse(
			$this->client->delete($this->fetchUrl($path)),
			204
		);
	}

	/**
	 * Get organization membership
	 *
	 * In order to get a user's membership with an organization, the authenticated user must be an organization owner.
	 *
	 * @param   string  $org   The name of the organization.
	 * @param   string  $user  The name of the user.
	 *
	 * @return  object
	 *
	 * @since   1.4.0
	 */
	public function getMembership($org, $user)
	{
		// Build the request path.
		$path = '/orgs/' . $org . '/memberships/' . $user;

		return $this->processResponse($this->client->get($this->fetchUrl($path)));
	}

	/**
	 * Add or update organization membership
	 *
	 * In order to create or update a user's membership with an organization, the authenticated user must be an organization owner.
	 *
	 * @param   string  $org   The name of the organization.
	 * @param   string  $user  The name of the user.
	 * @param   string  $role  The role to give the user in the organization. Can be either 'member' or 'admin'.
	 *
	 * @return  object
	 *
	 * @since   1.4.0
	 */
	public function updateMembership($org, $user, $role = 'member')
	{
		$allowedRoles = array('member', 'admin');

		if (!in_array($role, $allowedRoles))
		{
			throw new \InvalidArgumentException(sprintf("The user's role must be: %s", implode(', ', $allowedRoles)));
		}

		// Build the request path.
		$path = "/orgs/$org/memberships/$user";

		$data = array(
			'role' => $role,
		);

		return $this->processResponse($this->client->put($this->fetchUrl($path), json_encode($data)));
	}

	/**
	 * Remove organization membership
	 *
	 * In order to remove a user's membership with an organization, the authenticated user must be an organization owner.
	 *
	 * @param   string  $org   The name of the organization.
	 * @param   string  $user  The name of the user.
	 *
	 * @return  object
	 *
	 * @since   1.4.0
	 */
	public function removeMembership($org, $user)
	{
		// Build the request path.
		$path = '/orgs/' . $org . '/memberships/' . $user;

		return $this->processResponse(
			$this->client->delete($this->fetchUrl($path)),
			204
		);
	}

	/**
	 * List your organization memberships
	 *
	 * @return  object
	 *
	 * @since   1.4.0
	 */
	public function listMemberships()
	{
		// Build the request path.
		$path = '/user/memberships/orgs';

		return $this->processResponse($this->client->get($this->fetchUrl($path)));
	}

	/**
	 * Get your organization membership
	 *
	 * @param   string  $org  The name of the organization.
	 *
	 * @return  object
	 *
	 * @since   1.4.0
	 */
	public function listOrganizationMembership($org)
	{
		// Build the request path.
		$path = '/user/memberships/orgs/' . $org;

		return $this->processResponse($this->client->get($this->fetchUrl($path)));
	}

	/**
	 * Edit your organization membership
	 *
	 * @param   string  $org    The name of the organization.
	 * @param   string  $state  The state that the membership should be in.
	 *
	 * @return  object
	 *
	 * @since   1.4.0
	 */
	public function editOrganizationMembership($org, $state)

	{
		// The API only accepts $state == 'active' at present
		if ($state != 'active')
		{
			throw new \InvalidArgumentException('The state must be "active".');
		}

		// Build the request path.
		$path = '/user/memberships/orgs/' . $org;

		return $this->processResponse($this->client->patch($this->fetchUrl($path), array('state' => $state)));
	}
}


1		<?php
2		/**
3		* Part of the Joomla Framework Github Package
4		*
5		* @copyright Copyright (C) 2005 - 2015 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\Orgs;
10
11		use Joomla\Github\AbstractPackage;
12
13		/**
14		* GitHub API Orgs Members class for the Joomla Framework.
15		*
16		* @documentation http://developer.github.com/v3/orgs/members/
17		*
18		* @since 1.0
19		*/
20		class Members extends AbstractPackage
21		{
22		/**
23		* Members list.
24		*
25		* List all users who are members of an organization.
26		* A member is a user that belongs to at least 1 team in the organization.
27		* If the authenticated user is also a member of this organization then
28		* both concealed and public members will be returned.
29		* If the requester is not a member of the organization the query will be
30		* redirected to the public members list.
31		*
32		* @param string $org The name of the organization.
33		*
34		* @throws \UnexpectedValueException
35		* @since 1.0
36		*
37		* @return boolean\|mixed
38		*/
39	View Code Duplication	public function getList($org)
		0 ignored issues – show Duplication introduced 2017-12-19 12:54 UTC by Report Bug Copy Issue Report 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...
40		{
41		// Build the request path.
42		$path = '/orgs/' . $org . '/members';
43
44		$response = $this->client->get($this->fetchUrl($path));
45
46		switch ($response->code)
47		{
48		case 302 :
49		// Requester is not an organization member.
50		return false;
51		break;
		0 ignored issues – show Unused Code introduced 2017-12-19 12:54 UTC by Report Bug Copy Issue Report `break` is not strictly necessary here and could be removed. The `break` statement is not necessary if it is preceded for example by a `return` statement: switch ($x) { case 1: return 'foo'; break; // This break is not necessary and can be left off. } If you would like to keep this construct to be consistent with other `case` statements, you can safely mark this issue as a false-positive. Loading history...
52
53		case 200 :
54		return json_decode($response->body);
55		break;
		0 ignored issues – show Unused Code introduced 2017-12-19 12:54 UTC by Report Bug Copy Issue Report `break` is not strictly necessary here and could be removed. The `break` statement is not necessary if it is preceded for example by a `return` statement: switch ($x) { case 1: return 'foo'; break; // This break is not necessary and can be left off. } If you would like to keep this construct to be consistent with other `case` statements, you can safely mark this issue as a false-positive. Loading history...
56
57		default :
58		throw new \UnexpectedValueException('Unexpected response code: ' . $response->code);
59		break;
		0 ignored issues – show Unused Code introduced 2017-12-19 12:54 UTC by Report Bug Copy Issue Report `break;` does not seem to be reachable. This check looks for unreachable code. It uses sophisticated control flow analysis techniques to find statements which will never be executed. Unreachable code is most often the result of `return`, `die` or `exit` statements that have been added for debug purposes. function fx() { try { doSomething(); return true; } catch (\Exception $e) { return false; } return false; } In the above example, the last `return false` will never be executed, because a return statement has already been met in every possible execution path. Loading history...
60		}
61		}
62
63		/**
64		* Check membership.
65		*
66		* Check if a user is, publicly or privately, a member of the organization.
67		*
68		* @param string $org The name of the organization.
69		* @param string $user The name of the user.
70		*
71		* @throws \UnexpectedValueException
72		* @since 1.0
73		*
74		* @return boolean
75		*/
76	View Code Duplication	public function check($org, $user)
		0 ignored issues – show Duplication introduced 2017-12-19 12:54 UTC by Report Bug Copy Issue Report 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...
77		{
78		// Build the request path.
79		$path = '/orgs/' . $org . '/members/' . $user;
80
81		$response = $this->client->get($this->fetchUrl($path));
82
83		switch ($response->code)
84		{
85		case 204 :
86		// Requester is an organization member and user is a member.
87		return true;
88		break;
		0 ignored issues – show Unused Code introduced 2017-12-19 12:54 UTC by Report Bug Copy Issue Report `break` is not strictly necessary here and could be removed. The `break` statement is not necessary if it is preceded for example by a `return` statement: switch ($x) { case 1: return 'foo'; break; // This break is not necessary and can be left off. } If you would like to keep this construct to be consistent with other `case` statements, you can safely mark this issue as a false-positive. Loading history...
89
90		case 404 :
91		// Requester is an organization member and user is not a member.
92		// Requester is not an organization member and is inquiring about themselves.
93		return false;
94		break;
		0 ignored issues – show Unused Code introduced 2017-12-19 12:54 UTC by Report Bug Copy Issue Report `break` is not strictly necessary here and could be removed. The `break` statement is not necessary if it is preceded for example by a `return` statement: switch ($x) { case 1: return 'foo'; break; // This break is not necessary and can be left off. } If you would like to keep this construct to be consistent with other `case` statements, you can safely mark this issue as a false-positive. Loading history...
95
96		case 302 :
97		// Requester is not an organization member.
98		return false;
99		break;
		0 ignored issues – show Unused Code introduced 2017-12-19 12:54 UTC by Report Bug Copy Issue Report `break` is not strictly necessary here and could be removed. The `break` statement is not necessary if it is preceded for example by a `return` statement: switch ($x) { case 1: return 'foo'; break; // This break is not necessary and can be left off. } If you would like to keep this construct to be consistent with other `case` statements, you can safely mark this issue as a false-positive. Loading history...
100
101		default :
102		throw new \UnexpectedValueException('Unexpected response code: ' . $response->code);
103		break;
		0 ignored issues – show Unused Code introduced 2017-12-19 12:54 UTC by Report Bug Copy Issue Report `break;` does not seem to be reachable. This check looks for unreachable code. It uses sophisticated control flow analysis techniques to find statements which will never be executed. Unreachable code is most often the result of `return`, `die` or `exit` statements that have been added for debug purposes. function fx() { try { doSomething(); return true; } catch (\Exception $e) { return false; } return false; } In the above example, the last `return false` will never be executed, because a return statement has already been met in every possible execution path. Loading history...
104		}
105		}
106
107		/**
108		* Add a member.
109		*
110		* To add someone as a member to an org, you must add them to a team.
111		*/
112
113		/**
114		* Remove a member.
115		*
116		* Removing a user from this list will remove them from all teams and they will no longer have
117		* any access to the organization’s repositories.
118		*
119		* @param string $org The name of the organization.
120		* @param string $user The name of the user.
121		*
122		* @since 1.0
123		*
124		* @return object
125		*/
126		public function remove($org, $user)
127		{
128		// Build the request path.
129		$path = '/orgs/' . $org . '/members/' . $user;
130
131		return $this->processResponse(
132		$this->client->delete($this->fetchUrl($path)),
133		204
134		);
135		}
136
137		/**
138		* Public members list.
139		*
140		* Members of an organization can choose to have their membership publicized or not.
141		*
142		* @param string $org The name of the organization.
143		*
144		* @since 1.0
145		*
146		* @return object
147		*/
148	View Code Duplication	public function getListPublic($org)
		0 ignored issues – show Duplication introduced 2017-12-19 12:54 UTC by Report Bug Copy Issue Report 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...
149		{
150		// Build the request path.
151		$path = '/orgs/' . $org . '/public_members';
152
153		return $this->processResponse(
154		$this->client->get($this->fetchUrl($path))
155		);
156		}
157
158		/**
159		* Check public membership.
160		*
161		* @param string $org The name of the organization.
162		* @param string $user The name of the user.
163		*
164		* @throws \UnexpectedValueException
165		* @since 1.0
166		*
167		* @return boolean
168		*/
169	View Code Duplication	public function checkPublic($org, $user)
		0 ignored issues – show Duplication introduced 2017-12-19 12:54 UTC by Report Bug Copy Issue Report 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...
170		{
171		// Build the request path.
172		$path = '/orgs/' . $org . '/public_members/' . $user;
173
174		$response = $this->client->get($this->fetchUrl($path));
175
176		switch ($response->code)
177		{
178		case 204 :
179		// Response if user is a public member.
180		return true;
181		break;
		0 ignored issues – show Unused Code introduced 2017-12-19 12:54 UTC by Report Bug Copy Issue Report `break` is not strictly necessary here and could be removed. The `break` statement is not necessary if it is preceded for example by a `return` statement: switch ($x) { case 1: return 'foo'; break; // This break is not necessary and can be left off. } If you would like to keep this construct to be consistent with other `case` statements, you can safely mark this issue as a false-positive. Loading history...
182
183		case 404 :
184		// Response if user is not a public member.
185		return false;
186		break;
		0 ignored issues – show Unused Code introduced 2017-12-19 12:54 UTC by Report Bug Copy Issue Report `break` is not strictly necessary here and could be removed. The `break` statement is not necessary if it is preceded for example by a `return` statement: switch ($x) { case 1: return 'foo'; break; // This break is not necessary and can be left off. } If you would like to keep this construct to be consistent with other `case` statements, you can safely mark this issue as a false-positive. Loading history...
187
188		default :
189		throw new \UnexpectedValueException('Unexpected response code: ' . $response->code);
190		break;
		0 ignored issues – show Unused Code introduced 2017-12-19 12:54 UTC by Report Bug Copy Issue Report `break;` does not seem to be reachable. This check looks for unreachable code. It uses sophisticated control flow analysis techniques to find statements which will never be executed. Unreachable code is most often the result of `return`, `die` or `exit` statements that have been added for debug purposes. function fx() { try { doSomething(); return true; } catch (\Exception $e) { return false; } return false; } In the above example, the last `return false` will never be executed, because a return statement has already been met in every possible execution path. Loading history...
191		}
192		}
193
194		/**
195		* Publicize a user’s membership.
196		*
197		* @param string $org The name of the organization.
198		* @param string $user The name of the user.
199		*
200		* @since 1.0
201		*
202		* @return object
203		*/
204		public function publicize($org, $user)
205		{
206		// Build the request path.
207		$path = '/orgs/' . $org . '/public_members/' . $user;
208
209		return $this->processResponse(
210		$this->client->put($this->fetchUrl($path), ''),
211		204
212		);
213		}
214
215		/**
216		* Conceal a user’s membership.
217		*
218		* @param string $org The name of the organization.
219		* @param string $user The name of the user.
220		*
221		* @since 1.0
222		*
223		* @return object
224		*/
225		public function conceal($org, $user)
226		{
227		// Build the request path.
228		$path = '/orgs/' . $org . '/public_members/' . $user;
229
230		return $this->processResponse(
231		$this->client->delete($this->fetchUrl($path)),
232		204
233		);
234		}
235
236		/**
237		* Get organization membership
238		*
239		* In order to get a user's membership with an organization, the authenticated user must be an organization owner.
240		*
241		* @param string $org The name of the organization.
242		* @param string $user The name of the user.
243		*
244		* @return object
245		*
246		* @since 1.4.0
247		*/
248		public function getMembership($org, $user)
249		{
250		// Build the request path.
251		$path = '/orgs/' . $org . '/memberships/' . $user;
252
253		return $this->processResponse($this->client->get($this->fetchUrl($path)));
254		}
255
256		/**
257		* Add or update organization membership
258		*
259		* In order to create or update a user's membership with an organization, the authenticated user must be an organization owner.
260		*
261		* @param string $org The name of the organization.
262		* @param string $user The name of the user.
263		* @param string $role The role to give the user in the organization. Can be either 'member' or 'admin'.
264		*
265		* @return object
266		*
267		* @since 1.4.0
268		*/
269		public function updateMembership($org, $user, $role = 'member')
270		{
271		$allowedRoles = array('member', 'admin');
272
273		if (!in_array($role, $allowedRoles))
274		{
275		throw new \InvalidArgumentException(sprintf("The user's role must be: %s", implode(', ', $allowedRoles)));
276		}
277
278		// Build the request path.
279		$path = "/orgs/$org/memberships/$user";
280
281		$data = array(
282		'role' => $role,
283		);
284
285		return $this->processResponse($this->client->put($this->fetchUrl($path), json_encode($data)));
286		}
287
288		/**
289		* Remove organization membership
290		*
291		* In order to remove a user's membership with an organization, the authenticated user must be an organization owner.
292		*
293		* @param string $org The name of the organization.
294		* @param string $user The name of the user.
295		*
296		* @return object
297		*
298		* @since 1.4.0
299		*/
300		public function removeMembership($org, $user)
301		{
302		// Build the request path.
303		$path = '/orgs/' . $org . '/memberships/' . $user;
304
305		return $this->processResponse(
306		$this->client->delete($this->fetchUrl($path)),
307		204
308		);
309		}
310
311		/**
312		* List your organization memberships
313		*
314		* @return object
315		*
316		* @since 1.4.0
317		*/
318		public function listMemberships()
319		{
320		// Build the request path.
321		$path = '/user/memberships/orgs';
322
323		return $this->processResponse($this->client->get($this->fetchUrl($path)));
324		}
325
326		/**
327		* Get your organization membership
328		*
329		* @param string $org The name of the organization.
330		*
331		* @return object
332		*
333		* @since 1.4.0
334		*/
335		public function listOrganizationMembership($org)
336		{
337		// Build the request path.
338		$path = '/user/memberships/orgs/' . $org;
339
340		return $this->processResponse($this->client->get($this->fetchUrl($path)));
341		}
342
343		/**
344		* Edit your organization membership
345		*
346		* @param string $org The name of the organization.
347		* @param string $state The state that the membership should be in.
348		*
349		* @return object
350		*
351		* @since 1.4.0
352		*/
353	View Code Duplication	public function editOrganizationMembership($org, $state)
		0 ignored issues – show Duplication introduced 2017-12-19 12:54 UTC by Report Bug Copy Issue Report 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...
354		{
355		// The API only accepts $state == 'active' at present
356		if ($state != 'active')
357		{
358		throw new \InvalidArgumentException('The state must be "active".');
359		}
360
361		// Build the request path.
362		$path = '/user/memberships/orgs/' . $org;
363
364		return $this->processResponse($this->client->patch($this->fetchUrl($path), array('state' => $state)));
365		}
366		}
367

joomla-projects / com_localise

GitHub Access Token became invalid

Push — develop ( b5e3c2...7799bc )

Members A

Complexity

Size/Duplication

Coupling/Cohesion

Importance

13 Methods

How to fix Duplicated Code

Duplicated Code

Duplication Side-by-Side

Filter issues like