Points for bbPress topic author when topic is favorited

Yes, that’s it! Now it works ?? !!

Uhm, is there a function defined by your plugin that checks the data array of the sql table already or do I’ve to start a new query for this question (https://www.ads-software.com/support/topic/points-for-bbpress-topic-author-when-topic-is-favorited?replies=15#post-4324849 above):

instead of checking for a unique event if ( $this->has_entry( ‘topic_favorited’, $topic_id, $topic_author ) ) return; I’d like to check if the sam user already gave a favorite. Therefore I added the ref_user to the data field. Is there already a function to check it directly? (I saw that you added array( ‘ref_type’ => ‘post’ ) for the bbpress new_topic, so I had this idea to put the ref_user in here in my function)

Yay!

When it comes to the data variable, yes, the default array is added to indicate that this template tag uses “post related” template tags. You can add to this array like with any other associative array.

Example:

array( 'ref_type' => 'post', 'some_key' => 'some_value', 'another_key' => 'more_values' )

Now, when an array is used in the data variable, myCRED will serialize this before inserting it to the database. So searching for values inside a serialized array can be tricky.

IF, you know you will only have these two values in your data array (ref_type and your own custom key) then the following custom function will allow you to search for it:

function my_custom_has_entry_check( $action = '', $ref_id = '', $user_id = '', $data = '' ) {
	global $wpdb;

	$where = $prep = array();
	if ( !empty( $action ) ) {
		$where[] = 'ref = %s';
		$prep[] = $action;
	}

	if ( !empty( $ref_id ) ) {
		$where[] = 'ref_id = %d';
		$prep[] = $ref_id;
	}

	if ( !empty( $user_id ) ) {
		$where[] = 'user_id = %d';
		$prep[] = abs( $user_id );
	}

	if ( !empty( $data ) ) {
		$where[] = 'data = %s';
		$prep[] = '%' . trim( $data ) . '%';
	}

	$where = implode( ' AND ', $where );

	if ( !empty( $where ) ) {
		$sql = "SELECT * FROM " . $wpdb->prefix . 'myCRED_log' . " WHERE $where";
		$wpdb->get_results( $wpdb->prepare( $sql, $prep ) );
		if ( $wpdb->num_rows > 0 ) return true;
	}

	return false;
}

NOTE! This function will search your data with the use of wildcards. This means that if you at a later point add more to the data array, lets say another value that also contains a numeric value, then you might run in into getting false positives!

Hope it makes sense.

ok, your’re great ?? thanks!
I changed this line:
$prep[] = '%' . trim( $data ) . '%';
to
$prep[] = maybe_serialize($data);
I think it’s a BuddyPress function, but my theme is a BuddyPress theme, so it’s ok. Now I can use more items in an array, if I want. It just has to be the same in the entry and the same in the has_entry_check. So this is the complete function now (if anybody else is looking for it…):

public function fav_topic( $user_id, $topic_id ) {

			// $user_id is loggedin_user, not author, so get topic author
			$topic_author = get_post_field( 'post_author', $topic_id );

			// Check if user is excluded (required)
			if ( $this->core->exclude_user( $topic_author ) || $topic_author == $user_id  ) return;

			// Make sure this is a unique event (favorite not from same user)
			if( my_custom_has_entry_check( 'topic_favorited', $topic_id, $topic_author, array('ref_user' => $user_id) ) ) return;

			// Execute
			$this->core->add_creds(
				'topic_favorited',
				$topic_author,
				$this->prefs['topic_fav']['creds'],
				$this->prefs['topic_fav']['log'],
				$topic_id,
				array( 'ref_user' => $user_id )
			);

			// Clean up
			//unset( $this );
		}

Makes sense, nicely done and great work on your first myCRED Custom Hook!

Let me know if you have any further issues otherwise Ill mark this topic “Resolved”.

I have added this in 1.1.1.

Hey Gabriel, that’s nice, thanks ??
Just one hint:
maybe_serialize makes serialized data out of arrays, so it’s correct to pass the array to it array('ref_user' => $user_id)
and is a WordPress function (I was wrong with saying, it’s BuddyPress). So you don’t need to pass serialized data as string like ‘s:8:”ref_user”;i:’ . $user_id . ‘;’
The function makes it automatically looking like this in the sql query. Personally I think it’s more beautiful to use an array.

Hey Fee.

Yeah, I use maybe_serialize in the has_entry() method.
If we are to just pass on an array there then i think we will need to add some wildcards around the maybe_serialize since the new data array no longer contains only the ref_user but also the ref_type (to enable user related template tags).