google_places 0.33.0 → 0.34.0

data/CHANGELOG.md

@@ -0,0 +1,5 @@
+## 0.34.0
+
+- Added the option `details: true` to all methods that return a collection of spots
+- Added `permanently_closed` to the returned fields for a spot
+- Introduced the CHANGELOG

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    google_places (0.33.0)
+    google_places (0.34.0)
       httparty (>= 0.13.1, < 0.14.1)
 
 GEM

data/README.rdoc

@@ -99,6 +99,20 @@ Search by multiple types and exclude multiple types
 
   @client.spots_by_query('Pizza near Miami Florida', :types => ['restaurant', 'food'], :exclude => ['cafe', 'establishment'])
 
+=== Retrieving a collection of spots with complete details
+
+Google limits the details that are returned in any API calls for a collection,
+so you'll often find that details such as phone numbers are missing in a
+collection of spots, but are filled in when retrieving a single spot.
+
+If you require these extra details to be completed when retrieving a number of
+results, you can pass in the <tt>detail: true</tt> option to any method that returns
+a collection of spots.
+
+This option should be used with care, as it adds an additional API call
+for EACH spot in the collection. E.g. a spots collection of 100 spots
+will use 101 API calls when the <tt>detail: true</tt> option is set.
+
 === Retrieving a single spot
 
 First register a new Client:

data/google_places.gemspec

@@ -3,7 +3,7 @@ $:.push File.expand_path('../lib', __FILE__)
 
 Gem::Specification.new do |s|
   s.name        = 'google_places'
-  s.version     = '0.33.0'
+  s.version     = '0.34.0'
   s.platform    = Gem::Platform::RUBY
   s.authors     = ['Marcel de Graaf']
   s.email       = ['mail@marceldegraaf.net']

data/lib/google_places/client.rb

@@ -93,14 +93,11 @@ module GooglePlaces
     # @see http://spreadsheets.google.com/pub?key=p9pdwsai2hDMsLkXsoM05KQ&gid=1 List of supported languages
     # @see https://developers.google.com/maps/documentation/places/supported_types List of supported types
     def spots(lat, lng, options = {})
-      detail_spot = options.delete(:detail)
-      spots = Spot.list(lat, lng, @api_key, @options.merge(options))
-
-      if detail_spot
-        spots.map {|spot| Spot.find(spot.place_id, @api_key, @options.merge(options))}
-      else
-        spots
-      end
+      detail = @options.merge!(options).delete(:detail)
+      collection_detail_level(
+        Spot.list(lat, lng, @api_key, @options),
+        detail
+      )
     end
 
     # Search for a Spot with a reference key
@@ -153,11 +150,18 @@ module GooglePlaces
     # @option options [Object] :retry_options[:status] ([])
     # @option options [Integer] :retry_options[:max] (0) the maximum retries
     # @option options [Integer] :retry_options[:delay] (5) the delay between each retry in seconds
+    # @option options [Boolean] :detail
+    #   A boolean to return spots with full detail information(its complete address, phone number, user rating, reviews, etc)
+    #   Note) This makes an extra call for each spot for more information.
     #
     # @see http://spreadsheets.google.com/pub?key=p9pdwsai2hDMsLkXsoM05KQ&gid=1 List of supported languages
     # @see https://developers.google.com/maps/documentation/places/supported_types List of supported types
     def spots_by_query(query, options = {})
-      Spot.list_by_query(query, @api_key, @options.merge(options))
+      detail = @options.merge!(options).delete(:detail)
+      collection_detail_level(
+        Spot.list_by_query(query, @api_key, @options),
+        detail
+      )
     end
     # Search for Spots within a give SW|NE bounds with query
     #
@@ -190,10 +194,17 @@ module GooglePlaces
     # @option options [Object] :retry_options[:status] ([])
     # @option options [Integer] :retry_options[:max] (0) the maximum retries
     # @option options [Integer] :retry_options[:delay] (5) the delay between each retry in seconds
+    # @option options [Boolean] :detail
+    #   A boolean to return spots with full detail information(its complete address, phone number, user rating, reviews, etc)
+    #   Note) This makes an extra call for each spot for more information.
     #
     # @see https://developers.google.com/maps/documentation/places/supported_types List of supported types
     def spots_by_bounds(bounds, options = {})
-      Spot.list_by_bounds(bounds, @api_key, @options.merge(options))
+      detail = @options.merge!(options).delete(:detail)
+      collection_detail_level(
+        Spot.list_by_bounds(bounds, @api_key, @options),
+        detail
+      )
     end
     # Search for Spots with a pagetoken
     #
