laforge has uploaded this change for review. (
https://gerrit.osmocom.org/c/pysim/+/37624?usp=email )
Change subject: ara_m: Fix pySim.tlv.IE.from_dict() calls
......................................................................
ara_m: Fix pySim.tlv.IE.from_dict() calls
Historically, to_dict and from_dict were not symmetric; this has been
fixed in I07e4feb3800b420d8be7aae8911f828f1da9dab8 in December 2023.
This however broke the ara_m legacy use of the from_dict() methods.
We've just introduced a from_val_dict() method in
I81654ea54aed9e598943f41a26a57dcc3a7f10c2, let's make use of it.
Change-Id: I3aaec40eb665d6254be7b103444c04ff48aac36d
---
M pySim/ara_m.py
1 file changed, 19 insertions(+), 3 deletions(-)
git pull ssh://gerrit.osmocom.org:29418/pysim refs/changes/24/37624/1
diff --git a/pySim/ara_m.py b/pySim/ara_m.py
index 5776e3d..5d0f35d 100644
--- a/pySim/ara_m.py
+++ b/pySim/ara_m.py
@@ -298,8 +298,8 @@
@staticmethod
def get_config(tp, v_major=0, v_minor=0, v_patch=1):
cmd_do = DeviceConfigDO()
- cmd_do.from_dict([{'device_interface_version_do': {
- 'major': v_major, 'minor': v_minor,
'patch': v_patch}}])
+ cmd_do.from_val_dict([{'device_interface_version_do': {
+ 'major': v_major, 'minor': v_minor,
'patch': v_patch}}])
return ADF_ARAM.xceive_apdu_tlv(tp, '80cadf21', cmd_do,
ResponseAramConfigDO)
@with_default_category('Application-Specific Commands')
@@ -372,7 +372,7 @@
ar_do_content += [{'perm_ar_do': {'permissions':
opts.android_permissions}}]
d = [{'ref_ar_do': [{'ref_do': ref_do_content},
{'ar_do': ar_do_content}]}]
csrado = CommandStoreRefArDO()
- csrado.from_dict(d)
+ csrado.from_val_dict(d)
res_do = ADF_ARAM.store_data(self._cmd.lchan.scc._tp, csrado)
if res_do:
self._cmd.poutput_json(res_do.to_dict())
--
To view, visit
https://gerrit.osmocom.org/c/pysim/+/37624?usp=email
To unsubscribe, or for help writing mail filters, visit
https://gerrit.osmocom.org/settings
Gerrit-Project: pysim
Gerrit-Branch: master
Gerrit-Change-Id: I3aaec40eb665d6254be7b103444c04ff48aac36d
Gerrit-Change-Number: 37624
Gerrit-PatchSet: 1
Gerrit-Owner: laforge <laforge(a)osmocom.org>
Gerrit-MessageType: newchange