tomk32-flickr_fu 0.3.0 → 0.3.1

data/README

@@ -170,6 +170,20 @@ token_cache: "token_cache.yml"
   
   photos = flickr.photos.search(:user_id => 'your_user_id_here')
 
+== Authenticatd Search Example
+
+  For a few searches you will need to authenticate. E.g. to receive the original_url
+  staight away (and thus saving an extra getInfo call), or to search for the private
+  photos of a user. In order to do so you need to pass the user's token and his
+  flickr_nsid (though the API docs also suggest 'me', both work fine).
+
+  This example appends the :url_o to the extras, mapping it to :original_url:
+
+  flickr.photos.extras << {:url_o => :original_url}
+  photos = flickr.photos.search(:per_page => 5, :page => 1,
+      :user_id => 'me', :privacy_filter => 5, :auth_token => user_token,
+      :safe_search => 3))
+
 == Patch Contributers
 
   Chris Ledet

data/VERSION.yml

@@ -1,4 +1,4 @@
 --- 
 :major: 0
 :minor: 3
-:patch: 0
+:patch: 1

data/{tomk32-flickr_fu.gemspec → flickr_fu.gemspec}

@@ -1,17 +1,14 @@
-# Generated by jeweler
-# DO NOT EDIT THIS FILE DIRECTLY
-# Instead, edit Jeweler::Tasks in Rakefile, and run the gemspec command
 # -*- encoding: utf-8 -*-
 
 Gem::Specification.new do |s|
   s.name = %q{tomk32-flickr_fu}
-  s.version = "0.3.0"
+  s.version = "0.3.1"
 
   s.required_rubygems_version = Gem::Requirement.new(">= 0") if s.respond_to? :required_rubygems_version=
-  s.authors = ["Ben Wyrosdick", "Maciej Bilas"]
-  s.date = %q{2009-11-08}
+  s.authors = ["Ben Wyrosdick", "Maciej Bilas", "Thomas R. Koll"]
+  s.date = %q{2010-04-17}
   s.description = %q{Provides a ruby interface to flickr via the REST api}
-  s.email = %q{ben@commonthread.com}
+  s.email = %q{tomk32@gmx.de}
   s.extra_rdoc_files = [
     "README"
   ]
@@ -21,7 +18,7 @@ Gem::Specification.new do |s|
      "README",
      "Rakefile",
      "VERSION.yml",
-     "tomk32-flickr_fu.gemspec",
+     "flickr_fu.gemspec",
      "lib/flickr/auth.rb",
      "lib/flickr/base.rb",
      "lib/flickr/comment.rb",
@@ -84,22 +81,22 @@ Gem::Specification.new do |s|
      "spec/spec.opts",
      "spec/spec_helper.rb"
   ]
