MP3THEQUE

Commentaire de xque19 le 07/06/2006 15:59:45

J'ai passé pas mal de temps à faire cette petite application et je ne compte pas m'arrêter en si bon chemin, c'est pour çà que j'ai décidé de la publier histoire d'avoir des avis divers et varier pour m'aider à l'améliorer, alors n'hésitez svp à laisser des commentaires.

Mehdi

Commentaire de Eliays le 07/06/2006 20:01:05

il demande un login&mdp .. mais comment le connaitre? aucune indication :/

Commentaire de xque19 le 08/06/2006 09:56:49

Désolé, j'ai pas pensé à mettre le login et le mot de passe. je viens de mettre à jour les Explications, j'y ai ajouté ces infos.

Commentaire de J_G le 08/06/2006 11:48:02

Salut,


Tu souhaite des commentaires...? Alors feu !

Je test ta source sur :
PHP => Version 5.0.4
Server API => Apache 2.0 Handler
System => Linux localhost 2.6.12-12mdk #1 Fri Sep 9 17:26:56 CEST 2005 x86_64

Avec... entre autres :
register_globals Off (BIEN  SUR)
safe_mode Off (sinon j'ai pas acces à ma musique)
magic_quotes_gpc Off
error_reporting 2047 (E_ALL)
...


Et ben c'est pas joli-joli...

Notice: Undefined variable: skin in /var/www/html/test/mp3theque/index2.php on line 35
Template Error: filename: file skins/main_template.tpl does not exist.
Halted.

Notice: Undefined variable: nb in /var/www/html/test/mp3theque/index2.php on line 67

Notice: Undefined variable: nb in /var/www/html/test/mp3theque/index2.php on line 77

Notice: Undefined variable: login in /var/www/html/test/mp3theque/index2.php on line 109

Notice: Undefined variable: type_user in /var/www/html/test/mp3theque/index2.php on line 117

<br />
<b>Notice</b>:  Undefined variable: login in <b>/var/www/html/test/mp3theque/play.php</b> on line <b>37</b><br />
<br />
<b>Notice</b>:  Undefined variable: file in <b>/var/www/html/test/mp3theque/include/class.id3.inc</b> on line <b>177</b><br />
<br />

Tu n'initialise quasiment jamais tes variable... Mets un error_reporting(E_ALL); en haut de tes sripts, et recommence tout...



<br />
<b>Warning</b>:  fopen(Deftones\White Pony\Deftones - White Pony - 04 - Rx Queen.mp3) : failed to open stream: No such file or directory in <b>/.../test/mp3theque/include/class.id3.inc</b> on line <b>176</b><br />

BIIIP... au lieu d'utiliser '/' ou '\' comme séparateur de dossier, il faut utiliser DIRECTORY_SEPARATOR. Car, dans le cas Deftones présenté, j'imagine que tu recomposes le chemin avec implode('\\',$array_path); Or, manque de bol, mon serveur est sous Linux (ce qui est courrant pour un serveur). Donc pour lui, \ n'est pas le séparateur de dossier.



Il serait bienvenu de trier le contenu des répertoires de musique


Sinon, tu utilises les fonctions désuées session_register et ses soeurs (unregister, is_registered, ...). isset($_SESSION['']) $_SESSION['salut'] = 'hop'; etc serait mieux


Mais la réalisation est plutôt pas mal... Bonne continuation !

Commentaire de xque19 le 08/06/2006 12:06:31

Désolé J_G, je n'ai pas très bien saisi ton dernier msg, si j'ai bien compris en utilisant $_SESSION on a pas besoin de mettre  register_globals sur on?

Sinon comme je le dis plus haut je n'ai pas encore testé sous linux, je l'ai testé sous easyphp seulement donc je ne connais pas toutes les erreurs qui remontent sous des systèmes autres que windobe. mais en tout cas, je prend note de tes remarques et je ferai les changements necessaires.

Mehdi

Commentaire de xque19 le 08/06/2006 13:19:14

Voili voilà! c'est fait, j'ai tout mis avec $_SESSION, c'est vrai que c'est plus propre.

Merci bien J_G!!

Commentaire de xque19 le 08/06/2006 14:45:06

j'avais pas fait gaffe que sur quand on envoie un zip sur le site, tous les dossiers vides sont supprimé, du coup j'ai fait une petite vérif au début qui recrée tous les dossiers inexistants. (dossier LOG, dossier temporaire, ...)

Commentaire de xque19 le 08/06/2006 16:36:49

C'est tout à fait normal que le contenu du fichier m3u soit comme çà.
sinon je pense que çà doit venir du fait que dans "config.ini", il y a juste le nom du dossier, en faisant les tests tout à l'heure, je me suis rendu compte que çà ne marche que qd on met le chemin absolu (c:\...\mp3 par ex). sinon g corrigé pour firefox çà n'envoie plus un fichier play.php mais un fichier du type "stream_44882d1e2e94b.m3u" qui sert temporairement à lancer le streaming dans winamp ou wmp ou xmms.

Commentaire de galadriann le 08/06/2006 16:57:24

Je confirme ... ca marche avec firefox ...
en mettant le chemin absolu sur le serveur ca fonctionne nickel ...

DONC ...   grand bravo pour cette source ...

(on va matter un peu le code maintenant ... ;-) )

