- πΊπΈUnited States brainiacque
Is this patch still required for Drupal 10? Composer will not apply this patch; however, I still get errors when building views with this module.
API page: https://api.drupal.org/api/drupal/core%21modules%21views%21src%21Plugin%...
I would suggest to make StylePluginBase::renderFields(array $hresult) public instead of protected.
The module Views Merge Rows is a display extender plugin that modify Views output by implementing hook_views_pre_render()
. As the module needs to access $view->style_plugin->rendered_fields
during pre-rendering process, and because $view->style_plugin->rendered_fields
is not rendered prior to the execution ofhook_views_pre_render
, invoking $view->style_plugin->renderFields($view->result)
is made necessary.
The access in the code is as follows:
$view->style_plugin->renderFields($view->result);
$rendered_fields = $view->style_plugin->getRenderedFields();
Prior to the proposed solution, creating a derived class of StylePluginBase was considered. However, the required property would not spread throughout all style plugins. The latter is pre-requisite because Views Merge Rows is a display extender intended to work with any style plugin.
Needs work
11.0 π₯
Last updated
Issue summaries save everyone time if they are kept up-to-date. See Update issue summary task instructions.
It denotes an issue that prevents porting of a contributed project to the stable version of Drupal due to missing APIs, regressions, and so on.
Not all content is available!
It's likely this issue predates Contrib.social: some issue and comment data are missing.
Is this patch still required for Drupal 10? Composer will not apply this patch; however, I still get errors when building views with this module.