[asterisk-commits] elguero: branch 1.8 r400622 - /branches/1.8/apps/app_queue.c

[SVN commits to the Asterisk project]

Author: elguero
Date: Sun Oct  6 12:07:32 2013
New Revision: 400622

URL: http://svnview.digium.com/svn/asterisk?view=rev&rev=400622
Log:
Fix Regression With Queuelog EXITWITHKEY Only Logging Two Out Of Four Fields

Commit r62462 added two extra fields for logging "the original position the 
caller entered the queue at, and the amount of time the caller was waiting in
the queue."  But when r75969 was merged from 1.4 into trunk (r75977), these two
fields disappeared. Those two extra fields were not logged in 1.4 and when the
patch was merged, those fields went away.

Therefore, this is a regression and was caught by the reporter because he was
reading the awesome "Asterisk: The Definitive Guide" book.

(closes issue ASTERISK-22197)
Reported by: Dalius M.
Tested by: Dalius M.
Patches:
    asterisk-22197-q-log-exitwithkey.diff
				     uploaded by Michael L. Young (license 5026)

Review: https://reviewboard.asterisk.org/r/2901/

Modified:
    branches/1.8/apps/app_queue.c

Modified: branches/1.8/apps/app_queue.c
URL: http://svnview.digium.com/svn/asterisk/branches/1.8/apps/app_queue.c?view=diff&rev=400622&r1=400621&r2=400622
==============================================================================
--- branches/1.8/apps/app_queue.c (original)
+++ branches/1.8/apps/app_queue.c Sun Oct  6 12:07:32 2013
@@ -6352,7 +6352,7 @@
 			}
 		} else if (qe.valid_digits) {
 			ast_queue_log(args.queuename, chan->uniqueid, "NONE", "EXITWITHKEY",
-				"%s|%d", qe.digits, qe.pos);
+				"%s|%d|%d|%ld", qe.digits, qe.pos, qe.opos, (long) time(NULL) - qe.start);
 		}
 	}