Rework mod/msearch
- Remove deprecated q()/killme() calls - Ensure an output on empty search string - Fix page number nonsense (-1, +1,...)
This commit is contained in:
parent
1938ec3ebe
commit
b962fc989a
1 changed files with 52 additions and 33 deletions
|
@ -4,45 +4,64 @@ use Friendica\App;
|
||||||
use Friendica\Core\System;
|
use Friendica\Core\System;
|
||||||
use Friendica\Database\DBA;
|
use Friendica\Database\DBA;
|
||||||
|
|
||||||
function msearch_post(App $a) {
|
function msearch_post(App $a)
|
||||||
|
{
|
||||||
$perpage = (($_POST['n']) ? $_POST['n'] : 80);
|
$search = defaults($_POST, 's', '');
|
||||||
$page = (($_POST['p']) ? intval($_POST['p'] - 1) : 0);
|
$perpage = intval(defaults($_POST, 'n', 80));
|
||||||
$startrec = (($page+1) * $perpage) - $perpage;
|
$page = intval(defaults($_POST, 'p', 1));
|
||||||
|
$startrec = ($page - 1) * $perpage;
|
||||||
$search = $_POST['s'];
|
|
||||||
if(! strlen($search))
|
|
||||||
killme();
|
|
||||||
|
|
||||||
$r = q("SELECT COUNT(*) AS `total` FROM `profile` LEFT JOIN `user` ON `user`.`uid` = `profile`.`uid` WHERE `is-default` = 1 AND `user`.`hidewall` = 0 AND MATCH `pub_keywords` AGAINST ('%s') ",
|
|
||||||
DBA::escape($search)
|
|
||||||
);
|
|
||||||
|
|
||||||
if (DBA::isResult($r))
|
|
||||||
$total = $r[0]['total'];
|
|
||||||
|
|
||||||
|
$total = 0;
|
||||||
$results = [];
|
$results = [];
|
||||||
|
|
||||||
$r = q("SELECT `pub_keywords`, `username`, `nickname`, `user`.`uid` FROM `user` LEFT JOIN `profile` ON `user`.`uid` = `profile`.`uid` WHERE `is-default` = 1 AND `user`.`hidewall` = 0 AND MATCH `pub_keywords` AGAINST ('%s') LIMIT %d , %d ",
|
if (!strlen($search)) {
|
||||||
DBA::escape($search),
|
$output = ['total' => 0, 'items_page' => $perpage, 'page' => $page, 'results' => $results];
|
||||||
intval($startrec),
|
echo json_encode($output);
|
||||||
intval($perpage)
|
exit();
|
||||||
|
}
|
||||||
|
|
||||||
|
$count_stmt = DBA::p(
|
||||||
|
"SELECT COUNT(*) AS `total`
|
||||||
|
FROM `profile`
|
||||||
|
JOIN `user` ON `user`.`uid` = `profile`.`uid`
|
||||||
|
WHERE `is-default` = 1
|
||||||
|
AND `user`.`hidewall` = 0
|
||||||
|
AND MATCH(`pub_keywords`) AGAINST (?)",
|
||||||
|
$search
|
||||||
);
|
);
|
||||||
|
|
||||||
if (DBA::isResult($r)) {
|
if (DBA::isResult($count_stmt)) {
|
||||||
foreach($r as $rr)
|
$row = DBA::fetch($count_stmt);
|
||||||
|
$total = $row['total'];
|
||||||
|
}
|
||||||
|
|
||||||
|
DBA::close($count_stmt);
|
||||||
|
|
||||||
|
$search_stmt = DBA::p(
|
||||||
|
"SELECT `pub_keywords`, `username`, `nickname`, `user`.`uid`
|
||||||
|
FROM `user`
|
||||||
|
JOIN `profile` ON `user`.`uid` = `profile`.`uid`
|
||||||
|
WHERE `is-default` = 1
|
||||||
|
AND `user`.`hidewall` = 0
|
||||||
|
AND MATCH(`pub_keywords`) AGAINST (?)
|
||||||
|
LIMIT ?, ?",
|
||||||
|
$search,
|
||||||
|
$startrec,
|
||||||
|
$perpage
|
||||||
|
);
|
||||||
|
|
||||||
|
while($search_result = DBA::fetch($search_stmt)) {
|
||||||
$results[] = [
|
$results[] = [
|
||||||
'name' => $rr['name'],
|
'name' => $search_result['name'],
|
||||||
'url' => System::baseUrl() . '/profile/' . $rr['nickname'],
|
'url' => System::baseUrl() . '/profile/' . $search_result['nickname'],
|
||||||
'photo' => System::baseUrl() . '/photo/avatar/' . $rr['uid'] . '.jpg',
|
'photo' => System::baseUrl() . '/photo/avatar/' . $search_result['uid'] . '.jpg',
|
||||||
'tags' => str_replace([',',' '],[' ',' '],$rr['pub_keywords'])
|
'tags' => str_replace([',', ' '], [' ', ' '], $search_result['pub_keywords'])
|
||||||
];
|
];
|
||||||
}
|
}
|
||||||
|
|
||||||
$output = ['total' => $total, 'items_page' => $perpage, 'page' => $page + 1, 'results' => $results];
|
$output = ['total' => $total, 'items_page' => $perpage, 'page' => $page, 'results' => $results];
|
||||||
|
|
||||||
echo json_encode($output);
|
echo json_encode($output);
|
||||||
|
|
||||||
killme();
|
exit();
|
||||||
|
|
||||||
}
|
}
|
||||||
|
|
Loading…
Reference in a new issue