erector_to_fortitude 0.0.1

checksums.yaml

@@ -0,0 +1,7 @@
+---
+SHA1:
+  metadata.gz: f9cded875e3387ed11fd00281a2936de45d37374
+  data.tar.gz: 6f2487e34aed2cbbd99a7ac3b8f4ccbd5f6ca77b
+SHA512:
+  metadata.gz: 6db22dda91469c94c42f5a784aaecfd67d378b413f9ca6b6cbf28e156362014032d5272544495e1f29d72d68d8fc4ebc3d163c94059d9597daa9d85c17834ea8
+  data.tar.gz: 00c9c44ffac2af7f6aab976df3622d48e959b59f782783402039f6270ba2bf5947457dca35c7af8e5d9aa5f17d794902a1d00c95eb2816ad4c18794f2eb6e65e

data/.gitignore

@@ -0,0 +1 @@
+Gemfile.lock

data/Gemfile

@@ -0,0 +1,2 @@
+source 'https://rubygems.org'
+gemspec

data/Guardfile

@@ -0,0 +1,4 @@
+guard :rspec, cmd: 'bundle exec rspec', failed_mode: :focus do
+  watch(%r{^spec/.+_spec\.rb$})
+  watch(%r{^lib/(.+)\.rb$})         { |m| "spec/lib/erector_to_fortitude/converter_spec.rb" }
+end

data/README.md

@@ -0,0 +1,14 @@
+Erector to Fortitude
+----
+
+Converts Erector views to Fortitude syntax. In practice, this means assigning IDs and classes in the options hash: https://github.com/ageweke/fortitude/blob/master/README-erector.md#what-doesnt-it-support
+
+## Usage
+
+```
+bin/erector_to_fortitude path/to/views
+```
+
+## Known issues
+
+- Can't transform the contents of a proc that's the value of a hash (!??)

data/bin/erector_to_fortitude

@@ -0,0 +1,34 @@
+#!/usr/bin/env ruby
+
+here = File.expand_path(File.dirname(__FILE__))
+
+if File.directory?("#{here}/../lib/erector_to_fortitude")
+  $:.unshift "#{here}/../lib"
+end
+
+require 'erector_to_fortitude'
+
+filepaths = []
+
+ARGV.each do |arg|
+  if File.directory?(arg)
+    arg = arg.gsub(/\/$/, '')
+    Dir["#{arg}/**/*.rb"].each do |file|
+      filepaths << file
+    end
+  elsif File.exist?(arg)
+    filepaths << arg
+  else
+    puts "Didn't know how to parse the argument \"#{arg}\""
+  end
+end
+
+puts "Converting #{filepaths.length} files"
+
+filepaths.each do |fp|
+  puts "Converting #{fp}..."
+  new_code = ErectorToFortitude::Converter.convert(File.read(fp))
+  File.open(fp, 'w') { |f| f.write(new_code) }
+end
+
+puts 'Done!'

data/erector_to_fortitude.gemspec

@@ -0,0 +1,25 @@
+$:.push File.expand_path("../lib", __FILE__)
+
+# Maintain your gem's version:
+require 'erector_to_fortitude/version'
+
+# Describe your gem and declare its dependencies:
+Gem::Specification.new do |s|
+  s.name = 'erector_to_fortitude'
+  s.version = ErectorToFortitude::VERSION
+  s.required_ruby_version = Gem::Requirement.new('>= 2.0.0')
+  s.authors = ['Adam Becker']
+  s.summary = 'Converts Erector views to Fortitude syntax.'
+  s.email = 'adam@dobt.co'
+  s.license = 'MIT'
+  s.files = `git ls-files`.split("\n")
+  s.test_files = `git ls-files -- {features,spec}/*`.split("\n")
+  s.homepage = 'http://github.com/ajb/erector_to_fortitude'
+
+  s.add_dependency 'parser'
+
+  s.add_development_dependency 'pry'
+  s.add_development_dependency 'rspec'
+  s.add_development_dependency 'guard'
+  s.add_development_dependency 'guard-rspec'
+end

data/lib/erector_to_fortitude/converter.rb

@@ -0,0 +1,13 @@
+module ErectorToFortitude
+  class Converter
+    def self.convert(code)
+      buffer = Parser::Source::Buffer.new('(example)')
+      buffer.source = code
+      parser = Parser::CurrentRuby.new
+      ast = parser.parse(buffer)
+      rewriter = Rewriter.new
+
+      rewriter.rewrite(buffer, ast)
+    end
+  end
+end

