Change in pysim[master]: fileystem: Use human-readable ADF name if available.

laforge has submitted this change. ( https://gerrit.osmocom.org/c/pysim/+/28627 ) Change subject: fileystem: Use human-readable ADF name if available. ...................................................................... fileystem: Use human-readable ADF name if available. When using __str__ for a CardDF we would get "DF(DF.TELECOM)" but when using it on CardADF we would get ADF(a0000000871002)" instead of "ADF(ADF.USIM)". Let's fix that. Change-Id: I5801a08bcc28cb222734af6d9ee835227f4fee69 --- M pySim/filesystem.py 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Jenkins Builder: Verified fixeria: Looks good to me, approved diff --git a/pySim/filesystem.py b/pySim/filesystem.py index 813bebb..a3e83d6 100644 --- a/pySim/filesystem.py +++ b/pySim/filesystem.py @@ -488,7 +488,7 @@ mf.add_application_df(self) def __str__(self): - return "ADF(%s)" % (self.aid) + return "ADF(%s)" % (self.name if self.name else self.aid) def _path_element(self, prefer_name: bool): if self.name and prefer_name: 1 is the latest approved patch-set. No files were changed between the latest approved patch-set and the submitted one. -- To view, visit https://gerrit.osmocom.org/c/pysim/+/28627 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: pysim Gerrit-Branch: master Gerrit-Change-Id: I5801a08bcc28cb222734af6d9ee835227f4fee69 Gerrit-Change-Number: 28627 Gerrit-PatchSet: 2 Gerrit-Owner: laforge <laforge(a)osmocom.org> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: dexter <pmaier(a)sysmocom.de> Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-MessageType: merged