X-Git-Url: https://git.mxchange.org/?a=blobdiff_plain;ds=sidebyside;f=mod%2Fdirectory.php;h=256c9bbbd597e69918a4d9ae979b73cea1a02e72;hb=6c341962648b73c935c2de26996b07332b776a1f;hp=6c8be7c3c5da26587de1f35753114299abe330e3;hpb=7c0b591043b8c5183b1b697be9de772667f4778b;p=friendica.git
diff --git a/mod/directory.php b/mod/directory.php
index 6c8be7c3c5..256c9bbbd5 100644
--- a/mod/directory.php
+++ b/mod/directory.php
@@ -7,13 +7,15 @@ use Friendica\App;
use Friendica\Content\Nav;
use Friendica\Content\Pager;
use Friendica\Content\Widget;
-use Friendica\Core\Addon;
use Friendica\Core\Config;
+use Friendica\Core\Hook;
use Friendica\Core\L10n;
+use Friendica\Core\Renderer;
use Friendica\Database\DBA;
use Friendica\Model\Contact;
use Friendica\Model\Profile;
use Friendica\Util\Proxy as ProxyUtils;
+use Friendica\Util\Strings;
function directory_init(App $a)
{
@@ -28,7 +30,7 @@ function directory_init(App $a)
function directory_post(App $a)
{
- if (x($_POST, 'search')) {
+ if (!empty($_POST['search'])) {
$a->data['search'] = $_POST['search'];
}
}
@@ -43,12 +45,14 @@ function directory_content(App $a)
}
$o = '';
+ $entries = [];
+
Nav::setSelected('directory');
- if (x($a->data, 'search')) {
- $search = notags(trim($a->data['search']));
+ if (!empty($a->data['search'])) {
+ $search = Strings::escapeTags(trim($a->data['search']));
} else {
- $search = ((x($_GET, 'search')) ? notags(trim(rawurldecode($_GET['search']))) : '');
+ $search = (!empty($_GET['search']) ? Strings::escapeTags(trim(rawurldecode($_GET['search']))) : '');
}
$gdirpath = '';
@@ -96,7 +100,7 @@ function directory_content(App $a)
$limit = $pager->getStart()."," . $pager->getItemsPerPage();
$r = DBA::p("SELECT `profile`.*, `profile`.`uid` AS `profile_uid`, `user`.`nickname`, `user`.`timezone` , `user`.`page-flags`,
- `contact`.`addr`, `contact`.`url` AS profile_url FROM `profile`
+ `contact`.`addr`, `contact`.`url` AS `profile_url` FROM `profile`
LEFT JOIN `user` ON `user`.`uid` = `profile`.`uid`
LEFT JOIN `contact` ON `contact`.`uid` = `user`.`uid`
WHERE `is-default` $publish AND NOT `user`.`blocked` AND NOT `user`.`account_removed` AND `contact`.`self`
@@ -110,114 +114,117 @@ function directory_content(App $a)
}
while ($rr = DBA::fetch($r)) {
- $itemurl= '';
-
- $itemurl = (($rr['addr'] != "") ? $rr['addr'] : $rr['profile_url']);
-
- $profile_link = 'profile/' . ((strlen($rr['nickname'])) ? $rr['nickname'] : $rr['profile_uid']);
-
- $pdesc = (($rr['pdesc']) ? $rr['pdesc'] . '
' : '');
-
- $details = '';
- if (strlen($rr['locality'])) {
- $details .= $rr['locality'];
- }
- if (strlen($rr['region'])) {
- if (strlen($rr['locality'])) {
- $details .= ', ';
- }
- $details .= $rr['region'];
- }
- if (strlen($rr['country-name'])) {
- if (strlen($details)) {
- $details .= ', ';
- }
- $details .= $rr['country-name'];
- }
-// if(strlen($rr['dob'])) {
-// if(($years = age($rr['dob'],$rr['timezone'],'')) != 0)
-// $details .= '
' . L10n::t('Age: ') . $years ;
-// }
-// if(strlen($rr['gender']))
-// $details .= '
' . L10n::t('Gender: ') . $rr['gender'];
-
- $profile = $rr;
-
- if ((x($profile, 'address') == 1)
- || (x($profile, 'locality') == 1)
- || (x($profile, 'region') == 1)
- || (x($profile, 'postal-code') == 1)
- || (x($profile, 'country-name') == 1)
- ) {
- $location = L10n::t('Location:');
- } else {
- $location = '';
- }
-
- $gender = ((x($profile, 'gender') == 1) ? L10n::t('Gender:') : false);
- $marital = ((x($profile, 'marital') == 1) ? L10n::t('Status:') : false);
- $homepage = ((x($profile, 'homepage') == 1) ? L10n::t('Homepage:') : false);
- $about = ((x($profile, 'about') == 1) ? L10n::t('About:') : false);
-
- $location_e = $location;
-
- $photo_menu = [
- 'profile' => [L10n::t("View Profile"), Contact::magicLink($profile_link)]
- ];
-
- $entry = [
- 'id' => $rr['id'],
- 'url' => $profile_link,
- 'itemurl' => $itemurl,
- 'thumb' => ProxyUtils::proxifyUrl($rr[$photo], false, ProxyUtils::SIZE_THUMB),
- 'img_hover' => $rr['name'],
- 'name' => $rr['name'],
- 'details' => $details,
- 'account_type' => Contact::getAccountType($rr),
- 'profile' => $profile,
- 'location' => $location_e,
- 'tags' => $rr['pub_keywords'],
- 'gender' => $gender,
- 'pdesc' => $pdesc,
- 'marital' => $marital,
- 'homepage' => $homepage,
- 'about' => $about,
- 'photo_menu' => $photo_menu,
-
- ];
-
- $arr = ['contact' => $rr, 'entry' => $entry];
-
- Addon::callHooks('directory_item', $arr);
-
- unset($profile);
- unset($location);
-
- if (!$arr['entry']) {
- continue;
- }
-
- $entries[] = $arr['entry'];
+ $entries[] = format_directory_entry($rr, $photo);
}
DBA::close($r);
-
- $tpl = get_markup_template('directory_header.tpl');
-
- $o .= replace_macros($tpl, [
- '$search' => $search,
- '$globaldir' => L10n::t('Global Directory'),
- '$gdirpath' => $gdirpath,
- '$desc' => L10n::t('Find on this site'),
- '$contacts' => $entries,
- '$finding' => L10n::t('Results for:'),
- '$findterm' => (strlen($search) ? $search : ""),
- '$title' => L10n::t('Site Directory'),
- '$submit' => L10n::t('Find'),
- '$paginate' => $pager->renderFull($total),
- ]);
} else {
info(L10n::t("No entries \x28some entries may be hidden\x29.") . EOL);
}
+ $tpl = Renderer::getMarkupTemplate('directory_header.tpl');
+
+ $o .= Renderer::replaceMacros($tpl, [
+ '$search' => $search,
+ '$globaldir' => L10n::t('Global Directory'),
+ '$gdirpath' => $gdirpath,
+ '$desc' => L10n::t('Find on this site'),
+ '$contacts' => $entries,
+ '$finding' => L10n::t('Results for:'),
+ '$findterm' => (strlen($search) ? $search : ""),
+ '$title' => L10n::t('Site Directory'),
+ '$search_mod' => 'directory',
+ '$submit' => L10n::t('Find'),
+ '$paginate' => $pager->renderFull($total),
+ ]);
+
return $o;
}
+
+/**
+ * Format contact/profile/user data from the database into an usable
+ * array for displaying directory entries.
+ *
+ * @param array $arr The directory entry from the database.
+ * @param string $photo_size Avatar size (thumb, photo or micro).
+ *
+ * @return array
+ */
+function format_directory_entry(array $arr, $photo_size = 'photo')
+{
+ $itemurl = (($arr['addr'] != "") ? $arr['addr'] : $arr['profile_url']);
+
+ $profile_link = $arr['profile_url'];
+
+ $pdesc = (($arr['pdesc']) ? $arr['pdesc'] . '
' : '');
+
+ $details = '';
+ if (strlen($arr['locality'])) {
+ $details .= $arr['locality'];
+ }
+ if (strlen($arr['region'])) {
+ if (strlen($arr['locality'])) {
+ $details .= ', ';
+ }
+ $details .= $arr['region'];
+ }
+ if (strlen($arr['country-name'])) {
+ if (strlen($details)) {
+ $details .= ', ';
+ }
+ $details .= $arr['country-name'];
+ }
+
+ $profile = $arr;
+
+ if (!empty($profile['address'])
+ || !empty($profile['locality'])
+ || !empty($profile['region'])
+ || !empty($profile['postal-code'])
+ || !empty($profile['country-name'])
+ ) {
+ $location = L10n::t('Location:');
+ } else {
+ $location = '';
+ }
+
+ $gender = (!empty($profile['gender']) ? L10n::t('Gender:') : false);
+ $marital = (!empty($profile['marital']) ? L10n::t('Status:') : false);
+ $homepage = (!empty($profile['homepage']) ? L10n::t('Homepage:') : false);
+ $about = (!empty($profile['about']) ? L10n::t('About:') : false);
+
+ $location_e = $location;
+
+ $photo_menu = [
+ 'profile' => [L10n::t("View Profile"), Contact::magicLink($profile_link)]
+ ];
+
+ $entry = [
+ 'id' => $arr['id'],
+ 'url' => Contact::magicLInk($profile_link),
+ 'itemurl' => $itemurl,
+ 'thumb' => ProxyUtils::proxifyUrl($arr[$photo_size], false, ProxyUtils::SIZE_THUMB),
+ 'img_hover' => $arr['name'],
+ 'name' => $arr['name'],
+ 'details' => $details,
+ 'account_type' => Contact::getAccountType($arr),
+ 'profile' => $profile,
+ 'location' => $location_e,
+ 'tags' => $arr['pub_keywords'],
+ 'gender' => $gender,
+ 'pdesc' => $pdesc,
+ 'marital' => $marital,
+ 'homepage' => $homepage,
+ 'about' => $about,
+ 'photo_menu' => $photo_menu,
+
+ ];
+
+ $hook = ['contact' => $arr, 'entry' => $entry];
+
+ Hook::callAll('directory_item', $hook);
+
+ unset($profile);
+ unset($location);
+
+ return $hook['entry'];
+}