to_spreadsheet 1.0.0 → 1.0.1

data/lib/to_spreadsheet.rb

@@ -1,9 +1,7 @@
-require 'nokogiri'
-require 'to_spreadsheet/action_pack_renderers'
-require 'to_spreadsheet/mime_types'
 require 'to_spreadsheet/version'
-require 'to_spreadsheet/helpers'
 require 'to_spreadsheet/context'
+require 'to_spreadsheet/renderer'
+require 'to_spreadsheet/railtie' if defined?(Rails)
 
 module ToSpreadsheet
   class << self

data/lib/to_spreadsheet/context.rb

@@ -17,10 +17,26 @@ module ToSpreadsheet
       def global
         @global ||= new
       end
+
+      def current
+        Thread.current[:_to_spreadsheet_ctx]
+      end
+
+      def current=(ctx)
+        Thread.current[:_to_spreadsheet_ctx] = ctx
+      end
+
+      def with_context(ctx, &block)
+        old = current
+        self.current = ctx
+        r = block.call(ctx)
+        self.current = old
+        r
+      end
     end
 
     def initialize(wb_options = nil)
-      @rules          = []
+      @rules = []
       workbook wb_options if wb_options
     end
 
@@ -83,7 +99,7 @@ module ToSpreadsheet
 
     # A new context
     def merge(other_context)
-      ctx = Context.new()
+      ctx       = Context.new()
       ctx.rules = rules + other_context.rules
       ctx
     end

data/lib/to_spreadsheet/{action_pack_renderers.rb → rails/action_pack_renderers.rb}

@@ -1,32 +1,24 @@
-require 'active_support'
-require 'action_controller/metal/renderers'
-require 'action_controller/metal/responder'
-
-require 'to_spreadsheet/renderer'
-
-# This will let us do thing like `render :xlsx => 'index'`
-# This is similar to how Rails internally implements its :json and :xml renderers
-ActionController::Renderers.add :xlsx do |template, options|
-  filename = options[:filename] || options[:template] || 'data'
-
-  html = with_context ToSpreadsheet::Context.global.merge(ToSpreadsheet::Context.new) do
-    # local context
-    @local_formats.each do |selector, &block|
-      context.process_dsl selector, &block
-    end if @local_formats
-    render_to_string(options[:template], options)
-  end
-
-  data = ToSpreadsheet::Axlsx::Renderer.to_data(html)
-  send_data data, type: :xlsx, disposition: %(attachment; filename="#{filename}.xlsx")
-end
-
-class ActionController::Responder
-  # This sets up a default render call for when you do
-  # respond_to do |format|
-  #   format.xlsx
-  # end
-  def to_xlsx
-    controller.render xlsx: controller.action_name
-  end
-end
+require 'active_support'
+require 'action_controller/metal/renderers'
+require 'action_controller/metal/responder'
+
+# This will let us do thing like `render :xlsx => 'index'`
+# This is similar to how Rails internally implements its :json and :xml renderers
+ActionController::Renderers.add :xlsx do |template, options|
+  filename = options[:filename] || options[:template] || 'data'
+  data = ToSpreadsheet::Context.with_context ToSpreadsheet::Context.global.merge(ToSpreadsheet::Context.new) do |context|
+    html = render_to_string(template, options.merge(template: template, formats: ['html']))
+    ToSpreadsheet::Renderer.to_data(html, context)
+  end
+  send_data data, type: :xlsx, disposition: %(attachment; filename="#{filename}.xlsx")
+end
+
+class ActionController::Responder
+  # This sets up a default render call for when you do
+  # respond_to do |format|
+  #   format.xlsx
+  # end
+  def to_xlsx
+    controller.render xlsx: controller.action_name
+  end
+end

data/lib/to_spreadsheet/{mime_types.rb → rails/mime_types.rb}

@@ -1,2 +1,2 @@
-require 'action_dispatch/http/mime_type'
-Mime::Type.register "application/vnd.openxmlformats-officedocument.spreadsheetml.sheet", :xlsx unless Mime::Type.lookup_by_extension(:xlsx)
+require 'action_dispatch/http/mime_type'
+Mime::Type.register "application/vnd.openxmlformats-officedocument.spreadsheetml.sheet", :xlsx unless Mime::Type.lookup_by_extension(:xlsx)