data/lib/erector_to_fortitude/html_tags.rb

@@ -0,0 +1,115 @@
+module ErectorToFortitude
+  # From https://github.com/erector/erector/blob/37d38613fd1f485988150383a29db6a72a642d42/lib/erector/html_widget.rb#L66
+  HTML_TAGS = %w(
+    area
+    base
+    br
+    col
+    embed
+    frame
+    hr
+    img
+    input
+    link
+    meta
+    param
+    a
+    abbr
+    acronym
+    address
+    article
+    aside
+    audio
+    b
+    bdo
+    big
+    blockquote
+    body
+    button
+    canvas
+    caption
+    center
+    cite
+    code
+    colgroup
+    command
+    datalist
+    dd
+    del
+    details
+    dfn
+    dialog
+    div
+    dl
+    dt
+    em
+    fieldset
+    figure
+    footer
+    form
+    frameset
+    h1
+    h2
+    h3
+    h4
+    h5
+    h6
+    head
+    header
+    hgroup
+    html
+    i
+    iframe
+    ins
+    keygen
+    kbd
+    label
+    legend
+    li
+    map
+    mark
+    meter
+    nav
+    noframes
+    noscript
+    object
+    ol
+    optgroup
+    option
+    p
+    pre
+    progress
+    q
+    ruby
+    rt
+    rp
+    s
+    samp
+    script
+    section
+    select
+    small
+    source
+    span
+    strike
+    strong
+    style
+    sub
+    sup
+    table
+    tbody
+    td
+    textarea
+    tfoot
+    th
+    thead
+    time
+    title
+    tr
+    tt
+    u
+    ul
+    var
+    video
+  )
+end

data/lib/erector_to_fortitude/rewriter.rb

@@ -0,0 +1,108 @@
+require 'pry'
+
+module ErectorToFortitude
+  class Rewriter < Parser::Rewriter
+    def on_send(node)
+      # Why is this necessary?
+      return unless node.children.length > 1
+
+      # How do I explain this? lol
+      return unless node.children[0]
+
+      tag_node = get_first_node(node.children[0])
+
+      # The tag method is an instance method
+      return unless tag_node.children[0] == nil
+
+      # Validate tag name
+      return unless ErectorToFortitude::HTML_TAGS.include?(tag_node.children[1].to_s)
+
+      ids = []
+      classes = []
+
+      get_classes_and_ids_sent(node).each do |class_or_id|
+        if class_or_id[-1] == '!'
+          ids << class_or_id[0..-2]
+        else
+          classes << class_or_id
+        end
+      end
+
+      new_opts = []
+
+      if classes.length > 0
+        new_opts << %{class: '#{classes.join(' ')}'}
+      end
+
+      if ids.length > 0
+        new_opts << %{id: '#{ids.join(' ')}'}
+      end
+
+      new_opt_str = new_opts.join(', ')
+
+      # Replace the existing dots
+      get_sends(node).each do |child|
+        replace child.loc.dot, ''
+        replace child.loc.selector, ''
+      end
+
+      if (text_node = get_node_after_classes_and_ids(node)) && text_node.is_a?(Parser::AST::Node) && text_node.type != :hash
+
+        insert_after text_node.loc.expression, ', ' + new_opt_str
+
+      # Add class to existing hash
+      elsif node.children[2] && node.children[2].type == :hash
+        insert_before node.children[2].children[0].loc.expression, new_opt_str + ', '
+
+      # Add a new options hash
+      else
+        # Remove the dot
+        insert_after node.loc.selector, "(#{new_opt_str})"
+      end
+    end
+
+    def get_first_node(node)
+      while node.children[0].is_a?(Parser::AST::Node) && node.children[0].type == :send
+        node = node.children[0]
+      end
+
+      node
+    end
+
+    def get_sends(node)
+      arr = []
+
+      while node.children[0] && node.children[0].type == :send
+        arr << node
+        node = node.children[0]
+      end
+
+      arr
+    end
+
+    def get_classes_and_ids_sent(node)
+      arr = []
+
+      while node.children[0] && node.children[0].type == :send
+        arr << node.children[1]
+        node = node.children[0]
+      end
+
+      arr.reverse
+    end
+
+    def get_node_after_classes_and_ids(node)
+      found_classes_and_ids = false
+
+      node.children.each do |child|
+        if child.is_a?(Symbol)
+          found_classes_and_ids = true
+        end
+
+        if found_classes_and_ids && !child.is_a?(Symbol)
+          return child
+        end
+      end
+    end
+  end
+end

