remote_table 0.2.9 → 0.2.10

data/VERSION

@@ -1 +1 @@
-0.2.9
+0.2.10

data/lib/remote_table.rb

@@ -1,12 +1,12 @@
 require 'digest/md5'
 require 'iconv'
+require 'uri'
+require 'tmpdir'
 require 'active_support'
 begin; require 'active_support/core_ext/object/blank'; rescue MissingSourceFile; end
 begin; require 'active_support/core_ext/string/inflections'; rescue MissingSourceFile; end
 begin; require 'active_support/core_ext/array/wrap'; rescue MissingSourceFile; end
 begin; require 'active_support/core_ext/hash/except'; rescue MissingSourceFile; end
-begin; require 'active_support/core_ext/string/starts_ends_with'; rescue MissingSourceFile; end
-require 'tempfile'
 require 'fastercsv'
 require 'slither'
 require 'roo'

data/lib/remote_table/request.rb

@@ -3,10 +3,10 @@ class RemoteTable
     attr_accessor :parsed_url, :post_data, :username, :password
     attr_accessor :form_data
     
-    # TODO: support post_data
     # TODO: support HTTP basic auth
     def initialize(bus)
-      @parsed_url = URI.parse(bus[:url]) or raise "need url"
+      raise(ArgumentError, "RemoteTable needs :url option") unless bus[:url].present?
+      @parsed_url = URI.parse bus[:url]
       if @parsed_url.host == 'spreadsheets.google.com' and (bus[:format].blank? or bus[:format].to_s == 'csv')
         @parsed_url.query = 'output=csv&' + @parsed_url.query.sub(/\&*output=.*(\&|\z)/, '')
       end
@@ -36,8 +36,7 @@ class RemoteTable
     
     def staging_dir_path    
       return @_staging_dir_path if @_staging_dir_path
-      @_staging_dir_path = Tempfile.open(parsed_url.to_s.gsub(/[^a-z0-9]+/i, '_')[0,100]).path
-      FileUtils.rm_f @_staging_dir_path
+      @_staging_dir_path = ::File.join Dir.tmpdir, rand.to_s
       FileUtils.mkdir @_staging_dir_path
       at_exit { FileUtils.rm_rf @_staging_dir_path }
       @_staging_dir_path

data/remote_table.gemspec

@@ -5,11 +5,11 @@
 
 Gem::Specification.new do |s|
   s.name = %q{remote_table}
-  s.version = "0.2.9"
+  s.version = "0.2.10"
 
   s.required_rubygems_version = Gem::Requirement.new(">= 0") if s.respond_to? :required_rubygems_version=
   s.authors = ["Seamus Abshere", "Andy Rossmeissl"]
-  s.date = %q{2010-04-15}
+  s.date = %q{2010-04-16}
   s.description = %q{Remotely open and parse Excel XLS, ODS, CSV and fixed-width tables.}
   s.email = %q{seamus@abshere.net}
   s.extra_rdoc_files = [

data/test/remote_table_test.rb

@@ -14,7 +14,7 @@ class FuelOilParser
     row.keys.grep(/(.+) Residual Fuel Oil/) do |location_column_name|
       first_part = $1
       next if (cost = row[location_column_name]).blank? or (date = row['Date']).blank?
-      if first_part.starts_with?('U.S.')
+      if first_part.start_with?('U.S.')
         locatable = "united_states (Country)"
       elsif first_part.include?('PADD')
         /\(PADD (.*)\)/.match(first_part)

metadata

@@ -5,8 +5,8 @@ version: !ruby/object:Gem::Version
   segments: 
   - 0
   - 2
-  - 9
-  version: 0.2.9
+  - 10
+  version: 0.2.10
 platform: ruby
 authors: 
 - Seamus Abshere
@@ -15,7 +15,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2010-04-15 00:00:00 -04:00
+date: 2010-04-16 00:00:00 -04:00
 default_executable: 
 dependencies: 
 - !ruby/object:Gem::Dependency