]> git.basschouten.com Git - openhab-addons.git/commitdiff
Fix clearing Now Playing channels (#17185)
authorJacob Laursen <jacob-github@vindvejr.dk>
Thu, 1 Aug 2024 10:32:11 +0000 (12:32 +0200)
committerGitHub <noreply@github.com>
Thu, 1 Aug 2024 10:32:11 +0000 (12:32 +0200)
Signed-off-by: Jacob Laursen <jacob-github@vindvejr.dk>
bundles/org.openhab.binding.denonmarantz/src/main/java/org/openhab/binding/denonmarantz/internal/connector/http/DenonMarantzHttpConnector.java

index 73d643654e9384469cf2480400090923c180cc4d..7d284e5d6aff437745518abda9e8807b372d5519 100644 (file)
@@ -19,6 +19,7 @@ import java.math.BigDecimal;
 import java.net.URLEncoder;
 import java.nio.charset.Charset;
 import java.nio.charset.StandardCharsets;
+import java.util.Objects;
 import java.util.concurrent.ExecutionException;
 import java.util.concurrent.ScheduledExecutorService;
 import java.util.concurrent.ScheduledFuture;
@@ -340,18 +341,9 @@ public class DenonMarantzHttpConnector extends DenonMarantzConnector {
                 return;
             }
             CommandRx titleInfo = response.getCommands().get(0);
-            String artist = titleInfo.getText("artist");
-            if (artist != null) {
-                state.setNowPlayingArtist(artist);
-            }
-            String album = titleInfo.getText("album");
-            if (album != null) {
-                state.setNowPlayingAlbum(album);
-            }
-            String track = titleInfo.getText("track");
-            if (track != null) {
-                state.setNowPlayingTrack(track);
-            }
+            state.setNowPlayingArtist(Objects.requireNonNullElse(titleInfo.getText("artist"), ""));
+            state.setNowPlayingAlbum(Objects.requireNonNullElse(titleInfo.getText("album"), ""));
+            state.setNowPlayingTrack(Objects.requireNonNullElse(titleInfo.getText("track"), ""));
         } catch (HttpCommunicationException e) {
             logger.debug("Failed to update display info: {}", e.getMessage());
         }