Beakerboy / Beakerboy/VBA-SQL-Library

SQLSelect — AddHaving passes wrong argument order

Ouverte
#64 0 commentaires 0 réactions 0 personnes assignées Voir sur GitHub
Langage dominant
VBA
Étoiles
90
Forks
18
Métriques de merge des PR
Aucune PR mergée en 30 j

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.

---

Guide de contribution

Aucun guide de contribution indexé pour ce dépôt

Piste de recherche

Ouvrez src/ClassModules/SQLSelect.cls autour des lignes 71–95 et examinez AddHaving dans les deux branches, en commençant par les appels à SQLCondition.Create. Reproduisez un HAVING utilisant un opérateur autre que celui par défaut et vérifiez que les deux branches génèrent la comparaison avec l’ordre d’arguments attendu.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Évaluation

Stack technique
sql
Domaine
database
Type d'issue
Bug
Difficulté
1/5
Temps estimé
Moins d'une heure
Activité
À l'abandon
Clarté
Clairement spécifiée
Accessibilité débutants
55/100

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.