lm_docstache 1.1.2 → 1.2.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 4da1923074bce295e621bf0fff86882e16fe9f8bef9bf7373e955ce4b1a1f8f7
-  data.tar.gz: 7fb1709820b0d692c68b0ca8f5b2cea3769170e2387254f309bc77cd2cb9c730
+  metadata.gz: b345230c53f335f5de05dccdaca946bf30edaee21031c219dbdad1315c61c915
+  data.tar.gz: 554faf30a6988614877f20d72d172d1a66c11bcb2a145f4eb4efd45f7a598fb0
 SHA512:
-  metadata.gz: b4d8662856ee7a9b1239b4ca5e1427ae57132b0da6133b7cc955e85966c90facfa20da9261b6d4654a5fa2b96fe5bedf2dd61ea20d70039a276aa9628d412223
-  data.tar.gz: 65912e5b0715e750de569d69b3da542e1f7ebbe58a58cab115b8090e64de4e8b329b18f7c95e45bc2a234a15045728acab13611303889ba7dde923f54830c86d
+  metadata.gz: 66732e83ef4cbd515c10904dd293a29346bb1278f1bcfa3ad7becc6e027798f8ca2ca1534f2a9ffc86ee1e4db2d632cd6c926fd703975f386524fd42ccd48516
+  data.tar.gz: f3ff05f0d5e5b55e3c6eb3e070713b625f74544d667816a71a761d48521ffda12146ff72b3a1189b9a794b8a190a60ea7227a374c0c58711471fba1786de0ce4

data/.gitignore

@@ -2,4 +2,3 @@
 spec/example_input/tmp
 .idea/
 *.gem
-Gemfile.lock

data/CHANGELOG.md

@@ -1,7 +1,11 @@
 # Changelog
 
+## 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/Gemfile.lock

@@ -0,0 +1,48 @@
+PATH
+  remote: .
+  specs:
+    lm_docstache (1.2.0)
+      nokogiri (~> 1.6)
+      rubyzip (~> 1.1)
+
+GEM
+  remote: https://rubygems.org/
+  specs:
+    byebug (11.1.3)
+    coderay (1.1.3)
+    diff-lcs (1.4.4)
+    method_source (1.0.0)
+    mini_portile2 (2.4.0)
+    nokogiri (1.10.10)
+      mini_portile2 (~> 2.4.0)
+    pry (0.13.1)
+      coderay (~> 1.1)
+      method_source (~> 1.0)
+    pry-byebug (3.9.0)
+      byebug (~> 11.0)
+      pry (~> 0.13.0)
+    rspec (3.9.0)
+      rspec-core (~> 3.9.0)
+      rspec-expectations (~> 3.9.0)
+      rspec-mocks (~> 3.9.0)
+    rspec-core (3.9.2)
+      rspec-support (~> 3.9.3)
+    rspec-expectations (3.9.2)
+      diff-lcs (>= 1.2.0, < 2.0)
+      rspec-support (~> 3.9.0)
+    rspec-mocks (3.9.1)
+      diff-lcs (>= 1.2.0, < 2.0)
+      rspec-support (~> 3.9.0)
+    rspec-support (3.9.3)
+    rubyzip (1.3.0)
+
+PLATFORMS
+  ruby
+
+DEPENDENCIES
+  lm_docstache!
+  pry-byebug (>= 1)
+  rspec (>= 3.1.0)
+
+BUNDLED WITH
+   1.17.2

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: name, data: data, element: match, inverted: inverted, condition: 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: name, data: data, element: element, inverted: inverted, condition: 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)
         unless condition
           block.content_elements.each(&:unlink)
         end
@@ -60,6 +68,40 @@ module LMDocstache
       block.closing_element.unlink
     end
 
+    def replace_conditionals(block)
+      condition = get_condition(block)
+
+      @content.css('w|t').each do |text_el|
+        rendered_string = text_el.text
+
+        if !(results = rendered_string.scan(/{{#(.*?)}}(.*?){{\/(.*?)}}/)).empty?
+          results.each do |r|
+            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,
+        # and the condition being inverted. maybe combine them?
+        if !(results = rendered_string.scan(/{{\^(.*?)}}(.*?){{\/(.*?)}}/)).empty?
+          results.each do |r|
+            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 +114,19 @@ module LMDocstache
       end
       return elements
     end
+
+    private
+
+    def get_condition(block)
+      case condition = @data.get(block.name, condition: block.condition)
+      when Array
+        condition = !condition.empty?
+      else
+        condition = !!condition
+      end
+      condition = !condition if block.inverted
+
+      condition
+    end
   end
 end

data/lib/lm_docstache/version.rb

@@ -1,3 +1,3 @@
 module LMDocstache
-  VERSION = "1.1.2"
+  VERSION = "1.2.0"
 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.}

data/spec/conditional_block_spec.rb

@@ -0,0 +1,82 @@
+require 'spec_helper'
+require 'nokogiri'
+
+module LMDocstache
+  module TestData
+    DATA = {
+      gender: 'Male',
+      first_name: 'Hector',
+      last_name: 'Jones'
+    }
+  end
+end
+
+describe LMDocstache::Renderer do
+  let(:data) { Marshal.load(Marshal.dump(LMDocstache::TestData::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 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.2
+  version: 1.2.0
 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-15 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: nokogiri
@@ -71,6 +72,7 @@ description: Integrates data into MS Word docx template files. Processing suppor
   loops and replacement of strings of data both outside and within loops.
 email:
 - roey@lawmatics.com
+- jonathan@lawmatics.com
 - will@willcosgrove.com
 executables: []
 extensions: []
@@ -79,6 +81,7 @@ files:
 - ".gitignore"
 - CHANGELOG.md
 - Gemfile
+- Gemfile.lock
 - LICENSE.txt
 - README.md
 - Rakefile
@@ -89,9 +92,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 +125,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