Problem/Motivation

grep -in variable_get -d recurse *
views/ajax_comments_handler_field_list_comments.inc:54: $mode = variable_get('comment_default_mode_' . $node->type, COMMENT_MODE_THREADED);
views/ajax_comments_handler_field_list_comments.inc:55: $comments_per_page = variable_get('comment_default_per_page_' . $node->type, 150);
views/ajax_comments_handler_field_list_comments.inc:69: && (variable_get('comment_form_location_' . $node->type, COMMENT_FORM_BELOW) == COMMENT_FORM_BELOW

Steps to reproduce

Proposed resolution

Remaining tasks

User interface changes

API changes

Data model changes

CommentFileSizeAuthor
#5 trailing_whitespace_ajax_comments.png38.95 KBguilhermevp
#4 3213248-remove-variable_get-ajax_comments.patch1.74 KBsocialnicheguru
#2 3213238-remove-variable_get.patch29.27 KBsocialnicheguru

Comments

SocialNicheGuru created an issue. See original summary.

socialnicheguru’s picture

socialnicheguru commented
StatusFileSize
new 3213238-remove-variable_get.patch29.27 KB
socialnicheguru’s picture

socialnicheguru commented
Status: Active » Needs review
socialnicheguru’s picture

socialnicheguru commented
StatusFileSize
new 3213248-remove-variable_get-ajax_comments.patch1.74 KB
guilhermevp’s picture

guilhermevp commented
Status: Needs review » Needs work
StatusFileSize
new trailing_whitespace_ajax_comments.png38.95 KB

Patch works as intended but there are trailing whitespaces in the patch that should be removed.

1

qzmenko’s picture

qzmenko
Primary language Russian
Location Novosibirsk
commented

This view handlers not used in D8/9 versions of module. Was removed in #3081873: Drupal 9 Deprecated Code Report

qzmenko’s picture

qzmenko
Primary language Russian
Location Novosibirsk
commented
Status: Needs work » Fixed

Status: Fixed » Closed (fixed)

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