Merge pull request #14 from clue-labs/redraw-echo

Do not redraw() readline if echo replacement results in same line

Author: clue

src/Readline.php

@@ -203,11 +203,15 @@ public function setInput($input)
             return $this;
         }
 
+        // remember old input length if echo replacement is used
+        $oldlen = (is_string($this->echo)) ? $this->strlen($this->linebuffer) : null;
+
         $this->linebuffer = $input;
         $this->linepos = $this->strlen($this->linebuffer);
 
         // only redraw if input should be echo'ed (i.e. is not hidden anyway)
-        if ($this->echo !== false) {
+        // and echo replacement is used, make sure the input length changes
+        if ($this->echo !== false && $this->linepos !== $oldlen) {
             $this->redraw();
         }
 

tests/ReadlineTest.php

@@ -124,6 +124,16 @@ public function testSettingInputWithEchoAsteriskDoesRedraw()
         $this->assertSame($this->readline, $this->readline->setInput('test'));
     }
 
+    public function testSettingInputWithSameLengthWithEchoAsteriskDoesNotNeedToRedraw()
+    {
+        $this->readline->setInput('test');
+        $this->readline->setEcho('*');
+
+        $this->output->expects($this->never())->method('write');
+
+        $this->assertSame($this->readline, $this->readline->setInput('demo'));
+    }
+
     public function testSettingInputWithoutEchoDoesNotNeedToRedraw()
     {
         $this->readline->setEcho(false);