Jump to content

Recommended Posts

Hello,

We are happy to have this great application purchased and installed.

The only concern we found, is that your application creates logs for every new registered users, regardless the NOT filling the custom profile field "Steam". This field is not required to be filled in, so even the users, who didn't fill in their Steam IDs in that field, are still will be logged into the "View Profiles" section on the ACP.

It will say like "ERROR: steam_no_steamid". See the picture for better understanding.

steam.PNG

We have almost 200 new users registering on our board on the daily basis, so if continue keeping them logged, the logs on the "View Profiles" will have hundreds of pages :) if not thousands...

So, how about to implement a "Button" for "hiding" the users from that list, who don't fill their "Steam" profile field?

Thank you,

Alexey

Share this post


Link to post
Share on other sites

What version are you running?  This is likely to be a bug that's persistent or remnants from when the bug existed prior to installing a bug fix version.

The Sign in Through Steam app used to store NULL in the members table when a user did not link their steam account, now it appears to store in different formats depending on the database configuration.  

I'll log a bug shortly and look at adding pruning of this errant data in the cleanup routine.

Share this post


Link to post
Share on other sites

I've got the fix half written, but the cleanup routine needs to be rewritten entirely.  A bit more involved than I thought.   

Share this post


Link to post
Share on other sites

I went about this a slightly different way.  

1) I disabled storing of empty profiles.  
2) Added a filter to only show Active profiles in the table, any profile with a valid SteamID
3) Added an uninstall routine that delete all steam_no_steamid profiles.  Policy is to only store profiles with some kind of steam value, even if it's bad, or they manually attempted to update their profile.  

Share this post


Link to post
Share on other sites

Create an account or sign in to comment

You need to be a member in order to leave a comment

Create an account

Sign up for a new account in our community. It's easy!

Register a new account

Sign in

Already have an account? Sign in here.

Sign In Now


×