read csv-data and put it in an array

[sdetweil]

@Perlchamp the classname is a string

and the front part is the same

so , you had duplicated code, just to change the classname…

I just used your if(entrie) to decide it the name needed the’DIMMED" part added
move the _ to the “DIMMED”, like this “_DIMMED”

then they will be TD-IMAGE and TD-IMAGE_DIMMED , just like you had it…

its the same row data and parentage, just a class change… (nice work having common names, so you could do this), got rid of that big else…

[Perlchamp]

css i don’t get to work. here’s my getDom:

getDom: function() {
	var wrapper = this.createEl("div",null,null,null,null);
	
	// tell MM to call and get our content
	Log.log(JSON.stringify(this.active_birthdays));

	if ((moment() > this.midnight) || (!this.loaded)) {
	    var month = moment().month();
	    var year = moment().year();
	    var monthName = moment().format("MMMM");
	    var monthLength = moment().daysInMonth();
	    var now = moment();

	    if(Object.keys(this.active_birthdays).length > 0) {
	    
		// create table
		var table = this.createEl("table", "birthday-table", "TABLE" , wrapper,null)

		// create tableheader here, array of column names
		var table_header = this.createTableHeader(table, null, [" "," "]);
		
		// create TBODY section with day names
		var tBody = this.createEl('tBody', "birthday-tbody", "TBODY" , table, null);

		var birthdays_seen= {}
		var spacer = " ";
		for(var birthday of Object.keys(this.active_birthdays)) {
		    for(var person of this.active_birthdays[birthday]) {

		    				
			// create looped row section
			var bodyTR = this.createEl('tr', "birthday-tr-body", "TR-BODY" , tBody, null);

			let now = moment().format('DD.MM').valueOf();				
			let entrie=now;
			if(this.config.dimmEntries) {				
			    entrie = moment(birthday,'DD.MM').valueOf();
			}
			
			// delete leading 0 and month
			var bd = "";
			
			if(birthdays_seen[birthday] == undefined) {
			    bd = (birthday.startsWith("0")? birthday.substring(1): birthday).split('.')[0];

			    var imageTD = this.createEl('td', null, "TD-IMAGE" + (entrie < now)?"_DIMMED":'' , bodyTR, bd);		
			    var nameTD = this.createEl('td', null, "TD-BODY" + (entrie < now)?"_DIMMED":'' , bodyTR, person.name);
			    this.createEl("span", null, null, nameTD, " ");						
			    
			    // needs class for width
			    var spanTDo = this.createEl("span", null, "TD-AGE" + (entrie < now)?"_DIMMED":'', nameTD, person.age);
			}
			else{
			    // add a break
			    this.createEl('br', null , null , spanTDo,null);
			    // add a span with name
			    var nameTD = this.createEl('span', null, "TD-BODY" + (entrie < now)?"_DIMMED":'' ,spanTDo, person.name);
			    // add a span with age
			    var spanTD = this.createEl("span", null, "TD-AGE" + (entrie < now)?"_DIMMED":'', spanTDo, person.age);						
			}					
			
			var spacerTR = this.createEl('tr', null, null , tBody, null);
			var spacerTD = this.createEl('td', null, "SPACER" , spacerTR, " ");
			spacerTD.colSpan = "2";
			birthdays_seen[birthday] = true;
			
		    }
		}

		// Create TFOOT section -- currently used for debugging only
		if (this.config.debugging) {
		    var table_footer = this.createTableFooter(tBody, null, [" "," "]);
		    //footerTD.innerHTML = "Birthdaylist is currently in DEBUG mode!<br />Please see console log.";
		}
		else {
		    var table_footer = this.createTableFooter(tBody, null, [" "," "]);
		}

	    }

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

	}
	// Dom is loaded
	this.loaded = true;

    },

there should be an error i don’t see.

[sdetweil]

@Perlchamp use the debugger, expand the elements tree, find the table and look at the classes assigned

[Perlchamp]

hi sam,
if i do this:

   var imageTD = this.createEl("td", null, "TD-IMAGE", bodyTR, bd);

instead of this :

var imageTD = this.createEl("td", null, "TD-IMAGE" + (entrie < now)?"_DIMMED":'' , bodyTR, bd);

css are addressed. maybe something has to change in the createEl function ?

[sdetweil]

@Perlchamp let me look…

[sdetweil]

@Perlchamp

