enwikipedia-acc / waca

includes/DataObjects/Comment.php

Doc Comments +2 added lines, -2 removed lines

@@ -87,7 +87,7 @@ 
 
 	/**
 	 * Summary of getUserObject
-	 * @return User|null
+	 * @return DataObject|null
 	 */
 	public function getUserObject()
 	{
@@ -126,7 +126,7 @@ 
 
 	/**
 	 * Summary of getRequestObject
-	 * @return Request|null
+	 * @return DataObject|null
 	 */
 	public function getRequestObject()
 	{

includes/DataObjects/Log.php

Doc Comments +2 added lines, -2 removed lines

@@ -85,7 +85,7 @@ 
 	
 	/**
 	 * Summary of getUserObject
-	 * @return User|null
+	 * @return DataObject|null
 	 */
 	public function getUserObject()
 	{
@@ -94,7 +94,7 @@ 
 
 	/**
 	 * Summary of setUser
-	 * @param int|User $user 
+	 * @param User $user 
 	 */
 	public function setUser($user)
 	{

includes/DataObjects/User.php

Braces +4 added lines, -2 removed lines

@@ -822,11 +822,13 @@
     
 	#endregion
     
-	public function getForgottenPasswordHash() {
+	public function getForgottenPasswordHash()
+	{
 		return md5($this->username . $this->email . $this->welcome_template . $this->id . $this->password);
 	}
 
-	public function getApprovalDate() {
+	public function getApprovalDate()
+	{
 		$query = $this->dbObject->prepare(<<<SQL
 			SELECT timestamp 
 			FROM log 

includes/Helpers/Logger.php

Braces +1 added lines, -2 removed lines

@@ -425,8 +425,7 @@
 			);
 		
 		$statement = $database->query("SELECT CONCAT('Closed ', id) as k, CONCAT('closed (',name,')') as v FROM emailtemplate;");
-		foreach ($statement->fetchAll(PDO::FETCH_ASSOC) as $row)
-		{
+		foreach ($statement->fetchAll(PDO::FETCH_ASSOC) as $row) {
 			$lookup[$row['k']] = $row['v'];
 		}
 		

acc.php

Braces +2 added lines, -2 removed lines

@@ -1281,7 +1281,7 @@ 
 		if ($_POST['action'] == EmailTemplate::CREATED || $_POST['action'] == EmailTemplate::NOT_CREATED) {
 			$request->setStatus('Closed');
 
-			if($_POST['action'] == EmailTemplate::CREATED){
+			if($_POST['action'] == EmailTemplate::CREATED) {
 				$gem  = 'custom-y';
 				$crea = "Custom, Created";
 			}
@@ -1331,7 +1331,7 @@ 
 		BootstrapSkin::displayInternalFooter();
 		die();		
 	}
-	else{
+	else {
 		// Not a custom close, just a normal close
 	    
 		$request->setStatus('Closed');

includes/strings.php

Doc Comments +2 added lines

@@ -35,6 +35,7 @@ 
 	/**
 	 * Formats a string to be used as a username.
 	 * @deprecated StringFunctions class
+	 * @param string $username
 	 */
 	public function struname($username)
 	{
@@ -50,6 +51,7 @@ 
 	 * from the beginning/end of the Email, as well as immediately before/after
 	 * the @ in the Email).
 	 * @deprecated StringFunctions class
+	 * @param string $email
 	 */
 	public function stremail($email)
 	{

tests/includes/IrcColourCodeTest.php

Braces +4 added lines, -2 removed lines

@@ -10,11 +10,13 @@
 		$this->irc = new IrcColourCode();
 	}
 
-	public function tearDown() {
+	public function tearDown()
+	{
         $this->irc = NULL;
     }
 
-    public function test_colorCodes() {
+    public function test_colorCodes()
+    {
 
         $i = $this->irc;
 

tests/includes/StringFunctionsTest.php

Braces +6 added lines, -3 removed lines

@@ -10,11 +10,13 @@ 
 		$this->e = new StringFunctions();
 	}
 
-	public function tearDown() {
+	public function tearDown()
+	{
         $this->e = NULL;
     }
 
-    public function test_formatAsUsername() {
+    public function test_formatAsUsername()
+    {
         // Happy path
         $this->assertEquals($this->e->formatAsUsername("this"), "This");
         $this->assertEquals($this->e->formatAsUsername("1this"), "1this");
@@ -29,7 +31,8 @@ 
         $this->assertNotEquals($this->e->formatAsUsername("1this"), "1This");
     }
 
-    public function test_formatAsEmail() {
+    public function test_formatAsEmail()
+    {
         $this->assertEquals($this->e->formatAsEmail("this@example.com"), "this@example.com");
         $this->assertEquals($this->e->formatAsEmail("1this12345@example.com"), "1this12345@example.com");
         $this->assertEquals($this->e->formatAsEmail(" 1this12345@example.com"), "1this12345@example.com");

tests/includes/stringsTest.php

Braces +6 added lines, -3 removed lines

@@ -10,11 +10,13 @@ 
 		$this->e = new strings();
 	}
 
-	public function tearDown() {
+	public function tearDown()
+	{
         $this->e = NULL;
     }
 
-    public function test_struname() {
+    public function test_struname()
+    {
         // Happy path
         $this->assertEquals($this->e->struname("this"), "This");
         $this->assertEquals($this->e->struname("1this"), "1this");
@@ -29,7 +31,8 @@ 
         $this->assertNotEquals($this->e->struname("1this"), "1This");
     }
 
-    public function test_stremail() {
+    public function test_stremail()
+    {
         $this->assertEquals($this->e->stremail("this@example.com"), "this@example.com");
         $this->assertEquals($this->e->stremail("1this12345@example.com"), "1this12345@example.com");
         $this->assertEquals($this->e->stremail(" 1this12345@example.com"), "1this12345@example.com");