om 3.0.0 → 3.0.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 7b79d1d1f65ee425169a4ccbf20bf083ce6b7194
-  data.tar.gz: 0f4bd140fffda33f93e14bc2a9ddc21da17e186c
+  metadata.gz: ad2b503450c856d7a7e3dbd55bab9545419c04f9
+  data.tar.gz: 2c97e1ce0d2f2b2524e1cca742726d50a2fc814f
 SHA512:
-  metadata.gz: 7042ec3c299b99a8cce474667d3bf88a1a66b2f79c401d78f0b27c57562e2929e1332f3200b35bb12376688ba2f9dfb648cc335449653407aff83f6328233624
-  data.tar.gz: 8b6c21a6e6b5511ababcb568ab9af7e895704dfd643d14f893ebacb2a404d212a7e976ebba86ea5f1445e703e5f17a64f2ffa0d7f1b1bfefaddb7951aedf7bc0
+  metadata.gz: b9cf738d9afa3d3c08d1e2a2cb349b2c6a13b7dfc42b7400384094f3fed29f324cdf83cf1511e69452c24fb07f937a817b9759fdbbb19a2cda6b30b3bc1e16fc
+  data.tar.gz: 879531480232ee5d238745cba12be106c0b3566b38827053d930afbdcfffe78f88738159b7f63753057bb6e0a03b6e0dae2a76c7f22f79a5c2684a55b1eb31c7

data/History.textile

@@ -1,3 +1,7 @@
+h3. 3.0.1 (25 Jun 2013)
+  Fix bug where values that were the same as the existing values were 
+    removed from the update list
+
 h3. 3.0.0 (20 Jun 2013)
   Return an array instead of a hash Term#update_values
   When passing an array to Term#update_values, it will overwrite all of

data/lib/om/version.rb

@@ -1,3 +1,3 @@
 module Om
-  VERSION = "3.0.0"
+  VERSION = "3.0.1"
 end

data/lib/om/xml/term_value_operators.rb

@@ -89,13 +89,6 @@ module OM::XML::TermValueOperators
       # Populate the response hash appropriately, using hierarchical names for terms as keys rather than the given pointers.
       result = new_values.dup
       
-      # Skip any submitted values if the new value matches the current values
-      new_values.each_with_index do |val, index|
-        if !val.nil? && current_values[index] == val
-          new_values.delete_at(index)
-        end
-      end 
-
       # Fill out the pointer completely if the final term is a NamedTermProxy
       if term.kind_of? OM::XML::NamedTermProxy
         pointer.pop

data/spec/unit/term_value_operators_spec.rb

@@ -92,6 +92,12 @@ describe "OM::XML::TermValueOperators" do
       @article.update_values( { [{"person"=>"0"}, "role"]=>"the role" }).should == expected_result
       @article.update_values( { [{:person=>0}, :role]=>"the role" }).should == expected_result
     end
+
+    it "should replace stuff with the same value (in this case 'one')" do
+      @article.update_values( { [{:person=>0}, :role]=>["one"] })
+      @article.update_values( { [{:person=>0}, :role]=>["one", "two"] })
+      @article.term_values( {:person=>0}, :role).should == ["one", "two"]
+    end
     
     it "should traverse named term proxies transparently" do
       @article.term_values( :journal, :issue, :start_page).should_not == ["108"]      

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: om
 version: !ruby/object:Gem::Version
-  version: 3.0.0
+  version: 3.0.1
 platform: ruby
 authors:
 - Matt Zumwalt
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-06-21 00:00:00.000000000 Z
+date: 2013-06-26 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activesupport