solrizer 1.0.2 → 1.0.3

data/History.txt

@@ -1,3 +1,7 @@
+h2. 1.0.3
+
+BUG: Extractor.format_field_values was choking when value was nil
+
 h2. 1.0.0
 
 Deprecated

data/VERSION

@@ -1 +1 @@
-1.0.2
+1.0.3

data/lib/solrizer/extractor.rb

@@ -28,9 +28,16 @@ class Extractor
   end
   
   # Strips the majority of whitespace from the values array and then joins them with a single blank delimitter
-  # @param [Array] values Array of strings representing the values returned 
+  # Returns an empty string if values argument is nil
+  #
+  # @param [Array] values Array of strings representing the values to be formatted
+  # @return [String] 
   def self.format_node_value values
-    values.map{|val| val.gsub(/\s+/,' ').strip}.join(" ")
+    if values.nil?
+      return ""
+    else
+      return values.map{|val| val.gsub(/\s+/,' ').strip}.join(" ")
+    end
   end
   
   # Instance Methods

data/solrizer.gemspec

@@ -5,11 +5,11 @@
 
 Gem::Specification.new do |s|
   s.name = %q{solrizer}
-  s.version = "1.0.2"
+  s.version = "1.0.3"
 
   s.required_rubygems_version = Gem::Requirement.new(">= 0") if s.respond_to? :required_rubygems_version=
   s.authors = ["Matt Zumwalt"]
-  s.date = %q{2011-03-31}
+  s.date = %q{2011-04-18}
   s.description = %q{Use solrizer to populate solr indexes from Fedora repository content or from other sources.  You can run solrizer from within your apps, using the provided rake tasks, or as a JMS listener}
   s.email = %q{matt.zumwalt@yourmediashelf.com}
   s.executables = ["solrizer", "solrizerd"]

data/spec/units/extractor_spec.rb

@@ -12,6 +12,9 @@ describe Solrizer::Extractor do
       Solrizer::Extractor.format_node_value([" test    \n   node    \t value \t"]).should == "test node value"
       Solrizer::Extractor.format_node_value([" test ", "     \n   node ", "   \t value \t"]).should == "test node value"
     end
+    it "should return an empty string if given an argument of nil" do
+      Solrizer::Extractor.format_node_value(nil).should == ""
+    end
   end
   
 end

metadata

@@ -1,13 +1,13 @@
 --- !ruby/object:Gem::Specification 
 name: solrizer
 version: !ruby/object:Gem::Version 
-  hash: 19
+  hash: 17
   prerelease: false
   segments: 
   - 1
   - 0
-  - 2
-  version: 1.0.2
+  - 3
+  version: 1.0.3
 platform: ruby
 authors: 
 - Matt Zumwalt
@@ -15,12 +15,12 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2011-03-31 00:00:00 -04:00
+date: 2011-04-18 00:00:00 -04:00
 default_executable: 
 dependencies: 
 - !ruby/object:Gem::Dependency 
-  prerelease: false
   type: :runtime
+  prerelease: false
   name: nokogiri
   version_requirements: &id001 !ruby/object:Gem::Requirement 
     none: false
@@ -33,8 +33,8 @@ dependencies:
         version: "0"
   requirement: *id001
 - !ruby/object:Gem::Dependency 
-  prerelease: false
   type: :runtime
+  prerelease: false
   name: xml-simple
   version_requirements: &id002 !ruby/object:Gem::Requirement 
     none: false
@@ -47,8 +47,8 @@ dependencies:
         version: "0"
   requirement: *id002
 - !ruby/object:Gem::Dependency 
-  prerelease: false
   type: :runtime
+  prerelease: false
   name: om
   version_requirements: &id003 !ruby/object:Gem::Requirement 
     none: false
@@ -63,8 +63,8 @@ dependencies:
         version: 1.0.0
   requirement: *id003
 - !ruby/object:Gem::Dependency 
-  prerelease: false
   type: :runtime
+  prerelease: false
   name: mediashelf-loggable
   version_requirements: &id004 !ruby/object:Gem::Requirement 
     none: false
@@ -77,8 +77,8 @@ dependencies:
         version: "0"
   requirement: *id004
 - !ruby/object:Gem::Dependency 
-  prerelease: false
   type: :runtime
+  prerelease: false
   name: stomp
   version_requirements: &id005 !ruby/object:Gem::Requirement 
     none: false
@@ -91,8 +91,8 @@ dependencies:
         version: "0"
   requirement: *id005
 - !ruby/object:Gem::Dependency 
