diff --git modules/comment/comment.admin.inc modules/comment/comment.admin.inc index 738cc07..85dd24e 100644 --- modules/comment/comment.admin.inc +++ modules/comment/comment.admin.inc @@ -72,26 +72,37 @@ function comment_admin_overview($form, &$form_state, $arg) { $query->addField('u', 'name', 'registered_name'); $query->addField('n', 'title', 'node_title'); $result = $query - ->fields('c', array('subject', 'nid', 'cid', 'comment', 'changed', 'status', 'name', 'homepage')) + ->fields('c', array('subject', 'nid', 'cid', 'changed', 'status', 'name', 'homepage')) ->fields('u', array('uid')) ->condition('c.status', $status) ->limit(50) ->orderByHeader($header) ->execute(); - + + // We need to load the comment object to get the comment_body field which is + // stored in its own table. To do this, get an array of cids. + foreach ($result as $comment) { + $comments[] = $comment; + $cid[] = $comment->cid; + } + $comment_objects = comment_load_multiple($cid); // Build a table listing the appropriate comments. $options = array(); $destination = drupal_get_destination(); - foreach ($result as $comment) { + foreach ($comments as $comment) { + // Retrieve the first comment object in the array and add the comment_body + // field to the result from the database query. + $comment_object = array_shift($comment_objects); + $comment->comment_body = $comment_object->comment_body; $options[$comment->cid] = array( 'subject' => array( 'data' => array( '#type' => 'link', '#title' => $comment->subject, '#href' => 'comment/' . $comment->cid, - '#options' => array('attributes' => array('title' => truncate_utf8($comment->comment, 128)), 'fragment' => 'comment-' . $comment->cid), + '#options' => array('attributes' => array('title' => truncate_utf8($comment->comment_body['zxx'][0]['value'], 128)), 'fragment' => 'comment-' . $comment->cid), ), ), 'author' => theme('username', array('account' => $comment)), @@ -160,7 +171,7 @@ function comment_admin_overview_submit($form, &$form_state) { // Allow modules to respond to the updating of a comment. module_invoke_all('comment_' . $form_state['values']['operation'], $comment); // Add an entry to the watchdog log. - watchdog('content', 'Comment: updated %subject.', array('%subject' => $comment->subject), WATCHDOG_NOTICE, l(t('view'), 'node/' . $comment->nid, array('fragment' => 'comment-' . $comment->cid))); + watchdog('content', 'Comment: updated %cid.', array('%cid' => $comment->cid), WATCHDOG_NOTICE, l(t('view'), 'node/' . $comment->nid, array('fragment' => 'comment-' . $comment->cid))); } } cache_clear_all(); @@ -193,7 +204,7 @@ function comment_multiple_delete_confirm($form, &$form_state) { $comment = comment_load($cid); if (is_object($comment) && is_numeric($comment->cid)) { $subject = db_query('SELECT subject FROM {comment} WHERE cid = :cid', array(':cid' => $cid))->fetchField(); - $form['comments'][$cid] = array('#type' => 'hidden', '#value' => $cid, '#prefix' => '
  • ', '#suffix' => check_plain($subject) . '
  • '); + $form['comments'][$cid] = array('#type' => 'hidden', '#value' => $cid); $comment_counter++; } } @@ -235,7 +246,7 @@ function comment_confirm_delete($form, &$form_state, $comment) { $form['#comment'] = $comment; return confirm_form( $form, - t('Are you sure you want to delete the comment %title?', array('%title' => $comment->subject)), + t('Are you sure you want to delete the comment by %name', array('%name' => $comment->name)), 'node/' . $comment->nid, t('Any replies to this comment will be lost. This action cannot be undone.'), t('Delete'), diff --git modules/comment/comment.install modules/comment/comment.install index a83885c..4b0f545 100644 --- modules/comment/comment.install +++ modules/comment/comment.install @@ -10,6 +10,9 @@ * Implement hook_uninstall(). */ function comment_uninstall() { + // Delete comment_body field. + field_delete_field('comment_body'); + // Remove variables. variable_del('comment_block_count'); $node_types = array_keys(node_type_get_types()); @@ -43,6 +46,19 @@ function comment_enable() { db_insert('node_comment_statistics') ->from($query) ->execute(); + + // Create comment body field. + // @todo this should be done in comment_install, but causes exceptions + // in simpletest due to dependencies between entity_get_info() and + // taxonomy_install(). + $field = array( + 'field_name' => 'comment_body', + 'type' => 'text_long', + ); + field_create_field($field); + + // Create instances on all existing node types. + node_types_rebuild(); } /** @@ -207,6 +223,92 @@ function comment_update_7010() { db_add_index('node_comment_statistics', 'cid', array('cid')); } + /** + * Create the comment_body field. + */ +function comment_update_7012() { + $ret = array(); + // Create comment body field. + $field = array( + 'field_name' => 'comment_body', + 'type' => 'text_long', + ); + field_create_field($field); + + // Add the field to comments for all existing bundles. + $body_instance = array( + 'field_name' => 'comment_body', + 'label' => 'Body', + 'object_type' => 'comment', + 'settings' => array('text_processing' => 1), + // Hide field labels by default. + 'display' => array( + 'full' => array( + 'label' => 'hidden', + ), + ), + ); + foreach (node_type_get_types() as $info) { + $body_instance['bundle'] = 'comment_node_' . $info->type; + field_create_instance($body_instance); + } + + return array(); +} + +/** + * Migrate data from the comment field to field storage. + */ +function comment_update_7013(&$context) { + $ret = array(); + + // This is a multipass update. First set up some comment variables. + if (empty($context['total'])) { + $comments = (bool) db_query_range('SELECT 1 FROM {comment}', 0, 1)->fetchField(); + if (!$comments) { + // The update will finish when $context['types'] is an empty array. + // Since we have nothing to do if there's no comments on the site, except + // drop the columns, set that here for convenience. + $context['types'] = array(); + } + else { + $context['etid'] = _field_sql_storage_etid('comment'); + $context['types'] = node_type_get_types(); + } + $context['total'] = count($context['types']); + } + if (!empty($context['types'])) { + $type = array_shift($context['types']); + + $query = db_select('comment', 'c'); + $query->innerJoin('node', 'n', 'c.nid = n.nid AND n.type = :type', array(':type' => $type->type)); + $query->addField('c', 'cid', 'entity_id'); + $query->addExpression("'comment_node_$type->type'", 'bundle'); + $query->addExpression($context['etid'], 'etid'); + $query->addExpression('0', 'deleted'); + $query->addExpression("'" . FIELD_LANGUAGE_NONE . "'", 'language'); + $query->addExpression('0', 'delta'); + $query->addField('c', 'comment', 'comment_body_value'); + $query->addField('c', 'format', 'comment_body_format'); + + $comment_body = field_info_field('comment_body'); + $comment_body_table = _field_sql_storage_tablename($comment_body); + + db_insert($comment_body_table) + ->from($query) + ->execute(); + } + + // On the last pass of the update, $context['types'] will be empty. + if (empty($context['types'])) { + db_drop_field($ret, 'comment', 'subject'); + } + + $ret['#finished'] = 1 - count($context['types']) / $context['total']; + + return $ret; +} + /** * @} End of "defgroup updates-6.x-to-7.x" * The next series of updates should start at 8000. @@ -249,12 +351,6 @@ function comment_schema() { 'default' => '', 'description' => 'The comment title.', ), - 'comment' => array( - 'type' => 'text', - 'not null' => TRUE, - 'size' => 'big', - 'description' => 'The comment body.', - ), 'hostname' => array( 'type' => 'varchar', 'length' => 128, @@ -282,13 +378,6 @@ function comment_schema() { 'size' => 'tiny', 'description' => 'The published status of a comment. (0 = Not Published, 1 = Published)', ), - 'format' => array( - 'type' => 'int', - 'size' => 'small', - 'not null' => TRUE, - 'default' => 0, - 'description' => 'The {filter_format}.format of the comment body.', - ), 'thread' => array( 'type' => 'varchar', 'length' => 255, diff --git modules/comment/comment.module modules/comment/comment.module index 1bdbd60..8fc0d81 100644 --- modules/comment/comment.module +++ modules/comment/comment.module @@ -236,6 +236,22 @@ function comment_count_unpublished() { */ function comment_node_type_insert($info) { field_attach_create_bundle('comment', 'comment_node_' . $info->type); + + // Attach the body field by default. + $instance = array( + 'field_name' => 'comment_body', + 'label' =>'Body', + 'object_type' => 'comment', + 'bundle' => 'comment_node_' . $info->type, + 'settings' => array('text_processing' => 1), + // Hide field labels by default. + 'display' => array( + 'full' => array( + 'label' => 'hidden', + ), + ), + ); + field_create_instance($instance); } /** @@ -456,6 +472,7 @@ function comment_new_page_count($num_comments, $new_replies, $node) { function theme_comment_block() { $items = array(); $number = variable_get('comment_block_count', 10); + foreach (comment_get_recent($number) as $comment) { $items[] = l($comment->subject, 'comment/' . $comment->cid, array('fragment' => 'comment-' . $comment->cid)) . '
    ' . t('@time ago', array('@time' => format_interval(REQUEST_TIME - $comment->changed))); } @@ -825,11 +842,6 @@ function comment_build_content($comment, $node, $build_mode = 'full') { // Remove previously built content, if exists. $comment->content = array(); - // Build comment body. - $comment->content['comment_body'] = array( - '#markup' => check_markup($comment->comment, $comment->format, '', TRUE), - ); - // Build fields content. field_attach_prepare_view('comment', array($comment->cid => $comment), $build_mode); $comment->content += field_attach_view('comment', $comment, $build_mode); @@ -1146,17 +1158,14 @@ function comment_node_delete($node) { * Implement hook_node_update_index(). */ function comment_node_update_index($node) { - $text = ''; - if ($node->comment != COMMENT_NODE_HIDDEN) { - $comments = db_query('SELECT subject, comment, format FROM {comment} WHERE nid = :nid AND status = :status', array( - ':nid' => $node->nid, - ':status' => COMMENT_PUBLISHED - )); - foreach ($comments as $comment) { - $text .= '

    ' . check_plain($comment->subject) . '

    ' . check_markup($comment->comment, $comment->format, '', TRUE); - } + $mode = variable_get('comment_default_mode_' . $node->type, COMMENT_MODE_THREADED); + $comments_per_page = variable_get('comment_default_per_page_' . $node->type, 50); + if ($node->comment && $cids = comment_get_thread($node, $mode, $comments_per_page)) { + $comments = comment_load_multiple($cids); + comment_prepare_thread($comments); + $build = comment_build_multiple($comments, $node); } - return $text; + return drupal_render($build); } /** @@ -1275,8 +1284,6 @@ function comment_save($comment) { 'created' => $comment->created, 'changed' => $comment->changed, 'subject' => $comment->subject, - 'comment' => $comment->comment, - 'format' => $comment->comment_format, 'uid' => $comment->uid, 'name' => $comment->name, 'mail' => $comment->mail, @@ -1289,7 +1296,7 @@ function comment_save($comment) { // Allow modules to respond to the updating of a comment. module_invoke_all('comment_update', $comment); // Add an entry to the watchdog log. - watchdog('content', 'Comment: updated %subject.', array('%subject' => $comment->subject), WATCHDOG_NOTICE, l(t('view'), 'comment/' . $comment->cid, array('fragment' => 'comment-' . $comment->cid))); + watchdog('content', 'Comment: updated %cid.', array('%subject' => $comment->cid), WATCHDOG_NOTICE, l(t('view'), 'comment/' . $comment->cid, array('fragment' => 'comment-' . $comment->cid))); } else { // Add the comment to database. This next section builds the thread field. @@ -1351,8 +1358,6 @@ function comment_save($comment) { 'pid' => empty($comment->pid) ? 0 : $comment->pid, 'uid' => $comment->uid, 'subject' => $comment->subject, - 'comment' => $comment->comment, - 'format' => $comment->comment_format, 'hostname' => ip_address(), 'created' => $comment->created, 'changed' => $comment->changed, @@ -1418,6 +1423,7 @@ function comment_delete_multiple($cids) { comment_delete_multiple($child_cids); _comment_update_node_statistics($comment->nid); } + //parent::attachLoad($comments); } } @@ -1828,15 +1834,6 @@ function comment_form($form, &$form_state, $comment) { $default = ''; } - $form['comment'] = array( - '#type' => 'textarea', - '#title' => t('Comment'), - '#rows' => 15, - '#default_value' => $default, - '#text_format' => isset($comment->format) ? $comment->format : filter_default_format(), - '#required' => TRUE, - ); - $form['cid'] = array( '#type' => 'value', '#value' => !empty($comment->cid) ? $comment->cid : NULL, @@ -1911,8 +1908,7 @@ function comment_preview($comment) { $node = node_load($comment->nid); if (!form_get_errors()) { - $comment->format = $comment->comment_format; - + $comment->format = $comment->comment_body['zxx'][0]['format']; // Attach the user and time information. if (!empty($comment->author)) { $account = user_load_by_name($comment->author); @@ -2041,13 +2037,15 @@ function comment_submit($comment) { // 1) Filter it into HTML // 2) Strip out all HTML tags // 3) Convert entities back to plain-text. - $comment['subject'] = truncate_utf8(trim(decode_entities(strip_tags(check_markup($comment['comment'], $comment['comment_format'])))), 29, TRUE); + + $comment['subject'] = truncate_utf8(trim(decode_entities(strip_tags(check_markup($comment['comment_body']['zxx'][0]['value'], $comment['comment_body']['zxx'][0]['format'])))), 29, TRUE); // Edge cases where the comment body is populated only by HTML tags will // require a default subject. if ($comment['subject'] == '') { $comment['subject'] = t('(No subject)'); } } + return (object)$comment; } @@ -2096,8 +2094,8 @@ function comment_form_submit($form, &$form_state) { $redirect = array('node/' . $node->nid, array('query' => $query, 'fragment' => 'comment-' . $comment->cid)); } else { - watchdog('content', 'Comment: unauthorized comment submitted or comment submitted to a closed post %subject.', array('%subject' => $comment->subject), WATCHDOG_WARNING); - drupal_set_message(t('Comment: unauthorized comment submitted or comment submitted to a closed post %subject.', array('%subject' => $comment->subject)), 'error'); + watchdog('content', 'Comment: unauthorized comment submitted or comment submitted to a closed post %title.', array('%title' => $node->title), WATCHDOG_WARNING); + drupal_set_message(t('Comment: unauthorized comment submitted or comment submitted to a closed post %title.', array('%title' => $node->title)), 'error'); // Redirect the user to the node they are commenting on. $redirect = 'node/' . $node->nid; } @@ -2123,6 +2121,7 @@ function template_preprocess_comment(&$variables) { $variables['picture'] = theme_get_setting('toggle_comment_user_picture') ? theme('user_picture', array('account' => $comment)) : ''; $variables['signature'] = $comment->signature; $variables['title'] = l($comment->subject, 'comment/' . $comment->cid, array('fragment' => "comment-$comment->cid")); + $variables['permalink'] = l('#', 'comment/' . $comment->cid, array('fragment' => "comment-$comment->cid")); $variables['template_files'][] = 'comment-' . $variables['node']->type; // Helpful $content variable for templates. @@ -2400,7 +2399,7 @@ function comment_unpublish_action($comment, $context = array()) { ->condition('cid', $cid) ->execute(); } - watchdog('action', 'Unpublished comment %subject.', array('%subject' => $subject)); + watchdog('action', 'Unpublished comment %cid.', array('%cid' => $cid));; } /** @@ -2417,9 +2416,10 @@ function comment_unpublish_action($comment, $context = array()) { */ function comment_unpublish_by_keyword_action($comment, $context) { foreach ($context['keywords'] as $keyword) { - if (strpos($comment->comment, $keyword) !== FALSE || strpos($comment->subject, $keyword) !== FALSE) { + $text = drupal_render($comment); + if (strpos($text, $keyword) !== FALSE) { $comment->status = COMMENT_NOT_PUBLISHED; - watchdog('action', 'Unpublished comment %subject.', array('%subject' => $comment->subject)); + watchdog('action', 'Unpublished comment %cid.', array('%cid' => $comment->cid)); break; } } @@ -2489,16 +2489,6 @@ function comment_menu_alter(&$items) { } /** - * Implement hook_filter_format_delete(). - */ -function comment_filter_format_delete($format, $fallback) { - db_update('comment') - ->fields(array('format' => $fallback->format)) - ->condition('format', $format->format) - ->execute(); -} - -/** * Implements hook_rdf_mapping(). */ function comment_rdf_mapping() { diff --git modules/comment/comment.pages.inc modules/comment/comment.pages.inc index e9acb81..9566622 100644 --- modules/comment/comment.pages.inc +++ modules/comment/comment.pages.inc @@ -108,7 +108,7 @@ function comment_reply($node, $pid = NULL) { */ function comment_approve($comment) { $comment->status = COMMENT_PUBLISHED; - $comment->comment_format = $comment->format; +// $comment->comment_format = $comment->format; comment_save($comment); drupal_set_message(t('Comment approved.')); diff --git modules/comment/comment.test modules/comment/comment.test index 3b1bab1..5b75d79 100644 --- modules/comment/comment.test +++ modules/comment/comment.test @@ -24,14 +24,14 @@ class CommentHelperCase extends DrupalWebTestCase { */ function postComment($node, $comment, $subject = '', $contact = NULL) { $edit = array(); - $edit['comment'] = $comment; - + $edit['comment_body[zxx][0][value]'] = $comment; + $preview_mode = variable_get('comment_preview_article', DRUPAL_OPTIONAL); $subject_mode = variable_get('comment_subject_field_article', 1); // Must get the page before we test for fields. if ($node !== NULL) { - $this->drupalGet('comment/reply/' . $node->nid); + $test = $this->drupalGet('comment/reply/' . $node->nid); } if ($subject_mode == TRUE) { @@ -44,6 +44,7 @@ class CommentHelperCase extends DrupalWebTestCase { if ($contact !== NULL && is_array($contact)) { $edit += $contact; } + switch ($preview_mode) { case DRUPAL_REQUIRED: // Preview required so no save button should be found. @@ -67,6 +68,7 @@ class CommentHelperCase extends DrupalWebTestCase { // Get comment ID preg_match('/#comment-([0-9]+)/', $this->getURL(), $match); + // Get comment. if ($contact !== TRUE) { // If true then attempting to find error message. if ($subject) { @@ -355,7 +357,7 @@ class CommentInterfaceTest extends CommentHelperCase { $this->assertTrue($this->node, t('Article node created.')); $this->drupalGet('comment/reply/' . $this->node->nid); $this->assertNoText('This discussion is closed', t('Posting to node with comments enabled')); - $this->assertField('edit-comment', t('Comment body field found.')); + $this->assertField('edit-comment-body-zxx-0-value', t('Comment body field found.')); // Delete comment and make sure that reply is also removed. $this->drupalLogout(); @@ -413,17 +415,17 @@ class CommentPreviewTest extends CommentHelperCase { $this->drupalLogin($this->web_user); $edit = array(); $edit['subject'] = $this->randomName(8); - $edit['comment'] = $this->randomName(16); + $edit['comment_body[zxx][0][value]'] = $this->randomName(16); $this->drupalPost('node/' . $this->node->nid, $edit, t('Preview')); // Check that the preview is displaying the title and body. $this->assertTitle(t('Preview comment | Drupal'), t('Page title is "Preview comment".')); $this->assertText($edit['subject'], t('Subject displayed.')); - $this->assertText($edit['comment'], t('Comment displayed.')); + $this->assertText($edit['comment_body[zxx][0][value]'], t('Comment displayed.')); // Check that the title and body fields are displayed with the correct values. $this->assertFieldByName('subject', $edit['subject'], t('Subject field displayed.')); - $this->assertFieldByName('comment', $edit['comment'], t('Comment field displayed.')); + $this->assertFieldByName('comment_body[zxx][0][value]', $edit['comment_body[zxx][0][value]'], t('Comment field displayed.')); } /** @@ -439,23 +441,23 @@ class CommentPreviewTest extends CommentHelperCase { $edit = array(); $edit['subject'] = $this->randomName(8); - $edit['comment'] = $this->randomName(16); + $edit['comment_body[zxx][0][value]'] = $this->randomName(16); $edit['author'] = $web_user->name; $edit['date'] = '2008-03-02 17:23 +0300'; $expected_date = format_date(strtotime($edit['date'])); - $comment = $this->postComment($this->node, $edit['subject'], $edit['comment'], TRUE); + $comment = $this->postComment($this->node, $edit['subject'], $edit['comment_body[zxx][0][value]'], TRUE); $this->drupalPost('comment/' . $comment->id . '/edit', $edit, t('Preview')); // Check that the preview is displaying the subject, comment, author and date correctly. $this->assertTitle(t('Preview comment | Drupal'), t('Page title is "Preview comment".')); $this->assertText($edit['subject'], t('Subject displayed.')); - $this->assertText($edit['comment'], t('Comment displayed.')); + $this->assertText($edit['comment_body[zxx][0][value]'], t('Comment displayed.')); $this->assertText($edit['author'], t('Author displayed.')); $this->assertText($expected_date, t('Date displayed.')); // Check that the title and body fields are displayed with the correct values. $this->assertFieldByName('subject', $edit['subject'], t('Subject field displayed.')); - $this->assertFieldByName('comment', $edit['comment'], t('Comment field displayed.')); + $this->assertFieldByName('comment_body[zxx][0][value]', $edit['comment_body[zxx][0][value]'], t('Comment field displayed.')); $this->assertFieldByName('author', $edit['author'], t('Author field displayed.')); $this->assertFieldByName('date', $edit['date'], t('Date field displayed.')); } @@ -572,6 +574,7 @@ class CommentAnonymous extends CommentHelperCase { $this->assertPattern('/
    ]*?)id="comments"([^>]*?)>/', t('Comments were displayed.')); $this->assertLink('Login', 1, t('Link to login was found.')); $this->assertLink('register', 1, t('Link to register was found.')); + } } diff --git modules/comment/comment.tpl.php modules/comment/comment.tpl.php index 483f689..1d410ef 100644 --- modules/comment/comment.tpl.php +++ modules/comment/comment.tpl.php @@ -18,6 +18,7 @@ * Preprocess functions can reformat it by calling format_date() with the * desired parameters on the $comment->changed variable. * - $new: New comment marker. + * - $permalink: Comment permalink. * - $picture: Authors picture. * - $signature: Authors signature. * - $status: Comment status. Possible values are: @@ -64,6 +65,7 @@ >