[asterisk-bugs] [Asterisk 0009939]: Transfer implementation

[noreply at bugs.digium.com]

A NOTE has been added to this issue. 
====================================================================== 
http://bugs.digium.com/view.php?id=9939 
====================================================================== 
Reported By:                wedhorn
Assigned To:                qwell
====================================================================== 
Project:                    Asterisk
Issue ID:                   9939
Category:                   Channels/chan_skinny
Reproducibility:            N/A
Severity:                   feature
Priority:                   normal
Status:                     assigned
Asterisk Version:            SVN 
SVN Branch (only for SVN checkouts, not tarball releases):  trunk 
SVN Revision (number only!): 67843 
Disclaimer on File?:        Yes 
Request Review:              
====================================================================== 
Date Submitted:             06-11-2007 04:33 CDT
Last Modified:              02-07-2008 16:25 CST
====================================================================== 
Summary:                    Transfer implementation
Description: 
Initial patch adding transfer to chan_skinny. Works, but lots of bugs and
more work to be done. Basic functionality is <XFER> to start transfer, puts
call on hold and gives you a dialtone. Dial number and after connected
press <XFER> again and call is transferred.

After first <XFER> press and before call ringing/connected, you can press
<XFER> again to use a blind transfer. In this case transfer occurs when
either channel being dialed indicates ringing or answering. When in blind
transfer you can press <XFER> again to go back to attended transfer.

<HOLD> toggles between the transferee and transferor.
====================================================================== 

---------------------------------------------------------------------- 
 DEA - 02-07-08 16:25  
---------------------------------------------------------------------- 
That was not so bad.  A bunch of fuzzy an single reject involving an
already merged change involving a single ast_verbose() call.

One aspect of the patch that I am not happy with, but which I do not
think
should block it, is the open coded linked list handling of the
subchannels.

I think those sections should be replaced with the list handling macro or
better yet the astobj2 style.  Sadly both are more than I can bite off at
the
moment...

I do not have any of the old model phones that wedhorn has, so I am not
sure that they still work, but I suspect that they do. 

Issue History 
Date Modified   Username       Field                    Change               
====================================================================== 
02-07-08 16:25  DEA            Note Added: 0081890                          
======================================================================