Skip to content

Commit 69d0f42

Browse files
committed
[Misc] Apply mechanical SonarCloud code cleanups
Fixes 58 SonarCloud issues across 22 modules; all changes are mechanical and behaviour-preserving: * java:S1128 (26) - remove unused imports. * java:S1197 (18) - move array designators from the variable name to the type. * java:S1116 (14) - remove redundant empty statements. Co-Authored-By: Vincent Massol <vincent@massol.net>
1 parent ad50a7b commit 69d0f42

30 files changed

Lines changed: 31 additions & 58 deletions

File tree

xwiki-platform-core/xwiki-platform-captcha/xwiki-platform-captcha-jcaptcha/xwiki-platform-captcha-jcaptcha-api/src/test/java/org/xwiki/captcha/internal/CaptchaServiceManagerTest.java

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -21,19 +21,15 @@
2121

2222
import javax.inject.Named;
2323

24-
import org.junit.jupiter.api.BeforeEach;
2524
import org.junit.jupiter.api.Test;
2625
import org.xwiki.captcha.CaptchaException;
27-
import org.xwiki.component.phase.InitializationException;
2826
import org.xwiki.configuration.ConfigurationSource;
2927
import org.xwiki.test.junit5.mockito.ComponentTest;
3028
import org.xwiki.test.junit5.mockito.InjectMockComponents;
3129
import org.xwiki.test.junit5.mockito.MockComponent;
3230

3331
import static org.junit.jupiter.api.Assertions.assertEquals;
34-
import static org.junit.jupiter.api.Assertions.assertNotNull;
3532
import static org.junit.jupiter.api.Assertions.assertThrows;
36-
import static org.junit.jupiter.api.Assertions.assertTrue;
3733
import static org.mockito.Mockito.when;
3834

3935
/**

xwiki-platform-core/xwiki-platform-chart/xwiki-platform-chart-renderer/src/test/java/org/xwiki/chart/internal/DefaultChartGeneratorTest.java

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,6 @@
2222
import java.util.Map;
2323

2424
import org.jfree.chart.plot.Plot;
25-
import org.junit.jupiter.api.BeforeEach;
2625
import org.junit.jupiter.api.Test;
2726
import org.mockito.Mock;
2827
import org.xwiki.chart.ChartGenerator;

xwiki-platform-core/xwiki-platform-extension/xwiki-platform-extension-handlers/xwiki-platform-extension-handler-xar/src/main/java/org/xwiki/extension/xar/question/ConflictQuestion.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -109,7 +109,7 @@ public List<GlobalAction> getActions()
109109
{
110110
return this.actions;
111111
}
112-
};
112+
}
113113

114114
public enum GlobalAction
115115
{
@@ -143,7 +143,7 @@ public enum GlobalAction
143143
* configure default answers in the request for example.
144144
*/
145145
ASK
146-
};
146+
}
147147

148148
// Question datas
149149

xwiki-platform-core/xwiki-platform-filter/xwiki-platform-filter-streams/xwiki-platform-filter-stream-xar/src/main/java/org/xwiki/filter/xar/internal/XARDocumentModel.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -99,5 +99,5 @@ ELEMENT_REVISIONS, new EventParameter(XWikiWikiDocumentFilter.PARAMETER_JRCSREVI
9999
new EventParameter(WikiDocumentFilter.PARAMETER_TITLE));
100100
DOCUMENTREVISION_PARAMETERS.put(ELEMENT_VALIDATIONSCRIPT,
101101
new EventParameter(WikiDocumentFilter.PARAMETER_VALIDATIONSCRIPT));
102-
};
102+
}
103103
}

xwiki-platform-core/xwiki-platform-flamingo/xwiki-platform-flamingo-theme/xwiki-platform-flamingo-theme-ui/src/test/java/org/xwiki/flamingo/WebHomeSheetPageTest.java

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,6 @@
2727
import org.jsoup.nodes.Element;
2828
import org.junit.jupiter.api.BeforeEach;
2929
import org.junit.jupiter.api.Test;
30-
import org.xwiki.localization.macro.internal.TranslationMacro;
3130
import org.xwiki.model.reference.DocumentReference;
3231
import org.xwiki.model.script.ModelScriptService;
3332
import org.xwiki.query.Query;

