rails_table_for 0.2.0 → 0.2.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 7771134d7d3b6bbf57a19ca741f08c113e3f21cedffecf3aded0353ddc444663
-  data.tar.gz: faaf9c18db501ed6abccbca4d937b9fbd3f2e5343ab0a01a1549654c8970e93d
+  metadata.gz: bc39558936421adf2e8f1fe5cc4f79d404f423ad2a48ac96aed9b951de5cfb05
+  data.tar.gz: 2459ffa6e9167052e2a2f1e2fe02817c65eb2dafb9267cff706a0b7be9e4dd4b
 SHA512:
-  metadata.gz: 7e69b040c738f65d0133e65f6e3dbca8e19b0aed58267235cbfe949469b3fa06db8c8f77cde0935acd8cf875c62cc1694494c8714e21ad66137054db21e9555b
-  data.tar.gz: d51079e75e4d0c6d28d5c74a4a79cc1319ff4d1a02a01460f09fd0eb119c400d0dd7217a8ff9e0e8cc24e9a9635db71bd9b299f9eb9d20ce336505b039c44c9c
+  metadata.gz: 1ae6589f69b463b6d997b2e4a110c63bdbe682554612cdbc7611b0d665344c14c7f703de6b1f834ca3e8477c9ebb7e35eb762d861114153b6a5c242416f36d98
+  data.tar.gz: d6f88eb4659eeda945bfda4c9f23cd924c720052651125a7c7f5d2e9d0d44e2af4425d6d9109dc8c8036453616b57b62a8cedd05cf485522f6baf4a083f5e1a2

data/.github/workflows/{ruby.yml → tests.yml}

@@ -1,4 +1,4 @@
-name: Run Minitest
+name: Tests
 
 on:
   push:
@@ -17,7 +17,10 @@ jobs:
       uses: actions/setup-ruby@v1
       with:
         ruby-version: 2.6.x
-    - name: Build and test with Rake
+    - name: Install libsqlite3-dev
+      run: |
+        sudo apt-get install -y libsqlite3-dev
+    - name: Build and test
       run: |
         gem install bundler
         bundle install --jobs 4 --retry 3

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    rails_table_for (0.2.0)
+    rails_table_for (0.2.1)
       rails (>= 5)
 
 GEM
@@ -142,6 +142,7 @@ DEPENDENCIES
   bundler (~> 2.1, >= 2.1.4)
   byebug (~> 11.1, >= 11.1.1)
   minitest (~> 5.14)
+  nokogiri (~> 1.10, >= 1.10.9)
   rails_table_for!
   rake (~> 13.0, >= 13.0.1)
   sqlite3 (~> 1.4, >= 1.4.2)

data/docs/index.md

@@ -100,4 +100,18 @@ Assuming the route exists to show that record, this will generate HTML like this
     </tr>
   </tbody>
 </table>
-```
+```
+
+# Auto-linking columns
+
+Automatically get a link to the record represented by each row like this:
+
+```
+<%=
+table_for @records do |table|
+  table.column :id, auto_link: true
+end
+%>
+```
+
+If no route can be found for the record, a `NoMethodError`  will be thrown.

data/lib/rails_table_for/action_view_ext.rb

@@ -0,0 +1,19 @@
+require 'rails_table_for/elements/table'
+
+module TableHelper
+  include ActionView::Helpers::TagHelper
+
+  def table_for(records, **options)
+    def table_for(records, **options)
+      table = Elements::Table.new(options)
+      if block_given?
+        yield table
+      end
+      table.build(records)
+    end
+  end
+end
+
+ActionView::Base.class_eval do
+  include TableHelper
+end

data/lib/rails_table_for/elements/block_column.rb

@@ -0,0 +1,20 @@
+require 'rails_table_for/helpers/auto_link'
+require 'rails_table_for/elements/column'
+
+module Elements
+  class BlockColumn < Column
+    include Helpers::AutoLink
+    attr_reader :title
+
+    def initialize(block, **options)
+      @block = block
+      @title = options[:title]
+      @auto_link = options[:auto_link] || false
+    end
+
+    def value_for(record)
+      text = @block.call(record)
+      @auto_link ? auto_link(record, text) : text
+    end
+  end
+end

data/lib/rails_table_for/elements/column.rb

