From 497a048b595b6a1a09683fa8c6939aae4681e69f Mon Sep 17 00:00:00 2001 From: silverstreak Date: Sat, 25 May 2024 14:03:48 +0200 Subject: [PATCH] Bugfix in helper.py Bugfix for the error: Traceback (most recent call last): File "/home/xxx/.local/bin/fabric", line 8, in sys.exit(cli()) ^^^^^ File "/home/xxx/.local/share/pipx/venvs/fabric/lib/python3.12/site-packages/installer/client/cli/fabric.py", line 148, in main session.list_sessions() File "/home/xxx/.local/share/pipx/venvs/fabric/lib/python3.12/site-packages/installer/client/cli/helper.py", line 67, in list_sessions most_recent = self.find_most_recent_file().split("/")[-1] ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ AttributeError: 'NoneType' object has no attribute 'split' --- installer/client/cli/helper.py | 22 +++++++++++++--------- 1 file changed, 13 insertions(+), 9 deletions(-) diff --git a/installer/client/cli/helper.py b/installer/client/cli/helper.py index afde5ec..3f5db8a 100644 --- a/installer/client/cli/helper.py +++ b/installer/client/cli/helper.py @@ -60,12 +60,16 @@ class Session: def list_sessions(self): sessionlist = os.listdir(self.sessions_folder) - most_recent = self.find_most_recent_file().split("/")[-1] - for session in sessionlist: - with open(os.path.join(self.sessions_folder, session), "r") as f: - firstline = f.readline().strip() - secondline = f.readline().strip() - if session == most_recent: - print(f"{session} **default** \"{firstline}\n{secondline}\n\"") - else: - print(f"{session} \"{firstline}\n{secondline}\n\"") + find_most_recent_file_result = self.find_most_recent_file() + if find_most_recent_file_result is not None: + most_recent = find_most_recent_file_result.split("/")[-1] + for session in sessionlist: + with open(os.path.join(self.sessions_folder, session), "r") as f: + firstline = f.readline().strip() + secondline = f.readline().strip() + if session == most_recent: + print(f"{session} **default** \"{firstline}\n{secondline}\n\"") + else: + print(f"{session} \"{firstline}\n{secondline}\n\"") + else: + print('No files present in sessions directory')