Transmissor\Services\NotificationService

Complexity

Total Complexity

21

Size/Duplication

Total Lines	244
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	1

Importance

Changes

0

15 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	7	1
A	all()	0	4	1
A	paginated()	0	4	1
A	userBasedPaginated()	0	4	1
A	userBased()	0	4	1
A	search()	0	17	3
A	notify()	0	12	1
A	create()	0	41	4
A	getUser()	0	4	1
A	find()	0	4	1
A	findByUuid()	0	4	1
A	update()	0	17	1
A	markAsRead()	0	5	1
A	destroy()	0	4	1
A	usersAsOptions()	0	10	2

<?php

namespace Transmissor\Services;

use Crypto;
use Illuminate\Support\Facades\Schema;
use Transmissor\Models\Notification;
use Transmissor\Notifications\GeneralNotification;
use Transmissor\Services\UserService;

class NotificationService
{
    public function __construct(
        Notification $model,
        UserService $userService
    ) {
        $this->model = $model;

The property model does not exist. Did you maybe forget to declare it?

        $this->userService = $userService;

The property userService does not exist. Did you maybe forget to declare it?

    }

    /**
     * All notifications
     *
     * @return Collection
     */
    public function all()
    {
        return $this->model->orderBy('created_at', 'desc')->get();
    }

    /**
     * Paginated notifications
     *
     * @return PaginatedCollection
     */
    public function paginated()
    {
        return $this->model->orderBy('created_at', 'desc')->paginate(env('PAGINATE', 25));
    }

    /**
     * User based paginated notifications
     *
     * @param  integer $id
     * @return PaginatedCollection
     */
    public function userBasedPaginated($id)
    {
        return $this->model->where('notificable_type', User::class)->where('notificable_id', $id)->orderBy('created_at', 'desc')->paginate(env('PAGINATE', 25));
    }

    /**
     * User based notifications
     *
     * @param  integer $id
     * @return Collection
     */
    public function userBased($id)
    {
        return $this->model->where('notificable_type', User::class)->where('notificable_id', $id)->where('deleted_at', null)->orderBy('created_at', 'desc')->get();
    }

    /**
     * Search notifications
     *
     * @param  string  $input
     * @param  integer $id
     * @return Collection
     */
    public function search($input, $id)
    {
        $query = $this->model->orderBy('created_at', 'desc');
        $query->where('id', 'LIKE', '%'.$input.'%');

        $columns = Schema::getColumnListing('notifications');

        foreach ($columns as $attribute) {
            if (is_null($id)) {
                $query->orWhere($attribute, 'LIKE', '%'.$input.'%');
            } else {
                $query->orWhere($attribute, 'LIKE', '%'.$input.'%')->where('user_id', $id);
            }
        };

        return $query->paginate(env('PAGINATE', 25));
    }

    /**
     * Create a notificaton
     *
     * @param  integer $userId
     * @param  string  $flag
     * @param  string  $title
     * @param  string  $details
     * @return void
     */
    public function notify($userId, $flag, $title, $details)
    {
        $input = [
            'notificable_type' => User::class,
            'notificable_id' => $userId,
            'flag' => $flag,
            'title' => $title,
            'details' => $details,
        ];

        $this->create($input);
    }

    /**
     * Create a notification
     *
     * @param  array $input
     * @return boolean|exception
     */
    public function create($input)
    {
        try {
            if ($input['user_id'] == 0) {
                $users = $this->userService->all();

                foreach ($users as $user) {
                    $input['uuid'] = Crypto::uuid();
                    $input['user_id'] = $user->id;
                    $this->model->create($input);
                }

                $user->notify(

The variable $user seems to be defined by a foreach iteration on line 122. Are you sure the iterator is never empty, otherwise this variable is not defined?

                    new GeneralNotification(
                        [
                        'title' => $input['title'],
                        'details' => $input['details'],
                        ]
                    )
                );

                return true;
            }

            $input['uuid'] = Crypto::uuid();

            $user = $this->userService->find($input['user_id']);
            $user->notify(
                new GeneralNotification(
                    [
                    'title' => $input['title'],
                    'details' => $input['details'],
                    ]
                )
            );

            return $this->model->create($input);
        } catch (Exception $e) {

The class Transmissor\Services\Exception does not exist. Did you forget a USE statement, or did you not list all dependencies?

            throw new Exception("Could not send notifications please try agian.", 1);
        }
    }

    /**
     * Get a user
     *
     * @param  integer $id
     * @return User
     */
    public function getUser($id)
    {
        return $this->userService->find($id);
    }

    /**
     * Find a notification
     *
     * @param  integer $id
     * @return Notification
     */
    public function find($id)
    {
        return $this->model->find($id);
    }

    /**
     * Find a notification by UUID
     *
     * @param  string $uuid
     * @return Notification
     */
    public function findByUuid($uuid)
    {
        return $this->model->where('uuid', $uuid)->first();
    }

    /**
     * Update a notification
     *
     * @param  integer $id
     * @param  array   $input
     * @return Notification
     */
    public function update($id, $input)
    {
        $notification = $this->model->find($id);
        $notification->update($input);

        $user = $this->userService->find($notification->user_id);
        $user->notify(
            new GeneralNotification(
                [
                'title' => $input['title'],
                'details' => $input['details'],
                ]
            )
        );

        return $notification;
    }

    /**
     * Mark notification as read
     *
     * @param  integer $id
     * @return boolean
     */
    public function markAsRead($id)
    {
        $input['is_read'] = true;

$input was never initialized. Although not strictly required by PHP, it is generally a good practice to add $input = array(); before regardless.

        return $this->model->find($id)->update($input);
    }

    /**
     * Destroy a Notification
     *
     * @param  integer $id
     * @return boolean
     */
    public function destroy($id)
    {
        return $this->model->find($id)->delete();
    }

    /**
     * Users as Select options array
     *
     * @return Array
     */
    public function usersAsOptions()
    {
        $users = ['All' => 0];

        foreach ($this->userService->all() as $user) {
            $users[$user->name] = $user->id;
        }

        return $users;
    }
}