@@ -0,0 +1,7 @@
+module Elements
+  class Column
+    def value_for(record)
+      raise "Not implemented"
+    end
+  end
+end

data/lib/rails_table_for/elements/field_column.rb

@@ -0,0 +1,23 @@
+require 'rails_table_for/helpers/auto_link'
+require 'rails_table_for/elements/column'
+
+module Elements
+  class FieldColumn < Column
+    include Helpers::AutoLink
+    attr_reader :title
+
+    def initialize(field, **options)
+      if field.nil?
+        raise ArgumentError.new("Field cannot be nil")
+      end
+      @field = field
+      @title = options[:title] || field.to_s.humanize
+      @auto_link = options[:auto_link] || false
+    end
+
+    def value_for(record)
+      text = record.send(@field)
+      @auto_link ? auto_link(record, text) : text
+    end
+  end
+end

data/lib/rails_table_for/elements/table.rb

@@ -0,0 +1,63 @@
+require 'rails_table_for/elements/block_column'
+require 'rails_table_for/elements/field_column'
+
+module Elements
+  class Table
+    include ActionView::Helpers::TagHelper
+
+    attr_accessor :output_buffer
+
+    def initialize(**options)
+      columns = options[:columns] || []
+      @columns = columns.map do |field|
+        FieldColumn.new(field)
+      end
+      @options = options
+    end
+
+    def column(field=nil, **options, &block)
+      if field.nil? && !block_given?
+        raise 'Must provide either field or block'
+      end
+
+      if block_given?
+        @columns << BlockColumn.new(block, options)
+      else
+        @columns << FieldColumn.new(field, options)
+      end
+    end
+
+    def build(records)
+      return '' if records.nil? || records.empty?
+      return '' if @columns.nil? || @columns.empty?
+      table(records)
+    end
+
+    private
+    def table(records)
+      content_tag :table, class: @options[:class] do
+        [head, body(records)].join.html_safe
+      end
+    end
+
+    def head
+      content_tag :thead do
+        content_tag :tr do
+          @columns.map {|column| content_tag :th, column.title }.join.html_safe
+        end
+      end
+    end
+
+    def body(records)
+      content_tag :tbody do
+        records.map {|record| body_row(record) }.join.html_safe
+      end
+    end
+
+    def body_row(record)
+      content_tag :tr do
+        @columns.map {|column| content_tag :td, column.value_for(record) }.join.html_safe
+      end
+    end
+  end
+end

data/lib/rails_table_for/helpers/auto_link.rb

@@ -0,0 +1,10 @@
+module Helpers
+  module AutoLink
+    include ActionView::Helpers::UrlHelper
+
+    def auto_link(record, text)
+      path = Rails.application.routes.url_helpers.polymorphic_path(record)
+      link_to(text, path)
+    end
+  end
+end

data/lib/rails_table_for/version.rb

@@ -1,3 +1,3 @@
 module RailsTableFor
-  VERSION = '0.2.0'
+  VERSION = '0.2.1'
 end

data/lib/rails_table_for.rb

@@ -1,5 +1,5 @@
 require 'rails_table_for/railtie'
-require 'rails_table_for/table_helper'
+require 'rails_table_for/action_view_ext'
 
 module RailsTableFor
   # Your code goes here...

data/rails_table_for.gemspec

@@ -25,6 +25,7 @@ Gem::Specification.new do |spec|
   spec.add_development_dependency 'bundler', '~> 2.1', '>= 2.1.4'
   spec.add_development_dependency 'byebug', '~> 11.1', '>= 11.1.1'
   spec.add_development_dependency 'minitest', '~> 5.14'
+  spec.add_development_dependency 'nokogiri', '~> 1.10', '>= 1.10.9'
   spec.add_development_dependency 'rake', '~> 13.0', '>= 13.0.1'
   spec.add_development_dependency 'sqlite3', '~> 1.4', '>= 1.4.2'
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: rails_table_for
 version: !ruby/object:Gem::Version
-  version: 0.2.0
+  version: 0.2.1
 platform: ruby
 authors:
 - Austin Roos
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2020-03-25 00:00:00.000000000 Z
+date: 2020-03-28 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails
@@ -92,6 +92,26 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '5.14'
+- !ruby/object:Gem::Dependency
+  name: nokogiri
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '1.10'
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 1.10.9
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '1.10'
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 1.10.9
 - !ruby/object:Gem::Dependency
   name: rake
   requirement: !ruby/object:Gem::Requirement
