[vsolovei-smartling]

Problem

With PublishPress Revisions active, translation uploads could fail before the file was sent, leaving the Smartling job batch empty. The submission failed with:

MetaFieldProcessorManager::getProcessor(): Argument #1 ($fieldName) must be of type string, array given

Root cause

PublishPressRevisions hooked sanitizeTargetField onto smartling_metadata_string_process at priority 5 and returned the field value. WordPress passes the first argument of a filter through the callback chain, and that argument is the field name, so MetaFieldProcessorManager::handlePostTranslationFields received the value as its name:

• array values (for example the revision meta _rvy_updated_by) raised a TypeError;
• scalar values silently selected the wrong processor;
• the post_mime_type reset never took effect.

The existing unit tests called sanitizeTargetField directly, so the real filter chain was never exercised.

Fix

• New smartling_target_entity_field filter (ExportedAPI::FILTER_SMARTLING_TARGET_ENTITY_FIELD) that takes the value first, then the field name, then the submission. SmartlingCoreTrait applies it after the metadata filter for each target entity field.
• PublishPressRevisions::sanitizeTargetField($value, $name, $submission) now hooks that filter and no longer touches the metadata filter.
• UploadJob logs upload failures at ERROR with submission ids, exception class and file:line (previously only a NOTICE).
• Changelog entry under 5.8.2 in readme.txt, and version bumped to 5.8.2 (composer.json, smartling-connector.php, readme.txt stable tag).

Tests

Written first and watched fail, including a filter-chain test that reproduces the TypeError. Full unit suite: 788 tests passing. Integration tests not run.

Jira: WP-1026

🤖 Generated with Claude Code