Skip to content

Fix Buy Similar searches for punctuated unique names - #10139

Merged
LocalIdentity merged 2 commits into
PathOfBuildingCommunity:devfrom
mcagnion:bugfix/buy-similar-apostrophe-name
Aug 6, 2026
Merged

Fix Buy Similar searches for punctuated unique names#10139
LocalIdentity merged 2 commits into
PathOfBuildingCommunity:devfrom
mcagnion:bugfix/buy-similar-apostrophe-name

Conversation

@mcagnion

Copy link
Copy Markdown
Contributor

Description of the problem being solved:

Buy Similar could generate an invalid Trade search for unique names containing punctuation, such as apostrophes. The unique name is now kept intact while an optional trailing numeric seed is still removed.

Steps taken to verify a working solution:

  • Verified Buy Similar opens a valid Trade search for unique names containing apostrophes and hyphens.
  • Added coverage for URL generation from every currently loaded unique name, as well as trailing numeric seeds.
  • Verified that existing whitespace normalization remains intact.

Keep official unique names intact while stripping user-added trailing numeric IDs.
@mcagnion
mcagnionforce-pushed the bugfix/buy-similar-apostrophe-name branch from 8718bb9 to 5738bfeCompareAugust 5, 2026 22:03
@mcagnion
mcagnion marked this pull request as ready for review August 5, 2026 22:06
@LocalIdentityLocalIdentity added the bug: behaviour Behavioral differences label Aug 6, 2026
@LocalIdentity
LocalIdentity merged commit 8b6a8e6 into PathOfBuildingCommunity:devAug 6, 2026
3 checks passed
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug: behaviourBehavioral differences

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@mcagnion@LocalIdentity