- ๐บ๐ธUnited States smustgrave
Thank you for creating this issue to improve Drupal.
We are working to decide if this task is still relevant to a currently supported version of Drupal. There hasn't been any discussion here for over 8 years which suggests that this has either been implemented or is no longer relevant. Your thoughts on this will allow a decision to be made.
Since we need more information to move forward with this issue, the status is now Postponed (maintainer needs more info). If we don't receive additional information to help with the issue, it may be closed after three months.
Thanks!
- ๐บ๐ธUnited States smustgrave
Thank you for creating this issue to improve Drupal.
We are working to decide if this task is still relevant to a currently supported version of Drupal. There hasn't been any discussion here for over 8 years which suggests that this has either been implemented or is no longer relevant. Your thoughts on this will allow a decision to be made.
Since we need more information to move forward with this issue, the status is now Postponed (maintainer needs more info). If we don't receive additional information to help with the issue, it may be closed after three months.
Thanks!
- ๐บ๐ธUnited States smustgrave
Maybe, may be a change that canโt be backported
- ๐ซ๐ทFrance prudloff Lille
Well, the issue is about making sure that adding a relationship to a translatable entities does not cause duplicate rows.
The test was previously expecting duplicates so it had to be updated to only expect one instance of each entity.This might be a breaking change though, if people rely on the view returning duplicates.
- ๐บ๐ธUnited States smustgrave
Not 100% sure about the changes to core/modules/views/tests/src/Kernel/Entity/RowEntityRenderersTest.php isn't this updating the tests to make it pass?
- ๐บ๐ธUnited States smustgrave
Thank you for creating this issue to improve Drupal.
We are working to decide if this task is still relevant to a currently supported version of Drupal. There hasn't been any discussion here for over 8 years which suggests that this has either been implemented or is no longer relevant. Your thoughts on this will allow a decision to be made.
Since we need more information to move forward with this issue, the status is now Postponed (maintainer needs more info). If we don't receive additional information to help with the issue, it may be closed after three months.
Thanks!
- ๐บ๐ธUnited States smustgrave
Thank you for creating this issue to improve Drupal.
We are working to decide if this task is still relevant to a currently supported version of Drupal. There hasn't been any discussion here for over 8 years which suggests that this has either been implemented or is no longer relevant. Your thoughts on this will allow a decision to be made.
Since we need more information to move forward with this issue, the status is now Postponed (maintainer needs more info). If we don't receive additional information to help with the issue, it may be closed after three months.
Thanks!
- ๐ซ๐ทFrance prudloff Lille
I rebased and added an additional fix for non-reverse relationships.
Tests are green and this fixes my use case but I have not manually tested the scenario from the issue summary. - First commit to issue fork.
- ๐จ๐ญSwitzerland sir_squall
This module work:
https://www.drupal.org/project/translatable_menu_link_uri โ - ๐ฌ๐งUnited Kingdom catch
There has been lots of profiling of core since and this hasn't shown up as a problem, closing out.