flickraw 0.9.9 → 0.9.10

checksums.yaml

@@ -1,7 +1,7 @@
 ---
-SHA1:
-  metadata.gz: cdf1d2ebe95cae5115bb7e53b0e4e8f4dc9fa56a
-  data.tar.gz: 0e68d6ee1844edab7fb300051993cf071711e036
+SHA256:
+  metadata.gz: 160c81f7e624a74d0659ac6af8c2113e31eddb1dca007aa8c476473473805070
+  data.tar.gz: 33fe9c9c542955b211d8127f3b093b48a4a2cb7e37ccdc3949abd59444c5a112
 SHA512:
-  metadata.gz: 3d28e97c5452a9a8f662b565b02c24100bf91462b77db11a179e638ec51425efaad66a9d203b6fad997cdaac9b25c9935cb6145cb8230e62d7c16a7f16361881
-  data.tar.gz: eea3e153758348555ca757fd996089b3dad4bfda8b43eb774f2c052ef4d8f1c46411107d8ee2e87a09a9a8a245839153033a8a7749a2521fc26d2868357013d8
+  metadata.gz: de6bfc36428610b9d7d090a6b8e70a6c36d70ecf36eeae6023717eb4481096d75c52c7435118576b565ef529aa88110b9edf5ea00a9d88e89fa34afed1b72522
+  data.tar.gz: 76c6fc51e76ff719bffe5da9c80ec9bcdf9b5a57668b61d2d8bc5ea3dc563a88050b85e24f51292f969b914f4014a0f08fc7c73d183eeb6427f5e39a66c1f560

data/README.rdoc

@@ -62,7 +62,7 @@ This will recreate the documentation by fetching the methods descriptions from f
   token = flickr.get_request_token
   auth_url = flickr.get_authorize_url(token['oauth_token'], :perms => 'delete')
 
-  puts "Open this url in your process to complete the authication process : #{auth_url}"
+  puts "Open this url in your browser to complete the authentication process : #{auth_url}"
   puts "Copy here the number given when you complete the process."
   verify = gets.strip
 

data/examples/auth.rb

@@ -11,7 +11,7 @@ FlickRaw.shared_secret=SHARED_SECRET
 token = flickr.get_request_token
 auth_url = flickr.get_authorize_url(token['oauth_token'], :perms => 'delete')
 
-puts "Open this url in your process to complete the authication process : #{auth_url}"
+puts "Open this url in your browser to complete the authentication process : #{auth_url}"
 puts "Copy here the number given when you complete the process."
 verify = gets.strip
 

data/lib/flickraw.rb

@@ -7,9 +7,9 @@ require 'flickraw/response'
 require 'flickraw/api'
 
 module FlickRaw
-  VERSION='0.9.9'
+  VERSION='0.9.10'
   USER_AGENT = "FlickRaw/#{VERSION}"
-  
+
   self.secure = true
   self.check_certificate = true
 end

data/lib/flickraw/api.rb

@@ -2,11 +2,11 @@ module FlickRaw
   END_POINT='http://api.flickr.com/services'.freeze
   END_POINT2='http://www.flickr.com/services'.freeze
   END_POINT_SECURE='https://api.flickr.com/services'.freeze
-  
+
   FLICKR_OAUTH_REQUEST_TOKEN=(END_POINT2 + '/oauth/request_token').freeze
   FLICKR_OAUTH_AUTHORIZE=(END_POINT2 + '/oauth/authorize').freeze
   FLICKR_OAUTH_ACCESS_TOKEN=(END_POINT2 + '/oauth/access_token').freeze
-  
+
   FLICKR_OAUTH_REQUEST_TOKEN_SECURE=(END_POINT_SECURE + '/oauth/request_token').freeze
   FLICKR_OAUTH_AUTHORIZE_SECURE=(END_POINT_SECURE + '/oauth/authorize').freeze
   FLICKR_OAUTH_ACCESS_TOKEN_SECURE=(END_POINT_SECURE + '/oauth/access_token').freeze
@@ -14,7 +14,7 @@ module FlickRaw
   REST_PATH=(END_POINT + '/rest/').freeze
   UPLOAD_PATH=(END_POINT + '/upload/').freeze
   REPLACE_PATH=(END_POINT + '/replace/').freeze