-  prerelease: false
   type: :runtime
+  prerelease: false
   name: daemons
   version_requirements: &id006 !ruby/object:Gem::Requirement 
     none: false
@@ -105,8 +105,8 @@ dependencies:
         version: "0"
   requirement: *id006
 - !ruby/object:Gem::Dependency 
-  prerelease: false
   type: :development
+  prerelease: false
   name: jeweler
   version_requirements: &id007 !ruby/object:Gem::Requirement 
     none: false
@@ -119,8 +119,8 @@ dependencies:
         version: "0"
   requirement: *id007
 - !ruby/object:Gem::Dependency 
-  prerelease: false
   type: :development
+  prerelease: false
   name: ruby-debug
   version_requirements: &id008 !ruby/object:Gem::Requirement 
     none: false
@@ -133,8 +133,8 @@ dependencies:
         version: "0"
   requirement: *id008
 - !ruby/object:Gem::Dependency 
-  prerelease: false
   type: :development
+  prerelease: false
   name: ruby-debug-base
   version_requirements: &id009 !ruby/object:Gem::Requirement 
     none: false
@@ -147,8 +147,8 @@ dependencies:
         version: "0"
   requirement: *id009
 - !ruby/object:Gem::Dependency 
-  prerelease: false
   type: :development
+  prerelease: false
   name: rspec
   version_requirements: &id010 !ruby/object:Gem::Requirement 
     none: false
@@ -163,8 +163,8 @@ dependencies:
         version: 2.0.0
   requirement: *id010
 - !ruby/object:Gem::Dependency 
-  prerelease: false
   type: :development
+  prerelease: false
   name: mocha
   version_requirements: &id011 !ruby/object:Gem::Requirement 
     none: false
@@ -177,8 +177,8 @@ dependencies:
         version: "0"
   requirement: *id011
 - !ruby/object:Gem::Dependency 
-  prerelease: false
   type: :runtime
+  prerelease: false
   name: nokogiri
   version_requirements: &id012 !ruby/object:Gem::Requirement 
     none: false
@@ -191,8 +191,8 @@ dependencies:
         version: "0"
   requirement: *id012
 - !ruby/object:Gem::Dependency 
-  prerelease: false
   type: :runtime
+  prerelease: false
   name: om
   version_requirements: &id013 !ruby/object:Gem::Requirement 
     none: false
@@ -205,8 +205,8 @@ dependencies:
         version: "0"
   requirement: *id013
 - !ruby/object:Gem::Dependency 
-  prerelease: false
   type: :runtime
+  prerelease: false
   name: nokogiri
   version_requirements: &id014 !ruby/object:Gem::Requirement 
     none: false
@@ -219,8 +219,8 @@ dependencies:
         version: "0"
   requirement: *id014
 - !ruby/object:Gem::Dependency 
-  prerelease: false
   type: :runtime
+  prerelease: false
   name: mediashelf-loggable
   version_requirements: &id015 !ruby/object:Gem::Requirement 
     none: false
@@ -233,8 +233,8 @@ dependencies:
         version: "0"
   requirement: *id015
 - !ruby/object:Gem::Dependency 
-  prerelease: false
   type: :development
+  prerelease: false
   name: jeweler
   version_requirements: &id016 !ruby/object:Gem::Requirement 
     none: false
@@ -247,8 +247,8 @@ dependencies:
         version: "0"
   requirement: *id016
 - !ruby/object:Gem::Dependency 
-  prerelease: false
   type: :development
+  prerelease: false
   name: ruby-debug
   version_requirements: &id017 !ruby/object:Gem::Requirement 
     none: false
@@ -261,8 +261,8 @@ dependencies:
         version: "0"
   requirement: *id017
 - !ruby/object:Gem::Dependency 
-  prerelease: false
   type: :development
+  prerelease: false
   name: ruby-debug-base
   version_requirements: &id018 !ruby/object:Gem::Requirement 
     none: false
@@ -275,8 +275,8 @@ dependencies:
         version: "0"
   requirement: *id018
 - !ruby/object:Gem::Dependency 
-  prerelease: false
   type: :development
+  prerelease: false
   name: rspec
   version_requirements: &id019 !ruby/object:Gem::Requirement 
     none: false
@@ -291,8 +291,8 @@ dependencies:
         version: 2.0.0
   requirement: *id019
 - !ruby/object:Gem::Dependency 
-  prerelease: false
   type: :development
+  prerelease: false
   name: mocha
   version_requirements: &id020 !ruby/object:Gem::Requirement 
     none: false