polytexnic 1.5.1 → 1.5.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 6095a0ec5936b87a0a296c91bc61b772197013bade5d9a28f7045f78f3221583
-  data.tar.gz: 4e4af1905ea37ff5d8a124a209d6c6c49faa672433156b91590cf712324012db
+  metadata.gz: 6c446b2fb998709a77a87e98ba9af120969006b56484734a32293cbaca231033
+  data.tar.gz: 3f47c7501336770234680a56af3b4115aeb3c831b04410256a0f6d52d602a407
 SHA512:
-  metadata.gz: 3320219ee29e5e0bb74fdc702aa80ad51554ed7ef810833c657f805128a0db01f22940c04a9668e7e54a2cbd0ecbb8eb3dd697da3ac7af2a16c81f7d7d7866cc
-  data.tar.gz: b804d40cf685c3db85f0327fb4a22aab99dca3875755dab4052db4679cdf3131ded4e1dac3ea4883b9811beb1361a5a76101d9732da6f441ad822f1c0673794e
+  metadata.gz: 70d14a3eceafb2fb6bf73bf6f8bebff97467942ac3cf1097bf92c4958667d4a163162766383ec9c3e03f9d637ec555ff801fd547d0536e656aa4ffe958d44fdf
+  data.tar.gz: 390ea1b062ad18eec86bca19632cf66c8f1832a6d7e9d417441e90e653ee38decfbf0759732e494f655eb41b96d1137e302aa15a61b16fec95cb442d1728ad52

data/lib/polytexnic/preprocessors/html.rb

@@ -68,23 +68,29 @@ module Polytexnic
           end
           not_a_capital = '[^A-Z]'
           # Case of "foo. A"
-          doc.gsub!(/(#{not_a_capital})(#{end_of_sentence})[ ]+([^\s])/) do
+          doc.gsub!(/(#{not_a_capital})(#{end_of_sentence})[ \t]+([^\s])/) do
             $1 + $2 + xmlelement('intersentencespace') + ' ' + $3
           end
-          # Case of "foo.\n A"
-          doc.gsub!(/(#{not_a_capital})(#{end_of_sentence})\n[ ]+([^\s])/) do
-            $1 + $2 + xmlelement('intersentencespace') + ' ' + $3
-          end
-          # Case of "foo.\nA"
-          doc.gsub!(/(#{not_a_capital})(#{end_of_sentence})\n([^\n])/) do
+          # Case of "foo.\nA" or "foo.\n A"
+          doc.gsub!(/(#{not_a_capital})(#{end_of_sentence})[ \t]*\n[ \t]*([^\s])/) do
             $1 + $2 + xmlelement('intersentencespace') + ' ' + $3
           end
           # Case of "foo.} A" or "foo.}} A"
-          doc.gsub!(/(#{not_a_capital})(#{end_of_sentence})(\}+)[ ]+([^\s])/) do
+          doc.gsub!(/(#{not_a_capital})(#{end_of_sentence})(\}+)[ \t]+([^\s])/) do
             $1 + $2 + $3 + xmlelement('intersentencespace') + ' ' + $4
           end
           # Case of "foo.'') A"
-          doc.gsub!(/(#{not_a_capital})(#{end_of_sentence})('+)(\))[ ]+([^\s])/) do
+          doc.gsub!(/(#{not_a_capital})(#{end_of_sentence})('+)(\))[ \t]+([^\s])/) do
+            $1 + $2 + $3 + $4 + xmlelement('intersentencespace') + ' ' + $5
+          end
+          # Case of "foo.}\nA" or "foo.}}\nA"
+          # Matching '.}\n\label' messes up aside boxes, so as a compromise
+          # we detect '.}\n\w', which matches virtually all use cases.
+          doc.gsub!(/(#{not_a_capital})(#{end_of_sentence})(\}+)\n[ \t]*(\w+)/) do
+            $1 + $2 + $3 + xmlelement('intersentencespace') + ' ' + $4
+          end
+          # Case of "foo.')\nA" or "foo.'')\nA"
+          doc.gsub!(/(#{not_a_capital})(#{end_of_sentence})('+)(\))\n([^\s])/) do
             $1 + $2 + $3 + $4 + xmlelement('intersentencespace') + ' ' + $5
           end
           # Handle the manual override to force an intersentence space, '\@',

data/lib/polytexnic/version.rb

@@ -1,3 +1,3 @@
 module Polytexnic
-  VERSION = "1.5.1"
+  VERSION = "1.5.2"
 end

data/spec/to_html/characters_and_punctuation_spec.rb

@@ -93,7 +93,12 @@ describe 'Polytexnic::Pipeline#to_html' do
           it { should resemble "#{intsp} Bar" }
         end
 
-        context "with a sentence ending with single quote and a paren" do
+        context "with a sentence ending with curly braces and a newline" do
+          let(:polytex) { "\\emph{\\textbf{Foo.}}\nBar." }
+          it { should resemble "#{intsp} Bar" }
+        end
+
+        context "with a sentence ending with a single quote and a paren" do
           let(:polytex) { "(It 'isn't.') Is it?"}
           it { should resemble "(It ’isn’t.’)#{intsp} Is it?" }
         end
@@ -103,12 +108,37 @@ describe 'Polytexnic::Pipeline#to_html' do
           it { should resemble "(It ”isn’t.”)#{intsp} Is it?" }
         end
 
+        context "with a sentence ending with a single quote, paren, newline" do
+          let(:polytex) { "(It 'isn't.')\nIs it?"}
+          it { should resemble "(It ’isn’t.’)#{intsp} Is it?" }
+        end
+
+        context "with a sentence ending with double quotes, paren, newline" do
+          let(:polytex) { "(It ''isn't.'')\nIs it?"}
+          it { should resemble "(It ”isn’t.”)#{intsp} Is it?" }
+        end
+
         context "with a mid-sentence footnote ending with a period" do
           let(:polytex) { 'Lorem\footnote{From \emph{Cicero}.} ipsum.' }
           it { should     include 'ipsum' }
           it { should_not include 'intersentencespace' }
         end
 
+        context "with only a newline" do
+          let(:polytex) { "Lorem ipsum.\nDolor sit amet." }
+          it { should resemble "Lorem ipsum.#{intsp} Dolor sit amet." }
+        end
+
+        context "with a newline and space" do
+          let(:polytex) { "Lorem ipsum.  \n  Dolor sit amet." }
+          it { should resemble "Lorem ipsum.#{intsp} Dolor sit amet." }
+        end
+
+        context "with two newlines" do
+          let(:polytex) { "Lorem ipsum.\n\nDolor sit amet." }
+          it { should_not resemble "Lorem ipsum.#{intsp} Dolor sit amet." }
+        end
+
         context "forced inter-sentence override" do
           let(:polytex) { 'Superman II\@. Lorem ipsum.' }
           it { should resemble "Superman II.#{intsp} Lorem ipsum." }

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: polytexnic
 version: !ruby/object:Gem::Version
-  version: 1.5.1
+  version: 1.5.2
 platform: ruby
 authors:
 - Michael Hartl