Skip to content

Embed preset DPI in rendered images with optional overrides - #3667

Merged
chubes4 merged 1 commit into
mainfrom
fix/issue-3665-data-machine
Oct 24, 2026
Merged

chubes4 merged 1 commit into
mainfrom
fix/issue-3665-data-machine

Conversation

@chubes4

@chubes4 chubes4 commented Oct 24, 2026

Copy link
Copy Markdown
Member

Closes #3665

Summary

Implemented and committed as 97e9eabb. All 25 standalone media tests, four required smokes, and scoped lint passed.

What changed

  • Added machine-readable DPI to print presets.
  • Applied preset resolution when saving PNG and JPEG images.
  • Added validated ability dpi and CLI --dpi overrides.
  • Added standalone tests for resolution metadata, defaults, overrides, and invalid inputs.

Compatibility

Non-print behavior, pixel dimensions, version strings, and CHANGELOG remain unchanged.

Verification

Gate gate-1 passed on b3ec767, base main at 1af0b24.

Gate command and provenance
set -e
[ -f vendor/autoload.php ] || composer install -q --no-interaction --no-progress
git fetch -q origin main || true
homeboy review lint data-machine --path "$PWD" --changed-since origin/main
EXT=$(homeboy extension show wordpress | grep -m1 '"path"' | sed -E 's/.*"path": "([^"]+)".*/\1/')
PU="$EXT/vendor/bin/phpunit"
for t in tests/Unit/Abilities/Media/*Test.php; do grep -q WP_UnitTestCase "$t" && continue; echo "== $t"; php "$PU" --bootstrap vendor/autoload.php "$t"; done
for f in tests/poster-template-smoke.php tests/abilities-image-template-load-order-smoke.php tests/image-template-file-urls-smoke.php tests/lightweight-ability-manifest-smoke.php tests/abilities-categories-load-order-smoke.php tests/abilities-send-email-load-order-smoke.php; do php "$f" >/dev/null || { echo "FAIL $f"; exit 1; }; done
echo "gate ok"
  • gate-1: commit b3ec767da686766291acbcfc9f8905ac615d7b7d, tree 608820663bedad86cbaa4e320ae43e562003e32f, fingerprint 2fec898efc4e7ec8caf30bbbdc374a2ad98af0abbad6626e5ba66c212b25d65b

AI disclosure: cook-heavy via Homeboy (pi). Used for: Producing print images with intended physical-size metadata.

@chubes4
chubes4 merged commit 0c6b75e into main Oct 24, 2026
20 of 26 checks passed
@chubes4
chubes4 deleted the fix/issue-3665-data-machine branch October 24, 2026 19:07
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.

image render: embed preset DPI in output files (print presets)

1 participant