lm_docstache 1.1.0 → 1.2.2.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: f8d96c1d658331a8ef93e7ace164759a64e26f9fc6f579ccab6aaf37032ad6fd
-  data.tar.gz: abaf62fe5a057c4adc66cbf2ed57ec87c75b55d349259953b02bba568b55dd35
+  metadata.gz: 10a795ca947b082eca640e84db42ca03588902b010c4e564599dd13a61a2ba6b
+  data.tar.gz: d18772ec72ef30d8ac01fe790456bc881167c6908664b11c0d73b3d419a05c79
 SHA512:
-  metadata.gz: e1b98280644b99d84502cf026887d6be64b038950afe96cefb97ce5364de9b7b760c577ea052a377e4a3e7cf1af6b743a1d3d8fcbe8837b7d8a47d83b75f7625
-  data.tar.gz: 694650b1c8b433244cfc7ea891de75d77ab70816aaa69442faee72f8efc3c8de71c9201d096c24fbdcf8a02ed32b47fa471a08d2a890be11c60380d5011f1d94
+  metadata.gz: 3d7314dead61fe490028624a2900ea55095e6e95d0ebf6ac6487e2f42f624702644eade733208d27d3a1f1166e24baa591e646f8cf963c56887c2b43a2743d83
+  data.tar.gz: 9c030a05bbf6ea2ecc96163520690813594b1345a56f85e7f554ef2201647d7ee2d3859a8e620da7afb86a0e94fe15507dee3dbe1ca6877b413ec1212267720d

data/.github/workflows/ruby.yml

@@ -0,0 +1,29 @@
+# This workflow uses actions that are not certified by GitHub.
+# They are provided by a third-party and are governed by
+# separate terms of service, privacy policy, and support
+# documentation.
+# This workflow will download a prebuilt Ruby version, install dependencies and run tests with Rake
+# For more information see: https://github.com/marketplace/actions/setup-ruby-jruby-and-truffleruby
+
+name: Ruby
+
+on: push
+
+jobs:
+  test:
+
+    runs-on: ubuntu-latest
+
+    steps:
+    - uses: actions/checkout@v2
+    - name: Set up Ruby
+    # To automatically get bug fixes and new Ruby versions for ruby/setup-ruby,
+    # change this to (see https://github.com/ruby/setup-ruby#versioning):
+    # uses: ruby/setup-ruby@v1
+      uses: ruby/setup-ruby@ec106b438a1ff6ff109590de34ddc62c540232e0
+      with:
+        ruby-version: 2.6
+    - name: Install dependencies
+      run: bundle install
+    - name: Run tests
+      run: rspec

data/CHANGELOG.md

@@ -1,7 +1,17 @@
 # Changelog
 
+## 1.2.2
+* Remove uneccessary runtime dependency on active support.
+
+## 1.2.1
+* Fix inline conditional issues such as no support for multi conditionals.
+
+## 1.2.0
+* Add support for inline conditionals.
+* Add some tests.
+
 ## 1.0.0
-  Fork the Docstache codebase and rename all references to LMDocstache.
+* Fork the Docstache codebase and rename all references to LMDocstache.
 
 ## 0.3.2
 * [b3b66a0cd7ae67834cdbe7c18cefdb1498bcc5ab] I'm an idiot.  Fixed an error in the `Document#unusable_tags` method

data/lib/lm_docstache/block.rb

@@ -1,7 +1,7 @@
 module LMDocstache
   class Block
-    attr_reader :name, :opening_element, :content_elements, :closing_element, :inverted, :condition
-    def initialize(name:, data:, opening_element:, content_elements:, closing_element:, inverted:, condition: nil)
+    attr_reader :name, :opening_element, :content_elements, :closing_element, :inverted, :condition, :inline
+    def initialize(name:, data:, opening_element:, content_elements:, closing_element:, inverted:, condition: nil, inline: false)
       @name = name
       @data = data
       @opening_element = opening_element
@@ -9,6 +9,7 @@ module LMDocstache
       @closing_element = closing_element
       @inverted = inverted
       @condition = condition
+      @inline = inline
     end
 
     def type
@@ -31,7 +32,7 @@ module LMDocstache
       type == :conditional
     end
 
