Fix checkstyle

AB-xdev · AB-xdev · commit 1b7bfea2d936 · 2024-01-11T17:15:08.000+01:00
diff --git a/vaadin-maps-leaflet-flow-demo/src/main/java/software/xdev/vaadin/maps/leaflet/flow/demo/ComplexDemo.java b/vaadin-maps-leaflet-flow-demo/src/main/java/software/xdev/vaadin/maps/leaflet/flow/demo/ComplexDemo.java
@@ -47,6 +47,7 @@
 
 
 @Route(ComplexDemo.NAV)
+@SuppressWarnings("checkstyle:MagicNumber")
 public class ComplexDemo extends AbstractDemo
 {
 	public static final String NAV = "/complex";
@@ -106,6 +107,7 @@ public ComplexDemo()
 			new LLatLng(this.reg, 49.674662, 12.162869),
 			new LMarkerOptions().withIcon(divIconInfo));
 		
+		@SuppressWarnings("checkstyle:LineLength")
 		final LIcon iconXDEV = new LIcon(this.reg, new LIconOptions()
 			.withIconUrl("""
 				data:image/svg+xml;utf8,<svg xmlns="http://www.w3.org/2000/svg" width="1000" height="200" viewBox="0 0 18300 4500" style="background-color:rgba(180,180,180,0.7)">
diff --git a/vaadin-maps-leaflet-flow-demo/src/main/java/software/xdev/vaadin/maps/leaflet/flow/demo/FreeingUpResourceBenchmarkDemo.java b/vaadin-maps-leaflet-flow-demo/src/main/java/software/xdev/vaadin/maps/leaflet/flow/demo/FreeingUpResourceBenchmarkDemo.java
@@ -23,6 +23,7 @@
 
 
 @Route(FreeingUpResourceBenchmarkDemo.NAV)
+@SuppressWarnings("checkstyle:MagicNumber")
 public class FreeingUpResourceBenchmarkDemo extends AbstractDemo
 {
 	public static final String NAV = "/freeing-up-resources-benchmark";
diff --git a/vaadin-maps-leaflet-flow-demo/src/main/java/software/xdev/vaadin/maps/leaflet/flow/demo/MinimalisticDemo.java b/vaadin-maps-leaflet-flow-demo/src/main/java/software/xdev/vaadin/maps/leaflet/flow/demo/MinimalisticDemo.java
@@ -13,6 +13,7 @@
 
 
 @Route(MinimalisticDemo.NAV)
+@SuppressWarnings("checkstyle:MagicNumber")
 public class MinimalisticDemo extends VerticalLayout
 {
 	public static final String NAV = "/minimalistic";
diff --git a/vaadin-maps-leaflet-flow/src/main/java/software/xdev/vaadin/maps/leaflet/basictypes/LLatLng.java b/vaadin-maps-leaflet-flow/src/main/java/software/xdev/vaadin/maps/leaflet/basictypes/LLatLng.java
@@ -29,7 +29,11 @@ public LLatLng(final LComponentManagementRegistry componentRegistry, final doubl
 		super(componentRegistry, "L.latLng($0, $1)", lat, lng);
 	}
 	
-	public LLatLng(final LComponentManagementRegistry componentRegistry, final double lat, final double lng, final double altitude)
+	public LLatLng(
+		final LComponentManagementRegistry componentRegistry,
+		final double lat,
+		final double lng,
+		final double altitude)
 	{
 		super(componentRegistry, "L.latLng($0, $1, $2)", lat, lng, altitude);
 	}
diff --git a/vaadin-maps-leaflet-flow/src/main/java/software/xdev/vaadin/maps/leaflet/basictypes/LLatLngBounds.java b/vaadin-maps-leaflet-flow/src/main/java/software/xdev/vaadin/maps/leaflet/basictypes/LLatLngBounds.java
@@ -29,6 +29,7 @@ public LLatLngBounds(
 		final LLatLng corner1,
 		final LLatLng corner2)
 	{
-		super(compReg, "L.latLngBounds(" + corner1.clientComponentJsAccessor() + "," + corner2.clientComponentJsAccessor() + ")");
+		super(compReg, "L.latLngBounds(" + corner1.clientComponentJsAccessor()
+			+ "," + corner2.clientComponentJsAccessor() + ")");
 	}
 }
diff --git a/vaadin-maps-leaflet-flow/src/main/java/software/xdev/vaadin/maps/leaflet/layer/LAbstractImageOverlayOptions.java b/vaadin-maps-leaflet-flow/src/main/java/software/xdev/vaadin/maps/leaflet/layer/LAbstractImageOverlayOptions.java
@@ -15,7 +15,8 @@
  */
 package software.xdev.vaadin.maps.leaflet.layer;
 
-public abstract class LAbstractImageOverlayOptions<S extends LAbstractImageOverlayOptions<S>> extends LInteractiveLayerOptions<S>
+public abstract class LAbstractImageOverlayOptions<S extends LAbstractImageOverlayOptions<S>>
+	extends LInteractiveLayerOptions<S>
 {
 	private Double opacity;
 	private String alt;
diff --git a/vaadin-maps-leaflet-flow/src/main/java/software/xdev/vaadin/maps/leaflet/layer/LLayer.java b/vaadin-maps-leaflet-flow/src/main/java/software/xdev/vaadin/maps/leaflet/layer/LLayer.java
@@ -111,7 +111,9 @@ public S removeFrom(final LLayerGroup layerGroup)
 	 */
 	public S bindPopup(final String content, final LPopupOptions options)
 	{
-		this.invokeSelf(".bindPopup($0, " + this.componentRegistry().writeOptionsOptionalParameter(options) + ")", content);
+		this.invokeSelf(
+			".bindPopup($0, " + this.componentRegistry().writeOptionsOptionalParameter(options) + ")",
+			content);
 		return this.self();
 	}
 	
@@ -184,7 +186,9 @@ public S closePopup()
 	 */
 	public S bindTooltip(final String content, final LTooltipOptions options)
 	{
-		this.invokeSelf(".bindTooltip($0, " + this.componentRegistry().writeOptionsOptionalParameter(options) + ")", content);
+		this.invokeSelf(
+			".bindTooltip($0, " + this.componentRegistry().writeOptionsOptionalParameter(options) + ")",
+			content);
 		return this.self();
 	}
 	
diff --git a/vaadin-maps-leaflet-flow/src/main/java/software/xdev/vaadin/maps/leaflet/layer/LLayerGroup.java b/vaadin-maps-leaflet-flow/src/main/java/software/xdev/vaadin/maps/leaflet/layer/LLayerGroup.java
@@ -36,10 +36,14 @@ public LLayerGroup(
 			compReg,
 			"L.layerGroup("
 				+ (layers != null
-				? "[" + Stream.of(layers).map(LLayer::clientComponentJsAccessor).collect(Collectors.joining(",")) + "]"
+				? "[" + Stream.of(layers)
+				.map(LLayer::clientComponentJsAccessor)
+				.collect(Collectors.joining(",")) + "]"
 				: "")
 				+ (options != null
-				? ", ["+ Stream.of(options).map(compReg::writeOptionsOrEmptyObject).collect(Collectors.joining(","))+ "]"
+				? ", [" + Stream.of(options)
+				.map(compReg::writeOptionsOrEmptyObject)
+				.collect(Collectors.joining(",")) + "]"
 				: "")
 				+ ")");
 	}
diff --git a/vaadin-maps-leaflet-flow/src/main/java/software/xdev/vaadin/maps/leaflet/layer/raster/LTileLayer.java b/vaadin-maps-leaflet-flow/src/main/java/software/xdev/vaadin/maps/leaflet/layer/raster/LTileLayer.java
@@ -60,6 +60,7 @@ public LTileLayer(
 	 * Use OpenStreetMap's tile server <br/>
 	 * <b>Important: By using this you accept the <a href="https://operations.osmfoundation.org/policies/tiles/">Usage Policy</a></b>
 	 */
+	@SuppressWarnings("checkstyle:MagicNumber") // Default max Zoom
 	public static LTileLayer createDefaultForOpenStreetMapTileServer(final LComponentManagementRegistry compReg)
 	{
 		return new LTileLayer(
diff --git a/vaadin-maps-leaflet-flow/src/main/java/software/xdev/vaadin/maps/leaflet/layer/ui/LMarker.java b/vaadin-maps-leaflet-flow/src/main/java/software/xdev/vaadin/maps/leaflet/layer/ui/LMarker.java
@@ -43,7 +43,10 @@ public LMarker(
 	{
 		super(
 			compReg,
-			"L.marker(" + compReg.clientComponentJsAccessor(latLng) + compReg.writeOptionsOptionalParameter(options) + ")");
+			"L.marker("
+				+ compReg.clientComponentJsAccessor(latLng)
+				+ compReg.writeOptionsOptionalParameter(options)
+				+ ")");
 	}
 	
 	/**
diff --git a/vaadin-maps-leaflet-flow/src/main/java/software/xdev/vaadin/maps/leaflet/layer/ui/LPopup.java b/vaadin-maps-leaflet-flow/src/main/java/software/xdev/vaadin/maps/leaflet/layer/ui/LPopup.java
@@ -27,7 +27,10 @@ public LPopup(
 		final LPopupOptions options,
 		final LLayer<?> source)
 	{
-		super(compReg, "L.popup(" + compReg.writeOptions(options) + (source != null ? ", " + source.clientComponentJsAccessor() : "") + ")");
+		super(compReg, "L.popup("
+			+ compReg.writeOptions(options)
+			+ (source != null ? ", " + source.clientComponentJsAccessor() : "")
+			+ ")");
 	}
 	
 	public LPopup(
@@ -47,7 +50,10 @@ public LPopup(
 		final LLatLng latLng,
 		final LPopupOptions options)
 	{
-		super(compReg, "L.popup(" + latLng.clientComponentJsAccessor() + compReg.writeOptionsOptionalParameter(options) + ")");
+		super(compReg, "L.popup("
+			+ latLng.clientComponentJsAccessor()
+			+ compReg.writeOptionsOptionalParameter(options)
+			+ ")");
 	}
 	
 	public LPopup(
diff --git a/vaadin-maps-leaflet-flow/src/main/java/software/xdev/vaadin/maps/leaflet/layer/ui/LTooltip.java b/vaadin-maps-leaflet-flow/src/main/java/software/xdev/vaadin/maps/leaflet/layer/ui/LTooltip.java
@@ -27,7 +27,10 @@ public LTooltip(
 		final LTooltipOptions options,
 		final LLayer<?> source)
 	{
-		super(compReg, "L.tooltip(" + compReg.writeOptions(options) + (source != null ? ", " + source.clientComponentJsAccessor() : "") + ")");
+		super(compReg, "L.tooltip("
+			+ compReg.writeOptions(options)
+			+ (source != null ? ", " + source.clientComponentJsAccessor() : "")
+			+ ")");
 	}
 	
 	public LTooltip(
@@ -47,7 +50,10 @@ public LTooltip(
 		final LLatLng latLng,
 		final LTooltipOptions options)
 	{
-		super(compReg, "L.tooltip(" + latLng.clientComponentJsAccessor() + compReg.writeOptionsOptionalParameter(options) + ")");
+		super(compReg, "L.tooltip("
+			+ latLng.clientComponentJsAccessor()
+			+ compReg.writeOptionsOptionalParameter(options)
+			+ ")");
 	}
 	
 	public LTooltip(
diff --git a/vaadin-maps-leaflet-flow/src/main/java/software/xdev/vaadin/maps/leaflet/map/LMapZoomPanOptions.java b/vaadin-maps-leaflet-flow/src/main/java/software/xdev/vaadin/maps/leaflet/map/LMapZoomPanOptions.java
@@ -18,7 +18,8 @@
 /**
  * @see <a href="https://leafletjs.com/reference.html#zoom/pan-options">Leaflet docs</a>
  */
-public class LMapZoomPanOptions implements LMapZoomBaseOptions<LMapZoomPanOptions>, LMapPanBaseOptions<LMapZoomPanOptions>
+public class LMapZoomPanOptions
+	implements LMapZoomBaseOptions<LMapZoomPanOptions>, LMapPanBaseOptions<LMapZoomPanOptions>
 {
 	private Boolean animate;
 	private Double duration;
diff --git a/vaadin-maps-leaflet-flow/src/main/java/software/xdev/vaadin/maps/leaflet/registry/LDefaultComponentManagementRegistry.java b/vaadin-maps-leaflet-flow/src/main/java/software/xdev/vaadin/maps/leaflet/registry/LDefaultComponentManagementRegistry.java
@@ -122,7 +122,9 @@ public <C extends LComponent<C>> C add(
 	{
 		final int currentId = this.nextComponentId.getAndIncrement();
 		this.getElement()
-			.executeJs(this.clientComponents() + ".set(" + currentId + ", " + jsConstructorCallExpression + ");", parameters);
+			.executeJs(
+				this.clientComponents() + ".set(" + currentId + ", " + jsConstructorCallExpression + ");",
+				parameters);
 		this.clientMapSize.incrementAndGet();
 		this.componentIndexMap.put(component, currentId);
 		

Original file line number	Diff line number	Diff line change
`@@ -23,6 +23,7 @@`
`23`	`23`
`24`	`24`
`25`	`25`	`@Route(FreeingUpResourceBenchmarkDemo.NAV)`
	`26`	`+@SuppressWarnings("checkstyle:MagicNumber")`
`26`	`27`	`public class FreeingUpResourceBenchmarkDemo extends AbstractDemo`
`27`	`28`	`{`
`28`	`29`	`public static final String NAV = "/freeing-up-resources-benchmark";`
Original file line number	Diff line number	Diff line change
`@@ -13,6 +13,7 @@`
`13`	`13`
`14`	`14`
`15`	`15`	`@Route(MinimalisticDemo.NAV)`
	`16`	`+@SuppressWarnings("checkstyle:MagicNumber")`
`16`	`17`	`public class MinimalisticDemo extends VerticalLayout`
`17`	`18`	`{`
`18`	`19`	`public static final String NAV = "/minimalistic";`
Original file line number	Diff line number	Diff line change
`@@ -29,7 +29,11 @@ public LLatLng(final LComponentManagementRegistry componentRegistry, final doubl`
`29`	`29`	`super(componentRegistry, "L.latLng($0, $1)", lat, lng);`
`30`	`30`	`}`
`31`	`31`
`32`		`- public LLatLng(final LComponentManagementRegistry componentRegistry, final double lat, final double lng, final double altitude)`
	`32`	`+ public LLatLng(`
	`33`	`+ final LComponentManagementRegistry componentRegistry,`
	`34`	`+ final double lat,`
	`35`	`+ final double lng,`
	`36`	`+ final double altitude)`
`33`	`37`	`{`
`34`	`38`	`super(componentRegistry, "L.latLng($0, $1, $2)", lat, lng, altitude);`
`35`	`39`	`}`
Original file line number	Diff line number	Diff line change
`@@ -29,6 +29,7 @@ public LLatLngBounds(`
`29`	`29`	`final LLatLng corner1,`
`30`	`30`	`final LLatLng corner2)`
`31`	`31`	`{`
`32`		`- super(compReg, "L.latLngBounds(" + corner1.clientComponentJsAccessor() + "," + corner2.clientComponentJsAccessor() + ")");`
	`32`	`+ super(compReg, "L.latLngBounds(" + corner1.clientComponentJsAccessor()`
	`33`	`+ + "," + corner2.clientComponentJsAccessor() + ")");`
`33`	`34`	`}`
`34`	`35`	`}`
Original file line number	Diff line number	Diff line change
`@@ -15,7 +15,8 @@`
`15`	`15`	`*/`
`16`	`16`	`package software.xdev.vaadin.maps.leaflet.layer;`
`17`	`17`
`18`		`-public abstract class LAbstractImageOverlayOptions<S extends LAbstractImageOverlayOptions<S>> extends LInteractiveLayerOptions<S>`
	`18`	`+public abstract class LAbstractImageOverlayOptions<S extends LAbstractImageOverlayOptions<S>>`
	`19`	`+ extends LInteractiveLayerOptions<S>`
`19`	`20`	`{`
`20`	`21`	`private Double opacity;`
`21`	`22`	`private String alt;`
Original file line number	Diff line number	Diff line change
`@@ -111,7 +111,9 @@ public S removeFrom(final LLayerGroup layerGroup)`
`111`	`111`	`*/`
`112`	`112`	`public S bindPopup(final String content, final LPopupOptions options)`
`113`	`113`	`{`
`114`		`- this.invokeSelf(".bindPopup($0, " + this.componentRegistry().writeOptionsOptionalParameter(options) + ")", content);`
	`114`	`+ this.invokeSelf(`
	`115`	`+ ".bindPopup($0, " + this.componentRegistry().writeOptionsOptionalParameter(options) + ")",`
	`116`	`+ content);`
`115`	`117`	`return this.self();`
`116`	`118`	`}`
`117`	`119`
`@@ -184,7 +186,9 @@ public S closePopup()`
`184`	`186`	`*/`
`185`	`187`	`public S bindTooltip(final String content, final LTooltipOptions options)`
`186`	`188`	`{`
`187`		`- this.invokeSelf(".bindTooltip($0, " + this.componentRegistry().writeOptionsOptionalParameter(options) + ")", content);`
	`189`	`+ this.invokeSelf(`
	`190`	`+ ".bindTooltip($0, " + this.componentRegistry().writeOptionsOptionalParameter(options) + ")",`
	`191`	`+ content);`
`188`	`192`	`return this.self();`
`189`	`193`	`}`
`190`	`194`
Original file line number	Diff line number	Diff line change
`@@ -60,6 +60,7 @@ public LTileLayer(`
`60`	`60`	`* Use OpenStreetMap's tile server <br/>`
`61`	`61`	`* <b>Important: By using this you accept the <a href="https://operations.osmfoundation.org/policies/tiles/">Usage Policy</a></b>`
`62`	`62`	`*/`
	`63`	`+ @SuppressWarnings("checkstyle:MagicNumber") // Default max Zoom`
`63`	`64`	`public static LTileLayer createDefaultForOpenStreetMapTileServer(final LComponentManagementRegistry compReg)`
`64`	`65`	`{`
`65`	`66`	`return new LTileLayer(`
Original file line number	Diff line number	Diff line change
`@@ -43,7 +43,10 @@ public LMarker(`
`43`	`43`	`{`
`44`	`44`	`super(`
`45`	`45`	`compReg,`
`46`		`- "L.marker(" + compReg.clientComponentJsAccessor(latLng) + compReg.writeOptionsOptionalParameter(options) + ")");`
	`46`	`+ "L.marker("`
	`47`	`+ + compReg.clientComponentJsAccessor(latLng)`
	`48`	`+ + compReg.writeOptionsOptionalParameter(options)`
	`49`	`+ + ")");`
`47`	`50`	`}`
`48`	`51`
`49`	`52`	`/**`