Map name regex for City square matches multipe other maps

veiset / poe-vendor-string

Path of Exile Vendor Search tool

61 stars 17 forks source link

Map name regex for City square matches multipe other maps #151

Open dy95rabo opened 4 months ago

dy95rabo commented 4 months ago

your shorthand for city square "ty s" also matches multiple other maps like:

cemetry
academy

changing it to "ty sq" fixes that problem, not sure if its the most efficient tho.

best regards

veiset commented 4 months ago

Thanks for the report, yeah this looks like an error.

This bug is part of a larger issue, "ty s" shouldnt match cemetry ty s|..., and is parsed as: contains ty or s when searching the map tab, I'll have to make a special rule to ensure that when searching the map tab you cant use words with spaces in them.