@@ -207,10 +218,17 @@ module GooglePlaces
     # @option options [Object] :retry_options[:status] ([])
     # @option options [Integer] :retry_options[:max] (0) the maximum retries
     # @option options [Integer] :retry_options[:delay] (5) the delay between each retry in seconds
+    # @option options [Boolean] :detail
+    #   A boolean to return spots with full detail information(its complete address, phone number, user rating, reviews, etc)
+    #   Note) This makes an extra call for each spot for more information.
     #
     # @see https://developers.google.com/maps/documentation/places/supported_types List of supported types
     def spots_by_pagetoken(pagetoken, options = {})
-      Spot.list_by_pagetoken(pagetoken, @api_key, @options.merge(options))
+      detail = @options.merge!(options).delete(:detail)
+      collection_detail_level(
+        Spot.list_by_pagetoken(pagetoken, @api_key, @options),
+        detail
+      )
     end
 
     # Radar Search Service allows you to search for up to 200 Places at once, but with less detail than is typically returned from a Text Search or Nearby Search request. The search response will include up to 200 Places, identified only by their geographic coordinates and reference. You can send a Place Details request for more information about any of them.
@@ -244,10 +262,17 @@ module GooglePlaces
     #   Restrict your search to only those locations that are Zagat selected businesses.
     #   This parameter does not require a true or false value, simply including the parameter in the request is sufficient to restrict your search.
     #   The zagatselected parameter is experimental, and only available to Places API enterprise customers.
+    # @option options [Boolean] :detail
+    #   A boolean to return spots with full detail information(its complete address, phone number, user rating, reviews, etc)
+    #   Note) This makes an extra call for each spot for more information.
     #
     # @see https://developers.google.com/places/documentation/search#RadarSearchRequests
     def spots_by_radar(lat, lng, options = {})
-      Spot.list_by_radar(lat, lng, @api_key, @options.merge(options))
+      detail = @options.merge!(options).delete(:detail)
+      collection_detail_level(
+        Spot.list_by_radar(lat, lng, @api_key, @options),
+        detail
+      )
     end
 
     # Query for Place Predictions
@@ -272,5 +297,17 @@ module GooglePlaces
     def predictions_by_input(input, options = {})
       Prediction.list_by_input(input, @api_key, @options.merge(options))
     end
+
+    private
+
+    def collection_detail_level(spots, detail = false)
+      if detail
+        spots.map do |spot|
+          Spot.find(spot.place_id, @api_key, @options)
+        end
+      else
+        spots
+      end
+    end
   end
 end

data/lib/google_places/review.rb

@@ -1,14 +1,16 @@
 module GooglePlaces
   class Review
-    attr_accessor :rating, :type, :author_name, :author_url, :text, :time
+    attr_accessor :rating, :type, :author_name, :author_url, :text, :time,
+                  :profile_photo_url
 
-    def initialize(rating, type, author_name, author_url, text, time)
-      @rating       = rating
-      @type         = type
-      @author_name  = author_name
-      @author_url   = author_url
-      @text         = text
-      @time         = time
+    def initialize(rating, type, author_name, author_url, text, time, profile_photo_url)
+      @rating            = rating
+      @type              = type
+      @author_name       = author_name
+      @author_url        = author_url
+      @text              = text
+      @time              = time
+      @profile_photo_url = profile_photo_url
     end
   end
 end

data/lib/google_places/spot.rb

@@ -1,7 +1,7 @@
 require 'google_places/review'
 module GooglePlaces
   class Spot
-    attr_accessor :lat, :lng, :viewport, :name, :icon, :reference, :vicinity, :types, :id, :formatted_phone_number, :international_phone_number, :formatted_address, :address_components, :street_number, :street, :city, :region, :postal_code, :country, :rating, :url, :cid, :website, :reviews, :aspects, :zagat_selected, :zagat_reviewed, :photos, :review_summary, :nextpagetoken, :price_level, :opening_hours, :events, :utc_offset, :place_id
+    attr_accessor :lat, :lng, :viewport, :name, :icon, :reference, :vicinity, :types, :id, :formatted_phone_number, :international_phone_number, :formatted_address, :address_components, :street_number, :street, :city, :region, :postal_code, :country, :rating, :url, :cid, :website, :reviews, :aspects, :zagat_selected, :zagat_reviewed, :photos, :review_summary, :nextpagetoken, :price_level, :opening_hours, :events, :utc_offset, :place_id, :permanently_closed
 
     # Search for Spots at the provided location
     #
@@ -441,6 +441,7 @@ module GooglePlaces
       @nextpagetoken              = json_result_object['nextpagetoken']
       @events                     = events_component(json_result_object['events'])
       @utc_offset                 = json_result_object['utc_offset']
