[SOLVED] Simple IF statments not working

Setup, Configuration and Use Beginners
1

HI All, new to openHAB

I am using pushover to send notifications to my android device but for some reason if statements will not work
what so ever. what am I missing?

this code here works:

rule "Enable/Disable Alarm"
when
    Item Arm_Alarm changed to ON
then
    sendPushoverMessage(pushoverBuilder("Kitchen Door Opened").withEmergencyPriority())
end

This does not work for some reason:

rule "Enable/Disable Alarm"
when
    Item Arm_Alarm changed to ON
then
    if (XiaomiSensorKitchen_OpenStatus.state == OPEN){
        sendPushoverMessage(pushoverBuilder("Kitchen Door Opened").withEmergencyPriority())
    }
end

Am I missing some sort of Import or have I forgotten to enable something in a config file somewhere?

Thanks in advance

2

What is this item, please?

3

its a window / door sensor: https://www.openhab.org/addons/bindings/mihome/

so i think it comes under a contact state

4

Does it?
Show the channel and item,please?

5

Perhaps this isnā€™t going to do what you expect. It isnā€™t an ā€œalarmā€ for the kitchen door. The rule runs only once when ā€˜Armā€™ is turned on, checks if the kitchen door is open at that time, and messages if so.

Did you mean to have a rule that runs when the door is opened, checks to see if the alam is armed, and messages if so?

6

Yeahhhhh thats what I need to doā€¦

so how would that be created?
I can see the logic Iā€™m using is flawed

7

Screenshot_1
Screenshot_1.png1169Ɨ86 4.63 KB

not sure what you mean by Channel

8

Come on, itā€™s just a case of altering what change triggers your existing rule, and what state you check within it.

2 Likes
9

Well, you need to do a bit of reading to understand the basics of openHAB:

TIP: Itā€™s the wrong way aroundā€¦
Follow this:

10

Thanks guys, got it working! :slight_smile:

rule "Enable/Disable Alarm"
when
    Item XiaomiSensorKitchen_OpenStatus changed from CLOSED or 
    Item XiaomiSensorFrontDoor_OpenStatus changed from CLOSED
then
    if(Arm_Alarm.state == ON)
    {
        if(XiaomiSensorKitchen_OpenStatus.state == OPEN){
        sendPushoverMessage(pushoverBuilder("Kitchen Door Opened").withEmergencyPriority())
        }else if(XiaomiSensorFrontDoor_OpenStatus.state == OPEN){
        sendPushoverMessage(pushoverBuilder("Front Door Opened").withEmergencyPriority())
        }
    }
end
1 Like
11

Another way:

rule "Enable/Disable Alarm"
when
    Item XiaomiSensorKitchen_OpenStatus changed to OPEN or 
    Item XiaomiSensorFrontDoor_OpenStatus changed to OPEN
then
    if(Arm_Alarm.state == ON) {
        val String doorName = triggeringItem.name.toString.split("_").get(0).substring(12) // returns "Kitchen or FrontDoor"
        sendPushoverMessage(pushoverBuilder(doorName + " Door Opened").withEmergencyPriority())
    }
end
1 Like
12

Thank you, appreciate it a lot