search

tgstation / tgstation13.org

Website for tgstation13
GNU General Public License v2.0
6 stars 44 forks source link

Tries to replace lastadminrank #40

Closed Jordie0608 closed 6 years ago

Jordie0608 commented 6 years ago

So it can be later be removed altogether. I don't know php to know if this is valid code but this is mostly copy-paste-edited so it's probably fine. Not too sure about the RIGHT JOIN, I'll have to test it out when I get home.

MrStonedOne commented 6 years ago

You would want to just left join the admin table into the player table. This would let you keep the existing sql format, and just add a new column that polls from the admin tables to replace lastadminrank

Jordie0608 commented 6 years ago

The right join does seem to work as intended (sans two typos), would you just rather a column for rank to filter with instead of constricting the result set?