+      @permanently_closed         = json_result_object['permanently_closed']
     end
 
     def [] (key)

data/spec/google_places/client_spec.rb

@@ -1,69 +1,165 @@
 require 'spec_helper'
 
 describe GooglePlaces::Client do
+  let(:client) { GooglePlaces::Client.new(api_key) }
+  let(:fake_spot) { Object.new }
+
+  before do
+    allow(fake_spot).to receive(:place_id) { 1 }
+  end
+
   it 'should initialize with an api_key' do
-    @client = GooglePlaces::Client.new(api_key)
-    expect(@client.api_key).to eq(api_key)
+    expect(client.api_key).to eq(api_key)
   end
 
-  it 'should request spots' do
-    lat, lng = '-33.8670522', '151.1957362'
-    @client = GooglePlaces::Client.new(api_key)
-    expect(GooglePlaces::Spot).to receive(:list).with(lat, lng, api_key, {})
+  describe '::spots' do
+    let(:lat) { '-33.8670522' }
+    let(:lng) { '151.1957362' }
+    it 'should request spots' do
+      expect(GooglePlaces::Spot).to receive(:list).with(lat, lng, api_key, {})
+      client.spots(lat, lng)
+    end
 
-    @client.spots(lat, lng)
-  end
+    it 'does not call find on GooglePlces::Spot' do
+      allow(GooglePlaces::Spot).to receive(:list) { [fake_spot] }
+      expect(GooglePlaces::Spot).not_to receive(:find)
+      client.spots(lat, lng)
+    end
 
-  it 'should request a single spot by place_id' do
-    place_id = 'ChIJu46S-ZZhLxMROG5lkwZ3D7k'
-    @client = GooglePlaces::Client.new(api_key)
-    expect(GooglePlaces::Spot).to receive(:find).with(place_id, api_key, {})
+    context 'with detail set to true' do
+      it 'calls find on GooglePlaces::Spot' do
+        allow(GooglePlaces::Spot).to receive(:list) { [fake_spot] }
+        expect(GooglePlaces::Spot).to receive(:find)
+        client.spots(lat, lng, detail: true)
+      end
+    end
+  end
 
-    @client.spot(place_id)
+  describe '::spot' do
+    let(:place_id) { 'ChIJu46S-ZZhLxMROG5lkwZ3D7k' }
+    it 'should request a single spot by place_id' do
+      expect(GooglePlaces::Spot).to receive(:find).with(place_id, api_key, {})
+      client.spot(place_id)
+    end
   end
 
-  it 'should request spots by query' do
-    query = 'Statue of liberty, New York'
-    @client = GooglePlaces::Client.new(api_key)
-    expect(GooglePlaces::Spot).to receive(:list_by_query).with(query, api_key, {})
+  describe '::spots_by_query' do
+    let(:query) { 'Statue of liberty, New York' }
+    it 'should request spots by query' do
+      expect(GooglePlaces::Spot).to receive(:list_by_query).with(
+        query,
+        api_key,
+        {}
+      )
+      client.spots_by_query(query)
+    end
+
+    it 'does not call find on GooglePlces::Spot' do
+      allow(GooglePlaces::Spot).to receive(:list_by_query) { [fake_spot] }
+      expect(GooglePlaces::Spot).not_to receive(:find)
+      client.spots_by_query(query)
+    end
 
-    @client.spots_by_query(query)
+    context 'with detail set to true' do
+      it 'calls find on GooglePlaces::Spot' do
+        allow(GooglePlaces::Spot).to receive(:list_by_query) { [fake_spot] }
+        expect(GooglePlaces::Spot).to receive(:find)
+        client.spots_by_query(query, detail: true)
+      end
+    end
   end
