scholtzm / punk

⚡ Alternative cross-platform Steam client for desktop
MIT License
100 stars 29 forks source link

Differentiate blocked users #35

Closed losuler closed 7 years ago

losuler commented 7 years ago

Issue description

Currently blocked users are indistinguishable from other offline friends.

Suggestions

Potential ways I think this could be dealt with are:

scholtzm commented 7 years ago

Thank you for your suggestions, I'll probably go with option number two. Just need to find someone to block. 😄

scholtzm commented 7 years ago

It's been a while since this report but I finally looked at this and I'm not able to reproduce this behaviour.

There are 4 people who I permanently blocked, they do not appear in my Punk friends list.

I have also tried blocking my friend, his name instantly disappeared from Punk friends list. After unblocking him via steamcommunity.com, his name instantly reappears in my friends list.

Can you provide steps to reproduce this behaviour?

losuler commented 7 years ago

In looking into this, these were the following steps I took and the observations I made from them.

Note: To ensure my observations weren't effected in anyway by #36, I made sure neither clients were running simultaneously. I also newly cloned and built Punk before I went ahead with the above.

Now beyond this, I discovered that people that I had blocked but which were not still my friends appeared in Punk at the bottom of the friends list as offline. Whereas in the Steam client these people didn't appear in my friends list at all, and were only accessible through the Steam website under Profile >> Friends >> Blocked List.

scholtzm commented 7 years ago

Thank you for a very thorough description, I will look into it this week. 👍

scholtzm commented 7 years ago

@losuler

Ok, so I have been digging into this issue and I managed to reproduce all 3 cases you described. Handling of blocked users is broken right now and I'll need to do some adjustments to personastate plugin which handles the friends list.

Relevant bits from SteamKit: https://github.com/SteamRE/SteamKit/blob/master/SteamKit2/SteamKit2/Steam/Handlers/SteamFriends/SteamFriends.cs#L737 https://github.com/SteamRE/SteamKit/blob/master/SteamKit2/SteamKit2/Steam/Handlers/SteamFriends/SteamFriends.cs#L647

scholtzm commented 7 years ago

Should be fixed in https://github.com/scholtzm/punk/commit/e6c4bffad13b05df0d46588de1f2e9baa6400f69 @ master.

Next release will contain this fix.