Scraping multi disc roms

HurricaneFan

@mediamogul I used the exact instructions pjft provided in his post.

mediamogul

I'll give it another go later this evening.

Exact steps on how you get it to crash can help.

I highlighted a folder in ES without any prior metadata attached to it and selected the scraping option from the menu. The folder name was 'The Oregon Trail', so it discovered quite a few options. I selected one, which then left me at the menu to save. After selecting the 'save' option, the crash happens immediately.

pjft

@mediamogul from your description, that's the exact issue I addressed. Let me know. Best.

mediamogul

@pjft

OK, I tried the one from your branch one more time and still got the crash. However, I then tried the dev branch and no crash at all. I tested it further by scraping multiple folders and game files. I also tested manual metadata additions and it still held up. I want to test a few more scenarios tomorrow such as scraping script files, but for now it's been a crazy long day and I'm looking to break out the Scotch and unwind a little. It really looks like you've nailed this thing down and it'll be great to have this feature back.

pjft

@mediamogul good to hear. Maybe I gave you the wrong instructions, so I wouldn't worry about my branch. Glad that the dev one works for you.

mediamogul

@pjft said in Scraping multi disc roms:

Maybe I gave you the wrong instructions

I'm sure the issue was on my end in some way. I'm just thrilled everything's set now. I plan on testing a few more scenarios around lunch, but I really think you got the crash licked.

mediamogul

@pjft

After testing several scenarios, your fix held up every time in both manually and automatically adding metadata to a directory. I also tested many individual game files for any possible issues that may have sprung up and everything seems to be working well there too. This was stellar work and is most appreciated. Also, thank you to @HurricaneFan for first bringing the issue to light and helping to troubleshoot.

pjft

@mediamogul not at all. A big part of the work was bisecting it, so thank you very much for all the help!

mediamogul

@pjft, @BuZz

It's been a while since this issue was fixed and it still doesn't seem to have made it into the main branch. Was there an issue of some sort, or has it just been lost in the mix? I haven't installed the development branch in a while, does it still exist there?

pjft

@mediamogul I wish I had a better answer here. I'm not really sure there's a proper graduation path from dev features to master. In fact, in hindsight, this breakdown was introduced when there was a lot of entropy because of multiple parallel developments that had an impact across the board - I recall power savings and some internal refactoring taking place at the same time.

Still, given that there's not a lot of meaty new development in ES these days, and the last meaningful changes took place quite a while back, it could make sense to consider a different approach so that new features are released faster?

mediamogul

@pjft said in Scraping multi disc roms:

Still, given that there's not a lot of meaty new development in ES these days, and the last meaningful changes took place quite a while back, it could make sense to consider a different approach so that new features are released faster?

Sorry for the late response... Holidays, ho ho ho and all. I know I would be all for that personally. Especially when it comes to releasing quality of life bug fixes like this. Editing the gamelists by hand is no fun. I believe the 'Last Played' limit has also yet to make it out of the development branch.