Vindi\Notification

Complexity

Total Complexity

4

Size/Duplication

Total Lines	49
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	1

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	endpoint()	0	4	1
A	getNotificationItem()	0	4	1
A	setNotificationItem()	0	4	1
A	removeNotificationItem()	0	4	1

<?php

namespace Vindi;

class Notification extends Resource
{
    /**
     * The endpoint that will hit the API.
     *
     * @return string
     */
    public function endpoint()
    {
        return 'notification';
    }

    /**
     * Make a GET request to notifications/{id}/notification_items.
     * @param int $id The resource's id.
     *
     * @return string
     */
    public function getNotificationItem($id, $page = 1, $per_page = 50)
    {
        return $this->get($id, 'notification_items', $page, $per_page);

The call to Notification::get() has too many arguments starting with $page.

    }

    /**
     * Make a POST request to notifications/{id}/notification_items.
     *
     * @param int   $id The resource's id.
     * @param array.
     *
     * @return mixed
     */
    public function setNotificationItem($id, array $form_params = [])
    {
        return $this->post($id, 'notification_items', $form_params);
    }

    /**
     * Make a DELETE request to notifications/{id}/notification_items.
     *
     * @param int   $id The resource's id.
     * @param int 	$item_id The resource's item id.

There is no parameter named $item_id. Did you maybe mean $item_Id?

     *
     * @return mixed
     */
    public function removeNotificationItem($id, $item_Id)

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

    {
        return $this->delete($id, 'notification_items', $item_id);

The variable $item_id does not exist. Did you mean $item_Id?

'notification_items' is of type string, but the function expects a array.

The call to Notification::delete() has too many arguments starting with $item_id.

    }
}