sfneal / google-places

src/Actions/Abstracts/PlacesSearchAction.php

<?php

namespace Sfneal\GooglePlaces\Actions\Abstracts;

use Sfneal\Actions\Action;
use Sfneal\GooglePlaces\Actions\CurlRequestAction;

abstract class PlacesSearchAction extends Action
{
    /**
     * @var array Longitude & Latitude of the location biases (Milford, MA)
     */
    private $location_bias_coords;

    /**
     * @var int A radius in miles from location bias coordinates
     */
    private $radius_bias;

    /**
     * @var string Google places country code (defaults to 'us')
     */
    private $country;

    /**
     * @var string Query parameters
     */
    public $query;

    /**
     * @var array Places predictions results
     */
    public $results;

    /**
     * @var null Place ID for retrieving a zip or other information about a place
     */
    public $place_id;

    /**
     * PlacesSearchAction constructor.
     *
     * @param string $query
     * @param null $place_id

Are you sure the doc-type for parameter $place_id is correct as it would always require null to be passed?

     */
    public function __construct(string $query, $place_id = null)
    {
        // Set params
        $this->query = self::sanitize($query);
        $this->place_id = $place_id;

        // Set optional env values
        $this->location_bias_coords = config('google-places.location_bias');
        $this->radius_bias = config('google-places.radius');
        $this->country = config('google-places.country');
    }

    /**
     * Retrieve the API endpoint for a google places AutocompleteCity search.
     *
     * @return string
     */
    public function getEndpoint(): string
    {
        $endpoint = 'https://maps.googleapis.com/maps/api/place/autocomplete/json?input='.$this->query;
        $endpoint .= '&types=(regions)';
        $endpoint .= '&region=us';

        // Add location bias if set
        if (isset($this->location_bias_coords)) {
            $endpoint .= '&location='.str_replace(' ', '', $this->location_bias_coords);

Are you sure str_replace(' ', '', $this->location_bias_coords) of type string[] can be used in concatenation?

        }

        // Add radius bias if set
        if (isset($this->radius_bias)) {
            $endpoint .= '&radius='.$this->radius_bias;
        }

        // Add language
        $endpoint .= '&language=en_EN';
        $endpoint .= '&components=country:'.$this->country;
        $endpoint .= '&key='.config('google-places.api_key');

        return $endpoint;
    }

    /**
     * Parse the Google Places API response to retrieve results set.
     *
     * @param $response
     * @return array|string
     */
    abstract public function parseResponse($response);

    /**
     * Remove spaces and forbidden characters from query string.
     *
     * @param $query
     * @return string|string[]
     */
    private static function sanitize($query)
    {
        return str_replace(' ', '-', $query);
    }

    /**
     * Retrieve the API URL endpoint, cURL execute the request and return response.
     *
     * @return mixed
     */
    public function execute()
    {
        return $this->parseResponse((new CurlRequestAction($this->getEndpoint()))->execute());
    }
}