-  
+
   REST_PATH_SECURE=(END_POINT_SECURE + '/rest/').freeze
   UPLOAD_PATH_SECURE=(END_POINT_SECURE + '/upload/').freeze
   REPLACE_PATH_SECURE=(END_POINT_SECURE + '/replace/').freeze
@@ -30,10 +30,10 @@ module FlickRaw
   class Flickr < Request
     # Authenticated access token
     attr_accessor :access_token
-    
+
     # Authenticated access token secret
     attr_accessor :access_secret
-    
+
     def self.build(methods); methods.each { |m| build_request m } end
 
     def initialize(api_key: FlickRaw.api_key,
@@ -51,17 +51,17 @@ module FlickRaw
       @oauth_consumer.ca_path = FlickRaw.ca_path
       @oauth_consumer.user_agent = USER_AGENT
       @access_token = @access_secret = nil
-      
+
       Flickr.build(call('flickr.reflection.getMethods')) if Flickr.flickr_objects.empty?
       super self
     end
-    
+
     # This is the central method. It does the actual request to the flickr server.
     #
     # Raises FailedResponse if the response status is _failed_.
     def call(req, args={}, &block)
       oauth_args = args.delete(:oauth) || {}
-      rest_path = FlickRaw.secure ? REST_PATH_SECURE :  REST_PATH
+      rest_path = FlickRaw.secure ? REST_PATH_SECURE : REST_PATH
       http_response = @oauth_consumer.post_form(rest_path, @access_secret, {:oauth_token => @access_token}.merge(oauth_args), build_args(args, req))
       process_response(req, http_response.body)
     end
@@ -73,7 +73,7 @@ module FlickRaw
       flickr_oauth_request_token = FlickRaw.secure ? FLICKR_OAUTH_REQUEST_TOKEN_SECURE : FLICKR_OAUTH_REQUEST_TOKEN
       @oauth_consumer.request_token(flickr_oauth_request_token, args)
     end
-    
+
     # Get the oauth authorize url.
     #
     #  auth_url = flickr.get_authorize_url(token['oauth_token'], :perms => 'delete')
@@ -125,19 +125,19 @@ module FlickRaw
           code = response[/code="([^"]+)"/, 1]
           raise FailedResponse.new(msg, code, req)
         end
-        
+
         type = response[/<(\w+)/, 1]
         h = {
-          "secret" => response[/secret="([^"]+)"/, 1],
-          "originalsecret" => response[/originalsecret="([^"]+)"/, 1],
-          "_content" => response[/>([^<]+)<\//, 1]
+          'secret' => response[/secret="([^"]+)"/, 1],
+          'originalsecret' => response[/originalsecret="([^"]+)"/, 1],
+          '_content' => response[/>([^<]+)<\//, 1]
         }.delete_if {|k,v| v.nil? }
-        
+
         Response.build h, type
       else
-        json = JSON.load(response.empty? ? "{}" : response)
+        json = JSON.load(response.empty? ? '{}' : response)
         raise FailedResponse.new(json['message'], json['code'], req) if json.delete('stat') == 'fail'
-        type, json = json.to_a.first if json.size == 1 and json.all? {|k,v| v.is_a? Hash}
+        type, json = json.to_a.first if json.size == 1 and json.all? { |k,v| v.is_a? Hash }
 
         Response.build json, type
       end
@@ -152,7 +152,7 @@ module FlickRaw
         args['photo'] = open(file, 'rb')
         close_after = true
       end
-      
+
       http_response = @oauth_consumer.post_multipart(method, @access_secret, {:oauth_token => @access_token}.merge(oauth_args), args)
       args['photo'].close if close_after
       process_response(method, http_response.body)
@@ -162,26 +162,26 @@ module FlickRaw
   class << self
     # Your flickr API key, see http://www.flickr.com/services/api/keys for more information
     attr_accessor :api_key
-    
+
     # The shared secret of _api_key_, see http://www.flickr.com/services/api/keys for more information
     attr_accessor :shared_secret
-    
+
     # Use a proxy
     attr_accessor :proxy