je met donc 9/10

Commentaire de johann1 le 08/06/2006 17:26:54

Je vais scruter tout ça ! Ca m'a l'air bien sympa !

Commentaire de xque19 le 08/06/2006 19:38:07

j'ai testé une fois pour voir si çà marche, mais çà n'a pas l'air de fonctionner, je crois que çà vient du serveur qui bloque les fonctions sur les fichiers, mais j'ai pas poussé mes recherches plus loin.
Si quelqu'un a une idée là dessus, je suis preneur. :)

Commentaire de J_G le 09/06/2006 13:45:49

Salut,

ton appli est intéressante. Alors je fais un deuxième passage pour voir comment ça évolue. Je te fais mes commentaires en cours de routes :

Bon, je commence par changer tous les '<?' en '<?php'.
Et oui : short_tag = Off (Pour ne pas analyser le <? des spécifications XML)

Affichage du contenu d'un répertoire
Notice: Uninitialized string offset: 10 in /var/www/html/test/mp3theque/include/class.id3.inc on line 369
La ligne 369 de clas.id3.inc correspond à la condition d'un do{}while();
Le while test les caractère 8,9,10 d'une chaine nommée $bits sans en vérifier la longueure.
Ce problème est résolu en remplaçant le test par :
(strlen($bits)<=10) or (!$bits[8] and !$bits[9] and !$bits[10])
=> on ne prend pas la peine de tester les carac 8 , 9 ou 10 si la chaine ne fait pas au moins 11 caractères de long !

