----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/3508/#review11817 -----------------------------------------------------------
/branches/11/apps/app_queue.c <https://reviewboard.asterisk.org/r/3508/#comment21678> Annoyingly, I think you'll need to also unsubscribe from the extension here as well. Since this is slipping the reloaded member back into the same position in the queue, you can't just call member_remove_from_queue here. - Matt Jordan On May 2, 2014, 9:52 a.m., sysreq wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviewboard.asterisk.org/r/3508/ > ----------------------------------------------------------- > > (Updated May 2, 2014, 9:52 a.m.) > > > Review request for Asterisk Developers. > > > Bugs: ASTERISK-18411 > https://issues.asterisk.org/jira/browse/ASTERISK-18411 > > > Repository: Asterisk > > > Description > ------- > > The rationale behind this patch is that when a hint is used as a > 'state_interface', app_queue should explicitly subscribe to it and become a > watcher. That will prevent a dynamic hint from disappearing after a 'dialplan > reload' because Asterisk thinks it's not being "watched" by anything. We > should also unsubscribe from the hint when the agent is removed from the > queue. > > > Diffs > ----- > > /branches/11/apps/app_queue.c 413115 > > Diff: https://reviewboard.asterisk.org/r/3508/diff/ > > > Testing > ------- > > I have tested this patch by using either a dynamic hint, a static hint, a > local channel or a SIP device as a queue member's 'state_interface'. I have > also added/removed the queue member both through Realtime and the CLI. > > I've had this patch on about half a dozen 1.8/11 production systems (all > averaging a thousand users each) for a little more than a month with no > apparent issue. > > > Thanks, > > sysreq > >
-- _____________________________________________________________________ -- Bandwidth and Colocation Provided by http://www.api-digital.com -- asterisk-dev mailing list To UNSUBSCRIBE or update options visit: http://lists.digium.com/mailman/listinfo/asterisk-dev
