FlexyProject / GitHubAPI

lib/GitHub/AbstractApi.php

Doc Comments +1 added lines, -1 removed lines

@@ -156,7 +156,7 @@
     /**
      * Set accept
      *
-     * @param array|string $accept
+     * @param string $accept
      *
      * @return AbstractApi
      */

Spacing +2 added lines, -2 removed lines

@@ -517,7 +517,7 @@ 
                 break;
         }
 
-        $curl->success(function (CurlClient $instance) {
+        $curl->success(function(CurlClient $instance) {
             $this->headers = $instance->getHeaders();
             $this->success = $instance->getResponse();
             $data          = json_decode($this->success, true);
@@ -525,7 +525,7 @@ 
                 $this->success = $data;
             }
         });
-        $curl->error(function (CurlClient $instance) {
+        $curl->error(function(CurlClient $instance) {
             $this->headers = $instance->getHeaders();
             $this->failure = $instance->getResponse();
             $data          = json_decode($this->failure, true);

Indentation +615 added lines, -615 removed lines

@@ -13,628 +13,628 @@
 abstract class AbstractApi
 {
 
-    /** API version */
-    const API_VERSION = 'v3';
-
-    /** API constants */
-    const API_URL        = 'https://api.github.com';
-    const API_UPLOADS    = 'https://uploads.github.com';
-    const API_RAW_URL    = 'https://raw.github.com';
-    const CONTENT_TYPE   = 'application/json';
-    const DEFAULT_ACCEPT = 'application/vnd.github.' . self::API_VERSION . '+json';
-    const USER_AGENT     = 'FlexyProject-GitHubAPI';
-
-    /** Archive constants */
-    const ARCHIVE_TARBALL = 'tarball';
-    const ARCHIVE_ZIPBALL = 'zipball';
-
-    /** Authentication constants */
-    const OAUTH_AUTH             = 0;
-    const OAUTH2_HEADER_AUTH     = 1;
-    const OAUTH2_PARAMETERS_AUTH = 2;
-
-    /** Branch constants */
-    const BRANCH_MASTER  = 'master';
-    const BRANCH_DEVELOP = 'develop';
-
-    /** Direction constants */
-    const DIRECTION_ASC  = 'asc';
-    const DIRECTION_DESC = 'desc';
-
-    /** Environment constants */
-    const ENVIRONMENT_PRODUCTION = 'production';
-    const ENVIRONMENT_STAGING    = 'staging';
-    const ENVIRONMENT_QA         = 'qa';
-
-    /** Events constants */
-    const EVENTS_PULL         = 'pull';
-    const EVENTS_PULL_REQUEST = 'pull_request';
-    const EVENTS_PUSH         = 'push';
-
-    /** Filter constants */
-    const FILTER_ALL        = 'all';
-    const FILTER_ASSIGNED   = 'assigned';
-    const FILTER_CREATED    = 'created';
-    const FILTER_MENTIONED  = 'mentioned';
-    const FILTER_SUBSCRIBED = 'subscribed';
-
-    /** Media types constants */
-    const MEDIA_TYPE_JSON = 'json';
-    const MEDIA_TYPE_RAW  = 'raw';
-    const MEDIA_TYPE_FULL = 'full';
-    const MEDIA_TYPE_TEXT = 'text';
-
-    /** Modes constants */
-    const MODE_MARKDOWN = 'markdown';
-    const MODE_GFM      = 'gfm';
-
-    /** Permissions constants */
-    const PERMISSION_ADMIN = 'admin';
-    const PERMISSION_PULL  = 'pull';
-    const PERMISSION_PUSH  = 'push';
-
-    /** Sort constants */
-    const SORT_COMPLETENESS = 'completeness';
-    const SORT_CREATED      = 'created';
-    const SORT_DUE_DATE     = 'due_date';
-    const SORT_FULL_NAME    = 'full_name';
-    const SORT_NEWEST       = 'newest';
-    const SORT_OLDEST       = 'oldest';
-    const SORT_PUSHED       = 'pushed';
-    const SORT_STARGAZERS   = 'stargazers';
-    const SORT_UPDATED      = 'updated';
-
-    /** State constants */
-    const STATE_ACTIVE  = 'active';
-    const STATE_ALL     = 'all';
-    const STATE_CLOSED  = 'closed';
-    const STATE_ERROR   = 'error';
-    const STATE_FAILURE = 'failure';
-    const STATE_OPEN    = 'open';
-    const STATE_PENDING = 'pending';
-    const STATE_SUCCESS = 'success';
-
-    /** Task constants */
-    const TASK_DEPLOY            = 'deploy';
-    const TASK_DEPLOY_MIGRATIONS = 'deploy:migrations';
-
-    /** Type constants */
-    const TYPE_ALL        = 'all';
-    const TYPE_COMMENTS   = 'comments';
-    const TYPE_GISTS      = 'gists';
-    const TYPE_HOOKS      = 'hooks';
-    const TYPE_ISSUES     = 'issues';
-    const TYPE_MEMBER     = 'member';
-    const TYPE_MILESTONES = 'milestones';
-    const TYPE_ORGS       = 'orgs';
-    const TYPE_OWNER      = 'owner';
-    const TYPE_PAGES      = 'pages';
-    const TYPE_PUBLIC     = 'public';
-    const TYPE_PULLS      = 'pulls';
-    const TYPE_PRIVATE    = 'private';
-    const TYPE_REPOS      = 'repos';
-    const TYPE_USERS      = 'users';
-
-    /** Properties */
-    protected $accept         = self::DEFAULT_ACCEPT;
-    protected $apiUrl         = self::API_URL;
-    protected $authentication = self::OAUTH_AUTH;
-    protected $clientId;
-    protected $clientSecret;
-    protected $contentType    = self::CONTENT_TYPE;
-    protected $failure;
-    protected $headers        = [];
-    protected $httpAuth       = ['username' => '', 'password' => ''];
-    protected $pagination;
-    protected $request;
-    protected $success;
-    protected $timeout        = 240;
-    protected $token          = '';
-
-    /**
-     * Constructor
-     */
-    public function __construct()
-    {
-        $this->request = Request::createFromGlobals();
-    }
-
-    /**
-     * Get request
-     *
-     * @return Request
-     */
-    public function getRequest(): Request
-    {
-        return $this->request;
-    }
-
-    /**
-     * Get accept
-     *
-     * @return mixed
-     */
-    public function getAccept()
-    {
-        return $this->accept;
-    }
-
-    /**
-     * Set accept
-     *
-     * @param array|string $accept
-     *
-     * @return AbstractApi
-     */
-    public function setAccept($accept): AbstractApi
-    {
-        $this->accept = $accept;
-
-        return $this;
-    }
-
-    /**
-     * Get authentication
-     *
-     * @return int
-     */
-    public function getAuthentication(): int
-    {
-        return $this->authentication;
-    }
-
-    /**
-     * Set authentication
-     *
-     * @param int $authentication
-     *
-     * @return AbstractApi
-     */
-    public function setAuthentication(int $authentication): AbstractApi
-    {
-        $this->authentication = $authentication;
-
-        return $this;
-    }
-
-    /**
-     * Get apiUrl
-     *
-     * @return string
-     */
-    public function getApiUrl(): string
-    {
-        return $this->apiUrl;
-    }
-
-    /**
-     * Set apiUrl
-     *
-     * @param mixed $apiUrl
-     *
-     * @return AbstractApi
-     */
-    public function setApiUrl($apiUrl): AbstractApi
-    {
-        $this->apiUrl = $apiUrl;
-
-        return $this;
-    }
-
-    /**
-     * Get clientId
-     *
-     * @return null|int
-     */
-    public function getClientId()
-    {
-        return $this->clientId;
-    }
-
-    /**
-     * Set clientId
-     *
-     * @param mixed $clientId
-     *
-     * @return AbstractApi
-     */
-    public function setClientId($clientId): AbstractApi
-    {
-        $this->clientId = $clientId;
-
-        return $this;
-    }
-
-    /**
-     * Get clientSecret
-     *
-     * @return null|string
-     */
-    public function getClientSecret()
-    {
-        return $this->clientSecret;
-    }
-
-    /**
-     * Set clientSecret
-     *
-     * @param mixed $clientSecret
-     *
-     * @return AbstractApi
-     */
-    public function setClientSecret($clientSecret): AbstractApi
-    {
-        $this->clientSecret = $clientSecret;
-
-        return $this;
-    }
-
-    /**
-     * Get httpAuth
-     *
-     * @return array
-     */
-    public function getHttpAuth(): array
-    {
-        return $this->httpAuth;
-    }
-
-    /**
-     * Set httpAuth
-     *
-     * @param string $username
-     * @param string $password
-     *
-     * @return AbstractApi
-     */
-    public function setHttpAuth(string $username, string $password = ''): AbstractApi
-    {
-        $this->httpAuth['username'] = $username;
-        $this->httpAuth['password'] = $password;
-
-        return $this;
-    }
-
-    /**
-     * Get token
-     *
-     * @return string
-     */
-    public function getToken(): string
-    {
-        return $this->token;
-    }
-
-    /**
-     * Set token
-     *
-     * @param string $token
-     * @param int    $authentication
-     *
-     * @return AbstractApi
-     */
-    public function setToken(string $token, int $authentication = self::OAUTH_AUTH): AbstractApi
-    {
-        $this->token = $token;
-        $this->setAuthentication($authentication);
-
-        return $this;
-    }
-
-    /**
-     * Get timeout
-     *
-     * @return int
-     */
-    public function getTimeout(): int
-    {
-        return $this->timeout;
-    }
-
-    /**
-     * Set timeout
-     *
-     * @param int $timeout
-     *
-     * @return AbstractApi
-     */
-    public function setTimeout(int $timeout): AbstractApi
-    {
-        $this->timeout = $timeout;
-
-        return $this;
-    }
-
-    /**
-     * Get contentType
-     *
-     * @return string
-     */
-    public function getContentType(): string
-    {
-        return $this->contentType;
-    }
-
-    /**
-     * Set contentType
-     *
-     * @param string $contentType
-     *
-     * @return AbstractApi
-     */
-    public function setContentType(string $contentType): AbstractApi
-    {
-        $this->contentType = $contentType;
-
-        return $this;
-    }
-
-    /**
-     * Get headers
-     *
-     * @return array
-     */
-    public function getHeaders(): array
-    {
-        return $this->headers;
-    }
-
-    /**
-     * Get pagination
-     *
-     * @return Pagination|null
-     */
-    public function getPagination()
-    {
-        return $this->pagination;
-    }
-
-    /**
-     * Set pagination
-     *
-     * @param Pagination $pagination
-     *
-     * @return AbstractApi
-     */
-    public function setPagination(Pagination $pagination): AbstractApi
-    {
-        $this->pagination = $pagination;
-
-        return $this;
-    }
-
-    /**
-     * Curl request
-     *
-     * @param string      $url
-     * @param string      $method
-     * @param array       $postFields
-     * @param null|string $apiUrl
-     *
-     * @return array
-     */
-    public function request(
-        string $url,
-        string $method = Request::METHOD_GET,
-        array $postFields = [],
-        string $apiUrl = null
-    ): array {
-        /** Building url */
-        if (null === $apiUrl) {
-            $apiUrl = $this->getApiUrl();
-        }
-        $url   = $apiUrl . $url;
-        $query = [];
-
-        /**
-         * OAuth2 Key/Secret authentication
-         *
-         * @see https://developer.github.com/v3/#oauth2-keysecret
-         */
-        if (null !== $this->getClientId() && null !== $this->getClientSecret()) {
-            $query['client_id']     = $this->getClientId();
-            $query['client_secret'] = $this->getClientSecret();
-        } /**
-         * Basic authentication via OAuth2 Token (sent as a parameter)
-         *
-         * @see https://developer.github.com/v3/#oauth2-token-sent-as-a-parameter
-         */ elseif ($this->getAuthentication() === self::OAUTH2_PARAMETERS_AUTH) {
-            $query['access_token'] = $this->getToken();
+	/** API version */
+	const API_VERSION = 'v3';
+
+	/** API constants */
+	const API_URL        = 'https://api.github.com';
+	const API_UPLOADS    = 'https://uploads.github.com';
+	const API_RAW_URL    = 'https://raw.github.com';
+	const CONTENT_TYPE   = 'application/json';
+	const DEFAULT_ACCEPT = 'application/vnd.github.' . self::API_VERSION . '+json';
+	const USER_AGENT     = 'FlexyProject-GitHubAPI';
+
+	/** Archive constants */
+	const ARCHIVE_TARBALL = 'tarball';
+	const ARCHIVE_ZIPBALL = 'zipball';
+
+	/** Authentication constants */
+	const OAUTH_AUTH             = 0;
+	const OAUTH2_HEADER_AUTH     = 1;
+	const OAUTH2_PARAMETERS_AUTH = 2;
+
+	/** Branch constants */
+	const BRANCH_MASTER  = 'master';
+	const BRANCH_DEVELOP = 'develop';
+
+	/** Direction constants */
+	const DIRECTION_ASC  = 'asc';
+	const DIRECTION_DESC = 'desc';
+
+	/** Environment constants */
+	const ENVIRONMENT_PRODUCTION = 'production';
+	const ENVIRONMENT_STAGING    = 'staging';
+	const ENVIRONMENT_QA         = 'qa';
+
+	/** Events constants */
+	const EVENTS_PULL         = 'pull';
+	const EVENTS_PULL_REQUEST = 'pull_request';
+	const EVENTS_PUSH         = 'push';
+
+	/** Filter constants */
+	const FILTER_ALL        = 'all';
+	const FILTER_ASSIGNED   = 'assigned';
+	const FILTER_CREATED    = 'created';
+	const FILTER_MENTIONED  = 'mentioned';
+	const FILTER_SUBSCRIBED = 'subscribed';
+
+	/** Media types constants */
+	const MEDIA_TYPE_JSON = 'json';
+	const MEDIA_TYPE_RAW  = 'raw';
+	const MEDIA_TYPE_FULL = 'full';
+	const MEDIA_TYPE_TEXT = 'text';
+
+	/** Modes constants */
+	const MODE_MARKDOWN = 'markdown';
+	const MODE_GFM      = 'gfm';
+
+	/** Permissions constants */
+	const PERMISSION_ADMIN = 'admin';
+	const PERMISSION_PULL  = 'pull';
+	const PERMISSION_PUSH  = 'push';
+
+	/** Sort constants */
+	const SORT_COMPLETENESS = 'completeness';
+	const SORT_CREATED      = 'created';
+	const SORT_DUE_DATE     = 'due_date';
+	const SORT_FULL_NAME    = 'full_name';
+	const SORT_NEWEST       = 'newest';
+	const SORT_OLDEST       = 'oldest';
+	const SORT_PUSHED       = 'pushed';
+	const SORT_STARGAZERS   = 'stargazers';
+	const SORT_UPDATED      = 'updated';
+
+	/** State constants */
+	const STATE_ACTIVE  = 'active';
+	const STATE_ALL     = 'all';
+	const STATE_CLOSED  = 'closed';
+	const STATE_ERROR   = 'error';
+	const STATE_FAILURE = 'failure';
+	const STATE_OPEN    = 'open';
+	const STATE_PENDING = 'pending';
+	const STATE_SUCCESS = 'success';
+
+	/** Task constants */
+	const TASK_DEPLOY            = 'deploy';
+	const TASK_DEPLOY_MIGRATIONS = 'deploy:migrations';
+
+	/** Type constants */
+	const TYPE_ALL        = 'all';
+	const TYPE_COMMENTS   = 'comments';
+	const TYPE_GISTS      = 'gists';
+	const TYPE_HOOKS      = 'hooks';
+	const TYPE_ISSUES     = 'issues';
+	const TYPE_MEMBER     = 'member';
+	const TYPE_MILESTONES = 'milestones';
+	const TYPE_ORGS       = 'orgs';
+	const TYPE_OWNER      = 'owner';
+	const TYPE_PAGES      = 'pages';
+	const TYPE_PUBLIC     = 'public';
+	const TYPE_PULLS      = 'pulls';
+	const TYPE_PRIVATE    = 'private';
+	const TYPE_REPOS      = 'repos';
+	const TYPE_USERS      = 'users';
+
+	/** Properties */
+	protected $accept         = self::DEFAULT_ACCEPT;
+	protected $apiUrl         = self::API_URL;
+	protected $authentication = self::OAUTH_AUTH;
+	protected $clientId;
+	protected $clientSecret;
+	protected $contentType    = self::CONTENT_TYPE;
+	protected $failure;
+	protected $headers        = [];
+	protected $httpAuth       = ['username' => '', 'password' => ''];
+	protected $pagination;
+	protected $request;
+	protected $success;
+	protected $timeout        = 240;
+	protected $token          = '';
+
+	/**
+	 * Constructor
+	 */
+	public function __construct()
+	{
+		$this->request = Request::createFromGlobals();
+	}
+
+	/**
+	 * Get request
+	 *
+	 * @return Request
+	 */
+	public function getRequest(): Request
+	{
+		return $this->request;
+	}
+
+	/**
+	 * Get accept
+	 *
+	 * @return mixed
+	 */
+	public function getAccept()
+	{
+		return $this->accept;
+	}
+
+	/**
+	 * Set accept
+	 *
+	 * @param array|string $accept
+	 *
+	 * @return AbstractApi
+	 */
+	public function setAccept($accept): AbstractApi
+	{
+		$this->accept = $accept;
+
+		return $this;
+	}
+
+	/**
+	 * Get authentication
+	 *
+	 * @return int
+	 */
+	public function getAuthentication(): int
+	{
+		return $this->authentication;
+	}
+
+	/**
+	 * Set authentication
+	 *
+	 * @param int $authentication
+	 *
+	 * @return AbstractApi
+	 */
+	public function setAuthentication(int $authentication): AbstractApi
+	{
+		$this->authentication = $authentication;
+
+		return $this;
+	}
+
+	/**
+	 * Get apiUrl
+	 *
+	 * @return string
+	 */
+	public function getApiUrl(): string
+	{
+		return $this->apiUrl;
+	}
+
+	/**
+	 * Set apiUrl
+	 *
+	 * @param mixed $apiUrl
+	 *
+	 * @return AbstractApi
+	 */
+	public function setApiUrl($apiUrl): AbstractApi
+	{
+		$this->apiUrl = $apiUrl;
+
+		return $this;
+	}
+
+	/**
+	 * Get clientId
+	 *
+	 * @return null|int
+	 */
+	public function getClientId()
+	{
+		return $this->clientId;
+	}
+
+	/**
+	 * Set clientId
+	 *
+	 * @param mixed $clientId
+	 *
+	 * @return AbstractApi
+	 */
+	public function setClientId($clientId): AbstractApi
+	{
+		$this->clientId = $clientId;
+
+		return $this;
+	}
+
+	/**
+	 * Get clientSecret
+	 *
+	 * @return null|string
+	 */
+	public function getClientSecret()
+	{
+		return $this->clientSecret;
+	}
+
+	/**
+	 * Set clientSecret
+	 *
+	 * @param mixed $clientSecret
+	 *
+	 * @return AbstractApi
+	 */
+	public function setClientSecret($clientSecret): AbstractApi
+	{
+		$this->clientSecret = $clientSecret;
+
+		return $this;
+	}
+
+	/**
+	 * Get httpAuth
+	 *
+	 * @return array
+	 */
+	public function getHttpAuth(): array
+	{
+		return $this->httpAuth;
+	}
+
+	/**
+	 * Set httpAuth
+	 *
+	 * @param string $username
+	 * @param string $password
+	 *
+	 * @return AbstractApi
+	 */
+	public function setHttpAuth(string $username, string $password = ''): AbstractApi
+	{
+		$this->httpAuth['username'] = $username;
+		$this->httpAuth['password'] = $password;
+
+		return $this;
+	}
+
+	/**
+	 * Get token
+	 *
+	 * @return string
+	 */
+	public function getToken(): string
+	{
+		return $this->token;
+	}
+
+	/**
+	 * Set token
+	 *
+	 * @param string $token
+	 * @param int    $authentication
+	 *
+	 * @return AbstractApi
+	 */
+	public function setToken(string $token, int $authentication = self::OAUTH_AUTH): AbstractApi
+	{
+		$this->token = $token;
+		$this->setAuthentication($authentication);
+
+		return $this;
+	}
+
+	/**
+	 * Get timeout
+	 *
+	 * @return int
+	 */
+	public function getTimeout(): int
+	{
+		return $this->timeout;
+	}
+
+	/**
+	 * Set timeout
+	 *
+	 * @param int $timeout
+	 *
+	 * @return AbstractApi
+	 */
+	public function setTimeout(int $timeout): AbstractApi
+	{
+		$this->timeout = $timeout;
+
+		return $this;
+	}
+
+	/**
+	 * Get contentType
+	 *
+	 * @return string
+	 */
+	public function getContentType(): string
+	{
+		return $this->contentType;
+	}
+
+	/**
+	 * Set contentType
+	 *
+	 * @param string $contentType
+	 *
+	 * @return AbstractApi
+	 */
+	public function setContentType(string $contentType): AbstractApi
+	{
+		$this->contentType = $contentType;
+
+		return $this;
+	}
+
+	/**
+	 * Get headers
+	 *
+	 * @return array
+	 */
+	public function getHeaders(): array
+	{
+		return $this->headers;
+	}
+
+	/**
+	 * Get pagination
+	 *
+	 * @return Pagination|null
+	 */
+	public function getPagination()
+	{
+		return $this->pagination;
+	}
+
+	/**
+	 * Set pagination
+	 *
+	 * @param Pagination $pagination
+	 *
+	 * @return AbstractApi
+	 */
+	public function setPagination(Pagination $pagination): AbstractApi
+	{
+		$this->pagination = $pagination;
+
+		return $this;
+	}
+
+	/**
+	 * Curl request
+	 *
+	 * @param string      $url
+	 * @param string      $method
+	 * @param array       $postFields
+	 * @param null|string $apiUrl
+	 *
+	 * @return array
+	 */
+	public function request(
+		string $url,
+		string $method = Request::METHOD_GET,
+		array $postFields = [],
+		string $apiUrl = null
+	): array {
+		/** Building url */
+		if (null === $apiUrl) {
+			$apiUrl = $this->getApiUrl();
+		}
+		$url   = $apiUrl . $url;
+		$query = [];
+
+		/**
+		 * OAuth2 Key/Secret authentication
+		 *
+		 * @see https://developer.github.com/v3/#oauth2-keysecret
+		 */
+		if (null !== $this->getClientId() && null !== $this->getClientSecret()) {
+			$query['client_id']     = $this->getClientId();
+			$query['client_secret'] = $this->getClientSecret();
+		} /**
+		 * Basic authentication via OAuth2 Token (sent as a parameter)
+		 *
+		 * @see https://developer.github.com/v3/#oauth2-token-sent-as-a-parameter
+		 */ elseif ($this->getAuthentication() === self::OAUTH2_PARAMETERS_AUTH) {
+			$query['access_token'] = $this->getToken();
 }
 
-        /**
-         * Pagination
-         * Requests that return multiple items will be paginated to 30 items by default.
-         * You can specify further pages with the ?page parameter.
-         * For some resources, you can also set a custom page size up to 100 with the ?per_page parameter.
-         * Note that for technical reasons not all endpoints respect the ?per_page parameter,
-         *
-         * @see https://developer.github.com/v3/#pagination
-         */
+		/**
+		 * Pagination
+		 * Requests that return multiple items will be paginated to 30 items by default.
+		 * You can specify further pages with the ?page parameter.
+		 * For some resources, you can also set a custom page size up to 100 with the ?per_page parameter.
+		 * Note that for technical reasons not all endpoints respect the ?per_page parameter,
+		 *
+		 * @see https://developer.github.com/v3/#pagination
+		 */
 if (null !== $this->getPagination()) {
-    if (null !== $this->getPagination()->getPage()) {
-        $query['page'] = $this->getPagination()->getPage();
-    }
-    if (null !== $this->getPagination()->getLimit()) {
-        $query['per_page'] = $this->getPagination()->getLimit();
-    }
+	if (null !== $this->getPagination()->getPage()) {
+		$query['page'] = $this->getPagination()->getPage();
+	}
+	if (null !== $this->getPagination()->getLimit()) {
+		$query['per_page'] = $this->getPagination()->getLimit();
+	}
 }
 
-        /**
-         * Add URL-encoded query string parameters
-         */
+		/**
+		 * Add URL-encoded query string parameters
+		 */
 if (!empty($query)) {
-    $url .= (strstr($url, '?') !== false ? '&' : '?');
-    $url .= http_build_query($query);
+	$url .= (strstr($url, '?') !== false ? '&' : '?');
+	$url .= http_build_query($query);
 }
 
-        /** Call curl */
-        $curl = new CurlClient();
-        $curl->setOption([
-            CURLOPT_USERAGENT      => self::USER_AGENT,
-            CURLOPT_TIMEOUT        => $this->getTimeout(),
-            CURLOPT_HEADER         => false, // Use $client->getHeaders() to get full header
-            CURLOPT_FOLLOWLOCATION => true,
-            CURLOPT_HTTPHEADER     => [
-                'Accept: ' . $this->getAccept(),
-                'Content-Type: ' . $this->getContentType()
-            ],
-            CURLOPT_URL            => $url
-        ]);
-
-        /**
-         * Basic authentication via username and Password
-         *
-         * @see https://developer.github.com/v3/auth/#via-username-and-password
-         */
-        if (!empty($this->getHttpAuth())) {
-            if (!isset($this->getHttpAuth()['password']) || empty($this->getHttpAuth()['password'])) {
-                $curl->setOption([
-                    CURLOPT_HTTPAUTH => CURLAUTH_BASIC,
-                    CURLOPT_USERPWD  => $this->getHttpAuth()['username']
-                ]);
-            } else {
-                $curl->setOption([
-                    CURLOPT_HTTPAUTH => CURLAUTH_BASIC,
-                    CURLOPT_USERPWD  => sprintf(
-                        '%s:%s',
-                        $this->getHttpAuth()['username'],
-                        $this->getHttpAuth()['password']
-                    )
-                ]);
-            }
-        }
-
-        if (!empty($this->getToken()) && $this->getAuthentication() !== self::OAUTH2_PARAMETERS_AUTH) {
-            /**
-             * Basic authentication via OAuth token
-             *
-             * @see https://developer.github.com/v3/auth/#via-oauth-tokens
-             **/
-            if ($this->getAuthentication() === self::OAUTH_AUTH) {
-                $curl->setOption([
-                    CURLOPT_HTTPAUTH => CURLAUTH_BASIC,
-                    CURLOPT_USERPWD  => sprintf('%s:x-oauth-basic', $this->getToken())
-                ]);
-            } /**
-             * Basic authentication via OAuth2 Token (sent in a header)
-             *
-             * @see https://developer.github.com/v3/#oauth2-token-sent-in-a-header
-             */ elseif ($this->getAuthentication() === self::OAUTH2_HEADER_AUTH) {
-                $curl->setOption([
-                    CURLOPT_HTTPAUTH   => CURLAUTH_BASIC,
-                    CURLOPT_HTTPHEADER => [sprintf('Authorization: token %s', $this->getToken())]
-                ]);
+		/** Call curl */
+		$curl = new CurlClient();
+		$curl->setOption([
+			CURLOPT_USERAGENT      => self::USER_AGENT,
+			CURLOPT_TIMEOUT        => $this->getTimeout(),
+			CURLOPT_HEADER         => false, // Use $client->getHeaders() to get full header
+			CURLOPT_FOLLOWLOCATION => true,
+			CURLOPT_HTTPHEADER     => [
+				'Accept: ' . $this->getAccept(),
+				'Content-Type: ' . $this->getContentType()
+			],
+			CURLOPT_URL            => $url
+		]);
+
+		/**
+		 * Basic authentication via username and Password
+		 *
+		 * @see https://developer.github.com/v3/auth/#via-username-and-password
+		 */
+		if (!empty($this->getHttpAuth())) {
+			if (!isset($this->getHttpAuth()['password']) || empty($this->getHttpAuth()['password'])) {
+				$curl->setOption([
+					CURLOPT_HTTPAUTH => CURLAUTH_BASIC,
+					CURLOPT_USERPWD  => $this->getHttpAuth()['username']
+				]);
+			} else {
+				$curl->setOption([
+					CURLOPT_HTTPAUTH => CURLAUTH_BASIC,
+					CURLOPT_USERPWD  => sprintf(
+						'%s:%s',
+						$this->getHttpAuth()['username'],
+						$this->getHttpAuth()['password']
+					)
+				]);
+			}
+		}
+
+		if (!empty($this->getToken()) && $this->getAuthentication() !== self::OAUTH2_PARAMETERS_AUTH) {
+			/**
+			 * Basic authentication via OAuth token
+			 *
+			 * @see https://developer.github.com/v3/auth/#via-oauth-tokens
+			 **/
+			if ($this->getAuthentication() === self::OAUTH_AUTH) {
+				$curl->setOption([
+					CURLOPT_HTTPAUTH => CURLAUTH_BASIC,
+					CURLOPT_USERPWD  => sprintf('%s:x-oauth-basic', $this->getToken())
+				]);
+			} /**
+			 * Basic authentication via OAuth2 Token (sent in a header)
+			 *
+			 * @see https://developer.github.com/v3/#oauth2-token-sent-in-a-header
+			 */ elseif ($this->getAuthentication() === self::OAUTH2_HEADER_AUTH) {
+				$curl->setOption([
+					CURLOPT_HTTPAUTH   => CURLAUTH_BASIC,
+					CURLOPT_HTTPHEADER => [sprintf('Authorization: token %s', $this->getToken())]
+				]);
 }
-        }
-
-        /** Methods */
-        switch ($method) {
-            /** @see http://www.w3.org/Protocols/rfc2616/rfc2616-sec9.html#sec9.7 */
-            case Request::METHOD_DELETE:
-                /** @see http://tools.ietf.org/html/rfc5789 */
-            case Request::METHOD_PATCH:
-                $curl->setOption([
-                    CURLOPT_CUSTOMREQUEST => $method,
-                    CURLOPT_POST          => true,
-                    CURLOPT_POSTFIELDS    => json_encode(array_filter($postFields))
-                ]);
-                break;
-
-            /** @see http://www.w3.org/Protocols/rfc2616/rfc2616-sec9.html#sec9.3 */
-            case Request::METHOD_GET:
-                $curl->setOption(CURLOPT_HTTPGET, true);
-                break;
-
-            /** @see http://www.w3.org/Protocols/rfc2616/rfc2616-sec9.html#sec9.4 */
-            case Request::METHOD_HEAD:
-                $curl->setOption([
-                    CURLOPT_CUSTOMREQUEST => $method,
-                    CURLOPT_NOBODY        => true
-                ]);
-                break;
-
-            /** @see http://www.w3.org/Protocols/rfc2616/rfc2616-sec9.html#sec9.5 */
-            case Request::METHOD_POST:
-                $curl->setOption([
-                    CURLOPT_POST       => true,
-                    CURLOPT_POSTFIELDS => json_encode(array_filter($postFields))
-                ]);
-                break;
-
-            /** @see http://www.w3.org/Protocols/rfc2616/rfc2616-sec9.html#sec9.6 */
-            case Request::METHOD_PUT:
-                $curl->setOption([
-                    CURLOPT_CUSTOMREQUEST => $method,
-                    CURLOPT_PUT           => true,
-                    CURLOPT_HTTPHEADER    => [
-                        'X-HTTP-Method-Override: PUT',
-                        'Content-type: application/x-www-form-urlencoded'
-                    ]
-                ]);
-                break;
-
-            default:
-                break;
-        }
-
-        $curl->success(function (CurlClient $instance) {
-            $this->headers = $instance->getHeaders();
-            $this->success = $instance->getResponse();
-            $data          = json_decode($this->success, true);
-            if (JSON_ERROR_NONE === json_last_error()) {
-                $this->success = $data;
-            }
-        });
-        $curl->error(function (CurlClient $instance) {
-            $this->headers = $instance->getHeaders();
-            $this->failure = $instance->getResponse();
-            $data          = json_decode($this->failure, true);
-            if (JSON_ERROR_NONE === json_last_error()) {
-                $this->failure = $data;
-            }
-        });
-        $curl->perform();
-
-        return (array)($this->success ?? $this->failure);
-    }
-
-    /**
-     * Return a formatted string. Modified version of sprintf using colon(:)
-     *
-     * @param string $string
-     * @param array  $params
-     *
-     * @return String
-     * @throws Exception
-     */
-    public function sprintf(string $string, ...$params): string
-    {
-        preg_match_all('/\:([A-Za-z0-9_]+)/', $string, $matches);
-        $matches = $matches[1];
-
-        if (count($matches)) {
-            $tokens   = [];
-            $replaces = [];
-
-            foreach ($matches as $key => $value) {
-                if (count($params) > 1 || !is_array($params[0])) {
-                    if (!array_key_exists($key, $params)) {
-                        throw new Exception('Too few arguments, missing argument: ' . $key);
-                    }
-                    $replaces[] = $params[$key];
-                } else {
-                    if (!array_key_exists($value, $params[0])) {
-                        throw new Exception('Missing array argument: ' . $key);
-                    }
-                    $replaces[] = $params[0][$value];
-                }
-                $tokens[] = ':' . $value;
-            }
-
-            $string = str_replace($tokens, $replaces, $string);
-        }
-
-        return $string;
-    }
+		}
+
+		/** Methods */
+		switch ($method) {
+			/** @see http://www.w3.org/Protocols/rfc2616/rfc2616-sec9.html#sec9.7 */
+			case Request::METHOD_DELETE:
+				/** @see http://tools.ietf.org/html/rfc5789 */
+			case Request::METHOD_PATCH:
+				$curl->setOption([
+					CURLOPT_CUSTOMREQUEST => $method,
+					CURLOPT_POST          => true,
+					CURLOPT_POSTFIELDS    => json_encode(array_filter($postFields))
+				]);
+				break;
+
+			/** @see http://www.w3.org/Protocols/rfc2616/rfc2616-sec9.html#sec9.3 */
+			case Request::METHOD_GET:
+				$curl->setOption(CURLOPT_HTTPGET, true);
+				break;
+
+			/** @see http://www.w3.org/Protocols/rfc2616/rfc2616-sec9.html#sec9.4 */
+			case Request::METHOD_HEAD:
+				$curl->setOption([
+					CURLOPT_CUSTOMREQUEST => $method,
+					CURLOPT_NOBODY        => true
+				]);
+				break;
+
+			/** @see http://www.w3.org/Protocols/rfc2616/rfc2616-sec9.html#sec9.5 */
+			case Request::METHOD_POST:
+				$curl->setOption([
+					CURLOPT_POST       => true,
+					CURLOPT_POSTFIELDS => json_encode(array_filter($postFields))
+				]);
+				break;
+
+			/** @see http://www.w3.org/Protocols/rfc2616/rfc2616-sec9.html#sec9.6 */
+			case Request::METHOD_PUT:
+				$curl->setOption([
+					CURLOPT_CUSTOMREQUEST => $method,
+					CURLOPT_PUT           => true,
+					CURLOPT_HTTPHEADER    => [
+						'X-HTTP-Method-Override: PUT',
+						'Content-type: application/x-www-form-urlencoded'
+					]
+				]);
+				break;
+
+			default:
+				break;
+		}
+
+		$curl->success(function (CurlClient $instance) {
+			$this->headers = $instance->getHeaders();
+			$this->success = $instance->getResponse();
+			$data          = json_decode($this->success, true);
+			if (JSON_ERROR_NONE === json_last_error()) {
+				$this->success = $data;
+			}
+		});
+		$curl->error(function (CurlClient $instance) {
+			$this->headers = $instance->getHeaders();
+			$this->failure = $instance->getResponse();
+			$data          = json_decode($this->failure, true);
+			if (JSON_ERROR_NONE === json_last_error()) {
+				$this->failure = $data;
+			}
+		});
+		$curl->perform();
+
+		return (array)($this->success ?? $this->failure);
+	}
+
+	/**
+	 * Return a formatted string. Modified version of sprintf using colon(:)
+	 *
+	 * @param string $string
+	 * @param array  $params
+	 *
+	 * @return String
+	 * @throws Exception
+	 */
+	public function sprintf(string $string, ...$params): string
+	{
+		preg_match_all('/\:([A-Za-z0-9_]+)/', $string, $matches);
+		$matches = $matches[1];
+
+		if (count($matches)) {
+			$tokens   = [];
+			$replaces = [];
+
+			foreach ($matches as $key => $value) {
+				if (count($params) > 1 || !is_array($params[0])) {
+					if (!array_key_exists($key, $params)) {
+						throw new Exception('Too few arguments, missing argument: ' . $key);
+					}
+					$replaces[] = $params[$key];
+				} else {
+					if (!array_key_exists($value, $params[0])) {
+						throw new Exception('Missing array argument: ' . $key);
+					}
+					$replaces[] = $params[0][$value];
+				}
+				$tokens[] = ':' . $value;
+			}
+
+			$string = str_replace($tokens, $replaces, $string);
+		}
+
+		return $string;
+	}
 }

lib/GitHub/Event/Payload.php

Indentation +200 added lines, -200 removed lines

@@ -14,204 +14,204 @@
  */
 class Payload implements EventInterface
 {
-    /** @var WebHook */
-    protected $webHook;
-
-    /** @var null|string */
-    protected $secret = null;
-
-    /** @var string|resource */
-    protected $rawData;
-
-    /** @var mixed */
-    protected $parsedData;
-
-    /** @var ServerBag */
-    protected $serverBag;
-
-    /**
-     * Constructor, pass a WebHook object
-     *
-     * @param WebHook $webHook
-     */
-    public function __construct(WebHook $webHook)
-    {
-        $this->setWebHook($webHook);
-        $request = $webHook->getRequest();
-        $this->setRawData($request->getContent());
-        $this->serverBag = $request->server;
-    }
-
-    /**
-     * Get webHook
-     *
-     * @return null|WebHook
-     */
-    public function getWebHook()
-    {
-        return $this->webHook;
-    }
-
-    /**
-     * Set webHook
-     *
-     * @param mixed $webHook
-     *
-     * @return Payload
-     */
-    public function setWebHook($webHook): Payload
-    {
-        $this->webHook = $webHook;
-
-        return $this;
-    }
-
-    /**
-     * Set secret, encode this secret with Hmac, SHA1 method
-     *
-     * @param string $secret
-     *
-     * @return Payload
-     */
-    public function setSecret(string $secret): Payload
-    {
-        $this->secret = hash_hmac('sha1', $this->rawData, $secret);
-
-        return $this;
-    }
-
-    /**
-     * Get secret
-     *
-     * @return null|string
-     */
-    public function getSecret()
-    {
-        return $this->secret;
-    }
-
-    /**
-     * Get rawData
-     *
-     * @return resource|string
-     */
-    public function getRawData()
-    {
-        return $this->rawData;
-    }
-
-    /**
-     * Set rawData
-     *
-     * @param resource|string $rawData
-     *
-     * @return Payload
-     */
-    public function setRawData($rawData): Payload
-    {
-        $this->rawData = $rawData;
-
-        return $this;
-    }
-
-    /**
-     * Get parsedData
-     *
-     * @return mixed
-     */
-    public function getData()
-    {
-        return $this->parsedData;
-    }
-
-    /**
-     * Set parsedData
-     *
-     * @param mixed $parsedData
-     *
-     * @return Payload
-     */
-    protected function setParsedData($parsedData): Payload
-    {
-        $data = json_decode($parsedData);
-        if (JSON_ERROR_NONE === json_last_error()) {
-            $this->parsedData = $data;
-        }
-
-        return $this;
-    }
-
-    /**
-     * Debugger
-     *
-     * @return array
-     */
-    public function __debugInfo(): array
-    {
-        return [
-            'ramData'     => (array)$this->getRawData(),
-            'jsonEncoded' => json_decode($this->getRawData())
-        ];
-    }
-
-    /**
-     * Parse raw data
-     *
-     * @return Payload
-     * @throws BadSignatureException
-     * @throws \Exception
-     */
-    public function parse(): Payload
-    {
-        /** Check signature from header */
-        if (!$this->_checkSignature()) {
-            throw new BadSignatureException('Hook secret does not match.');
-        }
-
-        /** Get data from different locations according to content-type */
-        switch ($this->serverBag->get('CONTENT_TYPE')) {
-            case 'application/json':
-                $data = $this->getRawData();
-                break;
-
-            case 'application/x-www-form-urlencoded':
-                $data = $_POST['payload'];
-                break;
-
-            default:
-                throw new Exception('Unsupported content type: "' . $this->serverBag->get('CONTENT_TYPE') . '"');
-        }
-        $this->setParsedData($data);
-
-        return $this;
-    }
-
-    /**
-     * Check X-Hub-Signature
-     *
-     * @throws BadSignatureException
-     * @return bool
-     */
-    private function _checkSignature(): bool
-    {
-        $secret           = $this->getSecret();
-        $httpHubSignature = $this->serverBag->get('HTTP_X_HUB_SIGNATURE');
-
-        if (null !== $secret) {
-            if ($httpHubSignature) {
-                /**
-                 * Split signature into algorithm and hash
-                 *
-                 * @link http://isometriks.com/verify-github-webhooks-with-php
-                 */
-                list(, $hash) = explode('=', $httpHubSignature, 2);
-
-                return $secret == $hash;
-            }
-
-            throw new BadSignatureException('HTTP header "X-Hub-Signature" is missing.');
-        }
-
-        return true;
-    }
+	/** @var WebHook */
+	protected $webHook;
+
+	/** @var null|string */
+	protected $secret = null;
+
+	/** @var string|resource */
+	protected $rawData;
+
+	/** @var mixed */
+	protected $parsedData;
+
+	/** @var ServerBag */
+	protected $serverBag;
+
+	/**
+	 * Constructor, pass a WebHook object
+	 *
+	 * @param WebHook $webHook
+	 */
+	public function __construct(WebHook $webHook)
+	{
+		$this->setWebHook($webHook);
+		$request = $webHook->getRequest();
+		$this->setRawData($request->getContent());
+		$this->serverBag = $request->server;
+	}
+
+	/**
+	 * Get webHook
+	 *
+	 * @return null|WebHook
+	 */
+	public function getWebHook()
+	{
+		return $this->webHook;
+	}
+
+	/**
+	 * Set webHook
+	 *
+	 * @param mixed $webHook
+	 *
+	 * @return Payload
+	 */
+	public function setWebHook($webHook): Payload
+	{
+		$this->webHook = $webHook;
+
+		return $this;
+	}
+
+	/**
+	 * Set secret, encode this secret with Hmac, SHA1 method
+	 *
+	 * @param string $secret
+	 *
+	 * @return Payload
+	 */
+	public function setSecret(string $secret): Payload
+	{
+		$this->secret = hash_hmac('sha1', $this->rawData, $secret);
+
+		return $this;
+	}
+
+	/**
+	 * Get secret
+	 *
+	 * @return null|string
+	 */
+	public function getSecret()
+	{
+		return $this->secret;
+	}
+
+	/**
+	 * Get rawData
+	 *
+	 * @return resource|string
+	 */
+	public function getRawData()
+	{
+		return $this->rawData;
+	}
+
+	/**
+	 * Set rawData
+	 *
+	 * @param resource|string $rawData
+	 *
+	 * @return Payload
+	 */
+	public function setRawData($rawData): Payload
+	{
+		$this->rawData = $rawData;
+
+		return $this;
+	}
+
+	/**
+	 * Get parsedData
+	 *
+	 * @return mixed
+	 */
+	public function getData()
+	{
+		return $this->parsedData;
+	}
+
+	/**
+	 * Set parsedData
+	 *
+	 * @param mixed $parsedData
+	 *
+	 * @return Payload
+	 */
+	protected function setParsedData($parsedData): Payload
+	{
+		$data = json_decode($parsedData);
+		if (JSON_ERROR_NONE === json_last_error()) {
+			$this->parsedData = $data;
+		}
+
+		return $this;
+	}
+
+	/**
+	 * Debugger
+	 *
+	 * @return array
+	 */
+	public function __debugInfo(): array
+	{
+		return [
+			'ramData'     => (array)$this->getRawData(),
+			'jsonEncoded' => json_decode($this->getRawData())
+		];
+	}
+
+	/**
+	 * Parse raw data
+	 *
+	 * @return Payload
+	 * @throws BadSignatureException
+	 * @throws \Exception
+	 */
+	public function parse(): Payload
+	{
+		/** Check signature from header */
+		if (!$this->_checkSignature()) {
+			throw new BadSignatureException('Hook secret does not match.');
+		}
+
+		/** Get data from different locations according to content-type */
+		switch ($this->serverBag->get('CONTENT_TYPE')) {
+			case 'application/json':
+				$data = $this->getRawData();
+				break;
+
+			case 'application/x-www-form-urlencoded':
+				$data = $_POST['payload'];
+				break;
+
+			default:
+				throw new Exception('Unsupported content type: "' . $this->serverBag->get('CONTENT_TYPE') . '"');
+		}
+		$this->setParsedData($data);
+
+		return $this;
+	}
+
+	/**
+	 * Check X-Hub-Signature
+	 *
+	 * @throws BadSignatureException
+	 * @return bool
+	 */
+	private function _checkSignature(): bool
+	{
+		$secret           = $this->getSecret();
+		$httpHubSignature = $this->serverBag->get('HTTP_X_HUB_SIGNATURE');
+
+		if (null !== $secret) {
+			if ($httpHubSignature) {
+				/**
+				 * Split signature into algorithm and hash
+				 *
+				 * @link http://isometriks.com/verify-github-webhooks-with-php
+				 */
+				list(, $hash) = explode('=', $httpHubSignature, 2);
+
+				return $secret == $hash;
+			}
+
+			throw new BadSignatureException('HTTP header "X-Hub-Signature" is missing.');
+		}
+
+		return true;
+	}
 }

lib/GitHub/Event/EventInterface.php

Indentation +12 added lines, -12 removed lines

@@ -11,17 +11,17 @@
 interface EventInterface
 {
 
-    /**
-     * Constructor, pass a WebHook object
-     *
-     * @param WebHook $webHook
-     */
-    public function __construct(WebHook $webHook);
+	/**
+	 * Constructor, pass a WebHook object
+	 *
+	 * @param WebHook $webHook
+	 */
+	public function __construct(WebHook $webHook);
 
-    /**
-     * Parse raw data
-     *
-     * @return Payload
-     */
-    public function parse(): Payload;
+	/**
+	 * Parse raw data
+	 *
+	 * @return Payload
+	 */
+	public function parse(): Payload;
 }

lib/GitHub/Receiver/Search.php

Indentation +85 added lines, -85 removed lines

@@ -12,93 +12,93 @@
 class Search extends AbstractReceiver
 {
 
-    /** Available sub-Receiver */
-    const REPOSITORIES  = 'Repositories';
-    const CODE          = 'Code';
-    const ISSUES        = 'Issues';
-    const USERS         = 'Users';
-    const LEGACY_SEARCH = 'LegacySearch';
+	/** Available sub-Receiver */
+	const REPOSITORIES  = 'Repositories';
+	const CODE          = 'Code';
+	const ISSUES        = 'Issues';
+	const USERS         = 'Users';
+	const LEGACY_SEARCH = 'LegacySearch';
 
-    /**
-     * Search repositories
-     *
-     * @link https://developer.github.com/v3/search/#search-repositories
-     *
-     * @param string $q
-     * @param string $sort
-     * @param string $order
-     *
-     * @return array
-     * @throws \Exception
-     */
-    public function searchRepositories(
-        string $q,
-        string $sort = null,
-        string $order = AbstractApi::DIRECTION_DESC
-    ): array {
-        return $this->getApi()->request($this->getApi()->sprintf(
-            '/search/repositories?:args',
-            http_build_query(['q' => $q, 'sort' => $sort, 'order' => $order])
-        ));
-    }
+	/**
+	 * Search repositories
+	 *
+	 * @link https://developer.github.com/v3/search/#search-repositories
+	 *
+	 * @param string $q
+	 * @param string $sort
+	 * @param string $order
+	 *
+	 * @return array
+	 * @throws \Exception
+	 */
+	public function searchRepositories(
+		string $q,
+		string $sort = null,
+		string $order = AbstractApi::DIRECTION_DESC
+	): array {
+		return $this->getApi()->request($this->getApi()->sprintf(
+			'/search/repositories?:args',
+			http_build_query(['q' => $q, 'sort' => $sort, 'order' => $order])
+		));
+	}
 
-    /**
-     * Search code
-     *
-     * @link https://developer.github.com/v3/search/#search-code
-     *
-     * @param string $q
-     * @param string $sort
-     * @param string $order
-     *
-     * @return array
-     * @throws \Exception
-     */
-    public function searchCode(string $q, string $sort = null, string $order = AbstractApi::DIRECTION_DESC): array
-    {
-        return $this->getApi()->request($this->getApi()->sprintf(
-            '/search/code?:args',
-            http_build_query(['q' => $q, 'sort' => $sort, 'order' => $order])
-        ));
-    }
+	/**
+	 * Search code
+	 *
+	 * @link https://developer.github.com/v3/search/#search-code
+	 *
+	 * @param string $q
+	 * @param string $sort
+	 * @param string $order
+	 *
+	 * @return array
+	 * @throws \Exception
+	 */
+	public function searchCode(string $q, string $sort = null, string $order = AbstractApi::DIRECTION_DESC): array
+	{
+		return $this->getApi()->request($this->getApi()->sprintf(
+			'/search/code?:args',
+			http_build_query(['q' => $q, 'sort' => $sort, 'order' => $order])
+		));
+	}
 
-    /**
-     * Search issues
-     *
-     * @link https://developer.github.com/v3/search/#search-issues
-     *
-     * @param string $q
-     * @param string $sort
-     * @param string $order
-     *
-     * @return array
-     * @throws \Exception
-     */
-    public function searchIssues(string $q, string $sort = null, string $order = AbstractApi::DIRECTION_DESC): array
-    {
-        return $this->getApi()->request($this->getApi()->sprintf(
-            '/search/issues?:args',
-            http_build_query(['q' => $q, 'sort' => $sort, 'order' => $order])
-        ));
-    }
+	/**
+	 * Search issues
+	 *
+	 * @link https://developer.github.com/v3/search/#search-issues
+	 *
+	 * @param string $q
+	 * @param string $sort
+	 * @param string $order
+	 *
+	 * @return array
+	 * @throws \Exception
+	 */
+	public function searchIssues(string $q, string $sort = null, string $order = AbstractApi::DIRECTION_DESC): array
+	{
+		return $this->getApi()->request($this->getApi()->sprintf(
+			'/search/issues?:args',
+			http_build_query(['q' => $q, 'sort' => $sort, 'order' => $order])
+		));
+	}
 
-    /**
-     * Search users
-     *
-     * @link https://developer.github.com/v3/search/#search-users
-     *
-     * @param string $q
-     * @param string $sort
-     * @param string $order
-     *
-     * @return array
-     * @throws \Exception
-     */
-    public function searchUsers(string $q, string $sort = null, string $order = AbstractApi::DIRECTION_DESC): array
-    {
-        return $this->getApi()->request($this->getApi()->sprintf(
-            '/search/users?:args',
-            http_build_query(['q' => $q, 'sort' => $sort, 'order' => $order])
-        ));
-    }
+	/**
+	 * Search users
+	 *
+	 * @link https://developer.github.com/v3/search/#search-users
+	 *
+	 * @param string $q
+	 * @param string $sort
+	 * @param string $order
+	 *
+	 * @return array
+	 * @throws \Exception
+	 */
+	public function searchUsers(string $q, string $sort = null, string $order = AbstractApi::DIRECTION_DESC): array
+	{
+		return $this->getApi()->request($this->getApi()->sprintf(
+			'/search/users?:args',
+			http_build_query(['q' => $q, 'sort' => $sort, 'order' => $order])
+		));
+	}
 }

lib/GitHub/Receiver/PullRequests.php

Indentation +199 added lines, -199 removed lines

@@ -14,213 +14,213 @@
 class PullRequests extends AbstractReceiver
 {
 
-    /** Available sub-Receiver */
-    const REVIEW_COMMENTS = 'ReviewComments';
+	/** Available sub-Receiver */
+	const REVIEW_COMMENTS = 'ReviewComments';
 
-    /**
-     * List pull requests
-     *
-     * @link https://developer.github.com/v3/pulls/#list-pull-requests
-     *
-     * @param string $state
-     * @param string $head
-     * @param string $base
-     * @param string $sort
-     * @param string $direction
-     *
-     * @return array
-     * @throws \Exception
-     */
-    public function listPullRequests(
-        string $state = AbstractApi::STATE_OPEN,
-        string $head = null,
-        string $base = null,
-        string $sort = AbstractApi::SORT_CREATED,
-        string $direction = AbstractApi::DIRECTION_ASC
-    ): array {
-        return $this->getApi()->request($this->getApi()->sprintf(
-            '/repos/:owner/:repo/pulls?:args',
-            $this->getOwner(),
-            $this->getRepo(),
-            http_build_query([
-                'state'     => $state,
-                'head'      => $head,
-                'base'      => $base,
-                'sort'      => $sort,
-                'direction' => $direction
-            ])
-        ));
-    }
+	/**
+	 * List pull requests
+	 *
+	 * @link https://developer.github.com/v3/pulls/#list-pull-requests
+	 *
+	 * @param string $state
+	 * @param string $head
+	 * @param string $base
+	 * @param string $sort
+	 * @param string $direction
+	 *
+	 * @return array
+	 * @throws \Exception
+	 */
+	public function listPullRequests(
+		string $state = AbstractApi::STATE_OPEN,
+		string $head = null,
+		string $base = null,
+		string $sort = AbstractApi::SORT_CREATED,
+		string $direction = AbstractApi::DIRECTION_ASC
+	): array {
+		return $this->getApi()->request($this->getApi()->sprintf(
+			'/repos/:owner/:repo/pulls?:args',
+			$this->getOwner(),
+			$this->getRepo(),
+			http_build_query([
+				'state'     => $state,
+				'head'      => $head,
+				'base'      => $base,
+				'sort'      => $sort,
+				'direction' => $direction
+			])
+		));
+	}
 
-    /**
-     * Get a single pull request
-     *
-     * @link https://developer.github.com/v3/pulls/#get-a-single-pull-request
-     *
-     * @param int $number
-     *
-     * @return array
-     * @throws \Exception
-     */
-    public function getSinglePullRequest(int $number): array
-    {
-        return $this->getApi()->request($this->getApi()->sprintf(
-            '/repos/:owner/:repo/pulls/:number',
-            $this->getOwner(),
-            $this->getRepo(),
-            $number
-        ));
-    }
+	/**
+	 * Get a single pull request
+	 *
+	 * @link https://developer.github.com/v3/pulls/#get-a-single-pull-request
+	 *
+	 * @param int $number
+	 *
+	 * @return array
+	 * @throws \Exception
+	 */
+	public function getSinglePullRequest(int $number): array
+	{
+		return $this->getApi()->request($this->getApi()->sprintf(
+			'/repos/:owner/:repo/pulls/:number',
+			$this->getOwner(),
+			$this->getRepo(),
+			$number
+		));
+	}
 
-    /**
-     * Create a pull request
-     *
-     * @link https://developer.github.com/v3/pulls/#create-a-pull-request
-     *
-     * @param string $title
-     * @param string $head
-     * @param string $base
-     * @param string $body
-     *
-     * @return array
-     * @throws \Exception
-     */
-    public function createPullrequest(string $title, string $head, string $base, string $body): array
-    {
-        return $this->getApi()->request($this->getApi()->sprintf(
-            '/repos/:owner/:repo/pulls',
-            $this->getOwner(),
-            $this->getRepo()
-        ), Request::METHOD_POST, [
-                'title' => $title,
-                'head'  => $head,
-                'base'  => $base,
-                'body'  => $body
-            ]);
-    }
+	/**
+	 * Create a pull request
+	 *
+	 * @link https://developer.github.com/v3/pulls/#create-a-pull-request
+	 *
+	 * @param string $title
+	 * @param string $head
+	 * @param string $base
+	 * @param string $body
+	 *
+	 * @return array
+	 * @throws \Exception
+	 */
+	public function createPullrequest(string $title, string $head, string $base, string $body): array
+	{
+		return $this->getApi()->request($this->getApi()->sprintf(
+			'/repos/:owner/:repo/pulls',
+			$this->getOwner(),
+			$this->getRepo()
+		), Request::METHOD_POST, [
+				'title' => $title,
+				'head'  => $head,
+				'base'  => $base,
+				'body'  => $body
+			]);
+	}
 
-    /**
-     * Update a pull request
-     *
-     * @link https://developer.github.com/v3/pulls/#update-a-pull-request
-     *
-     * @param int    $number
-     * @param string $title
-     * @param string $body
-     * @param string $state
-     *
-     * @return array
-     * @throws \Exception
-     */
-    public function updatePullRequest(
-        int $number,
-        string $title = null,
-        string $body = null,
-        string $state = null
-    ): array {
-        return $this->getApi()->request($this->getApi()->sprintf(
-            '/repos/:owner/:repo/pulls/:number',
-            $this->getOwner(),
-            $this->getRepo(),
-            $number
-        ), Request::METHOD_PATCH, [
-                'title' => $title,
-                'body'  => $body,
-                'state' => $state
-            ]);
-    }
+	/**
+	 * Update a pull request
+	 *
+	 * @link https://developer.github.com/v3/pulls/#update-a-pull-request
+	 *
+	 * @param int    $number
+	 * @param string $title
+	 * @param string $body
+	 * @param string $state
+	 *
+	 * @return array
+	 * @throws \Exception
+	 */
+	public function updatePullRequest(
+		int $number,
+		string $title = null,
+		string $body = null,
+		string $state = null
+	): array {
+		return $this->getApi()->request($this->getApi()->sprintf(
+			'/repos/:owner/:repo/pulls/:number',
+			$this->getOwner(),
+			$this->getRepo(),
+			$number
+		), Request::METHOD_PATCH, [
+				'title' => $title,
+				'body'  => $body,
+				'state' => $state
+			]);
+	}
 
-    /**
-     * List commits on a pull request
-     *
-     * @link https://developer.github.com/v3/pulls/#list-commits-on-a-pull-request
-     *
-     * @param int $number
-     *
-     * @return array
-     * @throws \Exception
-     */
-    public function listCommits(int $number): array
-    {
-        return $this->getApi()->request($this->getApi()
-                                             ->sprintf(
-                                                 '/repos/:owner/:repo/pulls/:number/commits',
-                                                 $this->getOwner(),
-                                                 $this->getRepo(),
-                                                 $number
-                                             ));
-    }
+	/**
+	 * List commits on a pull request
+	 *
+	 * @link https://developer.github.com/v3/pulls/#list-commits-on-a-pull-request
+	 *
+	 * @param int $number
+	 *
+	 * @return array
+	 * @throws \Exception
+	 */
+	public function listCommits(int $number): array
+	{
+		return $this->getApi()->request($this->getApi()
+											 ->sprintf(
+												 '/repos/:owner/:repo/pulls/:number/commits',
+												 $this->getOwner(),
+												 $this->getRepo(),
+												 $number
+											 ));
+	}
 
-    /**
-     * List pull requests files
-     *
-     * @link https://developer.github.com/v3/pulls/#list-pull-requests-files
-     *
-     * @param int $number
-     *
-     * @return array
-     * @throws \Exception
-     */
-    public function listPullRequestsFiles(int $number): array
-    {
-        return $this->getApi()->request($this->getApi()
-                                             ->sprintf(
-                                                 '/repos/:owner/:repo/pulls/:number/files',
-                                                 $this->getOwner(),
-                                                 $this->getRepo(),
-                                                 $number
-                                             ));
-    }
+	/**
+	 * List pull requests files
+	 *
+	 * @link https://developer.github.com/v3/pulls/#list-pull-requests-files
+	 *
+	 * @param int $number
+	 *
+	 * @return array
+	 * @throws \Exception
+	 */
+	public function listPullRequestsFiles(int $number): array
+	{
+		return $this->getApi()->request($this->getApi()
+											 ->sprintf(
+												 '/repos/:owner/:repo/pulls/:number/files',
+												 $this->getOwner(),
+												 $this->getRepo(),
+												 $number
+											 ));
+	}
 
-    /**
-     * Get if a pull request has been merged
-     *
-     * @link https://developer.github.com/v3/pulls/#get-if-a-pull-request-has-been-merged
-     *
-     * @param int $number
-     *
-     * @return bool
-     * @throws \Exception
-     */
-    public function checkPullRequestsMerged(int $number): bool
-    {
-        $this->getApi()->request($this->getApi()->sprintf(
-            '/repos/:owner/:repo/pulls/:number/merge',
-            $this->getOwner(),
-            $this->getRepo(),
-            $number
-        ));
+	/**
+	 * Get if a pull request has been merged
+	 *
+	 * @link https://developer.github.com/v3/pulls/#get-if-a-pull-request-has-been-merged
+	 *
+	 * @param int $number
+	 *
+	 * @return bool
+	 * @throws \Exception
+	 */
+	public function checkPullRequestsMerged(int $number): bool
+	{
+		$this->getApi()->request($this->getApi()->sprintf(
+			'/repos/:owner/:repo/pulls/:number/merge',
+			$this->getOwner(),
+			$this->getRepo(),
+			$number
+		));
 
-        if ($this->getApi()->getHeaders()['Status'] == '204 No Content') {
-            return true;
-        }
+		if ($this->getApi()->getHeaders()['Status'] == '204 No Content') {
+			return true;
+		}
 
-        return false;
-    }
+		return false;
+	}
 
-    /**
-     * Merge a pull request (Merge Button)
-     *
-     * @link https://developer.github.com/v3/pulls/#merge-a-pull-request-merge-button
-     *
-     * @param int    $number
-     * @param string $commitMessage
-     * @param string $sha
-     *
-     * @return array
-     * @throws \Exception
-     */
-    public function mergePullRequest(int $number, string $commitMessage = null, string $sha = null): array
-    {
-        return $this->getApi()->request($this->getApi()
-                                             ->sprintf(
-                                                 '/repos/:owner/:repo/pulls/:number/merge',
-                                                 $this->getOwner(),
-                                                 $this->getRepo(),
-                                                 $number
-                                             ), Request::METHOD_PUT, [
-                'commit_message' => $commitMessage,
-                'sha'            => $sha
-                                                 ]);
-    }
+	/**
+	 * Merge a pull request (Merge Button)
+	 *
+	 * @link https://developer.github.com/v3/pulls/#merge-a-pull-request-merge-button
+	 *
+	 * @param int    $number
+	 * @param string $commitMessage
+	 * @param string $sha
+	 *
+	 * @return array
+	 * @throws \Exception
+	 */
+	public function mergePullRequest(int $number, string $commitMessage = null, string $sha = null): array
+	{
+		return $this->getApi()->request($this->getApi()
+											 ->sprintf(
+												 '/repos/:owner/:repo/pulls/:number/merge',
+												 $this->getOwner(),
+												 $this->getRepo(),
+												 $number
+											 ), Request::METHOD_PUT, [
+				'commit_message' => $commitMessage,
+				'sha'            => $sha
+												 ]);
+	}
 }

lib/GitHub/Receiver/Issues/Events.php

Indentation +50 added lines, -50 removed lines

@@ -10,56 +10,56 @@
 class Events extends AbstractIssues
 {
 
-    /**
-     * List events for an issue
-     *
-     * @link https://developer.github.com/v3/issues/events/#list-events-for-an-issue
-     *
-     * @param int $issueNumber
-     *
-     * @return array
-     */
-    public function listIssueEvents(int $issueNumber): array
-    {
-        return $this->getApi()->request($this->getApi()->sprintf(
-            '/repos/:owner/:repo/issues/:issue_number/events',
-            $this->getIssues()->getOwner(),
-            $this->getIssues()->getRepo(),
-            $issueNumber
-        ));
-    }
+	/**
+	 * List events for an issue
+	 *
+	 * @link https://developer.github.com/v3/issues/events/#list-events-for-an-issue
+	 *
+	 * @param int $issueNumber
+	 *
+	 * @return array
+	 */
+	public function listIssueEvents(int $issueNumber): array
+	{
+		return $this->getApi()->request($this->getApi()->sprintf(
+			'/repos/:owner/:repo/issues/:issue_number/events',
+			$this->getIssues()->getOwner(),
+			$this->getIssues()->getRepo(),
+			$issueNumber
+		));
+	}
 
-    /**
-     * List events for a repository
-     *
-     * @link https://developer.github.com/v3/issues/events/#list-events-for-a-repository
-     * @return array
-     */
-    public function listRepositoryEvents(): array
-    {
-        return $this->getApi()->request($this->getApi()->sprintf(
-            '/repos/:owner/:repo/issues/events',
-            $this->getIssues()->getOwner(),
-            $this->getIssues()->getRepo()
-        ));
-    }
+	/**
+	 * List events for a repository
+	 *
+	 * @link https://developer.github.com/v3/issues/events/#list-events-for-a-repository
+	 * @return array
+	 */
+	public function listRepositoryEvents(): array
+	{
+		return $this->getApi()->request($this->getApi()->sprintf(
+			'/repos/:owner/:repo/issues/events',
+			$this->getIssues()->getOwner(),
+			$this->getIssues()->getRepo()
+		));
+	}
 
-    /**
-     * Get a single event
-     *
-     * @link https://developer.github.com/v3/issues/events/#get-a-single-event
-     *
-     * @param int $id
-     *
-     * @return array
-     */
-    public function getEvent(int $id): array
-    {
-        return $this->getApi()->request($this->getApi()->sprintf(
-            '/repos/:owner/:repo/issues/events/:id',
-            $this->getIssues()->getOwner(),
-            $this->getIssues()->getRepo(),
-            $id
-        ));
-    }
+	/**
+	 * Get a single event
+	 *
+	 * @link https://developer.github.com/v3/issues/events/#get-a-single-event
+	 *
+	 * @param int $id
+	 *
+	 * @return array
+	 */
+	public function getEvent(int $id): array
+	{
+		return $this->getApi()->request($this->getApi()->sprintf(
+			'/repos/:owner/:repo/issues/events/:id',
+			$this->getIssues()->getOwner(),
+			$this->getIssues()->getRepo(),
+			$id
+		));
+	}
 }

lib/GitHub/Receiver/Issues/Milestones.php

Indentation +129 added lines, -129 removed lines

@@ -14,139 +14,139 @@
 class Milestones extends AbstractIssues
 {
 
-    /**
-     * List milestones for a repository
-     *
-     * @link https://developer.github.com/v3/issues/milestones/#list-milestones-for-a-repository
-     *
-     * @param string $state
-     * @param string $sort
-     * @param string $direction
-     *
-     * @return array
-     */
-    public function listMilestones(
-        string $state = AbstractApi::STATE_OPEN,
-        string $sort = AbstractApi::SORT_DUE_DATE,
-        string $direction = AbstractApi::DIRECTION_ASC
-    ): array {
-        return $this->getApi()->request($this->getApi()->sprintf(
-            '/repos/:owner/:repo/milestones?:args',
-            $this->getIssues()->getOwner(),
-            $this->getIssues()->getRepo(),
-            http_build_query([
-                'state'     => $state,
-                'sort'      => $sort,
-                'direction' => $direction
-            ])
-        ));
-    }
+	/**
+	 * List milestones for a repository
+	 *
+	 * @link https://developer.github.com/v3/issues/milestones/#list-milestones-for-a-repository
+	 *
+	 * @param string $state
+	 * @param string $sort
+	 * @param string $direction
+	 *
+	 * @return array
+	 */
+	public function listMilestones(
+		string $state = AbstractApi::STATE_OPEN,
+		string $sort = AbstractApi::SORT_DUE_DATE,
+		string $direction = AbstractApi::DIRECTION_ASC
+	): array {
+		return $this->getApi()->request($this->getApi()->sprintf(
+			'/repos/:owner/:repo/milestones?:args',
+			$this->getIssues()->getOwner(),
+			$this->getIssues()->getRepo(),
+			http_build_query([
+				'state'     => $state,
+				'sort'      => $sort,
+				'direction' => $direction
+			])
+		));
+	}
 
-    /**
-     * Get a single milestone
-     *
-     * @link https://developer.github.com/v3/issues/milestones/#get-a-single-milestone
-     *
-     * @param int $number
-     *
-     * @return array
-     */
-    public function getMilestone(int $number): array
-    {
-        return $this->getApi()->request($this->getApi()->sprintf(
-            '/repos/:owner/:repo/milestones/:number',
-            $this->getIssues()->getOwner(),
-            $this->getIssues()->getRepo(),
-            $number
-        ));
-    }
+	/**
+	 * Get a single milestone
+	 *
+	 * @link https://developer.github.com/v3/issues/milestones/#get-a-single-milestone
+	 *
+	 * @param int $number
+	 *
+	 * @return array
+	 */
+	public function getMilestone(int $number): array
+	{
+		return $this->getApi()->request($this->getApi()->sprintf(
+			'/repos/:owner/:repo/milestones/:number',
+			$this->getIssues()->getOwner(),
+			$this->getIssues()->getRepo(),
+			$number
+		));
+	}
 
-    /**
-     * Create a milestone
-     *
-     * @link https://developer.github.com/v3/issues/milestones/#create-a-milestone
-     *
-     * @param string $title
-     * @param string $state
-     * @param string $description
-     * @param string $dueOn
-     *
-     * @return array
-     */
-    public function createMilestone(
-        string $title,
-        string $state = AbstractApi::STATE_OPEN,
-        string $description = '',
-        string $dueOn = ''
-    ): array {
-        return $this->getApi()->request($this->getApi()
-                                             ->sprintf(
-                                                 '/repos/:owner/:repo/milestones',
-                                                 $this->getIssues()->getOwner(),
-                                                 $this->getIssues()->getRepo()
-                                             ), Request::METHOD_POST, [
-                'title'       => $title,
-                'state'       => $state,
-                'description' => $description,
-                'due_on'      => (new DateTime($dueOn))->format(DateTime::ATOM)
-                                                 ]);
-    }
+	/**
+	 * Create a milestone
+	 *
+	 * @link https://developer.github.com/v3/issues/milestones/#create-a-milestone
+	 *
+	 * @param string $title
+	 * @param string $state
+	 * @param string $description
+	 * @param string $dueOn
+	 *
+	 * @return array
+	 */
+	public function createMilestone(
+		string $title,
+		string $state = AbstractApi::STATE_OPEN,
+		string $description = '',
+		string $dueOn = ''
+	): array {
+		return $this->getApi()->request($this->getApi()
+											 ->sprintf(
+												 '/repos/:owner/:repo/milestones',
+												 $this->getIssues()->getOwner(),
+												 $this->getIssues()->getRepo()
+											 ), Request::METHOD_POST, [
+				'title'       => $title,
+				'state'       => $state,
+				'description' => $description,
+				'due_on'      => (new DateTime($dueOn))->format(DateTime::ATOM)
+												 ]);
+	}
 
-    /**
-     * Update a milestone
-     *
-     * @link https://developer.github.com/v3/issues/milestones/#update-a-milestone
-     *
-     * @param int    $number
-     * @param string $title
-     * @param string $state
-     * @param string $description
-     * @param string $dueOn
-     *
-     * @return array
-     */
-    public function updateMilestone(
-        int $number,
-        string $title = '',
-        string $state = AbstractApi::STATE_OPEN,
-        string $description = '',
-        string $dueOn = ''
-    ): array {
-        return $this->getApi()->request($this->getApi()->sprintf(
-            '/repos/:owner/:repo/milestones/:number',
-            $this->getIssues()->getOwner(),
-            $this->getIssues()->getRepo(),
-            $number
-        ), Request::METHOD_PATCH, [
-                'title'       => $title,
-                'state'       => $state,
-                'description' => $description,
-                'due_on'      => (new DateTime($dueOn))->format(DateTime::ATOM)
-            ]);
-    }
+	/**
+	 * Update a milestone
+	 *
+	 * @link https://developer.github.com/v3/issues/milestones/#update-a-milestone
+	 *
+	 * @param int    $number
+	 * @param string $title
+	 * @param string $state
+	 * @param string $description
+	 * @param string $dueOn
+	 *
+	 * @return array
+	 */
+	public function updateMilestone(
+		int $number,
+		string $title = '',
+		string $state = AbstractApi::STATE_OPEN,
+		string $description = '',
+		string $dueOn = ''
+	): array {
+		return $this->getApi()->request($this->getApi()->sprintf(
+			'/repos/:owner/:repo/milestones/:number',
+			$this->getIssues()->getOwner(),
+			$this->getIssues()->getRepo(),
+			$number
+		), Request::METHOD_PATCH, [
+				'title'       => $title,
+				'state'       => $state,
+				'description' => $description,
+				'due_on'      => (new DateTime($dueOn))->format(DateTime::ATOM)
+			]);
+	}
 
-    /**
-     * Delete a milestone
-     *
-     * @link https://developer.github.com/v3/issues/milestones/#delete-a-milestone
-     *
-     * @param int $number
-     *
-     * @return bool
-     */
-    public function deleteMilestone(int $number): bool
-    {
-        $this->getApi()->request($this->getApi()->sprintf(
-            '/repos/:owner/:repo/milestones/:number',
-            $this->getIssues()->getOwner(),
-            $this->getIssues()->getRepo(),
-            $number
-        ), Request::METHOD_DELETE);
+	/**
+	 * Delete a milestone
+	 *
+	 * @link https://developer.github.com/v3/issues/milestones/#delete-a-milestone
+	 *
+	 * @param int $number
+	 *
+	 * @return bool
+	 */
+	public function deleteMilestone(int $number): bool
+	{
+		$this->getApi()->request($this->getApi()->sprintf(
+			'/repos/:owner/:repo/milestones/:number',
+			$this->getIssues()->getOwner(),
+			$this->getIssues()->getRepo(),
+			$number
+		), Request::METHOD_DELETE);
 
-        if ($this->getApi()->getHeaders()['Status'] == '204 No Content') {
-            return true;
-        }
+		if ($this->getApi()->getHeaders()['Status'] == '204 No Content') {
+			return true;
+		}
 
-        return false;
-    }
+		return false;
+	}
 }

lib/GitHub/Receiver/Issues/Comments.php

Indentation +127 added lines, -127 removed lines

@@ -13,138 +13,138 @@
 class Comments extends AbstractIssues
 {
 
-    /**
-     * List comments on an issue
-     *
-     * @link https://developer.github.com/v3/issues/comments/#list-comments-on-an-issue
-     *
-     * @param int $number
-     *
-     * @return array
-     */
-    public function listIssueComments(int $number): array
-    {
-        return $this->getApi()->request($this->getApi()->sprintf(
-            '/repos/:owner/:repo/issues/:number/comments',
-            $this->getIssues()->getOwner(),
-            $this->getIssues()->getRepo(),
-            $number
-        ));
-    }
+	/**
+	 * List comments on an issue
+	 *
+	 * @link https://developer.github.com/v3/issues/comments/#list-comments-on-an-issue
+	 *
+	 * @param int $number
+	 *
+	 * @return array
+	 */
+	public function listIssueComments(int $number): array
+	{
+		return $this->getApi()->request($this->getApi()->sprintf(
+			'/repos/:owner/:repo/issues/:number/comments',
+			$this->getIssues()->getOwner(),
+			$this->getIssues()->getRepo(),
+			$number
+		));
+	}
 
-    /**
-     * List comments in a repository
-     *
-     * @link https://developer.github.com/v3/issues/comments/#list-comments-in-a-repository
-     *
-     * @param string $sort
-     * @param string $direction
-     * @param string $since
-     *
-     * @return array
-     */
-    public function listRepositoryComments(
-        string $sort = AbstractApi::SORT_CREATED,
-        string $direction = AbstractApi::DIRECTION_DESC,
-        string $since = 'now'
-    ): array {
-        return $this->getApi()->request($this->getApi()->sprintf(
-            '/repos/:owner/:repo/issues/comments?:args',
-            $this->getIssues()->getOwner(),
-            $this->getIssues()->getRepo(),
-            http_build_query([
-                'sort'      => $sort,
-                'direction' => $direction,
-                'since'     => $since
-            ])
-        ));
-    }
+	/**
+	 * List comments in a repository
+	 *
+	 * @link https://developer.github.com/v3/issues/comments/#list-comments-in-a-repository
+	 *
+	 * @param string $sort
+	 * @param string $direction
+	 * @param string $since
+	 *
+	 * @return array
+	 */
+	public function listRepositoryComments(
+		string $sort = AbstractApi::SORT_CREATED,
+		string $direction = AbstractApi::DIRECTION_DESC,
+		string $since = 'now'
+	): array {
+		return $this->getApi()->request($this->getApi()->sprintf(
+			'/repos/:owner/:repo/issues/comments?:args',
+			$this->getIssues()->getOwner(),
+			$this->getIssues()->getRepo(),
+			http_build_query([
+				'sort'      => $sort,
+				'direction' => $direction,
+				'since'     => $since
+			])
+		));
+	}
 
-    /**
-     * Get a single comment
-     *
-     * @link https://developer.github.com/v3/issues/comments/#get-a-single-comment
-     *
-     * @param int $id
-     *
-     * @return array
-     */
-    public function getComment(int $id): array
-    {
-        return $this->getApi()->request($this->getApi()->sprintf(
-            '/repos/:owner/:repo/issues/comments/:id',
-            $this->getIssues()->getOwner(),
-            $this->getIssues()->getRepo(),
-            $id
-        ));
-    }
+	/**
+	 * Get a single comment
+	 *
+	 * @link https://developer.github.com/v3/issues/comments/#get-a-single-comment
+	 *
+	 * @param int $id
+	 *
+	 * @return array
+	 */
+	public function getComment(int $id): array
+	{
+		return $this->getApi()->request($this->getApi()->sprintf(
+			'/repos/:owner/:repo/issues/comments/:id',
+			$this->getIssues()->getOwner(),
+			$this->getIssues()->getRepo(),
+			$id
+		));
+	}
 
-    /**
-     * Create a comment
-     *
-     * @link https://developer.github.com/v3/issues/comments/#create-a-comment
-     *
-     * @param int    $number
-     * @param string $body
-     *
-     * @return array
-     */
-    public function createComment(int $number, string $body): array
-    {
-        return $this->getApi()->request($this->getApi()->sprintf(
-            '/repos/:owner/:repo/issues/:number/comments',
-            $this->getIssues()->getOwner(),
-            $this->getIssues()->getRepo(),
-            $number
-        ), Request::METHOD_POST, [
-                'body' => $body
-            ]);
-    }
+	/**
+	 * Create a comment
+	 *
+	 * @link https://developer.github.com/v3/issues/comments/#create-a-comment
+	 *
+	 * @param int    $number
+	 * @param string $body
+	 *
+	 * @return array
+	 */
+	public function createComment(int $number, string $body): array
+	{
+		return $this->getApi()->request($this->getApi()->sprintf(
+			'/repos/:owner/:repo/issues/:number/comments',
+			$this->getIssues()->getOwner(),
+			$this->getIssues()->getRepo(),
+			$number
+		), Request::METHOD_POST, [
+				'body' => $body
+			]);
+	}
 
-    /**
-     * Edit a comment
-     *
-     * @link https://developer.github.com/v3/issues/comments/#edit-a-comment
-     *
-     * @param int    $id
-     * @param string $body
-     *
-     * @return array
-     */
-    public function editComment(int $id, string $body): array
-    {
-        return $this->getApi()->request($this->getApi()->sprintf(
-            '/repos/:owner/:repo/issues/comments/:id',
-            $this->getIssues()->getOwner(),
-            $this->getIssues()->getRepo(),
-            $id
-        ), Request::METHOD_PATCH, [
-                'body' => $body
-            ]);
-    }
+	/**
+	 * Edit a comment
+	 *
+	 * @link https://developer.github.com/v3/issues/comments/#edit-a-comment
+	 *
+	 * @param int    $id
+	 * @param string $body
+	 *
+	 * @return array
+	 */
+	public function editComment(int $id, string $body): array
+	{
+		return $this->getApi()->request($this->getApi()->sprintf(
+			'/repos/:owner/:repo/issues/comments/:id',
+			$this->getIssues()->getOwner(),
+			$this->getIssues()->getRepo(),
+			$id
+		), Request::METHOD_PATCH, [
+				'body' => $body
+			]);
+	}
 
-    /**
-     * Delete a comment
-     *
-     * @link https://developer.github.com/v3/issues/comments/#delete-a-comment
-     *
-     * @param int $id
-     *
-     * @return bool
-     */
-    public function deleteComment(int $id): bool
-    {
-        $this->getApi()->request($this->getApi()->sprintf(
-            '/repos/:owner/:repo/issues/comments/:id',
-            $this->getIssues()->getOwner(),
-            $this->getIssues()->getRepo(),
-            $id
-        ), Request::METHOD_DELETE);
+	/**
+	 * Delete a comment
+	 *
+	 * @link https://developer.github.com/v3/issues/comments/#delete-a-comment
+	 *
+	 * @param int $id
+	 *
+	 * @return bool
+	 */
+	public function deleteComment(int $id): bool
+	{
+		$this->getApi()->request($this->getApi()->sprintf(
+			'/repos/:owner/:repo/issues/comments/:id',
+			$this->getIssues()->getOwner(),
+			$this->getIssues()->getRepo(),
+			$id
+		), Request::METHOD_DELETE);
 
-        if ($this->getApi()->getHeaders()['Status'] == '204 No Content') {
-            return true;
-        }
+		if ($this->getApi()->getHeaders()['Status'] == '204 No Content') {
+			return true;
+		}
 
-        return false;
-    }
+		return false;
+	}
 }

lib/GitHub/Receiver/Issues/AbstractIssues.php

Indentation +35 added lines, -35 removed lines

@@ -12,44 +12,44 @@
  */
 abstract class AbstractIssues
 {
-    /** Api trait */
-    use Api;
+	/** Api trait */
+	use Api;
 
-    /** Properties */
-    protected $issues;
+	/** Properties */
+	protected $issues;
 
-    /**
-     * Constructor
-     *
-     * @param Issues $issues
-     */
-    public function __construct(Issues $issues)
-    {
-        $this->setIssues($issues);
-        $this->setApi($issues->getApi());
-    }
+	/**
+	 * Constructor
+	 *
+	 * @param Issues $issues
+	 */
+	public function __construct(Issues $issues)
+	{
+		$this->setIssues($issues);
+		$this->setApi($issues->getApi());
+	}
 
-    /**
-     * Get issues
-     *
-     * @return Issues
-     */
-    public function getIssues(): Issues
-    {
-        return $this->issues;
-    }
+	/**
+	 * Get issues
+	 *
+	 * @return Issues
+	 */
+	public function getIssues(): Issues
+	{
+		return $this->issues;
+	}
 
-    /**
-     * Set issues
-     *
-     * @param Issues $issues
-     *
-     * @return AbstractIssues
-     */
-    public function setIssues(Issues $issues): AbstractIssues
-    {
-        $this->issues = $issues;
+	/**
+	 * Set issues
+	 *
+	 * @param Issues $issues
+	 *
+	 * @return AbstractIssues
+	 */
+	public function setIssues(Issues $issues): AbstractIssues
+	{
+		$this->issues = $issues;
 
-        return $this;
-    }
+		return $this;
+	}
 }