单击提交无法更新用户信息

Question

Everytime an user register in my website, they have a "free account". In my website there're 4 types of accounts:

Free - 2GB

Basic - 5GB

Ultra - 10GB

Plus - 25GB

Each one with an unique id. I have created this code. Depending of the submit you click, your user get one of those accounts (Basic Ultra or Plus).

    public static function ComprobarTarifa($user){
        $sql = "SELECT id_tipocuenta FROM canal WHERE id = '$user'";
        $resultado = self::Conexion($sql);
        $verificacion = false;

        if(isset($resultado)) {
            $fila = $resultado->fetch();
            if($fila !== false){
                $verificacion=true;
            }
        }
        return $verificacion;
    }

    public static function AmpliarCuenta($user, $tarifa){
        $sql = "UPDATE canal SET id_tipocuenta = '$tarifa' WHERE id = '$user'";
        $resultado = self::Conexion($sql);
        return $resultado;
    }

    public static function AmpliarCuentaErrores(){
        $error = "";
        $tarifa = 0;

        if(isset($_POST["tarifaBasic"])){
            $tarifa = 2;

            if(isset($_SESSION["usuario"])){
                if(self::ComprobarTarifa($_SESSION["usuario"]) != $tarifa){
                    DataBase::AmpliarCuenta($_SESSION["usuario"], $tarifa);
                    $error = "<div id='error_alert'><div class='error_container'><i class='fa fa-times awesome error_close btn'></i><div class='error_text'>Ya estás usando esa tarifa...</div></div></div>";
                }
            } else {
                $error = "<div id='error_alert'><div class='error_container'><i class='fa fa-times awesome error_close btn'></i><div class='error_text'>Debes iniciar sesión primero...</div></div></div>";
            }
        }

        if(isset($_POST["tarifaUltra"])){
            $tarifa = 3;

            if(isset($_SESSION["usuario"])){
                if(self::ComprobarTarifa($_SESSION["usuario"]) != $tarifa){
                    DataBase::AmpliarCuenta($_SESSION["usuario"], $tarifa);
                    $error = "<div id='error_alert'><div class='error_container'><i class='fa fa-times awesome error_close btn'></i><div class='error_text'>Ya estás usando esa tarifa...</div></div></div>";
                }
            } else {
                $error = "<div id='error_alert'><div class='error_container'><i class='fa fa-times awesome error_close btn'></i><div class='error_text'>Debes iniciar sesión primero...</div></div></div>";
            }
        }

        if(isset($_POST["tarifaPlus"])){
            $tarifa = 4;

            if(isset($_SESSION["usuario"])){
                if(self::ComprobarTarifa($_SESSION["usuario"]) != $tarifa){
                    DataBase::AmpliarCuenta($_SESSION["usuario"], $tarifa);
                    $error = "<div id='error_alert'><div class='error_container'><i class='fa fa-times awesome error_close btn'></i><div class='error_text'>Ya estás usando esa tarifa...</div></div></div>";
                }
            } else {
                $error = "<div id='error_alert'><div class='error_container'><i class='fa fa-times awesome error_close btn'></i><div class='error_text'>Debes iniciar sesión primero...</div></div></div>";
            }
        }
        return $error;
    }

The problem is that my database doesnt update... What could it be? Error divs dont appear...

Answer 1

You're printing the error message even if everything is OK. You forgot the else at all the instances here:

if(self::ComprobarTarifa($_SESSION["usuario"]) != $tarifa){
    DataBase::AmpliarCuenta($_SESSION["usuario"], $tarifa);
    $error = "<div id='error_alert'><div class='error_container'><i class='fa fa-times awesome error_close btn'></i><div class='error_text'>Ya estás usando esa tarifa...</div></div></div>";
}

That should actually be:

if(self::ComprobarTarifa($_SESSION["usuario"]) != $tarifa){
    DataBase::AmpliarCuenta($_SESSION["usuario"], $tarifa);
} else {
    $error = "<div id='error_alert'><div class='error_container'><i class='fa fa-times awesome error_close btn'></i><div class='error_text'>Ya estás usando esa tarifa...</div></div></div>";
}