App\Rest\Traits\Methods\DeleteMethod::deleteMethod()

Complexity

Conditions	2
Paths	2

Size

Total Lines	11
Code Lines	7

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	7
CRAP Score	2

Importance

Changes

0

<?php
    public function deleteMethod(Request $request, string $id, ?array $allowedHttpMethods = null): Response
    {
        $resource = $this->getResourceForMethod($request, $allowedHttpMethods ?? [Request::METHOD_DELETE]);

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

        try {
            // Fetch data from database
            return $this
                ->getResponseHandler()

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

                ->createResponse($request, $resource->delete($id), $resource);
        } catch (Throwable $exception) {
            throw $this->handleRestMethodException($exception, $id);

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

        }