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 <lafo...@osmocom.org> Gerrit-MessageType: newchange