CampaignMonitorAPIConnector

Complexity

Total Complexity

132

Size/Duplication

Total Lines	1688
Duplicated Lines	26.84 %

Coupling/Cohesion

Components	1
Dependencies	13

Importance

Changes

0

55 Methods

Rating	Name	Duplication	Size	Complexity
A	init()	0	5	1
A	setDebug()	0	4	1
A	setAllowCaching()	0	4	1
A	getAllowCaching()	0	4	1
C	getAuth()	0	48	7
B	returnResult()	0	26	5
A	getHttpStatusCode()	0	4	1
A	getFromCache()	12	12	3
A	saveToCache()	10	10	2
A	getCampaigns()	11	11	1
A	getDrafts()	11	11	1
A	getLists()	11	11	1
A	getScheduled()	0	4	1
A	getSuppressionlist()	0	19	1
A	getTemplates()	13	13	1
A	getTemplate()	13	13	1
B	createTemplate()	39	39	6
B	updateTemplate()	37	37	6
A	deleteTemplate()	10	10	1
B	createList()	0	25	2
C	createCustomField()	0	34	7
A	deleteCustomField()	10	10	1
A	deleteList()	11	11	1
A	getList()	11	11	1
A	getActiveSubscribers()	17	17	1
A	getUnconfirmedSubscribers()	17	17	1
A	getBouncedSubscribers()	17	17	1
A	getUnsubscribedSubscribers()	17	17	1
A	getDeletedSubscribers()	0	17	1
A	updateList()	0	22	2
A	getSegments()	13	13	1
A	getListStats()	11	11	1
A	getListCustomFields()	10	10	1
D	createCampaign()	0	89	11
A	deleteCampaign()	10	10	1
A	getBounces()	0	4	1
A	getClicks()	0	4	1
A	getSummary()	10	10	1
A	getEmailClientUsage()	10	10	1
A	getListsAndSegments()	0	4	1
A	getOpens()	0	4	1
A	getRecipients()	0	4	1
A	getSpam()	0	4	1
A	getUnsubscribes()	17	17	1
A	getListsForEmail()	0	14	2
B	addSubscriber()	10	34	4
B	updateSubscriber()	10	39	5
C	addSubscribers()	10	48	8
A	deleteSubscriber()	13	13	2
A	unsubscribeSubscriber()	13	13	2
B	getSubscriberExistsForThisList()	17	17	6
B	getSubscriberCanReceiveEmailsForThisList()	19	19	7
B	getSubscriberCanNoLongerReceiveEmailsForThisList()	0	17	5
A	getSubscriber()	0	19	4
A	getHistory()	13	13	2

<?php

/**
 * Main Holder page for Recipes
 *@author nicolaas [at] sunnysideup.co.nz
 */
class CampaignMonitorAPIConnector extends Object

PSR1 recommends that each class must be in a namespace of at least one level to avoid collisions.

