Beakerboy / Beakerboy/VBA-SQL-Library

SQLSelect — AddHaving passes wrong argument order

未关闭
#64 0 条评论 0 个 reaction 已指派 0 人 在 GitHub 查看
主要语言
VBA
星标
90
派生
18
PR 合并指标
30 天内没有已合并 PR

描述

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.

---

贡献指南

这个仓库没有索引到贡献指南

调研方向

打开 src/ClassModules/SQLSelect.cls 的第 71–95 行附近,检查两个分支中的 AddHaving,从调用 SQLCondition.Create 的地方开始。复现一个使用非默认运算符的 HAVING,并验证两个分支生成的比较具有预期的参数顺序。

由索引模型根据 Issue 内容生成。

评估

技术栈
sql
领域
database
Issue 类型
缺陷
难度
1/5
预计耗时
1 小时以内
活跃度
停滞
描述清晰度
描述清楚
新手友好度
55/100

把新 issue 发到你的邮箱

精选适合新手参与的 GitHub issue 摘要。