[
https://issues.apache.org/jira/browse/LANG-1780?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=18009272#comment-18009272
]
tza commented on LANG-1780:
---------------------------
I am not really familiar with Pull Requests / needed commit messages etc. in
this project.
Did my best, please let me know if you need adaptions.
https://github.com/apache/commons-lang/pull/1419
> Javadoc of new Strings API
> --------------------------
>
> Key: LANG-1780
> URL: https://issues.apache.org/jira/browse/LANG-1780
> Project: Commons Lang
> Issue Type: Improvement
> Components: lang.*
> Affects Versions: 3.18.0
> Reporter: tza
> Priority: Minor
>
> Several Javadoc of the new Strings-API was created with old method calls -
> where you had xx and xxIgnoreCase in StringUtils. Like contains and
> containsIgnoreCase looking at the new javadoc of Strings.CS/CI.contains:
>
> {code:java}
> * Strings.CI.containsIgnoreCase("abc", "A") = true
> * Strings.CI.containsIgnoreCase("abc", "Z") = false
> * </pre>
> *
> * @param seq the CharSequence to check, may be null
> * @param searchSeq the CharSequence to find, may be null
> * @return true if the CharSequence contains the search CharSequence, false if
> not or {@code null} string input */
> public abstract boolean contains(CharSequence seq, CharSequence
> searchSeq);{code}
>
> but CS/CI only supports the method contains (containsIgnoreCase was the old
> name from StringUtils). I would expect Strings.CI.contains() here in the
> examples.
>
> This javadoc is the same most likely for all the methods which had an
> IgnoreCase variant. Same with the javadoc of: compare, equals, indexOf,
> lastIndexOf, prependIfMissing, remove, removeEnd, removeStart, replace,
> replaceOnce, startsWith.
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
