Hey @vedanshujain @andfinally @kloon , I have added this PR #30641 , please review it , thank you

Ok , now I see that the changes I have made are wrong, but how do I know that no search results are being returned

now I see that the changes I have made are wrong, but how do I know that no search results are being returned , is any variable storing the number of results returned, please help me out @vedanshujain @andfinally @kloon , thanks

@san-coding you should be able to use $addon to see if there are search results. Can you check if the presence of both $search and more than one in $addon gives you expected behavior?

@san-coding you should be able to use $addon to see if there are search results. Can you check if the presence of both $search and more than one in $addon gives you expected behavior?

Thanks @vedanshujain , I have made the changes , instead of checking if(empty($search)) , I am now checking if(sizeof($addons)==0)

I have also added 0` ) : ?> , so that the message that the message search results for "" is displayed only when && sizeof($addons)>0

please verify and review if my changes are correct, thanks @vedanshujain

Hey @vedanshujain , could you please review this

Have added required space and replaced sizeof() with count()

@vedanshujain , looks like all checks have passed, you may review and merge

@vedanshujain @rodelgc @andfinally someone please review this

Thank you , I have made the changes, please review this PR

can you please add a label hacktoberfest-accepted to this PR, thank you @eopws @kloon

@kloon @eopws @vedanshujain please review this

@kloon thanks

Requesting a review from the core team before merging just to double check, should be a quick one.

Hi @vedanshujain, thanks for merging this pull request. Please take a look at these follow-up tasks you may need to perform:

• Add the status: needs changelog label
• Add the status: needs testing instructions label

Thanks you 👍🏻

A pull request is considered approved once it has an overall approving review from maintainers, or has been merged by maintainers, or has been given the 'hacktoberfest-accepted' label.

My reading of the rules is that the label is not necessary, but please let us know if you think we need to add it. Thx!

Hi @san-coding @vedanshujain - there is a slight issue for this PR when there is no search term the error message still displays (ie. on the landing page when you first access Marketplace).

To reproduce:

• login to wp-admin
• navigate to WooCommerce -> Marketplace
• you will see the error message Sorry, could not find anything. Try searching again using a different term. on this page when it should not show here

I am happy to open a PR to correct this if you like. Just let me know.

Ohh i will look into this and get back to you soon, can you revert the merge, adding a condition to check if search term exists would be enough i guess

@drjamesj , checkout this file in the trunk branch, maybe it has been solved