Summary:	ASTERISK-24867: Docs for 'e' option in ResetCDR say to use CDR_PROP instead, CDR_PROP docs are unclear
Reporter:	Rusty Newton (rnewton)	Labels:
Date Opened:	2015-03-11 16:12:29	Date Closed:	2015-07-20 18:32:37
Priority:	Trivial	Regression?
Status:	Closed/Complete	Components:	Documentation
Versions:		Frequency of Occurrence	Constant
Related Issues:
Environment:		Attachments:
Description:	The 'e' option for app ResetCDR is intended to enable CDR on a channel, where CDR had previously been disabled via app NoCDR. The help text for ResetCDR says: {quote} The e option is deprecated. Please use the CDR_PROP function instead. {quote} The full help text for function CDR_PROP says: {quote}  -= Info about function 'CDR_PROP' =- [Synopsis] Set a property on a channel's CDR. [Description] This function sets a property on a channel's CDR. Properties alter the behavior of how the CDR operates for that channel. [Syntax] CDR_PROP(name) [Arguments] name    The property to set on the CDR.    party_a - Set this channel as the preferred Party A when channels    are associated together.    Write-Only    disable - Disable CDRs for this channel.    Write-Only [See Also] Not available {quote} It isn't clear how to enable CDR on a channel using CDR_PROP. We had a discussion in IRC {quote} <newtonr> Does anyone know how you would enable CDR on a channel after CDR_PROP(disable) has been set ? ... <coreyfarrell> newtonr: not positive, but it looks like CDR_PROP(disable)=0 should 'unset' it?  looks like ast_true is used on the value passed, so if it's false then it unsets ... <coreyfarrell> seems weird to unset disable, double negatives are not nice <file> can you even... do that? <mjordan> file: yes, you can {quote} We should modify the documentation for CDR_PROP at least to describe how to enable CDR. Or, if someone wants, provide a patch for an 'enable' argument.
Comments: