[ 
https://issues.apache.org/jira/browse/LANG-1780?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

tza resolved LANG-1780.
-----------------------
    Fix Version/s: 3.18.1
       Resolution: Fixed

Fixed with PR 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
>             Fix For: 3.18.1
>
>
> 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)

Reply via email to