hydra-derivatives 0.0.3 → 0.0.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 6cb23951238b5918e0ea720629b80abbbf7c36f9
-  data.tar.gz: c518b9251a6869b1f76dc6095fcaa818f447e959
+  metadata.gz: 7e62821ad5807bc96b9df2c9e2fda733081bb06a
+  data.tar.gz: c3d04f3be7dbb1241f6c51043fbca299370a255b
 SHA512:
-  metadata.gz: fcc714f73eeda9db4e5c9d8bc3cfd62675d7f8af210abca8876d68ca9315479ff751624ac2c78c65b3314f08e1e5c369cff4c9edae21ec1c51fc20fb82ff977b
-  data.tar.gz: 08f350ea7eb44750e37e19c1e3082badc71c01258541f3fe5533887b4718517cad0f8591f538ce54959ed543d9991509fdafbeb0d819fe2738fc6b3e092aec5b
+  metadata.gz: 01e2f3cfdc4a190d122219cbf16f455f50a98fa1b7eab0598350aecdd58ac3bcaa53e07d5d7b0af0845788aabd91d9025e34aa9c35c17020165f0f69c6dc6adc
+  data.tar.gz: 7004d501623dc2667189951292421b436d8c441b5e8373f7d393a951a793ae226a1f0f721ec8b234373044c4d8900724f1bc3012924ac14d894e14d8c0abf5af

data/History.md

@@ -0,0 +1,19 @@
+## 0.0.4 (2013-07-25)
+- Handle invalid mime-type on the datastream [Justin Coyne]
+
+## 0.0.3 (2013-07-25)
+- Added LibreOffice support [Justin Coyne]
+- Updating README [Matt Zumwalt]
+- explicitly testing support for using makes_derivatives with callback methods
+[Matt Zumwalt]
+- support both block syntax and callbacks [Matt Zumwalt]
+- Break out the config [Justin Coyne]
+- sample implementation of block syntax [Matt Zumwalt]
+
+## 0.0.2 (2013-07-24)
+- Video and PDF support [Justin Coyne]
+- API Change
+- Video and audio processor accepts ':datastream' as an argument [Justin Coyne]
+
+## 0.0.1 (2013-07-23)
+- initial release

data/VERSION

@@ -1 +1 @@
-0.0.3
+0.0.4

data/lib/hydra/derivatives/extract_metadata.rb

@@ -14,8 +14,9 @@ module Hydra
 
       def to_tempfile &block
         return unless has_content?
-        list_of_extensions = MIME::Types[mimeType].first.extensions
-        extension = ".#{list_of_extensions.first}" if list_of_extensions
+        type = MIME::Types[mimeType].first
+        logger.warn "Unable to find a registered mime type for #{mimeType.inspect} on #{pid}" unless type
+        extension = type ? ".#{type.extensions.first}" : ''
 
         f = Tempfile.new(["#{pid}-#{dsVersionID}", extension])
         f.binmode

data/spec/units/transcoding_spec.rb

@@ -82,6 +82,18 @@ describe "Transcoder" do
     end
   end
 
+  describe "when the source datastrem has an unknown mime_type" do
+    let(:attachment) { File.open(File.expand_path('../../fixtures/piano_note.wav', __FILE__))}
+    let(:file) { GenericFile.new(mime_type: 'audio/wav').tap { |t| t.content.content = attachment; t.content.mimeType = 'audio/vnd.wav'; t.save } }
+
+    it "should transcode" do
+      expect(logger).to receive(:warn).with("Unable to find a registered mime type for \"audio/vnd.wav\" on #{file.pid}").twice
+      file.create_derivatives
+      file.datastreams['content_mp3'].should have_content
+      file.datastreams['content_mp3'].mimeType.should == 'audio/mpeg'
+    end
+  end
+
   describe "with an attached video", unless: ENV['TRAVIS'] == 'true' do
     let(:attachment) { File.open(File.expand_path('../../fixtures/countdown.avi', __FILE__))}
     let(:file) { GenericFile.new(mime_type: 'video/avi').tap { |t| t.content.content = attachment; t.save } }
@@ -108,6 +120,7 @@ describe "Transcoder" do
     end
   end
 
+
   describe "with an attached Powerpoint", unless: ENV['TRAVIS'] == 'true' do
     let(:attachment) { File.open(File.expand_path('../../fixtures/FlashPix.ppt', __FILE__))}
     let(:file) { GenericFile.new(mime_type: 'application/vnd.ms-powerpoint').tap { |t| t.content.content = attachment; t.save } }

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: hydra-derivatives
 version: !ruby/object:Gem::Version
-  version: 0.0.3
+  version: 0.0.4
 platform: ruby
 authors:
 - Justin Coyne
@@ -126,6 +126,7 @@ files:
 - .travis.yml
 - CONTRIBUTING.md
 - Gemfile
+- History.md
 - LICENSE.txt
 - README.md
 - Rakefile