From: Kai Kreuzer <kai@openhab.org>
Date: Sat, 23 Sep 2023 15:59:40 +0000 (+0200)
Subject: Fix config option provider to only become active for chatgpt things (#15578)
X-Git-Url: https://git.basschouten.com/?a=commitdiff_plain;h=4059003e9e3ff6cf8aa5844d588f04e58cd7bd8a;p=openhab-addons.git

Fix config option provider to only become active for chatgpt things (#15578)

Signed-off-by: Kai Kreuzer <kai@openhab.org>
---

diff --git a/bundles/org.openhab.binding.chatgpt/src/main/java/org/openhab/binding/chatgpt/internal/ChatGPTModelOptionProvider.java b/bundles/org.openhab.binding.chatgpt/src/main/java/org/openhab/binding/chatgpt/internal/ChatGPTModelOptionProvider.java
index 9057398de3..f0964ce184 100644
--- a/bundles/org.openhab.binding.chatgpt/src/main/java/org/openhab/binding/chatgpt/internal/ChatGPTModelOptionProvider.java
+++ b/bundles/org.openhab.binding.chatgpt/src/main/java/org/openhab/binding/chatgpt/internal/ChatGPTModelOptionProvider.java
@@ -39,15 +39,17 @@ public class ChatGPTModelOptionProvider implements ThingHandlerService, ConfigOp
     @Override
     public @Nullable Collection<ParameterOption> getParameterOptions(URI uri, String param, @Nullable String context,
             @Nullable Locale locale) {
-        if ("model".equals(param)) {
-            List<ParameterOption> options = new ArrayList<>();
-            if (thingHandler instanceof ChatGPTHandler chatGPTHandler) {
-                chatGPTHandler.getModels().forEach(model -> options.add(new ParameterOption(model, model)));
+        String accountParameterUrl = "thing-type:" + ChatGPTBindingConstants.THING_TYPE_ACCOUNT.getAsString();
+        if (accountParameterUrl.equals(uri.toString())) {
+            if ("model".equals(param)) {
+                List<ParameterOption> options = new ArrayList<>();
+                if (thingHandler instanceof ChatGPTHandler chatGPTHandler) {
+                    chatGPTHandler.getModels().forEach(model -> options.add(new ParameterOption(model, model)));
+                }
+                return options;
             }
-            return options;
-        } else {
-            return null;
         }
+        return null;
     }
 
     @Override