ParadoxAlarmInterface / pai

Paradox Magellan, Spectra and EVO, with MQTT, Signal, Pushbullet, Pushover and others
https://gitter.im/paradox-alarm-interface
Eclipse Public License 2.0
348 stars 92 forks source link

Core update 2024.6.1 breaks the arming function #468

Closed TamasAntal closed 3 months ago

TamasAntal commented 3 months ago

Hi yozik04!

I have an EVO192. after installing core update 2024.5.5 and 2024.6.1 together, arming stopped working. It returns a message saying that arming requires a code, but pay did not return any. Or something similar. I have restored to core 2024.5.4, and arming works again... Oh, and I have not had arming enabled before.

It would be so nice to get an update for this, as the alarm system is quite essential :)

thanks!

Alarm system

Describe your panel model (ex: EVO192, Spectra SP4000, Magellan MG5050), access mode (Serial, IP150, etc..).

Environment

Git branch

Which Git branch you use. Did you tried "dev" branch? Please try before posting.

Describe the bug

A clear and concise description of what the bug is.

To Reproduce

Steps to reproduce the behavior:

  1. Start script '...'
  2. Do something '....'
  3. See error

Logs

PLEASE DO NOT SEND LOGS IN CLEAR TEXT AS THEY MAY CONTAIN SENSITIVE INFORMATION If you wish to add a log (which will greatly help us), please follow the following instructions:

  1. Go to https://keybase.io/encrypt
  2. Write to us
    1. Write 'jpbarraca' as the recipient if you have SP/MG panel
    2. Write 'yozik04' as the recipient if you have EVO panel
  3. Write your log or add a link to the log file (if using services such as dropbox).
  4. Click Encrypt
  5. Save the result to a text file and add it as an attachment.
uwan-cm commented 3 months ago

I have the exact same issue. Waiting for a work around on this or an update to the integration.

yozik04 commented 3 months ago

Take a look into #466

yozik04 commented 3 months ago

I am not sure if I need to update anything on my end or it is a HA bug.

uwan-cm commented 3 months ago

I am not sure if I need to update anything on my end or it is a HA bug.

It is definitely a new HA feature. Alarm panels are required to pass in code. Maybe a setting on this extension can be introduced to include a code?

EtienneMD commented 3 months ago

Code validation is now a mandatory option and defaults to True.

code_arm_required needs to be set to False in the configuration.

https://developers.home-assistant.io/blog/2024/05/22/alarm_control_panel_validation/

karamfil commented 3 months ago

Is this something we need to do in the config somewhere and how, or is this PAI code change?

uwan-cm commented 3 months ago

Is this something we need to do in the config somewhere and how, or is this PAI code change?

It was something new that got introduced in HA. To fix it, just go to Settings > Devices & Services > Entities tab and click on your Alarm Panel entity and then on the settings icon on the popup. You will see there is a setting for "Default Code", so just enter your alarm panel arming code there and save.

TamasAntal commented 3 months ago

Thanks a lot for the advice, adding the default code at the entity solved the issue. And the speedy answer... awesome!