Adding imposters to admin

[Wild1145]

For some reason when you add someone who should actually be a imposer to admin it duplicates the entry and forms a new one. This means if a senior logs in they dont get flagged (see other issue) but then you cant add them back to senior.

This looks to be caused by a broken UUID either in Superadmin adding or the converters system from old Superadmin.yml to new.

One to add the what should have been imposter crafter

 3c6233a0-802c-3669-a807-16722684bd83:
last_login_name: CrafterSmith12
is_activated: true
is_telnet_admin: false
is_senior_admin: false
last_login: Tue, 22 Apr 2014 22:20:02 +0100
custom_login_message: ''
console_aliases: []
ips:
- 12.345.678.90

One converted from old Superadmin.yml

 222e1cb5-18a5-4825-af5b-780de658fe33:
last_login_name: CrafterSmith12
is_activated: true
is_telnet_admin: true
is_senior_admin: true
last_login: Tue, 15 Apr 2014 21:27:28 +0100
custom_login_message: '&da Probationary Senior Admin'
console_aliases: []
ips:
- 12.345.678.90

Please note that IP's have been removed for security.

[JeromSar]

Just verified that it works as intended. You might have an actual impostor on your hands.

  b3c3b05a-a52d-33a6-a9c6-6e5063e00f0a:
    last_login_name: DarthSalamon
    is_activated: true
    is_telnet_admin: true
    is_senior_admin: true
    last_login: Wed, 23 Apr 2014 15:24:22 +0200
    custom_login_message: the &5Lead Executive Developer
    console_aliases: []
    ips:
    - 1.1.1.1
    - 1.2.3.4

After adding:

  b3c3b05a-a52d-33a6-a9c6-6e5063e00f0a:
    last_login_name: DarthSalamon
    is_activated: true
    is_telnet_admin: true
    is_senior_admin: true
    last_login: Wed, 23 Apr 2014 15:24:22 +0200
    custom_login_message: the &5Lead Executive Developer
    console_aliases: []
    ips:
    - 1.1.1.1
    - 1.2.3.4
    - 127.0.0.1

[Wild1145]

I used the Vanilla TFM to do it... Unless something has changed getting UUID's then there is a problem.

It also cant be a "Fake" as:

1. You cant change your name
2. The actual UUID's were different
3. They verified.

Its happened with multiple admins... I will try again with the latest source code, but if not il re-open the issue here.

[JeromSar]

Try running on CraftBukkit instead of Spigot.

[Wild1145]

I dont see how that can even make a difference.

From what I understand, TFM is getting the UUID from the mojang servers through a API. What difference does spigot and Craftbukkit have? Il try the latest version anyway unless it was fixed without realizing before, if not il try craftbukkit, but i am 99% sure it wont make any difference.

[Wild1145]

I give you https://pravi.us/CoQ

While it is no longer actually adding duplicate entries from the latest pull, this is happening.

[Wild1145]

This is not fixed and needs to be re-opened.

[JeromSar]

This is a duplicate of #156