-    
+
     # Use ssl connection
     attr_accessor :secure
 
     # Check the server certificate (ssl connection only)
     attr_accessor :check_certificate
-    
+
     # Set path of a CA certificate file in PEM format (ssl connection only)
     attr_accessor :ca_file
 
     # Set path to a directory of CA certificate files in PEM format (ssl connection only)
     attr_accessor :ca_path
 
-    BASE58_ALPHABET="123456789abcdefghijkmnopqrstuvwxyzABCDEFGHJKLMNPQRSTUVWXYZ".freeze
+    BASE58_ALPHABET='123456789abcdefghijkmnopqrstuvwxyzABCDEFGHJKLMNPQRSTUVWXYZ'.freeze
     def base58(id)
       id = id.to_i
       alphabet = BASE58_ALPHABET.split(//)
@@ -193,35 +193,37 @@ module FlickRaw
       r
     end
 
-    def url(r);   PHOTO_SOURCE_URL % [r.farm, r.server, r.id, r.secret, "",   "jpg"]   end
-    def url_m(r); PHOTO_SOURCE_URL % [r.farm, r.server, r.id, r.secret, "_m", "jpg"] end
-    def url_s(r); PHOTO_SOURCE_URL % [r.farm, r.server, r.id, r.secret, "_s", "jpg"] end
-    def url_t(r); PHOTO_SOURCE_URL % [r.farm, r.server, r.id, r.secret, "_t", "jpg"] end
-    def url_b(r); PHOTO_SOURCE_URL % [r.farm, r.server, r.id, r.secret, "_b", "jpg"] end
-    def url_z(r); PHOTO_SOURCE_URL % [r.farm, r.server, r.id, r.secret, "_z", "jpg"] end
-    def url_q(r); PHOTO_SOURCE_URL % [r.farm, r.server, r.id, r.secret, "_q", "jpg"] end
-    def url_n(r); PHOTO_SOURCE_URL % [r.farm, r.server, r.id, r.secret, "_n", "jpg"] end
-    def url_c(r); PHOTO_SOURCE_URL % [r.farm, r.server, r.id, r.secret, "_c", "jpg"] end
-    def url_o(r); PHOTO_SOURCE_URL % [r.farm, r.server, r.id, r.originalsecret, "_o", r.originalformat] end
-    def url_profile(r); URL_PROFILE + (r.owner.respond_to?(:nsid) ? r.owner.nsid : r.owner) + "/" end
+    def url(r);   PHOTO_SOURCE_URL % [r.farm, r.server, r.id, r.secret, '',   'jpg'] end
+    def url_m(r); PHOTO_SOURCE_URL % [r.farm, r.server, r.id, r.secret, '_m', 'jpg'] end
+    def url_s(r); PHOTO_SOURCE_URL % [r.farm, r.server, r.id, r.secret, '_s', 'jpg'] end
+    def url_t(r); PHOTO_SOURCE_URL % [r.farm, r.server, r.id, r.secret, '_t', 'jpg'] end
+    def url_b(r); PHOTO_SOURCE_URL % [r.farm, r.server, r.id, r.secret, '_b', 'jpg'] end
+    def url_z(r); PHOTO_SOURCE_URL % [r.farm, r.server, r.id, r.secret, '_z', 'jpg'] end
+    def url_q(r); PHOTO_SOURCE_URL % [r.farm, r.server, r.id, r.secret, '_q', 'jpg'] end
+    def url_n(r); PHOTO_SOURCE_URL % [r.farm, r.server, r.id, r.secret, '_n', 'jpg'] end
+    def url_c(r); PHOTO_SOURCE_URL % [r.farm, r.server, r.id, r.secret, '_c', 'jpg'] end
+    def url_h(r); PHOTO_SOURCE_URL % [r.farm, r.server, r.id, r.secret, '_h', 'jpg'] end
+    def url_k(r); PHOTO_SOURCE_URL % [r.farm, r.server, r.id, r.secret, '_k', 'jpg'] end
+    def url_o(r); PHOTO_SOURCE_URL % [r.farm, r.server, r.id, r.originalsecret, '_o', r.originalformat] end
+    def url_profile(r); URL_PROFILE + (r.owner.respond_to?(:nsid) ? r.owner.nsid : r.owner) + '/' end
     def url_photopage(r); url_photostream(r) + r.id end
-    def url_photosets(r); url_photostream(r) + "sets/" end
+    def url_photosets(r); url_photostream(r) + 'sets/' end
     def url_photoset(r); url_photosets(r) + r.id end
     def url_short(r); URL_SHORT + base58(r.id) end
-    def url_short_m(r); URL_SHORT + "img/" + base58(r.id) + "_m.jpg" end
-    def url_short_s(r); URL_SHORT + "img/" + base58(r.id) + ".jpg" end
-    def url_short_t(r); URL_SHORT + "img/" + base58(r.id) + "_t.jpg" end
-    def url_short_q(r); URL_SHORT + "img/" + base58(r.id) + "_q.jpg" end
-    def url_short_n(r); URL_SHORT + "img/" + base58(r.id) + "_n.jpg" end
+    def url_short_m(r); URL_SHORT + 'img/' + base58(r.id) + '_m.jpg' end
+    def url_short_s(r); URL_SHORT + 'img/' + base58(r.id) + '.jpg' end
+    def url_short_t(r); URL_SHORT + 'img/' + base58(r.id) + '_t.jpg' end
+    def url_short_q(r); URL_SHORT + 'img/' + base58(r.id) + '_q.jpg' end
+    def url_short_n(r); URL_SHORT + 'img/' + base58(r.id) + '_n.jpg' end
     def url_photostream(r)
       URL_PHOTOSTREAM +
-        if r.respond_to?(:pathalias) and r.pathalias
+        if r.respond_to?(:pathalias) && r.pathalias
           r.pathalias
         elsif r.owner.respond_to?(:nsid)
           r.owner.nsid
         else
           r.owner
-        end + "/"
+        end + '/'
     end
   end
 end

data/lib/flickraw/oauth.rb

@@ -13,78 +13,78 @@ module FlickRaw
 
     class << self
       def encode_value(v)
-        v = v.to_s.encode("utf-8").force_encoding("ascii-8bit") if RUBY_VERSION >= "1.9"
+        v = v.to_s.encode('utf-8').force_encoding('ascii-8bit') if RUBY_VERSION >= '1.9'
         v.to_s
       end
 
       def escape(s)
-        encode_value(s).gsub(/[^a-zA-Z0-9\-\.\_\~]/) { |special|
-          special.unpack("C*").map{|i| sprintf("%%%02X", i) }.join
-        }
+        encode_value(s).gsub(/[^a-zA-Z0-9\-\.\_\~]/) do |special|
+          special.unpack("C*").map { |i| sprintf("%%%02X", i) }.join
+        end
       end
 
-      def parse_response(text); Hash[text.split("&").map {|s| s.split("=")}] end
+      def parse_response(text); Hash[text.split('&').map { |s| s.split('=') }] end
 
       def signature_base_string(method, url, params)
-        params_norm = params.map {|k,v| escape(k) + "=" + escape(v)}.sort.join("&")
-        method.to_s.upcase + "&" + escape(url) + "&" + escape(params_norm)
+        params_norm = params.map { |k,v| "#{escape(k)}=#{escape(v)}" }.sort.join('&')
+        "#{method.to_s.upcase}&#{escape(url)}&#{escape(params_norm)}"
       end
-      
+
       def sign_plaintext(method, url, params, token_secret, consumer_secret)
-        escape(consumer_secret) + "&" + escape(token_secret)
+        "#{escape(consumer_secret)}&#{escape(token_secret)}"
       end
-        
+
       def sign_rsa_sha1(method, url, params, token_secret, consumer_secret)
         text = signature_base_string(method, url, params)
         key = OpenSSL::PKey::RSA.new(consumer_secret)
         digest = OpenSSL::Digest::SHA1.new
         [key.sign(digest, text)].pack('m0').gsub(/\n$/,'')
       end
-            
+
       def sign_hmac_sha1(method, url, params, token_secret, consumer_secret)
         text = signature_base_string(method, url, params)
-        key = escape(consumer_secret) + "&" + escape(token_secret)
+        key = "#{escape(consumer_secret)}&#{escape(token_secret)}"
         digest = OpenSSL::Digest::SHA1.new
         [OpenSSL::HMAC.digest(digest, key, text)].pack('m0').gsub(/\n$/,'')
       end
-    
+
       def gen_timestamp; Time.now.to_i end
       def gen_nonce; [OpenSSL::Random.random_bytes(32)].pack('m0').gsub(/\n$/,'') end
       def gen_default_params
-        { :oauth_version => "1.0", :oauth_signature_method => "HMAC-SHA1",
+        { :oauth_version => "1.0", :oauth_signature_method => 'HMAC-SHA1',
           :oauth_nonce => gen_nonce, :oauth_timestamp => gen_timestamp }
       end
-    
+
       def authorization_header(url, params)
-        params_norm = params.map {|k,v| %(#{escape(k)}="#{escape(v)}")}.sort.join(", ")
+        params_norm = params.map { |k, v| %(#{escape(k)}="#{escape(v)}")}.sort.join(', ')
         %(OAuth realm="#{url.to_s}", #{params_norm})
       end
     end
-    
+
     attr_accessor :user_agent
     attr_reader :proxy
     attr_accessor :check_certificate
     attr_accessor :ca_file
     attr_accessor :ca_path
     def proxy=(url); @proxy = URI.parse(url || '') end
-    
+
     def initialize(consumer_key, consumer_secret)
       @consumer_key, @consumer_secret = consumer_key, consumer_secret
       self.proxy = nil
     end
 
     def request_token(url, oauth_params = {})
-      r = post_form(url, nil, {:oauth_callback => "oob"}.merge(oauth_params))
+      r = post_form(url, nil, {:oauth_callback => 'oob'}.merge(oauth_params))
       OAuthClient.parse_response(r.body)
     end
-    
+
     def authorize_url(url, oauth_params = {})
-      params_norm = oauth_params.map {|k,v| OAuthClient.escape(k) + "=" + OAuthClient.escape(v)}.sort.join("&")
-      url =  URI.parse(url)
-      url.query = url.query ? url.query + "&" + params_norm : params_norm
+      params_norm = oauth_params.map { |k,v| "#{OAuthClient.escape(k)}=#{OAuthClient.escape(v)}" }.sort.join('&')
+      url = URI.parse(url)
+      url.query = url.query ? "#{url.query}&#{params_norm}" : params_norm
       url.to_s
     end
-    
+
     def access_token(url, token_secret, oauth_params = {})
       r = post_form(url, token_secret, oauth_params)
       OAuthClient.parse_response(r.body)
@@ -92,32 +92,32 @@ module FlickRaw
 
     def post_form(url, token_secret, oauth_params = {}, params = {})
       encoded_params = Hash[*params.map {|k,v| [OAuthClient.encode_value(k), OAuthClient.encode_value(v)]}.flatten]
-      post(url, token_secret, oauth_params, params) {|request| request.form_data = encoded_params}
+      post(url, token_secret, oauth_params, params) { |request| request.form_data = encoded_params }
     end
-    
+
     def post_multipart(url, token_secret, oauth_params = {}, params = {})
       post(url, token_secret, oauth_params, params) {|request|
         boundary = "FlickRaw#{OAuthClient.gen_nonce}"
         request['Content-type'] = "multipart/form-data, boundary=#{boundary}"
 
         request.body = ''
-        params.each { |k, v|
+        params.each do |k, v|
           if v.respond_to? :read
             basename = File.basename(v.path.to_s) if v.respond_to? :path
             basename ||= File.basename(v.base_uri.to_s) if v.respond_to? :base_uri
             basename ||= "unknown"
             request.body << "--#{boundary}\r\n" <<
-              "Content-Disposition: form-data; name=\"#{OAuthClient.encode_value(k)}\"; filename=\"#{OAuthClient.encode_value(basename)}\"\r\n" <<
-              "Content-Transfer-Encoding: binary\r\n" <<
-              "Content-Type: image/jpeg\r\n\r\n" <<
-              v.read << "\r\n"
+            "Content-Disposition: form-data; name=\"#{OAuthClient.encode_value(k)}\"; filename=\"#{OAuthClient.encode_value(basename)}\"\r\n" <<
+            "Content-Transfer-Encoding: binary\r\n" <<
+            "Content-Type: image/jpeg\r\n\r\n" <<
+            v.read << "\r\n"
           else
             request.body << "--#{boundary}\r\n" <<
-              "Content-Disposition: form-data; name=\"#{OAuthClient.encode_value(k)}\"\r\n\r\n" <<
-              "#{OAuthClient.encode_value(v)}\r\n"
+            "Content-Disposition: form-data; name=\"#{OAuthClient.encode_value(k)}\"\r\n\r\n" <<
+            "#{OAuthClient.encode_value(v)}\r\n"
           end
-        }
-        
+        end
+
         request.body << "--#{boundary}--"
       }
     end
@@ -125,11 +125,11 @@ module FlickRaw
     private
     def sign(method, url, params, token_secret = nil)
       case params[:oauth_signature_method]
-      when "HMAC-SHA1"
+      when 'HMAC-SHA1'
         OAuthClient.sign_hmac_sha1(method, url, params, token_secret, @consumer_secret)
-      when "RSA-SHA1"
+      when 'RSA-SHA1'
         OAuthClient.sign_rsa_sha1(method, url, params, token_secret, @consumer_secret)
-      when "PLAINTEXT"
+      when 'PLAINTEXT'
         OAuthClient.sign_plaintext(method, url, params, token_secret, @consumer_secret)
       else
         raise UnknownSignatureMethod, params[:oauth_signature_method]
@@ -140,9 +140,9 @@ module FlickRaw
       url = URI.parse(url)
       default_oauth_params = OAuthClient.gen_default_params
       default_oauth_params[:oauth_consumer_key] = @consumer_key
-      default_oauth_params[:oauth_signature_method] = "PLAINTEXT" if url.scheme == 'https'
+      default_oauth_params[:oauth_signature_method] = 'PLAINTEXT' if url.scheme == 'https'
       oauth_params = default_oauth_params.merge(oauth_params)
-      params_signed = params.reject {|k,v| v.respond_to? :read}.merge(oauth_params)
+      params_signed = params.reject { |k,v| v.respond_to? :read }.merge(oauth_params)
       oauth_params[:oauth_signature] = sign(:post, url, params_signed, token_secret)
 
       http = Net::HTTP.new(url.host, url.port, @proxy.host, @proxy.port, @proxy.user, @proxy.password)
@@ -150,18 +150,17 @@ module FlickRaw
       http.verify_mode = (@check_certificate ? OpenSSL::SSL::VERIFY_PEER : OpenSSL::SSL::VERIFY_NONE)
       http.ca_file = @ca_file
       http.ca_path = @ca_path
-      r = http.start {|agent|
+      r = http.start do |agent|
         request = Net::HTTP::Post.new(url.path)
         request['User-Agent'] = @user_agent if @user_agent
         request['Authorization'] = OAuthClient.authorization_header(url, oauth_params)
 
         yield request
         agent.request(request)
-      }
-      
+      end
+
       raise FailedResponse.new(r.body) if r.is_a? Net::HTTPClientError
       r
     end
   end
-
 end

data/lib/flickraw/request.rb

@@ -1,50 +1,49 @@
 module FlickRaw
-class Request
-  def initialize(flickr = nil) # :nodoc:
-    @flickr = flickr
-
-    self.class.flickr_objects.each {|name|
-      klass = self.class.const_get name.capitalize
-      instance_variable_set "@#{name}", klass.new(@flickr)
-    }
-  end
+  class Request
+    def initialize(flickr = nil) # :nodoc:
+      @flickr = flickr
+
+      self.class.flickr_objects.each {|name|
+        klass = self.class.const_get name.capitalize
+        instance_variable_set "@#{name}", klass.new(@flickr)
+      }
+    end
 
-  def self.build_request(req) # :nodoc:
-    method_nesting = req.split '.'
-    raise "'#{@name}' : Method name mismatch" if method_nesting.shift != request_name.split('.').last
+    def self.build_request(req) # :nodoc:
+      method_nesting = req.split '.'
+      raise "'#{@name}' : Method name mismatch" if method_nesting.shift != request_name.split('.').last
+
+      if method_nesting.size > 1
+        name = method_nesting.first
+        class_name = name.capitalize
+        if flickr_objects.include? name
+          klass = const_get(class_name)
+        else
+          klass = Class.new Request
+          const_set(class_name, klass)
+          attr_reader name
+          flickr_objects << name
+        end
 
-    if method_nesting.size > 1
-      name = method_nesting.first
-      class_name = name.capitalize
-      if flickr_objects.include? name
-        klass = const_get(class_name)
+        klass.build_request method_nesting.join('.')
       else
-        klass = Class.new Request
-        const_set(class_name, klass)
-        attr_reader name
-        flickr_objects << name
-      end
-
-      klass.build_request method_nesting.join('.')
-    else
-      req = method_nesting.first
-      module_eval %{
+        req = method_nesting.first
+        module_eval %{
         def #{req}(*args, &block)
           @flickr.call("#{request_name}.#{req}", *args, &block)
         end
-      } if Util.safe_for_eval?(req)
-      flickr_methods << req
+        } if Util.safe_for_eval?(req)
+        flickr_methods << req
+      end
     end
-  end
 
-  # List the flickr subobjects of this object
-  def self.flickr_objects; @flickr_objects ||= [] end
+    # List the flickr subobjects of this object
+    def self.flickr_objects; @flickr_objects ||= [] end
 
-  # List the flickr methods of this object
-  def self.flickr_methods; @flickr_methods ||= [] end
-
-  # Returns the prefix of the request corresponding to this class.
-  def self.request_name; name.downcase.gsub(/::/, '.').sub(/[^\.]+\./, '') end
-end
+    # List the flickr methods of this object
+    def self.flickr_methods; @flickr_methods ||= [] end
 
+    # Returns the prefix of the request corresponding to this class.
+    def self.request_name; name.downcase.gsub(/::/, '.').sub(/[^\.]+\./, '') end
+  end
 end

data/lib/flickraw/response.rb

@@ -1,58 +1,62 @@
 module FlickRaw
-class Response
-  def self.build(h, type) # :nodoc:
-    if h.is_a? Response
-      h
-    elsif type =~ /s$/ and (a = h[$`]).is_a? Array
-      ResponseList.new(h, type, a.collect {|e| Response.build(e, $`)})
-    elsif h.keys == ["_content"]
-      h["_content"]
-    else
-      Response.new(h, type)
+  class Response
+    def self.build(h, type) # :nodoc:
+      if h.is_a? Response
+        h
+      elsif type =~ /s$/ and (a = h[$`]).is_a? Array
+        ResponseList.new(h, type, a.collect { |e| Response.build(e, $`) })
+      elsif h.keys == ['_content']
+        h['_content']
+      else
+        Response.new(h, type)
+      end
     end
-  end
 
-  attr_reader :flickr_type
-  def initialize(h, type) # :nodoc:
-    @flickr_type, @h = type, {}
-    methods = "class << self;"
-    h.each {|k,v|
-      @h[k] = case v
-        when Hash  then Response.build(v, k)
-        when Array then v.collect {|e| Response.build(e, k)}
-        else v
+    attr_reader :flickr_type
+    def initialize(h, type) # :nodoc:
+      @flickr_type, @h = type, {}
+      methods = 'class << self;'
+      h.each do |k, v|
+        @h[k] = case v
+                when Hash  then Response.build(v, k)
+                when Array then v.collect { |e| Response.build(e, k) }
+                else v
+                end
+        methods << "def #{k}; @h['#{k}'] end;" if Util.safe_for_eval?(k)
       end
-      methods << "def #{k}; @h['#{k}'] end;" if Util.safe_for_eval?(k)
-    }
-    eval methods << "end"
+      eval methods << 'end'
+    end
+    def [](k); @h[k] end
+    def to_s; @h['_content'] || super end
+    def inspect; @h.inspect end
+    def to_hash; @h end
+    def marshal_dump; [@h, @flickr_type] end
+    def marshal_load(data); initialize(*data) end
   end
-  def [](k); @h[k] end
-  def to_s; @h["_content"] || super end
-  def inspect; @h.inspect end
-  def to_hash; @h end
-  def marshal_dump; [@h, @flickr_type] end
-  def marshal_load(data); initialize(*data) end
-end
-
-class ResponseList < Response
-  include Enumerable
-  def initialize(h, t, a); super(h, t); @a = a end
-  def [](k); k.is_a?(Fixnum) ? @a[k] : super(k) end
-  def each; @a.each{|e| yield e} end
-  def to_a; @a end
-  def inspect; @a.inspect end
-  def size; @a.size end
-  def marshal_dump; [@h, @flickr_type, @a] end
-  alias length size
-end
 
-class FailedResponse < Error
-  attr_reader :code
-  alias :msg :message
-  def initialize(msg, code, req)
-    @code = code
-    super("'#{req}' - #{msg}")
+  class ResponseList < Response
+    include Enumerable
+    def initialize(h, t, a); super(h, t); @a = a end
+    def [](k); k.is_a?(Integer) ? @a[k] : super(k) end
+    def each
+       @a.each { |e| yield e }
+    end
+    def to_a; @a end
+    def inspect; @a.inspect end
+    def size; @a.size end
+    def map!
+      @a = @a.map { |e| yield e }
+    end
+    def marshal_dump; [@h, @flickr_type, @a] end
+    alias length size
   end
-end
 
+  class FailedResponse < Error
+    attr_reader :code
+    alias :msg :message
+    def initialize(msg, code, req)
+      @code = code
+      super("'#{req}' - #{msg}")
+    end
+  end
 end

data/lib/flickraw/util.rb

@@ -9,6 +9,5 @@ module FlickRaw
     def safe_for_eval?(string)
       string == sanitize(string)
     end
-
   end
 end

data/test/test_request.rb

@@ -11,6 +11,7 @@ class TestReqeust < Test::Unit::TestCase
   def test_flickr_api_is_accessible_via_methods
     # Reset FlickRaw (was initialized in test/helper.rb) so the request methods
     # are properly built
+    old_flickraw = $flickraw
     $flickraw = nil
     FlickRaw::Request.instance_variable_set(:@flickr_objects, nil)
 
@@ -18,6 +19,9 @@ class TestReqeust < Test::Unit::TestCase
 
     assert_equal true, flickr.methods.include?(:fully)
     assert_equal true, flickr.fully.methods.include?(:legal)
+    
+    # Fix for failing subsequent tests
+    $flickraw = old_flickraw
   end
 
   def test_invalid_keys_are_skipped

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: flickraw
 version: !ruby/object:Gem::Version
-  version: 0.9.9
+  version: 0.9.10
 platform: ruby
 authors:
 - Mael Clerambault
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2016-06-30 00:00:00.000000000 Z
+date: 2019-03-13 00:00:00.000000000 Z
 dependencies: []
 description: 
 email: mael@clerambault.fr
@@ -16,28 +16,28 @@ executables: []
 extensions: []
 extra_rdoc_files: []
 files:
+- LICENSE
+- README.rdoc
 - examples/auth.rb
 - examples/interestingness.rb
 - examples/search.rb
 - examples/sinatra.rb
 - examples/upload.rb
 - examples/web_oauth.rb
-- test/helper.rb
-- test/test.rb
-- test/test_request.rb
-- test/test_response.rb
-- test/test_upload.rb
+- flickraw_rdoc.rb
+- lib/flickraw.rb
 - lib/flickraw/api.rb
 - lib/flickraw/error.rb
 - lib/flickraw/oauth.rb
 - lib/flickraw/request.rb
 - lib/flickraw/response.rb
 - lib/flickraw/util.rb
-- lib/flickraw.rb
-- flickraw_rdoc.rb
-- LICENSE
-- README.rdoc
 - rakefile
+- test/helper.rb
+- test/test.rb
+- test/test_request.rb
+- test/test_response.rb
+- test/test_upload.rb
 homepage: http://hanklords.github.com/flickraw/
 licenses:
 - MIT
@@ -57,8 +57,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubyforge_project: 
-rubygems_version: 2.0.14
+rubygems_version: 3.0.2
 signing_key: 
 specification_version: 4
 summary: Flickr library with a syntax close to the syntax described on http://www.flickr.com/services/api