Beakerboy / Beakerboy/VBA-SQL-Library

SQLSelect — AddHaving passes wrong argument order

オープン
#64 コメント 0 件 リアクション 0 件 担当者 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行付近を開き、SQLCondition.Create の呼び出しから始めて、両方の分岐にある AddHaving を調べてください。デフォルト以外の演算子を使う HAVING を再現し、両方の分岐が期待される引数の順序で比較を生成することを確認してください。

索引モデルが issue の本文から書いたものです。

評価

技術スタック
sql
領域
database
issue の種類
バグ
難易度
1/5
見積もり時間
1時間未満
活発さ
停滞
明瞭さ
明確に書かれている
初心者へのやさしさ
55/100

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。