Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

TypeError: novels is undefined #7

Closed
SaltatorMortis opened this issue Mar 2, 2019 · 7 comments
Closed

TypeError: novels is undefined #7

SaltatorMortis opened this issue Mar 2, 2019 · 7 comments
Assignees
Labels
bug Something isn't working
Milestone

Comments

@SaltatorMortis
Copy link

after the Last update the addon doesnt load the Readinglist
a login via addon is still possible

tested with Firefox + chome

TypeError: novels is undefined
TypeError: can't access dead object
@SaltatorMortis SaltatorMortis changed the title update Fetch fail TypeError: novels is undefined Mar 2, 2019
@Spark-NF
Copy link
Owner

Spark-NF commented Mar 2, 2019

Crap, I'll take a look
Do you have the stack trace? (i.e. in which file/where in said file the error occurred)

EDIT: sorry for the spam

@Spark-NF Spark-NF self-assigned this Mar 2, 2019
@Spark-NF Spark-NF added the bug Something isn't working label Mar 2, 2019
@Spark-NF
Copy link
Owner

Spark-NF commented Mar 2, 2019

Could it be that you have so many novels that keeping all of their chapters in memory would take too much RAM? How many novels do you have among all your reading lists?

@SaltatorMortis
Copy link
Author

SaltatorMortis commented Mar 2, 2019

after addon reenabling:

18:04:00.515 Event received <unavailable>
amo-c08279ab41f512d05228.js:1:1256327
18:04:04.612 Warning: attempting to write 8559 bytes to preference browser.uiCustomization.state. This is bad for general performance and memory usage. Such an amount of data should rather be written to an external file. This preference will not be sent to any content processes.
18:04:04.868 Event received <unavailable> amo-c08279ab41f512d05228.js:1:1256327
18:04:04.957 TypeError: chromeWin.gBrowserInit is undefined[Weitere Informationen]
ext-webNavigation.js:128:1
18:04:25.867 CSI/start
m=consumer:73:260

after opening the popup

18:04:27.428 TypeError: novels is undefined bundle.js:115:1

PS.
jeah it could be my blacklist thats causing the problems (PPS around 160)

@Spark-NF Spark-NF modified the milestones: v1.1.0, v1.1.1 Mar 2, 2019
@Spark-NF
Copy link
Owner

Spark-NF commented Mar 2, 2019

I'll try to add a few hundred novels with lots of chapters to my reading list to see if I can reproduce, then I'll push a fix and release v1.1.1 within the weekend.

Sorry and thanks for noticing it so fast! 😅

@SaltatorMortis
Copy link
Author

hmm
after a littlebit testing i found out with my Readinglist if i go above 18 Novels it "crashes"

@Spark-NF
Copy link
Owner

Spark-NF commented Mar 2, 2019

Ok so after some investigation there seems to be 3 things here:

  • Some parsing error while loading the reading list or a novel's chapters cause the loading to fail (very likely is a loading error occurred while loading a novel's chapter)
  • If you open the popup after a failed loading, it will try to show in an empty novel list, which causes the TypeError: novels is undefined
  • If you open the popup before the loading is finished then close the popup, when the loading finishes it will cause a TypeError: can't access dead object which can be safely ignored (I'll try to make it not occur however)

(so nothing to do with the number of novels in the list it would seem)

@Spark-NF
Copy link
Owner

Spark-NF commented Mar 2, 2019

Note that I also improved the logging/reporting so if there are loading errors later, they'll be much more easily to investigate (because that was truly not obvious here). 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants