The docblock from CommentController::getReplyForm() contains "an associative array containing" twice. I think it the second list should be removed.

   *   An associative array containing:
   *   - An array for rendering the entity or parent comment.
   *     - comment_entity: If the comment is a reply to the entity.
   *     - comment_parent: If the comment is a reply to another comment.
   *   - comment_form: The comment form as a renderable array.
   *   - An associative array containing:
   *     - An array for rendering the entity or parent comment.
   *        - comment_entity: If the comment is a reply to the entity.
   *        - comment_parent: If the comment is a reply to another comment.
   *     - comment_form: The comment form as a renderable array.

Beta phase evaluation

Reference: https://www.drupal.org/core/beta-changes
Issue category Bug: flaw in API documentation
Issue priority Normal
Unfrozen changes Unfrozen because it only changes documentation.
Prioritized changes none
Disruption None
CommentFileSizeAuthor
#1 2460479-1.patch1.04 KBAnonymous (not verified)

Comments

Anonymous’s picture

Status: Active » Needs review
StatusFileSize
new1.04 KB

And a patch.

Anonymous’s picture

Issue summary: View changes

Added beta eval in summary.

jhodgdon’s picture

Category: Task » Bug report
Issue summary: View changes
Status: Needs review » Reviewed & tested by the community

Wrong docs are actually a bug. Thanks for the issue and the patch!

alexpott’s picture

Status: Reviewed & tested by the community » Fixed

Committed 14470fe and pushed to 8.0.x. Thanks!

Thanks for adding the beta evaluation to the issue summary.

  • alexpott committed 14470fe on 8.0.x
    Issue #2460479 by pjonckiere: CommentController::getReplyForm() docblock...

Status: Fixed » Closed (fixed)

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