-    def self.find_all(name:, data:, elements:, inverted:, condition: nil, ignore_missing: true)
+    def self.find_all(name:, data:, elements:, inverted:, condition: nil, ignore_missing: true, child: false)
       inverted_op = inverted ? '\^' : '\#'
       full_tag_regex = /\{\{#{inverted_op}(#{name})\s?#{condition}\}\}.+?\{\{\/\k<1>\}\}/m
       start_tag_regex = /\{\{#{inverted_op}#{name}\s?#{condition}\}\}/m
@@ -41,7 +42,11 @@ module LMDocstache
         if elements.any? { |e| e.text.match(full_tag_regex) }
           matches = elements.select { |e| e.text.match(full_tag_regex) }
           return matches.flat_map do |match|
-            find_all(name: name, data: data, elements: match.elements, inverted: inverted, condition: condition)
+            if match.elements.any?
+              find_all(name: name, data: data, elements: match.elements, inverted: inverted, condition: condition, child: true)
+            else
+              extract_block_from_element(name, data, match, inverted, condition)
+            end
           end
         else
           opening = elements.find { |e| e.text.match(start_tag_regex) }
@@ -58,5 +63,9 @@ module LMDocstache
         raise "Block not found in given elements" unless ignore_missing
       end
     end
+
+    def self.extract_block_from_element(name, data, element, inverted, condition)
+      return Block.new(name: name, data: data, opening_element: element.parent.previous, content_elements: [element.parent], closing_element: element.parent.next, inverted: inverted, condition: condition, inline: true)
+    end
   end
 end

data/lib/lm_docstache/document.rb

@@ -68,6 +68,16 @@ module LMDocstache
       File.open(output, "w") { |f| f.write buffer.string }
     end
 
+    def render_replace(output, text)
+      rendered_documents = Hash[
+        @documents.map do |(path, document)|
+          [path, LMDocstache::Renderer.new(document.dup, {}).render_replace(text)]
+        end
+      ]
+      buffer = zip_buffer(rendered_documents)
+      File.open(output, "w") { |f| f.write buffer.string }
+    end
+
     def render_stream(data={})
       rendered_documents = Hash[
         @documents.map do |(path, document)|
@@ -79,6 +89,16 @@ module LMDocstache
       return buffer.sysread
     end
 
+    def render_xml(data={})
+      rendered_documents = Hash[
+        @documents.map do |(path, document)|
+          [path, LMDocstache::Renderer.new(document.dup, data).render]
+        end
+      ]
+
+      rendered_documents
+    end
+
     private
 
     def problem_paragraphs

data/lib/lm_docstache/renderer.rb

@@ -13,6 +13,16 @@ module LMDocstache
       return @content
     end
 
+    def render_replace(text)
+      @content.css('w|t').each do |text_el|
+        rendered_string = text_el.text
+        if !(results = rendered_string.scan(/\|-Lawmatics Test-\|/)).empty?
+          text_el.content = text
+        end
+      end
+      return @content
+    end
+
     private
 
     def find_and_expand_blocks
@@ -22,20 +32,18 @@ module LMDocstache
         Block.find_all(name: block[1], elements: @content.elements, data: @data, inverted: inverted, condition: block[2])
       end
       found_blocks.each do |block|
-        expand_and_replace_block(block) if block.present?
+        if block.inline
+          replace_conditionals(block)
+        else
+          expand_and_replace_block(block) if block.present?
+        end
       end
     end
 
     def expand_and_replace_block(block)
       case block.type
       when :conditional
-        case condition = @data.get(block.name, condition: block.condition)
-        when Array
-          condition = !condition.empty?
-        else
-          condition = !!condition
-        end
-        condition = !condition if block.inverted
+        condition = get_condition(block.name, block.condition, block.inverted)
         unless condition
           block.content_elements.each(&:unlink)
         end
@@ -60,6 +68,42 @@ module LMDocstache
       block.closing_element.unlink
     end
 
+    def replace_conditionals(block)
+      @content.css('w|t').each do |text_el|
+        rendered_string = text_el.text
+
+        if !(results = rendered_string.scan(/{{#(.*?)}}(.*?){{\/(.*?)}}/)).empty?
+          results.each do |r|
+            vals = r[0].split('==')
+            condition = get_condition(vals[0].strip, "== #{vals[1]}")
+            if condition
+              rendered_string.sub!("{{##{r[0]}}}", "")
+              rendered_string.sub!("{{/#{r[2]}}}", "")
+            else
+              rendered_string.sub!("{{##{r[0]}}}#{r[1]}{{/#{r[2]}}}", "")
+            end
+          end
+        end
+
+        # the only difference in this code block is caret instead of pound in three places,
+        # the inverted value passed to get_condition, and the condition being inverted. maybe combine them?
+        if !(results = rendered_string.scan(/{{\^(.*?)}}(.*?){{\/(.*?)}}/)).empty?
+          results.each do |r|
+            vals = r[0].split('==')
+            condition = get_condition(vals[0].strip, "== #{vals[1]}", true)
+            if condition
+              rendered_string.sub!("{{^#{r[0]}}}", "")
+              rendered_string.sub!("{{/#{r[2]}}}", "")
+            else
+              rendered_string.sub!("{{^#{r[0]}}}#{r[1]}{{/#{r[2]}}}", "")
+            end
+          end
+        end
+
+        text_el.content = rendered_string
+      end
+    end
+
     def replace_tags(elements, data)
       elements.css('w|t').each do |text_el|
         if !(results = text_el.text.scan(/\{\{([\w\.]+)\}\}/).flatten).empty?
@@ -72,5 +116,19 @@ module LMDocstache
       end
       return elements
     end
+
+    private
+
+    def get_condition(name, condition, inverted = false)
+      case condition = @data.get(name, condition: condition)
+      when Array
+        condition = !condition.empty?
+      else
+        condition = !!condition
+      end
+      condition = !condition if inverted
+
+      condition
+    end
   end
 end

data/lib/lm_docstache/version.rb

@@ -1,3 +1,3 @@
 module LMDocstache
-  VERSION = "1.1.0"
+  VERSION = "1.2.2.1"
 end

data/lm_docstache.gemspec

@@ -5,8 +5,8 @@ require "lm_docstache/version"
 Gem::Specification.new do |s|
   s.name        = "lm_docstache"
   s.version     = LMDocstache::VERSION
-  s.authors     = ["Roey Chasman", "Will Cosgrove"]
-  s.email       = ["roey@lawmatics.com", "will@willcosgrove.com"]
+  s.authors     = ["Roey Chasman", "Jonathan Stevens", "Will Cosgrove"]
+  s.email       = ["roey@lawmatics.com", "jonathan@lawmatics.com", "will@willcosgrove.com"]
   s.homepage    = "https://www.lawmatics.com"
   s.summary     = %q{Merges Hash of Data into Word docx template files using mustache syntax}
   s.description = %q{Integrates data into MS Word docx template files. Processing supports loops and replacement of strings of data both outside and within loops.}
@@ -20,6 +20,7 @@ Gem::Specification.new do |s|
   s.add_runtime_dependency 'nokogiri', '~> 1.6'
   s.add_runtime_dependency 'rubyzip', '~> 1.1'
 
-  s.add_development_dependency 'rspec', '>= 3.1.0'
-  s.add_development_dependency 'pry-byebug', '>= 1'
+  s.add_development_dependency 'rspec', '~> 3.1', '>= 3.1.0'
+  s.add_development_dependency 'pry-byebug', '~> 1'
+  s.add_development_dependency 'activesupport', '~> 1'
 end

data/spec/conditional_block_spec.rb

@@ -0,0 +1,101 @@
+require 'spec_helper'
+require 'nokogiri'
+
+module LMDocstache
+  module BlockTestData
+    DATA = {
+      gender: 'Male',
+      num: '2',
+      first_name: 'Hector',
+      last_name: 'Jones'
+    }
+  end
+end
+
+describe LMDocstache::Renderer do
+  let(:data) { Marshal.load(Marshal.dump(LMDocstache::BlockTestData::DATA)) } # deep copy
+  let(:base_path) { SPEC_BASE_PATH.join('example_input') }
+  let(:blank_doc_path) { "#{base_path}/blank.docx" }
+  let(:blank_doc) { LMDocstache::Document.new(blank_doc_path) }
+  let(:output_dir) { "#{base_path}/tmp" }
+  let(:temp_file) { "#{output_dir}/temp.docx" }
+  let(:result_file) { "#{output_dir}/result.docx" }
+
+  def render_docx(doc_text)
+    # create doc from blank
+    blank_doc.render_replace(temp_file, doc_text)
+
+    doc = LMDocstache::Document.new(temp_file).render_file(result_file, data)
+
+    result_doc = LMDocstache::Document.new(result_file).render_xml(data)
+    result_doc["word/document.xml"].text
+  end
+
+  before do
+    FileUtils.rm_rf(output_dir) if File.exist?(output_dir)
+    Dir.mkdir(output_dir)
+  end
+
+  after do
+    File.delete(temp_file)
+    File.delete(result_file)
+  end
+
+  it 'should handle inline conditional tags' do
+    result_text = render_docx("Refer to the matter as {{#gender == Male}}he{{/gender}}{{^gender == Male}}she{{/gender}} please")
+    expected_text = "Refer to the matter as he please"
+    expect(result_text).to eq(expected_text)
+  end
+
+  it 'should handle else statements with inline conditional tags' do
+    result_text = render_docx("Refer to the matter as {{#gender == 'Female'}}he{{/gender}}{{^gender == 'Female'}}she{{/gender}} please")
+    expected_text = "Refer to the matter as she please"
+    expect(result_text).to eq(expected_text)
+  end
+
+  it 'should handle inline conditional tags with no matches' do
+    result_text = render_docx("Refer to the matter as {{#gender == 'none'}}he{{/gender}} please")
+    expected_text = "Refer to the matter as  please"
+    expect(result_text).to eq(expected_text)
+  end
+
+  it 'should handle inline conditional tags with tags inside' do
+    result_text = render_docx("Refer to the matter as {{#gender == 'Male'}}{{first_name}}{{/gender}}{{^gender == 'Male'}}{{last_name}}{{/gender}} please")
+    expected_text = "Refer to the matter as Hector please"
+    expect(result_text).to eq(expected_text)
+  end
+
+  it 'should handle multiple positive checks in one line' do
+    result_text = render_docx("Refer to the matter as {{#gender == 'Male'}}him{{/gender}}{{#gender == 'Female'}}her{{/gender}} please")
+    expected_text = "Refer to the matter as him please"
+    expect(result_text).to eq(expected_text)
+  end
+
+  it 'should handle multiple positive checks and multiple negative checks in one line' do
+    result_text = render_docx("be {{#num == 1}}1{{/num}}{{#num == 2}}2{{/num}} and {{^num == 2}}!2{{/num}}{{^num == 1}}!1{{/num}} please")
+    expected_text = "be 2 and !1 please"
+    expect(result_text).to eq(expected_text)
+  end
+
+  it 'should handle multiple types of conditionals in one line' do
+    result_text = render_docx("be{{#gender == Male}} he {{/gender}}{{#num == 1}}1{{/num}}{{#num == 2}}2{{/num}} and {{^num == 2}}!2{{/num}}{{^num == 1}}!1{{/num}} please")
+    expected_text = "be he 2 and !1 please"
+    expect(result_text).to eq(expected_text)
+  end
+
+  it 'should handle multiline conditional tags' do
+    text = [
+      "Refer to the matter as",
+      "{{#gender == 'Male'}}",
+      "{{first_name}}",
+      "{{/gender}}",
+      "{{^gender == 'Male'}}",
+      "{{last_name}}",
+      "{{/gender}}Thank you"
+    ].join("\r\n")
+
+    result_text = render_docx(text)
+    expected_text = "Refer to the matter as\r\rHector\r\rThank you"
+    expect(result_text).to eq(expected_text)
+  end
+end

metadata

@@ -1,15 +1,16 @@
 --- !ruby/object:Gem::Specification
 name: lm_docstache
 version: !ruby/object:Gem::Version
-  version: 1.1.0
+  version: 1.2.2.1
 platform: ruby
 authors:
 - Roey Chasman
+- Jonathan Stevens
 - Will Cosgrove
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2020-06-09 00:00:00.000000000 Z
+date: 2020-07-16 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: nokogiri
@@ -46,6 +47,9 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: 3.1.0
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '3.1'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
@@ -53,29 +57,48 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: 3.1.0
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '3.1'
 - !ruby/object:Gem::Dependency
   name: pry-byebug
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - "~>"
       - !ruby/object:Gem::Version
         version: '1'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '1'
+- !ruby/object:Gem::Dependency
+  name: activesupport
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '1'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
       - !ruby/object:Gem::Version
         version: '1'
 description: Integrates data into MS Word docx template files. Processing supports
   loops and replacement of strings of data both outside and within loops.
 email:
 - roey@lawmatics.com
+- jonathan@lawmatics.com
 - will@willcosgrove.com
 executables: []
 extensions: []
 extra_rdoc_files: []
 files:
+- ".github/workflows/ruby.yml"
 - ".gitignore"
 - CHANGELOG.md
 - Gemfile
@@ -89,9 +112,11 @@ files:
 - lib/lm_docstache/renderer.rb
 - lib/lm_docstache/version.rb
 - lm_docstache.gemspec
+- spec/conditional_block_spec.rb
 - spec/data_scope_spec.rb
 - spec/empty_data_scope_spec.rb
 - spec/example_input/ExampleTemplate.docx
+- spec/example_input/blank.docx
 - spec/example_input/word/document.xml
 - spec/integration_spec.rb
 - spec/spec_helper.rb
@@ -120,9 +145,11 @@ signing_key:
 specification_version: 4
 summary: Merges Hash of Data into Word docx template files using mustache syntax
 test_files:
+- spec/conditional_block_spec.rb
 - spec/data_scope_spec.rb
 - spec/empty_data_scope_spec.rb
 - spec/example_input/ExampleTemplate.docx
+- spec/example_input/blank.docx
 - spec/example_input/word/document.xml
 - spec/integration_spec.rb
 - spec/spec_helper.rb