Hey Jason,

I am wondering if I could have permission to fork your code OR enhance it with a pull request. I wanted to add some more ways to search for posts and this seems like a great start to what I had in mind. Your thoughts?

Bryce

I have a small translation issue with this component: https://www.opensource-socialnetwork.org/discussion/view/3549/search-posts-cannot-find-the-locale-for-view-post

nice job =)

Hey Bryce,
Yep, It's good to go!

Any more development on this to fix issues, I would love to have this component

Next: You can't always be sure your site is accessed from a logged in user. Log off and try

https://yoursite.com/search?type=posts

and most likely you'll get a crash

You'd better make use of if(ossn_isLoggedin()) { ....

Another step ahead, yes!
Still irritating:

$count_options = array("search_type" => true, "description" => $query, "count" => true);

What is it good for?

Hi Z-Man,
Thanks for the help with this. Moving the friends part into !isadmin of course only saves time and resources for admin users, but still a good call. Thank you for helping me understand the pagination a bit better. I have uploaded V1.3 ;)

Best,
Jason

Aside from that you should think about moving your friends part completely inside the !ossn_isAdminLoggedin() condition to avoid unnecessary queries if the admin is logged in. Actually I don't think this is the final solution anyway because it's missing all public records.

(Sorry, but I was unable to post the complete stuff all in one.)

  if(ossn_loggedin_user()->guid){
    		//bof build array of friends user id's
    		$user    = new OssnUser;
    		$friends = $user->getFriends(ossn_loggedin_user()->guid);
    		//add self user id so we can find self's posts too
    		$fuid_list = array(ossn_loggedin_user()->guid);
    		// suppress PHP warning if we have no friends !
    		if ($friends) {
    			foreach($friends as $friend){
    				$fuid_list[] = $friend->guid;
    			}
    		}
    		//eof build array of friends user id's
    		
    		$wall = new OssnWall;
    		$query = input('q');
    		$search_options = array("search_type" => true, "description" => $query);
    		
    		//if not admin user add where caluse to limit reach
    		if(!ossn_isAdminLoggedin()){
    			$search_options['wheres'] = 'owner_guid IN ('.implode(",",$fuid_list) .')';
    		}
    		
    		// since we use 2 different (admin/no admin) record queries now, we can't stay with only 1 count query any longer
    		// $count_options = array("search_type" => true, "description" => $query, "count" => true);
    		// get posts
    		$posts = $wall->GetPosts($search_options);
    		// take currently in use query - plus count option in order to return the number of records found
    		$search_options['count'] = true;
    		// get count
    		$count = $wall->GetPosts($search_options);
    		// $data = $posts;  no idea for what reason we need an extra array here ?
    		$found['users'] = $posts;
    		// $found['count'] = $count; no idea why this is passed to the view.php ? there's no 'count' being use in the code
    		$search         = ossn_plugin_view('searchposts/search/view', $found);
    		$search .= ossn_view_pagination($count);
    		if(empty($posts)) {
    			return ossn_print('ossn:search:no:result');
    		}
    		return $search;
    	}else{
    		return ossn_print('ossn:search:no:result');
    	}