diff --git a/src/MetadataExtractorMVP.php b/src/MetadataExtractorMVP.php index c25e105..b2244f5 100644 --- a/src/MetadataExtractorMVP.php +++ b/src/MetadataExtractorMVP.php @@ -308,8 +308,6 @@ class MetadataExtractorMVP } // 4. Update ARTESIA.FIELD.ASSET DESCRIPTION from filename subject_title - // TEMPORARILY DISABLED - Testing if this causes upload failure - /* if ($parsedFilename && !empty($parsedFilename['subject_title'])) { $description = $parsedFilename['subject_title']; foreach ($mvpFields as &$field) { @@ -324,11 +322,8 @@ class MetadataExtractorMVP } } } - */ - // 5. Force FERRERO.FIELD.STATE to "Local" for all uploads - // TEMPORARILY DISABLED - Testing if this causes upload failure - /* + // 5. Force FERRERO.FIELD.STATE to "Local" for all uploads (valid domain value) foreach ($mvpFields as &$field) { if ($field['id'] === 'FERRERO.FIELD.STATE') { if (isset($field['value']['value']['value'])) { @@ -340,7 +335,6 @@ class MetadataExtractorMVP break; } } - */ // 6. Update video technical metadata if available if (!empty($videoMetadata)) { @@ -348,28 +342,12 @@ class MetadataExtractorMVP // Let DAM analyze video files automatically - don't send technical metadata } - // 7. CREATOR field - TEMPORARILY DISABLED + // 7. CREATOR field - DISABLED (causes upload failure) // "Oliver Agency" is NOT in the domain values (1893 specific emails/usernames) // Valid values include: "ExternalAgency", "ExternalAgencynew" - // Commenting out until we confirm which value to use - /* - $creatorFieldIndex = null; - foreach ($mvpFields as $index => &$field) { - if ($field['id'] === 'FERRERO.MARKETING.CREATOR') { - $creatorFieldIndex = $index; - // Update existing field - if (isset($field['value']['value']['value'])) { - $field['value']['value']['value'] = 'ExternalAgency'; // Valid domain value - } elseif (isset($field['value']['value']['field_value']['value'])) { - $field['value']['value']['field_value']['value'] = 'ExternalAgency'; - } - error_log("MVP Extractor: Updated CREATOR to: ExternalAgency"); - break; - } - } - */ + // Leaving this field from master as-is to avoid validation errors - error_log("MVP Extractor: Field updates temporarily disabled for testing"); + error_log("MVP Extractor: Field updates applied - Description, State, Language"); return $mvpFields; } diff --git a/workflow_v3.php b/workflow_v3.php index 12a5ca3..7dbf48c 100644 --- a/workflow_v3.php +++ b/workflow_v3.php @@ -2954,6 +2954,7 @@ $envInfo = $configV3->getEnvironmentInfo();