A3Armory / A3A_A3Wasteland.Altis

A3Wasteland is a survival sandbox mission for Arma 3 where two teams and independent players fight for survival. Built in collaboration between GoT, TPG, KoS, 404Games, and others.
http://a3wasteland.com
GNU Affero General Public License v3.0
9 stars 6 forks source link

Admin log not working on low and high admins #13

Closed Gigatek1 closed 8 years ago

Gigatek1 commented 8 years ago

For some reason the admin panel is only logging serverOwners use of the admin panel. Not sure what broke it but it was probably this commit https://github.com/A3Armory/A3A_A3Wasteland.Altis/commit/7a72d1f4e6326263490543009958e63b7203bc6f

When I commented out this line, logging worked. Any thoughts on if this was the right fix or will it screw up anything else? It looks like isAdmin function is broken for lowAdmins and highAdmins but I'm at a loss.

Gigatek1 commented 8 years ago

Fixed it. Still don't get the need for the 3 in this line but oh well. https://github.com/A3Armory/A3A_A3Wasteland.Altis/commit/73f251d1a1bc41a3f79716f07b1cbc15c02ae756

pAxton- commented 8 years ago

You are defaulting it's case selection in this switch statement in the isAdmin.sqf On Thu, Sep 15, 2016 at 4:00 PM, Gigatek1 notifications@github.com wrote:

Fixed it. Still don't get the need for the 3 in this line https://github.com/A3Armory/A3A_A3Wasteland.Altis/commit/73f251d1a1bc41a3f79716f07b1cbc15c02ae756#diff-1c3d38cc2a26bd4d6432a11e808cfb77L11 but oh well. 73f251d https://github.com/A3Armory/A3A_A3Wasteland.Altis/commit/73f251d1a1bc41a3f79716f07b1cbc15c02ae756

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/A3Armory/A3A_A3Wasteland.Altis/issues/13#issuecomment-247436784, or mute the thread https://github.com/notifications/unsubscribe-auth/AGFZ5Yn3bIZnEaUCji4qAbX3ZFxNbCzKks5qqaPugaJpZM4J8J5b .

AgentRev commented 8 years ago

the 3 was not needed. your fix is invalid and will cause the function to fail.

Gigatek1 commented 8 years ago

Thanks for the info guys. I don't believe I had logging off for low and high admins for 3 months. I thought they just weren't using it too much... lol