BazaarNotifier icon indicating copy to clipboard operation
BazaarNotifier copied to clipboard

Split calls should be replaced with Regular Expressions

Open symt opened this issue 2 years ago • 1 comments

https://github.com/symt/BazaarNotifier/blob/beta/src/main/java/dev/meyi/bn/handlers/EventHandler.java#L49-L50

As of now, this causes a problem with "Farming for Dummies". The "for" means the item name won't be registered correctly. Use of more robust regular expressions can fix this.

For example, the bank calculator uses it properly: https://github.com/symt/BazaarNotifier/blob/beta/src/main/java/dev/meyi/bn/modules/calc/BankCalculator.java

symt avatar Mar 21 '23 21:03 symt

Every single split call that is used to parse out the amount, price, name, etc. of an order should be replaced with regex

This will be done with 1.8.0

symt avatar Feb 10 '24 16:02 symt