data/lib/to_spreadsheet/rails/view_helpers.rb

@@ -0,0 +1,11 @@
+module ToSpreadsheet
+  module Rails
+    module ViewHelpers
+      def format_xls(selector = nil, &block)
+        ctx = ToSpreadsheet::Context.current
+        return unless ctx
+        ctx.format_xls selector, &block
+      end
+    end
+  end
+end

data/lib/to_spreadsheet/railtie.rb

@@ -0,0 +1,11 @@
+require 'rails/railtie'
+require 'to_spreadsheet/rails/mime_types'
+require 'to_spreadsheet/rails/action_pack_renderers'
+require 'to_spreadsheet/rails/view_helpers'
+module ToSpreadsheet
+  class Railtie < ::Rails::Railtie
+    initializer "to_spreadsheet.view_helpers" do
+      ActionView::Base.send :include, ToSpreadsheet::Rails::ViewHelpers
+    end
+  end
+end

data/lib/to_spreadsheet/renderer.rb

@@ -1,34 +1,30 @@
 require 'axlsx'
+require 'nokogiri'
+
 module ToSpreadsheet
   module Renderer
     extend self
 
-    def to_stream(html, local_context = nil)
-      to_package(html, local_context).to_stream
+    def to_stream(html, context = nil)
+      to_package(html, context).to_stream
     end
 
-    def to_data(html, local_context = nil)
-      to_package(html, local_context).to_stream.read
+    def to_data(html, context = nil)
+      to_package(html, context).to_stream.read
     end
 
-    def to_package(html, local_context = nil)
-      with_context init_context(local_context) do
-        package = build_package(html, context)
-        context.rules.each do |rule|
-          puts "Applying #{rule}"
-          rule.apply(context, package)
-        end
-        package
+    def to_package(html, context = nil)
+      context ||= ToSpreadsheet::Context.global.merge(Context.new)
+      package = build_package(html, context)
+      context.rules.each do |rule|
+        #Rails.logger.debug "Applying #{rule}"
+        rule.apply(context, package)
       end
+      package
     end
 
     private
 
-    def init_context(local_context)
-      local_context ||= ToSpreadsheet::Context.new
-      ToSpreadsheet::Context.global.merge local_context
-    end
-
     def build_package(html, context)
       package     = ::Axlsx::Package.new
       spreadsheet = package.workbook

data/lib/to_spreadsheet/rule/format.rb

@@ -5,10 +5,11 @@ module ToSpreadsheet
     class Format < Base
       include ::ToSpreadsheet::TypeFromValue
       def apply(context, sheet)
+        wb = sheet.workbook
         case selector_type
           when :css
             css_match selector_query, context.to_xml_node(sheet) do |xml_node|
-              apply_inline_styles context, context.to_xls_entity(xml_node)
+              add_and_apply_style wb, context, context.to_xls_entity(xml_node)
             end
           when :row
             sheet.row_style selector_query, options if options.present?
@@ -17,7 +18,7 @@ module ToSpreadsheet
             sheet.col_style selector_query, inline_styles if inline_styles.present?
             apply_col_info sheet.column_info[selector_query]
           when :range
-            apply_inline_styles range_match(selector_type, sheet), context
+            add_and_apply_style wb, range_match(selector_type, sheet), context
         end
       end
 
@@ -32,7 +33,7 @@ module ToSpreadsheet
         end
       end
 
-      def apply_inline_styles(context, xls_ent)
+      def add_and_apply_style(wb, context, xls_ent)
         # Custom format rule
         # format 'td.sel', lambda { |node| ...}
         if self.options.is_a?(Proc)
@@ -47,17 +48,9 @@ module ToSpreadsheet
           options[k] = context.instance_exec(xls_ent, &v) if v.is_a?(Proc)
         end
 
