riazXrazor / udemy-dl

Nodejs script to download a udemy.com course, for personal offline use
282 stars 67 forks source link

Cannot read property 'parent' of undefined #27

Closed oscar-dario-pabon closed 6 years ago

oscar-dario-pabon commented 6 years ago

As the title says:

image /usr/local/lib/node_modules/udemy-dl/node_modules/cheerio/lib/parse.js:55 var oldParent = node.parent || node.root, ^

TypeError: Cannot read property 'parent' of undefined at Function.exports.update (/usr/local/lib/node_modules/udemy-dl/node_modules/cheerio/lib/parse.js:55:26) at module.exports (/usr/local/lib/node_modules/udemy-dl/node_modules/cheerio/lib/parse.js:17:11) at Function.exports.load (/usr/local/lib/node_modules/udemy-dl/node_modules/cheerio/lib/static.js:22:14) at Request._callback (/usr/local/lib/node_modules/udemy-dl/lib/core.js:198:31) at Request.self.callback (/usr/local/lib/node_modules/udemy-dl/node_modules/request/request.js:186:22) at emitTwo (events.js:126:13) at Request.emit (events.js:214:7) at Request. (/usr/local/lib/node_modules/udemy-dl/node_modules/request/request.js:1163:10) at emitOne (events.js:116:13) at Request.emit (events.js:211:7) image

htkcodes commented 6 years ago

same here

RonyRizk commented 6 years ago

same here

sirStephen commented 6 years ago

same here

riazXrazor commented 6 years ago

May be udemy code has changed i'll look into it.

On Fri 27 Apr, 2018, 6:05 PM sirStephen, notifications@github.com wrote:

same here

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/riazXrazor/udemy-dl/issues/27#issuecomment-384956972, or mute the thread https://github.com/notifications/unsubscribe-auth/AMlUe9byhthDmjHi4k_yM3DKDXt3Ig3_ks5tsxB-gaJpZM4TnEwU .

riazXrazor commented 6 years ago

yup i visited the site, it seems slugish, and they some portions has been worked on, so my code also has to be updated accordingly.

RonyRizk commented 6 years ago

any update ?

JC-JackClark commented 6 years ago

FYI I have a fix for this, but I need to readd the functionality to pick resolution.

Will make a pull request shortly