Merge pull request #246 from mzur/patch-1

Fix EmailSelect action cleaning of 'recipient'
mzur · Apr 5, 2023 · 4d1dbe4 · 4d1dbe4
2 parents 547040e + db46be6
commit 4d1dbe4
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/src/Actions/EmailSelectAction.php b/src/Actions/EmailSelectAction.php
@@ -22,7 +22,7 @@ class EmailSelectAction extends EmailAction
     public function perform()
     {
         $this->options['to'] = $this->getRecipient();
-        unset($this->data[self::RECIPIENT_FIELD]);
+        $this->form->forget(self::RECIPIENT_FIELD);
         unset($this->options['allowed-recipients']);
 
         return parent::perform();

diff --git a/tests/Actions/EmailSelectActionTest.php b/tests/Actions/EmailSelectActionTest.php
@@ -36,7 +36,7 @@ public function testRecipient()
         ]);
         $action->perform();
         $this->assertEquals('[email protected]', $action->params['to']);
-        $this->assertFalse(array_key_exists('recipient', $action->params));
+        $this->assertArrayNotHasKey('recipient', $action->params['data']);
     }
 
     public function testRecipientNotAllowed()