Skip to content

Conversation

@QuickStick123
Copy link
Contributor

@QuickStick123 QuickStick123 commented Apr 24, 2023

Description of the problem being solved:

Some mods on the trade site are listed as being negative value of increased or similar instead of in reduced format. This adds code to switch these around as well as fixing a bug related to flasks not being parsed correctly and projectile pierce.

The only remaining problems are weird parsing behaviour from trade site (e.g. "Arrows Pierce an additional Target" has ranges for each of the mods but no #), legacy mods(still exist but have a weight of zero or unreleased), missing mods (on trade site) or multi line mods e.g. trigger mod or similar.

@QuickStick123 QuickStick123 changed the title Fix inverse mods and flasks Fix inverse mods and improve flask behaviour in trader Apr 24, 2023
@QuickStick123 QuickStick123 added bug Something isn't working enhancement New feature, calculation, or mod and removed bug Something isn't working labels Apr 24, 2023
@QuickStick123 QuickStick123 changed the title Fix inverse mods and improve flask behaviour in trader Support inverse mods and improve flask behaviour in trader Apr 24, 2023
@QuickStick123 QuickStick123 changed the title Support inverse mods and improve flask behaviour in trader Support trade inversed mods and improve flask behaviour in trader Apr 24, 2023
@QuickStick123 QuickStick123 changed the title Support trade inversed mods and improve flask behaviour in trader Support trade inversed mods and improve flask mod's behaviour in trader Apr 24, 2023
@LocalIdentity LocalIdentity merged commit ce01a7b into PathOfBuildingCommunity:dev Apr 27, 2023
@QuickStick123 QuickStick123 deleted the fix-inverse-mods-and-flasks branch April 27, 2023 15:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature, calculation, or mod

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants