| 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-0600 | Date Closed: | 2008-01-15 14:49:03.000-0600 |
| Priority: | Major | Regression? | No |
| Status: | Closed/Complete | Components: | Core/General |
| Versions: | Frequency of Occurrence | ||
| Related Issues: | |||
| 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) ------------------------------------------------------------------------ http://svn.digium.com/view/asterisk?view=rev&revision=2566 | ||