Posted by Dane Powell on August 5, 2009 at 7:23pm
| Project: | Workflow |
| Version: | 6.x-1.3 |
| Component: | Code |
| Category: | bug report |
| Priority: | normal |
| Assigned: | Unassigned |
| Status: | closed (fixed) |
Issue Summary
My understanding is that when a user is deleted, their uid should be changed to 0 throughout the database - is that correct? Workflow does not seem to respect this, so that it's possible to have invalid uids in the Workflow tables. This causes subsequent problems in other modules.
I think we should be using hook_user() to capture deletes and update deleted users' uid to 0 in all workflow tables.
Comments
#1
I'm happy to patch this myself but I'd appreciate it if someone could let me know if this is indeed the correct solution before I go to the trouble. Thanks.
#2
I believe the original reason for this was European-style privacy laws that require data retention. But one could argue that in that case the user should not be deleted, only blocked.
#3
Here is a patch that takes care of this.
#4
This patch works great for me, can someone review it? Or it's so simple can you just commit it?
#5
Bueller? Anyone?
#6
Is there something preventing this from being committed? This is certainly a bug and goes against Drupal standards, and really does cause weird errors with other modules...
#7
If you're not willing to commit this please at least have to the courtesy to mark it as "won't fix"... it drives me nuts when RTBC issues sit for years without getting committed (and I have quite a few such issues in my queue right now, cumulatively representing a huge amount of wasted effort.)
#8
Committed. Will appear in 6.x-1.5. Sorry for the delay.
#9
Automatically closed -- issue fixed for 2 weeks with no activity.