Bugfix to escape document names during file type detection.

Author: knowtheory

docsplit.gemspec

@@ -1,6 +1,6 @@
 Gem::Specification.new do |s|
   s.name      = 'docsplit'
-  s.version   = '0.6.1'         # Keep version in sync with docsplit.rb
+  s.version   = '0.6.2'         # Keep version in sync with docsplit.rb
   s.date      = '2011-11-18'
 
   s.homepage    = "http://documentcloud.github.com/docsplit/"

index.html

@@ -277,6 +277,11 @@ <h2 id="internals">Internals</h2>
 
     <h2 id="changes">Change Log</h2>
 
+    <p>
+      <b class="header">0.6.2</b><small> &ndash; Nov. 22, 2011</small><br />
+      Bugfix to escape document names during file type detection.
+    </p>
+    
     <p>
       <b class="header">0.6.1</b><small> &ndash; Nov. 18, 2011</small><br />
       Docsplit now supports converting documents using LibreOffice

lib/docsplit.rb

@@ -1,7 +1,7 @@
 # The Docsplit module delegates to the Java PDF extractors.
 module Docsplit
 
-  VERSION       = '0.6.0' # Keep in sync with gemspec.
+  VERSION       = '0.6.2' # Keep in sync with gemspec.
 
   ROOT          = File.expand_path(File.dirname(__FILE__) + '/..')
 
@@ -68,7 +68,7 @@ def self.extract_pdf(docs, opts={})
       basename = File.basename(doc, ext)
       escaped_doc, escaped_out, escaped_basename = [doc, out, basename].map(&ESCAPE)
 
-      if GM_FORMATS.include?(`file -b --mime #{doc}`.strip.split(/[:;]\s+/)[0])
+      if GM_FORMATS.include?(`file -b --mime #{ESCAPE[doc]]}`.strip.split(/[:;]\s+/)[0])
         `gm convert #{escaped_doc} #{escaped_out}/#{escaped_basename}.pdf`
       else
         options = "-jar #{ROOT}/vendor/jodconverter/jodconverter-core-3.0-beta-4.jar -r #{ROOT}/vendor/conf/document-formats.js"