app_queue: Log reason for PAUSEALL/UNPAUSEALL

We needed the reason for our reporting when agents pause/unpause all of
their queues at once.  This is a small, simple patch that adds a reason
for PAUSEALL and UNPAUSEALL.  I have been using it in production for years.

ASTERISK-26920 #close

Change-Id: Ifb3f0d1a0abd5194253d9794023546e1395baf3d

diff --git a/CHANGES b/CHANGES
index f7f8505b8f938c7a55221127247390aebab1476d..9a82e1fe33b85915ea538e1f7d72df1d63666e9d 100644 (file)
--- a/CHANGES
+++ b/CHANGES
@@ -12,6 +12,11 @@
 --- Functionality changes from Asterisk 14 to Asterisk 15 --------------------
 ------------------------------------------------------------------------------
 
+app_queue
+------------------
+ * PAUSEALL/UNPAUSEALL now sets the pause reason in the queue_log if it has
+   been defined.
+
 ------------------------------------------------------------------------------
 --- Functionality changes from Asterisk 14.4.0 to Asterisk 14.5.0 ------------
 ------------------------------------------------------------------------------

diff --git a/apps/app_queue.c b/apps/app_queue.c
index 9bba6762d6e795515453c879d179305852b6ff02..ae2d645e9455280e2c3e36afebb2eccf6c53e37e 100644 (file)
--- a/apps/app_queue.c
+++ b/apps/app_queue.c
@@ -7332,7 +7332,7 @@ static int set_member_paused(const char *queuename, const char *interface, const
                                         * but since this affects all queues, we cannot.
                                         */
                                        ast_queue_log("NONE", "NONE", mem->membername,
-                                               (paused ? "PAUSEALL" : "UNPAUSEALL"), "%s", "");
+                                               (paused ? "PAUSEALL" : "UNPAUSEALL"), "%s", S_OR(reason, ""));
                                }
 
                                set_queue_member_pause(q, mem, reason, paused);