[asterisk-commits] rmudgett: branch rmudgett/cel_accountcode r416662 - in /team/rmudgett/cel_acc...

[SVN commits to the Asterisk project]

Author: rmudgett
Date: Wed Jun 18 22:56:25 2014
New Revision: 416662

URL: http://svnview.digium.com/svn/asterisk?view=rev&rev=416662
Log:
Update the UPGRADE.txt accountcode change documentation.

Modified:
    team/rmudgett/cel_accountcode/UPGRADE.txt
    team/rmudgett/cel_accountcode/main/channel.c

Modified: team/rmudgett/cel_accountcode/UPGRADE.txt
URL: http://svnview.digium.com/svn/asterisk/team/rmudgett/cel_accountcode/UPGRADE.txt?view=diff&rev=416662&r1=416661&r2=416662
==============================================================================
--- team/rmudgett/cel_accountcode/UPGRADE.txt (original)
+++ team/rmudgett/cel_accountcode/UPGRADE.txt Wed Jun 18 22:56:25 2014
@@ -27,15 +27,16 @@
    channels before dialing.  In addition, if a channel already has an
    accountcode then it will not be changed unless explicitly set by
    CHANNEL(accountcode) or an originate method that can specify an
-   accountcode value.  The change has four visible effects.  One, if the
+   accountcode value.  The change has three visible effects.  One, if the
    channel driver is configured with an accountcode for new channels then
    those channels will always have that accountcode unless explicitly
    changed.  Two, local channels now cross accountcode and peeraccount
-   over the local bridge between the ;1 and ;2 channels just like channels
-   in normal bridges.  Three, FollowMe outgoing channels and Queue local
-   channel members now propagate accountcodes.  Four, the
-   CHANNEL(peeraccount) value can be set before Dial to set an empty
-   accountcode on the outgoing channel.
+   across the special bridge between the ;1 and ;2 channels just like
+   channels between normal bridges.  Three, the CHANNEL(peeraccount) value
+   now can be set before Dial, FollowMe, and Queue to set an empty
+   accountcode on the outgoing channel(s).  For Queue this now includes
+   local channel members where the accountcodes are propagated early
+   enough to be useful on the ;2 channel.
 
  - The safe_asterisk script was previously not installed on top of an existing
    version. This caused bug-fixes in that script not to be deployed. If your

Modified: team/rmudgett/cel_accountcode/main/channel.c
URL: http://svnview.digium.com/svn/asterisk/team/rmudgett/cel_accountcode/main/channel.c?view=diff&rev=416662&r1=416661&r2=416662
==============================================================================
--- team/rmudgett/cel_accountcode/main/channel.c (original)
+++ team/rmudgett/cel_accountcode/main/channel.c Wed Jun 18 22:56:25 2014
@@ -6015,6 +6015,11 @@
 	 * The primary reason for the existence of this function is
 	 * so local channels can propagate accountcodes to the ;2
 	 * channel before ast_call().
+	 *
+	 * The secondary reason is to propagate the CHANNEL(peeraccount)
+	 * value set before Dial, FollowMe, and Queue while maintaining
+	 * the historic straight across accountcode propagation as a
+	 * fallback.
 	 */
 	switch (relationship) {
 	case AST_CHANNEL_REQUESTOR_BRIDGE_PEER: