vassili/seitime-frappe
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions 4

refactor: Cleanup name confusion

Browse source 
Using `delete_session` name for a function and also as a method name is
confusing. Cleaned that up.

(cherry picked from commit 9200192c1c)
This commit is contained in:
leela 2021-03-30 12:04:09 +05:30 • committed by mergify-bot
parent 79233f007d
commit bda37b541b
1 changed files with 3 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

7
frappe/sessions.py
View file

@ -296,8 +296,7 @@ class Session:
expiry = get_expiry_in_seconds(session_data.get("session_expiry"))
if self.time_diff > expiry:
print('deleting...')
self.delete_session()
self._delete_session()
data = None
return data and data.data
@ -316,12 +315,12 @@ class Session:
data = frappe._dict(eval(rec and rec[0][1] or '{}'))
data.user = rec[0][0]
else:
self.delete_session()
self._delete_session()
data = None
return data
def delete_session(self):
def _delete_session(self):
delete_session(self.sid, reason="Session Expired")
def start_as_guest(self):