Skip to content

Commit

Permalink
Maybe fix logic problem due to order of operations on &&. Maybe fix t…
Browse files Browse the repository at this point in the history
…he Not sending event alarm problem
  • Loading branch information
connortechnology committed Oct 3, 2022
1 parent 9e1f100 commit cc66eb5
Showing 1 changed file with 1 addition and 2 deletions.
3 changes: 1 addition & 2 deletions zmeventnotification.pl
Original file line number Diff line number Diff line change
Expand Up @@ -3840,8 +3840,7 @@ sub processNewAlarmsInFork {

# note that this end_notify_if_start is default yes, even if you comment it out
# so if you disable all hooks params, you won't get end notifs
# # FIXME order of operations unclear
if ( $event_end_notify_if_start_success && $startHookResult != 0 ) {
if ( $event_end_notify_if_start_success && ($startHookResult != 0) ) {
printInfo(
'Not sending event end alarm, as we did not send a start alarm for this, or start hook processing failed'
);
Expand Down

0 comments on commit cc66eb5

Please sign in to comment.