e107inc/e107

Avoid too many return statements within this method.

Open

        return AUTH_SUCCESS;

Saw an @param annotation for pass, but it was not found in the param list of function login(string $uname, $pword, \pointer &$newvals, bool|false $connect_only = null) : int

Open

     *    @param string $pass - Password to use (usually plain text)

• Exclude checks
  • Disable engine
  • Disable check

Reference to undeclared constant \AUTH_BADPASSWORD

Open

            return AUTH_BADPASSWORD;

• Exclude checks
  • Disable engine
  • Disable check

Reference to undeclared constant \AUTH_SUCCESS

Open

        return AUTH_SUCCESS;

• Exclude checks
  • Disable engine
  • Disable check

Reference to undeclared constant \AUTH_SUCCESS

Open

        if ($connect_only) return AUTH_SUCCESS;            // Big problem if can't connect to our own DB!

• Exclude checks
  • Disable engine
  • Disable check

Reference to undeclared constant \AUTH_NOCONNECT

Open

            return AUTH_NOCONNECT;            // Debateable return code - really a DB error. But consistent with other handler

• Exclude checks
  • Disable engine
  • Disable check

Class \auth_login defined at /code/e107_plugins/alt_auth/importdb_auth.php:33 was previously defined as Class \auth_login at /code/e107_plugins/alt_auth/e107db_auth.php:34

Open

class auth_login extends alt_auth_base

• Exclude checks
  • Disable engine
  • Disable check

Parameter $newvals has undeclared type \pointer

Open

    public function login($uname, $pword, &$newvals, $connect_only = FALSE)

• Exclude checks
  • Disable engine
  • Disable check

Reference to undeclared constant \AUTH_NOUSER

Open

            return AUTH_NOUSER;

• Exclude checks
  • Disable engine
  • Disable check