Pull requests

#13 Merged
Repository
dgrinberg dgrinberg
Branch
default
Repository
ubernostrum ubernostrum
Branch
default

delete_expired_users fixed

Author
  1. dgrinberg
Reviewers
Description

delete_expired_users did not actually delete profiles for which users are deleted. In addition, User.DoesNotExist try/except catches when User objects don't exists as references from RegistrationProfile, and removes those profiles. This would happen after a first execution of original delete_expired_users, as it leaves profiles referencing users that don't exist, in a setup that doesn't do referential integrity checks nor on delete triggers, (like MongoDB)

  • Learn about pull requests

Comments (0)