@@ -142,7 +162,7 @@ extra_rdoc_files: []
 files:
 - ".github/ISSUE_TEMPLATE/bug_report.md"
 - ".github/ISSUE_TEMPLATE/new-user-story.md"
-- ".github/workflows/ruby.yml"
+- ".github/workflows/tests.yml"
 - ".gitignore"
 - Gemfile
 - Gemfile.lock
@@ -157,12 +177,13 @@ files:
 - docs/_config.yml
 - docs/index.md
 - lib/rails_table_for.rb
-- lib/rails_table_for/block_column.rb
-- lib/rails_table_for/column.rb
-- lib/rails_table_for/field_column.rb
+- lib/rails_table_for/action_view_ext.rb
+- lib/rails_table_for/elements/block_column.rb
+- lib/rails_table_for/elements/column.rb
+- lib/rails_table_for/elements/field_column.rb
+- lib/rails_table_for/elements/table.rb
+- lib/rails_table_for/helpers/auto_link.rb
 - lib/rails_table_for/railtie.rb
-- lib/rails_table_for/table.rb
-- lib/rails_table_for/table_helper.rb
 - lib/rails_table_for/version.rb
 - lib/tasks/rails_table_for_tasks.rake
 - rails_table_for.gemspec

data/lib/rails_table_for/block_column.rb

@@ -1,14 +0,0 @@
-require 'rails_table_for/column'
-
-class BlockColumn < Column
-  attr_reader :title
-
-  def initialize(block, title)
-    @block = block
-    @title = title
-  end
-
-  def value_for(record)
-    @block.call(record)
-  end
-end

data/lib/rails_table_for/column.rb

@@ -1,5 +0,0 @@
-class Column
-  def value_for(record)
-    raise "Not implemented"
-  end
-end

data/lib/rails_table_for/field_column.rb

@@ -1,17 +0,0 @@
-require 'rails_table_for/column'
-
-class FieldColumn < Column
-  attr_reader :title
-
-  def initialize(field, title = nil)
-    if field.nil?
-      raise ArgumentError.new("Field cannot be nil")
-    end
-    @field = field
-    @title = title || field.to_s.humanize
-  end
-
-  def value_for(record)
-    record.send(@field)
-  end
-end

data/lib/rails_table_for/table.rb

@@ -1,61 +0,0 @@
-require 'rails_table_for/block_column'
-require 'rails_table_for/field_column'
-
-class Table
-  include ActionView::Helpers::TagHelper
-
-  attr_accessor :output_buffer
-
-  def initialize(**options)
-    columns = options[:columns] || []
-    @columns = columns.map do |field|
-      FieldColumn.new(field)
-    end
-    @options = options
-  end
-
-  def column(field=nil, **options, &block)
-    if field.nil? && !block_given?
-      raise 'Must provide either field or block'
-    end
-    title = options[:title] || options['title']
-
-    if block_given?
-      @columns << BlockColumn.new(block, title)
-    else
-      @columns << FieldColumn.new(field, title)
-    end
-  end
-
-  def build(records)
-    return '' if records.nil? || records.empty?
-    table(records)
-  end
-
-  private
-  def table(records)
-    content_tag :table, class: @options[:class] do
-      [head, body(records)].join.html_safe
-    end
-  end
-
-  def head
-    content_tag :thead do
-      content_tag :tr do
-        @columns.map {|column| content_tag :th, column.title }.join.html_safe
-      end
-    end
-  end
-
-  def body(records)
-    content_tag :tbody do
-      records.map {|record| body_row(record) }.join.html_safe
-    end
-  end
-
-  def body_row(record)
-    content_tag :tr do
-      @columns.map {|column| content_tag :td, column.value_for(record) }.join.html_safe
-    end
-  end
-end

data/lib/rails_table_for/table_helper.rb

@@ -1,17 +0,0 @@
-require 'rails_table_for/table'
-
-module TableHelper
-  include ActionView::Helpers::TagHelper
-
-  def table_for(records, **options)
-    table = Table.new(options)
-    if block_given?
-      yield table
-    end
-    table.build(records)
-  end
-end
-
-ActionView::Base.class_eval do
-  include TableHelper
-end