Problem/Motivation

Paragraph summaries don't show. The issue was introduced along with the changes in #3601861: Automated Drupal 12 compatibility fixes for paragraphs 1.x-dev

Steps to reproduce

The .module file code contains this:

function template_preprocess_paragraphs_summary(&$variables) {
  \Drupal::service(ParagraphsThemeHooks::class)->preprocessParagraphsActions($variables);
}

But it should be this:

function template_preprocess_paragraphs_summary(&$variables) {
  \Drupal::service(ParagraphsThemeHooks::class)->preprocessParagraphsSummary($variables);
}

My paragraph summaries don't show on the most recent release.

Proposed resolution

Remaining tasks

User interface changes

API changes

Data model changes

CommentFileSizeAuthor
#6 paragraphs-3614739-m274.diff617 bytessolideogloria

Issue fork paragraphs-3614739

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

solideogloria created an issue. See original summary.

solideogloria opened merge request !274

solideogloria’s picture

solideogloria commented
Status: Active » Needs review
berdir’s picture

berdir
Primary language German
Location Switzerland
commented
Status: Needs review » Reviewed & tested by the community

Strange that previous major tests didn't catch this or did I not run them anymore in the end?

harkonn’s picture

harkonn commented

I can't answer your question @berdir, just a reminder that you mentioned not running some in #3601861-8: Automated Drupal 12 compatibility fixes for paragraphs 1.x-dev.
Can we please get a bugfix release for this? :)
Edit: Forgot to mention the fix is working as desired for me in 10.6.14.

solideogloria’s picture

solideogloria commented
StatusFileSize
new paragraphs-3614739-m274.diff617 bytes

Patch for ease of use in composer.patches.json.

berdir’s picture

berdir
Primary language German
Location Switzerland
commented
Status: Reviewed & tested by the community » Fixed

yes, but I definitely did run previous major tests at some point, but apparently not anymore after certain changes, awkward. Merged.

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.