| Summary: | ASTERISK-27479: [patch] app_queue: While using queues with realtime, setting back to an empty announce don't stop the old announce to be played | ||||
| Reporter: | Matt Fredrickson (mfredrickson) | Labels: | |||
| Date Opened: | 2017-12-11 16:30:49.000-0600 | Date Closed: | 2017-12-11 16:31:18.000-0600 | ||
| Priority: | Minor | Regression? | |||
| Status: | Closed/Complete | Components: | Applications/app_queue | ||
| Versions: | 13.18.3 | Frequency of Occurrence | Constant | ||
| Related Issues: |
| ||||
| Environment: | CentOS 6 64bit, MySQL 5.7 used with ODBC | Attachments: | ( 0) app_queue.c.patch | ||
| Description: | Like in ASTERISK-26462, where the affected field was "context", I noticed also the field "announce" is not correctly reset on queue init.
--- /usr/local/src/asterisk-13.18.2/apps/app_queue.c 2017-11-10 16:41:55.000000000 +0100 +++ app_queue.c 2017-11-26 11:44:59.281418114 +0100 @@ -2656,6 +2656,7 @@ q->maxlen = 0; ast_string_field_set(q, context, ""); + ast_string_field_set(q, announce, ""); q->announcefrequency = 0; q->minannouncefrequency = DEFAULT_MIN_ANNOUNCE_FREQUENCY; | ||||
| Comments: | By: Asterisk Team (asteriskteam) 2017-12-08 03:20:58.401-0600 Thanks for creating a report! The issue has entered the triage process. That means the issue will wait in this status until a Bug Marshal has an opportunity to review the issue. Once the issue has been reviewed you will receive comments regarding the next steps towards resolution. A good first step is for you to review the [Asterisk Issue Guidelines|https://wiki.asterisk.org/wiki/display/AST/Asterisk+Issue+Guidelines] if you haven't already. The guidelines detail what is expected from an Asterisk issue report. Then, if you are submitting a patch, please review the [Patch Contribution Process|https://wiki.asterisk.org/wiki/display/AST/Patch+Contribution+Process]. By: Matthew Fredrickson (mattf) 2017-12-11 15:53:19.050-0600 Thanks for the report and patch [~ldardini], feel free to push your patch up to gerrit. | ||||