CFenner / MMM-Smappee

MMM-Smappee.js

/* Magic Mirror
 * Module: Smappee
 *
 * By Christopher Fenner http://github.com/CFenner
 * MIT Licensed.
 */
Module.register('MMM-Smappee', {

The variable Module seems to be never declared. If this is a global, consider adding a /** global: Module */ comment.

  defaults: {
    animationSpeed: 1000,
    client: {
      id: '',
      secret: ''
    },
    user: {
      id: '',
      password: ''
    },
    location: {
      id: '',
      name: ''
    },
    showTimestamp: false,
    updateInterval: 60000
  },
  // Load translations files
  getTranslations: function() {
    return {
      en: "i18n/en.json",
      de: "i18n/de.json"
    };
  },
  //getScripts: function() {return [];},
  //getStyles: function() {return [];},
  getDom: function() {
    var wrapper = document.createElement("div");

    if (!this.loaded) {
      wrapper.innerHTML = this.translate('LOADING');
      return wrapper;
    } else {
      var moduleInfo = document.createElement('div'), valueWrapper, value;
  
      valueWrapper = document.createElement('div');
      valueWrapper.setAttribute('class', 'small');
      valueWrapper.innerHTML = this.translate('CURRENT_CONSUMPTION') + ' ';
      value = document.createElement('span');
      value.setAttribute('class', 'wi weathericon wi-lightning bright');
      valueWrapper.appendChild(value);
      if(this.config.showTimestamp){
        value = document.createElement('div');
        value.innerHTML = moment(this.consumption.timestamp).fromNow();
        valueWrapper.appendChild(value);
      }
      moduleInfo.appendChild(valueWrapper);

      valueWrapper = document.createElement('div');
      value = document.createElement('span');
      value.innerHTML = this.consumption.consumption * 10 + ' W';
      value.setAttribute('class', 'bright');
      valueWrapper.appendChild(value);
      moduleInfo.appendChild(valueWrapper);

      valueWrapper = document.createElement('div');
      valueWrapper.setAttribute('class', 'small');
      valueWrapper.innerHTML = this.translate('PERMANENT_CONSUMPTION') + ' ';
      value = document.createElement('span');
      value.setAttribute('class', 'wi weathericon wi-night-clear bright');
      valueWrapper.appendChild(value);
      moduleInfo.appendChild(valueWrapper);

      valueWrapper = document.createElement('div');
      value = document.createElement('span');
      value.innerHTML = this.consumption.alwaysOn + ' W';
      value.setAttribute('class', 'bright');
      valueWrapper.appendChild(value);
      moduleInfo.appendChild(valueWrapper);
  
      wrapper.appendChild(moduleInfo);
    }
    return wrapper;
  },
  start: function() {
    Log.info('Starting module: ' + this.name);

The variable Log seems to be never declared. If this is a global, consider adding a /** global: Log */ comment.

    Log.info(this.data.classes);
    this.loaded = false;
    this.consumption = '';
    this.update(this);
  },
  update: function(self) {
    var location = self.config.location;

The assignment to variable location seems to be never used. Consider removing it.

    self.sendSocketNotification('SMAPPEE_LOAD', self.config);
    setTimeout(self.update, self.config.updateInterval, self);
  },
  process: function(payload){
    this.consumption = payload;
		this.loaded = true;
		this.updateDom(this.config.animationSpeed);
  },
  socketNotificationReceived: function(notification, payload) {
    if (notification === 'SMAPPEE_DATA') { //  && this.config.location === payload.location
      Log.info('received SMAPPEE_DATA');

The variable Log seems to be never declared. If this is a global, consider adding a /** global: Log */ comment.

      this.process(payload);
    }
  }
});