[asterisk-bugs] [JIRA] (ASTERISK-26029) parking: ast_parking_park_call should return parking_space instead of parking_exten

[Diederik de Groot (JIRA)]

    [ https://issues.asterisk.org/jira/browse/ASTERISK-26029?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=230721#comment-230721 ] 

Diederik de Groot commented on ASTERISK-26029:
----------------------------------------------

Hi Richard,

Thank you very much for all the detail provided (I am sure it will be usefull to me in other respects). Looks like a whole slew of work, so a simple statusbar message :-( Because of it being asynchronously and having multiple hazards, i think i should give on this request. We are already monitoring the parking related AMI events, so i should be able to update the statusbar that way (only as a last resort).

It might be a good idea to fix the function documentation, to clarify what it's returning, so you don't get the same request again. And I would deprecate the returned 'exten' param complete, as it's now pretty useless.

> parking: ast_parking_park_call should return parking_space instead of parking_exten
> -----------------------------------------------------------------------------------
>
>                 Key: ASTERISK-26029
>                 URL: https://issues.asterisk.org/jira/browse/ASTERISK-26029
>             Project: Asterisk
>          Issue Type: Bug
>      Security Level: None
>          Components: Channels/chan_skinny, Features/Parking
>    Affects Versions: 13.9.1
>            Reporter: Diederik de Groot
>            Severity: Minor
>
> Occording to the source documentation in:
> include/asterisk/parking.h and res/parking/parking_bridge_features.c the call to 'parking_park_call' should return the extension where the call has been parked via the exten parameter. At the moment it is returning the 'parkext' instead, which is a bit useless. Asterisk-11 (and before) used to provide the parkingslot or parking_space information when calling this function. 
> For example chan_skinny.c is using the returned 'exten' to display the the parkingslot information, but ends up displaying the parkingextension, which is useless.
> Note: If this is not possible, would it be possible to add a pbx_buildin_setvar_helper(char, "PARKING_SPACE",.... while the call occupies the parking_space, which gets assigned at the moment when the call returns because of time.
> Note: If the change is not made, is there another way to get at the required information ?



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