active-fedora 1.1.11 → 1.1.13

data/History.txt

@@ -1,3 +1,11 @@
+1.1.13
+
+MetadataDatastream.update_indexed_attributes was no longer marking the object dirty when triggered.  Resulted in changed datastreams not saving.  Fixed now.
+
+1.1.12
+
+* Minor fix to Base.get_values_from_datstream (wasn't returning default values when necessary)
+
 1.1.11
 
 * fix to update_index preventing double-metadata

data/Rakefile

@@ -21,7 +21,7 @@ begin
     gem.add_dependency('mime-types', '>= 1.16')
     gem.add_dependency('multipart-post')
     gem.add_dependency('nokogiri')
-    gem.add_dependency('om', '>= 0.1.7')
+    gem.add_dependency('om', '>= 0.1.9')
     # gem.add_dependency('yaml')
 
     gem.add_development_dependency "rspec", ">= 1.2.9"

data/VERSION

@@ -1 +1 @@
-1.1.11
+1.1.13

data/active-fedora.gemspec

@@ -5,11 +5,11 @@
 
 Gem::Specification.new do |s|
   s.name = %q{active-fedora}
-  s.version = "1.1.11"
+  s.version = "1.1.13"
 
   s.required_rubygems_version = Gem::Requirement.new(">= 0") if s.respond_to? :required_rubygems_version=
   s.authors = ["Matt Zumwalt", "McClain Looney"]
-  s.date = %q{2010-07-02}
+  s.date = %q{2010-07-17}
   s.description = %q{ActiveFedora provides for creating and managing objects in the Fedora Repository Architecture.}
   s.email = %q{matt.zumwalt@yourmediashelf.com}
   s.extra_rdoc_files = [
@@ -345,7 +345,7 @@ Gem::Specification.new do |s|
       s.add_runtime_dependency(%q<mime-types>, [">= 1.16"])
       s.add_runtime_dependency(%q<multipart-post>, [">= 0"])
       s.add_runtime_dependency(%q<nokogiri>, [">= 0"])
-      s.add_runtime_dependency(%q<om>, [">= 0.1.7"])
+      s.add_runtime_dependency(%q<om>, [">= 0.1.9"])
       s.add_development_dependency(%q<rspec>, [">= 1.2.9"])
       s.add_development_dependency(%q<mocha>, [">= 1.2.9"])
       s.add_development_dependency(%q<ruby-debug>, [">= 0"])
@@ -355,7 +355,7 @@ Gem::Specification.new do |s|
       s.add_dependency(%q<mime-types>, [">= 1.16"])
       s.add_dependency(%q<multipart-post>, [">= 0"])
       s.add_dependency(%q<nokogiri>, [">= 0"])
-      s.add_dependency(%q<om>, [">= 0.1.7"])
+      s.add_dependency(%q<om>, [">= 0.1.9"])
       s.add_dependency(%q<rspec>, [">= 1.2.9"])
       s.add_dependency(%q<mocha>, [">= 1.2.9"])
       s.add_dependency(%q<ruby-debug>, [">= 0"])
@@ -366,7 +366,7 @@ Gem::Specification.new do |s|
     s.add_dependency(%q<mime-types>, [">= 1.16"])
     s.add_dependency(%q<multipart-post>, [">= 0"])
     s.add_dependency(%q<nokogiri>, [">= 0"])
-    s.add_dependency(%q<om>, [">= 0.1.7"])
+    s.add_dependency(%q<om>, [">= 0.1.9"])
     s.add_dependency(%q<rspec>, [">= 1.2.9"])
     s.add_dependency(%q<mocha>, [">= 1.2.9"])
     s.add_dependency(%q<ruby-debug>, [">= 0"])

data/lib/active_fedora/metadata_datastream.rb

@@ -79,6 +79,7 @@ module ActiveFedora
           values = instance_eval("#{field_name}_values=(new_values)")
           result[field_name] = {"0"=>values}           
         end
+        self.dirty = true
       end
       return result
     end

data/lib/active_fedora/nokogiri_datastream.rb

@@ -136,8 +136,11 @@ class ActiveFedora::NokogiriDatastream < ActiveFedora::Datastream
         self.class.accessor_xpath(*OM.destringify(field_key) ).nil?
       end
     end
-    result = update_properties( params )
-    self.dirty = true
+    result = {}
+    unless params.empty?
+      result = update_properties( params )
+      self.dirty = true
+    end
     return result
   end
   

data/spec/unit/metadata_datastream_spec.rb

@@ -54,7 +54,9 @@ describe ActiveFedora::MetadataDatastream do
       @test_ds.field "swank", :text
     end
     
-    it "should apply submitted hash to corresponding datastream field values" do
+    it "should apply submitted hash to corresponding datastream field values and mark the object dirty" do
+      @test_ds.should_not be_dirty
+      
       att= {"fubar"=>{"-1"=>"mork", "0"=>"york"}}
       @test_ds.update_indexed_attributes(att)
       @test_ds.fubar_values.should == ['mork', 'york']
@@ -68,6 +70,8 @@ describe ActiveFedora::MetadataDatastream do
       result = @test_ds.update_indexed_attributes(att)
       result.should == {"fubar"=>{"0"=>"hork", "1"=>"tork", '3'=>'dang'}}
       @test_ds.fubar_values.should == ['hork', 'tork', 'mangle', 'dang']
+      
+      @test_ds.should be_dirty
     end
     
     it "should support single-value arguments (as opposed to a hash of values with array indexes as keys)" do

metadata

@@ -1,13 +1,13 @@
 --- !ruby/object:Gem::Specification 
 name: active-fedora
 version: !ruby/object:Gem::Version 
-  hash: 5
+  hash: 9
   prerelease: false
   segments: 
   - 1
   - 1
-  - 11
-  version: 1.1.11
+  - 13
+  version: 1.1.13
 platform: ruby
 authors: 
 - Matt Zumwalt
@@ -16,7 +16,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2010-07-02 00:00:00 -05:00
+date: 2010-07-17 00:00:00 +02:00
 default_executable: 
 dependencies: 
 - !ruby/object:Gem::Dependency 
@@ -102,12 +102,12 @@ dependencies:
     requirements: 
     - - ">="
       - !ruby/object:Gem::Version 
-        hash: 21
+        hash: 9
         segments: 
         - 0
         - 1
-        - 7
-        version: 0.1.7
+        - 9
+        version: 0.1.9
   type: :runtime
   version_requirements: *id006
 - !ruby/object:Gem::Dependency