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

KuiLIU closed STANBOL-1469.
---------------------------
    Resolution: Incomplete

> Inconsistent method name
> ------------------------
>
>                 Key: STANBOL-1469
>                 URL: https://issues.apache.org/jira/browse/STANBOL-1469
>             Project: Stanbol
>          Issue Type: Improvement
>            Reporter: KuiLIU
>            Priority: Major
>         Attachments: rename-methods.patch
>
>
> The following method is name "select" which seems inconsistent with the 
> method body code.
> "select" is prone to an action of selecting something.
> But the method is checking whether the status of the parameter "collection" 
> is specified or not, which is a query.
> So, the name "isSelected" should be more clear than "select" to describe the 
> method.
> {code:java}
> public boolean select(Collection collection) {
>         for (String feature : features) {
>             Status status = FeaturesHelper.getFeatureStatus(collection, 
> feature);
>             if (status == Status.SPECIFIED)
>                 return true;
>         }
>         return false;
>     }
> {code}



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Reply via email to