Summary:ASTERISK-01291: [patch] set/append CDR User Field for manager interface
Reporter:exomorph (exomorph)Labels:
Date Opened:2004-03-25 13:07:09.000-0600Date Closed:2008-01-15 14:49:03.000-0600
Versions:Frequency of
Environment:Attachments:( 0) manager.c.patch
Description:Make app_agi be able to set or append to the CDR User Field. This is quite useful for adding notes to the CDR from an AGI script.
Comments:By: exomorph (exomorph) 2004-03-25 14:23:08.000-0600

As per JerJer's request... I've also modified the manager to allow for the same functionality.

By: James Golovich (jamesgolovich) 2004-03-25 23:43:51.000-0600

Instead of making an agi command for this it would be better to add a regular * app, that will cover agi and regular extensions.conf processing.

I'm also curious if an append and set is the way to go.  Why not have a get and set, and if you want to append then you just get the existing data and tack it on yourself and then set

By: exomorph (exomorph) 2004-03-26 00:45:26.000-0600

Ok... I had a brain fart when I did this work.  James is right, we have an agi command to exec dial plan apps, and we should use that instead.  Also these commands are already in for the dial plan.

So the app_agi.c patch is by by... and the manager patch needs to be moved to app_setcdruserfield.c

edited on: 03-25-04 23:56

By: James Golovich (jamesgolovich) 2004-03-26 01:33:31.000-0600

Ok I've commited my updated patch to CVS HEAD.  I changed things around a bit.
Heres the manager syntax:

Action: SetCDRUserField
Channel: Zap/1
UserField: Test Field

to append instead of replace add:
Append: yes (or any value of ast_true)

Closing the bug out.  Reopen if there is anything else

By: Digium Subversion (svnbot) 2008-01-15 14:49:03.000-0600

Repository: asterisk
Revision: 2566

U   trunk/apps/app_setcdruserfield.c

r2566 | citats | 2008-01-15 14:49:03 -0600 (Tue, 15 Jan 2008) | 2 lines

Add manager action 'SetCDRUserField' (bug ASTERISK-1291)