Simple Music Player Module

DrCastor

Hi aro28,

I had the same problem, so I decided to take a look at why this module couldn’t be loaded.

The issues is in the Node_Helper.js given, which is mistaken. The structure is wrong. It looks like a quick adaptation of a ctrl+C/ctrl+V made from index.js, found in the node_helper folder (MM core folders).

I’m actually tweeking it a bit. I’ll give you news when i get it worked nicely.
(In fact, it seems i can now load it, but since my speaker are broke, i cant be sure it reads files… I’ll change them tomorrow)

aro28

@DrCastor
Great! Thank you for reply. I hope you will succeed.

DrCastor

Ok, found bad speakers, but at least, they allow me to test it. It works :). I’m a real noob, so some useless things may still be there. But at least, it will do the trick

Go to the node_helper.js file, and change it this way:

var Class = require("../../js/class.js");
var express = require("express");
var path = require("path");


module.exports = NodeHelper.create({

	init: function() {
		console.log("Initializing mm-music-player module helper ...");
	},

	start: function() {
		this.expressApp.get('/music', function (req, res) {
			var fs = require('fs');
			var path = require('path');

			var files = [];

			fs.readdir('modules/mm-music-player/music', (err, data) => {
				for(i = 0; i < data.length; i++){
					if(path.extname(data[i]) == ".mp3"){
						files.push(data[i]);
					}
				}
				
				res.send(files);
			});
		});
	},



	/* setExpressApp(app)
	 * Sets the express app object for this module.
	 * This allows you to host files from the created webserver.
	 *
	 * argument app Express app - The Express app object.
	 */
	setExpressApp: function(app) {
		this.expressApp = app;

		var publicPath = this.path + "/public";
		app.use("/" + this.name, express.static(publicPath));
	},

});

If luck is by your side, it will work ;)

aro28

@DrCastor said in Simple Music Player Module:

Hi,
Could you please send a full code of “node_helper.js” file?
Thanks in advance,

DrCastor

It seems that I don’t have enough priviledges for uploading a file directly in this reply, and I don’t really want to provide you a link to some uploading site, since I don’t know if it is allowed in the rules of this site. I’ll try to find a way.

Since then, just try what I told you. You juste have to cut and paste what I wrote. I’m sure you can do it, be confident in yourself ;)

aro28

@DrCastor
Manay thanks!
Resolved.

DrCastor

Happy to have helped you :). It is sad that just for a problem of syntax, the source file was never edited :$

aro28

@DrCastor

Last question.
Do you know how to write a code for a volume buttons for that player?
I know that “MMM-TouchPlayerBasic” module has that feature.

DrCastor

no, I’m a noob in this. You should take a look at the files of MMM-TouchPlayerBasic, and try to enderstand how the volume is managed. Good luck!

Milk

would it be possible to make it so I could use a keyboard to control the music?