warning one ( was missing, fixed
SORRY… tested now… the + append didn’t work, had to force with concat()…

Its the two values, now and entrie

				let now = moment().format('DD.MM').valueOf();				 // this is string dd.mm
				let entrie=now
			    if(this.config.dimmEntries) {				
					entrie = moment(birthday,'DD.MM').format.valueOf();     // this is a moment 

so later when compared, it doesn;t work

and comparing strings with < doesn’t really work…

I would do

let now=moment()
let entrie = moment(birthday,‘DD.MM’)
then in the compare
(entrie.isBefore(now))?..

	getDom: function(){
		var wrapper = this.createEl("div",null,null,null);

		if(Object.keys(this.active_birthdays).length>0){

			// create your table  here
			var table = this.createEl("table", "birthday-table", null, wrapper );

			// table header here, array of column names
			var table_header=this.createTableHeader(table,null, ["date","Name","age"])		

			// create looped row section
			var tBody = this.createEl('tbody', "birthday-t-body", "tbody", table, null);		

			var birthdays_seen= {}
			for(var birthday of Object.keys(this.active_birthdays)) {
			    for(var person of this.active_birthdays[birthday]) {			    				

					// create looped row section
					var bodyTR = this.createEl('tr', null, "TR-BODY" ,tBody, null);	

					let now = moment()		
					let entrie=now
				    if(this.config.dimmEntries) {				
						entrie = moment(birthday,'DD.MM')
					}
					var bd = "";					
			    	if(birthdays_seen[birthday]==undefined){
						bd =(birthday.startsWith("0")? birthday.substring(1): birthday).split('.')[0];
						var imageTD = this.createEl('td', null, "TD-IMAGE".concat(entrie.isBefore(now)?"_DIMMED":'') , bodyTR, bd);		
						var nameTD = this.createEl('td', null, "TD-BODY".concat(entrie.isBefore(now)?"_DIMMED":'') , bodyTR, person.name);
									 this.createEl("span", null, null, nameTD, " ");						// needs class for width
						var spanTDo = this.createEl("span", null, "TD-AGE".concat(entrie.isBefore(now)?"_DIMMED":''), nameTD, person.age);
					}	
					else{
						// add a break
						this.createEl('br', null , null , spanTDo,null);
						// add a span with name
						var nameTD = this.createEl('span', null, "TD-BODY".concat(entrie.isBefore(now)?"_DIMMED":'') ,spanTDo, person.name);
						// add a span with age
						var spanTD = this.createEl("span", null, "TD-AGE".concat(entrie.isBefore(now)?"_DIMMED":''), spanTDo, person.age);						
					}					


					var spacerTR = this.createEl('tr', null, null , tBody, null);
					var spacerTD = this.createEl('td', null, "SPACER" , spacerTR, " ");
					spacerTD.colSpan = "2";
					birthdays_seen[birthday] = true;

				}
		    }
		}
		return wrapper;
	},

[sdetweil]

@Perlchamp see my update above

[Perlchamp]

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

moment(birthday,‘DD.MM’)

sorry, doesn’t work. blackscreen. if i change back (my post) css is addressed. i didn’t change the jet = part. i think, something’s wrong with the cerateEl function, but you know i’m not an expert.but to control. here is my code :

for(var birthday of Object.keys(this.active_birthdays)) {
		    for(var person of this.active_birthdays[birthday]) {
		    				
			// create looped row section
			var bodyTR = this.createEl('tr', "birthday-tr-body", "TR-BODY", tBody, null);

			let now = moment();
			let entrie = moment(birthday,"DD.MM");
			
			if(this.config.dimmEntries) {				
			    entrie = moment(birthday,'DD.MM');
			}
			
			// delete leading 0 and month
			var bd = "";
			
			if(birthdays_seen[birthday] == undefined) {
			    bd = (birthday.startsWith("0")? birthday.substring(1): birthday).split('.')[0];
			    //var imageTD = this.createEl('td', null, "TD-IMAGE".concat(entrie.isBefore(now)?"_DIMMED":'', bodyTR, bd);
			    var imageTD = this.createEl("td", null, "TD-IMAGE", bodyTR, bd);
			    //var nameTD = this.createEl("td", null, "TD-BODY".concat(entrie.isBefore(now)?"_DIMMED":'', bodyTR, person.name);
			    var nameTD = this.createEl("td", null, "TD-BODY", bodyTR, person.name);
			    this.createEl("span", null, "TD-AGE", nameTD, " ");						
			    
			    // needs class for width
			    //var spanTDo = this.createEl("span", null, "TD-AGE".concat(entrie.isBefore(now)?"_DIMMED":'', nameTD, person.age);
			    var spanTDo = this.createEl("span", null, "TD-AGE", nameTD, person.age);
			}
			else{
			    // add a break
			    this.createEl("br", null , null , spanTDo, null);
			    // add a span with name
			    //var nameTD = this.createEl("span", null, "TD-BODY".concat(entrie.isBefore(now)?"_DIMMED":'',spanTDo, person.name);
			    var nameTD = this.createEl("span", null, "TD-BODY", spanTDo, person.name);
			    // add a span with age
			    //var spanTD = this.createEl("span", null, "TD-AGE".concat(entrie.isBefore(now)?"_DIMMED":'', spanTDo, person.age);
			    var spanTD = this.createEl("span", null, "TD-AGE", spanTDo, person.age);
			}					
			
			var spacerTR = this.createEl("tr", null, null , tBody, null);
			var spacerTD = this.createEl("td", null, "SPACER", spacerTR, " ");
			spacerTD.colSpan = "2";
			birthdays_seen[birthday] = true;
			
		    }
		}

[sdetweil]

@Perlchamp ah well. both work here… your latest and mine produce the same results

your output

date	Name	age
12	Sam 69
2	Frank 81
Sue81
5	mary 65
10	eee EEEEE 51
16	fff FFFFF 8
ggg GGGGG32
27	hhh HHHHH 18

my output

date	Name	age
12	Sam 69
2	Frank 81
Sue81
5	mary 65
10	eee EEEEE 51
16	fff FFFFF 8
ggg GGGGG32
27	hhh HHHHH 18

[Perlchamp]

what does that mean? but you don’t use css, do you ? YES, but didn’t you see, that i have annotate the

.concat(entrie.isBefore(now)?"_DIMMED":''

lines in the code, i posted ? without entrie.isBefore … css is addressed, with it’s not addressed