Include drupal:views as dependency b/c of lb_claro_theme_registry_alter

Created on 28 September 2022, about 2 years ago
Updated 16 February 2023, almost 2 years ago

Problem/Motivation

lb_claro_theme_registry_alter() relies upon Drupal core views module's hook_theme implementation to re-use its theme registry for the following new items

  $theme_registry['views_view__media_library__widget'] = $theme_registry['views_view'];
  $theme_registry['views_view__media_library__widget']['template'] = 'views-view--media-library';
  $theme_registry['views_view__media_library__widget']['path'] = $lb_claro_path;

  $theme_registry['views_view__media_library__widget_table'] = $theme_registry['views_view'];
  $theme_registry['views_view__media_library__widget_table']['template'] = 'views-view--media-library';
  $theme_registry['views_view__media_library__widget_table']['path'] = $lb_claro_path;

  $theme_registry['views_view_unformatted__media_library'] = $theme_registry['views_view_unformatted'];
  $theme_registry['views_view_unformatted__media_library']['template'] = 'views-view-unformatted--media-library';
  $theme_registry['views_view_unformatted__media_library']['path'] = $lb_claro_path;

Steps to reproduce

* Install Drupal using the minimal (without views module)
* Enable the LB Claro module
* Observe php notices

Proposed resolution

Add drupal:views as a module dependency to issue necessary theme registry items are present.

Remaining tasks

* Patch

User interface changes

API changes

Data model changes

πŸ› Bug report
Status

Needs work

Version

1.0

Component

Code

Created by

πŸ‡ΊπŸ‡ΈUnited States jasonawant New Orleans, USA

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Comments & Activities

Not all content is available!

It's likely this issue predates Contrib.social: some issue and comment data are missing.

Production build 0.71.5 2024