Warnings when using non-standard comment field formatter

Created on 23 May 2023, almost 2 years ago

Updated 25 May 2023, almost 2 years ago

I created a comment field formatter that doesn't load and render the whole comment thread, but when using it warnings are generated:

Warning: Undefined array key "comments" in comment_preprocess_field() (line 724 of core/modules/comment/comment.module).

Create a comment field formatter that doesn't provide the comments or comment_form variable to the field template.

Add checks to comment_preprocess_field to make sure the array indexes it's accessing actually exist.

🐛 Bug report

Status

Needs work

Version

9.5

Component

Comment →

Last updated 9 days ago

Created by

🇧🇪Belgium dieterholvoet Brussels

Live updates comments and jobs are added and updated live.

Needs tests
The change is currently missing an automated test that fails when run with the original code, and succeeds when the bug has been fixed.

Comments & Activities

Issue created by @dieterholvoet
Open on Drupal.org →
Environment: PHP 8.1 & MySQL 5.7
last update almost 2 years ago
Not currently mergeable.
@dieterholvoet opened merge request.
Open in Jenkins → Open on Drupal.org →
Environment: PHP 8.1 & MySQL 5.7
last update almost 2 years ago
30,334 pass
Status changed to Needs review almost 2 years ago4:13pm 23 May 2023
Comment almost 2 years ago →
🇧🇪Belgium dieterholvoet Brussels
Status changed to Needs work almost 2 years ago10:01pm 25 May 2023
Comment almost 2 years ago →
🇺🇸United States smustgrave
As a bug will need a test showing the bug.

Would consider updating MR to 11.x as that's the current development branch. Code will be merged there first and backported.

Production build 0.71.5 2024