[asterisk-commits] mmichelson: branch 12 r413173 - /branches/12/res/res_pjsip_exten_state.c

SVN commits to the Asterisk project asterisk-commits at lists.digium.com
Thu May 1 10:41:50 CDT 2014


Author: mmichelson
Date: Thu May  1 10:41:43 2014
New Revision: 413173

URL: http://svnview.digium.com/svn/asterisk?view=rev&rev=413173
Log:
Remove unnecessary repetition checks from res_pjsip_exten_state

The PBX core already takes care of ensuring that repeated state changes
are not communicated to exten state consumers. Because the check in res_pjsip_exten_state
was incomplete, it was causing valid presence state changes not to be sent out. For instance,
if the presence state did not change but the message or subtype did, then no presence-related
NOTIFY request would be sent out.

closes issue ASTERISK-23672
Reported by Mark Michelson


Modified:
    branches/12/res/res_pjsip_exten_state.c

Modified: branches/12/res/res_pjsip_exten_state.c
URL: http://svnview.digium.com/svn/asterisk/branches/12/res/res_pjsip_exten_state.c?view=diff&rev=413173&r1=413172&r2=413173
==============================================================================
--- branches/12/res/res_pjsip_exten_state.c (original)
+++ branches/12/res/res_pjsip_exten_state.c Thu May  1 10:41:43 2014
@@ -334,11 +334,6 @@
 	struct notify_task_data *task_data;
 	struct exten_state_subscription *exten_state_sub = data;
 
-	if (exten_state_sub->last_exten_state == info->exten_state &&
-		exten_state_sub->last_presence_state == info->presence_state) {
-		return 0;
-	}
-
 	if (!(task_data = alloc_notify_task_data(exten, exten_state_sub, info))) {
 		return -1;
 	}




More information about the asterisk-commits mailing list