When cleanup_database deletes patrons based on the PatronRemovalDelay system preference, it sends those patrons to the deleteborrowers table. It seems like it would be more in keeping with user/patron expectations to fully delete those borrowers, skipping deletedborrowers entirely. We're looking at developing a feature for recovering borrowers from deletedborrowers, and we definitely would not want to allow restoration of patrons who declined GDPR consent. That'd be easier to avoid if we just delete them entirely in the first place.