Skip to content

Replace TRY_PARSE(... AS money) with TRY_PARSE(... AS decimal(19,2) f…#657

Merged
erikdarlingdata merged 5 commits intoerikdarlingdata:devfrom
gflpcantor31:issue_656
Jan 15, 2026
Merged

Replace TRY_PARSE(... AS money) with TRY_PARSE(... AS decimal(19,2) f…#657
erikdarlingdata merged 5 commits intoerikdarlingdata:devfrom
gflpcantor31:issue_656

Conversation

@gflpcantor31
Copy link
Copy Markdown
Contributor

This PR addresses issue #656.

Changes:

  • Replace TRY_PARSE(... AS money) with TRY_PARSE(... AS decimal(19,2)) in the ORDER BY logic of sp_QuickieStore.

Rationale:

  • Aligns with decimal(19,2) used in temp tables.
  • Avoids money-specific rounding behavior.
  • Improves multi-locale robustness when parsing formatted strings.
  • Does not change the visible output, only the numeric type used for sorting.

All tests performed locally show no regression.

@erikdarlingdata erikdarlingdata merged commit 8ee5d8e into erikdarlingdata:dev Jan 15, 2026
1 check passed
@gflpcantor31 gflpcantor31 deleted the issue_656 branch January 18, 2026 10:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants