read csv-data and put it in an array

[Perlchamp]

ok, nothing changed and i don’t see any messages of “HAVEDATA” (changed it in both files to big letter without space). i only see the header “Geburtstage” … :-(
node_helper.js:

var NodeHelper = require("node_helper");
var moment = require("moment");

// add require of other javascripot components here
// var xxx = require('yyy'); here
const csv = require("csvtojson");
var birthdaysArray = [];

module.exports = NodeHelper.create({
/*
	init(){
		console.log("init module helper perlchamp");
	},
*/
	start() {

	},
/*	
	stop(){
		console.log("Stopping module helper: " + this.name);
	},
*/
	// handle messages from our module// each notification indicates a different messages
	// payload is a data structure that is different per message.. up to you to design this
	socketNotificationReceived(notification, payload) {				

		var self = this;
		console.log("Starting module helper: " + this.name);

		// convert the csv-file into a JSON-String
		const csvFilePath = this.path + '/data/birthdays.csv';
		csv()
		.fromFile(csvFilePath)
		.then((jsonObj)=>{      
			birthdaysArray = JSON.stringify(jsonObj);
			
			// send data to [modulname].js
			self.sendSocketNotification("HAVEDATA", jsonObj);
		})		

		// if config message from module
		if (notification === "CONFIG") {
			// save payload config info
			this.config = payload;
			// wait 15 seconds, send a message back to module
			setTimeout(()=> {this.sendSocketNotification("message_from_helper"," this is a test_message")}, 15000);
		}
		else if(notification === "????2") {
	
		}
	},
});

maye change this part, cause i never need this variable birthdaysArray:

.then((jsonObj)=>{      
			birthdaysArray = JSON.stringify(jsonObj);

perlchamp.js:

Module.register("perlchamp", {

	defaults: {
		language: "de",
		dimmEntries: true   // true: dims entries and the associated 
							//       symbol when the date has expired.
							// false: delete entries and the associated 
							//        symbol when the date has expired.
	},
	
	active_birthdays: {
		
	},


	init: function(){
		Log.log(this.name + " is in init!");
	},

	start: function(){
		Log.log("Starting module: " + this.name);

		// Set locale
		moment.locale(config.language);
	},

	loaded: function(callback) {
		Log.log(this.name + " is loaded!");
		callback();
	},

	// return list of other functional scripts to use, if any (like require in node_helper)
	getScripts: function() {
		return	["moment.js"];
	}, 

	// return list of stylesheet files to use if any
	getStyles: function() {
		return 	[this.data.path + "/css/perlchamp.css"];
	},


	// only called if the module header was configured in module config in config.js
	getHeader: function() {
		return this.data.header;
	},

	// messages received from other modules and the system (NOT from your node helper)
	// payload is a notification dependent data structure
	notificationReceived: function(notification, payload, sender) {
		// once everybody is loaded up
		if(notification==="ALL_MODULES_STARTED"){
			// send our config to our node_helper
			this.sendSocketNotification("CONFIG",this.config);
		}
		if (sender) {
			Log.log(this.name + " received a module notification: " + notification + " from sender: " + sender.name);
		} else {
			Log.log(this.name + " received a system notification: " + notification);
		}
	},
	
	// messages received from from your node helper (NOT other modules or the system)
	// payload is a notification dependent data structure, up to you to design between module and node_helper
	socketNotificationReceived: function(notification, payload) {

		if(notification === "HAVEDATE"){
			Log.log(this.name + " received a socket notification: " + notification + " - Payload: " + payload);
			
			var now = moment();
			for(var birthday of payload) {
				// get 1st 5 chars of birthdate, thru month
				// we will use this as the key in the hash
				var birth_date = birthday.birth.substring(0,4);
				
				// get the birthday as a moment in this year, for comparing
				var birth_date_moment = moment(birth_date + now.getYear(),"DD.MMYYYY");

				// u can add days to a moment object and then compare 
				//     if the birthdate is before that date (and after now)
				//     so within the next xx days

				// if the date is the same or later, don't use time of day
				if(birth_date.startOf('day').isSame(now.startOf('day'))) {

					// birthday is in this month
					// check the hash if we've seen anything for today yet
					// if we haven't see this date yet
					if(this.active_birthdays[birth_date] == undefined){
           
						//  create the holder for its info (array of 
						//		names) in the hash
						this.active_birthdays[birth_date] = [];
					}
					// save the persons name on the list
					this.active_birthdays[birth_date].push(birthday.name);
				}
			}
     
			// tell MM to call and get our content
			self.updateDom();
		}     			
	},

	// system notification your module is being hidden
	// typically you would stop doing UI updates (getDom/updateDom) if the module is hidden
	suspend: function(){

	},

	// system notification your module is being unhidden/shown
	// typically you would resume doing UI updates (getDom/updateDom) if the module is shown
	resume: function(){

	},

	// this is the major worker of the module, it provides the displayable content for this module
	getDom: function() {
		var wrapper = document.createElement("div");

		for(var birthday of Object.keys(this.active_birthdays)){
			for(var name of this.active_birthdays[birthday]){
				var m = document.createElement('div');
				m.innerText = birthday + ' '+ name;
				wrapper.appendChild(m);
			}
		}

		// pass the created content back to MM to add to DOM.
		return wrapper;
	}
})

please have an eye on the getDom-sektion.

[sdetweil]

no… it will do the thig every time a notifcation comes in, only want it when the module sends down a message (say it is alive)

in node_help you are sending this “HAVEDATA”
and in module you are looking for this if(notification === “HAVEDATE”){

add a var self = this to start of socketNotificationReceived() and getDom()

[sdetweil]

@Perlchamp said in read csv-data and put it in an array:

if(birth_date.startOf(‘day’)

also should be

if(birth_date_moment.startOf('day')

comparing moments, not strings! oops

[Perlchamp]

only the header i see, BUT now i see the terminal-message “HAVEDATA” but an error, too:

[sdetweil]

@Perlchamp open the developers window, ctrl-shift-i on the mirror screen,
select console tab,
put perl in the filter field

select the sources tab, navigate the left nav tree,. to modules/perlchamp/perlcmap.js

find the socketnotificationreceived

click the number in the left column, hit f5 (refresh) on keyboard

code should stop at the line you clicked

in the right pane, top there is a blue arrow (resume executing) and a top half of a globe, stop on next )instruction)

[Perlchamp]

before looking in dev: i saw the following:

var birth_date_moment = moment(birth_date + now.getYear(),"DD.MMYYYY");

change to DD.MM.YYYY (point between MM and YYYY) ?

[sdetweil]

@Perlchamp no, cause we extract dd.mm from the info provided in file…

oh, the substring should be 0,5, not 0,4

var birth_date = birthday.birth.substring(0,5);

[Perlchamp]

error => now.getYear() is not a function

[sdetweil]

@Perlchamp yeh, sorry, rememeber just off the top of my head…
date uses that …
moment its just year()

var birth_date_moment = moment(birth_date + now.year(),“DD.MMYYYY”);

[Perlchamp]

nothing changed. but now i see the HAVEDATA notification again. could it be, that the Dom-update (from notification) not really goes ?
i mean i saw in the dev, that the birthday-data are right. it was the first entry of this data-string (or jsonObj) …
only the header i can see …