-  it 'should request spots by bounds' do
-    query = 'pizza'
-    bounds = {:start_point => {:lat => '36.06686213257888', :lng => '-86.94168090820312'},
-              :end_point => {:lat => '36.268635800737876', :lng => '-86.66152954101562'}}
-    @client = GooglePlaces::Client.new(api_key)
-    expect(GooglePlaces::Spot).to receive(:list_by_bounds).with(bounds, api_key, {:query => query})
-    res = @client.spots_by_bounds(bounds, :query => query)
+
+  describe '::spots_by_bounds' do
+    let(:query) { 'pizza' }
+    let(:bounds) do
+      {
+        start_point: { lat: '36.06686213257888', lng: '-86.94168090820312' },
+        end_point: { lat: '36.268635800737876', lng: '-86.66152954101562' }
+      }
+    end
+
+    it 'should request spots by bounds' do
+      expect(GooglePlaces::Spot).to receive(:list_by_bounds).with(
+        bounds, api_key,
+        query: query
+      )
+      client.spots_by_bounds(bounds, query: query)
+    end
+
+    it 'does not call find on GooglePlces::Spot' do
+      allow(GooglePlaces::Spot).to receive(:list_by_bounds) { [fake_spot] }
+      expect(GooglePlaces::Spot).not_to receive(:find)
+      client.spots_by_bounds(bounds, query: query)
+    end
+
+    context 'with detail set to true' do
+      it 'calls find on GooglePlaces::Spot' do
+        allow(GooglePlaces::Spot).to receive(:list_by_bounds) { [fake_spot] }
+        expect(GooglePlaces::Spot).to receive(:find)
+        client.spots_by_bounds(bounds, query: query, detail: true)
+      end
+    end
   end
-  it 'should request spots by radar' do
-    keywords = 'landmarks'
-    lat, lng = '51.511627', '-0.183778'
-    radius = 5000
-    @client = GooglePlaces::Client.new(api_key)
-    expect(GooglePlaces::Spot).to receive(:list_by_radar).with(lat, lng, api_key, {:radius=> radius, :keyword =>  keywords})
-
-    @client.spots_by_radar(lat, lng, :radius => radius, :keyword =>  keywords)
+
+  describe '::spots_by_radar' do
+    let(:keywords) { 'landmarks' }
+    let(:lat) { '51.511627' }
+    let(:lng) { '-0.183778' }
+    let(:radius) { 5000 }
+
+    it 'should request spots by radar' do
+      expect(GooglePlaces::Spot).to receive(:list_by_radar).with(
+        lat,
+        lng,
+        api_key,
+        radius: radius,
+        keyword: keywords
+      )
+      client.spots_by_radar(lat, lng, radius: radius, keyword: keywords)
+    end
+
+    it 'does not call find on GooglePlces::Spot' do
+      allow(GooglePlaces::Spot).to receive(:list_by_radar) { [fake_spot] }
+      expect(GooglePlaces::Spot).not_to receive(:find)
+      client.spots_by_radar(lat, lng, radius: radius, keyword: keywords)
+    end
+
+    context 'with detail set to true' do
+      it 'calls find on GooglePlaces::Spot' do
+        allow(GooglePlaces::Spot).to receive(:list_by_radar) { [fake_spot] }
+        expect(GooglePlaces::Spot).to receive(:find)
+        client.spots_by_radar(
+          lat,
+          lng,
+          radius: radius,
+          keyword: keywords,
+          detail: true
+        )
+      end
+    end
   end
 
-  it 'should request predictions by input' do
-    input = 'Atlanta'
-    @client = GooglePlaces::Client.new(api_key)
-    expect(GooglePlaces::Prediction).to receive(:list_by_input).with(input, api_key, {})
+  describe '::predictions_by_input' do
+    let(:input) { 'Atlanta' }
 
-    @client.predictions_by_input(input)
+    it 'should request predictions by input' do
+      expect(GooglePlaces::Prediction).to receive(:list_by_input).with(
+        input,
+        api_key,
+        {}
+      )
+      client.predictions_by_input(input)
+    end
   end
 
-  context 'List detailed spots', vcr: { cassette_name: 'list_spots_with_detail' } do
+  describe 'detailed spots', vcr: { cassette_name: 'list_spots_with_detail' } do
+    let(:lat) { '28.3852377' }
+    let(:lng) { '-81.566068' }
     it 'should return spots with detail information' do
-      lat, lng = '28.3852377', '-81.566068'
-      @client = GooglePlaces::Client.new(api_key)
-
-      spots = @client.spots(lat, lng, detail: true)
+      spots = client.spots(lat, lng, detail: true)
       expect(spots).to_not be_nil
 
-      for spot in spots
+      spots.each do |spot|
         expect(spot.address_components).not_to be_nil
         expect(spot.city).not_to be_nil
         expect(spot.country).not_to be_nil

data/spec/google_places/spot_spec.rb

@@ -161,7 +161,7 @@ describe GooglePlaces::Spot do
     it 'should be a Spot' do
       expect(@spot.class).to eq(GooglePlaces::Spot)
     end
-    %w(reference vicinity lat lng viewport name icon types id formatted_phone_number international_phone_number formatted_address address_components street_number street city region postal_code country rating url types website price_level opening_hours events utc_offset place_id).each do |attribute|
+    %w(reference vicinity lat lng viewport name icon types id formatted_phone_number international_phone_number formatted_address address_components street_number street city region postal_code country rating url types website price_level opening_hours events utc_offset place_id permanently_closed).each do |attribute|
       it "should have the attribute: #{attribute}" do
         expect(@spot.respond_to?(attribute)).to eq(true)
       end

