| Summary: | ASTERISK-11166: Set(CDR(accountcode)=value) does not work after ForkCDR | ||
| Reporter: | Jeff Pyle (jpyle) | Labels: | |
| Date Opened: | 2008-01-06 12:09:48.000-0600 | Date Closed: | 2008-02-14 18:23:20.000-0600 |
| Priority: | Minor | Regression? | No |
| Status: | Closed/Complete | Components: | Functions/func_cdr |
| Versions: | Frequency of Occurrence | ||
| Related Issues: | |||
| Environment: | Attachments: | ( 0) forkcdr-no-accountcode-console-output.txt | |
| Description: | I attempt to set the accountcode after forking the CDR. In previous versions, this changed only the accountcode value for the CDR line after the fork, which was exactly what I needed. Now, nothing seems to be set. I added the "v" option to ForkCDR from my previous working configuration in 1.2.14, but it didn't help. Running 1.2.17, CentOS 5.1 on i686. ****** ADDITIONAL INFORMATION ****** The extensions logic and console output are attached. I'm not sure which debugs are applicable in this case, so I'll refrain from posting any until requested. | ||
| Comments: | By: Michiel van Baak (mvanbaak) 2008-01-07 10:51:42.000-0600 Are you running 1.2.17 or 1.4.17 ? By: Jeff Pyle (jpyle) 2008-01-07 11:03:59.000-0600 1.4.17, presently. The last time I used this config was on 1.2.14, and it worked properly there. By: Loic Didelot (voipgate) 2008-01-09 19:40:55.000-0600 Just use the "well" documented parameter "l" ${CDR(accountcode|l)} or ${CDR(accountcode,l)} depending on which version of asterisk you are. I guess 'l' stands for 'last'. This option is not documented and the function should not work like this. It should by default give the last value and only with a parameter you should be abel to get the first value. By: Leif Madsen (lmadsen) 2008-02-14 13:42:42.000-0600 For Qwell and documentation: 'l' on a channel with multiple CDR records, use the most recent CDR record By: Digium Subversion (svnbot) 2008-02-14 13:47:53.000-0600 Repository: asterisk Revision: 103683 U branches/1.4/funcs/func_cdr.c ------------------------------------------------------------------------ r103683 | qwell | 2008-02-14 13:47:52 -0600 (Thu, 14 Feb 2008) | 5 lines Document the 'l' option to the CDR() function. (Thanks voipgate for pointing out the option, and Leif for providing text for it.) Closes issue ASTERISK-11166. ------------------------------------------------------------------------ http://svn.digium.com/view/asterisk?view=rev&revision=103683 By: Digium Subversion (svnbot) 2008-02-14 13:49:05.000-0600 Repository: asterisk Revision: 103685 _U trunk/ U trunk/funcs/func_cdr.c ------------------------------------------------------------------------ r103685 | qwell | 2008-02-14 13:49:05 -0600 (Thu, 14 Feb 2008) | 13 lines Merged revisions 103683 via svnmerge from https://origsvn.digium.com/svn/asterisk/branches/1.4 ........ r103683 | qwell | 2008-02-14 13:51:10 -0600 (Thu, 14 Feb 2008) | 5 lines Document the 'l' option to the CDR() function. (Thanks voipgate for pointing out the option, and Leif for providing text for it.) Closes issue ASTERISK-11166. ........ ------------------------------------------------------------------------ http://svn.digium.com/view/asterisk?view=rev&revision=103685 By: Digium Subversion (svnbot) 2008-02-14 18:23:20.000-0600 Repository: asterisk Revision: 103708 _U team/murf/bug11210/ U team/murf/bug11210/UPGRADE.txt U team/murf/bug11210/apps/app_externalivr.c U team/murf/bug11210/apps/app_queue.c U team/murf/bug11210/apps/app_voicemail.c U team/murf/bug11210/channels/chan_iax2.c U team/murf/bug11210/configs/queues.conf.sample U team/murf/bug11210/configure U team/murf/bug11210/configure.ac U team/murf/bug11210/doc/tex/imapstorage.tex U team/murf/bug11210/funcs/func_cdr.c U team/murf/bug11210/include/asterisk/autoconfig.h.in U team/murf/bug11210/res/res_agi.c U team/murf/bug11210/res/res_musiconhold.c ------------------------------------------------------------------------ r103708 | murf | 2008-02-14 18:23:17 -0600 (Thu, 14 Feb 2008) | 187 lines Merged revisions 103658,103662,103668,103677,103682,103685,103687,103689,103691,103694,103699-103700,103702,103704-103705 via svnmerge from https://origsvn.digium.com/svn/asterisk/trunk ................ r103658 | mmichelson | 2008-02-13 08:47:25 -0700 (Wed, 13 Feb 2008) | 10 lines 1. Deprecate SetMusicOnHold and WaitMusicOnHold. 2. Add a duration parameter to MusicOnHold (closes issue ASTERISK-11358) Reported by: dimas Patches: v2-moh.patch uploaded by dimas (license 88) Tested by: dimas ................ r103662 | jpeeler | 2008-02-13 14:04:31 -0700 (Wed, 13 Feb 2008) | 6 lines (closes issue ASTERISK-11286) Reported by: ctooley Patches: additional_eivr_commands.patch uploaded by ctooley (license 136) Tested by: ctooley ................ r103668 | oej | 2008-02-14 03:19:09 -0700 (Thu, 14 Feb 2008) | 2 lines Formatting fixes ................ r103677 | qwell | 2008-02-14 11:39:51 -0700 (Thu, 14 Feb 2008) | 11 lines Add periodic jitter stats to CLI and manager. (closes issue ASTERISK-7968) Reported by: stevedavies Patches: jblogging-trunk.patch uploaded by stevedavies jblogging-trunk_wmgrevent.patch uploaded by johann8384 updated_jbloggin-trunk_mgrevent.patch uploaded by johann8384 (license 190) (with additional changes by me) Tested by: stevedavies, johann8384 ................ r103682 | jpeeler | 2008-02-14 12:47:39 -0700 (Thu, 14 Feb 2008) | 1 line a few syntax changes and safer code ................ r103685 | qwell | 2008-02-14 12:52:21 -0700 (Thu, 14 Feb 2008) | 13 lines Merged revisions 103683 via svnmerge from https://origsvn.digium.com/svn/asterisk/branches/1.4 ........ r103683 | qwell | 2008-02-14 13:51:10 -0600 (Thu, 14 Feb 2008) | 5 lines Document the 'l' option to the CDR() function. (Thanks voipgate for pointing out the option, and Leif for providing text for it.) Closes issue ASTERISK-11166. ........ ................ r103687 | mmichelson | 2008-02-14 13:46:00 -0700 (Thu, 14 Feb 2008) | 10 lines Change the queue holdtime announcement to happen at any interval (not just greater than two minutes). Remove the saying of less-than for holdtime announcements since it can lead to awkward holdtime announcements. Using '1' as a queue-round-seconds value is no longer valid. (closes issue ASTERISK-9451) Reported by: caio1982 Patches: queue_announce5.diff uploaded by caio1982 (license 22) Tested by: caio1982, putnopvut ................ r103689 | mmichelson | 2008-02-14 13:58:30 -0700 (Thu, 14 Feb 2008) | 17 lines Merged revisions 103688 via svnmerge from https://origsvn.digium.com/svn/asterisk/branches/1.4 ........ r103688 | mmichelson | 2008-02-14 14:55:48 -0600 (Thu, 14 Feb 2008) | 9 lines Fix the new message count if delete=yes when using IMAP storage. (closes issue ASTERISK-10917) Reported by: jaroth Patches: deleteflag_v2.patch uploaded by jaroth (license 50) Tested by: jaroth ........ ................ r103691 | mmichelson | 2008-02-14 14:04:37 -0700 (Thu, 14 Feb 2008) | 11 lines Merged revisions 103690 via svnmerge from https://origsvn.digium.com/svn/asterisk/branches/1.4 ........ r103690 | mmichelson | 2008-02-14 15:03:02 -0600 (Thu, 14 Feb 2008) | 3 lines Fix build for non-IMAP builds ........ ................ r103694 | qwell | 2008-02-14 14:21:53 -0700 (Thu, 14 Feb 2008) | 8 lines Modify ldap autoconf function, so that an incorrect ldap library is not found on Solaris (it is incompatible). Also removes second check for awk, which causes Solaris to find an incompatible version of awk. (closes issue ASTERISK-11290) Reported by: snuffy Patches: bug-11829.diff uploaded by snuffy (license 35) ................ r103699 | mmichelson | 2008-02-14 16:35:21 -0700 (Thu, 14 Feb 2008) | 20 lines Blocked revisions 103698 via svnmerge ........ r103698 | mmichelson | 2008-02-14 17:30:17 -0600 (Thu, 14 Feb 2008) | 13 lines Change to the configure logic regarding IMAP. Prior to this commit, if you wished to configure Asterisk with IMAP support, you would use the --with-imap configure switch in one of the following two ways: --with-imap=/some/directory would look in the directory specified for a UW IMAP source installation --with-imap would assume that you had imap-2004g installed in .. relative to the Asterisk source With this set of changes the two above options still work the same, but there are two new behaviors, too. --with-imap=system will assume that you have -libc-client.so where you store your shared objects and will attempt to find c-client headers in your include path either in the imap or c-client directory. If either of the two original methods of specifying the imap option should fail, then the check for --with-imap =system will be performed in addition. It is only after this "system" check that failure can happen. ........ ................ r103700 | mmichelson | 2008-02-14 16:39:47 -0700 (Thu, 14 Feb 2008) | 5 lines See commit message for svn revision 103698. This behavior is the same as what is described there. The difference is that trunk already had the --with-imap=system option, but it only checked the include path for headers in the imap directory and not also the c-client directory. ................ r103702 | mmichelson | 2008-02-14 16:48:12 -0700 (Thu, 14 Feb 2008) | 10 lines Blocked revisions 103701 via svnmerge ........ r103701 | mmichelson | 2008-02-14 17:44:17 -0600 (Thu, 14 Feb 2008) | 3 lines Update documentation regarding configuration of IMAP ........ ................ r103704 | mmichelson | 2008-02-14 16:49:54 -0700 (Thu, 14 Feb 2008) | 10 lines Blocked revisions 103703 via svnmerge ........ r103703 | mmichelson | 2008-02-14 17:49:24 -0600 (Thu, 14 Feb 2008) | 3 lines Make a small clarification in the documentation ........ ................ r103705 | mmichelson | 2008-02-14 16:51:49 -0700 (Thu, 14 Feb 2008) | 3 lines Trunk version of 1.4's imap documentation updates ................ ------------------------------------------------------------------------ http://svn.digium.com/view/asterisk?view=rev&revision=103708 | ||