Merge pull request #38 from matecat/nbsp-double-encode

NBSP double encoding

src/Filters/NBSPDoubleEncode.php

@@ -0,0 +1,17 @@
+<?php
+
+
+namespace Matecat\SubFiltering\Filters;
+
+
+use Matecat\SubFiltering\Commons\AbstractHandler;
+
+class NBSPDoubleEncode extends AbstractHandler {
+
+    public function transform( $segment ) {
+        $segment = str_replace( "&nbsp;", "&amp;nbsp;", $segment );
+
+        return $segment;
+    }
+
+}

src/MateCatFilter.php

@@ -17,6 +17,7 @@
 use Matecat\SubFiltering\Filters\LtGtDoubleEncode;
 use Matecat\SubFiltering\Filters\LtGtEncode;
 use Matecat\SubFiltering\Filters\MateCatCustomPHToStandardPH;
+use Matecat\SubFiltering\Filters\NBSPDoubleEncode;
 use Matecat\SubFiltering\Filters\Percentages;
 use Matecat\SubFiltering\Filters\PercentNumberSnail;
 use Matecat\SubFiltering\Filters\PlaceBreakingSpacesInXliff;
@@ -93,6 +94,7 @@ public function fromLayer1ToLayer2( $segment ) {
         $channel->addLast( new RestoreTabsPlaceholders() );
         $channel->addLast( new HtmlPlainTextDecoder() );
         $channel->addLast( new LtGtDoubleEncode() );
+        $channel->addLast( new NBSPDoubleEncode() );
         $channel->addLast( new LtGtEncode() );
         $channel->addLast( new DataRefReplace() );
 

tests/MateCatSubFilteringTest.php

@@ -273,6 +273,16 @@ public function testNbsp() {
         $this->assertEquals( $string_from_UI, $filter->fromLayer0ToLayer2( $expected_segment ) );
     }
 
+    public function testNbspAsString() {
+        $filter = $this->getFilterInstance();
+
+        $expected_segment = '  Text';
+        $string_from_UI = '  Text';
+
+        $this->assertEquals( $expected_segment, $filter->fromLayer2ToLayer0( $string_from_UI ) );
+        $this->assertEquals( $string_from_UI, $filter->fromLayer0ToLayer2( $expected_segment ) );
+    }
+
     /**
      **************************
      * Sprintf