You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@karaf.apache.org by "Jean-Baptiste Onofré (JIRA)" <ji...@apache.org> on 2017/04/24 08:18:04 UTC
[jira] [Assigned] (KARAF-4986) A condition that is always true.
[ https://issues.apache.org/jira/browse/KARAF-4986?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Jean-Baptiste Onofré reassigned KARAF-4986:
-------------------------------------------
Assignee: Jean-Baptiste Onofré
> A condition that is always true.
> --------------------------------
>
> Key: KARAF-4986
> URL: https://issues.apache.org/jira/browse/KARAF-4986
> Project: Karaf
> Issue Type: Bug
> Components: cellar-features
> Reporter: JC
> Assignee: Jean-Baptiste Onofré
> Priority: Trivial
> Fix For: cellar-4.0.4, cellar-4.1.0
>
>
> I've found code smells and typos in a recent github snapshot. (karaf-cellar)
> Path: features/src/main/java/org/apache/karaf/cellar/features/management/internal/CellarFeaturesMBeanImpl.java
> {code:java}
> 559 for (String url : urls) {
> 560 // looking for the URL in the list
> 561 boolean found = false;
> 562 for (String repository : clusterRepositories.keySet()) {
> 563 if (repository.equals(repository)) {
> 564 found = true;
> 565 break;
> 566 }
> 567 }
> 568 if (found) {
> 569 Features repositoryModel = JaxbUtil.unmarshal(url, true);
> {code}
> The line 563 should be like this?
> {code:java}
> 563 if (repository.equals(url)) {
> {code}
> There is a similar issue in the following path as well: features/src/main/java/org/apache/karaf/cellar/features/shell/RepoRemoveCommand.java
> {code:java}
> 116 for (String url : urls) {
> 117 // looking for the URL in the list
> 118 boolean found = false;
> 119 for (String repository : clusterRepositories.keySet()) {
> 120 if (repository.equals(repository)) {
> 121 found = true;
> 122 break;
> 123 }
> {code}
> This might be trivial issues but wanted to report just in case.
--
This message was sent by Atlassian JIRA
(v6.3.15#6346)