[asterisk-bugs] [JIRA] (ASTERISK-24867) Docs for 'e' option in ResetCDR say to use CDR_PROP instead, CDR_PROP docs are unclear

[Rusty Newton (JIRA)]

Rusty Newton created ASTERISK-24867:
---------------------------------------

             Summary: Docs for 'e' option in ResetCDR say to use CDR_PROP instead, CDR_PROP docs are unclear
                 Key: ASTERISK-24867
                 URL: https://issues.asterisk.org/jira/browse/ASTERISK-24867
             Project: Asterisk
          Issue Type: Bug
      Security Level: None
          Components: Documentation
            Reporter: Rusty Newton
            Severity: Trivial


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.



--
This message was sent by Atlassian JIRA
(v6.2#6252)