Strime\Slackify\Api\UsersProfile::set()

Complexity

Conditions	15
Paths	84

Size

Total Lines

50

Duplication

Lines	50
Ratio	100 %

Importance

Changes

0

<?php
    public function set($user = NULL, $profile = NULL, $name = NULL, $value = NULL) {

This method seems to be duplicated in your project.

        // Check if the type of the variables is valid.
        if (!is_string($user) && ($user != NULL)) {
            throw new InvalidArgumentException("The type of the user variable is not valid.");
        }
        if (!is_string($profile) && ($profile != NULL)) {
            throw new InvalidArgumentException("The type of the profile variable is not valid.");
        }
        if (!is_string($name) && ($name != NULL)) {
            throw new InvalidArgumentException("The type of the name variable is not valid.");
        }
        if (!is_string($value) && ($value != NULL)) {
            throw new InvalidArgumentException("The type of the value variable is not valid.");
        }

        // Set the arguments of the request
        $arguments = array();

        if ($user != NULL) {

It seems like you are loosely comparing $user of type string|null against null; this is ambiguous if the string can be empty. Consider using a strict comparison !== instead.

            $arguments["user"] = $user;
        }
        if ($profile != NULL) {

It seems like you are loosely comparing $profile of type string|null against null; this is ambiguous if the string can be empty. Consider using a strict comparison !== instead.

            $arguments["profile"] = $profile;
        }
        if ($name != NULL) {

It seems like you are loosely comparing $name of type string|null against null; this is ambiguous if the string can be empty. Consider using a strict comparison !== instead.

            $arguments["name"] = $name;
        }
        if ($value != NULL) {

It seems like you are loosely comparing $value of type string|null against null; this is ambiguous if the string can be empty. Consider using a strict comparison !== instead.

            $arguments["value"] = $value;
        }

        $this->setUrl("users.profile.set", $arguments);

        // Send the request
        try {
            $client = new \GuzzleHttp\Client();
            $json_response = $client->request('GET', $this->getUrl(), []);
            $response = json_decode( $json_response->getBody() );
        }
        catch (RequestException $e) {
            throw new RuntimeException('The request to the API failed: '.$e->getMessage(), $e->getCode(), $e);
        }

        if($response->{'ok'} === FALSE) {
            throw new RuntimeException('The request to the API failed: '.$response->{'error'}.".");
        }

        return $json_response->getBody();
    }
}