laforge submitted this change.

Approvals:
  Jenkins Builder: Verified
  fixeria: Looks good to me, approved

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(-)

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 change 28627. To unsubscribe, or for help writing mail filters, visit settings.