MMM-Wunderlist-Enhanced javascript error

geeklimit

Running MMM-Wunderlist-Enhanced via PM2 on Raspbian Jessie Lite. MM 2.1.3 and Electron 1.4.5.

MMM-Wunderlist-Enhanced doesn’t show on screen. Checking PM2 error logs shows:

** Message: console message: [native code] @0: Load script: modules/MMM-Wunderlist-Enhanced//MMM-Wunderlist-Enhanced.js

** Message: console message: http://localhost:8080/modules/MMM-Wunderlist-Enhanced//MMM-Wunderlist-Enhanced.js @59: SyntaxError: Unexpected token '>'

I’ve tried the following, just to make sure all dependencies and files are up to date and correct:

pi@kitchenpi:~/MagicMirror/modules/MMM-Wunderlist-Enhanced $ git fetch --all
Fetching origin
pi@kitchenpi:~/MagicMirror/modules/MMM-Wunderlist-Enhanced $ git reset --hard origin/master
HEAD is now at 4f01f5d Update README.md
pi@kitchenpi:~/MagicMirror/modules/MMM-Wunderlist-Enhanced $ npm install
up to date in 37.436s

Seems OK. The section of code in MMM-Wunderlist-Enhanced.js that’s giving the problem is:

    this.config.lists.forEach((listValue, listKey) => {

and in context here (line 59):

  getTodos: function() {
    var tasks = [];
    this.config.lists.forEach((listValue, listKey) => {
      let list = this.tasks[listValue];
      if (list && list.length)
        list.forEach(todo => {
          if (this.config.order === 'reversed') {
            tasks.push(todo);
          } else {
            tasks.unshift(todo)
          }

It seems like the ‘=>’ is the problem?

I’m not a javascript coder, so I can’t tell if it’s supposed to be that way or not.

I tried changing it to just ‘=’, but that doesn’t fix the problem.

Sean

@geeklimit what is your device? => is ES6 style syntax, so old js engine cannot parse it.

geeklimit

It’s a Pi Zero W.

pi@kitchenpi:~/MagicMirror/modules/MMM-NOAA $ npm version
{ 'mmm-noaa': '1.0.0',
  npm: '5.4.2',
  ares: '1.10.1-DEV',
  cldr: '31.0.1',
  http_parser: '2.7.0',
  icu: '59.1',
  modules: '57',
  node: '8.3.0',
  openssl: '1.0.2l',
  tz: '2017b',
  unicode: '9.0',
  uv: '1.13.1',
  v8: '6.0.286.52',
  zlib: '1.2.11' }

Looks like node is at version 8.3.0.

So…downgrade node or update the syntax?

Sean

You can modify that with old style like this.

(...)=>{...}

to

function (...) {...}

Sean

@geeklimit I think pi zewo would use Midori instead Chromium as default browser. So you can try my above solution.

Sean

Watchout! You should change all of this to self in function and prepend var self = this before function.

geeklimit

I am using Midori as part of my startup script. Specifically:

#!/bin/sh

unclutter &
xset -dpms # disable DPMS (Energy Star) features.
xset s off # disable screen saver
xset s noblank # don’t blank the video device
matchbox-window-manager &
midori -e Fullscreen -a http://localhost:8080

Sean

@geeklimit can you change midori to chromium? I think its easier than code modifying.

geeklimit

You’re losing me, but I think you mean I should edit the line causing the error from:

this.config.lists.forEach((listValue, listKey) => {

to

this.config.lists.forEach ((listValue, listKey) {

Or does this section of the code need a complete rewrite?

Sean

@geeklimit sorry but i think there is not the only place where es6 style used. You should find and modify every place of that kinds. Browser changing is more easier.