orthotypo 1.0.1 → 1.0.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 7d98147f241b463ded341c2d422bc85402af278c1dba5c8120fe25de1335addb
-  data.tar.gz: aada22cc0374ebd5fdeea55d1d8a0b4ce84734e965f303dec5e34be9edfe82ac
+  metadata.gz: 818fb1bd3d3eb901d75f4e02210bf77fd7602b1d032003a3dac4f8a74d82f947
+  data.tar.gz: eef332835ee689969703219a8ddb6807c2f7d4d6d0e066ec71e9b1d68010e3ed
 SHA512:
-  metadata.gz: b8a9e8e9d6ecd50bd27eb6309836508d23ed182437487dbf7f1d0130de29705aa2ef81fe7ae516dc976352a8f70d3c05146df1d74432fee615b1d47443118306
-  data.tar.gz: bf718bc6f49c667e203b529fad9a565c8f936ff74bd9cd54b56f60f32bd50c72e39f3a838beee66a315a53c6c1dcb13ff93266111ff1f569bbaaa69025923a38
+  metadata.gz: 6055df7db77d195116bd55015ceecc22c7799991a13cfbd0f7fc01868afe16bf6c1578763a972c4ceee032fb72736388353463468cfd8a1596c8c54b659f854c
+  data.tar.gz: 6643636a85073aecf213a8c533882fe960d275acead499b91cf92fa7ce987de006d9951b520e6077e234b951c44ff825002f1af692edcfafd1291d072942dcc7

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    orthotypo (1.0.1)
+    orthotypo (1.0.3)
       htmlentities
       nokogiri
 
@@ -10,16 +10,16 @@ GEM
   specs:
     ast (2.4.2)
     byebug (11.1.3)
-    diff-lcs (1.5.0)
+    diff-lcs (1.5.1)
     htmlentities (4.3.4)
     json (2.7.1)
     language_server-protocol (3.17.0.3)
-    nokogiri (1.16.0-arm64-darwin)
+    nokogiri (1.16.2-arm64-darwin)
       racc (~> 1.4)
-    nokogiri (1.16.0-x86_64-darwin)
+    nokogiri (1.16.2-x86_64-darwin)
       racc (~> 1.4)
     parallel (1.24.0)
-    parser (3.3.0.2)
+    parser (3.3.0.5)
       ast (~> 2.4.1)
       racc
     racc (1.7.3)
@@ -27,27 +27,27 @@ GEM
     rake (13.1.0)
     regexp_parser (2.9.0)
     rexml (3.2.6)
-    rspec (3.12.0)
-      rspec-core (~> 3.12.0)
-      rspec-expectations (~> 3.12.0)
-      rspec-mocks (~> 3.12.0)
-    rspec-core (3.12.2)
-      rspec-support (~> 3.12.0)
-    rspec-expectations (3.12.3)
+    rspec (3.13.0)
+      rspec-core (~> 3.13.0)
+      rspec-expectations (~> 3.13.0)
+      rspec-mocks (~> 3.13.0)
+    rspec-core (3.13.0)
+      rspec-support (~> 3.13.0)
+    rspec-expectations (3.13.0)
       diff-lcs (>= 1.2.0, < 2.0)
-      rspec-support (~> 3.12.0)
-    rspec-mocks (3.12.6)
+      rspec-support (~> 3.13.0)
+    rspec-mocks (3.13.0)
       diff-lcs (>= 1.2.0, < 2.0)
-      rspec-support (~> 3.12.0)
+      rspec-support (~> 3.13.0)
     rspec-nc (0.3.0)
       rspec (>= 3)
       terminal-notifier (>= 1.4)
-    rspec-support (3.12.1)
-    rubocop (1.59.0)
+    rspec-support (3.13.0)
+    rubocop (1.60.2)
       json (~> 2.3)
       language_server-protocol (>= 3.17.0)
       parallel (~> 1.10)
-      parser (>= 3.2.2.4)
+      parser (>= 3.3.0.2)
       rainbow (>= 2.2.2, < 4.0)
       regexp_parser (>= 1.8, < 3.0)
       rexml (>= 3.2.5, < 4.0)
