diff options
| author | Steve Clay <steve@mrclay.org> | 2012-10-10 18:22:31 -0700 |
|---|---|---|
| committer | Steve Clay <steve@mrclay.org> | 2012-10-10 18:22:31 -0700 |
| commit | d134beadb79fcc90a75bda8bbcbfb9987b27470c (patch) | |
| tree | 3228eb49db250d8d1a5a0b36eb58c7051b4d8bbb /mod/groups | |
| parent | ec68ded7bc64e64d27f9c78efdfa6a7d3d25d14e (diff) | |
| parent | 5efa9426d40326b8d31c152dd2a433076b490308 (diff) | |
| download | elgg-d134beadb79fcc90a75bda8bbcbfb9987b27470c.tar.gz elgg-d134beadb79fcc90a75bda8bbcbfb9987b27470c.tar.bz2 | |
Merge pull request #381 from mrclay/4593-plaintext-titles
Fixes #4593: All titles are HTML-escaped plain text
Diffstat (limited to 'mod/groups')
| -rw-r--r-- | mod/groups/actions/discussion/save.php | 2 | ||||
| -rw-r--r-- | mod/groups/actions/groups/edit.php | 3 |
2 files changed, 2 insertions, 3 deletions
diff --git a/mod/groups/actions/discussion/save.php b/mod/groups/actions/discussion/save.php index de4afadfb..b3e9da654 100644 --- a/mod/groups/actions/discussion/save.php +++ b/mod/groups/actions/discussion/save.php @@ -4,7 +4,7 @@ */ // Get variables -$title = get_input("title"); +$title = htmlspecialchars(get_input('title', '', false), ENT_QUOTES, 'UTF-8'); $desc = get_input("description"); $status = get_input("status"); $access_id = (int) get_input("access_id"); diff --git a/mod/groups/actions/groups/edit.php b/mod/groups/actions/groups/edit.php index df2464a65..a4169461a 100644 --- a/mod/groups/actions/groups/edit.php +++ b/mod/groups/actions/groups/edit.php @@ -33,8 +33,7 @@ foreach ($CONFIG->group as $shortname => $valuetype) { } } -$input['name'] = get_input('name'); -$input['name'] = html_entity_decode($input['name'], ENT_COMPAT, 'UTF-8'); +$input['name'] = htmlspecialchars(get_input('name', '', false), ENT_QUOTES, 'UTF-8'); $user = elgg_get_logged_in_user_entity(); |