laforge has submitted this change. ( https://gerrit.osmocom.org/c/pysim/+/27177 ) Change subject: filesystem: Fix CardMF.get_app_names() ...................................................................... filesystem: Fix CardMF.get_app_names() This function was not used and doesn't work without this patch. Change-Id: Id3dad7d97fe29a25792d2f8f0e879666c1d9c136 --- 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 d52a16e..95f792a 100644 --- a/pySim/filesystem.py +++ b/pySim/filesystem.py @@ -384,7 +384,7 @@ def get_app_names(self): """Get list of completions (AID names)""" - return [x.name for x in self.applications] + return list(self.applications.values()) def get_selectables(self, flags=[]) -> dict: """Return a dict of {'identifier': File} that is selectable from the current DF. 2 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/+/27177 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: pysim Gerrit-Branch: master Gerrit-Change-Id: Id3dad7d97fe29a25792d2f8f0e879666c1d9c136 Gerrit-Change-Number: 27177 Gerrit-PatchSet: 3 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