illusionistpm / ddo-gear-planner

DDO Gear Planning Website
http://ddo-gear-planner.netlify.com/
2 stars 7 forks source link

Add support for selectable (random) properties (affix or set) detected from item processing #81

Closed joenuts1 closed 1 month ago

joenuts1 commented 2 months ago

Create recursive function to handle nested lists inside enchantments cell Function will update crafting map if unique property is detected Dinosaur bone crafting code uses cell affix parsing code; function updated to accommodate new arguments Reposition crafting loader to outside item parsing loop to reduce number of file reads/writes Update parse affix function to support reading in sets and crafting dict to update when unique property found

joenuts1 commented 1 month ago

Ok, tracked down the issue. For detected craftable affixes, it was storing those at a top level instead of pushing those down in to an affixes key map.

I've updated, but am going to wait for the other pull request to be reviewed/approved, as there are some code paths that both of these pulls touch.

joenuts1 commented 1 month ago

Closing until I can properly clean up the commit.

joenuts1 commented 1 month ago

Ready for review.

To do : housekeeping to resolve / standardize data coming in from wiki

Some edge cases where affix names need to be tracked down and updated (these will shake out in future updates of synonym and group entries)