Node revision UI will sometimes show no revisions on a page

Regarding #33, using the node language will be the correct language once the loaded node is correct for the interface language per 🐛 Revisions log on translated nodes should not show original language revisions, should show revisions of translated content RTBC

The Needs Review Queue Bot → tested this issue.

While you are making the above changes, we recommend that you convert this patch to a merge request → . Merge requests are preferred over patches. Be sure to hide the old patch files as well. (Converting an issue to a merge request without other contributions to the issue will not receive credit.)

samit.310@gmail.com → made their first commit to this issue’s fork.

romixua → changed the visibility of the branch 2916172-node-revision-ui to hidden.

By adding addMetaData('langcode', $langcode) we fix this issue and provide correct check by nodes grants

Issue summary appears incomplete, recommend using the standard issue template.

Also can a test case be added for this scenario to show the problem.

This is a duplicate of 📌 Consider showing all revisions on the revision overview Active I'll close this one as work has been started over there with a more fleshed out solution.

@acbramley I do not agree that #2722307 is a better solution. There the behavior has completely changed and not only the affected revisions are displayed.

With patch provided by @falco010 in #27, it is only filtering at the query level, but the same revisions are displayed as before, but we don't have empty pages anymore.

@romixua your suggestion did not work, I tried to apply the patch your provided in the MR, but I will have empty pages.

Hi @lisotton I understand where you're coming from but:
1. That issue also includes the langcode filter
2. There's no reason to have 2 issues open that touch the same code in similar ways. If we find in that issue that we don't want to remove the affected revisions check then we will remove it there, meaning it is exactly the same as this issue. Having both issues open is just splitting effort and making it confusing for contributors.

We should focus efforts on a single issue and since this one has the least recent activity and the MR is not up to date I think we should still close this one.

Please provide testing and feedback on 📌 Consider showing all revisions on the revision overview Active .