[Home]

Summary:ASTERISK-21224: [patch] Skinny groupPickup issues
Reporter:Damien Wedhorn (wedhorn)Labels:patch
Date Opened:2013-03-08 00:48:22.000-0600Date Closed:
Priority:MajorRegression?
Status:Open/NewComponents:Channels/chan_skinny
Versions:SVN 13.18.4 Frequency of
Occurrence
Related
Issues:
Environment:Attachments:( 0) skinny-pickupconf01.diff
( 1) skinny-pickupconf02.diff
( 2) skinny-pickupconf03.diff
Description:Skinny segfaults unless both pickupgroup and callgroup are specified.
Comments:By: Damien Wedhorn (wedhorn) 2013-03-08 00:50:17.560-0600

Small patch, sets callgroup to pickupgroup if callgroup left blank. Also disables the gpickup if no pickupgroup specified.

Could probably grey out the softkey as well.

By: Damien Wedhorn (wedhorn) 2013-03-14 15:20:13.525-0500

New patch (02) that grey's out the gpickup key when not set.

Added the (soft)keymask to line given that we can have different setups on each line (eg one line may have pickup and the other doesn't).

Had to add a transmit softkeyset to ONHOOK to ensure the default onhook keyset gpickup is appropriately greyed. The default keyset is based on the line you'd get if you lifted up the receiver.

May want to rethink gpickup when activated from an onhook device to possibly iterate through available lines.

By: Damien Wedhorn (wedhorn) 2013-03-15 13:52:22.671-0500

Small revision to 02.diff. Moves the setting of the l->Keymask from sub config to line config.