KAZOO-3208: track other leg, just don't bind

[jamesaimonetti]

@onnet See if this patch works for you. It is lighter weight than your approach, and takes advantage of the fact that we had the information, we just discarded it!

[onnet]

To make this patch play MOH to transferee, the next additions needed in my case:

-spec maybe_handle_aleg_code(state()) -> 'ok'.
maybe_handle_aleg_code(#state{numbers=Ns
,other_leg=OtherLeg
,patterns=Ps
,a_collected_dtmf = Collected
,call=Call
,call_id=CallId
}) ->
lager:debug("a DTMF timeout, let's check '~s'", [Collected]),
case has_metaflow(Collected, Ns, Ps) of
'false' -> lager:debug("no handler for '~s', unarming", [Collected]);
{'number', N} -> handle_number_metaflow(whapps_call:set_other_leg_call_id(OtherLeg, Call), N, CallId);
{'pattern', P} -> handle_pattern_metaflow(whapps_call:set_other_leg_call_id(OtherLeg, Call), P, CallId)
end.

handle_channel_bridged(#state{other_leg='undefined'}=State, _JObj, OtherLeg) ->
State#state{other_leg=OtherLeg};
handle_channel_bridged(#state{other_leg=_OL, call_id=_CallId}=State, _JObj, _OtherLeg) ->
State.

[jamesaimonetti]

@onnet thanks for the report; let me try this patch locally and add it here if I find similar results. And thanks for testing this out!

[onnet]

Hi James,

No thanks needed, just kick someone to give me an answer to this thread instead :))

https://groups.google.com/forum/#!topic/2600hz-dev/iGqJzOWKMp0

Please, please, please ...
... planning to write cb_call_recordings.erl for third_party_bigcouch stored call recordings, but feel that it could be a waste of time if it will be decided to store call recordings at MODB

Sory for impatience :)

Regards,
Kirill

[jamesaimonetti]

@onnet I am able to get the MOH without your patch above. Can you re-test using a recent 3.18 build to see if the issue is fixed for you? I'm going to close this pull request in the meantime as the work has been merged in.