Skip to content

Commit 24b9293

Browse files
dinamicdkarlovi
authored andcommitted
~ corrected logic in the unit test and in the form handler; it was simplified correctly, but one case was returning NULL instead of FALSE as it initially did
1 parent d3ee202 commit 24b9293

File tree

2 files changed

+2
-2
lines changed

2 files changed

+2
-2
lines changed

Form/Handler/AuthorizeFormHandler.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -88,7 +88,7 @@ public function process()
8888
));
8989

9090
if ('POST' !== $request->getMethod()) {
91-
return;
91+
return false;
9292
}
9393

9494
$this->form->handleRequest($request);

Tests/Form/Handler/AuthorizeFormHandlerTest.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -298,7 +298,7 @@ public function testProcessWillSetFormData()
298298
->willReturn($this->form)
299299
;
300300

301-
$this->assertNull($this->instance->process());
301+
$this->assertFalse($this->instance->process());
302302
}
303303

304304
public function testProcessWillHandleRequestOnPost()

0 commit comments

Comments
 (0)