? .cvsignore ? userpoints_nc_d7.patch ? userpoints_nc_d7_2.patch Index: userpoints_nc.info =================================================================== RCS file: /cvs/drupal-contrib/contributions/modules/userpoints_nc/userpoints_nc.info,v retrieving revision 1.1 diff -u -p -r1.1 userpoints_nc.info --- userpoints_nc.info 9 Jan 2009 07:46:35 -0000 1.1 +++ userpoints_nc.info 3 Sep 2010 16:18:22 -0000 @@ -3,4 +3,5 @@ name = Userpoints Nodes and Comments description = Grants userpoints for posting nodes or comments or moderating comments package = Userpoints dependencies[] = userpoints -core = 6.x +files[]=userpoints_nc.module +core = 7.x Index: userpoints_nc.js =================================================================== RCS file: userpoints_nc.js diff -N userpoints_nc.js --- /dev/null 1 Jan 1970 00:00:00 -0000 +++ userpoints_nc.js 3 Sep 2010 16:18:22 -0000 @@ -0,0 +1,20 @@ +// $Id$ +(function ($) { + Drupal.behaviors.userpointsNCNodeTypeFieldsetSummaries = { + attach: function (context) { + $('fieldset.userpoints-nc-nodetype-form', context).drupalSetSummary(function (context) { + if ($('#edit-userpoints-nc-points').val() > 0) { + var params = { + '@points': $('#edit-userpoints-nc-points', context).val(), + '@category': $('select#edit-userpoints-nc-category :selected', context).text() + }; + return Drupal.t('@points in category @category are assigned for posting content.', params); + } + else { + return Drupal.t('No points are assigned for posting content.'); + } + }); + } + }; + +})(jQuery); \ No newline at end of file Index: userpoints_nc.module =================================================================== RCS file: /cvs/drupal-contrib/contributions/modules/userpoints_nc/userpoints_nc.module,v retrieving revision 1.3.2.2 diff -u -p -r1.3.2.2 userpoints_nc.module --- userpoints_nc.module 2 Mar 2010 22:27:46 -0000 1.3.2.2 +++ userpoints_nc.module 3 Sep 2010 16:18:22 -0000 @@ -4,350 +4,374 @@ // Copyright 2005 Khalid Baheyeldin http://2bits.com -define('USERPOINTS_POST', 'userpoints_post_'); -define('USERPOINTS_POST_COMMENT', 'userpoints_post_comment'); -define('USERPOINTS_MODERATE_COMMENT', 'userpoints_moderate_comment'); -//define a variable to trigger the use of the v2bug please read drupal.org/node/183520 -define('USERPOINTS_USE_V2BUG', 'userpoints_use_v2bug'); - -function userpoints_nc_help($path, $arg) { - switch ($path) { - case 'admin/settings/userpoints_nc': - return t('UP: Some basic interfaces for userpoints, such as posting nodes, comments, ...etc.'); - } +function userpoints_nc_form_node_type_form_alter(&$form, &$form_state) { + $form['userpoints_nc'] = array( + '#type' => 'fieldset', + '#title' => t('Userpoints Options'), + '#collapsible' => TRUE, + '#collapsed' => FALSE, + '#access' => user_access('administer userpoints'), + '#weight' => 25, + '#group' => 'additional_settings', + '#attached' => array( + 'js' => array( + 'userpoints_nc' => drupal_get_path('module', 'userpoints_nc') . '/userpoints_nc.js', + ), + ), + '#attributes' => array('class' => array('userpoints-nc-nodetype-form')), + ); + + $type = $form['#node_type']->type; + $form['userpoints_nc']['userpoints_nc_points'] = array( + '#type' => 'textfield', + '#title' => t('!Points for posting content of this type', array_merge(userpoints_translation())), + '#default_value' => userpoints_nc_get_setting('points', $type), + '#size' => 5, + '#maxlength' => 5, + ); + + $form['userpoints_nc']['userpoints_nc_category'] = array( + '#type' => 'select', + '#title' => t('Assign !Points for posting content of this type to category', array_merge(userpoints_translation())), + '#options' => userpoints_get_categories(), + '#default_value' => userpoints_nc_get_setting('category', $type), + ); + + $form['userpoints_nc']['userpoints_nc_comment_points'] = array( + '#type' => 'textfield', + '#title' => t('!Points for posting a comment', userpoints_translation()), + '#default_value' => userpoints_nc_get_setting('comment_points', $type), + '#size' => 5, + '#maxlength' => 5, + ); + + $form['userpoints_nc']['userpoints_nc_comment_category'] = array( + '#type' => 'select', + '#title' => t('Assign !Points for posting a comment to category', array_merge(userpoints_translation())), + '#options' => userpoints_get_categories(), + '#default_value' => userpoints_nc_get_setting('comment_category', $type), + ); } +/** + * Implements hook_userpoints(). + */ function userpoints_nc_userpoints($op, $params = array()) { switch ($op) { case 'setting': - $group = 'node'; - $form[$group] = array( + $form['userpoints_nc'] = array( '#type' => 'fieldset', '#collapsible' => TRUE, '#collapsed' => TRUE, - '#title' => t('!Points for posting nodes', userpoints_translation()), + '#title' => t('Settings for posting nodes and comments'), ); - $form[$group][USERPOINTS_POST . '_undo_points_on_delete'] = array( + $form['userpoints_nc']['userpoints_nc_published_only'] = array( '#type' => 'checkbox', - '#title' => t('Take away !points on node delete', array_merge(userpoints_translation())), - '#default_value' => variable_get(USERPOINTS_POST . '_undo_points_on_delete', true), + '#title' => t('Only give points for published nodes and comments.', array_merge(userpoints_translation())), + '#default_value' => variable_get('userpoints_nc_published_only', TRUE), + '#description' => t('If checked, uses will only receive !points when their content is published and will be removed again when it is deleted or unpublished.', userpoints_translation()) ); - $tree = taxonomy_get_tree(userpoints_get_vid()); - $category_options = array(); - $category_options[0] = t(''); - if (!empty($tree)) { - foreach ($tree as $term) { - $category_options[$term->tid] = str_repeat('--', $term->depth) . $term->name; - } - } - foreach (node_get_types() as $type => $name) { - $form[$group][USERPOINTS_POST . $type] = array( - '#type' => 'textfield', - '#title' => t('!Points for posting a !node-name', array_merge(userpoints_translation(), array('!node-name' => $name->name))), - '#default_value' => variable_get(USERPOINTS_POST . $type, '0'), - '#size' => 5, - '#maxlength' => 5, - ); - - $form[$group][USERPOINTS_POST . $type . '_category'] = array( - '#type' => 'select', - '#title' => t('Assign !Points for posting a !node-name to category', array_merge(userpoints_translation(), array('!node-name' => $name->name))), - '#options' => $category_options, - '#default_value' => variable_get(USERPOINTS_POST . $type . '_category', 0), - ); - - } - $group = 'comment'; - $form[$group] = array( - '#type' => 'fieldset', - '#collapsible' => TRUE, - '#collapsed' => TRUE, - '#title' => t('!Points for posting comments', userpoints_translation()), - ); - $form[$group][USERPOINTS_POST_COMMENT] = array( + $form['userpoints_nc']['userpoints_nc_points'] = array( '#type' => 'textfield', - '#title' => t('!Points for posting a comment', userpoints_translation()), - '#default_value' => variable_get(USERPOINTS_POST_COMMENT, 0), + '#title' => t('!Points assigned by default for posting content.', array_merge(userpoints_translation())), + '#default_value' => userpoints_nc_get_setting('points'), '#size' => 5, '#maxlength' => 5, - ); + ); - $form[$group][USERPOINTS_POST_COMMENT . '_category'] = array( + $form['userpoints_nc']['userpoints_nc_category'] = array( '#type' => 'select', - '#title' => t('Assign !Points for posting a comment to category', array_merge(userpoints_translation())), - '#options' => $category_options, - '#default_value' => variable_get(USERPOINTS_POST_COMMENT . '_category', 0), - ); - - $form[$group][USERPOINTS_MODERATE_COMMENT] = array( + '#title' => t('Assign !Points by default for posting content to category', array_merge(userpoints_translation())), + '#options' => userpoints_get_categories(), + '#default_value' => userpoints_nc_get_setting('category'), + ); + $form['userpoints_nc']['userpoints_nc_comment_points'] = array( '#type' => 'textfield', - '#title' => t('!Points for moderating a comment', userpoints_translation()), - '#default_value' => variable_get(USERPOINTS_MODERATE_COMMENT, 0), + '#title' => t('!Points for posting a comment by default', userpoints_translation()), + '#default_value' => userpoints_nc_get_setting('comment_points'), '#size' => 5, '#maxlength' => 5, - ); + ); - $form[$group][USERPOINTS_MODERATE_COMMENT . '_category'] = array( + $form['userpoints_nc']['userpoints_nc_comment_category'] = array( '#type' => 'select', - '#title' => t('Assign !Points for posting a comment to category', array_merge(userpoints_translation())), - '#options' => $category_options, - '#default_value' => variable_get(USERPOINTS_MODERATE_COMMENT . '_category', 0), - ); - - $group = 'v2compatibility'; - $form[$group] = array( - '#type' => 'fieldset', - '#collapsible' => TRUE, - '#collapsed' => TRUE, - '#title' => t('userpoints_basic events version 2.14 Compatibility', userpoints_translation()), - '#description' => t('If you have used userpoints_basic prior to version 2.14 you may need to enable this fix.
- If you have not used userpoints_basic prior to v3 you do NOT need to (nor should) enable this fix.
- This fix only affects the behavior of how !points are handled when ownership of a node/comment - changes. In version 2 data on the original owner was not kept and a static variable was used to - store this information. Unfortunately people have experienced unusual side effects such as - !points moving from one owner to a completely random user. All !points accrued with version 3 will - not exhibit this behavior thus this fix is unnecessary. - Only check this box under the following conditions
- 1) You have existing !points granted by a userpoints version prior to version 3
- 2) If ownership changes on a node/comment you want points to follow ownership
- All !points accrued by version 3 will move ownership automatically this setting only affects !points accrued with v2 -
- 3) You have read Article 183520 on drupal.org and - understand that there could be unusual issues with ownership moves', userpoints_translation()), - ); - $form[$group][USERPOINTS_USE_V2BUG] = array( - '#type' => 'checkbox', - '#title' => t('I agree to the above description and want to use the V2 bug fix'), - '#default_value' => variable_get(USERPOINTS_USE_V2BUG, false), + '#title' => t('Assign !Points for posting a comment by default to category', array_merge(userpoints_translation())), + '#options' => userpoints_get_categories(), + '#default_value' => userpoints_nc_get_setting('comment_category'), ); return $form; break; } } - -function userpoints_nodeapi(&$node, $op, $teaser, $page) { - //static up_orig_uid please read drupal.org/node/183520 - static $up_orig_uid; - $points = variable_get(USERPOINTS_POST . $node->type, 0); - $tid = variable_get(USERPOINTS_POST . $node->type . '_category', 0); - switch ($op) { - case 'insert': + +/** + * Helper function to return a setting. + * + * If a node type is passed in, the specific setting will be used with a + * fallback to the global settings. + * + * @param $name + * Name of the setting: points, category, comment_points, comment_category. + * @param $type + * Node type, if passed in, the function looks for a node type specifc setting. + * + * @return + * The configured setting value. + */ +function userpoints_nc_get_setting($name, $type = NULL) { + if (!empty($type)) { + return variable_get('userpoints_nc_' . $name . '_' . $type, variable_get('userpoints_nc_' . $name, 0)); + } + else { + return variable_get('userpoints_nc_' . $name, 0); + } +} + +/** + * Implements hook_node_delete(). + */ +function userpoints_nc_node_delete($node) { + $tid = userpoints_nc_get_setting('category', $node->type); + $points = userpoints_nc_get_setting('points', $node->type); + $published_or_all = $node->status == NODE_PUBLISHED || !variable_get('userpoints_nc_published_only', TRUE); + + if ($published_or_all && $points != 0) { + $params = array( + 'points' => -$points, + 'tid' => $tid, + 'uid' => $node->uid, + 'operation' => 'delete', + 'entity_id' => $node->nid, + 'entity_type' => 'node', + ); + userpoints_userpointsapi($params); + } +} + +/** + * Implements hook_node_insert(). + */ +function userpoints_nc_node_insert($node) { + $tid = userpoints_nc_get_setting('category', $node->type); + $points = userpoints_nc_get_setting('points', $node->type); + $published_or_all = $node->status == NODE_PUBLISHED || !variable_get('userpoints_nc_published_only', TRUE); + + // Only add points if node is published or published only setting is FALSE. + if ($points != 0 && $published_or_all) { + $params = array( + 'points' => $points, + 'tid' => $tid, + 'uid' => $node->uid, + 'operation' => 'insert', + 'entity_id' => $node->nid, + 'entity_type' => 'node' + ); + userpoints_userpointsapi($params); + } +} + +/** + * Implements hook_node_update(). + */ +function userpoints_nc_node_presave($node) { + $tid = userpoints_nc_get_setting('category', $node->type); + $points = userpoints_nc_get_setting('points', $node->type); + $published_or_all = $node->status == NODE_PUBLISHED || !variable_get('userpoints_nc_published_only', TRUE); + $published_only = $node->status == NODE_PUBLISHED && variable_get('userpoints_nc_published_only', TRUE); + + // Check if it is a new node. + if (!$node->nid) { + return; + } + // Load the current status of the node from the database. + $old_status = db_query('SELECT status FROM {node} WHERE nid = :nid', array(':nid' => $node->nid))->fetchField(); + + // Find the last points granted on this node inserts and ownership gains. + $sql = "SELECT points, uid + FROM {userpoints_txn} + WHERE entity_id = :id AND entity_type = :type + AND (operation = :op1 OR operation = :op2) + ORDER BY time_stamp DESC"; + $last_owner = db_query_range($sql, 0, 1, array(':id' => $node->nid, ':type' => 'node', ':op1' => 'insert', ':op2' => 'Ownership gain'))->fetchObject(); + // If ownership has changed, add/substract points. + if ($last_owner && $node->uid != $last_owner->uid) { + // Add to the new node owner. + if ($points != 0 && $published_or_all) { $params = array( - 'points' => $points, + 'points' => $last_owner->points, 'tid' => $tid, 'uid' => $node->uid, - 'operation' => 'insert', + 'operation' => 'Ownership gain', 'entity_id' => $node->nid, 'entity_type' => 'node' ); userpoints_userpointsapi($params); - break; - case 'delete': - if (variable_get(USERPOINTS_POST . '_undo_points_on_delete', true)) { - $points = -$points; - $params = array( - 'points' => $points, - 'tid' => $tid, - 'uid' => $node->uid, - 'operation' => 'operation', - 'entity_id' => $node->nid, - 'entity_type' => 'node', - ); - userpoints_userpointsapi($params); - } - break; - case 'prepare': - $up_orig_uid = $node->uid; - break; - case 'update': - //Find the last points granted on this node inserts and ownership gains - $sql = "SELECT points, uid - FROM {userpoints_txn} - WHERE entity_id = %d AND entity_type = '%s' - AND (operation = '%s' OR operation ='%s') - ORDER BY time_stamp DESC - LIMIT 1 - "; - $last_owner = db_fetch_object(db_query($sql, $node->nid, 'node', 'insert', 'Ownership gain')); - - //Check the UID of the original to this user, if different add/substract points - if ($node->uid != $last_owner->uid && is_numeric($last_owner->uid) ) { - //Check to see if this user has already lost the points for - // Add to the new node owner - $params = array( - 'points' => $points, - 'tid' => $tid, - 'uid' => $node->uid, - 'operation' => 'Ownership gain', - 'entity_id' => $node->nid, - 'entity_type' => 'node' - ); - userpoints_userpointsapi($params); - // subtract from the original node owner - $params = array( - 'points' => -$points, - 'tid' => $tid, - 'uid' => $up_orig_uid, - 'operation' => 'Ownership loss', - 'entity_id' => $node->nid, - 'entity_type' => 'node' - ); - userpoints_userpointsapi($params); - } - else { - //We failed to pull a matching operation via the DB - //If the user wants to use the V2BUG we'll use it.. - //please read drupal.org/node/183520 - if (variable_get(USERPOINTS_USE_V2BUG, false)) { - if ($node->uid != $up_orig_uid) { - // Add to the new node owner - $params = array( - 'points' => $points, - 'tid' => $tid, - 'uid' => $node->uid, - 'operation' => 'Ownership gain', - 'entity_id' => $node->nid, - 'entity_type' => 'node' - ); - userpoints_userpointsapi($params); - // subtract from the original node owner - $params = array( - 'points' => -$points, - 'tid' => $tid, - 'uid' => $up_orig_uid, - 'operation' => 'Ownership loss', - 'entity_id' => $node->nid, - 'entity_type' => 'node' - ); - userpoints_userpointsapi($params); - } - } - } - break; - } -} - -function userpoints_comment($comment, $op) { - global $user; - //static up_orig_uid, please read this thread http://drupal.org/node/183520 - static $up_orig_com_uid; + } - $points = variable_get(USERPOINTS_POST_COMMENT, 0); - $tid = variable_get(USERPOINTS_POST_COMMENT . '_category', 0); - switch ($op) { - case 'insert': + // Only subtract points if he was published before. + if (($published_only && $old_status == NODE_PUBLISHED) || $published_or_all) { $params = array( - 'points' => $points, + 'points' => -$last_owner->points, 'tid' => $tid, - 'uid' => $user->uid, - 'operation' => 'insert', - 'entity_id' => $comment['cid'], - 'entity_type' => 'comment' + 'uid' => $last_owner->uid, + 'operation' => 'Ownership loss', + 'entity_id' => $node->nid, + 'entity_type' => 'node' ); userpoints_userpointsapi($params); - break; - case 'delete': - $points = -$points; + } + } + // Check if node has been published. + else if ($published_only && $old_status == NODE_NOT_PUBLISHED) { + $params = array( + 'points' => $points, + 'tid' => $tid, + 'uid' => $node->uid, + 'operation' => 'published', + 'entity_id' => $node->nid, + 'entity_type' => 'node' + ); + userpoints_userpointsapi($params); + } + // Check if node was unpublished and points need to be subtracted. + else if ($points != 0 && $old_status == NODE_PUBLISHED && $node->status == NODE_NOT_PUBLISHED && variable_get('userpoints_nc_published_only', TRUE)) { + $params = array( + 'points' => -$points, + 'tid' => $tid, + 'uid' => $node->uid, + 'operation' => 'unpublished', + 'entity_id' => $node->nid, + 'entity_type' => 'node' + ); + userpoints_userpointsapi($params); + } +} + +/** + * Implements hook_comment_insert(). + */ +function userpoints_nc_comment_insert($comment) { + $node = node_load($comment->nid); + $tid = userpoints_nc_get_setting('comment_category', $node->type); + $points = userpoints_nc_get_setting('comment_points', $node->type); + $published_or_all = $comment->status == COMMENT_PUBLISHED || !variable_get('userpoints_nc_published_only', TRUE); + + if ($points != 0 && $published_or_all) { + $params = array( + 'points' => $points, + 'tid' => $tid, + 'uid' => $comment->uid, + 'operation' => 'insert', + 'entity_id' => $comment->cid, + 'entity_type' => 'comment' + ); + userpoints_userpointsapi($params); + } +} + +/** + * Implements hook_comment_insert(). + */ +function userpoints_nc_comment_presave($comment) { + global $user; + + $node = node_load($comment->nid); + $tid = userpoints_nc_get_setting('comment_category', $node->type); + $points = userpoints_nc_get_setting('comment_points', $node->type); + $published_or_all = $comment->status == COMMENT_PUBLISHED || !variable_get('userpoints_nc_published_only', TRUE); + $published_only = $comment->status == COMMENT_PUBLISHED && variable_get('userpoints_nc_published_only', TRUE); + + if (!$comment->cid) { + return; + } + + // Load the current status of the node from the database. + $old_status = db_query('SELECT status FROM {comment} WHERE cid = :cid', array(':cid' => $comment->cid))->fetchField(); + + // Find the last points granted on this node inserts and ownership gains. + $sql = "SELECT points, uid + FROM {userpoints_txn} + WHERE entity_id = :id AND entity_type = :type + AND (operation = :op1 OR operation = :op2) + ORDER BY time_stamp DESC"; + $last_owner = db_query_range($sql, 0, 1, array(':id' => $comment->cid, ':type' => 'comment', ':op1' => 'insert', ':op2' => 'Ownership gain'))->fetchObject(); + + + // If ownership has changed, add/substract points. + if ($last_owner && $comment->uid != $last_owner->uid) { + // Add to the new node owner. + if ($points != 0 && $published_or_all) { $params = array( - 'points' => $points, + 'points' => $last_owner->points, 'tid' => $tid, 'uid' => $comment->uid, - 'operation' => 'delete', + 'operation' => 'Ownership gain', 'entity_id' => $comment->cid, - 'entity_type' => 'comment' + 'entity_type' => 'comment', ); userpoints_userpointsapi($params); - break; - case 'moderate': - $points = variable_get(USERPOINTS_MODERATE_COMMENT, 0); - $tid = variable_get(USERPOINTS_MODERATE_COMMENT . '_category', 0); + } + + // Only subtract points if he was published before. + if (($published_only && $old_status == NODE_PUBLISHED) || $published_or_all) { $params = array( - 'points' => $points, + 'points' => -$last_owner->points, 'tid' => $tid, - 'uid' => $comment->uid, - 'operation' => 'moderate', + 'uid' => $last_owner->uid, + 'operation' => 'Ownership loss', 'entity_id' => $comment->cid, - 'entity_type' => 'comment' + 'entity_type' => 'comment', ); userpoints_userpointsapi($params); - break; - case 'form': - $up_orig_com_uid = $comment['uid']['#value']; - break; - case 'update': - //Find the last points granted on this node inserts and ownership gains - $sql = "SELECT points, uid - FROM {userpoints_txn} - WHERE entity_id = %d AND entity_type = '%s' - AND (operation = '%s' OR operation ='%s') - ORDER BY time_stamp DESC - LIMIT 1 - "; - $cid = $comment['cid']; - $new_uid = $comment['uid']; - $last_owner = db_fetch_object(db_query($sql, $cid, 'comment', 'insert', 'Ownership gain')); - - //Check the UID of the original to this user, if different add/substract points - if ($new_uid != $last_owner->uid && is_numeric($last_owner->uid) ) { - //The owner has changed so we're removing from the - //the original owner and giving to the new owner - //Give to the original owner - $points = $last_owner->points; - $params = array( - 'points' => $points, - 'tid' => $tid, - 'uid' => $new_uid, - 'operation' => 'Ownership gain', - 'entity_id' => $cid, - 'entity_type' => 'comment' - ); - userpoints_userpointsapi($params); - - //Take away from the original owner - $params = array( - 'points' => -$points, - 'tid' => $tid, - 'uid' => $last_owner->uid, - 'operation' => 'Ownership loss', - 'entity_id' => $cid, - 'entity_type' => 'comment' - ); - userpoints_userpointsapi($params); - - } - else { - //We failed to pull a matching operation via the DB - //If the user wants to use the V2BUG we'll use it.. - //please read drupal.org/node/183520 - if (variable_get(USERPOINTS_USE_V2BUG, false)) { - if ($orig_uid != $comment['uid']) { - $params = array( - 'points' => $points, - 'tid' => $tid, - 'uid' => $new_uid, - 'operation' => 'Ownership gain', - 'entity_id' => $cid, - 'entity_type' => 'comment' - ); - userpoints_userpointsapi($params); - - //Take away from the original owner - $params = array( - 'points' => -$points, - 'tid' => $tid, - 'uid' => $comment['uid'], - 'operation' => 'Ownership loss', - 'entity_id' => $new_uid, - 'entity_type' => 'comment' - ); - userpoints_userpointsapi($params); - } - } - } - break; + } + } + // Check if node has been published. + else if ($published_only && $old_status == NODE_NOT_PUBLISHED) { + $params = array( + 'points' => $points, + 'tid' => $tid, + 'uid' => $comment->uid, + 'operation' => 'published', + 'entity_id' => $comment->cid, + 'entity_type' => 'comment', + ); + userpoints_userpointsapi($params); + } + // Check if node was unpublished and points need to be subtracted. + else if ($points != 0 && $old_status == COMMENT_PUBLISHED && $comment->status == COMMENT_NOT_PUBLISHED && variable_get('userpoints_nc_published_only', TRUE)) { + $params = array( + 'points' => -$points, + 'tid' => $tid, + 'uid' => $node->uid, + 'operation' => 'unpublished', + 'entity_id' => $comment->cid, + 'entity_type' => 'comment', + ); + userpoints_userpointsapi($params); } } +/** + * Implements hook_comment_delete(). + */ +function userpoints_nc_comment_delete($comment) { + $node = node_load($comment->nid); + $tid = userpoints_nc_get_setting('comment_category', $node->type); + $points = userpoints_nc_get_setting('comment_points', $node->type); + $published_or_all = $comment->status == COMMENT_PUBLISHED || !variable_get('userpoints_nc_published_only', TRUE); + + if ($points != 0 && $published_or_all) { + $params = array( + 'points' => -$points, + 'tid' => $tid, + 'uid' => $comment->uid, + 'operation' => 'delete', + 'entity_id' => $comment->cid, + 'entity_type' => 'comment' + ); + userpoints_userpointsapi($params); + } +}