JK3Y / NPS-Browser-macOS

macOS version of NPS Browser
The Unlicense
183 stars 21 forks source link

1.3.6 insta-crashes after bookmarking #20

Closed RaphaelDDL closed 5 years ago

RaphaelDDL commented 5 years ago

Heya :)

Downloaded 1.3.6, did rm -r ~/Library/Application\ Support/NPS\ Browser/ just to make sure.

What I did:

Tried opening multiple times, no luck. Selecting any of the two options in the dialog about opening windows does not do anything, app straight up crashes. RM'ing app support again does not help.

Here my crash dump: NPS Browser_2018-08-03-024051_Raphaels-MacBook-Pro.crash.log

Thanks for your hard work, bro :)

JK3Y commented 5 years ago

try deleting the user settings you have set: defaults delete JK3Y.NPS-Browser then rerun.

Does this make the app able to start? the error "Invalid path given: /Users/RaphaelDDL/Downloads/NPS Downloads/" makes me think it's the download folder's path that's the problem... I'll have to do more looking into this though.

RaphaelDDL commented 5 years ago

I never set that folder. My NPS folder (from previous versions) always was ~/Documents/NPS. Before deleting the defaults, I created that folder you mentioned. The app opened as normal. O__o

I checked the preferences, the path indeed is wrong, though if I press the Folder icon, it shows me the correct place (inside Documents). So I clicked on my folder, pressed open on the folder inside Documents again and now the path updated. Weird.

I guess you'll need an error handler for when folder on settings do not exist.


Well, just to make sure folder had nothing to do with the bookmark, I did the same steps just to make sure: search atelier, bookmarked Firis, the before last one - shallie - already have yellow star, crashed again (now with the folder).

But reopening does not crash instantly. What I noticed was that whenever I bookmark something, all games stay with yellow star, for example, I removed the search, went back to full list, all games are yellow star.

Doing the defaults delete didn't change anything. NPS Browser_2018-08-03-122814_Raphaels-MacBook-Pro.crash.log

Bookmarking, closing, reopening. The bookmarked game has no yellow star. It shows on the Bookmark list though. Clicking on it's star toggle between white/yellow, but it's not added/removed from the list. Doing the same on other game crashes too.

JK3Y commented 5 years ago

Thanks for trying so many things, it narrowed it down for me. It seems like the issue is with the bookmarks and how they're being handled. Does the app crash for you if you don't use the bookmark star?

Ps: doing the defaults delete command deletes the app's saved preferences. Deleting the application support folder does not do this. I was hoping it would have reset the downloads location folder to the main Downloads folder

RaphaelDDL commented 5 years ago

Glad to be of help :)

JK3Y commented 5 years ago

I found the stupid issue. I accidentally deleted one of the functions that checks if the selected row has a bookmark then changes the star accordingly. -____-

If you have XCode installed you can build from source, otherwise I'm gonna keep working on some other additions and will push out an update later.

RaphaelDDL commented 5 years ago

hahah no worries. That's what git is good for, pulling out diffs to see what was wrong :D