Problem/Motivation

This stems from the following issue: #1334830: Missing comment date
The feeds_comment_processor module uses the comment_save function from the drupal api comment.module. There are cases in which the $comment->hostname is passed into the comment_save function, but currently the hostname on a new comment is always reset to the ip_address().

Proposed resolution

Similar to the other comment fields, the $comment->hostname should be checked if it is set before setting a default with ip_address(). I will attach a proposed patch.

API changes

comment_save will not overwrite the hostname value if it is already set.

Comments

dooug’s picture

StatusFileSize
new525 bytes
dooug’s picture

dooug’s picture

dooug’s picture

StatusFileSize
new901 bytes

re-rolled to the latest dev. It is a very simple patch, it was somewhat confirmed here: http://drupal.org/node/1334830#comment-6934778. Please review!

dooug’s picture

Category: Task » Bug report
Issue summary: View changes

Status: Needs review » Closed (outdated)

Automatically closed because Drupal 7 security and bugfix support has ended as of 5 January 2025. If the issue verifiably applies to later versions, please reopen with details and update the version.