om 1.4.0 → 1.4.2

data/History.textile

@@ -1,3 +1,7 @@
+h3. 1.4.2
+  
+  "HYDRA-667":https://jira.duraspace.org/browse/HYDRA-667 Fixed bug where updating nodes wasn't marking the document as dirty
+
 h3. 1.4.0
 
   Added dynamic node access DSL. Added a warning when calling an index on a proxy term.

data/lib/om/version.rb

@@ -1,3 +1,3 @@
 module Om
-  VERSION = "1.4.0"
+  VERSION = "1.4.2"
 end

data/lib/om/xml/dynamic_node.rb

@@ -73,6 +73,7 @@ module OM
             @document.term_value_update(xpath, y.to_i, z)
           end
         end
+        @document.dirty = true
       end
 
       def sanitize_new_values(new_values)

data/lib/om/xml/term.rb

@@ -250,9 +250,9 @@ class OM::XML::Term
     if self.path.include?(":")
       ns_prefix = self.path[0..path.index(":")-1]
       path_name = self.path[path.index(":")+1..-1]
-      template = "xml[\"#{ns_prefix}\"].#{path_name}( #{OM::XML.delimited_list(node_options)} )" + node_child_template
+      template = "xml['#{ns_prefix}'].#{path_name}( #{OM::XML.delimited_list(node_options)} )" + node_child_template
     elsif !self.namespace_prefix.nil? and self.namespace_prefix != 'oxns'
-      template = "xml[\"#{self.namespace_prefix}\"].#{self.path}( #{OM::XML.delimited_list(node_options)} )" + node_child_template
+      template = "xml['#{self.namespace_prefix}'].#{self.path}( #{OM::XML.delimited_list(node_options)} )" + node_child_template
     else
       template = "xml.#{self.path}( #{OM::XML.delimited_list(node_options)} )" + node_child_template
     end

data/spec/unit/dynamic_node_spec.rb

@@ -67,6 +67,7 @@ describe "OM::XML::DynamicNode" do
     end
 
     it "should append nodes at the specified index if possible" do
+      @article.expects(:dirty=).with(true).twice
       @article.journal.title_info = ["all", "for", "the"]
       @article.journal.title_info(3, 'glory')
       @article.term_values(:journal, :title_info).should == ["all", "for", "the", "glory"]

data/spec/unit/term_spec.rb

@@ -173,9 +173,9 @@ describe "OM::XML::Term" do
     
     it "should work for namespaced nodes" do
       @ical_date = OM::XML::Term.new(:ical_date, :path=>"ical:date")
-      @ical_date.xml_builder_template.should == "xml[\"ical\"].date( '\#{builder_new_value}' )"
+      @ical_date.xml_builder_template.should == "xml[\'ical\'].date( '\#{builder_new_value}' )"
       @ical_date = OM::XML::Term.new(:ical_date, :path=>"date", :namespace_prefix=>"ical")
-      @ical_date.xml_builder_template.should == "xml[\"ical\"].date( '\#{builder_new_value}' )"
+      @ical_date.xml_builder_template.should == "xml[\'ical\'].date( '\#{builder_new_value}' )"
     end
     
     it "should work for nodes with default_content_path" do      

data/spec/unit/terminology_spec.rb

@@ -87,6 +87,8 @@ describe "OM::XML::Terminology" do
 
     it "should expand proxy and get sub terms" do
       @test_full_terminology.retrieve_node(:title, :main_title_lang).xpath.should == '//oxns:titleInfo/oxns:title/@xml:lang'
+      ### retrieve_term() will not cross proxies
+      @test_full_terminology.retrieve_term(:title_info, :main_title, :main_title_lang).xpath.should == '//oxns:titleInfo/oxns:title/@xml:lang'
     end
 
     it "constructs templates for value-driven searches" do

metadata

@@ -1,13 +1,13 @@
 --- !ruby/object:Gem::Specification 
 name: om
 version: !ruby/object:Gem::Version 
-  hash: 7
+  hash: 3
   prerelease: 
   segments: 
   - 1
   - 4
-  - 0
-  version: 1.4.0
+  - 2
+  version: 1.4.2
 platform: ruby
 authors: 
 - Matt Zumwalt
@@ -15,7 +15,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2011-08-29 00:00:00 -05:00
+date: 2011-09-21 00:00:00 -05:00
 default_executable: 
 dependencies: 
 - !ruby/object:Gem::Dependency 
@@ -213,7 +213,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
 requirements: []
 
 rubyforge_project: 
-rubygems_version: 1.5.2
+rubygems_version: 1.6.2
 signing_key: 
 specification_version: 3
 summary: "OM (Opinionated Metadata): A library to help you tame sprawling XML schemas like MODS."