The filearchive table stores information about the deleting user as a reference to user id, which for all anonymous users is 0. Instead, it should refer to the user's actor id, to properly reflect the actions of anonymous users, which have actor ids.
Description
Description
Details
Details
- Reference
- bz27615
Related Objects
Related Objects
- Mentioned In
- T297215: Drop unused filearchive.fa_deleted_reason_id and filearchive.fa_deleted_user
- Mentioned Here
- T28741: Migrate file tables to a modern layout (image/oldimage; file/file_revision; add primary keys)
T297215: Drop unused filearchive.fa_deleted_reason_id and filearchive.fa_deleted_user
T29872: add pt_user_text to protected_titles table to allow page protection by anon
T167246: Refactor "user" & "user_text" fields into "actor" reference table
Event Timeline
Comment Actions
Reopen, because this field is needed to allow anon users delete a file or to create a maintaince script, which delete files wihtout a exiting user account. The field should be set only, when the fa_deleted_user field is 0, because than that is an anon user/system user.
Comment Actions
fa_deleted_user is the user that deleted the file. Instead of adding a user text field, it should be replaced with a reference to the actor table
Comment Actions
This is not a blocker for calling actor migration done, on top of that, this ticket should be decline in favor of T297215: Drop unused filearchive.fa_deleted_reason_id and filearchive.fa_deleted_user or T28741: Migrate file tables to a modern layout (image/oldimage; file/file_revision; add primary keys)