Bugreport - Addon writes to users.txt still

[Bryantdl7]

Hi there,

in the latest version I realized ranks done either via ulx adduserid or ulx adduser end up in the users.txt. Surely I could just make the file read-only, however, I wanted to report this bug.

Also, it would be nice to have a verbose entry in console for when someone is updated in the database successfully, for peace of mind. I found out the hard way that this was still writing to the users.txt, and thus someone kept their rank despite me removing it!

[ApertureDevelopment]

This is not a bug and in fact intended behaviour. As I stated in the past, MSync is supposed to work alongside ULX and not intrusive and overwriting. And normally MSync should load the ranks after the user has already been loaded by ulx, so that the rank gets set to the default rank after the user loaded

[Bryantdl7]

The problem we run into, is sometimes the users.txt doesn't update, only msync does. As both are still considered, this can cause for people to not be properly adjusted, and in our case, someone who was supposed to be demoted wasn't. This problem did not occur prior to using your addon.

[ApertureDevelopment]

I have implemented a bugfix for that behaviour, users should now be removed from the MRSync database when you demote them, and if MRSync notices that the user is not in a nosync rank and that his rank isn't user, it removes him entirely from ULX

[ApertureDevelopment]

Well, the bugfix is not yet published

[ApertureDevelopment]

There has been no activity in this issue for 6 days. If there is no reply within the next 24 hours, this issue will be closed and marked as "resolved"

[Bryantdl7]

When you say the bug fix is not yet published, should I assume your development branch contains untested code you would like me to test?

…

Sent from my iPhone On Jul 23, 2020, at 9:17 AM, Aperture Development <notifications@github.com> wrote:  There has been no activity in this issue for 6 days. If there is no reply within the next 24 hours, this issue will be closed and marked as "resolved" — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub<#56 (comment)>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/ABKOR5JQBXLW3WMG4RRRCZTR5AZ5FANCNFSM4O63KSVQ>.

[ApertureDevelopment]

Yes, I would like you to try out the development branch if possible

[Bryantdl7]

This appears to be functional. I am able to pull my rank from the one server, and joining the other I keep the rank applied.

I still worry that users.txt will hold priority, but I will need further testing to see if that is a problem. The code seems solid functionality speaking, but when traversing between three different servers is when I will know for sure.

[ApertureDevelopment]

Alright, I will PR it into the development branch then