-        # Apply inline styles
-        options.each do |k, v|
-          next if v.nil?
-          setter = :"#{k}="
-          xls_ent.send setter, v if xls_ent.respond_to?(setter)
-          if xls_ent.respond_to?(:cells)
-            xls_ent.cells.each do |cell|
-              cell.send setter, v if cell.respond_to?(setter)
-            end
-          end
-        end
+        style = wb.styles.add_style options
+        cells = xls_ent.respond_to?(:cells) ? xls_ent.cells : [xls_ent]
+        cells.each { |cell| cell.style = style }
       end
     end
   end

data/lib/to_spreadsheet/version.rb

@@ -1,3 +1,3 @@
-module ToSpreadsheet
-  VERSION = '1.0.0'
-end
+module ToSpreadsheet
+  VERSION = '1.0.1'
+end

data/spec/format_spec.rb

@@ -6,7 +6,7 @@ describe ToSpreadsheet::Rule::Format do
 :ruby
   format_xls do
     format column: 0, width: 25
-    format 'tr', color: lambda { |row| 'cccccc' if row.index.odd? }
+    format 'tr', fg_color: lambda { |row| 'cccccc' if row.index.odd? }
   end
 %table
   %tr
@@ -23,7 +23,10 @@ describe ToSpreadsheet::Rule::Format do
       sheet.column_info[0].width.should == 25
     end
     it 'runs lambdas' do
-      sheet.rows[1].cells[0].color.rgb.should == Axlsx::Color.new(rgb: 'cccccc').rgb
+      cell = sheet.rows[1].cells[0]
+      styles = sheet.workbook.styles
+      font_id = styles.cellXfs[cell.style].fontId
+      styles.fonts[font_id].color.rgb.should == Axlsx::Color.new(rgb: 'cccccc').rgb
     end
   end
 end

metadata

@@ -1,111 +1,128 @@
---- !ruby/object:Gem::Specification 
+--- !ruby/object:Gem::Specification
 name: to_spreadsheet
-version: !ruby/object:Gem::Version 
-  prerelease: false
-  segments: 
-  - 1
-  - 0
-  - 0
-  version: 1.0.0
+version: !ruby/object:Gem::Version
+  prerelease: 
+  version: 1.0.1
 platform: ruby
-authors: 
+authors:
 - Gleb Mazovetskiy
 autorequire: 
 bindir: bin
 cert_chain: []
-
-date: 2012-11-29 00:00:00 +01:00
-default_executable: 
-dependencies: 
-- !ruby/object:Gem::Dependency 
+date: 2012-12-04 00:00:00.000000000 Z
+dependencies:
+- !ruby/object:Gem::Dependency
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+    none: false
   name: rails
-  prerelease: false
-  requirement: &id001 !ruby/object:Gem::Requirement 
-    requirements: 
-    - - ">="
-      - !ruby/object:Gem::Version 
-        segments: 
-        - 0
-        version: "0"
   type: :runtime
-  version_requirements: *id001
-- !ruby/object:Gem::Dependency 
-  name: nokogiri
   prerelease: false
-  requirement: &id002 !ruby/object:Gem::Requirement 
-    requirements: 
-    - - ">="
-      - !ruby/object:Gem::Version 
-        segments: 
-        - 0
-        version: "0"
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+    none: false
+- !ruby/object:Gem::Dependency
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+    none: false
+  name: nokogiri
   type: :runtime
-  version_requirements: *id002
-- !ruby/object:Gem::Dependency 
-  name: axlsx
   prerelease: false
-  requirement: &id003 !ruby/object:Gem::Requirement 
-    requirements: 
-    - - ">="
-      - !ruby/object:Gem::Version 
-        segments: 
-        - 0
-        version: "0"
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+    none: false
+- !ruby/object:Gem::Dependency
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+    none: false
+  name: axlsx
   type: :runtime
-  version_requirements: *id003
-- !ruby/object:Gem::Dependency 
-  name: chronic
   prerelease: false
-  requirement: &id004 !ruby/object:Gem::Requirement 
-    requirements: 
-    - - ">="
-      - !ruby/object:Gem::Version 
-        segments: 
-        - 0
-        version: "0"
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+    none: false
+- !ruby/object:Gem::Dependency
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+    none: false
+  name: chronic
   type: :runtime
