Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[deleted] added as user. #20

Open
ohhdemgirls opened this issue Apr 25, 2014 · 3 comments
Open

[deleted] added as user. #20

ohhdemgirls opened this issue Apr 25, 2014 · 3 comments

Comments

@ohhdemgirls
Copy link

When a user deletes their account/posts [deleted] gets added as a user to the database and /content/[deleted] is created.

Flags set --friend-zone --config save_thumbnails false

@ohhdemgirls ohhdemgirls changed the title [deleted] Added as user. [deleted] added as user. Apr 25, 2014
4pr0n added a commit that referenced this issue Apr 26, 2014
Also, add_top_users adds users to the reddit friends list as it finds
new users
@4pr0n
Copy link
Owner

4pr0n commented Apr 26, 2014

I... don't see how this is possible.

  1. Can't get [deleted] from user pages (only happens when user deletes account, so user page is 404 at that point)
  2. Can't get [deleted] from friends, because if they're deleted, then they shouldn't show up on friends list (I'm assuming).
  3. The add_top_users functionality purposefully ignores [deleted] accounts: Gonewild.py#L358, fix added 2013-12-01

I just checked /r/friends for the past few days with around 3,000 friends added, and didn't see any [deleted].


This never showed up before the friends functionality was added, and you are using it, so there's now a check while parsing /r/friends to ignore [deleted] accounts. Seems kinda weird that you can still view posts of deleted accounts if they're friends...

@ohhdemgirls
Copy link
Author

I... don't see how this is possible.

I didn't think it was, I wasn't logging when it happened though I am now, if it happens again I'll dump the log here.


Hmm, I can't even recreate this with test accounts, very annoying...


First report someone else had the issue, (before latest commit) - zzoJG1q.png

@mixbiscuit
Copy link

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants