Problem/Motivation
The piece of code if ($view && $view->access($display_id) && $view->setDisplay($display_id) && $view->display_handler->getOption('use_ajax')) { needs some helpful documentation why we need this.
Proposed resolution
- Explain that there could be any view passed along to this controller
- Given it could be any way, people might have not intended them to be rendered outside of the context, for example a panel.
- Given that the view might not have access configured, even the panel has some.
- Document also that its highly recommended to always add access checking to views as well.
Comments
Comment #11
smustgrave commentedThank 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!