From: Steve Clay <steve@mrclay.org>
Date: Sat, 21 Sep 2013 01:19:06 +0000 (-0400)
Subject: Test that get_user_by_username accepts URL encoded input
X-Git-Url: https://gitweb.fluxo.info/?a=commitdiff_plain;h=49ab3a17173aedb8b5e3a2a228cc6cfd0a510e49;p=lorea%2Felgg.git

Test that get_user_by_username accepts URL encoded input
---

diff --git a/engine/tests/objects/users.php b/engine/tests/objects/users.php
index dc9129326..7d2ef6961 100644
--- a/engine/tests/objects/users.php
+++ b/engine/tests/objects/users.php
@@ -159,6 +159,22 @@ class ElggCoreUserTest extends ElggCoreUnitTest {
 		$this->assertFalse($user);
 	}
 
+	public function testGetUserByUsernameAcceptsUrlEncoded() {
+		$username = (string)time();
+		$this->user->username = $username;
+		$guid = $this->user->save();
+
+		// percent encode first letter
+		$first_letter = $username[0];
+		$first_letter = str_pad('%' . dechex(ord($first_letter)), 2, '0', STR_PAD_LEFT);
+		$username =   $first_letter . substr($username, 1);
+
+		$user = get_user_by_username($username);
+		$this->assertTrue((bool) $user);
+		$this->assertEqual($guid, $user->guid);
+
+		$this->user->delete();
+	}
 
 	public function testElggUserMakeAdmin() {
 		global $CONFIG;