data/lib/erector_to_fortitude/version.rb

@@ -0,0 +1,3 @@
+module ErectorToFortitude
+  VERSION = '0.0.1'
+end

data/lib/erector_to_fortitude.rb

@@ -0,0 +1,8 @@
+module ErectorToFortitude
+end
+
+require 'parser'
+require 'parser/current'
+require 'erector_to_fortitude/converter'
+require 'erector_to_fortitude/rewriter'
+require 'erector_to_fortitude/html_tags'

data/script/cibuild

@@ -0,0 +1,3 @@
+#!/bin/sh
+
+bundle exec rspec

data/script/release

@@ -0,0 +1,38 @@
+#!/bin/sh
+# Tag and push a release.
+
+set -e
+
+# Make sure we're in the project root.
+
+cd $(dirname "$0")/..
+
+# Build a new gem archive.
+
+rm -rf erector_to_fortitude-*.gem
+gem build -q erector_to_fortitude.gemspec
+
+# Make sure we're on the master branch.
+
+(git branch | grep -q '* master') || {
+  echo "Only release from the master branch."
+  exit 1
+}
+
+# Figure out what version we're releasing.
+
+tag=v`ls erector_to_fortitude-*.gem | sed 's/^erector_to_fortitude-\(.*\)\.gem$/\1/'`
+
+# Make sure we haven't released this version before.
+
+git fetch -t origin
+
+(git tag -l | grep -q "$tag") && {
+  echo "Whoops, there's already a '${tag}' tag."
+  exit 1
+}
+
+# Tag it and bag it.
+
+gem push erector_to_fortitude-*.gem && git tag "$tag" &&
+  git push origin master && git push origin "$tag"

data/spec/lib/erector_to_fortitude/converter_spec.rb

@@ -0,0 +1,232 @@
+require 'spec_helper'
+
+RSpec.describe ErectorToFortitude::Converter do
+  it 'converts a basic class send' do
+    input = <<END
+  div.foo {
+    text 'hi'
+  }
+END
+
+    output = code = <<END
+  div(class: 'foo') {
+    text 'hi'
+  }
+END
+
+    expect(described_class.convert(input)).to eq output
+  end
+
+  it 'converts multiple classes' do
+    input = <<END
+  div.foo.bar.baz {
+    text 'hi'
+  }
+END
+
+    output = code = <<END
+  div(class: 'foo bar baz') {
+    text 'hi'
+  }
+END
+
+    expect(described_class.convert(input)).to eq output
+  end
+
+  it 'converts on non-div tags' do
+    input = <<END
+  span.foo {
+    text 'hi'
+  }
+END
+
+    output = code = <<END
+  span(class: 'foo') {
+    text 'hi'
+  }
+END
+
+    expect(described_class.convert(input)).to eq output
+  end
+
+  it 'ignores non-erector stuffs' do
+    input = <<END
+  span {
+    foos = Foo.with_deleted.all
+    text foos.count
+    bar.baz!
+  }
+END
+
+    output = code = <<END
+  span {
+    foos = Foo.with_deleted.all
+    text foos.count
+    bar.baz!
+  }
+END
+
+    expect(described_class.convert(input)).to eq output
+  end
+
+  it 'converts a basic id send' do
+    input = <<END
+  div.foo! {
+    text 'hi'
+  }
+END
+
+    output = code = <<END
+  div(id: 'foo') {
+    text 'hi'
+  }
+END
+
+    expect(described_class.convert(input)).to eq output
+  end
+
+  it 'handles existing hash options' do
+    input = <<END
+  div.foo('data-bar' => 'baz') {
+    text 'hi'
+  }
+END
+
+    output = code = <<END
+  div(class: 'foo', 'data-bar' => 'baz') {
+    text 'hi'
+  }
+END
+
+    expect(described_class.convert(input)).to eq output
+  end
+
+  it 'handles complex existing hash options' do
+    input = <<END
+  div.foo.bar.baz.booz!('data-bar' => 'baz') {
+    text 'hi'
+  }
+END
+
+    output = code = <<END
+  div(class: 'foo bar baz', id: 'booz', 'data-bar' => 'baz') {
+    text 'hi'
+  }
+END
+
+    expect(described_class.convert(input)).to eq output
+  end
+
+  it 'handles a text argument' do
+    input = <<END
+  span.foo 'Bar'
+END
+
+    output = code = <<END
+  span 'Bar', class: 'foo'
+END
+
+    expect(described_class.convert(input)).to eq output
+  end
+
+  it 'handles a text argument with existing hash' do
+    input = <<END
+  span.foo 'bar', 'data-baz' => 'booz'
+END
+
+    output = code = <<END
+  span 'bar', class: 'foo', 'data-baz' => 'booz'
+END
+
+    expect(described_class.convert(input)).to eq output
+  end
+
+  it 'handles a dynamic text argument' do
+    input = <<END
+  span.foo x, 'data-baz' => 'booz'
+END
+
+    output = code = <<END
+  span x, class: 'foo', 'data-baz' => 'booz'
+END
+
+    expect(described_class.convert(input)).to eq output
+  end
+
+  it 'handles a complex dynamic text argument' do
+    input = <<END
+  span.foo x.z.y(yeah: 'man'), 'data-baz' => 'booz'
+END
+
+    output = code = <<END
+  span x.z.y(yeah: 'man'), class: 'foo', 'data-baz' => 'booz'
+END
+
+    expect(described_class.convert(input)).to eq output
+  end
+
+  it 'handles a text argument inside another node' do
+    input = <<END
+  div.bar {
+    span.foo x
+  }
+END
+
+    output = code = <<END
+  div(class: 'bar') {
+    span x, class: 'foo'
+  }
+END
+
+    expect(described_class.convert(input)).to eq output
+  end
+
+  it 'handles empty tags' do
+    input = <<END
+  div.bar
+END
+
+    output = code = <<END
+  div(class: 'bar')
+END
+
+    expect(described_class.convert(input)).to eq output
+  end
+
+  it 'handles code within an inline widget' do
+    input = <<END
+  Erector.inline {
+    div.foo 'bar'
+  }
+END
+
+    output = code = <<END
+  Erector.inline {
+    div 'bar', class: 'foo'
+  }
+END
+
+    expect(described_class.convert(input)).to eq output
+  end
+
+  # known issue
+  xit 'handles code inside proc values' do
+    input = <<END
+bar({
+  proc: -> {
+    a.button.primary 'hi'
+  }
+})
+END
+
+  output = code = <<END
+bar({
+  proc: -> {
+    a 'hi', class: 'button primary'
+  }
+})
+END
+
+    expect(described_class.convert(input)).to eq output
+  end
+end

