Remove Max Rank/Token Checks

[Camer the Dragon]

I do not see why they are there still (although there might be a reason I'm missing)
if a hacker hacked themselves a high rank or lots of tokens surely it'd be pretty obvious

https://github.com/jacob-grahn/platform-.../login.php

The code referred to in lines 184 and 200 is what I'm talking about
(The code referred to in line 194 makes sense to me still though, in case there was a weird bug out where more tokens got used than were there or something)

EDIT: if you wanted a check still, you could soft check and then record the Name and IP if it failed the check

EDIT 2: ok this sounded kinda mean sorry I only meant as an idea

[Ashley766]

I do agree with the first line. Removing the max rank would make it possible for people to have "happy hour" stats and maybe if someone reaches a rank like 80, the exp calculator could decrease just a tab to make it easier to get to a higher rank than intended.

[Camer the Dragon]

I do agree with the first line. Removing the max rank would make it possible for people to have "happy hour" stats and maybe if someone reaches a rank like 80, the exp calculator could decrease just a tab to make it easier to get to a higher rank than intended.

that'd create too much of a gap imo, separating the most hardcore macroers from everyone else
there's enough of a gap already

[Eternal]

what

[Camer the Dragon]

what

There's a server check that doesn't let you on if you're over rank 100 which is a bit weird
also one for more than 5 permanent tokens

[ExceedZero]

also what

[Ashley766]

also one for more than 5 permanent tokens

I think there was an exploit that caused that check but ?‍♀️

[Camer the Dragon]

also what

Just say here what you did on discord where you actually said something about it lol

also E check the JV2 discord and scroll up we talk about it there

[Eternal]

There's a server check that doesn't let you on if you're over rank 100 which is a bit weird
also one for more than 5 permanent tokens

Why does the first part matter and the second part isn’t true.

[Camer the Dragon]

Why does the first part matter and the second part isn’t true.

1) Yeah I don't think it really matters and I do not know why it's there (but maybe it is a good check just in case anything did happen?)
2) This is 2nd part

Code:

// sanity check: do they have more than 5 permanent rank tokens?
    if ($rt_available > 5) {
        throw new Exception("Too many rank tokens. Please use a different account.");
    }

[Eternal]

Ah, the rented token check was removed. Again though, I don’t see why removing this is necessary at all. Just let it be.

[Stxtics]

Idk if you code but these things are more just there just in case it happens even though it's very unlikely they will.

Rank 100 is pretty much impossible unless someone buys a lot of rank tokens.

The rank token check is if someone somehow gets more than 5 permanent rank tokens, it probably won't happen but it's there just in case it does.

There is no harm in having them there, it doesn't make pr2 slower or anything like that so they are just there for security

[Ashley766]

Rank 100 is pretty much impossible unless someone buys a lot of rank tokens.

The rented rank token check got removed but I don't see a problem getting to rank 100 or over if it's for a short amount of time if someone rents a bunch of rank tokens. Getting to rank 100 alone would probably still be impossible.

[Stxtics]

The rented rank token check got removed but I don't see a problem getting to rank 100 or over if it's for a short amount of time if someone rents a bunch of rank tokens. Getting to rank 100 alone would probably still be impossible.

Unless there is a sale nobody is going to spend enough money to get 25 rank tokens or more. 

At the moment 25 rank tokens would cost 7250 kreds, and 1150 kreds is $120.

[Eternal]

When/if the issue ever arises where we need to remove the checks, we will, like what happened with the rented rank tokens. Until then, I'm not in favor of removing them since they're serving their purpose accordingly.

[bls1999]

As this is a matter of security and account integrity verification, I will not be removing this code. Sorry.