orthotypo 0.6.0 → 1.0.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: b41b05766be4f11e6494a7e89823a6ab730910aec4a679d22b764ae96fb05f10
-  data.tar.gz: 0c2a072f5c3ebd8583b70c7ee4bf0eafe064c827d4e0af250728bcf5762dc5ae
+  metadata.gz: 7d98147f241b463ded341c2d422bc85402af278c1dba5c8120fe25de1335addb
+  data.tar.gz: aada22cc0374ebd5fdeea55d1d8a0b4ce84734e965f303dec5e34be9edfe82ac
 SHA512:
-  metadata.gz: 20ff73ede6bf05d4b2b8842604b09267f36cf2d44fc17c6e4a467e9e703934a20f18616368cc5af88529122166130c8b868612af6caab95ff6f685236b862443
-  data.tar.gz: 55f9a2023f185176d52de197847f92ee61ea0e8ba20d70c920fec7e31b4d86dbae0ab87594adcd7c406fcdbbc2ef189fa8a113bcfeab9f60786c31dd77771f1e
+  metadata.gz: b8a9e8e9d6ecd50bd27eb6309836508d23ed182437487dbf7f1d0130de29705aa2ef81fe7ae516dc976352a8f70d3c05146df1d74432fee615b1d47443118306
+  data.tar.gz: bf718bc6f49c667e203b529fad9a565c8f936ff74bd9cd54b56f60f32bd50c72e39f3a838beee66a315a53c6c1dcb13ff93266111ff1f569bbaaa69025923a38

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    orthotypo (0.6.0)
+    orthotypo (1.0.1)
       htmlentities
       nokogiri
 
@@ -9,24 +9,23 @@ GEM
   remote: https://rubygems.org/
   specs:
     ast (2.4.2)
-    base64 (0.1.1)
     byebug (11.1.3)
     diff-lcs (1.5.0)
     htmlentities (4.3.4)
-    json (2.6.3)
+    json (2.7.1)
     language_server-protocol (3.17.0.3)
-    nokogiri (1.15.4-arm64-darwin)
+    nokogiri (1.16.0-arm64-darwin)
       racc (~> 1.4)
-    nokogiri (1.15.4-x86_64-darwin)
+    nokogiri (1.16.0-x86_64-darwin)
       racc (~> 1.4)
-    parallel (1.23.0)
-    parser (3.2.2.4)
+    parallel (1.24.0)
+    parser (3.3.0.2)
       ast (~> 2.4.1)
       racc
-    racc (1.7.1)
+    racc (1.7.3)
     rainbow (3.1.1)
-    rake (13.0.6)
-    regexp_parser (2.8.2)
+    rake (13.1.0)
+    regexp_parser (2.9.0)
     rexml (3.2.6)
     rspec (3.12.0)
       rspec-core (~> 3.12.0)
@@ -44,8 +43,7 @@ GEM
       rspec (>= 3)
       terminal-notifier (>= 1.4)
     rspec-support (3.12.1)
-    rubocop (1.57.0)
-      base64 (~> 0.1.1)
+    rubocop (1.59.0)
       json (~> 2.3)
       language_server-protocol (>= 3.17.0)
       parallel (~> 1.10)
@@ -53,10 +51,10 @@ GEM
       rainbow (>= 2.2.2, < 4.0)
       regexp_parser (>= 1.8, < 3.0)
       rexml (>= 3.2.5, < 4.0)
-      rubocop-ast (>= 1.28.1, < 2.0)
+      rubocop-ast (>= 1.30.0, < 2.0)
       ruby-progressbar (~> 1.7)
       unicode-display_width (>= 2.4.0, < 3.0)
-    rubocop-ast (1.29.0)
+    rubocop-ast (1.30.0)
       parser (>= 3.2.1.0)
     ruby-progressbar (1.13.0)
     terminal-notifier (2.0.0)

data/README.md

@@ -28,12 +28,27 @@ Ajout d'espace fine insécable avant les signes doubles en français.
 ## Roadmap
 
 ### v1
-1. S'adapter aux locales (détecter I18n)
-2. Ne pas endommager l'HTML et les HTML entities ( )
+Ne pas endommager l'HTML et les HTML entities ( )
 
 ### v2
+S'adapter aux locales (détecter I18n)
 Permettre les configs
 
+## Tests
+
+```
+rake
+```
+
+```
+bundle exec rspec
+```
+
+Pour jouer un seul test : 
+```
+bundle exec rspec ./spec/composer/fr_spec.rb:56
+```
+
 ## Sources
 
 - https://fr.wikipedia.org/wiki/Code_typographique

data/lib/orthotypo/composer.rb

@@ -19,6 +19,12 @@ module Orthotypo
       []
     end
 
+    def chars_with_space_before_after_digit
+      [
+        '%'
+      ]
+    end
+
     def chars_with_space_after
       [
         ',',
@@ -42,6 +48,22 @@ module Orthotypo
       []
     end
 
+    def chars_with_no_space_around_between_digits
+      [
+        '/',
+        ':'
+      ]
+    end
+
+    def chars_in_numbers
+      [
+        '.', 
+        ',',
+        '/',
+        ':'
+      ]
+    end
+
     def is_html?
       # TODO contains tags?
       @html || contains_html_entities?
@@ -53,13 +75,11 @@ module Orthotypo
 
     def prepare_ortho
       @ortho = string.dup
-      # @ortho = html_entities.decode(@ortho) if contains_html_entities?
       @nokogiri = Nokogiri::HTML.fragment @ortho
     end
 
     def clean_ortho
       @ortho = @nokogiri.to_s
-      # @ortho = html_entities.encode(@ortho) if contains_html_entities?
     end
 
     def parse
@@ -67,6 +87,7 @@ module Orthotypo
       preserve_precious_things
       # Chars
       parse_chars_with_space_before
+      parse_chars_with_space_before_after_digit
       parse_chars_with_space_after
       parse_chars_with_space_around
       parse_chars_with_no_space_around
@@ -74,8 +95,8 @@ module Orthotypo
       parse_pairs_with_space_around
       parse_pairs_with_no_space_around
       # Numbers
-      parse_numbers
-      # 
+      parse_chars_in_numbers
+      #
       clean_ortho
       restore_precious_things
     end
@@ -83,20 +104,40 @@ module Orthotypo
     def preserve_precious_things
       @precious_things = []
       @nokogiri.traverse do |node|
-        next unless node.text?
-        new_content = node.content.split(SPACE).map { |fragment|
-          if Analyzer::precious?(fragment)
-            token = "#{PRECIOUS_TOKEN}#{@precious_things.length}"
-            @precious_things << fragment
-            token
-          else
-            fragment
+        if node.text?
+          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)  
+          }.join(SPACE)
+          node.content = SPACE + node.content if has_leading_space
+          node.content = node.content + SPACE if has_trailing_space
+        elsif node.element?
+          if node.name == 'a'
+            node.attributes.each do |key, attribute|
+              if attribute.name == 'href'
+                attribute.value = store_precious_thing(attribute.value)
+              end
+            end
           end
-        }.join(SPACE)
-        node.content = new_content
+        end
       end
     end
 
+    def store_if_precious(string)
+      Analyzer::precious?(string) ? store_precious_thing(string)
+                                  : string
+    end
+
+    def store_precious_thing(string)
+      # Create token identifier
+      token = "#{PRECIOUS_TOKEN}#{@precious_things.length}"
+      # Store value
+      @precious_things << string
+      # Return identifier
+      token
+    end
+
     def restore_precious_things
       @precious_things.each_with_index do |value, index|
         @ortho.gsub! "#{PRECIOUS_TOKEN}#{index}", value
@@ -106,9 +147,15 @@ module Orthotypo
     def parse_chars_with_space_before
       chars_with_space_before.each do |char|
         # Espace normal avant -> espace fine insécable avant
-        fix(SPACE + '%', NNBSP + '%')
+        fix(SPACE + char, NNBSP + char)
         # Pas d'espace avant -> espace fine insécable avant
-        fix(/([[:alnum:]])%/, "\\1" + NNBSP + '%')
+        fix(/([[:alpha:]])[#{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)
       end
     end
 
@@ -117,7 +164,7 @@ module Orthotypo
         # Espace avant -> pas d'espace avant 
         fix(SPACE + char, char)
         # Pas d'espace après -> espace après
-        fix(/[#{char}]([[:alnum:]])/, char + SPACE + "\\1")
+        fix(/[#{char}]([[:alpha:]])/, char + SPACE + "\\1")
       end
     end
 
@@ -126,7 +173,7 @@ 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(/([[:alnum:]])[#{char}]/, "\\1" + NNBSP + char)
+        fix(/([[:alpha:]])[#{char}]/, "\\1" + NNBSP + char)
       end
     end
 
@@ -162,8 +209,10 @@ module Orthotypo
       end
     end
 
-    def parse_numbers
-      ['.', ','].each do |char|
+    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")
       end
     end

data/lib/orthotypo/version.rb

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

data/spec/composer/fr_spec.rb

@@ -5,7 +5,8 @@ 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"))
-    expect("1: suite".ortho).to(eq("1 : suite"))
+    # Pas automatisable, parce que 11:20
+    # expect("1: suite".ortho).to(eq("1 : suite"))
     expect("mot; suite".ortho).to(eq("mot ; suite"))
     expect("mot!".ortho).to(eq("mot !"))
     expect("mot !".ortho).to(eq("mot !"))
@@ -44,6 +45,18 @@ describe Orthotypo::Composer::Fr do
     expect("10 %".ortho).to(eq("10 %"))
   end
 
+  it 'fixes dates/time' do
+    expect("10/01/2023 16:00".ortho).to(eq("10/01/2023 16:00"))
+    expect("10/01/2023 16:00:00".ortho).to(eq("10/01/2023 16:00:00"))
+    expect("10 / 01 / 2023 16:00".ortho).to(eq("10/01/2023 16:00"))
+    expect("10 / 01 / 2023 16 : 00".ortho).to(eq("10/01/2023 16:00"))
+    expect("10 octobre 2023 16:00".ortho).to(eq("10 octobre 2023 16:00"))
+  end
+
+  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>"))
+  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."))
@@ -53,6 +66,7 @@ describe Orthotypo::Composer::Fr do
     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

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: orthotypo
 version: !ruby/object:Gem::Version
-  version: 0.6.0
+  version: 1.0.1
 platform: ruby
 authors:
 - Arnaud Levy
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2023-10-16 00:00:00.000000000 Z
+date: 2024-01-09 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.6
+rubygems_version: 3.4.10
 signing_key:
 specification_version: 4
 summary: Pour un texte correctement typographié