Skip to content

docs: further specify alpha-rules and the permit_empty rule - #9818

Merged
michalsn merged 4 commits into
codeigniter4:developfrom
wipped:patch-1
Dec 5, 2025
Merged

docs: further specify alpha-rules and the permit_empty rule#9818
michalsn merged 4 commits into
codeigniter4:developfrom
wipped:patch-1

Conversation

@wipped

Copy link
Copy Markdown
Contributor

Description
Specified that the alpha-rules fail on empty strings and that permit_empty will skip all other rules except for required_with and required_without.

Specified that the alpha-rules fail on empty strings and that permit_empty will skip all other rules except for required_with and required_without.
@mergeable

mergeableBot commented Dec 1, 2025

Copy link
Copy Markdown

Hi there, wipped! 👋

Thank you for sending this PR!

We expect the following in all Pull Requests (PRs).

Important

We expect all code changes or bug-fixes to be accompanied by one or more tests added to our test suite to prove the code works.

If pull requests do not comply with the above, they will likely be closed. Since we are a team of volunteers, we don't have any more time to work
on the framework than you do. Please make it as painless for your contributions to be included as possible.

See https://github.com/codeigniter4/CodeIgniter4/blob/develop/contributing/pull_request.md

Sincerely, the mergeable bot 🤖

Comment threaduser_guide_src/source/libraries/validation.rst Outdated
Comment threaduser_guide_src/source/libraries/validation.rst
Clarify behavior of permit_empty validation rule.
Comment threaduser_guide_src/source/libraries/validation.rst Outdated
@michalsnmichalsn added the documentation Pull requests for documentation only label Dec 4, 2025
Clarify the behavior of the 'permit_empty' validation rule.
@michalsn
michalsn merged commit b36e7d3 into codeigniter4:developDec 5, 2025
4 checks passed
@michalsn

Copy link
Copy Markdown
Member

Thank you @wipped

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentationPull requests for documentation only

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@wipped@michalsn@neznaika0@paulbalandan