Skip to content

fix USSD CDR issue not being saved to DB#1896

Merged
maria-farooq merged 1 commit intomasterfrom
issue#1820
Mar 21, 2017
Merged

fix USSD CDR issue not being saved to DB#1896
maria-farooq merged 1 commit intomasterfrom
issue#1820

Conversation

@croufay
Copy link
Copy Markdown
Contributor

@croufay croufay commented Feb 27, 2017

the CDR is now updated in the db for USSD request

@maria-farooq
Copy link
Copy Markdown

maria-farooq commented Mar 9, 2017

@croufay can you please share your findings about root cause why cdr was not being updated.
I see from code that eliminating following 2 conditions you were able to successfully add cdr:

  1. if (CallResponse.class.equals(klass)) {
  2. if (callInfo.direction().equals("inbound")) {

eliminating 1 is valid in my opinion as klass here can be either one of CallResponse or SipServletRequest.

But can you explain why you removed second check if (callInfo.direction().equals("inbound"))
As per my understanding inbound cdr is populated by Interpreter and outbound cdr is populated by Call/UssdCall so this check should be in place.

But maybe i am missing something so you can help me here..

@maria-farooq
Copy link
Copy Markdown

@croufay do we have any update here?

@croufay
Copy link
Copy Markdown
Contributor Author

croufay commented Mar 21, 2017

@maria-farooq ,

When I sent USSD request, the if (callInfo.direction().equals("inbound"))

didn't contain the word "inbound" so that part of the code was always false hence, not CDR will be recorded. I think this was because part of the USSD interpreter was copied from the VoiceInterpreter that had the inbound in the callInfo.direction.

Regards

Charles Roufay
Telestax

@maria-farooq
Copy link
Copy Markdown

thank for update @croufay i will merge it now

@maria-farooq maria-farooq merged commit eac4fc4 into master Mar 21, 2017
@gvagenas gvagenas added this to the 8.1.0 milestone Apr 10, 2017
@jaimecasero jaimecasero deleted the issue#1820 branch November 22, 2017 09:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants