lionel_richie 0.1.5 → 0.1.5.1

checksums.yaml

@@ -1,15 +1,15 @@
 ---
 !binary "U0hBMQ==":
   metadata.gz: !binary |-
-    MDg0NzdjY2E1ZDRiMzVhNzVjYzU2ZTdhNjhjZmZkMTc0MjFjYmUyZQ==
+    OGRkODE3YzNlNjVkZmUzNWVhZTZjZTdmNzYyYzY0OWUyZmNjYWI2OA==
   data.tar.gz: !binary |-
-    OGUwOTllMjkzYzhlNDVlZGQ5Yzk5YmU3N2YzYjIwZTBjNDhjMjk2Yw==
+    YTBlNWFlNDFjYWY0Y2ZlMzAyNTJhMmIzYTc1OWNmZmVlZTY5M2JmYg==
 !binary "U0hBNTEy":
   metadata.gz: !binary |-
-    ODUwNThhNDBmZTE1NTcyMTcyMWIzNWU3NTVlZmQ2ZDg1N2VhZDE2ZWRjNTY5
-    M2M5YWE4YmRhOTVlYjE2Y2FmNGNjYzBiMTEzZThmYjgxYWJmN2EzY2RkMTNk
-    NTQ0ZWRhZDdmM2ExZmY0NzE4MmZjMzIwNzU0MzdiYzM3ZTBlMzQ=
+    MGViMDI0ZTdiZmI5ZjFkZDA5MmMxMWI2Zjg0MDFhYzNlZDBkMzgyMjU2Yzc1
+    YmU2MDg4Zjc0NTQxZDI5MTZkYWEzMjZjYzBlOGIzNTBkOWZmZDg3Njc4Mjhm
+    NWZlYjY3OWIwMWQ0ZDlkMTg5MTAwMWY5OTU4Y2MxNTYyODIwMWM=
   data.tar.gz: !binary |-
-    NjU5ZWE1Mjk5ZTAzNTY4NTMzZTQwMDFmMDMwN2I4OWEzMjk4ZjAxYzhiYmY4
-    ZjBhN2MxMzhlMzZiOTdmZDViOWMyMzViM2M0MmYzNmJjOWNhMDRiMWE5MWU5
-    YzcyMTI2ZTA3OTNhNDRiZTg5ZTNiYzA4NTZkZTk5NjE2MTQwNWE=
+    YzEyZjNhZWUzMTM2N2M0MjI0YjFjN2NkZjA0YTdkOGYwNTk2NDgyODU3NmQw
+    ZTVlMjIzMmYwNjM3NjVmMzdiMzZiNzBhN2NhOTdmODI0Y2FjYWE3MjUxZDc5
+    NDg1NmU5MTY1ZmU3MDJmZTYzN2EzZjhkNzBjYjkyNzhjOTdmY2U=

data/lib/lionel/export.rb

@@ -57,30 +57,15 @@ module Lionel
     def download
       Lionel.logger.info "Exporting trello board '#{board.name}' (#{trello_board_id}) to " + "google doc '#{spreadsheet.title}' (#{google_doc_id})"
 
-      start_row = 2
-      rows = worksheet.size
-
-      card_rows = {}
-
-      # Find existing rows for current cards
-      (start_row..rows).each do |row|
-        cell_id = worksheet["B",row]
-        next unless cell_id.present?
-        card = cards.find { |c| c.id == cell_id }
-        next unless card.present?
-        card_rows[row] = card
-      end
-
-      # Set available rows for new cards
-      new_cards = cards - card_rows.values
-      new_cards.each_with_index do |card, i|
-        row = rows + i + 1
-        card_rows[row] = card
-      end
-
-      card_rows.each do |row, card|
+      card_map.each do |row, card|
         begin
-          Timeout.timeout(5) { sync_row(row, card) }
+          Timeout.timeout(5) {
+            Lionel.logger.info "row[#{row}] = #{card.name}"
+
+            sync_columns(card).each do |col, value|
+              worksheet[col,row] = value
+            end
+          }
         rescue Timeout::Error, Trello::Error => e
           Lionel.logger.warn e.inspect
           Lionel.logger.warn card.inspect
@@ -88,6 +73,10 @@ module Lionel
       end
     end
 
+    def card_map
+      @card_map ||= CardMap.new(cards, worksheet)
+    end
+
     def upload
       worksheet.save
     end