data/spec/spec_helper.rb

@@ -0,0 +1 @@
+require 'erector_to_fortitude'

metadata

@@ -0,0 +1,131 @@
+--- !ruby/object:Gem::Specification
+name: erector_to_fortitude
+version: !ruby/object:Gem::Version
+  version: 0.0.1
+platform: ruby
+authors:
+- Adam Becker
+autorequire: 
+bindir: bin
+cert_chain: []
+date: 2015-08-03 00:00:00.000000000 Z
+dependencies:
+- !ruby/object:Gem::Dependency
+  name: parser
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: pry
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: rspec
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: guard
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: guard-rspec
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+description: 
+email: adam@dobt.co
+executables: []
+extensions: []
+extra_rdoc_files: []
+files:
+- ".gitignore"
+- Gemfile
+- Guardfile
+- README.md
+- bin/erector_to_fortitude
+- erector_to_fortitude.gemspec
+- lib/erector_to_fortitude.rb
+- lib/erector_to_fortitude/converter.rb
+- lib/erector_to_fortitude/html_tags.rb
+- lib/erector_to_fortitude/rewriter.rb
+- lib/erector_to_fortitude/version.rb
+- script/cibuild
+- script/release
+- spec/lib/erector_to_fortitude/converter_spec.rb
+- spec/spec_helper.rb
+homepage: http://github.com/ajb/erector_to_fortitude
+licenses:
+- MIT
+metadata: {}
+post_install_message: 
+rdoc_options: []
+require_paths:
+- lib
+required_ruby_version: !ruby/object:Gem::Requirement
+  requirements:
+  - - ">="
+    - !ruby/object:Gem::Version
+      version: 2.0.0
+required_rubygems_version: !ruby/object:Gem::Requirement
+  requirements:
+  - - ">="
+    - !ruby/object:Gem::Version
+      version: '0'
+requirements: []
+rubyforge_project: 
+rubygems_version: 2.2.2
+signing_key: 
+specification_version: 4
+summary: Converts Erector views to Fortitude syntax.
+test_files:
+- spec/lib/erector_to_fortitude/converter_spec.rb
+- spec/spec_helper.rb
+has_rdoc: