text length must be recalculated after applying each shortcode replac…

…ement because of the convoluted replacement logic due to the issues with multibyte strings

src/Processor/Processor.php

@@ -115,11 +115,10 @@ private function processIteration($text, ProcessorContext $context, ProcessedSho
 
     private function applyReplaces($text, array $replaces)
     {
-        $textLength = mb_strlen($text, 'utf-8');
-
-        return array_reduce(array_reverse($replaces), function($state, ReplacedShortcode $s) use($textLength) {
+        return array_reduce(array_reverse($replaces), function($state, ReplacedShortcode $s) {
             $offset = $s->getOffset();
             $length = mb_strlen($s->getText(), 'utf-8');
+            $textLength = mb_strlen($state, 'utf-8');
 
             return mb_substr($state, 0, $offset, 'utf-8').$s->getReplacement().mb_substr($state, $offset + $length, $textLength, 'utf-8');
         }, $text);

tests/ProcessorTest.php

@@ -111,6 +111,15 @@ public function testProcessorParentContext()
         $this->assertSame($result.$result, $processor->process($text.$text));
     }
 
+    public function testReplacesLongerThanInputText()
+    {
+        $handlers = new HandlerContainer();
+        $handlers->add('x', function() { return '<length>'; });
+        $processor = new Processor(new RegularParser(), $handlers);
+
+        $this->assertSame('<length><length><length>', $processor->process('[x][x][x]'));
+    }
+
     public function testProcessorWithoutRecursion()
     {
         $processor = new Processor(new RegexParser(), $this->getHandlers());