e107inc/e107

Avoid too many return statements within this method.

Open

            return 'ip';

Avoid too many return statements within this method.

Open

        if (strpos($ip, 'fc00:') === 0) return FALSE;                            // local addresses

Avoid too many return statements within this method.

Open

        return 'unknown';

Avoid too many return statements within this method.

Open

        return TRUE;

Avoid too many return statements within this method.

Open

        if ($ip == '0000:0000:0000:0000:0000:0000:0000:0001') return FALSE;

Argument #2 of this call to \strpos is typically a literal or constant but isn't, but argument #1 (which is typically a variable) is a literal or constant. The arguments may be in the wrong order.

Open

        if (strpos(':', $ip) === FALSE) return TRUE;

• Exclude checks
  • Disable engine
  • Disable check

Saw possibly unextractable annotation for a fragment of comment '* @param string(?) $perms - required permissions as standard *nix 3-digit string': after string, did not see an element name (will guess based on comment order)

Open

     *    @param string(?) $perms - required permissions as standard *nix 3-digit string

• Exclude checks
  • Disable engine
  • Disable check

Argument 1 (string) is false but \strlen() takes string

Open

        while (($togo = (39 - strlen($address))) > 0)

• Exclude checks
  • Disable engine
  • Disable check

Saw a token Phan may have failed to parse after '* @return string. '0' if date is zero, else formatted in consistent way.': after string, saw '.'

Open

     *    @return string. '0' if date is zero, else formatted in consistent way.

• Exclude checks
  • Disable engine
  • Disable check

Variable $ret was undeclared, but array fields are being added to it.

Open

        $ret['banDate'] = $temp[0];

• Exclude checks
  • Disable engine
  • Disable check

Variable $pref was undeclared, but array fields are being added to it.

Open

        $pref['ban_durations'] = e107::getPref('ban_durations');

• Exclude checks
  • Disable engine
  • Disable check

Returning type false but ipDecode() is declared to return string

Open

        if (strpos($ip, ':') === false) return FALSE;            // Return on problem - no ':'!

• Exclude checks
  • Disable engine
  • Disable check

Variable $pref was undeclared, but array fields are being added to it.

Open

        $pref['ban_messages'] = e107::getPref('ban_messages');

• Exclude checks
  • Disable engine
  • Disable check

Argument #2 of this call to \strrpos is typically a literal or constant but isn't, but argument #1 (which is typically a variable) is a literal or constant. The arguments may be in the wrong order.

Open

        if (($tv = strrpos('@', $email)) !== FALSE)

• Exclude checks
  • Disable engine
  • Disable check

Saw a token Phan may have failed to parse after '* @param string(?) $perms - required permissions as standard *nix 3-digit string': after string, saw '('

Open

     *    @param string(?) $perms - required permissions as standard *nix 3-digit string

• Exclude checks
  • Disable engine
  • Disable check