Joomla\Github\Package\Orgs

Complexity

Total Complexity

7

Size/Duplication

Total Lines	85
Duplicated Lines	14.12 %

Coupling/Cohesion

Components	1
Dependencies	2

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	getList()	12	12	2
A	get()	0	10	1
B	edit()	0	24	4

<?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;

use Joomla\Github\AbstractPackage;

/**
 * GitHub API Activity class for the Joomla Framework.
 *
 * @since  1.0
 *
 * @documentation  http://developer.github.com/v3/orgs/
 *
 * @property-read  Orgs\Hooks    $hooks    GitHub API object for hooks.
 * @property-read  Orgs\Members  $members  GitHub API object for members.
 * @property-read  Orgs\Teams    $teams    GitHub API object for teams.
 */
class Orgs extends AbstractPackage
{
	/**
	 * List user organizations.
	 *
	 * If a user name is given, public and private organizations for the authenticated user will be listed.
	 *
	 * @param   string  $user  The user name.
	 *
	 * @return  object
	 *
	 * @since   1.0
	 */
	public function getList($user = '')

This method seems to be duplicated in your project.

	{
		// Build the request path.
		$path = ($user)
			? '/users/' . $user . '/orgs'
			: '/user/orgs';

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

	/**
	 * Get an organization.
	 *
	 * @param   string  $org  The organization name.
	 *
	 * @return  object
	 *
	 * @since   1.0
	 */
	public function get($org)
	{
		// Build the request path.
		$path = '/orgs/' . $org;

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

	/**
	 * Edit an organization.
	 *
	 * @param   string  $org           The organization name.
	 * @param   string  $billingEmail  Billing email address. This address is not publicized.
	 * @param   string  $company       The company name.
	 * @param   string  $email         The email address.
	 * @param   string  $location      The location name.
	 * @param   string  $name          The name.
	 *
	 * @return  object
	 *
	 * @since   1.0
	 */
	public function edit($org, $billingEmail = '', $company = '', $email = '', $location = '', $name = '')

The parameter $billingEmail is not used and could be removed.

The parameter $company is not used and could be removed.

The parameter $email is not used and could be removed.

The parameter $location is not used and could be removed.

The parameter $name is not used and could be removed.

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

		$args = array('billing_email', 'company', 'email', 'location', 'name');

		$data = array();

		$fArgs = func_get_args();

		foreach ($args as $i => $arg)
		{
			if (array_key_exists($i + 1, $fArgs) && $fArgs[$i + 1])
			{
				$data[$arg] = $fArgs[$i + 1];
			}
		}

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