Elgg / Elgg

views/default/input/form.php

<?php
/**
 * Create a form for data submission.
 * Use this view for forms as it provides protection against CSRF attacks.
 *
 * @package Elgg
 * @subpackage Core
 *
 * @uses $vars['body'] The body of the form (made up of other input/xxx views and html
 * @uses $vars['action'] The action URL of the form
 * @uses $vars['action_name'] The name of the action (for targeting particular forms while extending)
 * @uses $vars['method'] The submit method: post (default) or get
 * @uses $vars['enctype'] Set to 'multipart/form-data' if uploading a file
 * @uses $vars['disable_security'] turn off CSRF security by setting to true
 * @uses $vars['class'] Additional class for the form
 * @uses $vars['ignore_empty_body'] Boolean (default true) to determine if an empty body should return continue
 */

$defaults = [
	'method' => 'post',
	'disable_security' => false,
];

$vars = array_merge($defaults, $vars);

$vars['class'] = elgg_extract_class($vars, 'elgg-form');
$vars['action'] = elgg_normalize_url($vars['action']);

It seems like $vars['action'] can also be of type string[]; however, parameter $url of elgg_normalize_url() does only seem to accept string, maybe add an additional type check?

$vars['method'] = strtolower($vars['method']);

It seems like $vars['method'] can also be of type string[]; however, parameter $str of strtolower() does only seem to accept string, maybe add an additional type check?

$ignore_empty_body = (bool) elgg_extract('ignore_empty_body', $vars, true);
unset($vars['ignore_empty_body']);

$body = $vars['body'];
unset($vars['body']);

if (!$ignore_empty_body && empty($body)) {
	return;
}

// Generate a security header
if (!$vars['disable_security']) {
	$body = elgg_view('input/securitytoken') . $body;
}
unset($vars['disable_security']);
unset($vars['action_name']);

echo elgg_format_element('form', $vars, "<fieldset>$body</fieldset>");