Newcomer1989 / TSN-Ranksystem

A PHP Bot that assigns time based server groups on TeamSpeak3.
https://ts-ranksystem.com
GNU General Public License v3.0
144 stars 60 forks source link

Addon assign servergroups bug (if notification is disabled) #225

Closed nisutec closed 6 years ago

nisutec commented 7 years ago

My Users can't get any groups. The Webinterface says "Please choose at least one new group." but i have a group selected.

Shad86 commented 7 years ago

I Checked it - and it works without any issues.

You get this message only if you already have all the groups what are currently assigned.

So if you select a group what you already have, sure it says you have to choose because the assigner knows if you have the group or not.

Or in an easy sentence.

it doesn't make sense to save assigned groups, if your already own it.

nisutec commented 7 years ago

Ich habe gesehen das du deutsch kannst also können wir ja auch auf deutsch schreiben. Und es geht nicht auch wenn man keine der Server Gruppen besitzt. Wenn du möchtest kannst du dir das gerne mal ansehen.

Shad86 commented 7 years ago

Aktuell ist da wohl tatsächlich ein Bug - Wenn die Benachrichtigungen ausgeschaltet sind, funktioniert der Assigner für die Vergabe der Gruppen nicht.

Lösung:

Benachrichtigung anschalten.

Currently there is a bug, if "Rank up notification" (under Message) is disabled you can't give servergroups about the assigner.

Solution:

Enable "Rank up notifications"

mamayer commented 7 years ago

Ich habe das Problem auch, es scheint an folgenden Zeilen zu liegen:

<?PHP if(in_array($allowed_group, $cld_groups)) {
    echo '<input id="switch-animate" type="checkbox" checked data-size="mini" name="',$allowed_group,'" value="',$msgtouser,'">';
} else {
    echo '<input id="switch-animate" type="checkbox" data-size="mini" name="',$allowed_group,'" value="',$msgtouser,'">';
} ?>

ich bin noch nicht lange im Code des Systems unterwegs, aber mir erschließt sich nicht, warum der value $msgtouser sei sollte und nicht einfach "1". Vermutlich ein Copy'n'paste-Fehler des Buttons aus msg.php.

Korrekt wäre demnach:

<?PHP if(in_array($allowed_group, $cld_groups)) {
    echo '<input id="switch-animate" type="checkbox" checked data-size="mini" name="',$allowed_group,'" value="1">';
} else {
    echo '<input id="switch-animate" type="checkbox" data-size="mini" name="',$allowed_group,'" value="1">';
} ?>

Ich mache mal einen Pullrequest dazu...

Shad86 commented 7 years ago

Nicht notwendig, dennoch danke

  • fixed problem on assigning servergroup if rank up notification is disabled

-> https://ts-n.net/ranksystem.php#changelog

mamayer commented 7 years ago

Ah Danke, da ist auch mein anderes Problem gelöst... "/icons" - musste erstmal die Apache-Config anpassen und den Alias rauslöschen -.-

Shad86 commented 7 years ago

Ja apache managed das ganze recht seltsam, weil für gewöhnlich Leute ohne subdomains nicht in dieses problem laufen, da apache wohl bei subdomains die Ordner anders verwaltet...

Newcomer1989 commented 6 years ago

Will be fixed with the next official release 1.2.3

Newcomer1989 commented 6 years ago

Release 1.2.3 is published. The problem should be fixed. Please reopen this issue, if you should still have problems!