Skip to content

Remove INET_ATON WordPress dependency - #485

Merged
JanJakes merged 1 commit into
trunkfrom
agent/inet-aton-wordpress-dependency
Aug 11, 2026
Merged

Remove INET_ATON WordPress dependency#485
JanJakes merged 1 commit into
trunkfrom
agent/inet-aton-wordpress-dependency

Conversation

@JanJakes

Copy link
Copy Markdown
Member

Summary

Removes the driver's dependency on WordPress's absint() helper in INET_ATON(). The replacement uses native PHP logic, allowing the Composer package to run in standalone PHP applications.

A regression test covers the function through the public PDO-compatible API. The package test suite and coding-standards check pass.

Why

The standalone Composer package autoloads the driver outside WordPress. Calling INET_ATON() previously failed because absint() is not available in standalone PHP applications.

Use native PHP integer handling for the SQLite callback and cover the standalone Composer driver query with a regression test.
@JanJakes JanJakes mentioned this pull request Aug 11, 2026
14 tasks
@JanJakes
JanJakes marked this pull request as ready for review August 11, 2026 15:22
@JanJakes

Copy link
Copy Markdown
Member Author

This literally inlines what WP does, so I'll go ahead and merge.

@JanJakes
JanJakes merged commit fc3a703 into trunk Aug 11, 2026
24 checks passed
@JanJakes
JanJakes deleted the agent/inet-aton-wordpress-dependency branch August 11, 2026 19:05
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