When USER_LIMIT == 0 (no limit) $UserCount is always > USER_LIMIT so
the notice is always shown. Actually this commit just implements the
exact opposite of the logic on line 137.
When USER_LIMIT == 0 (no limit) $UserCount is always > USER_LIMIT so
the notice is always shown. Actually this commit just implements the
exact opposite of the logic on line 137.
If you make that change I’ll be happy to merge it.
I think it would be clearer as
```($UserCount >= USER_LIMIT && USER_LIMIT != 0 && !check_perms('site_can_invite_always'))```
instead of
```(!($UserCount <= USER_LIMIT || USER_LIMIT == 0 || check_perms('site_can_invite_always')))```.
If you make that change I'll be happy to merge it.
When USER_LIMIT == 0 (no limit) $UserCount is always > USER_LIMIT so the notice is always shown. Actually this commit just implements the exact opposite of the logic on line 137.
I think it would be clearer as
($UserCount >= USER_LIMIT && USER_LIMIT != 0 && !check_perms('site_can_invite_always'))
instead of
(!($UserCount <= USER_LIMIT || USER_LIMIT == 0 || check_perms('site_can_invite_always')))
.If you make that change I’ll be happy to merge it.
OK, I have pushed a new commit with those changes.