Elgg / Elgg

mod/profile/views/default/profile/fields.php

<?php
/**
 * @uses $vars['entity']       The user entity
 * @uses $vars['microformats'] Mapping of fieldnames to microformats
 * @uses $vars['fields']       Array of profile fields to show
 */

$microformats = [
	'mobile' => 'tel p-tel',
	'phone' => 'tel p-tel',
	'website' => 'url u-url',
	'contactemail' => 'email u-email',
];
$microformats = array_merge($microformats, (array) elgg_extract('microformats', $vars, []));

$user = elgg_extract('entity', $vars);
if (!($user instanceof ElggUser)) {
	return;
}

$fields = (array) elgg_extract('fields', $vars, []);
if (empty($fields)) {
	return;
}

// move description to the bottom of the list
if (isset($fields['description'])) {
	$temp = $fields['description'];
	unset($fields['description']);
	$fields['description'] = $temp;
}

foreach ($fields as $shortname => $valtype) {
	$annotations = $user->getAnnotations([
		'annotation_names' => "profile:$shortname",
		'limit' => false,
	]);
	$values = array_map(function (ElggAnnotation $a) {
		return $a->value;
	}, $annotations);

	if (!$values) {

The expression $values of type array is implicitly converted to a boolean; are you sure this is intended? If so, consider using empty($expr) instead to make it clear that you intend to check for an array without elements.

		continue;
	}
	// emulate metadata API
	$value = (count($values) === 1) ? $values[0] : $values;

	// validate urls
	if ($valtype == 'url' && !preg_match('~^https?\://~i', $value)) {
		$value = "http://$value";
	}

	$class = elgg_extract($shortname, $microformats, '');

	$field_title = elgg_echo("profile:{$shortname}");
	$field_value = elgg_format_element('span', [
		'class' => $class,
	], elgg_view("output/{$valtype}", [
		'value' => $value,
	]));

	echo <<<___FIELD
	<div class='clearfix profile-field'>
		<div class='elgg-col elgg-col-1of5'>
			<b>{$field_title}:</b>
		</div>
		<div class='elgg-col elgg-col-4of5'>
			{$field_value}
		</div>
	</div>
___FIELD;
}