shabados / presenter

Desktop app for presenting the Shabad OS Database on projectors, TVs, and live streams
https://shabados.com
MIT License
19 stars 15 forks source link

Fix crash on pressing up/down gui buttons on first/last line #476

Closed bhajneet closed 4 years ago

bhajneet commented 4 years ago

image

When it reloads (automatically on countdown expiration or manually by user) it simply opens the shabad as it should.

Cannot reproduce consistently.

Attaching logs when it happens going forward ...

bhajneet commented 4 years ago

2020-05-08_14-43-08.172Z.log

Went from bookmark ardaas, to the same line from search

At the bottom:

{"level":30,"time":1588951117408,"pid":4484,"hostname":"BSK-X1C","msg":"BSK-X1C disconnected","v":1} {"level":30,"time":1588951117565,"pid":4484,"hostname":"BSK-X1C","msg":"BSK-X1C connected","v":1}

bhajneet commented 4 years ago

Crashed again going from these two lines, then to next "shabad"

{"level":30,"time":1588951170080,"pid":4484,"hostname":"BSK-X1C","msg":"Checking for database updates satisfying @shabados/database@^4.7.0","v":1} {"level":30,"time":1588951177343,"pid":4484,"hostname":"BSK-X1C","msg":"Local Database Version: 4.7.0","v":1} {"level":30,"time":1588951177343,"pid":4484,"hostname":"BSK-X1C","msg":"Remote Database Version: 4.7.0","v":1} {"level":30,"time":1588951177343,"pid":4484,"hostname":"BSK-X1C","msg":"No database updates available","v":1} {"level":30,"time":1588951217982,"pid":4484,"hostname":"BSK-X1C","msg":"Setting Line ID to V26Z","v":1} {"level":30,"time":1588951218681,"pid":4484,"hostname":"BSK-X1C","msg":"Setting Shabad ID to 843","v":1} {"level":30,"time":1588951218681,"pid":4484,"hostname":"BSK-X1C","msg":"Setting Line ID to NVNP","v":1} {"level":30,"time":1588951218682,"pid":4484,"hostname":"BSK-X1C","msg":"Setting mainLineId to NVNP","v":1} {"level":30,"time":1588951218682,"pid":4484,"hostname":"BSK-X1C","msg":"Setting nextLineId to NVNP","v":1} {"level":30,"time":1588951230040,"pid":4484,"hostname":"BSK-X1C","msg":"BSK-X1C disconnected","v":1} {"level":30,"time":1588951230190,"pid":4484,"hostname":"BSK-X1C","msg":"BSK-X1C connected","v":1}

bhajneet commented 4 years ago

Went to next shabad, no issue, went back, and tried again -- another crash:

{"level":30,"time":1588951284852,"pid":4484,"hostname":"BSK-X1C","msg":"Setting Line ID to NVNP","v":1} {"level":30,"time":1588951285589,"pid":4484,"hostname":"BSK-X1C","msg":"Setting Shabad ID to SGB","v":1} {"level":30,"time":1588951285589,"pid":4484,"hostname":"BSK-X1C","msg":"Setting Line ID to 8EEH","v":1} {"level":30,"time":1588951285590,"pid":4484,"hostname":"BSK-X1C","msg":"Setting mainLineId to 8EEH","v":1} {"level":30,"time":1588951285590,"pid":4484,"hostname":"BSK-X1C","msg":"Setting nextLineId to 8EEH","v":1} {"level":30,"time":1588951286673,"pid":4484,"hostname":"BSK-X1C","msg":"Setting Shabad ID to 843","v":1} {"level":30,"time":1588951286674,"pid":4484,"hostname":"BSK-X1C","msg":"Setting Line ID to NVNP","v":1} {"level":30,"time":1588951286674,"pid":4484,"hostname":"BSK-X1C","msg":"Setting mainLineId to NVNP","v":1} {"level":30,"time":1588951286675,"pid":4484,"hostname":"BSK-X1C","msg":"Setting nextLineId to NVNP","v":1} {"level":30,"time":1588951287134,"pid":4484,"hostname":"BSK-X1C","msg":"Setting Shabad ID to CSQ","v":1} {"level":30,"time":1588951287135,"pid":4484,"hostname":"BSK-X1C","msg":"Setting Line ID to V26Z","v":1} {"level":30,"time":1588951287135,"pid":4484,"hostname":"BSK-X1C","msg":"Setting mainLineId to 2HY4","v":1} {"level":30,"time":1588951287136,"pid":4484,"hostname":"BSK-X1C","msg":"Setting nextLineId to 2HY4","v":1} {"level":30,"time":1588951288362,"pid":4484,"hostname":"BSK-X1C","msg":"Setting Shabad ID to 843","v":1} {"level":30,"time":1588951288362,"pid":4484,"hostname":"BSK-X1C","msg":"Setting Line ID to NVNP","v":1} {"level":30,"time":1588951288363,"pid":4484,"hostname":"BSK-X1C","msg":"Setting mainLineId to NVNP","v":1} {"level":30,"time":1588951288363,"pid":4484,"hostname":"BSK-X1C","msg":"Setting nextLineId to NVNP","v":1} {"level":30,"time":1588951299718,"pid":4484,"hostname":"BSK-X1C","msg":"BSK-X1C disconnected","v":1} {"level":30,"time":1588951299844,"pid":4484,"hostname":"BSK-X1C","msg":"BSK-X1C connected","v":1}

2020-05-08_14-43-08.172Z.log

Restarting Shabad OS now...

bhajneet commented 4 years ago

Recreate (one scenario! this happens in others as well, though I don't know how just yet):

Attached logs:

2020-05-08_15-23-21.110Z.log

saihaj commented 4 years ago

This is related to info popover because once we added that this started to happen. Me and @Harjot1Singh tried to fix most of the issue before merging it in but it crashes on first and last lines of shabad or bani

bhajneet commented 4 years ago

https://github.com/ShabadOS/desktop/pull/503/files/5a57a08ca5fcc2eb086c94b8c865e875786d5142#diff-077d3cb94c5f8ce4f2a0fb13436a5509L214

Originally posted by @saihaj in https://github.com/ShabadOS/desktop/pull/503

saihaj commented 4 years ago

Resolution took 15 minutes

bhajneet commented 4 years ago

So presumably it should have been tagged "Difficulty: 0" haha