file_url filter does not produce translated media

jjose.quevedo → made their first commit to this issue’s fork.

Please review.

Here is a patch to solve the issue. Please take a look

The last submitted patch, 8: resolve_translation.patch, failed testing. View results →
- codesniffer_fixes.patch Interdiff of automated coding standards fixes only.

Updating the patch

Besides UrlExtractor there is also a UriExtractor services. Does it have to be updated same way?

Chi is right and I pushed the corresponding changes into the MR.

Updated test coverage (and fixed a failing test) to cover this new feature. I'd mark RTBC, but I wrote too much test coverage.

Changing constructor signature could break existing sites that extends these services. Though the should not be considered as public API.
Anyway, I propose we change this the next major release. I've just created 4.x branch.

I am thinking of making all TT services final to avoid such issues in feature.