malhuda/CodeIgniter4
1
0
Fork 0
mirror of https://github.com/codeigniter4/CodeIgniter4.git synced 2025-02-20 11:44:28 +08:00
Code Issues Actions Packages Projects Releases Wiki Activity

Merge remote-tracking branch 'upstream/develop' into 4.4

Browse Source
This commit is contained in:
kenjis 2023-08-22 09:47:37 +09:00
commit e80b66f1bb
No known key found for this signature in database
GPG Key ID: BD254878922AF198
2 changed files with 27 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
system/Helpers/form_helper.php
View File

@ -629,8 +629,11 @@ if (! function_exists('set_checkbox')) {
return '';
}
$session = Services::session();
$hasOldInput = $session->has('_ci_old_input');
// Unchecked checkbox and radio inputs are not even submitted by browsers ...
if ((string) $input === '0' || ! empty($request->getPost()) || ! empty(old($field))) {
if ((string) $input === '0' || ! empty($request->getPost()) || $hasOldInput) {
return ($input === $value) ? ' checked="checked"' : '';
}

23
tests/system/Helpers/FormHelperTest.php
View File

@ -866,6 +866,29 @@ final class FormHelperTest extends CIUnitTestCase
$this->assertSame(' checked="checked"', set_checkbox('foo', '0', true));
}
/**
* @see https://github.com/codeigniter4/CodeIgniter4/issues/7814
*/
public function testSetCheckboxWithUnchecked(): void
{
$_SESSION = [
'_ci_old_input' => [
'post' => [
],
],
];
$this->assertSame(
'',
set_checkbox('fruit', 'apple', true)
);
$this->assertSame(
'',
set_checkbox('fruit', 'apple')
);
}
/**
* @runInSeparateProcess
* @preserveGlobalState disabled