[Home]

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:29Date Closed:2015-07-20 18:32:37
Priority:TrivialRegression?
Status:Closed/CompleteComponents: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: