Image links with multiple query parameters have ampersand encoded

Perhaps other also use svg_image, than you must perhaps use this patch: https://www.drupal.org/project/svg_image/issues/3337927#comment-14898139 🐛 Image links with multiple query parameters have ampersand encoded in REST view Fixed

Huh. I'm surprised we've even been using a #markup array there, instead of just using the same SerializedData object as immediately above. It looks like this is the only place in the entire module where a field formatter can return something that isn't wrapped in SerializedData.

(Git annotations show I wrote this part in the first few days while designing the module. My guess is that the plugin was only created in the first place to handle the case where alt and title attributes needed to be exported, and that this else branch was rarely used.)

Thanks for the patch! I'm going ahead and replacing that section with

      else {
        $elements[$delta] = [
          '#type' => 'data',
          '#data' => SerializedData::create($uri),
        ];
      }

for consistency, which from local testing seems to also fix it.

(Note for future reference: Maybe counterintuitively, the $uri variable here - from the moment it is retrieved from Entity::getFileUri() - is a string, not an instance of Drupal\Core\Url. Just noting that in case core changes that in the future and breaks this.)

Backporting to 2.0.x as well.

And pushed. Thanks all!

Automatically closed - issue fixed for 2 weeks with no activity.