[Home]

Summary:ASTERISK-02593: [patch] Possibility to disable some parts of the announce
Reporter:dsandras (dsandras)Labels:
Date Opened:2004-10-13 06:15:12Date Closed:2011-06-07 14:05:10
Priority:MajorRegression?No
Status:Closed/CompleteComponents:Applications/app_queue
Versions:Frequency of
Occurrence
Related
Issues:
Environment:Attachments:( 0) app_queue.patch
Description:I would like to be able to disable some parts of the announce. For example, the thank-you part is mandatory, but the "you are at pos" part is not useful to us. A good way to do this is to allow things like queue-next = in the configuration. Here is a trivial patch that allows this.
Comments:By: twisted (twisted) 2004-10-14 01:19:06

please add [patch] to summary fields when you are submitting a patch.  Also, when submitting a patch, note if you have disclaimer on file or not.  Thanks ;)

By: Mark Spencer (markster) 2004-10-14 01:53:30

What is the relationship between this bug and ASTERISK-2467?

By: dsandras (dsandras) 2004-10-14 02:10:29

No relationship. The current simple patch just prevents asterisk to complain if the user specifies an empty announce message, permitting him to disable some of the messages in that case.

By: twisted (twisted) 2004-10-27 17:14:56

what about ASTERISK-2572 ..?   We had several of these bugs open up close to one another, which I can't stress enough to search the bugs before posting new ones.  Is ASTERISK-2572 what you're trying to accomplish?

By: dsandras (dsandras) 2004-10-28 02:12:12

That's something else, unrelated. See my previous bugnotes.

By: Russell Bryant (russell) 2004-11-09 00:52:57.000-0600

!strcmp (filename, "")

should be ...

!ast_strlen_zero(filename)

By: Russell Bryant (russell) 2004-11-14 22:00:12.000-0600

Feel free to reopen this bug if you have an update for your patch.