copy_paste_pdf 0.0.1 → 0.0.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 136b95ea0d8543294bce8905905ae9f638d34367
-  data.tar.gz: c5f220fe4fcf77b4fedf3d6fc5e186487b492ef8
+  metadata.gz: 52f0830d2a7adfa5a108b6a2e73b47e9c384bc22
+  data.tar.gz: 7be6080ee75fc74faf820475c97b2718a9c9a494
 SHA512:
-  metadata.gz: 2522bdfe9adf4f87114d73d3b0d8ca11576fd517af6c1459c286ed22340ca6225aee4320a84e4a9513ad18f98d56117a48f9e1ffd8130260b0c05fb0130c233b
-  data.tar.gz: 91686dc0fdcc83700e5d4e72500fa2667ec212880176eb82587111bfa8c3bab7503c6476979b66130271f120e6604354c05a050feec0e8f76ef2079dff2cba86
+  metadata.gz: b639499bf827557c0f2c586dc27bd19bb6b2a1609b206060f650051252cb29d2fb73562a808f01317c21f9d53ec552501491cf5fc951549851392b0dc202a46d
+  data.tar.gz: b34b0222c2e000921d8006a3adf33ca45406a01b6610fda1c95afe0b2353941a5fa7c4bfc2aeb96b9866801e6c959395af507d48b27dd2269571ebb5da6cbfa5

data/bin/copy-paste-pdf.applescript

@@ -14,6 +14,7 @@ on run argv
   if count of argv is 2 then set close_applications to false
 
   tell application "Preview"
+    -- It seems Preview won't work if we `run` or `launch` it.
     activate
     open input
   end
@@ -51,7 +52,7 @@ on run argv
   end repeat
 
   tell application "Microsoft Excel"
-    activate
+    launch
     make new workbook
     paste worksheet active sheet
     save workbook as active workbook filename output file format CSV file format overwrite yes

data/lib/copy_paste_pdf.rb

@@ -1 +1,9 @@
 require 'copy_paste_pdf/table'
+
+module CopyPastePDF
+  class Error < StandardError; end
+  class CopyToNonEmptyCellError < Error; end
+  class MissingSourceError < Error; end
+  class InvalidRowError < Error; end
+  class MergeWithEmptyCellError < Error; end
+end

data/lib/copy_paste_pdf/table.rb

@@ -13,9 +13,11 @@ module CopyPastePDF
     # @param [Array] indices the cell indices to copy
     # @yieldparam [Array] row a row in the table
     # @yieldreturn [Boolean] whether to skip the row from the table
-    # @raise if a destination has no source
-    # @raise if a destination cell has a value
-    # @raise if a row is neither a source nor a destination
+    # @raise [CopyPastePDF::MissingSourceError] if a destination has no source
+    # @raise [CopyPastePDF::CopyToNonEmptyCellError] if a destination cell
+    #   already has a value
+    # @raise [CopyPastePDF::InvalidRowError] if a row is neither a source nor a
+    #   destination
     def copy_into_cell_below(*indices)
       source = nil
       each do |row|
@@ -28,16 +30,16 @@ module CopyPastePDF
             if source
               indices.each_with_index do |index,i|
                 if row[index]
-                  raise "#{index} contains #{row[index]}"
+                  raise CopyToNonEmptyCellError, "destination cell #{index} already has a value #{row[index]}"
                 else
                   row[index] = source[i]
                 end
               end
             else
-              raise "#{row} has no source"
+              raise MissingSourceError, "#{row} is a destination, but it has no source"
             end
           else
-            raise "#{row} is neither a source nor a destination"
+            raise InvalidRowError, "#{row} is neither a source nor a destination"
           end
         end
       end
@@ -47,7 +49,7 @@ module CopyPastePDF
     # empty, into the corresponding cells of the prececeding line.
     #
     # @param [Array] indices the cell indices to merge
-    # @raise if a destination cell is empty
+    # @raise [CopyPastePDF::MergeWithEmptyCellError] if a destination cell is empty
     def merge_into_cell_above(*indices)
       each_with_index.reverse_each do |row,i|
         if row.each_with_index.all?{|value,j| value.nil? || indices.include?(j)}
@@ -55,7 +57,7 @@ module CopyPastePDF
             if self[i - 1][index]
               self[i - 1][index] = "#{self[i - 1][index]}\n#{row[index]}"
             else
-              raise "#{index} is empty"
+              raise MergeWithEmptyCellError, "cell #{index} is empty and can't be merged"
             end
           end
           delete_at(i)

data/lib/copy_paste_pdf/version.rb

@@ -1,3 +1,3 @@
 module CopyPastePDF
-  VERSION = "0.0.1"
+  VERSION = "0.0.2"
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: copy_paste_pdf
 version: !ruby/object:Gem::Version
-  version: 0.0.1
+  version: 0.0.2
 platform: ruby
 authors:
 - Open North
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-10-10 00:00:00.000000000 Z
+date: 2013-10-11 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: coveralls