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()
 	{

tests/includes/stringsTest.php

Indentation +34 added lines, -34 removed lines

@@ -2,7 +2,7 @@ 
 
 class stringsTest extends PHPUnit_Framework_TestCase
 {
-    private $e;
+	private $e;
 
 	public function setUp()
 	{
@@ -11,38 +11,38 @@ 
 	}
 
 	public function tearDown() {
-        $this->e = NULL;
-    }
-
-    public function test_struname() {
-        // Happy path
-        $this->assertEquals($this->e->struname("this"), "This");
-        $this->assertEquals($this->e->struname("1this"), "1this");
-        $this->assertEquals($this->e->struname("This"), "This");
-        $this->assertEquals($this->e->struname("This "), "This");
-        $this->assertEquals($this->e->struname("This_"), "This");
-
-        // Sad Path
-        $this->assertNotEquals($this->e->struname("This "), "This ");
-        $this->assertNotEquals($this->e->struname("This_"), "This_");
-        $this->assertNotEquals($this->e->struname("this"), "this");
-        $this->assertNotEquals($this->e->struname("1this"), "1This");
-    }
-
-    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");
-        $this->assertEquals($this->e->stremail("1this12345@example.com "), "1this12345@example.com");
-        $this->assertEquals($this->e->stremail("1this12345 @example.com"), "1this12345@example.com");
-        $this->assertEquals($this->e->stremail("1this12345@ example.com"), "1this12345@example.com");
-
-        // Sad Path
-        $this->assertNotEquals($this->e->stremail(" 1this12345@example.com"), " 1this12345@example.com");
-        $this->assertNotEquals($this->e->stremail("1this12345@example.com "), "1this12345@example.com ");
-        $this->assertNotEquals($this->e->stremail("1this12345 @example.com"), "1this12345 @example.com");
-        $this->assertNotEquals($this->e->stremail("1this12345@ example.com"), "1this12345@ example.com");
-
-    }
+		$this->e = NULL;
+	}
+
+	public function test_struname() {
+		// Happy path
+		$this->assertEquals($this->e->struname("this"), "This");
+		$this->assertEquals($this->e->struname("1this"), "1this");
+		$this->assertEquals($this->e->struname("This"), "This");
+		$this->assertEquals($this->e->struname("This "), "This");
+		$this->assertEquals($this->e->struname("This_"), "This");
+
+		// Sad Path
+		$this->assertNotEquals($this->e->struname("This "), "This ");
+		$this->assertNotEquals($this->e->struname("This_"), "This_");
+		$this->assertNotEquals($this->e->struname("this"), "this");
+		$this->assertNotEquals($this->e->struname("1this"), "1This");
+	}
+
+	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");
+		$this->assertEquals($this->e->stremail("1this12345@example.com "), "1this12345@example.com");
+		$this->assertEquals($this->e->stremail("1this12345 @example.com"), "1this12345@example.com");
+		$this->assertEquals($this->e->stremail("1this12345@ example.com"), "1this12345@example.com");
+
+		// Sad Path
+		$this->assertNotEquals($this->e->stremail(" 1this12345@example.com"), " 1this12345@example.com");
+		$this->assertNotEquals($this->e->stremail("1this12345@example.com "), "1this12345@example.com ");
+		$this->assertNotEquals($this->e->stremail("1this12345 @example.com"), "1this12345 @example.com");
+		$this->assertNotEquals($this->e->stremail("1this12345@ example.com"), "1this12345@ example.com");
+
+	}
 
 }

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");

Upper-Lower-Casing +1 added lines, -1 removed lines

@@ -11,7 +11,7 @@
 	}
 
 	public function tearDown() {
-        $this->e = NULL;
+        $this->e = null;
     }
 
     public function test_struname() {

includes/DataObjects/Log.php

Doc Comments +1 added lines, -1 removed lines

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

tests/includes/StringFunctionsTest.php

Spacing +1 added lines, -1 removed lines

@@ -17,7 +17,7 @@
 		$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");

Braces +2 added lines, -1 removed lines

@@ -17,7 +17,8 @@
 		$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");