Skip to content

Commit ee749b9

Browse files
committed
Check if role has description before editing
1 parent a202a65 commit ee749b9

File tree

1 file changed

+4
-2
lines changed

1 file changed

+4
-2
lines changed

lib/galaxy/managers/users.py

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -241,11 +241,13 @@ def purge(self, user, flush=True):
241241
for role in user.all_roles():
242242
if self.app.config.redact_username_during_deletion:
243243
role.name = role.name.replace(user.username, uname_hash)
244-
role.description = role.description.replace(user.username, uname_hash)
244+
if role.description:
245+
role.description = role.description.replace(user.username, uname_hash)
245246

246247
if self.app.config.redact_email_during_deletion:
247248
role.name = role.name.replace(user.email, email_hash)
248-
role.description = role.description.replace(user.email, email_hash)
249+
if role.description:
250+
role.description = role.description.replace(user.email, email_hash)
249251
self.session().add(role)
250252
private_role.name = email_hash
251253
private_role.description = f"Private Role for {email_hash}"

0 commit comments

Comments
 (0)