active_fedora-registered_attributes 0.0.2 → 0.0.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 1f3ebbd73ebefa0ab6879e1c931ba1e910d57cc3
-  data.tar.gz: 9f3e8bffeb4f9a856436ab3eeb14fc845481613e
+  metadata.gz: 391408142f61cbfc8e7a066c9679201590b6280f
+  data.tar.gz: aadb6192c4be35d014b382f527cee72ead010413
 SHA512:
-  metadata.gz: 9f3dbba63bb918a6ba656116f3d04336ba738b1f7af4928bff84eb538c665d3d09fcefc4760e15b4331ac9404827197e4c4f87c3a5005b23cb18591071c6c37f
-  data.tar.gz: be2a203b103986dc7ed68f03c6f2de2770be7719fc932c74bbfaa88de0c79033e7ca0b1cc9dff8c5020e77c68e9b6f7f75c69b0d20d29d65b6b5f1fbf67ea26f
+  metadata.gz: 1e8ce89fa90924ac1108905f79d1e28a6f6edbd4f2a95cef43aa99d3a051a0284621ace0e83cd19b26dc67d109923f9ac1a1fbad21349e08fb1055ec34b2af68
+  data.tar.gz: 9e419729a17762bf442afb7c54c3f604926415c2c372fd0a0ed1a0a1c9f8269396883f9a9e78516e2cdbe31b25b510c7328b1f481829b597e1ec372ab57a6b79

data/lib/active_fedora/registered_attributes/attribute.rb

@@ -46,6 +46,10 @@ module ActiveFedora
         @options[:form] ||= {}
       end
 
+      def multiple?
+        options[:multiple]
+      end
+
       def displayable?
         @options.fetch(:displayable, true)
       end
@@ -75,7 +79,7 @@ module ActiveFedora
         options[:form].tap {|hash|
           hash[:hint] ||= options[:hint] if options[:hint]
           hash[:label] ||= options[:label] if options[:label]
-          if options[:multiple]
+          if multiple?
             hash[:as] = 'multi_value'
             hash[:input_html] ||= {}
             hash[:input_html][:multiple] = 'multiple'
@@ -118,22 +122,27 @@ module ActiveFedora
 
         def options_for_delegation
           if datastream.is_a?(Hash)
-            datastream.merge(unique: !options[:multiple])
+            datastream.merge(unique: !multiple?)
           else
             {
               to: datastream,
-              unique: !options[:multiple]
+              unique: !multiple?
             }
           end
         end
 
         def with_writer_method_wrap
+          method_name = "#{name}=".to_sym
           if writer = options[:writer]
-            method_name = "#{name}=".to_sym
             proc = writer.respond_to?(:call) ?
               writer :
               lambda { |value| send(writer, value) }
             yield(method_name, proc)
+          elsif multiple?
+            proc = lambda {|values|
+              values.select {|value| value.present? }
+            }
+            yield(method_name, proc)
           end
         end
 

data/lib/active_fedora/registered_attributes/version.rb

@@ -1,5 +1,5 @@
 module ActiveFedora
   module RegisteredAttributes
-    VERSION = "0.0.2"
+    VERSION = "0.0.3"
   end
 end

data/spec/active_fedora/registered_attributes_spec.rb

@@ -145,6 +145,14 @@ describe 'ActiveFedora::RegisteredAttributes' do
       expect(subject.file_names).to eq(values.reverse)
     end
 
+    it "intercepts blanks and omits them" do
+      values = ['foo.rb', '', 'bar.rb', '']
+      expected_values = ['foo.rb', 'bar.rb']
+      subject.description = values
+      expect(subject.properties.description).to eq(expected_values)
+      expect(subject.description).to eq(expected_values)
+    end
+
     it "allows for a writer that is a symbol" do
       values = ['South Bend', 'State College', 'Minneapolis']
       expected_values = values.dup

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: active_fedora-registered_attributes
 version: !ruby/object:Gem::Version
-  version: 0.0.2
+  version: 0.0.3
 platform: ruby
 authors:
 - Jeremy Friesen
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-07-19 00:00:00.000000000 Z
+date: 2013-07-23 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: active-fedora