-  s.homepage = %q{http://github.com/commonthread/flickr_fu}
+  s.homepage = %q{http://github.com/tomk32/flickr_fu}
   s.rdoc_options = ["--main", "README"]
   s.require_paths = ["lib"]
-  s.rubygems_version = %q{1.3.5}
+  s.rubygems_version = %q{1.3.3}
   s.summary = %q{Provides a ruby interface to flickr via the REST api}
   s.test_files = [
-    "spec/flickr/base_spec.rb",
+    "spec/spec_helper.rb",
+     "spec/flickr/test_spec.rb",
+     "spec/flickr/geo_spec.rb",
      "spec/flickr/contacts_spec.rb",
+     "spec/flickr/urls_spec.rb",
      "spec/flickr/errors_spec.rb",
-     "spec/flickr/geo_spec.rb",
+     "spec/flickr/base_spec.rb",
      "spec/flickr/photo_spec.rb",
-     "spec/flickr/photos_spec.rb",
      "spec/flickr/photosets_spec.rb",
-     "spec/flickr/test_spec.rb",
-     "spec/flickr/urls_spec.rb",
-     "spec/spec_helper.rb"
+     "spec/flickr/photos_spec.rb"
   ]
 
   if s.respond_to? :specification_version then
@@ -118,4 +115,3 @@ Gem::Specification.new do |s|
     s.add_dependency(%q<xml-magic>, ["> 0.0.0"])
   end
 end
-

data/lib/flickr/base.rb

@@ -84,8 +84,9 @@ module Flickr
     def send_request(method, options = {}, http_method = :get, endpoint = REST_ENDPOINT)
       options.merge!(:api_key => @api_key, :method => method)
       sign_request(options)
-      
       rsp = request_over_http(options, http_method, endpoint)
+puts y options
+puts y rsp
       
       rsp = '<rsp stat="ok"></rsp>' if rsp == ""
       xm = XmlMagic.new(rsp)

data/lib/flickr/photo.rb

@@ -15,6 +15,11 @@ class Flickr::Photos::Photo
   def initialize(flickr, attributes)
     @flickr = flickr
     attributes.each do |k,v|
+      # necessary for anything that comes via 'extras' or of course
+      # possible future changes in the api returns
+      if ! respond_to?(k)
+        self.class.send(:attr_accessor, k)
+      end
       send("#{k}=", v)
     end
   end

data/lib/flickr/photos.rb

@@ -1,17 +1,34 @@
 class Flickr::Photos < Flickr::Base
+  attr_accessor :extras
   def initialize(flickr)
     @flickr = flickr
+    self.extras ||= {
+      # flickr => flickr_fu
+      :license => :license_id,
+      :date_upload => :date_upload,
+      :datetaken => :taken_at,
+      :ownername => :owner_name,
+      :icon_server => :icon_server,
+      :originalformat => :original_format,
+      :lastupdate => :updated_at,
+      :machine_tags => :machine_tags,
+      :geo => :geo,
+      :tags => :tags,
+      :o_dims => :o_dims,
+      :views => :views,
+      :media => :media
+    }
   end
 
-  # Return a list of photos matching some criteria. Only photos visible to the calling user will be returned. To return private or semi-private photos, 
-  # the caller must be authenticated with 'read' permissions, and have permission to view the photos. Unauthenticated calls will only return public photos.
-  # 
+  # Return a list of photos matching some criteria. Only photos visible to the calling user will be returned. To return private or semi-private photos,
+  # the caller must be authenticated with 'read' permissions (by passing :auth_token), and have permission to view the photos. Unauthenticated calls will only return public photos.
+  #
   # == Authentication
   # This method does not require authentication.
-  # 
+  #
   # == Options
   # * user_id (Optional)
-  #     The NSID of the user who's photo to search. If this parameter isn't passed then everybody's public photos will be searched. A value of "me" will 
+  #     The NSID of the user who's photo to search. If this parameter isn't passed then everybody's public photos will be searched. A value of "me" will
   #     search against the calling user's photos for authenticated calls.
   # * tags (Optional)
   #     A comma-delimited list of tags. Photos with one or more of the tags listed will be returned.
@@ -30,7 +47,7 @@ class Flickr::Photos < Flickr::Base
   # * license (Optional)
   #     The license id for photos (for possible values see the flickr.photos.licenses.getInfo method). Multiple licenses may be comma-separated.
   # * sort (Optional)
-  #     The order in which to sort returned photos. Deafults to date-posted-desc. The possible values are: date-posted-asc, date-posted-desc, date-taken-asc, 
+  #     The order in which to sort returned photos. Deafults to date-posted-desc. The possible values are: date-posted-asc, date-posted-desc, date-taken-asc,
   #     date-taken-desc, interestingness-desc, interestingness-asc, and relevance.
   # * privacy_filter (Optional)
   #     Return photos only matching a certain privacy level. This only applies when making an authenticated call to view photos you own. Valid values are:
@@ -40,18 +57,18 @@ class Flickr::Photos < Flickr::Base
   #       4 private photos visible to friends & family
   #       5 completely private photos
   # * bbox (Optional)
-  #     A comma-delimited list of 4 values defining the Bounding Box of the area that will be searched. 
-  # 
-  #     The 4 values represent the bottom-left corner of the box and the top-right corner, minimum_longitude, minimum_latitude, maximum_longitude, maximum_latitude. 
-  # 
-  #     Longitude has a range of -180 to 180 , latitude of -90 to 90. Defaults to -180, -90, 180, 90 if not specified. 
-  # 
-  #     Unlike standard photo queries, geo (or bounding box) queries will only return 250 results per page. 
-  # 
-  #     Geo queries require some sort of limiting agent in order to prevent the database from crying. This is basically like the check against "parameterless searches" 
-  #     for queries without a geo component. 
-  # 
-  #     A tag, for instance, is considered a limiting agent as are user defined min_date_taken and min_date_upload parameters — If no limiting factor is passed we 
+  #     A comma-delimited list of 4 values defining the Bounding Box of the area that will be searched.
+  #
+  #     The 4 values represent the bottom-left corner of the box and the top-right corner, minimum_longitude, minimum_latitude, maximum_longitude, maximum_latitude.
+  #
+  #     Longitude has a range of -180 to 180 , latitude of -90 to 90. Defaults to -180, -90, 180, 90 if not specified.
+  #
+  #     Unlike standard photo queries, geo (or bounding box) queries will only return 250 results per page.
+  #
+  #     Geo queries require some sort of limiting agent in order to prevent the database from crying. This is basically like the check against "parameterless searches"
+  #     for queries without a geo component.
+  #
+  #     A tag, for instance, is considered a limiting agent as are user defined min_date_taken and min_date_upload parameters — If no limiting factor is passed we
   #     return only photos added in the last 12 hours (though we may extend the limit in the future).
   # * accuracy (Optional)
   #     Recorded accuracy level of the location information. Current range is 1-16 :
@@ -85,27 +102,27 @@ class Flickr::Photos < Flickr::Base
   #       Find photos that have a title, in any namespace : "machine_tags" => "*:title="
   #       Find photos that have a title, in any namespace, whose value is "mr. camera" : "machine_tags" => "*:title=\"mr. camera\""
   #       Find photos, in the 'dc' namespace whose value is "mr. camera" : "machine_tags" => "dc:*=\"mr. camera\""
-  #     Multiple machine tags may be queried by passing a comma-separated list. The number of machine tags you can pass in a single query depends on 
+  #     Multiple machine tags may be queried by passing a comma-separated list. The number of machine tags you can pass in a single query depends on
   #     the tag mode (AND or OR) that you are querying with. "AND" queries are limited to (16) machine tags. "OR" queries are limited to (8).
   # * machine_tag_mode (Required)
   #     Either 'any' for an OR combination of tags, or 'all' for an AND combination. Defaults to 'any' if not specified.
   # * group_id (Optional)
   #     The id of a group who's pool to search. If specified, only matching photos posted to the group's pool will be returned.
   # * woe_id (Optional)
-  #     A 32-bit identifier that uniquely represents spatial entities. (not used if bbox argument is present). Experimental. 
-  # 
-  #     Geo queries require some sort of limiting agent in order to prevent the database from crying. This is basically like the check against "parameterless searches" 
-  #     for queries without a geo component. 
-  # 
-  #     A tag, for instance, is considered a limiting agent as are user defined min_date_taken and min_date_upload parameters &emdash; If no limiting factor is passed 
+  #     A 32-bit identifier that uniquely represents spatial entities. (not used if bbox argument is present). Experimental.
+  #
+  #     Geo queries require some sort of limiting agent in order to prevent the database from crying. This is basically like the check against "parameterless searches"
+  #     for queries without a geo component.
+  #
+  #     A tag, for instance, is considered a limiting agent as are user defined min_date_taken and min_date_upload parameters &emdash; If no limiting factor is passed
   #     we return only photos added in the last 12 hours (though we may extend the limit in the future).
   # * place_id (Optional)
-  #     A Flickr place id. (not used if bbox argument is present). Experimental. 
-  # 
-  #     Geo queries require some sort of limiting agent in order to prevent the database from crying. This is basically like the check against "parameterless searches" 
-  #     for queries without a geo component. 
-  # 
-  #     A tag, for instance, is considered a limiting agent as are user defined min_date_taken and min_date_upload parameters &emdash; If no limiting factor is passed 
+  #     A Flickr place id. (not used if bbox argument is present). Experimental.
+  #
+  #     Geo queries require some sort of limiting agent in order to prevent the database from crying. This is basically like the check against "parameterless searches"
+  #     for queries without a geo component.
+  #
+  #     A tag, for instance, is considered a limiting agent as are user defined min_date_taken and min_date_upload parameters &emdash; If no limiting factor is passed
   #     we return only photos added in the last 12 hours (though we may extend the limit in the future).
   # * per_page (Optional)
   #     Number of photos to return per page. If this argument is omitted, it defaults to 100. The maximum allowed value is 500.
@@ -113,9 +130,12 @@ class Flickr::Photos < Flickr::Base
   #     The page of results to return. If this argument is omitted, it defaults to 1.
   # * media (Optional)
   #     The type of media to search for. 'photo', 'video', or 'both' are allowed arguments, with 'both' being the default.
-  # 
+  # * extras (Optional)
+  #     Any extra fields of information you want. The default extra fields are defined
+  #     in Flickr::Photos.extras and can be overwritten
+  #
   def search(options)
-    options.merge!({:extras => "license,date_upload,date_taken,owner_name,icon_server,original_format,last_update,geo,tags,machine_tags,o_dims,views,media"})
+    options[:extras] ||= self.extras.keys.join(',')
 
     rsp = @flickr.send_request('flickr.photos.search', options)
 
@@ -134,12 +154,12 @@ class Flickr::Photos < Flickr::Base
       end if rsp.photos.photo
     end
   end
-    
+
   # Returns a list of the latest public photos uploaded to flickr.
-  # 
+  #
   # == Authentication
   # This method does not require authentication.
-  # 
+  #
   # == Options
   # * per_page (Optional)
   #     Number of photos to return per page. If this argument is omitted, it defaults to 100. The maximum allowed value is 500.
@@ -147,9 +167,9 @@ class Flickr::Photos < Flickr::Base
   #     The page of results to return. If this argument is omitted, it defaults to 1.
   # * media (Optional)
   #     The type of media to search for. 'photo', 'video', or 'both' are allowed arguments, with 'both' being the default.
-  # 
+  #
   def get_recent(options = {})
-    options.merge!({:extras => "license,date_upload,date_taken,owner_name,icon_server,original_format,last_update,geo,tags,machine_tags,o_dims,views,media"})
+    options[:extras] ||= self.extras.keys.join(',')
 
     rsp = @flickr.send_request('flickr.photos.getRecent', options)
 
@@ -167,9 +187,9 @@ class Flickr::Photos < Flickr::Base
       end if rsp.photos.photo
     end
   end
-  
+
   def interesting(options)
-    options.merge!({:extras => "license,date_upload,date_taken,owner_name,icon_server,original_format,last_update,geo,tags,machine_tags,o_dims,views,media"})
+    options[:extras] ||= self.extras.keys.join(',')
 
     rsp = @flickr.send_request('flickr.interestingness.getList', options)
 
@@ -189,11 +209,11 @@ class Flickr::Photos < Flickr::Base
       end if rsp.photos.photo
     end
   end
-  
+
   def licenses
     @licenses ||= begin
       rsp = @flickr.send_request('flickr.photos.licenses.getInfo')
-      
+
       returning Hash.new do |licenses|
         rsp.licenses.license.each do |license|
           licenses[license[:id].to_i] = Flickr::Photos::License.new(:id => license[:id].to_i, :name => license[:name], :url => license[:url])
@@ -206,72 +226,44 @@ class Flickr::Photos < Flickr::Base
   def geo
     @geo ||= Flickr::Photos::Geo.new(@flickr)
   end
-  
+
   # Returns a Flickr::Photos::Photo object of the given id.
   # Raises an error if photo not found
   def find_by_id(photo_id)
     rsp = @flickr.send_request('flickr.photos.getInfo', :photo_id => photo_id)
-    
-    tags = []
-    machine_tags = []
-    
-    if rsp.photo.tags.tag
-      rsp.photo.tags.tag.each do |tag|
-        if tag[:machine_tag] == '1'
-          machine_tags << tag[:raw]
-        else
-          tags << tag[:raw]
-        end
-      end
-    end
-    
-    Photo.new(@flickr,
-      :id => rsp.photo[:id].to_i,
-      :owner => rsp.photo.owner,
-      :secret => rsp.photo[:secret],
-      :server => rsp.photo[:server].to_i,
-      :farm => rsp.photo[:farm], 
-      :title => rsp.photo.title.to_s,
-      :is_public => rsp.photo.visibility[:ispublic],
-      :is_friend => rsp.photo.visibility[:isfriend],
-      :is_family => rsp.photo.visibility[:isfamily],
-      :license_id => rsp.photo[:license].to_i,
-      :uploaded_at => (Time.at(rsp.photo[:dateuploaded].to_i) rescue nil),
-      :taken_at => (Time.parse(rsp.photo.dates[:taken]) rescue nil),
-      :owner_name => rsp.photo.owner[:username],
-      :icon_server => rsp.photo[:icon_server],
-      :original_format => rsp.photo[:originalformat],
-      :updated_at => (Time.at(rsp.photo.dates[:lastupdate].to_i) rescue nil),
-      :tags => tags,
-      :machine_tags => machine_tags,
-      :views => rsp.photo[:views].to_i,
-      :media => rsp.photo[:media])
+    Photo.new(@flickr, :id => rsp.photo[:id].to_i, :owner => rsp.photo.owner,
+      :secret => rsp.photo[:secret], :server => rsp.photo[:server].to_i, :farm => rsp.photo[:farm],
+      :title => rsp.photo.title,
+      :is_public => rsp.photo.visibility[:public], :is_friend => rsp.photo.visibility[:is_friend], :is_family => rsp.photo.visibility[:is_family])
   end
-  
+
   protected
   def create_attributes(photo)
-    {:id => photo[:id], 
-     :owner => photo[:owner], 
-     :secret => photo[:secret], 
-     :server => photo[:server], 
-     :farm => photo[:farm], 
-     :title => photo[:title], 
-     :is_public => photo[:ispublic], 
-     :is_friend => photo[:isfriend], 
-     :is_family => photo[:isfamily],
+    attributes = {}
+
+    # map all attributes from flickr to `attributes`
+    {
+      # flickr -> flickr_fu
+      :id => :id,
+      :owner => :owner,
+      :secret => :secret,
+      :server => :server,
+      :farm => :farm,
+      :title => :title,
+      :ispublic => :is_public,
+      :isfriend => :is_friend,
+      :isfamily => :is_family
+    }.merge(self.extras).each do |flickr_key, photo_key|
+      attributes[photo_key] = photo[flickr_key]
+    end
+
+    # and a few conversions
+    attributes.merge({
      :license_id => photo[:license].to_i,
      :uploaded_at => (Time.at(photo[:dateupload].to_i) rescue nil),
      :taken_at => (Time.parse(photo[:datetaken]) rescue nil),
-     :owner_name => photo[:ownername],
-     :icon_server => photo[:icon_server],
-     :original_format => photo[:originalformat],
      :updated_at => (Time.at(photo[:lastupdate].to_i) rescue nil),
-     :geo => photo[:geo],
-     :tags => photo[:tags],
-     :machine_tags => photo[:machine_tags],
-     :o_dims => photo[:o_dims],
-     :views => photo[:views].to_i,
-     :media => photo[:media]}
+     :views => photo[:views].to_i
+    })
   end
-  
 end

data/lib/flickr_fu.rb

@@ -39,13 +39,10 @@ end
 
 include CommonThread::XML
 
-# Rails' ActiveSupport also contains this, we don't want to override it in that case.
-unless defined?(Object::returning)
-  class Object
-    # returning allows you to pass an object to a block that you can manipulate returning the manipulated object
-    def returning(value)
-      yield(value)
-      value
-    end
+class Object
+  # returning allows you to pass an object to a block that you can manipulate returning the manipulated object
+  def returning(value)
+    yield(value)
+    value
   end
 end

metadata

@@ -1,40 +1,54 @@
 --- !ruby/object:Gem::Specification 
 name: tomk32-flickr_fu
 version: !ruby/object:Gem::Version 
-  version: 0.3.0
+  prerelease: false
+  segments: 
+  - 0
+  - 3
+  - 1
+  version: 0.3.1
 platform: ruby
 authors: 
 - Ben Wyrosdick
 - Maciej Bilas
+- Thomas R. Koll
 autorequire: 
 bindir: bin
 cert_chain: []
 
-date: 2009-11-08 00:00:00 +01:00
+date: 2010-04-17 00:00:00 +02:00
 default_executable: 
 dependencies: 
 - !ruby/object:Gem::Dependency 
   name: mime-types
-  type: :runtime
-  version_requirement: 
-  version_requirements: !ruby/object:Gem::Requirement 
+  prerelease: false
+  requirement: &id001 !ruby/object:Gem::Requirement 
     requirements: 
     - - ">"
       - !ruby/object:Gem::Version 
+        segments: 
+        - 0
+        - 0
+        - 0
         version: 0.0.0
-    version: 
+  type: :runtime
+  version_requirements: *id001
 - !ruby/object:Gem::Dependency 
   name: xml-magic
-  type: :runtime
-  version_requirement: 
-  version_requirements: !ruby/object:Gem::Requirement 
+  prerelease: false
+  requirement: &id002 !ruby/object:Gem::Requirement 
     requirements: 
     - - ">"
       - !ruby/object:Gem::Version 
+        segments: 
+        - 0
+        - 0
+        - 0
         version: 0.0.0
-    version: 
+  type: :runtime
+  version_requirements: *id002
 description: Provides a ruby interface to flickr via the REST api
-email: ben@commonthread.com
+email: tomk32@gmx.de
 executables: []
 
 extensions: []
@@ -47,7 +61,7 @@ files:
 - README
 - Rakefile
 - VERSION.yml
-- tomk32-flickr_fu.gemspec
+- flickr_fu.gemspec
 - lib/flickr/auth.rb
 - lib/flickr/base.rb
 - lib/flickr/comment.rb
@@ -110,7 +124,7 @@ files:
 - spec/spec.opts
 - spec/spec_helper.rb
 has_rdoc: true
-homepage: http://github.com/commonthread/flickr_fu
+homepage: http://github.com/tomk32/flickr_fu
 licenses: []
 
 post_install_message: 
@@ -123,29 +137,31 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements: 
   - - ">="
     - !ruby/object:Gem::Version 
+      segments: 
+      - 0
       version: "0"
-  version: 
 required_rubygems_version: !ruby/object:Gem::Requirement 
   requirements: 
   - - ">="
     - !ruby/object:Gem::Version 
+      segments: 
+      - 0
       version: "0"
-  version: 
 requirements: []
 
 rubyforge_project: 
-rubygems_version: 1.3.5
+rubygems_version: 1.3.6
 signing_key: 
 specification_version: 3
 summary: Provides a ruby interface to flickr via the REST api
 test_files: 
-- spec/flickr/base_spec.rb
+- spec/spec_helper.rb
+- spec/flickr/test_spec.rb
+- spec/flickr/geo_spec.rb
 - spec/flickr/contacts_spec.rb
+- spec/flickr/urls_spec.rb
 - spec/flickr/errors_spec.rb
-- spec/flickr/geo_spec.rb
+- spec/flickr/base_spec.rb
 - spec/flickr/photo_spec.rb
-- spec/flickr/photos_spec.rb
 - spec/flickr/photosets_spec.rb
-- spec/flickr/test_spec.rb
-- spec/flickr/urls_spec.rb
-- spec/spec_helper.rb
+- spec/flickr/photos_spec.rb