public/pysim
1
0
Fork 1
mirror of https://gitea.osmocom.org/sim-card/pysim.git synced 2026-03-27 15:58:34 +03:00
Code Issues Packages Projects Releases Wiki Activity

cosmetic: Implement cmd2.Settable backwards-compat via wrapper class

Browse Source 
Let's avoid too many open-coded if-clauses and simply wrap it in
a compatibility class.

Change-Id: Id234f3fa56fe7eff8e1153d71b9be8a2e88dd112
This commit is contained in:
Harald Welte
2023-06-24 10:00:12 +02:00
parent 985ff31efa
commit 0ec01504ab
1 changed files with 18 additions and 24 deletions
Download Patch File Download Diff File Expand all files Collapse all files

42
pySim-shell.py
View File

@@ -149,11 +149,18 @@ class Cmd2Compat(cmd2.Cmd):
releases. See https://github.com/python-cmd2/cmd2/blob/master/CHANGELOG.md""" releases. See https://github.com/python-cmd2/cmd2/blob/master/CHANGELOG.md"""
def run_editor(self, file_path: Optional[str] = None) -> None: def run_editor(self, file_path: Optional[str] = None) -> None:
if version.parse(cmd2.__version__) < version.parse("2.0.0"): if version.parse(cmd2.__version__) < version.parse("2.0.0"):
# pylint: disable=no-member return self._run_editor(file_path) # pylint: disable=no-member
return self._run_editor(file_path)
else: else:
# pylint: disable=no-member return super().run_editor(file_path) # pylint: disable=no-member
return super().run_editor(file_path)
class Settable2Compat(cmd2.Settable):
"""Backwards-compatibility wrapper around cmd2.Settable to support older and newer
releases. See https://github.com/python-cmd2/cmd2/blob/master/CHANGELOG.md"""
def __init__(self, name, val_type, description, settable_object, **kwargs):
if version.parse(cmd2.__version__) < version.parse("2.0.0"):
super().__init__(name, val_type, description, **kwargs) # pylint: disable=no-value-for-parameter
else:
super().__init__(name, val_type, description, settable_object, **kwargs) # pylint: disable=too-many-function-args
class PysimApp(Cmd2Compat): class PysimApp(Cmd2Compat):
CUSTOM_CATEGORY = 'pySim Commands' CUSTOM_CATEGORY = 'pySim Commands'
@@ -181,26 +188,13 @@ class PysimApp(Cmd2Compat):
self.json_pretty_print = True self.json_pretty_print = True
self.apdu_trace = False self.apdu_trace = False
if version.parse(cmd2.__version__) < version.parse("2.0.0"): self.add_settable(Settable2Compat('numeric_path', bool, 'Print File IDs instead of names', self,
# pylint: disable=no-value-for-parameter onchange_cb=self._onchange_numeric_path))
self.add_settable(cmd2.Settable('numeric_path', bool, 'Print File IDs instead of names', self.add_settable(Settable2Compat('conserve_write', bool, 'Read and compare before write', self,
onchange_cb=self._onchange_numeric_path)) onchange_cb=self._onchange_conserve_write))
# pylint: disable=no-value-for-parameter self.add_settable(Settable2Compat('json_pretty_print', bool, 'Pretty-Print JSON output', self))
self.add_settable(cmd2.Settable('conserve_write', bool, 'Read and compare before write', self.add_settable(Settable2Compat('apdu_trace', bool, 'Trace and display APDUs exchanged with card', self,
onchange_cb=self._onchange_conserve_write)) onchange_cb=self._onchange_apdu_trace))
# pylint: disable=no-value-for-parameter
self.add_settable(cmd2.Settable('json_pretty_print', bool, 'Pretty-Print JSON output'))
# pylint: disable=no-value-for-parameter
self.add_settable(cmd2.Settable('apdu_trace', bool, 'Trace and display APDUs exchanged with card',
onchange_cb=self._onchange_apdu_trace))
else:
self.add_settable(cmd2.Settable('numeric_path', bool, 'Print File IDs instead of names', self, \
onchange_cb=self._onchange_numeric_path)) # pylint: disable=too-many-function-args
self.add_settable(cmd2.Settable('conserve_write', bool, 'Read and compare before write', self, \
onchange_cb=self._onchange_conserve_write)) # pylint: disable=too-many-function-args
self.add_settable(cmd2.Settable('json_pretty_print', bool, 'Pretty-Print JSON output', self)) # pylint: disable=too-many-function-args
self.add_settable(cmd2.Settable('apdu_trace', bool, 'Trace and display APDUs exchanged with card', self, \
onchange_cb=self._onchange_apdu_trace)) # pylint: disable=too-many-function-args
self.equip(card, rs) self.equip(card, rs)
def equip(self, card, rs): def equip(self, card, rs):