From 5dad6524e4e6fc235ef9b22783f9cfc36bc111c4 Mon Sep 17 00:00:00 2001 From: lsiepel Date: Thu, 19 Oct 2023 21:41:00 +0200 Subject: [PATCH] [jeelink] adapt to core StringUtils (#15771) Signed-off-by: Leo Siepel Signed-off-by: querdenker2k --- .../LaCrosseTemperatureSensorHandler.java | 6 ++- .../internal/lacrosse/LgwSensorHandler.java | 9 ++-- .../jeelink/internal/util/StringUtils.java | 48 ------------------- 3 files changed, 10 insertions(+), 53 deletions(-) delete mode 100644 bundles/org.openhab.binding.jeelink/src/main/java/org/openhab/binding/jeelink/internal/util/StringUtils.java diff --git a/bundles/org.openhab.binding.jeelink/src/main/java/org/openhab/binding/jeelink/internal/lacrosse/LaCrosseTemperatureSensorHandler.java b/bundles/org.openhab.binding.jeelink/src/main/java/org/openhab/binding/jeelink/internal/lacrosse/LaCrosseTemperatureSensorHandler.java index 91d54179f4b6e..82f69ab5265a9 100644 --- a/bundles/org.openhab.binding.jeelink/src/main/java/org/openhab/binding/jeelink/internal/lacrosse/LaCrosseTemperatureSensorHandler.java +++ b/bundles/org.openhab.binding.jeelink/src/main/java/org/openhab/binding/jeelink/internal/lacrosse/LaCrosseTemperatureSensorHandler.java @@ -20,13 +20,13 @@ import java.util.HashMap; import java.util.List; import java.util.Map; +import java.util.Objects; import org.openhab.binding.jeelink.internal.JeeLinkSensorHandler; import org.openhab.binding.jeelink.internal.ReadingPublisher; import org.openhab.binding.jeelink.internal.RollingAveragePublisher; import org.openhab.binding.jeelink.internal.RollingReadingAverage; import org.openhab.binding.jeelink.internal.config.LaCrosseTemperatureSensorConfig; -import org.openhab.binding.jeelink.internal.util.StringUtils; import org.openhab.core.library.types.OnOffType; import org.openhab.core.library.types.QuantityType; import org.openhab.core.library.unit.SIUnits; @@ -38,6 +38,7 @@ import org.openhab.core.thing.binding.builder.ChannelBuilder; import org.openhab.core.thing.binding.builder.ThingBuilder; import org.openhab.core.thing.type.ChannelTypeUID; +import org.openhab.core.util.StringUtils; import org.slf4j.Logger; import org.slf4j.LoggerFactory; @@ -93,7 +94,8 @@ private void createMissingChannels(int channelNo) { missingChannels.add(ChannelBuilder .create(new ChannelUID(getThing().getUID(), channelName + idSuffix), "Number") .withType(new ChannelTypeUID(getThing().getThingTypeUID().getBindingId(), channelName)) - .withLabel(StringUtils.capitalize(channelName + labelSuffix)).build()); + .withLabel(Objects.requireNonNull(StringUtils.capitalize(channelName + labelSuffix))) + .build()); } } missingChannels.addAll(getThing().getChannels()); diff --git a/bundles/org.openhab.binding.jeelink/src/main/java/org/openhab/binding/jeelink/internal/lacrosse/LgwSensorHandler.java b/bundles/org.openhab.binding.jeelink/src/main/java/org/openhab/binding/jeelink/internal/lacrosse/LgwSensorHandler.java index 3ad324bd03f00..b57becfbd37d8 100644 --- a/bundles/org.openhab.binding.jeelink/src/main/java/org/openhab/binding/jeelink/internal/lacrosse/LgwSensorHandler.java +++ b/bundles/org.openhab.binding.jeelink/src/main/java/org/openhab/binding/jeelink/internal/lacrosse/LgwSensorHandler.java @@ -17,11 +17,11 @@ import java.math.BigDecimal; import java.math.RoundingMode; +import java.util.Objects; import org.eclipse.jdt.annotation.NonNullByDefault; import org.openhab.binding.jeelink.internal.JeeLinkSensorHandler; import org.openhab.binding.jeelink.internal.ReadingPublisher; -import org.openhab.binding.jeelink.internal.util.StringUtils; import org.openhab.core.library.types.QuantityType; import org.openhab.core.library.unit.SIUnits; import org.openhab.core.library.unit.Units; @@ -31,6 +31,7 @@ import org.openhab.core.thing.binding.builder.ChannelBuilder; import org.openhab.core.thing.binding.builder.ThingBuilder; import org.openhab.core.thing.type.ChannelTypeUID; +import org.openhab.core.util.StringUtils; import org.slf4j.Logger; import org.slf4j.LoggerFactory; @@ -78,7 +79,8 @@ public void publish(LgwReading reading) { .create(new ChannelUID(getThing().getUID(), HUMIDITY_CHANNEL), "Number:Humidity") .withType(new ChannelTypeUID(getThing().getThingTypeUID().getBindingId(), HUMIDITY_CHANNEL)) - .withLabel(StringUtils.capitalize(HUMIDITY_CHANNEL)).build()); + .withLabel(Objects.requireNonNull(StringUtils.capitalize(HUMIDITY_CHANNEL))) + .build()); updateThing(thingBuilder.build()); hasHumidityChannel = true; @@ -94,7 +96,8 @@ public void publish(LgwReading reading) { .create(new ChannelUID(getThing().getUID(), PRESSURE_CHANNEL), "Number:Pressure") .withType(new ChannelTypeUID(getThing().getThingTypeUID().getBindingId(), PRESSURE_CHANNEL)) - .withLabel(StringUtils.capitalize(PRESSURE_CHANNEL)).build()); + .withLabel(Objects.requireNonNull(StringUtils.capitalize(PRESSURE_CHANNEL))) + .build()); updateThing(thingBuilder.build()); hasPressureChannel = true; diff --git a/bundles/org.openhab.binding.jeelink/src/main/java/org/openhab/binding/jeelink/internal/util/StringUtils.java b/bundles/org.openhab.binding.jeelink/src/main/java/org/openhab/binding/jeelink/internal/util/StringUtils.java deleted file mode 100644 index d5d93335f392c..0000000000000 --- a/bundles/org.openhab.binding.jeelink/src/main/java/org/openhab/binding/jeelink/internal/util/StringUtils.java +++ /dev/null @@ -1,48 +0,0 @@ -/** - * Copyright (c) 2010-2023 Contributors to the openHAB project - * - * See the NOTICE file(s) distributed with this work for additional - * information. - * - * This program and the accompanying materials are made available under the - * terms of the Eclipse Public License 2.0 which is available at - * http://www.eclipse.org/legal/epl-2.0 - * - * SPDX-License-Identifier: EPL-2.0 - */ -package org.openhab.binding.jeelink.internal.util; - -import org.eclipse.jdt.annotation.NonNullByDefault; - -/** - * Utility class for strings - * - * @author Leo Siepel - Initial contribution - */ -@NonNullByDefault -public final class StringUtils { - - /** - *

- * Capitalizes a String changing the first character to title case. - * No other characters are changed. - *

- * - * 
-     * StringUtils.capitalize(null)  = null
-     * StringUtils.capitalize("")    = ""
-     * StringUtils.capitalize("cat") = "Cat"
-     * StringUtils.capitalize("cAt") = "CAt"
-     * StringUtils.capitalize("'cat'") = "'cat'"
-     *
- * - * @param val the String to capitalize, may not be null - * @return the capitalized String - */ - public static String capitalize(String val) { - if (val.length() == 0) { - return val; - } - return val.substring(0, 1).toUpperCase() + val.substring(1); - } -}