Borfast\Socializr\Connectors\LinkedinPage

Complexity

Total Complexity

7

Size/Duplication

Total Lines	88
Duplicated Lines	37.5 %

Coupling/Cohesion

Components	1
Dependencies	7

Importance

Changes	3
Bugs	0	Features	0

3 Methods

Rating	Name	Duplication	Size	Complexity
B	post()	10	50	5
A	getProfile()	23	23	1
A	getStats()	0	7	1

<?php

namespace Borfast\Socializr\Connectors;

use Borfast\Socializr\Exceptions\ExpiredTokenException;
use Borfast\Socializr\Exceptions\GenericPostingException;
use Borfast\Socializr\Post;
use Borfast\Socializr\Profile;
use Borfast\Socializr\Page;
use Borfast\Socializr\Response;
use Borfast\Socializr\Connectors\AbstractConnector;
use OAuth\Common\Storage\TokenStorageInterface;

class LinkedinPage extends AbstractConnector
{
    public static $provider = 'linkedin';

    public function post(Post $post)
    {
        $page_id = $post->options['page_id'];
        $path = '/companies/'.$page_id.'/shares?format=json';
        $method = 'POST';
        $params = [
            'visibility' => [
                'code' => 'anyone'
            ],
            'comment' => '',
            'content' => [
                'title' => $post->title,
                'submitted-url' => $post->url,
                'description' => $post->body,
            ],
        ];

        if (!empty($post->media)) {
            $params['content']['submitted-image-url'] = $post->media[0];
        }

        $params = json_encode($params);

        // Linkedin API requires the Content-Type header set to application/json
        $header = ['Content-Type' => 'application/json'];
        $result = $this->request($path, $method, $params, $header);

$params is of type string, but the function expects a array.

        // The response comes in JSON
        $json_result = json_decode($result, true);

        if (isset($json_result['status']) && $json_result['status'] != 200) {

This code seems to be duplicated across your project.

            $msg = "Error posting to Linkedin page. Error code from Linkedin: %s. Error message from Linkedin: %s";
            $msg = sprintf($msg, $json_result['errorCode'], $json_result['message']);

            if ($json_result['status'] == '401') {
                throw new ExpiredTokenException($msg);
            } else {
                throw new GenericPostingException($msg, $json_result['status']);
            }
        }

        $response = new Response;
        $response->setRawResponse(json_encode($result));
        $response->setProvider(static::$provider);
        $result_json = json_decode($result);
        $response->setPostId($result_json->updateKey);
        $response->setPostUrl($result_json->updateUrl);

        return $response;
    }


    public function getProfile()

This method seems to be duplicated in your project.

    {
        $path = '/people/~:(id,first-name,last-name,maiden-name,public-profile-url,formatted-name,num-connections,email-address,num-recommenders)?format=json';
        $response = $this->service->request($path);
        $profile_json = json_decode($response, true);

        $mapping = [
            'id' => 'id',
            'email' => 'emailAddress',
            'name' => 'formattedName',
            'first_name' => 'firstName',
            'middle_name' => 'maidenName',
            'last_name' => 'lastName',
            // 'username' => 'username',

58% of this comment could be valid code. Did you maybe forget this after debugging?

            'link' => 'publicProfileUrl'
        ];

        $profile = Profile::create($mapping, $profile_json);
        $profile->provider = static::$provider;
        $profile->raw_response = $response;

        return $profile;
    }

    public function getStats()
    {
        $path = 'companies/'.$this->id.':(id,num-followers)?format=json';
        $response = json_decode($this->request($path));

        return $response->numFollowers;
    }
}