dans play.php
Warning: fopen() [function.fopen.html]: Unable to access Raspigaous\Raspigaous live - On ne peut pas tous_.mp3 in /var/www/html/test/mp3theque/include/class.id3.inc on line 176
L'erreur viend en fait de ta fonction create_m3u(), ligne 138
Un petit oubli de DIRECTORY_SEPARATOR
Modification de la ligne (j'ai pas cherché à comprendre le pourquoi le str_replace) :
$id3 = new id3(str_replace("/",DIRECTORY_SEPARATOR,base64_decode($lst[$i])), true);


Mais du coup :
Warning: unpack() [function.unpack.html]: Type H: outside of string in /var/www/html/test/mp3theque/include/class.id3.inc on line 187
Bon, là : @unpack.... (comme plus bas dans la classe id3)

J'y suis presque... j'arrrive dans stream.php
Warning: fopen(log/log_du_09_06_2006.log) [function.fopen.html]: failed to open stream: No such file or directory in /var/www/html/test/mp3theque/include/fonctions.inc on line 124
index.php ne m'avais pas créé le répertoire 'log'... Tu devrais prévoir une fonction remplaçant fopen est qui vérifie la présence du fichier, créer les répertoir automatiquement, etc...


ENFIN DE LA MUSIQUE !!!
(C'est pas top Raspigaous... un peu facile comme reagge)

Trois comportements étranges :
* J'aime pas qu'on me désactive mon menu contextuel (mais pas du tout !)
* Quand j'ajoute un fichier à la liste via le petit '+', cela efface la liste avant d'ajouter, (le fichier reste donc seul)
* pourquoi ne se passe t-il rien quand je clic sur le nom du fichier ?



Bon bah c'est très bien tout ça...
Il reste encore un bug chez moi : la gestion des listes de lectures sauver/charger/ajouter... (la lecture fonctionne).

Mais c'est super, bravo !

Commentaire de xque19 le 09/06/2006 14:07:41

Bonjour,

Je vais essayer de faire de mon mieux pour réparer toutes ces erreurs, sinon pour la gestion de la playliste, peut-être qu'il te manque le rep "listes", j'avais ajouté une fontions qui les crée tous les dossiers nécessaires mais sous linux çà marche pas si apache n'a pas les droits d'écriture sur le repertoire racine (en général 'public_html'). Sinon concernant les nombreux fichiers dans m3u et temp, il y a une fonction qui est executée à chaque fois pour vérifier et supprimer tous les fichiers de plus de 24h.

Commentaire de xque19 le 09/06/2006 16:59:31

Hola!

J'ai bossé un peu dessus cet après-midi, et j'ai donc pu mettre en ligne il y a quelques instants une versions avec moins de bugs (enfin je l'espère).
(merci pour tes nombreuses remarques constructive J_G et à tous les autres biensûr).

^_^

Commentaire de J_G le 09/06/2006 21:04:06

re-salut,

J'ai vu les fonctions verif_* C'est une bonne démarche...
Un petit "if( isset($_GET['msg']) ) $msg = $_GET['msg'];" en début de index.php pour register_global=Off


Bon, bon, bon... Passons aux choses sérieuses !!!

Vas-tu mettre ton site en ligne ?  Hum-hum et, les droits d'auteur ?

J'avais à l'époque l'idée de me faire un petit site serveur de musique, basé sur ce que tu as fait. Avec en plus un utilitaire de soumission de mp3. J'ai même effleuré l'idée de suivre les statistique d'écoute des morceaux. Le but étant de distribuer des droits d'auteur en fonction de l'audience de tel ou tel artiste. Les fonds nécessaires étant collecté grâce à un système d'abonnement ou mieux, de publicité... Sachant que les artiste ne touchent guère mieux que 0,5€ par CD vendu en magasin, il est facile d'arriver à un tel taux de rémunération.

Ce principe est utopique... Car brutalement illégal !

Bref, tout ça pour dire que si jamais tu mets un tel site en ligne, même pour usage privé, c'est un peu comme donner un baton pour ce faire battre. Et si jamais tu donnes des accès à tes copain, méfience... Le danger viend de ton hébergeur ! A moins bien sûr que tu es acheté les droits d'écoute de tout les morceaux MP3 que tu disposeras sur ton site...

A+

Commentaire de xque19 le 10/06/2006 03:39:17

c'est justement l'utilisation que j'en fait actuellement, c'est hebergé directement chez moi, et du coup j'ai accès à mes mp3 en écoute depuis n'importe quel pc qui a internet.
A la base je l'ai fait parce que j'ai testé une appli dans le même genre (Kplaylist) mais le souci c'est que çà necessite à chaque modification (ajout ou suppression de musiques, de rescanner le repertoire musical, vu que c'est basé sur une base mysql) du coup, je préférais plutôt quelque chose de plus réactif qui s'adapte automatiquement, d'où cette application sans bases de données.

Sinon dans le principe je pense que cette appli n'a rien d'illégal, après çà dépend de l'utilisation que chacun en fait.

Personellement je l'utilise uniquement pour pouvoir écouter mes musiques au boulot ou chez des amis, et c'est justement pour çà qu'il y a un système d'autentification, pour éviter que n'importe qui y accède,  même par accident.

A part çà, je n'y ai pas trop refléchi mais çà peut peut-être avoir d'autres utilités.

Commentaire de J_G le 10/06/2006 13:36:21

Salut,

pour le $_GET['msg'] :
Quand tu essaye d'accéder à index2.php sans être identifié, la page est renvoyé sur index.php?msg=Connexion%20obligatoire... Voilà pourquoi.

Connaissez-vous le site last.fm ?

C'est du pod-casting de musique... On selectionne une playlist soit par le style de musique ou par un artiste (joue une série d'artistes au style similaires). Le principe est intéressant, même si comme pour toute les radio, on fini par être saoulé.
(Il faut installer une petite appli pour lire le streaming, ne vous inquiétez pas, elle est très légère et ne va pas se coller partout dans la base de registre etc...)

A+

Commentaire de Dav57 le 14/06/2006 00:12:28

Bravo pour l'appli, c'est bien réalisé graphiquement je trouve. J'ai de bonnes connaissances en php mais pas assez solide pour analyser tout ton code.

Je voulais réagir sur l'aspect droits d'auteurs.
Pour une version publique, vous pouvez aussi choisir de diffuser de la musique libre c'est légal et encouragé !
Il faut se rendre sur des sites comme www.jamendo.com (plus de 1000 albums) ou musique-libre.org... et plein d'autres (ceux-là sont les 2 meilleurs francophones).

La musique est gérée par des contrats CC et donc en général, tant que tu ne fais pas une utilisation commerciale de ton interface et que tu cites l'auteur, c'est parfait :)

à bientôt

Commentaire de J_G le 15/06/2006 12:36:26

DAV57 :

Merci de m'avoir fait découvrir www.jamendo.com !

AAAAAH mais quel pied de trouver de la musique "indépendante", c'est à dire pas de la soupe commerciale. Avec un petit lien "écouter" qui balance du (faux) streaming en plutôt bonne qualité (128Kb). Et un autre - oh magie - "télécharger" avec à la clef du ogg-q8 à 300Kb dans mon client P2P !

Et le fin du fin : ce petit fichier license.txt qui accompagne tout mes téléchargements... Mon dieu quel pied!


Voilà, fallait que je le dise !
A+

Commentaire de xque19 le 27/07/2006 13:25:59

Salut,

