Get feed title in correct language

Updated the summary proposed resolution.

Since #39, several people have tested the patches. #49 also has been verified. So I propose we set this back to RTBC, so this issue can be closed finally.

The last submitted patch, 49: 2816447-49.patch, failed testing. View results →

Re-rolled #49 by removing the first code block which is completely unrelated and got removed from core by #3162827: Do not instantiate entity storages in constructors of services that do not always need them → - this should not be brought back and isn't required for this issue either. I guess this got in accidentally.

If the tests pass, this should be re-marked as RTBC

Seems to have some failures.

Created MR with patch from #58.
Let's see if the tests fail or not.

MR with patch from #58 and fixed remaining issues. Please review.

Added patch and interdiff.

Left comments on MR.

Fixed remarks.

Feedback appears to be addressed.

Thanks for fixing this issue.

@kriboogh, can you comment in the issue the reasoning for the changes you make? Sometime to commit message is enough to provide a clue, but here the message is always the same. It does help reviewer to know what it being done and why.

I left comments in the MR that need attention. This is adding deprecations so there should be a change record and the URL for that change record is to be in the deprecation message. And 10.4 is a maintenance minor → so I think this will be deprecated in 11.1.

I've tested the MR and it fixes the issue. Once MR comments are addressed it can go into RTBC.