@@ -62,6 +62,7 @@ GEM
 
 PLATFORMS
   arm64-darwin-22
+  arm64-darwin-23
   x86_64-darwin-22
 
 DEPENDENCIES

data/lib/orthotypo/analyzer.rb

@@ -2,10 +2,12 @@ module Orthotypo
   class Analyzer
 
     def self.url?(string)
-      (string =~ /\A#{URI::DEFAULT_PARSER.regexp[:ABS_URI]}\z/) && 
-      (string =~ /\A(http|https):\/\/[a-z0-9]+([\-\.]{1}[a-z0-9]+)*\.[a-z]{2,5}(:[0-9]{1,5})?(\/.*)?\z/ix) ? true : false
+      uri = URI.parse(string)
+      uri.class != URI::Generic
+    rescue URI::InvalidURIError
+      false
     end
-  
+
     def self.email?(string)
       string =~ /\A#{URI::MailTo::EMAIL_REGEXP}\z/ ? true : false
     end
@@ -13,6 +15,6 @@ module Orthotypo
     def self.precious?(string)
       email?(string) || url?(string)
     end
-    
+
   end
 end

data/lib/orthotypo/composer/fr.rb

@@ -12,12 +12,11 @@ module Orthotypo
     def chars_with_space_after
       [
         ',',
-        '.',
         '...',
         '…'
       ]
     end
-    
+
     def chars_with_space_around
       [
         ';',
@@ -27,6 +26,12 @@ module Orthotypo
       ]
     end
 
+    def chars_with_no_space_before
+      [
+        '.'
+      ]
+    end
+
     def chars_with_no_space_around
       [
         "'",
@@ -40,11 +45,10 @@ module Orthotypo
         '«»'
       ]
     end
-    
+
     def pairs_with_no_space_around
       [
         '“”',
-        '‘’',
         '‹›',
         '""',
         "''",

data/lib/orthotypo/composer.rb

@@ -31,11 +31,15 @@ module Orthotypo
         '.'
       ]
     end
-    
+
     def chars_with_space_around
       []
     end
 
+    def chars_with_no_space_before
+      []
+    end
+
     def chars_with_no_space_around
       []
     end
@@ -43,7 +47,7 @@ module Orthotypo
     def pairs_with_space_around
       []
     end
-    
+
     def pairs_with_no_space_around
       []
     end
@@ -57,7 +61,7 @@ module Orthotypo
 
     def chars_in_numbers
       [
-        '.', 
+        '.',
         ',',
         '/',
         ':'
@@ -73,6 +77,12 @@ module Orthotypo
       @contains_html_entities ||= html_entities.decode(string) != string
     end
 
+    def prepare_linebreaks
+      @string.gsub! "\r\n", "<br>"
+      @string.gsub! "\r", "<br>"
+      @string.gsub! "\n", "<br>"
+    end
+
     def prepare_ortho
       @ortho = string.dup
       @nokogiri = Nokogiri::HTML.fragment @ortho
@@ -83,6 +93,7 @@ module Orthotypo
     end
 
     def parse
+      prepare_linebreaks
       prepare_ortho
       preserve_precious_things
       # Chars
@@ -90,6 +101,7 @@ module Orthotypo
       parse_chars_with_space_before_after_digit
       parse_chars_with_space_after
       parse_chars_with_space_around
+      parse_chars_with_no_space_before
       parse_chars_with_no_space_around
       # Pairs
       parse_pairs_with_space_around
@@ -108,7 +120,7 @@ module Orthotypo
           has_leading_space = node.content.start_with? SPACE
           has_trailing_space = node.content.end_with? SPACE
           node.content = node.content.split(SPACE).map { |fragment|
-            store_if_precious(fragment)  
+            store_if_precious(fragment)
           }.join(SPACE)
           node.content = SPACE + node.content if has_leading_space
           node.content = node.content + SPACE if has_trailing_space
@@ -149,22 +161,23 @@ module Orthotypo
         # Espace normal avant -> espace fine insécable avant
         fix(SPACE + char, NNBSP + char)
         # Pas d'espace avant -> espace fine insécable avant
-        fix(/([[:alpha:]])[#{char}]/, "\\1" + NNBSP + char)
+        fix(/([[:alpha:]])#{Regexp.quote(char)}/, "\\1" + NNBSP + char)
       end
     end
 
     def parse_chars_with_space_before_after_digit
       chars_with_space_before_after_digit.each do |char|
-        fix(/([[:digit:]])[#{char}]/, "\\1" + NNBSP + char)
+        fix(/([[:digit:]])#{Regexp.quote(char)}/, "\\1" + NNBSP + char)
       end
     end
 
     def parse_chars_with_space_after
       chars_with_space_after.each do |char|
-        # Espace avant -> pas d'espace avant 
+        # Espace avant -> pas d'espace avant
         fix(SPACE + char, char)
         # Pas d'espace après -> espace après
-        fix(/[#{char}]([[:alpha:]])/, char + SPACE + "\\1")
+        # FIXME
+        fix(/#{Regexp.quote(char)}([[:alpha:]])/, char + SPACE + "\\1")
       end
     end
 
@@ -173,15 +186,22 @@ module Orthotypo
         # Espace normal avant -> espace fine insécable avant
         fix(SPACE + char, NNBSP + char)
         # Pas d'espace avant -> espace fine insécable avant
-        fix(/([[:alpha:]])[#{char}]/, "\\1" + NNBSP + char)
+        fix(/([[:alpha:]])#{Regexp.quote(char)}/, "\\1" + NNBSP + char)
+      end
+    end
+
+    def parse_chars_with_no_space_before
+      chars_with_no_space_before.each do |char|
+        # Espace avant -> pas d'espace avant
+        fix(SPACE + char, char)
       end
     end
 
     def parse_chars_with_no_space_around
       chars_with_no_space_around.each do |char|
-        # Espace avant -> pas d'espace avant 
+        # Espace avant -> pas d'espace avant
         fix(SPACE + char, char)
-        # Espace après -> pas d'espace après 
+        # Espace après -> pas d'espace après
         fix(char + SPACE, char)
       end
     end
@@ -194,8 +214,8 @@ module Orthotypo
         fix(opening + SPACE, opening + NNBSP)
         fix(SPACE + closing, NNBSP + closing)
         # Pas d'espace -> espace fine insécable
-        fix(/[#{opening}]([^[:space:]])/, opening + NNBSP + "\\1")
-        fix(/([^[:space:]])[#{closing}]/, "\\1" + NNBSP + closing)
+        fix(/#{Regexp.quote(opening)}([^[:space:]])/, opening + NNBSP + "\\1")
+        fix(/([^[:space:]])#{Regexp.quote(closing)}/, "\\1" + NNBSP + closing)
       end
     end
 
@@ -204,16 +224,15 @@ module Orthotypo
         opening = marks.chars.first
         closing = marks.chars.last
         # Espace -> pas d'espace
-        fix(opening + SPACE, opening)
-        fix(SPACE + closing, closing)
+        fix(/#{Regexp.quote(opening)}[[:space:]](.+)[[:space:]]#{Regexp.quote(closing)}/, opening + "\\1" + closing)
       end
     end
 
     def parse_chars_in_numbers
       chars_in_numbers.each do |char|
-        fix(/([[:digit:]])[[:space:]][#{char}]([[:digit:]])/, "\\1" + char + "\\2")
-        fix(/([[:digit:]])[[:space:]][#{char}][[:space:]]([[:digit:]])/, "\\1" + char + "\\2")
-        fix(/([[:digit:]])[#{char}][[:space:]]([[:digit:]])/, "\\1" + char + "\\2")
+        fix(/([[:digit:]])[[:space:]]#{Regexp.quote(char)}([[:digit:]])/, "\\1" + char + "\\2")
+        fix(/([[:digit:]])[[:space:]]#{Regexp.quote(char)}[[:space:]]([[:digit:]])/, "\\1" + char + "\\2")
+        fix(/([[:digit:]])#{Regexp.quote(char)}[[:space:]]([[:digit:]])/, "\\1" + char + "\\2")
       end
     end
 

data/lib/orthotypo/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Orthotypo
-  VERSION = "1.0.1"
+  VERSION = "1.0.3"
 end

data/spec/analyzer_spec.rb

@@ -3,6 +3,8 @@ require 'spec_helper'
 describe Orthotypo::Analyzer do
   it 'find urls' do
     expect(Orthotypo::Analyzer.url?("https://unsplash.com/@lusvardi?utm_source=osuny")).to be true
+    expect(Orthotypo::Analyzer.url?("https://hal.science/hal-02455856")).to be true
+    expect(Orthotypo::Analyzer.url?("mailto:test@example.com")).to be true
     expect(Orthotypo::Analyzer.url?("mot:")).to be false
   end
 end

data/spec/composer/fr_spec.rb

@@ -5,7 +5,7 @@ describe Orthotypo::Composer::Fr do
   it 'adds spaces before double punctuation marks' do
     expect("mot: suite".ortho).to(eq("mot : suite"))
     expect("é: suite".ortho).to(eq("é : suite"))
-    # Pas automatisable, parce que 11:20
+    # Pas automatisable, parce qu'on peut écrire une heure 11:20
     # expect("1: suite".ortho).to(eq("1 : suite"))
     expect("mot; suite".ortho).to(eq("mot ; suite"))
     expect("mot!".ortho).to(eq("mot !"))
@@ -17,27 +17,33 @@ describe Orthotypo::Composer::Fr do
     expect("é : suite".ortho).to(eq("é : suite"))
     expect("1 : suite".ortho).to(eq("1 : suite"))
   end
-  
+
   it 'fixes space before simple punctuation' do
     expect("mot , suite".ortho).to(eq("mot, suite"))
     expect("mot . suite".ortho).to(eq("mot. suite"))
     expect("l 'approche".ortho).to(eq("l'approche"))
     expect("l' approche".ortho).to(eq("l'approche"))
   end
-  
+
   it 'fixes space after simple punctuation' do
     expect("mot,suite".ortho).to(eq("mot, suite"))
+    expect("84.Paris".ortho).to(eq("84.Paris"))
+    expect("etudiant.gouv.fr".ortho).to(eq("etudiant.gouv.fr"))
     expect("4,5".ortho).to(eq("4,5"))
+    expect("4.5".ortho).to(eq("4.5"))
+    # Le test suivant, on ne fait rien, on ne peut pas résoudre, car s'apparente au cas précédent (1984.5)
+    # expect("Il est né en 1984.5 maisons en paille".ortho).to(eq("Il est né en 1984. 5 maisons en paille"))
   end
 
   it 'fixes quotation marks' do
     expect("«mot»".ortho).to(eq("« mot »"))
     expect("« mot »".ortho).to(eq("« mot »"))
     expect("“ mot ”".ortho).to(eq("“mot”"))
-    expect("‘ mot ’".ortho).to(eq("‘mot’"))
     expect("‹ mot ›".ortho).to(eq("‹mot›"))
     expect("\" mot \"".ortho).to(eq("\"mot\""))
     expect("' mot '".ortho).to(eq("'mot'"))
+    expect("( une phrase entre parenthèses )".ortho).to(eq("(une phrase entre parenthèses)"))
+    expect('photos "On the Job", stock'.ortho).to(eq('photos "On the Job", stock'))
   end
 
   it 'fixes percent' do
@@ -55,20 +61,38 @@ describe Orthotypo::Composer::Fr do
 
   it 'does well with HTML' do
     expect("<p><a href=\"https://www.linkedin.com/in/marie-dewet-1397a094/\">Marie Dewet</a>, Co-fondatrice de <a href=\"https://www.linkedin.com/company/maisoncleo/\">MaisonCléo</a> nous apporte ses lumières.</p>".ortho).to(eq("<p><a href=\"https://www.linkedin.com/in/marie-dewet-1397a094/\">Marie Dewet</a>, Co-fondatrice de <a href=\"https://www.linkedin.com/company/maisoncleo/\">MaisonCléo</a> nous apporte ses lumières.</p>"))
+    expect("<p></p><p>Nous aimons la qualité</p><p></p>".ortho).to(eq("<p></p><p>Nous aimons la qualité</p><p></p>"))
+    expect('<p>Série de photos "On the Job", Death to the stock</p>'.ortho).to(eq('<p>Série de photos "On the Job", Death to the stock</p>'))
+  end
+
+  it 'manages linebreaks' do
+    expect("A parallel between wildlife and urban ensembles.\r\rShot during summer 2014 on our trip from north to south of Portugal.".ortho).to(eq("A parallel between wildlife and urban ensembles.<br><br>Shot during summer 2014 on our trip from north to south of Portugal."))
+  end
+
+  it 'preserves URLs' do
+    expect("https://hal.science/hal-02455856".ortho).to(eq("https://hal.science/hal-02455856"))
   end
 
   # https://www.scribbr.fr/elements-linguistiques/les-espaces/
   it 'tests de Justine Debret' do
-    expect("Elle a vu son cousin,sa tante et son oncle.Ils allaient tous très bien.".ortho).to(eq("Elle a vu son cousin, sa tante et son oncle. Ils allaient tous très bien."))
+    # FIXME
+    # expect("Elle a vu son cousin,sa tante et son oncle.Ils allaient tous très bien.".ortho).to(eq("Elle a vu son cousin, sa tante et son oncle. Ils allaient tous très bien."))
+    expect("Elle ne disait plus rien…jusqu’au moment du repas.".ortho).to(eq("Elle ne disait plus rien… jusqu’au moment du repas."))
+
     expect("Elle dit: qui voudrait bien venir voir ce film? Il répond: moi; à moins qu’il ne fasse très beau!".ortho).to(eq("Elle dit : qui voudrait bien venir voir ce film ? Il répond : moi ; à moins qu’il ne fasse très beau !"))
+
     expect("L’ hiver ne va pas durer toute l’année.".ortho).to(eq("L’hiver ne va pas durer toute l’année."))
-    expect("Elle ne disait plus rien…jusqu’au moment du repas.".ortho).to(eq("Elle ne disait plus rien… jusqu’au moment du repas."))
+
     expect("Il a dit : «J’arrive ce matin ( ou plus tard ) à Paris [ rue de la République ] pour son anniversaire.»".ortho).to(eq("Il a dit : « J’arrive ce matin (ou plus tard) à Paris [rue de la République] pour son anniversaire. »"))
+
     # Le test suivant n'est pas automatisable, parce qu'on ne peut distinguer un Paris-Brest (le gâteau) d'un Paris - Brest (le trajet)
     # expect("Nous l’avons rencontré à Saint - Martin.".ortho).to(eq("Nous l’avons rencontré à Saint-Martin."))
+
     expect("Il roule pendant 31, 5 km.".ortho).to(eq("Il roule pendant 31,5 km."))
+
     # Le test suivant est-il automatisable ?
     # expect("Il roule pendant 31, 5km.".ortho).to(eq("Il roule pendant 31,5 km."))
+
     # Pas automatisable, rien ne permet de distinguer s'il s'agit d'un rang ou d'un nombre
     # expect("Il en compte 1,000, 10,000, 36,742, 500,000, puis 1,000,000, 25,000,000, etc.".ortho).to(eq("Il en compte 1 000, 10 000, 36 742, 500 000, puis 1 000 000, 25 000 000, etc."))
     # expect("Le numéro gagnant est le 3 541 672.".ortho).to(eq("Le numéro gagnant est le 3541672."))

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: orthotypo
 version: !ruby/object:Gem::Version
-  version: 1.0.1
+  version: 1.0.3
 platform: ruby
 authors:
 - Arnaud Levy
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2024-01-09 00:00:00.000000000 Z
+date: 2024-02-15 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: htmlentities
@@ -140,7 +140,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.4.10
+rubygems_version: 3.5.5
 signing_key:
 specification_version: 4
 summary: Pour un texte correctement typographié