search

buddypress / next-template-packs

is this the next BuddyPress template pack?
35 stars 9 forks source link

Members search by profile fields values doesn't work #105

Closed slaFFik closed 6 years ago

slaFFik commented 7 years ago

NEXT shows empty results (no members), while Legacy - a list of members for the same query.

Tested with fields generated by BP Default Data plugin, they contain space, which in URL results in a plus + sign.

slaFFik commented 7 years ago

This is an escaping bug.

aahani commented 7 years ago

I've noticed the same issue with Persian language characters.

code-casper commented 6 years ago

It seems to be a bug in buddypress/bp-templates/bp-nouveau/includes/functions.php

line 113 replace $qs[] = 'search_terms=' . urlencode( $_POST['search_terms'] ); by
$qs[] = 'search_terms=' . post_query['search_terms'] );

it works for me. Maybe a professional can check this and submit.

imath commented 6 years ago

Thanks for your feedback I’ll try to look at it asap, now BP Nouveau has been merged in BuddyPress, it can be a good idea to report bugs on https://buddypress.trac.wordpress.org

Le 12 août 2018 à 21:02, code-casper notifications@github.com a écrit :

It seems to be a bug in buddypress/bp-templates/bp-nouveau/includes/functions.php

line 113 replace $qs[] = 'search_terms=' . urlencode( $_POST['search_terms'] ); by $qs[] = 'search_terms=' . post_query['search_terms'] );

it works for me. Maybe a professional can check this and submit.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub, or mute the thread.

code-casper commented 6 years ago

done ;-)

https://buddypress.trac.wordpress.org/ticket/7944#ticket

imath commented 6 years ago

Awesome ! Thanks a lot, I'll reply on the ticket :)