Beakerboy / Beakerboy/VBA-SQL-Library
SQLSelect — AddHaving passes wrong argument order
- Dominant language
- VBA
- Stars
- 90
- Forks
- 18
- PR merge metrics
- No merged PRs in 30d
Description
SQLSelect: correct AddHaving argument order
- Component: `src/ClassModules/SQLSelect.cls` (around 71–95)
### Problem
- Calls `SQLCondition.Create` with `(Field, op, Value)` but signature is `(Expression1, Expression2, Operator)`.
### Snippet (actual)
```vb
NewHaving.Create Field, op, Value
```
### Expected
```vb
NewHaving.Create Field, Value, op
```
### Steps to Reproduce
1) Add a HAVING with a non-default operator.
### Actual
- Generates the comparison with swapped operands/operator.
### Proposed Fix
- Pass `(Field, Value, op)` consistently in both branches.
---
Contributor guide
No contributing guide indexed for this repository
Assessment
This issue has not been assessed yet.