Line 115 of userpoints_nc.module, the 'operation' you use for $op = 'delete' in hook_nodeapi is "operation" (it should be 'delete').

Current:

 case 'delete':
      if (variable_get(USERPOINTS_POST . '_undo_points_on_delete', true)) {
        $points = -$points;
        $params = array(
          'points' => $points,
          'uid' => $node->uid,
          'operation' => 'operation',
          'entity_id' => $node->nid,
          'entity_type' => 'node',
        );
        userpoints_userpointsapi($params);
      }
      break;

Should be:
'operation' => t('delete')

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

mstef’s picture

ezra-g’s picture

Status: Active » Needs review

Marking as "needs review".

Status: Needs review » Needs work

The last submitted patch, userpoints_nc-fixed-delete-operation-label.patch, failed testing.

quotesBro’s picture

Status: Needs work » Needs review
FileSize
471 bytes

Patch for Git

Status: Needs review » Needs work

The last submitted patch, userpoints_nc-875140.patch, failed testing.

quotesBro’s picture

Version: 6.x-1.1 » 6.x-1.x-dev
Status: Needs work » Needs review

patch against dev

quotesBro’s picture

#4: userpoints_nc-875140.patch queued for re-testing.

quotesBro’s picture

Status: Needs review » Reviewed & tested by the community
Berdir’s picture

Status: Reviewed & tested by the community » Fixed

Thanks, commited.

Status: Fixed » Closed (fixed)

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