MMM-PGA

swvalenti

Getting this error now…can anyone help? Tried update uninstall/reinstall etc

1|MagicMirror  | [2020-12-05 17:02:23.100] [ERROR]  App threw an error during load
1|MagicMirror  | [2020-12-05 17:02:23.102] [ERROR]  Error: Cannot find module 'jsdom'
1|MagicMirror  | Require stack:
1|MagicMirror  | - /home/pi/MagicMirror/modules/MMM-PGA/OWGR.js
1|MagicMirror  | - /home/pi/MagicMirror/modules/MMM-PGA/node_helper.js
1|MagicMirror  | - /home/pi/MagicMirror/js/app.js
1|MagicMirror  | - /home/pi/MagicMirror/js/electron.js
1|MagicMirror  | - /home/pi/MagicMirror/node_modules/electron/dist/resources/default_app.asar/main.js
1|MagicMirror  | -
1|MagicMirror  |     at Module._resolveFilename (internal/modules/cjs/loader.js:627:15)
1|MagicMirror  |     at Function.Module._resolveFilename (/home/pi/MagicMirror/node_modules/electron/dist/resources/electron.asar/common/reset-search-paths.js:41:16)
1|MagicMirror  |     at Function.Module._resolveFilename (/home/pi/MagicMirror/node_modules/module-alias/index.js:49:29)
1|MagicMirror  |     at Function.Module._load (internal/modules/cjs/loader.js:531:27)
1|MagicMirror  |     at Module.require (internal/modules/cjs/loader.js:685:19)
1|MagicMirror  |     at require (internal/modules/cjs/helpers.js:16:16)
1|MagicMirror  |     at Object.<anonymous> (/home/pi/MagicMirror/modules/MMM-PGA/OWGR.js:3:15)
1|MagicMirror  |     at Module._compile (internal/modules/cjs/loader.js:786:30)
1|MagicMirror  |     at Object.Module._extensions..js (internal/modules/cjs/loader.js:798:10)
1|MagicMirror  |     at Module.load (internal/modules/cjs/loader.js:645:32)
1|MagicMirror  | [2020-12-05 17:02:23.105] [ERROR]  Whoops! There was an uncaught exception...
1|MagicMirror  | [2020-12-05 17:02:23.106] [ERROR]
1|MagicMirror  | Error: Cannot find module 'jsdom'
1|MagicMirror  | Require stack:
1|MagicMirror  | - /home/pi/MagicMirror/modules/MMM-PGA/OWGR.js
1|MagicMirror  | - /home/pi/MagicMirror/modules/MMM-PGA/node_helper.js
1|MagicMirror  | - /home/pi/MagicMirror/js/app.js
1|MagicMirror  | - /home/pi/MagicMirror/js/electron.js
1|MagicMirror  | - /home/pi/MagicMirror/node_modules/electron/dist/resources/default_app.asar/main.js
1|MagicMirror  | -
1|MagicMirror  |     at Module._resolveFilename (internal/modules/cjs/loader.js:627:15)
1|MagicMirror  |     at Function.Module._resolveFilename (/home/pi/MagicMirror/node_modules/electron/dist/resources/electron.asar/common/reset-search-paths.js:41:16)
1|MagicMirror  |     at Function.Module._resolveFilename (/home/pi/MagicMirror/node_modules/module-alias/index.js:49:29)
1|MagicMirror  |     at Function.Module._load (internal/modules/cjs/loader.js:531:27)
1|MagicMirror  |     at Module.require (internal/modules/cjs/loader.js:685:19)
1|MagicMirror  |     at require (internal/modules/cjs/helpers.js:16:16)
1|MagicMirror  |     at Object.<anonymous> (/home/pi/MagicMirror/modules/MMM-PGA/OWGR.js:3:15)
1|MagicMirror  |     at Module._compile (internal/modules/cjs/loader.js:786:30)
1|MagicMirror  |     at Object.Module._extensions..js (internal/modules/cjs/loader.js:798:10)
1|MagicMirror  |     at Module.load (internal/modules/cjs/loader.js:645:32) {
1|MagicMirror  |   code: 'MODULE_NOT_FOUND',
1|MagicMirror  |   requireStack: [
1|MagicMirror  |     '/home/pi/MagicMirror/modules/MMM-PGA/OWGR.js',
1|MagicMirror  |     '/home/pi/MagicMirror/modules/MMM-PGA/node_helper.js',
1|MagicMirror  |     '/home/pi/MagicMirror/js/app.js',
1|MagicMirror  |     '/home/pi/MagicMirror/js/electron.js',
1|MagicMirror  |     '/home/pi/MagicMirror/node_modules/electron/dist/resources/default_app.asar/main.js',
1|MagicMirror  |     undefined
1|MagicMirror  |   ]
1|MagicMirror  | }
1|MagicMirror  | [2020-12-05 17:02:23.119] [ERROR]  MagicMirror will not quit, but it might be a good idea to check why this happened. Maybe no internet connection?
1|MagicMirror  | [2020-12-05 17:02:23.120] [ERROR]  If you think this really is an issue, please open an issue on GitHub: https://github.com/MichMich/MagicMirror/issues

