>>> [!note] Migrated issue <!-- Drupal.org comment --> <!-- Migrated from issue #3589577. --> Reported by: [nick hope](https://www.drupal.org/user/28698) >>> <h3 id="summary-problem-motivation">Problem/Motivation</h3> <p>The Media Remote Image URL field can be configured to allow more than one value.</p> <p>Multiple remote image URLs can be saved on a media entity, and the first value is used for metadata/thumbnail purposes. This is broadly similar to core image media behavior, where the first image is used as the representative image.</p> <p>However, there are currently edge cases where multi-value remote image URL fields can cause validation warnings or fatal errors.</p> <p>One case occurs when an additional exposed field item is left empty.</p> <p>Another case occurs when a media item contains both internal and external image URLs, and the values are reordered so that the internal image becomes the first value.</p> <h4 id="summary-steps-reproduce">Steps to reproduce</h4> <p>Scenario 1: empty additional value</p> <ol> <li>Install Media Entity Remote Image.</li> <li>Edit the Media Remote Image URL field on the Remote image media type.</li> <li>Set the allowed number of values to Unlimited, or to a limited value greater than 1.</li> <li>Add or edit a Remote image media entity.</li> <li>Enter a valid URL in the first value.</li> <li>Leave an additional exposed URL value empty.</li> <li>Save the media entity.</li> </ol> <p>On Drupal 10, this can produce warnings from Drupal core's LinkWidget validation, followed by:</p> <pre> The URL field is required when the field is specified. </pre><p>On Drupal 11, this can result in a fatal error:</p> <pre> TypeError: Drupal\Component\Utility\Html::escape(): Argument #1 ($text) must be of type string, null given </pre><p>Scenario 2: reordering internal and external image values</p> <ol> <li>Configure the Media Remote Image URL field to allow multiple values.</li> <li>Configure the field to allow both internal and external links.</li> <li>Create a Remote image media entity with one external image URL and one internal image URL.</li> <li>Edit the media entity.</li> <li>Reorder the values so that the internal image becomes the first value.</li> <li>Save the media entity.</li> </ol> <p>This can result in a fatal error:</p> <pre> Error: Call to a member function addError() on null in Drupal\media_entity_remote_image\Plugin\media\Source\RemoteImage-&gt;getMetadata() </pre><h3 id="summary-proposed-resolution">Proposed resolution</h3> <p>Ensure that multi-value Remote Image URL fields are handled safely.</p> <p>Empty additional values should be ignored/cleaned up before validation reaches code paths that expect removed link field properties, such as the link title property, to exist.</p> <p>When metadata or thumbnails are generated from the first value, internal image URLs should be converted to a fetchable URL where appropriate, as is already done elsewhere in the module.</p> <p>The media source plugin should also avoid calling a null messenger service when metadata fetching fails.</p> <p>This should be tested alongside the Media Library add form work in #3367270 and WebP/AVIF support in #3589328.</p> <h3 id="summary-remaining-tasks">Remaining tasks</h3> <ul> <li>Add a fix.</li> <li>Test single-value Remote Image URL fields.</li> <li>Test multi-value Remote Image URL fields.</li> <li>Test empty additional values.</li> <li>Test reordering external and internal image values.</li> <li>Test with alt text required and not required.</li> <li>Test on Drupal 10 and Drupal 11.</li> </ul> <h3 id="summary-ui-changes">User interface changes</h3> <p>None expected.</p> <h3 id="summary-api-changes">API changes</h3> <p>None expected.</p> <h3 id="summary-data-model-changes">Data model changes</h3> <p>None expected.</p>