fix: Trial Pairing Simulator — correct SQLite string quoting for sex filter #43

jason · 2026-03-10T14:38:32-05:00

jason commented

2026-03-10 14:38:32 -05:00

Bug

The Trial Pairing Simulator returns no such column: male when clicking Simulate Pairing.

Root Cause

In server/routes/pedigree.js (lines 164–165), the trial-pairing POST handler used double quotes around the string literals "male" and "female" in the SQL queries:

// BROKEN
db.prepare('SELECT * FROM dogs WHERE id = ? AND sex = "male"').get(sire_id);
db.prepare('SELECT * FROM dogs WHERE id = ? AND sex = "female"').get(dam_id);

SQLite follows the SQL standard where double quotes denote identifiers (column/table names), not string values. So SQLite interpreted "male" as a column name, throwing no such column: male.

Fix

Changed to single-quoted string literals, which SQLite correctly treats as values:

// FIXED
db.prepare("SELECT * FROM dogs WHERE id = ? AND sex = 'male'").get(sire_id);
db.prepare("SELECT * FROM dogs WHERE id = ? AND sex = 'female'").get(dam_id);

Files Changed

server/routes/pedigree.js — lines 164–165 only

## Bug The Trial Pairing Simulator returns `no such column: male` when clicking **Simulate Pairing**. ## Root Cause In `server/routes/pedigree.js` (lines 164–165), the `trial-pairing` POST handler used **double quotes** around the string literals `"male"` and `"female"` in the SQL queries: ```js // BROKEN db.prepare('SELECT * FROM dogs WHERE id = ? AND sex = "male"').get(sire_id); db.prepare('SELECT * FROM dogs WHERE id = ? AND sex = "female"').get(dam_id); ``` SQLite follows the SQL standard where **double quotes denote identifiers** (column/table names), not string values. So SQLite interpreted `"male"` as a column name, throwing `no such column: male`. ## Fix Changed to single-quoted string literals, which SQLite correctly treats as values: ```js // FIXED db.prepare("SELECT * FROM dogs WHERE id = ? AND sex = 'male'").get(sire_id); db.prepare("SELECT * FROM dogs WHERE id = ? AND sex = 'female'").get(dam_id); ``` ## Files Changed - `server/routes/pedigree.js` — lines 164–165 only

jason added 1 commit 2026-03-10 14:38:33 -05:00

fix: use single quotes for sex string literals in trial-pairing SQL (SQLite double-quote = identifier) 20fcc39a58

jason merged commit c7c0ec6530 into master

2026-03-10 14:39:17 -05:00

jason referenced this issue from a commit

2026-03-10 14:39:19 -05:00

Merge pull request 'fix: Trial Pairing Simulator — correct SQLite string quoting for sex filter' (#43) from fix/trial-pairing-sex-quote into master

jason referenced this pull request

2026-03-10 14:44:50 -05:00

fix: correct Wright COI algorithm — handle direct parent-offspring pairings #44

Sign in to join this conversation.