Backpack\CRUD\app\Exceptions\BackpackProRequiredException

Complexity

Total Complexity

2

Size/Duplication

Total Lines	19
Duplicated Lines	0 %

Importance

Changes

0

1 Method

Rating	Name	Duplication	Size	Complexity
A	render()	0	11	2

<?php

namespace Backpack\CRUD\app\Exceptions;

class BackpackProRequiredException extends \Exception
{
    /**
     * Render the exception into an HTTP response.
     *
     * @param  \Illuminate\Http\Request
     * @return \Illuminate\Http\Response
     */
    public function render($request)

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

    {
        // 0, by default, pass only what is a feature we construct the rest of the message
        // 1 use the provided message in full
        switch ($this->getCode()) {
            case 0:
                $this->message = $this->message.' is a Backpack PRO feature. Please purchase and install the Backpack\PRO addon from backpackforlaravel.com';
                break;
        }

        return abort(500, $this->getMessage(), ['developer-error-exception']);

Are you sure the usage of abort(500, $this->getMessage(), array('developer-error-exception')) is correct as it seems to always return null.

The expression return abort(500, $this->getMessage(), array('developer-error-exception')) returns the type void which is incompatible with the documented return type Illuminate\Http\Response.

    }
}