{

    /**
     * REQUIRED!
     * this is the CM url for logging in.
     * which can be used by the client.
     * @var String
     */
    private static $campaign_monitor_url = "";

The property $campaign_monitor_url is not used and could be removed.

    /**
     * REQUIRED!
     * @var String
     */
    private static $client_id = "";

The property $client_id is not used and could be removed.

    /**
     * OPTION 1: API KEY!
     * @var String
     */
    private static $api_key = "";

The property $api_key is not used and could be removed.

    /**
     * OPTION 2: OAUTH OPTION
     * @var String
     */
    private static $client_secret = "";

The property $client_secret is not used and could be removed.

    /**
     * OPTION 2: OAUTH OPTION
     * @var String
     */
    private static $redirect_uri = "";

The property $redirect_uri is not used and could be removed.

    /**
     * OPTION 2: OAUTH OPTION
     * @var String
     */
    private static $code = "";

The property $code is not used and could be removed.

    /**
     *
     * @var Boolean
     */
    protected $debug = false;

    /**
     *
     * @var Boolean
     */
    protected $allowCaching = false;

    /**
     *
     * @var Int
     */
    protected $httpStatusCode = 0;


    /**
     *
     * must be called to use this API.
     */
    public function init()
    {
        //require_once Director::baseFolder().'/'.SS_CAMPAIGNMONITOR_DIR.'/third_party/vendor/autoload.php';

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

        //require_once Director::baseFolder().'/'.SS_CAMPAIGNMONITOR_DIR.'/third_party/vendor/campaignmonitor/createsend-php/csrest_lists.php';

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

    }

    /**
     * turn debug on or off
     *
     * @param Boolean
     */
    public function setDebug($b)
    {
        $this->debug = $b;
    }

    /**
     *
     * @param Boolean $b
     */
    public function setAllowCaching($b)
    {
        $this->allowCaching = $b;
    }

    /**
     *
     * @return Boolean
     */
    public function getAllowCaching()
    {
        return $this->allowCaching;
    }

    /**
     * provides the Authorisation Array
     * @return Array
     */
    protected function getAuth()
    {
        if ($auth = $this->getFromCache("getAuth")) {
            return $auth;
        } else {
            if ($apiKey = $this->Config()->get("api_key")) {
                $auth = array('api_key' => $apiKey);
            } else {
                $client_id = $this->Config()->get("client_id");
                $client_secret = $this->Config()->get("client_secret");
                $redirect_uri = $this->Config()->get("redirect_uri");
                $code = $this->Config()->get("code");

                $result = CS_REST_General::exchange_token($client_id, $client_secret, $redirect_uri, $code);

                if ($result->was_successful()) {
                    $auth = array(
                        'access_token' => $result->response->access_token,
                        'refresh_token' => $result->response->refresh_token
                    );
                    //TODO: do we need to check expiry date?
                    //$expires_in = $result->response->expires_in;

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

                    # Save $access_token, $expires_in, and $refresh_token.

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

                    if ($this->debug) {
                        "access token: ".$result->response->access_token."\n";
                        "expires in (seconds): ".$result->response->expires_in."\n";
                        "refresh token: ".$result->response->refresh_token."\n";
                    }
                } else {
                    # If you receive '121: Expired OAuth Token', refresh the access token
                    if ($result->response->Code == 121) {
                        $wrap = new CS_REST_General($auth);

The call to CS_REST_General::__construct() has too many arguments starting with $auth.

                        list($new_access_token, $new_expires_in, $new_refresh_token) = $wrap->refresh_token();

The assignment to $new_expires_in is unused. Consider omitting it like so list($first,,$third).

                    }
                    $auth = array(
                        'access_token' => $new_access_token,

The variable $new_access_token does not seem to be defined for all execution paths leading up to this point.

                        'refresh_token' => $new_refresh_token

The variable $new_refresh_token does not seem to be defined for all execution paths leading up to this point.

                    );
                    if ($this->debug) {
                        'An error occurred:\n';
                        $result->response->error.': '.$result->response->error_description."\n";
                    }
                }
            }
            $this->saveToCache($auth, "getAuth");
            return $auth;
        }
    }

    /**
     * returns the result or NULL in case of an error
     * @param CS_REST_Wrapper_Result
     * @return Mixed | Null
     */
    protected function returnResult($result, $apiCall, $description)
    {
        if ($this->debug) {
            echo "<h1>$description ( $apiCall ) ...</h1>";
            if ($result->was_successful()) {
                echo "<h2>SUCCESS</h2>";
            } else {
                echo "<h2>FAILURE: ".$result->http_status_code."</h2>";
            }
            echo "<pre>";
            print_r($result);
            echo "</pre>";
            echo "<hr /><hr /><hr />";
            ob_flush();
            flush();
        }
        if ($result->was_successful()) {
            if (isset($result->response)) {
                return $result->response;
            }
            return true;
        } else {
            $this->httpStatusCode = $result->http_status_code;
            return null;
        }
    }

    /**
     * returns the HTTP code for the response.
     * This can be handy for debuging purposes.
     * @return Int
     */
    public function getHttpStatusCode()
    {
        return $this->httpStatusCode;
    }

    /*******************************************************
     * caching
     *
     *******************************************************/

    /**
     * @param String $name
     * @return Mixed
     */
    protected function getFromCache($name)

This method seems to be duplicated in your project.

    {
        if ($this->getAllowCaching()) {
            $name = "CampaignMonitorAPIConnector_".$name;
            $cache = SS_Cache::factory($name);
            $value = $cache->load($name);
            if (!$value) {
                return null;
            }
            return unserialize($value);
        }
    }

    /**
     * @param Mixed $unserializedValue
     * @param String $name
     */
    protected function saveToCache($unserializedValue, $name)

This method seems to be duplicated in your project.

    {
        if ($this->getAllowCaching()) {
            $serializedValue = serialize($unserializedValue);
            $name = "CampaignMonitorAPIConnector_".$name;
            $cache = SS_Cache::factory($name);
            $cache->save($serializedValue, $name);
            return true;
        }
    }


    /*******************************************************
     * client
     *
     *******************************************************/

    /**
     *
     * @return CS_REST_Wrapper_Result A successful response will be an object of the form
     * array(
     *     {
     *         'WebVersionURL' => The web version url of the campaign
     *         'WebVersionTextURL' => The web version url of the text version of the campaign
     *         'CampaignID' => The id of the campaign
     *         'Subject' => The campaign subject
     *         'Name' => The name of the campaign
     *         'FromName' => The from name for the campaign
     *         'FromEmail' => The from email address for the campaign
     *         'ReplyTo' => The reply to email address for the campaign
     *         'SentDate' => The sent data of the campaign
     *         'TotalRecipients' => The number of recipients of the campaign
     *     }
     */
    public function getCampaigns()

This method seems to be duplicated in your project.

    {
        //require_once '../../csrest_clients.php';
        $wrap = new CS_REST_Clients($this->Config()->get("client_id"), $this->getAuth());
        $result = $wrap->get_campaigns();
        return $this->returnResult(
            $result,
            "GET /api/v3.1/clients/{id}/campaigns",
            "Got sent campaigns"
        );
    }

    public function getDrafts()

This method seems to be duplicated in your project.

    {
        //require_once '../../csrest_clients.php';
        $wrap = new CS_REST_Clients($this->Config()->get("client_id"), $this->getAuth());
        $result = $wrap->get_drafts();
        return $this->returnResult(
            $result,
            "GET /api/v3.1/clients/{id}/drafts",
            "Got draft campaigns"
        );
    }

    /**
     * Gets all subscriber lists the current client has created
     * @return CS_REST_Wrapper_Result A successful response will be an object of the form
     * array(
     *     {
     *         'ListID' => The id of the list
     *         'Name' => The name of the list
     *     }
     * )
     */
    public function getLists()

This method seems to be duplicated in your project.

    {
        //require_once '../../csrest_clients.php';
        $wrap = new CS_REST_Clients($this->Config()->get("client_id"), $this->getAuth());
        $result = $wrap->get_lists();
        return $this->returnResult(
            $result,
            "GET /api/v3.1/clients/{id}/lists",
            "Got Lists"
        );
    }

    public function getScheduled()
    {
        user_error("This method is still to be implemented, see samples for an example");
    }

    /**
     * list of people that are definitely suppressed...
     * @param  int $page     page number
     * @param  int $pageSize size of page
     * @param  string $sortByField (email)
     * @param  string $sortDirection (asc)
     *
     * @return [type]                 [description]

The doc-type [type] could not be parsed: Unknown type name "" at position 0. [(view supported doc-types)

     */
    public function getSuppressionlist($page, $pageSize, $sortByField = 'email', $sortDirection = 'asc')
    {
        $wrap = new CS_REST_Clients(
            $this->Config()->get("client_id"),
            $this->getAuth()
        );
        $result = $wrap->get_suppressionlist(
            $page,
            $pageSize,
            $sortByField,
            $sortDirection
        );
        return $this->returnResult(
            $result,
            "GET /api/v3/clients/{id}/suppressionlist",
            "Get suppression list"
        );

    }

    public function getTemplates()

This method seems to be duplicated in your project.

    {
        $wrap = new CS_REST_Clients(
            $this->Config()->get("client_id"),
            $this->getAuth()
        );
        $result = $wrap->get_templates();
        return $this->returnResult(
            $result,
            "GET /api/v3/clients/{id}/templates",
            "Get Templates"
        );
    }

    public function getTemplate($templatID)

This method seems to be duplicated in your project.

    {
        $wrap = new CS_REST_Templates(
            $templatID,
            $this->getAuth()
        );
        $result = $wrap->get();
        return $this->returnResult(
            $result,
            "GET /api/v3/templates/{ID}",
            "Got Summary"
        );
    }

    /**
     *
     * @param CampaignMonitorCampaign $campaignMonitorCampaign
     *
     * @return CS_REST_Wrapper_Result
     */
    public function createTemplate($campaignMonitorCampaign)

This method seems to be duplicated in your project.

    {
        $siteConfig = SiteConfig::current_site_config();

$siteConfig is not used, you could remove the assignment.

        $name = "Template for ".$campaignMonitorCampaign->Name;

The property Name does not exist on object<CampaignMonitorCampaign>. Since you implemented __get, maybe consider adding a @property annotation.

        if (!$name) {
            $name = "no name set";
        }

        $wrap = new CS_REST_Templates(null, $this->getAuth());
        $result = $wrap->create(
            $this->Config()->get("client_id"),
            array(
                'Name' => $name,
                'HtmlPageURL' => $campaignMonitorCampaign->PreviewLink(),
                'ZipFileURL' => ''
            )
        );
        if (isset($result->http_status_code) && ($result->http_status_code == 201 || $result->http_status_code == 201)) {
            $code = $result->response;
            $campaignMonitorCampaign->CreateFromWebsite = false;

The property CreateFromWebsite does not exist on object<CampaignMonitorCampaign>. Since you implemented __set, maybe consider adding a @property annotation.

            $campaignMonitorCampaign->CreatedFromWebsite = true;

The property CreatedFromWebsite does not seem to exist. Did you mean Created?

            $campaignMonitorCampaign->TemplateID = $code;

The property TemplateID does not exist on object<CampaignMonitorCampaign>. Since you implemented __set, maybe consider adding a @property annotation.

        } else {
            $campaignMonitorCampaign->CreateFromWebsite = false;

The property CreateFromWebsite does not exist on object<CampaignMonitorCampaign>. Since you implemented __set, maybe consider adding a @property annotation.

            $campaignMonitorCampaign->CreatedFromWebsite = false;

The property CreatedFromWebsite does not seem to exist. Did you mean Created?

            $code = "Error";
            if (is_object($result->response)) {
                $code = $result->response->Code.":".$result->response->Message;
            }
            $campaignMonitorCampaign->MessageFromNewsletterServer = $code;

The property MessageFromNewsletterServer does not exist on object<CampaignMonitorCampaign>. Since you implemented __set, maybe consider adding a @property annotation.

        }
        $campaignMonitorCampaign->write();
        return $this->returnResult(
            $result,
            "POST /api/v3/templates/{clientID}",
            "Created Template"
        );
    }

    /**
     *
     * @param CampaignMonitorCampaign $campaignMonitorCampaign
     * @param string $templateID
     *
     * @return CS_REST_Wrapper_Result
     */
    public function updateTemplate($campaignMonitorCampaign, $templateID)

This method seems to be duplicated in your project.

    {
        $siteConfig = SiteConfig::current_site_config();

$siteConfig is not used, you could remove the assignment.

        $name = "Template for ".$campaignMonitorCampaign->Name;

The property Name does not exist on object<CampaignMonitorCampaign>. Since you implemented __get, maybe consider adding a @property annotation.

        if (!$name) {
            $name = "no name set";
        }
        $wrap = new CS_REST_Templates($templateID, $this->getAuth());
        $result = $wrap->create(
            $this->Config()->get("client_id"),
            array(
                'Name' => $name,
                'HtmlPageURL' => $campaignMonitorCampaign->PreviewLink(),
                'ZipFileURL' => ''
            )
        );
        if (isset($result->http_status_code) && ($result->http_status_code == 201 || $result->http_status_code == 201)) {
            $code = $result->response;

$code is not used, you could remove the assignment.

            $campaignMonitorCampaign->CreateFromWebsite = false;

The property CreateFromWebsite does not exist on object<CampaignMonitorCampaign>. Since you implemented __set, maybe consider adding a @property annotation.

            $campaignMonitorCampaign->CreatedFromWebsite = true;

The property CreatedFromWebsite does not seem to exist. Did you mean Created?

        } else {
            $campaignMonitorCampaign->CreateFromWebsite = false;

The property CreateFromWebsite does not exist on object<CampaignMonitorCampaign>. Since you implemented __set, maybe consider adding a @property annotation.

            $campaignMonitorCampaign->CreatedFromWebsite = false;

The property CreatedFromWebsite does not seem to exist. Did you mean Created?

            $code = "Error";
            if (is_object($result->response)) {
                $code = $result->response->Code.":".$result->response->Message;
            }
            $campaignMonitorCampaign->MessageFromNewsletterServer = $code;

The property MessageFromNewsletterServer does not exist on object<CampaignMonitorCampaign>. Since you implemented __set, maybe consider adding a @property annotation.

        }
        $campaignMonitorCampaign->write();
        return $this->returnResult(
            $result,
            "PUT /api/v3/templates/{ID}",
            "Updated Template"
        );
    }

    public function deleteTemplate($templateID)

This method seems to be duplicated in your project.

    {
        $wrap = new CS_REST_Templates($templateID, $this->getAuth());
        $result = $wrap->delete();
        return $this->returnResult(
            $result,
            "DELETE /api/v3/templates/{ID}",
            "Deleted Template"
        );
    }

    /*******************************************************
     * lists
     *
     *******************************************************/

    /**
     * Creates a new list based on the provided details.
     * Both the UnsubscribePage and the ConfirmationSuccessPage parameters are optional
     *
     * @param string $title - the page to redirect subscribers to when they unsubscribeThe list title
     * @param string $unsubscribePage - The page to redirect subscribers to when they unsubscribe
     * @param boolean $confirmedOptIn - Whether this list requires confirmation of subscription
     * @param string $confirmationSuccessPage - The page to redirect subscribers to when they confirm their subscription
     * @param string $unsubscribeSetting - Unsubscribe setting must be CS_REST_LIST_UNSUBSCRIBE_SETTING_ALL_CLIENT_LISTS or CS_REST_LIST_UNSUBSCRIBE_SETTING_ONLY_THIS_LIST.  See the documentation for details: http://www.campaignmonitor.com/api/lists/#creating_a_list

Should the type for parameter $unsubscribeSetting not be string|null?

     *
     * @return CS_REST_Wrapper_Result A successful response will be the ID of the newly created list
     */
    public function createList($title, $unsubscribePage, $confirmedOptIn = false, $confirmationSuccessPage, $unsubscribeSetting = null)

Parameters which have default values should be placed at the end.

    {
        //require_once '../../csrest_lists.php';
        $wrap = new CS_REST_Lists(null, $this->getAuth());
        //we need to do this afterwards otherwise the definition below
        //is not recognised
        if (!$unsubscribeSetting) {

The expression $unsubscribeSetting of type string|null is loosely compared to false; this is ambiguous if the string can be empty. You might want to explicitly use === null instead.

            $unsubscribeSetting = CS_REST_LIST_UNSUBSCRIBE_SETTING_ALL_CLIENT_LISTS;
        }
        $result = $wrap->create(
            $this->Config()->get("client_id"),
            array(
                'Title' => $title,
                'UnsubscribePage' => $unsubscribePage,
                'ConfirmedOptIn' => $confirmedOptIn,
                'ConfirmationSuccessPage' => $confirmationSuccessPage,
                'UnsubscribeSetting' => $unsubscribeSetting
            )
        );
        return $this->returnResult(
            $result,
            "POST /api/v3.1/lists/{clientID}",
            "Created with ID"
        );
    }

    /**
     * Creates custom field for list
     *
     * @param string $listID - list ID
     * @param string $type - type of custom field
     * @param string $title - field type
     * @param array $options - options for dropdown field type
     *
     * @return CS_REST_Wrapper_Result A successful response will be the key of the newly created custom field
     */
    public function createCustomField($listID, $visible, $type, $title, $options = array())
    {
        $wrap = new CS_REST_Lists($listID, $this->getAuth());
        switch ($type) {
            case "text":
                $type = CS_REST_CUSTOM_FIELD_TYPE_TEXT;
                break;
            case "number":
                $type = CS_REST_CUSTOM_FIELD_TYPE_NUMBER;
                break;
            case "multi_select_one":
                $type = CS_REST_CUSTOM_FIELD_TYPE_MULTI_SELECTONE;
                break;
            case "multi_select_many":
                $type = CS_REST_CUSTOM_FIELD_TYPE_MULTI_SELECTMANY;
                break;
            case "date":
                $type = CS_REST_CUSTOM_FIELD_TYPE_DATE;
                break;
            default:
                user_error("You must select one from text, number, multi_select_one, multi_select_many, date)");
        }
        $result = $wrap->create_custom_field(array(
            'FieldName' => $title,
            'DataType' => $type,
            'Options' => $options,
            'VisibleInPreferenceCenter' => $visible ? true : false
        ));
        return $this->returnResult(
            $result,
            "POST /api/v3/lists/{ID}/customfields",
            "Created Custom Field for $listID "
        );
    }

    /**
     * Creates custom field for list
     *
     * @param string $listID - list ID
     * @param string $key
     *
     * @return CS_REST_Wrapper_Result
     */
    public function deleteCustomField($listID, $key)

This method seems to be duplicated in your project.

    {
        $wrap = new CS_REST_Lists($listID, $this->getAuth());
        $result = $wrap->delete_custom_field($key);
        return $this->returnResult(
            $result,
            "DELETE /api/v3/lists/{ID}/{Key}",
            "Delete Custom Field for $listID with key $key"
        );
    }

    /**
     * Deletes an existing list from the system
     * @param Int $listID
     * @return CS_REST_Wrapper_Result A successful response will be empty
     */
    public function deleteList($listID)

This method seems to be duplicated in your project.

    {
        //require_once '../../csrest_lists.php';
        $wrap = new CS_REST_Lists($listID, $this->getAuth());
        $result = $wrap->delete();
        return $this->returnResult(
            $result,
            "DELETE /api/v3.1/lists/{ID}",
            "Deleted with code"
        );
    }

    /**
     * Gets the basic details of the current list
     *
     * @param Int $listID
     *
     * @return CS_REST_Wrapper_Result A successful response will be an object of the form
     * {
     *     'ListID' => The id of the list
     *     'Title' => The title of the list
     *     'UnsubscribePage' => The page which subscribers are redirected to upon unsubscribing
     *     'ConfirmedOptIn' => Whether the list is Double-Opt In
     *     'ConfirmationSuccessPage' => The page which subscribers are
     *         redirected to upon confirming their subscription
     *     'UnsubscribeSetting' => The unsubscribe setting for the list. Will
     *         be either CS_REST_LIST_UNSUBSCRIBE_SETTING_ALL_CLIENT_LISTS or
     *         CS_REST_LIST_UNSUBSCRIBE_SETTING_ONLY_THIS_LIST.
     * }
     */
    public function getList($listID)

This method seems to be duplicated in your project.

    {
        //require_once '../../csrest_lists.php';
        $wrap = new CS_REST_Lists($listID, $this->getAuth());
        $result = $wrap->get();
        return $this->returnResult(
            $result,
            "GET /api/v3.1/lists/{ID}",
            "Got list details"
        );
    }

    /**
     * Gets all active subscribers added since the given date
     *
     * @param Int $listID
     * @param string $daysAgo The date to start getting subscribers from

Should the type for parameter $daysAgo not be integer?

     * @param int $page The page number to get
     * @param int $pageSize The number of records per page
     * @param string $sortByField ('EMAIL', 'NAME', 'DATE')
     * @param string $sortDirection ('ASC', 'DESC')
     *
     * @return CS_REST_Wrapper_Result A successful response will be an object of the form
     * {
     *     'ResultsOrderedBy' => The field the results are ordered by
     *     'OrderDirection' => The order direction
     *     'PageNumber' => The page number for the result set
     *     'PageSize' => The page size used
     *     'RecordsOnThisPage' => The number of records returned
     *     'TotalNumberOfRecords' => The total number of records available
     *     'NumberOfPages' => The total number of pages for this collection
     *     'Results' => array(
     *         {
     *             'EmailAddress' => The email address of the subscriber
     *             'Name' => The name of the subscriber
     *             'Date' => The date that the subscriber was added to the list
     *             'State' => The current state of the subscriber, will be 'Active'
     *             'CustomFields' => array (
     *                 {
     *                     'Key' => The personalisation tag of the custom field
     *                     'Value' => The value of the custom field for this subscriber
     *                 }
     *             )
     *         }
     *     )
     * }
     */
    public function getActiveSubscribers($listID, $daysAgo = 3650, $page = 1, $pageSize = 999, $sortByField = "DATE", $sortDirection = "DESC")

This method seems to be duplicated in your project.

    {
        //require_once '../../csrest_lists.php';
        $wrap = new CS_REST_Lists($listID, $this->getAuth());
        $result = $wrap->get_active_subscribers(
            date('Y-m-d', strtotime('-'.$daysAgo.' days')),
            $page,
            $pageSize,
            $sortByField,
            $sortDirection
        );
        return $this->returnResult(
            $result,
            "GET /api/v3.1/lists/{ID}/active",
            "Got active subscribers"
        );
    }

    /**
     * Gets all unconfirmed subscribers added since the given date
     *
     * @param Int $listID
     * @param string $daysAgo The date to start getting subscribers from

Should the type for parameter $daysAgo not be integer?

     * @param int $page The page number to get
     * @param int $pageSize The number of records per page
     * @param string $sortByField ('EMAIL', 'NAME', 'DATE')
     * @param string $sortDirection ('ASC', 'DESC')
     *
     * @return CS_REST_Wrapper_Result A successful response will be an object of the form
     * {
     *     'ResultsOrderedBy' => The field the results are ordered by
     *     'OrderDirection' => The order direction
     *     'PageNumber' => The page number for the result set
     *     'PageSize' => The page size used
     *     'RecordsOnThisPage' => The number of records returned
     *     'TotalNumberOfRecords' => The total number of records available
     *     'NumberOfPages' => The total number of pages for this collection
     *     'Results' => array(
     *         {
     *             'EmailAddress' => The email address of the subscriber
     *             'Name' => The name of the subscriber
     *             'Date' => The date that the subscriber was added to the list
     *             'State' => The current state of the subscriber, will be 'Unconfirmed'
     *             'CustomFields' => array (
     *                 {
     *                     'Key' => The personalisation tag of the custom field
     *                     'Value' => The value of the custom field for this subscriber
     *                 }
     *             )
     *         }
     *     )
     * }
     */
    public function getUnconfirmedSubscribers($listID, $daysAgo = 3650, $page = 1, $pageSize = 999, $sortByField = "DATE", $sortDirection = "DESC")

This method seems to be duplicated in your project.

    {
        //require_once '../../csrest_lists.php';
        $wrap = new CS_REST_Lists($listID, $this->getAuth());
        $result = $wrap->get_unconfirmed_subscribers(
            date('Y-m-d', strtotime('-'.$daysAgo.' days')),
            $page,
            $pageSize,
            $sortByField,
            $sortDirection
        );
        return $this->returnResult(
            $result,
            "GET /api/v3.1/lists/{ID}/unconfirmed",
            "Got unconfimred subscribers"
        );
    }

    /**
     * Gets all bounced subscribers who have bounced out since the given date
     *
     * @param Int $listID
     * @param string $daysAgo The date to start getting subscribers from

Should the type for parameter $daysAgo not be integer?

     * @param int $page The page number to get
     * @param int $pageSize The number of records per page
     * @param string $sortByField ('EMAIL', 'NAME', 'DATE')
     * @param string $sortDirection ('ASC', 'DESC')
     *
     * @return CS_REST_Wrapper_Result A successful response will be an object of the form
     * {
     *     'ResultsOrderedBy' => The field the results are ordered by
     *     'OrderDirection' => The order direction
     *     'PageNumber' => The page number for the result set
     *     'PageSize' => The page size used
     *     'RecordsOnThisPage' => The number of records returned
     *     'TotalNumberOfRecords' => The total number of records available
     *     'NumberOfPages' => The total number of pages for this collection
     *     'Results' => array(
     *         {
     *             'EmailAddress' => The email address of the subscriber
     *             'Name' => The name of the subscriber
     *             'Date' => The date that the subscriber bounced out of the list
     *             'State' => The current state of the subscriber, will be 'Bounced'
     *             'CustomFields' => array (
     *                 {
     *                     'Key' => The personalisation tag of the custom field
     *                     'Value' => The value of the custom field for this subscriber
     *                 }
     *             )
     *         }
     *     )
     * }
     */
    public function getBouncedSubscribers($listID, $daysAgo = 3650, $page = 1, $pageSize = 999, $sortByField = "DATE", $sortDirection = "DESC")

This method seems to be duplicated in your project.

    {
        //require_once '../../csrest_lists.php';
        $wrap = new CS_REST_Lists($listID, $this->getAuth());
        $result = $wrap->get_bounced_subscribers(
            date('Y-m-d', strtotime('-'.$daysAgo.' days')),
            $page,
            $pageSize,
            $sortByField,
            $sortDirection
        );
        return $this->returnResult(
            $result,
            "GET /api/v3.1/lists/{ID}/bounced",
            "Got bounced subscribers"
        );
    }


    /**
     * Gets all unsubscribed subscribers who have unsubscribed since the given date
     *
     * @param Int $listID
     * @param string $daysAgo The date to start getting subscribers from

Should the type for parameter $daysAgo not be integer?

     * @param int $page The page number to get
     * @param int $pageSize The number of records per page
     * @param string $sortByField ('EMAIL', 'NAME', 'DATE')
     * @param string $sortDirection ('ASC', 'DESC')
     *
     * @return CS_REST_Wrapper_Result A successful response will be an object of the form
     * {
     *     'ResultsOrderedBy' => The field the results are ordered by
     *     'OrderDirection' => The order direction
     *     'PageNumber' => The page number for the result set
     *     'PageSize' => The page size used
     *     'RecordsOnThisPage' => The number of records returned
     *     'TotalNumberOfRecords' => The total number of records available
     *     'NumberOfPages' => The total number of pages for this collection
     *     'Results' => array(
     *         {
     *             'EmailAddress' => The email address of the subscriber
     *             'Name' => The name of the subscriber
     *             'Date' => The date that the subscriber was unsubscribed from the list
     *             'State' => The current state of the subscriber, will be 'Unsubscribed'
     *             'CustomFields' => array (
     *                 {
     *                     'Key' => The personalisation tag of the custom field
     *                     'Value' => The value of the custom field for this subscriber
     *                 }
     *             )
     *         }
     *     )
     * }
     */
    public function getUnsubscribedSubscribers($listID, $daysAgo = 3650, $page = 1, $pageSize = 999, $sortByField = "DATE", $sortDirection = "DESC")

This method seems to be duplicated in your project.

    {
        //require_once '../../csrest_lists.php';
        $wrap = new CS_REST_Lists($listID, $this->getAuth());
        $result = $wrap->get_unsubscribed_subscribers(
            date('Y-m-d', strtotime('-'.$daysAgo.' days')),
            $page,
            $pageSize,
            $sortByField,
            $sortDirection
        );
        return $this->returnResult(
            $result,
            "GET /api/v3.1/lists/{ID}/unsubscribed",
            "Got unsubscribed subscribers"
        );
    }

    /**
     * Gets all unsubscribed subscribers who have unsubscribed since the given date
     *
     * @param Int $listID
     * @param string $daysAgo The date to start getting subscribers from

Should the type for parameter $daysAgo not be integer?

     * @param int $page The page number to get
     * @param int $pageSize The number of records per page
     * @param string $sortByField ('EMAIL', 'NAME', 'DATE')
     * @param string $sortDirection ('ASC', 'DESC')
     *
     * @return CS_REST_Wrapper_Result A successful response will be an object of the form
     * {
     *     'ResultsOrderedBy' => The field the results are ordered by
     *     'OrderDirection' => The order direction
     *     'PageNumber' => The page number for the result set
     *     'PageSize' => The page size used
     *     'RecordsOnThisPage' => The number of records returned
     *     'TotalNumberOfRecords' => The total number of records available
     *     'NumberOfPages' => The total number of pages for this collection
     *     'Results' => array(
     *         {
     *             'EmailAddress' => The email address of the subscriber
     *             'Name' => The name of the subscriber
     *             'Date' => The date that the subscriber was unsubscribed from the list
     *             'State' => The current state of the subscriber, will be 'Unsubscribed'
     *             'CustomFields' => array (
     *                 {
     *                     'Key' => The personalisation tag of the custom field
     *                     'Value' => The value of the custom field for this subscriber
     *                 }
     *             )
     *         }
     *     )
     * }
     */
    public function getDeletedSubscribers($listID, $daysAgo = 3650, $page = 1, $pageSize = 999, $sortByField = "email", $sortDirection = "asc")
    {
        //require_once '../../csrest_lists.php';
        $wrap = new CS_REST_Lists($listID, $this->getAuth());
        $result = $wrap->get_deleted_subscribers(
            date('Y-m-d', strtotime('-'.$daysAgo.' days')),
            $page,
            $pageSize,
            $sortByField,
            $sortDirection
        );
        return $this->returnResult(
            $result,
            "GET /api/v3/lists/{ID}/delete",
            "Got deleted subscribers"
        );
    }

    /**
     * Updates the details of an existing list
     * Both the UnsubscribePage and the ConfirmationSuccessPage parameters are optional
     *
     * @param string $title - he page to redirect subscribers to when they unsubscribeThe list title
     * @param string $unsubscribePage - The page to redirect subscribers to when they unsubscribe
     * @param boolean $confirmedOptIn - Whether this list requires confirmation of subscription
     * @param string $confirmationSuccessPage - The page to redirect subscribers to when they confirm their subscription
     * @param string $unsubscribeSetting - Unsubscribe setting must be CS_REST_LIST_UNSUBSCRIBE_SETTING_ALL_CLIENT_LISTS or CS_REST_LIST_UNSUBSCRIBE_SETTING_ONLY_THIS_LIST.  See the documentation for details: http://www.campaignmonitor.com/api/lists/#creating_a_list
     * @param boolean $addUnsubscribesToSuppList -  When UnsubscribeSetting is CS_REST_LIST_UNSUBSCRIBE_SETTING_ALL_CLIENT_LISTS, whether unsubscribes from this list should be added to the suppression list.
     * @param boolean $acrubActiveWithSuppList - When UnsubscribeSetting is CS_REST_LIST_UNSUBSCRIBE_SETTING_ALL_CLIENT_LISTS, whether active subscribers should be scrubbed against the suppression list.

There is no parameter named $acrubActiveWithSuppList. Did you maybe mean $scrubActiveWithSuppList?

     *
     * @return CS_REST_Wrapper_Result A successful response will be empty
     */
    public function updateList($listID, $title, $unsubscribePage, $confirmedOptIn = false, $confirmationSuccessPage, $unsubscribeSetting, $addUnsubscribesToSuppList = true, $scrubActiveWithSuppList = true)

Parameters which have default values should be placed at the end.

    {
        //require_once '../../csrest_lists.php';
        if (!$unsubscribeSetting) {
            $unsubscribeSetting = CS_REST_LIST_UNSUBSCRIBE_SETTING_ALL_CLIENT_LISTS;
        }
        $wrap = new CS_REST_Lists($listID, $this->getAuth());
        $result = $wrap->update(array(
            'Title' => $title,
            'UnsubscribePage' => $unsubscribePage,
            'ConfirmedOptIn' => $confirmedOptIn,
            'ConfirmationSuccessPage' => $confirmationSuccessPage,
            'UnsubscribeSetting' => $unsubscribeSetting,
            'AddUnsubscribesToSuppList' => $addUnsubscribesToSuppList,
            'ScrubActiveWithSuppList' => $scrubActiveWithSuppList
        ));
        return $this->returnResult(
            $result,
            "PUT /api/v3.1/lists/{ID}",
            "Updated with code"
        );
    }

    public function getSegments($listID)

This method seems to be duplicated in your project.

    {
        //require_once '../../csrest_lists.php';
        $wrap = new CS_REST_Lists($listID, $this->getAuth());
        //we need to do this afterwards otherwise the definition below
        //is not recognised
        $result = $wrap->get_segments();
        return $this->returnResult(
            $result,
            "GET /api/v3.1/lists/{listid}/segments",
            "Got segment details"
        );
    }

    /**
     * Gets statistics for list subscriptions, deletions, bounces and unsubscriptions
     *
     * @param Int $listID
     *
     * @return CS_REST_Wrapper_Result A successful response will be an object of the form
     * {
     *     'TotalActiveSubscribers'
     *     'NewActiveSubscribersToday'
     *     'NewActiveSubscribersYesterday'
     *     'NewActiveSubscribersThisWeek'
     *     'NewActiveSubscribersThisMonth'
     *     'NewActiveSubscribersThisYeay'
     *     'TotalUnsubscribes'
     *     'UnsubscribesToday'
     *     'UnsubscribesYesterday'
     *     'UnsubscribesThisWeek'
     *     'UnsubscribesThisMonth'
     *     'UnsubscribesThisYear'
     *     'TotalDeleted'
     *     'DeletedToday'
     *     'DeletedYesterday'
     *     'DeletedThisWeek'
     *     'DeletedThisMonth'
     *     'DeletedThisYear'
     *     'TotalBounces'
     *     'BouncesToday'
     *     'BouncesYesterday'
     *     'BouncesThisWeek'
     *     'BouncesThisMonth'
     *     'BouncesThisYear'
     * }
     */
    public function getListStats($listID)

This method seems to be duplicated in your project.

    {
        //require_once '../../csrest_lists.php';
        $wrap = new CS_REST_Lists($listID, $this->getAuth());
        $result = $wrap->get_stats();
        return $this->returnResult(
            $result,
            "GET /api/v3.1/lists/{ID}/stats",
            "Got Lists Stats"
        );
    }

    public function getListCustomFields($listID)

This method seems to be duplicated in your project.

    {
        $wrap = new CS_REST_Lists($listID, $this->getAuth());
        $result = $wrap->get_custom_fields();
        return $this->returnResult(
            $result,
            "GET /api/v3.1/lists/{ID}/customfields",
            "Got Lists Custom Fields"
        );
    }

    /*******************************************************
     * create campaigns
     *
     *******************************************************/

    /**
     *
     *
     * @param CampaignMonitorCampaign $campaignMonitorCampaign
     * @param array listIDs
     * @param array segmentIDs
     * @param string templateID - OPTIONAL!
     * @param array templateContent - OPTIONAL!
     */
    public function createCampaign(
        $campaignMonitorCampaign,
        $listIDs = array(),

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

        $segmentIDs = array(),

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

        $templateID = "",
        $templateContent = array()
    ) {
        //require_once '../../csrest_lists.php';
        $siteConfig = SiteConfig::current_site_config();

        $subject = $campaignMonitorCampaign->Subject;

The property Subject does not exist on object<CampaignMonitorCampaign>. Since you implemented __set, maybe consider adding a @property annotation.

        if (!$subject) {
            $subject = "no subject set";
        }

        $name = $campaignMonitorCampaign->Name;

The property Name does not exist on object<CampaignMonitorCampaign>. Since you implemented __set, maybe consider adding a @property annotation.

        if (!$name) {
            $name = "no name set";
        }

        $fromName = $campaignMonitorCampaign->FromName;

The property FromName does not exist on object<CampaignMonitorCampaign>. Since you implemented __set, maybe consider adding a @property annotation.

        if (!$fromName) {
            $fromName = $siteConfig->Title;
        }

        $fromEmail = $campaignMonitorCampaign->FromEmail;

The property FromEmail does not exist on object<CampaignMonitorCampaign>. Since you implemented __set, maybe consider adding a @property annotation.

        if (!$fromEmail) {
            $fromEmail = Config::inst()->get('Email', 'admin_email');
        }

        $replyTo = $campaignMonitorCampaign->ReplyTo;

The property ReplyTo does not exist on object<CampaignMonitorCampaign>. Since you implemented __set, maybe consider adding a @property annotation.

        if (!$replyTo) {
            $replyTo = $fromEmail;
        }

        $listID = $campaignMonitorCampaign->Pages()->first()->ListID;

The method Pages does not exist on object<CampaignMonitorCampaign>? Since you implemented __call, maybe consider adding a @method annotation.

        $wrap = new CS_REST_Campaigns(null, $this->getAuth());
        if ($templateID) {
            $result = $wrap->create_from_template(
                $this->Config()->get("client_id"),
                array(
                    'Subject' => $subject,
                    'Name' => $name,
                    'FromName' => $fromName,
                    'FromEmail' => $fromEmail,
                    'ReplyTo' => $replyTo,
                    'ListIDs' => array($listID),
                    'SegmentIDs' => array(),
                    'TemplateID' => $templateID,
                    'TemplateContent' => $templateContent
                )
            );
        } else {
            $result = $wrap->create(
                $this->Config()->get("client_id"),
                array(
                    'Subject' => $subject,
                    'Name' => $name,
                    'FromName' => $fromName,
                    'FromEmail' => $fromEmail,
                    'ReplyTo' => $replyTo,
                    'HtmlUrl' => $campaignMonitorCampaign->PreviewLink(),
                    'TextUrl' => $campaignMonitorCampaign->PreviewLink("textonly"),
                    'ListIDs' => array($listID)
                )
            );
        }
        if (isset($result->http_status_code) && ($result->http_status_code == 201 || $result->http_status_code == 201)) {
            $code = $result->response;
            $campaignMonitorCampaign->CreateFromWebsite = false;

The property CreateFromWebsite does not exist on object<CampaignMonitorCampaign>. Since you implemented __set, maybe consider adding a @property annotation.

            $campaignMonitorCampaign->CreatedFromWebsite = true;

The property CreatedFromWebsite does not seem to exist. Did you mean Created?

            $campaignMonitorCampaign->CampaignID = $code;

The property CampaignID does not exist on object<CampaignMonitorCampaign>. Since you implemented __set, maybe consider adding a @property annotation.

        } else {
            $campaignMonitorCampaign->CreateFromWebsite = false;

The property CreateFromWebsite does not exist on object<CampaignMonitorCampaign>. Since you implemented __set, maybe consider adding a @property annotation.

            $campaignMonitorCampaign->CreatedFromWebsite = false;

The property CreatedFromWebsite does not seem to exist. Did you mean Created?

            $code = "Error";
            if (is_object($result->response)) {
                $code = $result->response->Code.":".$result->response->Message;
            }
            $campaignMonitorCampaign->MessageFromNewsletterServer = $code;

The property MessageFromNewsletterServer does not exist on object<CampaignMonitorCampaign>. Since you implemented __set, maybe consider adding a @property annotation.

        }
        $campaignMonitorCampaign->write();
        return $this->returnResult(
            $result,
            "CREATE /api/v3/campaigns/{clientID}",
            "Created Campaign"
        );
    }

    public function deleteCampaign($campaignID)

This method seems to be duplicated in your project.

    {
        $wrap = new CS_REST_Campaigns($campaignID, $this->getAuth());
        $result = $wrap->delete();
        return $this->returnResult(
            $result,
            "DELETE /api/v3/campaigns/{id}",
            "Deleted Campaign"
        );
    }

    /*******************************************************
     * information about the campaigns
     *
     *******************************************************/

    public function getBounces()
    {
        user_error("This method is still to be implemented, see samples for an example");
    }

    public function getClicks()
    {
        user_error("This method is still to be implemented, see samples for an example");
    }

    /**
     * Gets a summary of all campaign reporting statistics
     *
     * @param int $campaignID
     *
     * @return CS_REST_Wrapper_Result A successful response will be an object of the form
     * {
     *     'Recipients' => The total recipients of the campaign
     *     'TotalOpened' => The total number of opens recorded
     *     'Clicks' => The total number of recorded clicks
     *     'Unsubscribed' => The number of recipients who unsubscribed
     *     'Bounced' => The number of recipients who bounced
     *     'UniqueOpened' => The number of recipients who opened
     *     'WebVersionURL' => The url of the web version of the campaign
     *     'WebVersionTextURL' => The url of the web version of the text version of the campaign
     *     'WorldviewURL' => The public Worldview URL for the campaign
     *     'Forwards' => The number of times the campaign has been forwarded to a friend
     *     'Likes' => The number of times the campaign has been 'liked' on Facebook
     *     'Mentions' => The number of times the campaign has been tweeted about
     *     'SpamComplaints' => The number of recipients who marked the campaign as spam
     * }
     */
    public function getSummary($campaignID)

This method seems to be duplicated in your project.

    {
        $wrap = new CS_REST_Campaigns($campaignID, $this->getAuth());
        $result = $wrap->get_summary();
        return $this->returnResult(
            $result,
            "GET /api/v3.1/campaigns/{id}/summary",
            "Got Summary"
        );
    }

    /**
     * Gets the email clients that subscribers used to open the campaign
     *
     * @param Int $campaignID
     *
     * @return CS_REST_Wrapper_Result A successful response will be an object of the form
     * array(
     *     {
     *         Client => The email client name
     *         Version => The email client version
     *         Percentage => The percentage of subscribers who used this email client
     *         Subscribers => The actual number of subscribers who used this email client
     *     }
     * )
     */
    public function getEmailClientUsage($campaignID)

This method seems to be duplicated in your project.

    {
        $wrap = new CS_REST_Campaigns($campaignID, $this->getAuth());
        $result = $wrap->get_email_client_usage();
        return $this->returnResult(
            $result,
            "GET /api/v3.1/campaigns/{id}/emailclientusage",
            "Got email client usage"
        );
    }

    public function getListsAndSegments()
    {
        user_error("This method is still to be implemented, see samples for an example");
    }

    public function getOpens()
    {
        user_error("This method is still to be implemented, see samples for an example");
    }

    public function getRecipients()
    {
        user_error("This method is still to be implemented, see samples for an example");
    }

    public function getSpam()
    {
        user_error("This method is still to be implemented, see samples for an example");
    }

    /**
     * Gets all unsubscribes recorded for a campaign since the provided date
     *
     * @param int $campaignID ID of the Campaign
     * @param string $daysAgo The date to start getting subscribers from

Should the type for parameter $daysAgo not be integer?

     * @param int $page The page number to get
     * @param int $pageSize The number of records per page
     * @param string $sortByField ('EMAIL', 'NAME', 'DATE')
     * @param string $sortDirection ('ASC', 'DESC')
     *
     * @return CS_REST_Wrapper_Result A successful response will be an object of the form
     * {
     *     'ResultsOrderedBy' => The field the results are ordered by
     *     'OrderDirection' => The order direction
     *     'PageNumber' => The page number for the result set
     *     'PageSize' => The page size used
     *     'RecordsOnThisPage' => The number of records returned
     *     'TotalNumberOfRecords' => The total number of records available
     *     'NumberOfPages' => The total number of pages for this collection
     *     'Results' => array(
     *         {
     *             'EmailAddress' => The email address of the subscriber who unsubscribed
     *             'ListID' => The list id of the list containing the subscriber
     *             'Date' => The date of the unsubscribe
     *             'IPAddress' => The ip address where the unsubscribe originated
     *         }
     *     )
     * }
     */
    public function getUnsubscribes($campaignID, $daysAgo = 3650, $page =1, $pageSize = 999, $sortByField = "EMAIL", $sortDirection = "ASC")

This method seems to be duplicated in your project.

    {
        //require_once '../../csrest_campaigns.php';
        $wrap = new CS_REST_Campaigns($campaignID, $this->getAuth());
        $result = $wrap->get_unsubscribes(
            date('Y-m-d', strtotime('-'.$daysAgo.' days')),
            $page,
            $pageSize,
            $sortByField,
            $sortDirection
        );
        return $this->returnResult(
            $result,
            "GET /api/v3.1/campaigns/{id}/unsubscribes",
            "Got unsubscribes"
        );
    }



    /*******************************************************
     * user
     *
     * states:
     *
     * Active – Someone who is on a list and will receive any emails sent to that list.
     *
     * Unconfirmed – The individual signed up to a confirmed opt-in list
     * but has not clicked the link in the verification email sent to them.
     *
     * Unsubscribed – The subscriber has removed themselves from a list, or lists,
     * via an unsubscribe link or form.
     * You can also change a subscriber's status to unsubscribed through your account.
     *
     * Bounced – This describes an email address that campaigns cannot be delivered to,
     * which can happen for a number of reasons.
     *
     * Deleted – Means the subscriber has been deleted from a list through your account.
     *
     *******************************************************/

    /**
     * Gets the lists across a client to which a subscriber with a particular
     * email address belongs.
     *
     * @param string | Member $email Subscriber's email address (or Member)
     *
     * @return CS_REST_Wrapper_Result A successful response will be an object of the form
     * array(
     *     {
     *         'ListID' => The id of the list
     *         'ListName' => The name of the list
     *         'SubscriberState' => The state of the subscriber in the list
     *         'DateSubscriberAdded' => The date the subscriber was added
     *     }
     * )
     */
    public function getListsForEmail($member)
    {
        if ($member instanceof Member) {
            $member = $member->Email;
        }
        //require_once '../../csrest_clients.php';
        $wrap = new CS_REST_Clients($this->Config()->get("client_id"), $this->getAuth());
        $result = $wrap->get_lists_for_email($member);
        return $this->returnResult(
            $result,
            "/api/v3.1/clients/{id}/listsforemail",
            "Got lists to which email address ".$member." is subscribed"
        );
    }


    /**
     * Adds a new subscriber to the specified list
     *
     * @param Int $listID
     * @param Member $member
     * @param Array $customFields
     * @param array $customFields The subscriber details to use during creation.
     * @param boolean $resubscribe Whether we should resubscribe this subscriber if they already exist in the list
     * @param boolean $RestartSubscriptionBasedAutoResponders Whether we should restart subscription based auto responders which are sent when the subscriber first subscribes to a list.

There is no parameter named $RestartSubscriptionBasedAutoResponders. Did you maybe mean $restartSubscriptionBasedAutoResponders?

     *
     * NOTE that for the custom fields they need to be formatted like this:
     *    Array(
     *        'Key' => The custom fields personalisation tag
     *        'Value' => The value for this subscriber
     *        'Clear' => true/false (pass true to remove this custom field. in the case of a [multi-option, select many] field, pass an option in the 'Value' field to clear that option or leave Value blank to remove all options)
     *    )
     *
     * @return CS_REST_Wrapper_Result A successful response will be empty
     */
    public function addSubscriber(
        $listID,
        $member,
        $customFields = array(),
        $resubscribe = true,
        $restartSubscriptionBasedAutoResponders = false
    ) {
        //require_once '../../csrest_subscribers.php';
        $wrap = new CS_REST_Subscribers($listID, $this->getAuth());
        foreach ($customFields as $key => $customFieldValue) {

This code seems to be duplicated across your project.

            if (!is_array($customFields[$key])) {
                $customFields[] = array(
                    "Key" => $key,
                    "Value" => $customFieldValue,
                    "Clear" => $customFieldValue ? false : true
                );
                unset($customFields[$key]);
            }
        }
        $result = $wrap->add(
            $request = array(
                'EmailAddress' => $member->Email,
                'Name' => $member->getName(),
                'CustomFields' => $customFields,
                'Resubscribe' => $resubscribe,
                'RestartSubscriptionBasedAutoResponders' => $restartSubscriptionBasedAutoResponders
            )
        );
        return $this->returnResult(
            $result,
            "POST /api/v3.1/subscribers/{list id}.{format}",
            "Subscribed with code ..."
        );
    }

    /**
     * Updates an existing subscriber (email, name, state, or custom fields) in the specified list.
     * The update is performed even for inactive subscribers, but will return an error in the event of the
     * given email not existing in the list.
     *
     * @param Int $listID
     * @param String $oldEmailAddress
     * @param Member $member
     * @param array $customFields The subscriber details to use during creation.
     * @param boolean $resubscribe Whether we should resubscribe this subscriber if they already exist in the list
     * @param boolean $restartSubscriptionBasedAutoResponders Whether we should restart subscription based auto responders which are sent when the subscriber first subscribes to a list.
     *
     * NOTE that for the custom fields they need to be formatted like this:
     *    Array(
     *        'Key' => The custom fields personalisation tag
     *        'Value' => The value for this subscriber
     *        'Clear' => true/false (pass true to remove this custom field. in the case of a [multi-option, select many] field, pass an option in the 'Value' field to clear that option or leave Value blank to remove all options)
     *    )
     *
     * @return CS_REST_Wrapper_Result A successful response will be empty
     */
    public function updateSubscriber(
        $listID,
        $oldEmailAddress = "",
        Member $member,

Parameters which have default values should be placed at the end.

        $customFields = array(),
        $resubscribe = true,
        $restartSubscriptionBasedAutoResponders = false
    ) {
        if (!$oldEmailAddress) {
            $oldEmailAddress = $member->Email;
        }
        foreach ($customFields as $key => $customFieldValue) {

This code seems to be duplicated across your project.

            if (!is_array($customFields[$key])) {
                $customFields[] = array(
                    "Key" => $key,
                    "Value" => $customFieldValue,
                    "Clear" => $customFieldValue ? false : true
                );
                unset($customFields[$key]);
            }
        }
        //require_once '../../csrest_subscribers.php';
        $wrap = new CS_REST_Subscribers($listID, $this->getAuth());
        $result = $wrap->update(
            $oldEmailAddress,

It seems like $oldEmailAddress defined by $member->Email on line 1394 can also be of type array; however, CS_REST_Subscribers::update() does only seem to accept string, maybe add an additional type check?

            array(
                'EmailAddress' => $member->Email,
                'Name' => $member->getName(),
                'CustomFields' => $customFields,
                'Resubscribe' => $resubscribe,
                'RestartSubscriptionBasedAutoResponders' => $restartSubscriptionBasedAutoResponders,
            )
        );
        return $this->returnResult(
            $result,
            "PUT /api/v3.1/subscribers/{list id}.{format}?email={email}",
            "updated with email $oldEmailAddress ..."
        );
    }

    /**
     * Updates an existing subscriber (email, name, state, or custom fields) in the specified list.
     * The update is performed even for inactive subscribers, but will return an error in the event of the
     * given email not existing in the list.
     *
     * @param Int $listID
     * @param ArraySet $memberSet - list of mebers

There is no parameter named $memberSet. Did you maybe mean $membersSet?

     * @param array $customFields The subscriber details to use during creation. Each array item needs to have the same key as the member ID - e.g. array( 123 => array( [custom fields here] ), 456 => array( [custom fields here] ) )
     * @param $resubscribe Whether we should resubscribe any existing subscribers
     * @param $queueSubscriptionBasedAutoResponders By default, subscription based auto responders do not trigger during an import. Pass a value of true to override this behaviour
     * @param $restartSubscriptionBasedAutoResponders By default, subscription based auto responders will not be restarted
     *
     * NOTE that for the custom fields they need to be formatted like this:
     *    Array(
     *        'Key' => The custom fields personalisation tag
     *        'Value' => The value for this subscriber
     *        'Clear' => true/false (pass true to remove this custom field. in the case of a [multi-option, select many] field, pass an option in the 'Value' field to clear that option or leave Value blank to remove all options)
     *    )

     * @return CS_REST_Wrapper_Result A successful response will be empty
     */
    public function addSubscribers(
        $listID,
        $membersSet,
        $customFields = array(),
        $resubscribe,

Parameters which have default values should be placed at the end.

        $queueSubscriptionBasedAutoResponders = false,
        $restartSubscriptionBasedAutoResponders = false
    ) {
        //require_once '../../csrest_subscribers.php';
        $wrap = new CS_REST_Subscribers($listID, $this->getAuth());
        $importArray = array();
        foreach ($membersSet as $member) {
            $customFieldsForMember = array();
            if (isset($customFields[$member->ID])) {
                $customFieldsForMember = $customFields[$member->ID];
            } elseif (isset($customFields[$member->Email])) {
                $customFieldsForMember = $customFields[$member->Email];
            }
            foreach ($customFieldsForMember as $key => $customFieldValue) {

This code seems to be duplicated across your project.

                if (!is_array($customFieldsForMember[$key])) {
                    $customFieldsForMember[] = array(
                        "Key" => $key,
                        "Value" => $customFieldValue,
                        "Clear" => $customFieldValue ? false : true
                    );
                    unset($customFieldsForMember[$key]);
                }
            }
            if ($member instanceof Member) {
                $importArray[] = array(
                    'EmailAddress' => $member->Email,
                    'Name' => $member->getName(),
                    'CustomFields' => $customFieldsForMember
                );
            }
        }
        $result = $wrap->import(
            $importArray,
            $resubscribe,
            $queueSubscriptionBasedAutoResponders,
            $restartSubscriptionBasedAutoResponders
        );
        return $this->returnResult(
            $result,
            "POST /api/v3.1/subscribers/{list id}/import.{format}",
            "review details ..."
        );
    }

    /**
     * @param Int $listID
     * @param Member | String $member - email address or Member Object
     *
     * @return CS_REST_Wrapper_Result A successful response will be empty
     */
    public function deleteSubscriber($listID, $member)

This method seems to be duplicated in your project.

    {
        if ($member instanceof Member) {
            $member = $member->Email;
        }
        $wrap = new CS_REST_Subscribers($listID, $this->getAuth());
        $result = $wrap->delete($member);
        return $this->returnResult(
            $result,
            "DELETE /api/v3.1/subscribers/{list id}.{format}?email={emailAddress}",
            "Unsubscribed with code  ..."
        );
    }

    /**
     * Unsubscribes the given subscriber from the current list
     *
     * @param Int $listID
     * @param Member | String $member
     *
     * @return CS_REST_Wrapper_Result A successful response will be empty
     */
    public function unsubscribeSubscriber($listID, $member)

This method seems to be duplicated in your project.

    {
        if ($member instanceof Member) {
            $member = $member->Email;
        }
        $wrap = new CS_REST_Subscribers($listID, $this->getAuth());
        $result = $wrap->unsubscribe($member);
        return $this->returnResult(
            $result,
            "GET /api/v3.1/subscribers/{list id}/unsubscribe.{format}",
            "Unsubscribed with code  ..."
        );
    }

    /**
     * Is this user part of this list at all?
     *
     * @param Int $listID
     * @param Member | String $member
     *
     * @return Boolean
     */
    public function getSubscriberExistsForThisList($listID, $member)

This method seems to be duplicated in your project.

    {
        if ($member instanceof Member) {
            $member = $member->Email;
        }
        $outcome = $this->getSubscriber($listID, $member);
        if ($outcome && isset($outcome->State)) {
            if ($this->debug) {
                echo "<h3>Subscriber Exists For This List</h3>";
            }
            return true;
        }
        if ($this->debug) {
            echo "<h3>Subscriber does *** NOT *** Exist For This List</h3>";
        }
        return false;
    }

    /**
     * Can we send e-mails to this person in the future for this list?
     *
     * @param Int $listID
     * @param Member | String $member
     *
     * @return Boolean
     */
    public function getSubscriberCanReceiveEmailsForThisList($listID, $member)

This method seems to be duplicated in your project.

    {
        if ($member instanceof Member) {
            $member = $member->Email;
        }
        $outcome = $this->getSubscriber($listID, $member);
        if ($outcome && isset($outcome->State)) {
            if ($outcome->State == "Active") {

The property State does not seem to exist in CS_REST_Wrapper_Result.

                if ($this->debug) {
                    echo "<h3>Subscriber Can Receive Emails For This List</h3>";
                }
                return true;
            }
        }
        if ($this->debug) {
            echo "<h3>Subscriber Can *** NOT *** Receive Emails For This List</h3>";
        }
        return false;
    }

    /**
     * This e-mail / user has been banned from a list.
     *
     * @param Int $listID
     * @param Member | String $member
     *
     * @return Boolean
     */
    public function getSubscriberCanNoLongerReceiveEmailsForThisList($listID, $member)
    {
        $subscriberExistsForThisList = $this->getSubscriberExistsForThisList($listID, $member);
        $subscriberCanReceiveEmailsForThisList = $this->getSubscriberCanReceiveEmailsForThisList($listID, $member);
        if ($subscriberExistsForThisList) {
            if (!$subscriberCanReceiveEmailsForThisList) {
                if ($this->debug) {
                    echo "<h3>Subscriber Can No Longer Receive Emails For This List</h3>";
                }
                return true;
            }
        }
        if ($this->debug) {
            echo "<h3>Subscriber Can *** STILL *** Receive Emails For This List</h3>";
        }
        return false;
    }

    private static $_get_subscriber = array();

    /**
     * Gets a subscriber details, including custom fields
     *
     * @param Int $listID
     * @param Member | String $member
     *
     * @return CS_REST_Wrapper_Result A successful response will be an object of the form
     * {
     *     'EmailAddress' => The subscriber email address
     *     'Name' => The subscribers name
     *     'Date' => The date the subscriber was added to the list
     *     'State' => The current state of the subscriber
     *     'CustomFields' => array(
     *         {
     *             'Key' => The custom fields personalisation tag
     *             'Value' => The custom field value for this subscriber
     *         }
     *     )
     * }
     *
     */
    public function getSubscriber($listID, $member, $cacheIsOK = true)
    {
        if ($member instanceof Member) {
            $member = $member->Email;
        }
        $key = $listID."_".$member;
        if (isset(self::$_get_subscriber[$key]) && $cacheIsOK) {

This if statement is empty and can be removed.

            //do nothing
        } else {
            $wrap = new CS_REST_Subscribers($listID, $this->getAuth());
            $result = $wrap->get($member);
            self::$_get_subscriber[$key] = $this->returnResult(
                $result,
                "GET /api/v3.1/subscribers/{list id}.{format}?email={email}",
                "got subscribed subscriber"
            );
        }
        return self::$_get_subscriber[$key];
    }

    /**
     * Gets a subscriber details, including custom fields
     *
     * @param Int $listID
     * @param Member | String $member
     *
     * @return CS_REST_Wrapper_Result A successful response will be an object of the form
     * {
     *     'EmailAddress' => The subscriber email address
     *     'Name' => The subscribers name
     *     'Date' => The date the subscriber was added to the list
     *     'State' => The current state of the subscriber
     *     'CustomFields' => array(
     *         {
     *             'Key' => The custom fields personalisation tag
     *             'Value' => The custom field value for this subscriber
     *         }
     *     )
     * }
     *
     */
    public function getHistory($listID, $member)

This method seems to be duplicated in your project.

    {
        if ($member instanceof Member) {
            $member = $member->Email;
        }
        $wrap = new CS_REST_Subscribers($listID, $this->getAuth());
        $result = $wrap->get_history($member);
        return $this->returnResult(
            $result,
            "GET /api/v3.1/subscribers/{list id}/history.{format}?email={email}",
            "got subscriber history"
        );
    }
}