Skip to content
This repository has been archived by the owner on Feb 11, 2018. It is now read-only.

Update players.php #68

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Update players.php #68

wants to merge 1 commit into from

Conversation

ghost
Copy link

@ghost ghost commented Nov 14, 2016

Fixing the error from it trying to select playerid when the new table is pid.

Fixing the error from it trying to select `playerid` when the new table is `pid`.
@BoGuu
Copy link
Contributor

BoGuu commented Nov 15, 2016

See the relevant issue

@BoGuu BoGuu closed this Nov 15, 2016
@MightySCollins
Copy link
Member

Thanks for this PR, it seems this is needed for the newer versions of the database and if BooGuu is ok with this I would like to merge it in to fix #70.

@MightySCollins MightySCollins reopened this Dec 9, 2016
@Jason2605
Copy link
Contributor

This will only work for version 5, meaning version 4.4r4 below will break.

@MightySCollins
Copy link
Member

Yep so I am thinking feature switch or branch/release.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants