Giving users points for transference

Hey.

Change the following line:

if ( $result === false || $request['ref'] != 'transfer' || $request['type'] == 'mycred_default' ) return $result;

to:

if ( $result === false || $request['ref'] != 'transfer' ) return $result;

It will remove the point type check which was set to default.

Well doing that I get and endless loop, so I changed mycred->add_creds for mycred->updates_users_balance.
no my code is this:

add_filter( 'mycred_add_finished', 'mycred_give_points', 10, 2 );
function mycred_give_points( $result, $request, $mycred ) {

	extract( $request );

	$points_to_users = 0-$amount;

	if ( $result === false || $request['ref'] != 'transfer' ) return $result;

	if ( $request['type'] == 'mycred_default' || $request['type'] == 'token' || $request['type'] == 'coins') return $request;

	// Add points to users
	$mycred->update_users_balance( $user_id, $points_to_users, $type);

	return $result;
}

So my problem is: $type is calling only mycred_default value even if a user transfer token point. I’ve tried to use $request[‘type’] instead but no success.

Question: How can I call the $type that has been used for transfer?

That way when a user transfer any point he also get this point.

You are correct, using add_creds is not a good idea.

When it comes to transfers, you have to remember that the code will run twice. First to deduct the points from the sender and then to deposit into the recipients account.

What is the “transfer point type” key you set? It’s not “token” or “coins”?

Gabriel here is the addon transfer core:

// Let others play before we execute the transfer
			do_action( 'mycred_transfer_ready', $transaction_id, $post, $prefs, $this, $type );

			$data = apply_filters( 'mycred_transfer_data', array( 'ref_type' => 'user', 'tid' => $transaction_id ), $transaction_id, $post, $prefs, $type );

			// First take the amount from the sender
			$mycred->add_creds(
				$ref,
				$from,
				0-$amount,
				$prefs['logs']['sending'],
				$recipient_id,
				$data,
				$type
			);

			// Then add the amount to the receipient
			$mycred->add_creds(
				$ref,
				$recipient_id,
				$amount,
				$prefs['logs']['receiving'],
				$from,
				$data,
				$type
			);

I need that action be like below:

// Let others play before we execute the transfer
			do_action( 'mycred_transfer_ready', $transaction_id, $post, $prefs, $this, $type );

			$data = apply_filters( 'mycred_transfer_data', array( 'ref_type' => 'user', 'tid' => $transaction_id ), $transaction_id, $post, $prefs, $type );

			// First take the amount from the sender
			$mycred->add_creds(
				$ref,
				$from,
				0-$amount,
				$prefs['logs']['sending'],
				$recipient_id,
				$data,
				'my_custom_point'
			);

			// Then add the amount to the receipient
			$mycred->add_creds(
				$ref,
				$recipient_id,
				$amount,
				$prefs['logs']['receiving'],
				$from,
				$data,
				$type
			);

			// Last add the same amount to the sender
			$mycred->add_creds(
				$ref,
				$from,
				$amount,
				$prefs['logs']['sending'],
				$recipient_id,
				$data,
				$type
			);

How to do this without changing the core myCRED?

The following code snippet will do this for you. Requires 1.6 or higher.

The code assumes you make transfers under the default “transfer” reference. If you have changed this and use a different reference then the code must be updated to reflect this.