pespin submitted this change.

View Change


Approvals: laforge: Looks good to me, but someone else must approve Jenkins Builder: Verified osmith: Looks good to me, approved
asterisk/AMI_Functions: Add workaround for empty 'Hint: ' field

It sometimes shows up empty, let's skip it to avoid breaking the decoder
for now, similar to what we do with other fields.

Change-Id: I50fb51fab3755e4d63f3a0bc06d90b42814231d2
---
M asterisk/AMI_Functions.ttcn
1 file changed, 13 insertions(+), 0 deletions(-)

diff --git a/asterisk/AMI_Functions.ttcn b/asterisk/AMI_Functions.ttcn
index 1f3dc5e..c05403e 100644
--- a/asterisk/AMI_Functions.ttcn
+++ b/asterisk/AMI_Functions.ttcn
@@ -261,6 +261,7 @@
patched_txt := f_str_replace(patched_txt, "DestExten: \r\n", "");
patched_txt := f_str_replace(patched_txt, "Exten: \r\n", "");
patched_txt := f_str_replace(patched_txt, "Extension: \r\n", "");
+ patched_txt := f_str_replace(patched_txt, "Hint: \r\n", "");

/* "AppData" field sometimes has a value containing separator ": ", which makes
* TEXT dec not happy. Workaround it for now by removing the whole field line:

To view, visit change 36854. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: osmo-ttcn3-hacks
Gerrit-Branch: master
Gerrit-Change-Id: I50fb51fab3755e4d63f3a0bc06d90b42814231d2
Gerrit-Change-Number: 36854
Gerrit-PatchSet: 2
Gerrit-Owner: pespin <pespin@sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge@osmocom.org>
Gerrit-Reviewer: osmith <osmith@sysmocom.de>
Gerrit-Reviewer: pespin <pespin@sysmocom.de>
Gerrit-MessageType: merged