Skip to content

Audit fixes (2026-09) - #5

Merged
luthermonson merged 1 commit into
mainfrom
fix/audit-2026-09
Sep 11, 2026
Merged

luthermonson merged 1 commit into
mainfrom
fix/audit-2026-09

Conversation

@luthermonson

Copy link
Copy Markdown
Contributor

Audited DB-bridge fixes for db-wordpress. Adopts the newer ephpm_db_* natives (#259-#263: run/columns/in_transaction) behind function_exists (v0.6.3 floor preserved), fixes the \'->'' escaping on the tenant path where present, and refreshes docs/CI. Tests added/updated (agent ran suites green in a php:8.3 container).

… #262/#263)

- Add run() to DbOpsInterface (unified execute-and-report shape), implemented natively by SapiDbOps (behind function_exists('ephpm_db_run')) and by the pdo_sqlite test double.
- Db::_do_query() now calls run() once instead of hand-rolling first-keyword routing between ephpm_db_query()/ephpm_db_execute(); has_rowset is read from the executed statement.
- Column names come from the statement metadata (ephpm_db_run/ephpm_db_columns), so get_col_info() works after a zero-row SELECT — removing the documented empty-rowset limitation. Updated its test.
- SapiDbOps keeps the older-ePHPm floor via a query()/execute()+columns() fallback; the translation layer is untouched.
- Refresh current-release string v0.8.6 -> v0.10.2.
@luthermonson
luthermonson merged commit d52eae7 into main Sep 11, 2026
3 checks passed
@luthermonson
luthermonson deleted the fix/audit-2026-09 branch September 11, 2026 06:59
@ephpm

ephpm Bot commented Sep 11, 2026 •

Copy link
Copy Markdown

ePHPm Preview — removed

Preview deployment has been torn down.

@ephpm
ephpm Bot temporarily deployed to preview-pr-5 September 11, 2026 07:00 Inactive
@ephpm
ephpm Bot temporarily deployed to preview-pr-5 September 11, 2026 07:00 Inactive

This branch was previously deployed

1 inactive deployment
preview-pr-5 — 64b6a51b Deployed Sep 11, 2026 by ephpm[bot]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant