Skip to content

Skip empty strings when using capitalize #751#757

Merged
TobiasNx merged 2 commits into
masterfrom
751-skipEmptyStringWhenCapitalize
Apr 13, 2026
Merged

Skip empty strings when using capitalize #751#757
TobiasNx merged 2 commits into
masterfrom
751-skipEmptyStringWhenCapitalize

Conversation

@TobiasNx
Copy link
Copy Markdown
Contributor

Skip empty strings when using capitalize #751

Quick solution.

@TobiasNx TobiasNx requested a review from blackwinter March 18, 2026 09:15
@TobiasNx TobiasNx linked an issue Mar 18, 2026 that may be closed by this pull request
@TobiasNx TobiasNx marked this pull request as draft March 18, 2026 09:18
Comment thread metafix/src/main/java/org/metafacture/metafix/method/field/Capitalize.java Outdated
@blackwinter blackwinter assigned TobiasNx and unassigned blackwinter Mar 18, 2026
Co-authored-by: Jens Wille <jens.wille@hbz-nrw.de>
@TobiasNx TobiasNx marked this pull request as ready for review April 13, 2026 11:08
@TobiasNx TobiasNx requested a review from fsteeg April 13, 2026 11:08
@TobiasNx TobiasNx assigned blackwinter and fsteeg and unassigned TobiasNx Apr 13, 2026
@TobiasNx
Copy link
Copy Markdown
Contributor Author

I used the suggested code from @blackwinter.
@fsteeg you could functional review this scenario.

@TobiasNx TobiasNx requested a review from blackwinter April 13, 2026 11:09
@blackwinter blackwinter removed their assignment Apr 13, 2026
@fsteeg fsteeg assigned TobiasNx and unassigned fsteeg Apr 13, 2026
@TobiasNx TobiasNx merged commit 4a2743a into master Apr 13, 2026
2 checks passed
@TobiasNx TobiasNx deleted the 751-skipEmptyStringWhenCapitalize branch April 13, 2026 14:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Fix: capitalize breaks with empty strings

3 participants