Update MediaFileRedirect::getUrl() to match Linkit

Created on 15 October 2024, 6 months ago

Problem/Motivation

The Linkit module has changed the function signature of MediaFileRedirect::getUrl() to accept an optional array parameter $options. This parameter is missing in media_entity_file_redirect's implementation of the function, causing a fatal error.

Proposed resolution

Update the function signature.

Remaining tasks

Post a patch.

🐛 Bug report

Status

Needs review

Version

1.0

Component

Code

Created by

🇺🇸United States i.mcbride

Live updates comments and jobs are added and updated live.

Comments & Activities

Issue created by @i.mcbride
Comment 6 months ago →
🇺🇸United States i.mcbride
Status changed to Needs work 2 months ago9:00pm 3 February 2025
Comment 2 months ago →
🇺🇸United States bkosborne New Jersey, USA
Are you sure? I just looked thru the code in 7.x version of the module and I never see ->getUrl() invoked with that param.
Comment 14 days ago →
🇺🇸United States bkosborne New Jersey, USA

contrib.social Blog FAQ Discussions

Production build 0.71.5 2024