Pour la config j'ai pas encore fait la fenêtre qui correspond, sinon pour l'ID3 il est déjà géré dans une bulle au passage de la souris sur la chanson (dans les skins Rock et celui par defaut uniquement.

Tiens moi au courant si tu rajoute du code. Merci.

De mon côté çà fait un moment que j'essaie de l'étendre à d'autres formats (ogg, wma et wav principalement mais la methode de streaming actuelle ne fonctionne que pour le mp3. Si quelqu'un peut m'aider ce serait extra.

++

Commentaire de Dav57 le 27/07/2006 19:01:35

Salut J_G !

Tu me vois extrêment ravi de ton enthousisame, j'espère que tu découvriras des artistes qui te plaisnet (y en a des  vraiment talentueux et pas qu'un peu).
J'ai participé à la mise en place du projet les 6 premiers mois de sa vie (relations avec les artistes/com) et je suis toujours attaché de prêt au projet, j'oeuvre pour aider les artistes de différentes manières.
Clair que c'est un site qu'il ne faut pas garder pour soi, il faut en parler à tout le monde ! :)

autre concept de jamendo : un logiciel libre qui s'appelle le jamcorder (libre et open-source) qui permet aux artistes d'enregistrer leurs concerts en mp3 ou ogg sur un simple laptop. Puis à la fin du concert : tous ceux qui se pointent avec une clé usb/mp3 repart avec le concert complet :) c'est cool non ?

à bientôt,

Commentaire de codfish le 01/02/2007 16:59:00

Bonjour ;

Probleme avec le ini/users.ini, si quelqun fait une mise a jour de l'interface/theme tous les donnes dans ce dernier disparera;

ex:

[utilisateur_que_a_fait_la_mise_a_jour_de_son_interface]
skin = "Rock"

Commentaire de pixelk le 21/05/2007 22:20:04

Merci beaucoup d'avoir partagé ton travail, le résultat est exactement ce que je cherchais pour remplacer Jinzora.
C'est simple et léger, bravo.

Passons à mes "Request for improvments" :
* Proposer une version "sans login" pour l'utilisation en LAN (je stocke mes musiques sur un serveur, je les écoute sur un portable).
* La possibilité de lister tous les morceaux d'un coup (plustôt que "tout cocher" suivi de "lister")
* La possibilité de lister le contenu d'un dossier sans rentrer dedans (pas forcément les sous-dossiers).

Encore bravo et merci pour ton travail.

Commentaire de xque19 le 21/05/2007 23:39:57

par contre pour la version sans login, je pense rajouter une option configrable par l'admin qui permettra un accès invité, chaque invité aura ainsi un login généra de type "invite_xxxxxx" pour pouvoir acceder au chat etc... et les option de chargement et sauvegarde de la playlist seront grisées, pour éviter tout conflit.
Si vous avez une proposition là dessus, n'hésitez pas.

Commentaire de xque19 le 23/05/2007 00:51:34

ok merci, çà voudrai dire que mon code fonctionne aussi sous OpenBSD :D, je 'ai pas eu l'occaz de le tester là dessus. Sous Ubuntu, Fedora, Mandriva et Suze (ou j'ai pu le tester) çà a l'air de bien fonctionner aussi.

Pourrait-tu me renvoyer ta version modifiée sur mehdi[.]jaqir[@]free[.]fr, j'ajouterai quelques modifs et tes suggestions aussi, pour que tout le monde en profite.

merci...

Commentaire de clickclac13 le 11/07/2007 19:22:55

bonjour,
super travail interface allaichante mais,
j ai un probleme quand je click sur lire ma palylist dans winamp il met bad request avec
http://@IP:80/mp3/stream.php?user=admin&stream=469510e356948&file=12

commentresoudre le probleme


merci d'avance

Commentaire de sevirg le 22/10/2007 21:35:20

bonjour et gros gros taff merci a toi :)

repport de ce qui marche pas:

meme erreur que clickclac13
et ensuite grros probleme de login.

je me log en admin, je deco et je peux plus me reconnecter
pareil avec les comptes que j'ai crée : je crée un compte, on se log une fois et apres on peux plus :(

a part c nikel ;)

bonnne continuation

Commentaire de Astalavista le 02/01/2009 17:23:21

Faille de sécurité je pense (d'après ce que je connais)
Dans le fichier admin.php :
# else
# {
# if (isset($_SESSION["type_user"]) && $_SESSION["type_user"]!="admin")
# {
# print("<script language=\"javascript\">");
# print("window.close();");
# print("</script>");
# }
# }
Donc, sa signifie que si l'utilisateur n'es pas admin, on ferme la page grace au javascript ...Mais le reste du code est quand même exécuter ... Donc un post sera de toute façon envoyer ...
Tu devrais mettre des redirections avec Header ...