-  version_requirements: *id004
-- !ruby/object:Gem::Dependency 
-  name: mocha
   prerelease: false
-  requirement: &id005 !ruby/object:Gem::Requirement 
-    requirements: 
-    - - ">="
-      - !ruby/object:Gem::Version 
-        segments: 
-        - 0
-        version: "0"
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+    none: false
+- !ruby/object:Gem::Dependency
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+    none: false
+  name: mocha
   type: :development
-  version_requirements: *id005
-- !ruby/object:Gem::Dependency 
-  name: sqlite3-ruby
   prerelease: false
-  requirement: &id006 !ruby/object:Gem::Requirement 
-    requirements: 
-    - - ">="
-      - !ruby/object:Gem::Version 
-        segments: 
-        - 0
-        version: "0"
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+    none: false
+- !ruby/object:Gem::Dependency
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+    none: false
+  name: sqlite3-ruby
   type: :development
-  version_requirements: *id006
+  prerelease: false
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+    none: false
 description: Rendering spreadsheets from Rails made easy
 email: glex.spb@gmail.com
 executables: []
-
 extensions: []
-
-extra_rdoc_files: 
+extra_rdoc_files:
 - README.textile
-files: 
+files:
 - README.textile
 - LICENSE
 - Rakefile
-- lib/to_spreadsheet/action_pack_renderers.rb
 - lib/to_spreadsheet/context/pairing.rb
 - lib/to_spreadsheet/context.rb
-- lib/to_spreadsheet/helpers.rb
-- lib/to_spreadsheet/mime_types.rb
+- lib/to_spreadsheet/rails/action_pack_renderers.rb
+- lib/to_spreadsheet/rails/mime_types.rb
+- lib/to_spreadsheet/rails/view_helpers.rb
+- lib/to_spreadsheet/railtie.rb
 - lib/to_spreadsheet/renderer.rb
 - lib/to_spreadsheet/rule/base.rb
 - lib/to_spreadsheet/rule/container.rb
@@ -119,37 +136,35 @@ files:
 - lib/to_spreadsheet/type_from_value.rb
 - lib/to_spreadsheet/version.rb
 - lib/to_spreadsheet.rb
-has_rdoc: true
+- spec/defaults_spec.rb
+- spec/format_spec.rb
+- spec/types_spec.rb
+- spec/worksheets_spec.rb
 homepage: https://github.com/glebm/to_spreadsheet
 licenses: []
-
 post_install_message: 
 rdoc_options: []
-
-require_paths: 
+require_paths:
 - lib
-required_ruby_version: !ruby/object:Gem::Requirement 
-  requirements: 
-  - - ">="
-    - !ruby/object:Gem::Version 
-      segments: 
-      - 0
-      version: "0"
-required_rubygems_version: !ruby/object:Gem::Requirement 
-  requirements: 
-  - - ">="
-    - !ruby/object:Gem::Version 
-      segments: 
-      - 0
-      version: "0"
+required_ruby_version: !ruby/object:Gem::Requirement
+  requirements:
+  - - ! '>='
+    - !ruby/object:Gem::Version
+      version: '0'
+  none: false
+required_rubygems_version: !ruby/object:Gem::Requirement
+  requirements:
+  - - ! '>='
+    - !ruby/object:Gem::Version
+      version: '0'
+  none: false
 requirements: []
-
 rubyforge_project: 
-rubygems_version: 1.3.6
+rubygems_version: 1.8.24
 signing_key: 
 specification_version: 3
 summary: Render existing views as Excel documents with style!
-test_files: 
+test_files:
 - spec/defaults_spec.rb
 - spec/format_spec.rb
 - spec/types_spec.rb

data/lib/to_spreadsheet/helpers.rb

@@ -1,19 +0,0 @@
-module ToSpreadsheet
-  module Helpers
-    def format_xls(selector = nil, &block)
-      context.format_xls selector, &block
-    end
-
-    def context
-      @context || ToSpreadsheet::Context.global
-    end
-
-    def with_context(context, &block)
-      context_was = self.context
-      @context    = context
-      result      = block.call
-      @context    = context_was
-      result
-    end
-  end
-end