Bmatovu\LaravelXml\Http\Middleware\RequireXml

Complexity

Total Complexity

5

Size/Duplication

Total Lines	23
Duplicated Lines	0 %

Test Coverage

Coverage

50%

Importance

Changes	3
Bugs	0	Features	0

1 Method

Rating	Name	Duplication	Size	Complexity
A	handle()	0	11	5

<?php

namespace Bmatovu\LaravelXml\Http\Middleware;

use Bmatovu\LaravelXml\LaravelXml;
use Closure;

class RequireXml
{
    /**
     * Handle an incoming request.
     *
     * @see https://stackoverflow.com/a/11973933/2732184
     *
     * @param \Illuminate\Http\Request $request
     * @param bool $isValidXml
     *
     * @return mixed
     */
    public function handle($request, Closure $next, $isValidXml = false)
    {
        if ('xml' !== $request->getContentTypeFormat()) {
            return response()->xml(['message' => 'Only accepting content of type XML.'], 415);
        }

        if ($isValidXml && $request->getContent() && ! LaravelXml::is_valid($request->getContent())) {

The method is_valid() does not exist on Bmatovu\LaravelXml\LaravelXml. Since you implemented __callStatic, consider adding a @method annotation.

            return response()->xml(['message' => 'The given data was malformed.'], 400);
        }

        return $next($request);
    }
}