Skip to content

Commit 6df86c5

Browse files
authored
Merge pull request #124 from bluedynamics/fix/tika-enqueue-dict-row
fix: KeyError: 0 in Tika enqueue when cursor uses dict_row
2 parents 1430066 + 0b94836 commit 6df86c5

5 files changed

Lines changed: 1328 additions & 17 deletions

File tree

CHANGES.md

Lines changed: 16 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,21 @@
11
# Changelog
22

3+
## 1.0.0b52
4+
5+
### Fixed
6+
7+
- `CatalogStateProcessor._enqueue_tika_jobs` indexed result rows by
8+
integer position (`row[0]`, `row[1]`), but the request-scoped
9+
connection pool uses a `dict_row` factory, so every content save
10+
that produced an unresolved blob ref raised `KeyError: 0` during
11+
`tpc_vote` (e.g. uploading a Dexterity Image). Switched to
12+
column-name access.
13+
14+
Existing tests didn't catch this because the integration tests
15+
opened their cursor with `tuple_row` and the unit tests mocked
16+
`fetchall()` with tuple rows — both diverged from production.
17+
Tests updated to use `dict_row` to match the real pool.
18+
319
## 1.0.0b51
420

521
### Added

0 commit comments

Comments
 (0)