search

nornagon / cdda-guide

The Hitchhiker's Guide to the Cataclysm
https://cdda-guide.nornagon.net
GNU General Public License v3.0
112 stars 27 forks source link

fix computation of melee and ranged coverage #158

Closed pizzasgood closed 10 months ago

pizzasgood commented 10 months ago

Armors which define multiple coverage values for different sublocations weren't falling back on the default coverage value for that location when no cover_melee and cover_ranged values were set. This could be seen in the tshirt, where it would display separate and bogus values for melee and ranged coverage even though no such coverages were defined in the JSON.

pizzasgood commented 10 months ago

I guess that's fine, sure.

nornagon commented 10 months ago

Does this fix https://github.com/nornagon/cdda-guide/issues/96...?

pizzasgood commented 10 months ago

Only part of it. It gets rid of those Melee and Ranged values, but it doesn't address the 150% coverage down in the materials section of the output.