PDF search results

Hi,

1. In theory, something like this could do the trick:

add_filter( 'asp_results', 'asp_attachment_parent_in_category', 10, 1 );
function asp_attachment_parent_in_category( $results ) {
  // Category IDs
  $categories = array(1, 2, 3);
  
  foreach ($results as $k=>&$r) {
	if ( $r->content_type == 'attachment' ) {
		$parent_id = wp_get_post_parent_id( $r->id );
		if ( !is_wp_error($parent_id) && !empty($parent_id) ) {
			if ( !in_category($categories, $parent_id) ) {
				unset($results[$k]);
			}			
		}
	}
  } 
  return $results;
}

This only works with categories – no other taxonomy terms or anything else. The $categories variable holds the category IDs for which you want the parents to be restricted, don’t forget to change that.

2. This may not be possible without actual changes on the theme. The plugin forces the same titles/contents/links for the results page, but usually the results page queries these fields individually and therefore it refreshes them from the database.

There is still hope though, try this custom code:

add_filter( 'asp_regular_search_result', 'asp_force_guids', 10, 1 );
function asp_force_guids( $r ) {
	if ( isset($r->asp_data) ) {
		$r->guid = $r->asp_data->link;
		$r->post_excerpt = $r->asp_data->content;
		$r->post_content = $r->asp_data->content;
	}
	return $r;
}

3. Not yet, there are potential security vulerability concerns that has to be resolved to make it safe for release. It will be implemented for sure though.

Hi,

1. Try this variation:

add_filter( 'asp_results', 'asp_attachment_parent_in_category', 10, 1 );
function asp_attachment_parent_in_category( $results ) {
  // Category IDs
  $categories = array(1, 2, 3);
  
  foreach ($results as $k=>&$r) {
	if ( $r->content_type == 'attachment' ) {
		$parent_id = wp_get_post_parent_id( $r->id );
		if ( !is_wp_error($parent_id) && !empty($parent_id) ) {
			if ( $parent_id == 0 || !in_category($categories, $parent_id) ) {
				unset($results[$k]);
			}			
		} else {
			unset($results[$k]);
		}
	}
  } 
  return $results;
}

2. That is bad news, this may not be possible without theme modifications then. Maybe a bit more aggressive code will do it, try this:

add_filter( 'asp_regular_search_result', 'asp_force_guids', 10, 1 );
function asp_force_guids( $r ) {
	if ( isset($r->asp_data) ) {
		if ( $r->post_type == 'attachment' ) {
			$parent_id = wp_get_post_parent_id( $r->id );
			if ( !is_wp_error($parent_id) ) {
				$r->guid = get_permalink($parent_id);		
			}
		} else {
			$r->guid = $r->asp_data->link;
		}
		$r->post_excerpt = $r->asp_data->content;
		$r->post_content = $r->asp_data->content;
	}
	return $r;
}

Hi,

1.Okay, I have logged in to check and made a tiny modification. Can you please test if this changed anything.

For reference the new code is:

add_filter( 'asp_results', 'asp_attachment_parent_in_category', 10, 1 );
function asp_attachment_parent_in_category( $results ) {
  // Category IDs
  $categories = array(28);
  
  foreach ($results as $k=>&$r) {
	if ( isset($r->post_type) && $r->post_type == 'attachment' ) {
		$parent_id = wp_get_post_parent_id( $r->id );
		if ( !is_wp_error($parent_id) && !empty($parent_id) ) {
			if ( $parent_id == 0 || !in_category($categories, $parent_id) ) {
				unset($results[$k]);
			}			
		} else {
			unset($results[$k]);
		}
	}
  } 
  return $results;
}

Great 🙂 At least it works now.

I removed that extra text – I used that for debugging the code, and I forgot to delete that line. I have quickly logged in and corrected that. It should be fine now.