[smsmodem] Small fixes

update channel rename action to avoid colision with other binding and a too generic name Signed-off-by: Gwendal Roulleau <[email protected]>
openhab · Mar 7, 2022 · 82f91c8 · 82f91c8
1 parent 87c12f9
commit 82f91c8
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 3 deletions.
diff --git a/...smsmodem/src/main/java/org/openhab/binding/smsmodem/internal/actions/SMSModemActions.java b/...smsmodem/src/main/java/org/openhab/binding/smsmodem/internal/actions/SMSModemActions.java
@@ -47,7 +47,7 @@ public void setThingHandler(@Nullable ThingHandler handler) {
     }
 
     @RuleAction(label = "Send Message", description = "Send a message")
-    public void send(
+    public void sendSMS(
             @ActionInput(name = "recipient", label = "recipient", description = "Recipient of the message") @Nullable String recipient,
             @ActionInput(name = "message", label = "message", description = "Message to send") @Nullable String message) {
         if (recipient != null && !recipient.isEmpty() && message != null) {
@@ -57,9 +57,9 @@ public void send(
         }
     }
 
-    public static void send(@Nullable ThingActions actions, @Nullable String recipient, @Nullable String message) {
+    public static void sendSMS(@Nullable ThingActions actions, @Nullable String recipient, @Nullable String message) {
         if (actions instanceof SMSModemActions) {
-            ((SMSModemActions) actions).send(recipient, message);
+            ((SMSModemActions) actions).sendSMS(recipient, message);
         } else {
             throw new IllegalArgumentException("Instance is not an SMSModemActions class.");
         }

diff --git a/...m/src/main/java/org/openhab/binding/smsmodem/internal/handler/SMSConversationHandler.java b/...m/src/main/java/org/openhab/binding/smsmodem/internal/handler/SMSConversationHandler.java
@@ -99,6 +99,7 @@ public void handleCommand(ChannelUID channelUID, Command command) {
         }
         if (channelUID.getId().equals(SMSModemBindingConstants.CHANNEL_SEND)) {
             send(command.toString());
+            updateState(SMSModemBindingConstants.CHANNEL_SEND, new StringType(command.toString()));
         }
     }