LivePersonInc\LiveEngageLaravel\Collections\ConversationHistory

Complexity

Total Complexity

16

Size/Duplication

Total Lines	85
Duplicated Lines	0 %

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	5	1
A	find()	0	7	1
C	next()	0	31	7
C	prev()	0	31	7

<?php

namespace LivePersonInc\LiveEngageLaravel\Collections;

use Illuminate\Support\Collection;
use LivePersonInc\LiveEngageLaravel\LiveEngageLaravel;
use LivePersonInc\LiveEngageLaravel\Models\Conversation;
use LivePersonInc\LiveEngageLaravel\Models\Info;
use LivePersonInc\LiveEngageLaravel\Models\Visitor;
use LivePersonInc\LiveEngageLaravel\Models\Campaign;

class ConversationHistory extends Collection
{
    private $instance;

    public function __construct(array $models = [], LiveEngageLaravel $instance = null)
    {
        $this->instance = $instance;

        parent::__construct($models);
    }
    
    public function find($engagementID)
    {
	    $result = $this->filter(function ($value, $key) use ($engagementID) {

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

		    return $value->info->conversationId == $engagementID;
	    });
	    
	    return $result->first();
    }

    public function next()
    {
        if (! $this->instance) {
            return false;
        }

        $instance = $this->instance;

        $next = $instance->retrieveMsgHistory($instance->start, $instance->end, $instance->next);

The property $next is declared private in LivePersonInc\LiveEngageLaravel\LiveEngageLaravel. Since you implement __get, consider adding a @property or @property-read.

The property $end is declared private in LivePersonInc\LiveEngageLaravel\LiveEngageLaravel. Since you implement __get, consider adding a @property or @property-read.

The property $start is declared private in LivePersonInc\LiveEngageLaravel\LiveEngageLaravel. Since you implement __get, consider adding a @property or @property-read.

        if (property_exists($next->_metadata, 'next')) {

The property _metadata does not seem to exist on GuzzleHttp\Exception\ConnectException.

            $instance->next = $next->_metadata->next->href;

            $history = [];
            foreach ($next->conversationHistoryRecords as $item) {

The property conversationHistoryRecords does not seem to exist on GuzzleHttp\Exception\ConnectException.

	            if (property_exists($item, 'info')) {
	                $item->info = new Info((array) $item->info);
	            }
	
	            if (property_exists($item, 'visitorInfo')) {
	                $item->visitorInfo = new Visitor((array) $item->visitorInfo);
	            }
	
	            if (property_exists($item, 'campaign')) {
	                $item->campaign = new Campaign((array) $item->campaign);
	            }
                $history[] = new Conversation((array) $item);
            }

            return $this->merge(new self($history));
        } else {
            return false;
        }
    }

    public function prev()
    {
        if (! $this->instance) {
            return false;
        }

        $instance = $this->instance;

        $prev = $instance->retrieveMsgHistory($instance->start, $instance->end, $instance->prev);

The property $end is declared private in LivePersonInc\LiveEngageLaravel\LiveEngageLaravel. Since you implement __get, consider adding a @property or @property-read.

The property $start is declared private in LivePersonInc\LiveEngageLaravel\LiveEngageLaravel. Since you implement __get, consider adding a @property or @property-read.

The property $prev is declared private in LivePersonInc\LiveEngageLaravel\LiveEngageLaravel. Since you implement __get, consider adding a @property or @property-read.

        if (property_exists($prev->_metadata, 'prev')) {

The property _metadata does not seem to exist on GuzzleHttp\Exception\ConnectException.

            $instance->prev = $prev->_metadata->prev->href;

            $history = [];
            foreach ($next->conversationHistoryRecords as $item) {

The variable $next seems to be never defined.

	            if (property_exists($item, 'info')) {
	                $item->info = new Info((array) $item->info);
	            }
	
	            if (property_exists($item, 'visitorInfo')) {
	                $item->visitorInfo = new Visitor((array) $item->visitorInfo);
	            }
	
	            if (property_exists($item, 'campaign')) {
	                $item->campaign = new Campaign((array) $item->campaign);
	            }
                $history[] = new Conversation((array) $item);
            }

            return $this->merge(new self($history));
        } else {
            return false;
        }
    }
}