Ijeffro\Laralocker\LearningLocker\Queries\QueryHandler

Complexity

Total Complexity

5

Size/Duplication

Total Lines	43
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	3	1
A	update()	0	7	2
A	get()	0	7	2

<?php

namespace Ijeffro\Laralocker\LearningLocker\Queries;

use Ijeffro\Laralocker\LearningLocker\API\APIHandler;

class QueryHandler extends APIHandler implements QueryInterface {

    private $query = '/query';
    private $api = '/api';
    private $v1 = '/v1';

The private property $v1 is not used, and could be removed.

    private $v2 = '/v2';

    protected $headers = [
      'content-type' => 'application/json'
    ];

    function __construct($id = null) {
        parent::__construct();
        $this->id = $id;

The property id does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.

    }

    /**
     * Learning Locker: Request Organisation Details
     *
     * @return  $response

The doc comment $response at position 0 could not be parsed: Unknown type name '$response' at position 0 in $response.

     */
    public function get() {
        try {
            $url = $this->url . $this->api . $this->v2 . $this->query . '/' . $this->id ?? $this->id;
            $response = $this->request($url);
            return $response;
        } catch (Exception $e) {

The type Ijeffro\Laralocker\LearningLocker\Queries\Exception was not found. Did you mean Exception? If so, make sure to prefix the type with \.

            return $e;
        }
    }

    /**
     * Learning Locker: Request Organisation Details
     *
     * @return  $response

The doc comment $response at position 0 could not be parsed: Unknown type name '$response' at position 0 in $response.

     */
    public function update($data) {
        try {
            $url = $this->url . $this->api . $this->v2 . $this->query . '/' . $this->id ?? $this->id;
            $response = $this->save($url, $data);
            return $response;
        } catch (Exception $e) {
            return $e;
        }
    }
}