xwiki-platform-core/xwiki-platform-help/xwiki-platform-help-ui/src/test/java/org/xwiki/help/XWikiSyntaxMacrosListPageTest.java

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -30,7 +30,6 @@
3030
import org.junit.jupiter.api.BeforeEach;
3131
import org.junit.jupiter.api.Test;
3232
import org.xwiki.context.internal.concurrent.DefaultContextStoreManager;
33-
import org.xwiki.localization.macro.internal.TranslationMacro;
3433
import org.xwiki.model.reference.DocumentReference;
3534
import org.xwiki.query.Query;
3635
import org.xwiki.query.script.QueryManagerScriptService;

xwiki-platform-core/xwiki-platform-image/xwiki-platform-image-style/xwiki-platform-image-style-ui/src/test/java/org/xwiki/image/style/AdministrationPageTest.java

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,6 @@
2828
import org.junit.jupiter.api.Test;
2929
import org.mockito.Mock;
3030
import org.xwiki.csrf.script.CSRFTokenScriptService;
31-
import org.xwiki.localization.macro.internal.TranslationMacro;
3231
import org.xwiki.model.reference.DocumentReference;
3332
import org.xwiki.model.script.ModelScriptService;
3433
import org.xwiki.script.service.ScriptService;

xwiki-platform-core/xwiki-platform-legacy/xwiki-platform-legacy-notifications/xwiki-platform-legacy-notifications-filters/xwiki-platform-legacy-notifications-filters-api/src/main/java/org/xwiki/notifications/filters/internal/LegacyDefaultNotificationFilterManager.java

Lines changed: 0 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -21,33 +21,19 @@
2121

2222
import java.util.Collection;
2323
import java.util.Collections;
24-
import java.util.HashMap;
25-
import java.util.HashSet;
2624
import java.util.Iterator;
27-
import java.util.List;
2825
import java.util.Map;
2926
import java.util.Set;
30-
import java.util.stream.Stream;
3127

3228
import javax.inject.Inject;
33-
import javax.inject.Named;
3429
import javax.inject.Singleton;
3530

3631
import org.xwiki.component.annotation.Component;
37-
import org.xwiki.component.manager.ComponentLookupException;
38-
import org.xwiki.component.manager.ComponentManager;
39-
import org.xwiki.model.ModelContext;
4032
import org.xwiki.model.reference.DocumentReference;
41-
import org.xwiki.model.reference.WikiReference;
4233
import org.xwiki.notifications.NotificationConfiguration;
4334
import org.xwiki.notifications.NotificationException;
4435
import org.xwiki.notifications.filters.NotificationFilter;
45-
import org.xwiki.notifications.filters.NotificationFilterDisplayer;
4636
import org.xwiki.notifications.filters.NotificationFilterManager;
47-
import org.xwiki.notifications.filters.NotificationFilterPreference;
48-
import org.xwiki.notifications.preferences.NotificationPreference;
49-
import org.xwiki.rendering.block.Block;
50-
import org.xwiki.wiki.descriptor.WikiDescriptorManager;
5137

5238
/**
5339
* Default implementation of {@link NotificationFilterManager}.

xwiki-platform-core/xwiki-platform-localization/xwiki-platform-localization-api/src/main/java/org/xwiki/localization/internal/AbstractTranslationBundle.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -44,7 +44,7 @@ public abstract class AbstractTranslationBundle implements TranslationBundle
4444
public Translation getTranslation(String key, Locale locale)
4545
{
4646
return null;
47-
};
47+
}
4848
};
4949

5050
/**

xwiki-platform-core/xwiki-platform-logging/xwiki-platform-logging-ui/src/test/java/org/xwiki/logging/LoggingAdminPageTest.java

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,6 @@
2626
import org.junit.jupiter.api.Test;
2727
import org.xwiki.icon.IconManager;
2828
import org.xwiki.livedata.internal.macro.LiveDataMacroComponentList;
29-
import org.xwiki.localization.macro.internal.TranslationMacro;
3029
import org.xwiki.model.reference.DocumentReference;
3130
import org.xwiki.properties.internal.converter.EnumConverter;
3231
import org.xwiki.rendering.RenderingScriptServiceComponentList;

0 commit comments

Comments
 (0)