simple_scrobbler 0.1.0 → 0.1.1

data/lib/simple_scrobbler/version.rb

@@ -1,3 +1,3 @@
 class SimpleScrobbler
-  VERSION = "0.1.0"
+  VERSION = "0.1.1"
 end

data/lib/simple_scrobbler.rb

@@ -81,6 +81,7 @@ class SimpleScrobbler
   # :mb_trackid   :: MusicBrainz Track ID
   #
   def submit(artist, track, options={})
+    enforce_keys options, :time, :length, :album, :track_number, :mb_trackid
     if @source == "P" && !options[:length]
       raise DataError, "Track length must be specified if source is P"
     end
@@ -110,6 +111,7 @@ class SimpleScrobbler
   # :mb_trackid   :: MusicBrainz Track ID
   #
   def now_playing(artist, track, options={})
+    enforce_keys options, :length, :album, :track_number, :mb_trackid
     handshake
     parameters =
       generate_scrobbling_parameters(false,
@@ -163,16 +165,11 @@ private
     query_string = sort_parameters(parameters).
                    map{ |k, v| "#{k}=#{CGI.escape(v)}" }.
                    join("&")
-    Net::HTTP.get_response(
-      URI.parse(url + "?" + query_string)
-    ).body
+    Net::HTTP.get_response(URI.parse(url + "?" + query_string)).body
   end
 
   def post(url, parameters)
-    Net::HTTP.post_form(
-      URI.parse(url),
-      parameters
-    ).body
+    Net::HTTP.post_form(URI.parse(url), parameters).body
   end
 
   def md5(s)
@@ -215,4 +212,10 @@ private
       hash
     }
   end
+
+  def enforce_keys(hash, *keys)
+    if (extras = hash.keys - keys).any?
+      raise DataError, "Unknown hash key(s): #{extras.inspect}"
+    end
+  end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification 
 name: simple_scrobbler
 version: !ruby/object:Gem::Version 
-  version: 0.1.0
+  version: 0.1.1
 platform: ruby
 authors: 
 - Paul Battley
@@ -9,7 +9,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2010-07-10 00:00:00 +01:00
+date: 2010-07-11 00:00:00 +01:00
 default_executable: 
 dependencies: 
 - !ruby/object:Gem::Dependency