Follow-up to #2492585: Deprecate comment_view() & comment_view_multiple()

Problem/Motivation

Comment points to the function that not related to threading

Proposed resolution

Comment should point the place where threading actually used

Remaining tasks

review, commit

User interface changes

no

API changes

no

Beta phase evaluation

-->

Reference: https://www.drupal.org/core/beta-changes
Issue category Task because clean-up comment
Issue priority Normal because clarifies usage of API
Unfrozen changes Unfrozen because it only changes documentation in code
Disruption Non-disruptive
CommentFileSizeAuthor
#1 2526084-1.patch652 bytesandypost
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

andypost’s picture

andypost
he/him
Primary language Russian
CreditAttribution: andypost at Skilld commented
Issue summary: View changes
Status: Active » Needs review
FileSize
2526084-1.patch652 bytes

patch from parent issue

jibran’s picture

jibran
he/him
Primary language Urdu
Location Sydney, Australia
CreditAttribution: jibran as a volunteer commented
Status: Needs review » Reviewed & tested by the community
larowlan’s picture

larowlan
Location 🇦🇺🏝.au GMT+10
CreditAttribution: larowlan as a volunteer commented

+1 rtbc

xjm’s picture

xjm
she/her
Primary language English
CreditAttribution: xjm at Acquia commented
Status: Reviewed & tested by the community » Fixed

This issue only changes documentation, so per https://www.drupal.org/core/beta-changes, this can be completed any time during the Drupal 8 beta phase.
Committed and pushed to 8.0.x. Also added credit for @yarik.lutsiuk to the commit message from the change in the original issue. Thanks @andypost!

  • xjm committed 52b3deb on 8.0.x 
    Issue #2526084 by andypost, yarik.lutsiuk: Fix comment in Comment entity

Status: Fixed » Closed (fixed)

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