metadata

@@ -1,89 +1,100 @@
 --- !ruby/object:Gem::Specification
 name: google_places
 version: !ruby/object:Gem::Version
-  version: 0.33.0
+  version: 0.34.0
+  prerelease: 
 platform: ruby
 authors:
 - Marcel de Graaf
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2017-02-13 00:00:00.000000000 Z
+date: 2017-06-07 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: httparty
   requirement: !ruby/object:Gem::Requirement
+    none: false
     requirements:
-    - - ">="
+    - - ! '>='
       - !ruby/object:Gem::Version
         version: 0.13.1
-    - - "<"
+    - - <
       - !ruby/object:Gem::Version
         version: 0.14.1
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
+    none: false
     requirements:
-    - - ">="
+    - - ! '>='
       - !ruby/object:Gem::Version
         version: 0.13.1
-    - - "<"
+    - - <
       - !ruby/object:Gem::Version
         version: 0.14.1
 - !ruby/object:Gem::Dependency
   name: rspec
   requirement: !ruby/object:Gem::Requirement
+    none: false
     requirements:
-    - - "~>"
+    - - ~>
       - !ruby/object:Gem::Version
         version: '3.0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
+    none: false
     requirements:
-    - - "~>"
+    - - ~>
       - !ruby/object:Gem::Version
         version: '3.0'
 - !ruby/object:Gem::Dependency
   name: addressable
   requirement: !ruby/object:Gem::Requirement
+    none: false
     requirements:
-    - - "~>"
+    - - ~>
       - !ruby/object:Gem::Version
         version: 2.4.0
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
+    none: false
     requirements:
-    - - "~>"
+    - - ~>
       - !ruby/object:Gem::Version
         version: 2.4.0
 - !ruby/object:Gem::Dependency
   name: webmock
   requirement: !ruby/object:Gem::Requirement
+    none: false
     requirements:
-    - - "~>"
+    - - ~>
       - !ruby/object:Gem::Version
         version: '1.18'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
+    none: false
     requirements:
-    - - "~>"
+    - - ~>
       - !ruby/object:Gem::Version
         version: '1.18'
 - !ruby/object:Gem::Dependency
   name: vcr
   requirement: !ruby/object:Gem::Requirement
+    none: false
     requirements:
-    - - "~>"
+    - - ~>
       - !ruby/object:Gem::Version
         version: '2.9'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
+    none: false
     requirements:
-    - - "~>"
+    - - ~>
       - !ruby/object:Gem::Version
         version: '2.9'
 description: This gem provides a Ruby wrapper around the Google Places API for use
@@ -94,8 +105,9 @@ executables: []
 extensions: []
 extra_rdoc_files: []
 files:
-- ".gitignore"
-- ".travis.yml"
+- .gitignore
+- .travis.yml
+- CHANGELOG.md
 - Gemfile
 - Gemfile.lock
 - LICENSE.txt
@@ -139,26 +151,27 @@ files:
 homepage: https://github.com/marceldegraaf/google_places
 licenses:
 - MIT
-metadata: {}
 post_install_message: 
 rdoc_options: []
 require_paths:
 - lib
 required_ruby_version: !ruby/object:Gem::Requirement
+  none: false
   requirements:
-  - - ">="
+  - - ! '>='
     - !ruby/object:Gem::Version
       version: '0'
 required_rubygems_version: !ruby/object:Gem::Requirement
+  none: false
   requirements:
-  - - ">="
+  - - ! '>='
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.2.2
+rubygems_version: 1.8.23.2
 signing_key: 
-specification_version: 4
+specification_version: 3
 summary: A Ruby wrapper around the Google Places API.
 test_files:
 - spec/api_key.sample.rb

checksums.yaml

@@ -1,7 +0,0 @@
----
-SHA1:
-  metadata.gz: 3e89cd376d97e4bbadf049864a5df5f98d032630
-  data.tar.gz: c5d6c2bb190192b928dc3abab4956615a99d92f0
-SHA512:
-  metadata.gz: 568369db3f5e9d71a65363c2a806c36cabc94fe55fc5370b6afaede67e3a4d8f4ff1fb24ea81c3dba0ed282ee3f855bed00644a84b4e267e8f67e0e73907d52e
-  data.tar.gz: df20278fd0930297bf41446bece309703011177f31181dbbf6109a6cd7e9485f2d892e3fdaf06c770434914e202a0dbe9605f8d7f87016dfaf4872ecb597a692