sdetweil

@swvalenti go to the module folder
npm install jsdom

swvalenti

@sdetweil Yea sorry thought I deleted that I figured it out after the fact. Thanks!

swvalenti

Anyone know how to fix this the error has to do with “athlete”??? Module is stuck on loading…

swvalenti

Anyone able to help as it seems the developer has abandoned the project…something to do with “athlete” in the json requests etc but idk where to start to fix that but willing to give it a shot with some direction. Thanks!

sdetweil

@swvalenti I don’t know, but you could

console.log(JSON.stringify(data,null,2))

(replace ‘data’ with the variable name of the response from the request)

to dump out the data u got back and examine the structure to see if it changed
if u have the exact URL, you usually can do this in a browser too…

swvalenti

@sdetweil Not sure I follow Sam I am new to json

swvalenti

@cowboysdude @Mykle1 Calling on the big guns here if you have any ideas :face_with_tongue:

swvalenti

Can anyone explain or help me to try and get this module working again? Seems simple enough as ‘athlete’ is throwing the error but idk where to start.

sdetweil

@swvalenti as I said, u can use some tools in the coding to get the json data structure printed out so you can examine it

or, use the url in a browser to collect the info.

once u have that you can attempt to find the info the module needed
and maybe change the code to match

I use this site for validating json documents
https://codebeautify.org/jsonvalidator?/jsonvalidate

but it also allows you to format the data in a more readable fashion

the code is trying to extract the player from the data returned
Cannot read property ‘athlete’ of undefined
0|MagicMirror | at Request._callback (/home/pi/MagicMirror/modules/MMM-PGA/ESPN.js:49:65)

the code is the ESPN.js line 45, column 65

u also may get noting back from the api at all…

swvalenti

@sdetweil Ok but which url do I use to check the json?

swvalenti

I just commented that line out and now it works! HA

sdetweil

@swvalenti the url used by the code to get the data from the api…

I don’t know the module at all, just trying to give some help.

Someone (You) will have to become the programmer.
learn how the module works in enough detail to :

1. find where the error occurs (this is given to you)
1. develop some mechanism to capture the data that results in the failure
1. analyze the data to determine the cause
1. develop a strategy for fixing the ‘problem’
1. apply the strategy to the existing code
repeat 1-5 til you have resolved the problem(s) successfully, or abandon the task

sdetweil

@swvalenti you could also open an issue on the module github page and hope the author might look at it… has updates from about 9 months ago…

mumblebaj

@sdetweil It is a screen scraper. I see he says he commented some line and it is working again for him.

sdetweil

@mumblebaj cool. but doesn’t change the work

swvalenti

@sdetweil I tried pulling an issue ticket on github to no avail. @mumblebaj what is a screen scraper and can you assist in fixing?

sdetweil

@swvalenti remember this is volunteer work… we have other things in our lives…

many developers do this in the winter when they are stuck in the house. and when they can go outside, they do…

swvalenti

@sdetweil said in MMM-PGA:

@swvalenti remember this is volunteer work… we have other things in our lives…

many developers do this in the winter when they are stuck in the house. and when they can go outside, they do…

completely understand sir

mumblebaj

@swvalenti The issue is in the ESPN.js at line 49 or so. It is looking for defendingChampion.athlete but there isn’t one and it fails. I see you have created an issue on GitHub but sometimes the guys abandon their modules. Maybe the developer responds to you issue. You could change line 49 to the below and it should work.

I would have done it this way:

tournament.defendingChamp = event.defendingChampion ? event.defendingChampion.athlete.displayName : ""