App\Rest\Traits\Methods\FindMethod::findMethod()

Complexity

Conditions	2
Paths	4

Size

Total Lines	20
Code Lines	13

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	13
CRAP Score	2

Importance

Changes

0

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

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

        // Determine used parameters
        $orderBy = RequestHandler::getOrderBy($request);
        $limit = RequestHandler::getLimit($request);
        $offset = RequestHandler::getOffset($request);
        $search = RequestHandler::getSearchTerms($request);

        try {
            $criteria = RequestHandler::getCriteria($request);

            $this->processCriteria($criteria, $request, __METHOD__);

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

            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->find($criteria, $orderBy, $limit, $offset, $search), $resource);
        } catch (Throwable $exception) {
            throw $this->handleRestMethodException($exception);

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

        }