@@ -96,46 +85,46 @@ module Lionel
       worksheet.rows
     end
 
-    def sync_row(row, card)
-      Lionel.logger.info "row[#{row}] = #{card.name}"
-
-      worksheet["B",row] = card.id
+    def sync_columns(card)
+      {}.tap do |columns|
+        columns["B"] = card.id
 
-      # Card link
-      worksheet["C",row] = card.link
+        # Card link
+        columns["C"] = card.link(card.name.gsub(/^\[.*\]\s*/, ""))
 
-      # Ready date
-      ready_action = card.first_action do |a|
-        (a.create? && a.board_id == trello_board_id) || a.moved_to?("Ready")
-      end
-      worksheet["D",row] = card.format_date(ready_action.date) if ready_action
+        # Ready date
+        ready_action = card.first_action do |a|
+          (a.create? && a.board_id == trello_board_id) || a.moved_to?("Ready")
+        end
+        columns["D"] = card.format_date(ready_action.date) if ready_action
 
-      # In Progress date
-      worksheet["E",row] = card.date_moved_to("In Progress")
+        # In Progress date
+        columns["E"] = card.date_moved_to("In Progress")
 
-      # Code Review date
-      worksheet["F",row] = card.date_moved_to("Code Review")
+        # Code Review date
+        columns["F"] = card.date_moved_to("Code Review")
 
-      # Review date
-      worksheet["G",row] = card.date_moved_to("Review")
+        # Review date
+        columns["G"] = card.date_moved_to("Review")
 
-      # Deploy date
-      worksheet["H",row] = card.date_moved_to("Deploy")
+        # Deploy date
+        columns["H"] = card.date_moved_to("Deploy")
 
-      # Completed date
-      worksheet["I",row] = card.date_moved_to("Completed")
+        # Completed date
+        columns["I"] = card.date_moved_to("Completed")
 
-      # Type
-      worksheet["J",row] = card.type
+        # Type
+        columns["J"] = card.type
 
-      # Project
-      worksheet["K",row] = card.project
+        # Project
+        columns["K"] = card.project
 
-      # Estimate
-      worksheet["L",row] = card.estimate
+        # Estimate
+        columns["L"] = card.estimate
 
-      # Due Date
-      worksheet["M",row] = card.due_date
+        # Due Date
+        columns["M"] = card.due_date
+      end
     end
 
     def authenticate
@@ -187,14 +176,39 @@ module Lionel
       end
 
       def each(&block)
-        card_rows.each(&block)
+        rows.each(&block)
+      end
+
+      def rows
+        @rows ||= populate_rows
       end
 
-      def card_rows
-        @card_rows ||= {}.tap do |map|
+      private
+
+      def populate_rows
+        Lionel.logger.info "Using CardMap!!!!"
+        {}.tap do |card_rows|
+
+          start_row = 2
+          rows = worksheet.size
+
+          # Find existing rows for current cards
+          (start_row..rows).each do |row|
+            cell_id = worksheet["B",row]
+            next unless cell_id.present?
+            card = cards.find { |c| c.id == cell_id }
+            next unless card.present?
+            card_rows[row] = card
+          end
+
+          # Set available rows for new cards
+          new_cards = cards - card_rows.values
+          new_cards.each_with_index do |card, i|
+            row = rows + i + 1
+            card_rows[row] = card
+          end
         end
       end
     end
-
   end
 end

data/lib/lionel/proxy_card.rb

@@ -7,8 +7,8 @@ module Lionel
       @card = card
     end
 
-    def link
-      %Q[=HYPERLINK("#{card.url}", "#{card.name.gsub(/"/, "")}")]
+    def link(name = card.name)
+      %Q[=HYPERLINK("#{card.url}", "#{name.gsub(/"/, "")}")]
     end
 
     MAX_ACTIONS = 1000

data/lib/lionel/version.rb

@@ -1,3 +1,3 @@
 module Lionel
-  VERSION = "0.1.5"
+  VERSION = "0.1.5.1"
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: lionel_richie
 version: !ruby/object:Gem::Version
-  version: 0.1.5
+  version: 0.1.5.1
 platform: ruby
 authors:
 - Ross Kaffenberger
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-08-19 00:00:00.000000000 Z
+date: 2013-08-30 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: ruby-trello