Sfneal\CrudModelActions\Utils\HttpResponses

Complexity

Total Complexity

3

Size/Duplication

Total Lines	33
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	successResponse()	0	9	2
A	failResponse()	0	4	1

<?php


namespace Sfneal\CrudModelActions\Utils;


trait HttpResponses
{
    /**
     * @var string Response to be returned on success
     */
    private $successResponse;

    /**
     * Return the default success response
     *
     * @param string|null $response
     * @return string
     */
    protected function successResponse(string $response = null): string
    {
        // Set the response during runtime
        if (isset($response)) {
            $this->successResponse = $response;
        }

        // Redirect back to previous page
        return $this->successResponse ?? redirect()->back();
    }

    /**
     * Return the default error response
     *
     * @return string
     */
    protected function failResponse(): string
    {
        // Return JS alert
        return jsAlert($this->failMessage());

It seems like failMessage() must be provided by classes using this trait. How about adding it as abstract method to this trait?

The function jsAlert was not found. Maybe you did not declare it correctly or list all dependencies?

    }
}