dexter has submitted this change. ( 
https://gerrit.osmocom.org/c/onomondo-eim/+/42876?usp=email )

 (

1 is the latest approved patch-set.
No files were changed between the latest approved patch-set and the submitted 
one.
 )Change subject: esipa_asn1_handler: fix transactionId handling
......................................................................

esipa_asn1_handler: fix transactionId handling

The transactionId member is now called eimTransactionId. It is also
an optional member.

Change-Id: I1340e0c7aed091e8e5a5089ebd6e524512408cf7
Related: SYS#8100
---
M src/esipa_asn1_handler_utils.erl
1 file changed, 4 insertions(+), 4 deletions(-)

Approvals:
  Jenkins Builder: Verified
  fixeria: Looks good to me, but someone else must approve
  laforge: Looks good to me, approved




diff --git a/src/esipa_asn1_handler_utils.erl b/src/esipa_asn1_handler_utils.erl
index 5bf47da..c297638 100644
--- a/src/esipa_asn1_handler_utils.erl
+++ b/src/esipa_asn1_handler_utils.erl
@@ -14,21 +14,21 @@
             EuiccPackageResultDataSigned = maps:get(
                 euiccPackageResultDataSigned, EuiccPackageResultSigned
             ),
-            maps:get(transactionId, EuiccPackageResultDataSigned);
+            maps:get(eimTransactionId, EuiccPackageResultDataSigned, none);
         {euiccPackageErrorSigned, EuiccPackageErrorSigned} ->
             EuiccPackageErrorDataSigned = maps:get(
                 euiccPackageErrorDataSigned, EuiccPackageErrorSigned
             ),
-            maps:get(transactionId, EuiccPackageErrorDataSigned);
+            maps:get(eimTransactionId, EuiccPackageErrorDataSigned, none);
         _ ->
             none
     end.

 process_euiccPackageResult(Pid, EuiccPackageResult, EsipaReq, TransactionId) ->
     WorkBind = fun(Map) ->
-        case maps:is_key(transactionId, Map) of
+        case maps:is_key(eimTransactionId, Map) of
             true ->
-                mnesia_db:work_bind(Pid, maps:get(transactionId, Map));
+                mnesia_db:work_bind(Pid, maps:get(eimTransactionId, Map));
             _ ->
                 ok
         end

--
To view, visit https://gerrit.osmocom.org/c/onomondo-eim/+/42876?usp=email
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings?usp=email

Gerrit-MessageType: merged
Gerrit-Project: onomondo-eim
Gerrit-Branch: master
Gerrit-Change-Id: I1340e0c7aed091e8e5a5089ebd6e524512408cf7
Gerrit-Change-Number: 42876
Gerrit-PatchSet: 4
Gerrit-Owner: dexter <[email protected]>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: dexter <[email protected]>
Gerrit-Reviewer: fixeria <[email protected]>
Gerrit-Reviewer: laforge <[email protected]>

Reply via email to