Problema al ingresar datos en base de datos

[NedFock]

Hola amigos de FDW!! estoy modificando un código para subir archivos, que lo saqué de este foro hace tiempo y actualmente lo uso para subir imágenes, pero ahora que lo quiero para subir mp3 me está dando problemas

El código en cuestión es este:

Código PHP:

  include ('../acceso_db.php');
session_start();
 
$nombre = $_POST['nombre'];
if(isset($_POST['nombrealbum'])){
    $album = $_POST['nombrealbum'];
}else{
    if(isset($_POST['album'])){
    $album = $_POST['album'];
    }else{
    $album = "general";
     }
    
if(is_uploaded_file ($_FILES [ 'file' ][ 'tmp_name' ])){
 if($_POST){ 
// Creamos la cadena aletoria 
$str = "ABCDEFGHIJKLMNOPQRSTUVWXYZabcdefghijklmnopqrstuvwxyz1234567890"; 
$cad = ""; 
for($i=0;$i<12;$i++) { 
$cad .= substr($str,rand(0,62),1); 
} 
// Fin de la creacion de la cadena aletoria 
$tamano = $_FILES [ 'file' ][ 'size' ]; // Leemos el tamaño del fichero 
$tamaño_max="100000000000"; // Tamaño maximo permitido 
if( $tamano < $tamaño_max){ // Comprobamos el tamaño  
$ruta = "{$_SERVER['DOCUMENT_ROOT']}/canciones/{$_SESSION['usuario_id']}/{$album}/";
 
$ruta = "{$_SERVER['DOCUMENT_ROOT']}/canciones/{$_SESSION['usuario_id']}";
$perfil = '/{$album}';
if(!file_exists($ruta))
{
$oldumask = umask(0); 
mkdir ($ruta);
mkdir ($ruta . $perfil);
umask($oldumask); 
}
$destino = $ruta . $perfil; // Carpeta donde se guardara 
$sep=explode('image/',$_FILES["file"]["type"]); // Separamos image/ 
$tipo=$sep[1]; // Optenemos el tipo de imagen que es 
if($tipo == "mp3" || $tipo == "wav" || $tipo == "wma"){ // Si el tipo de archivo a subir es el mismo de los permitidos, segimos. Puedes agregar mas tipos de formatos
move_uploaded_file ( $_FILES [ 'file' ][ 'tmp_name' ], $destino . '/' .$cad.'.'.$tipo);  // Subimos el archivo 
$link = "{$_SERVER['DOCUMENT_ROOT']}/canciones/{$_SESSION['usuario_id']}/{$album}/".$cad.'.'.$tipo;
} 
else echo "el tipo de archivo no es de los permitidos";// Si no es el tipo permitido lo desimos 
} 
else echo "El archivo supera el peso permitido.";// Si supera el tamaño de permitido lo desimos 
} 
 
 
 
mysql_select_db("crunch");
//Creamos la sentencia SQL y la ejecutamos
$sql = mysql_query ("INSERT INTO canciones (nombre, album, link, usuario_id) VALUES ('$nombre', '$album', '$link', {$_SESSION['usuario_id']}") or die(mysql_error());
mysql_query($sql);
header("Location: /perfil.php?id=" . $_SESSION['usuario_id'] . "&modificado=ok");
}
} 
   

Perdón las faltas de ortografía en los comentarios, no fui yo quién lo escribio :D

La cuestión es que no me crea el registro en la base de datos, ni me redirige a perfil.php ... supongo tendría qe ser por estar mal armado los bloques, que el registro esté fuera de la condición, pero no, no encontré el problema! :S espero me puedan ayudar, y desde ya, muchas gracias por haber leído :D

[xIamAlex]

Dios, a ver si se acostumbra a indentar bien su código

Lo que no entiendo es el "if($_POST)", digo ¿Qué se quiere hacer con eso? porque $_POST es un Array y no una variable booleana, ¿no sería más bien "if(sizeof($_POST) > 0)"?

[NedFock]

xlamALex, lo que pasa es qe aprendí por internet y nunca vi nada de indentar, cosa que siempre quise aprender pero se me ha pasado. El código lo tenía mejor creo yo, pero lo pasé por beautify php y fue peor jaja.

lo del if($_POST) ni yo lo entendí, pero fue algo qe vino ya con el código del uploader y bueno, como el que lo hizo sabe más que yo, preferí dejarlo así.

Voy a ver lo de indentar :reloco: Pero aun así, sigo con el problema de este código xD :_ Gracias por tu respuesta, de todas formas

[luisalberti]

Cual es el problema que te da¿?

[luisalberti]

Iniciado por luisalberti

Cual es el problema que te da¿?

Algo con el tamaño recuarda que en la configuración de apache es donde tienes que indicar el tamaño.

$tamaño_max="100000000000"; // Tamaño maximo permitido

[NedFock]

Hola luisalberti, gracias por responder!! :D
El problema que tengo es que el código... no sube el archivo ni guarda los datos en la base de datos, directamente! jaja u.u y no logro enteder qe está mal, si es el mismo código que subo para las fotos. Será la parte de

Código PHP:

  $sep=explode('image/',$_FILES["file"]["type"]); // Separamos image/ 
   

Que al estar ahora subiendo algo que no es imagen, no me deja porque no separa "image/"? Pero que es lo qe tendría qe separar ahora al subir audio. Esa parte no entendí mucho.


Bien, voy a indicarlo en apache, gracias por el dato!!

[gldelossantos]

Porque esta ruta asi

$ruta = "{$_SERVER['DOCUMENT_ROOT']}/canciones/{$_SESSION['usuario_id']}/{$album}/";

Que quieres hacer con esto?

[NedFock]

la idea es que en caso que no exista la carpeta, se cree una una carpeta canciones/usuario/album y que luego se suba la canción ahí

[xIamAlex]

Si quieres saber si una ruta existe o no, estás cometiendo un error aquí:

Código PHP:

  if(!file_exists($ruta)) 
{ 
$oldumask = umask(0);  
mkdir ($ruta); 
mkdir ($ruta . $perfil); 
umask($oldumask);  
} 
   

Ya que deberá ser:

Código PHP:

  if(!is_dir($ruta)) 
{ 
$oldumask = umask(0);  
mkdir ($ruta); 
mkdir ($ruta . $perfil); 
umask($oldumask);  
} 
   

La función file_exists se utiliza para comprobar si un fichero existe, la función is_dir es la que se utiliza para comprobar si un directorio es el que existe, suerte