silverstripe / silverstripe-framework

tests/php/Dev/DevAdminControllerTest.php

Doc Comments +6 added lines

@@ -59,6 +59,9 @@ 
 
 
 
+    /**
+     * @param string $url
+     */
     protected function getCapture($url)
     {
         $this->logInWithPermission('ADMIN');
@@ -71,6 +74,9 @@ 
         return $r;
     }
 
+    /**
+     * @param string $url
+     */
     protected function getAndCheckForError($url)
     {
         $this->logInWithPermission('ADMIN');

tests/php/Forms/EmailFieldTest.php

Doc Comments +5 added lines

@@ -40,6 +40,11 @@
         $this->internalCheck("domain.but.no.user", "Invalid, no user part", false);
     }
 
+    /**
+     * @param string $email
+     * @param string $checkText
+     * @param boolean $expectSuccess
+     */
     public function internalCheck($email, $checkText, $expectSuccess)
     {
         $field = new EmailField("MyEmail");

tests/php/View/ContentNegotiatorTest.php

Doc Comments +1 added lines

@@ -14,6 +14,7 @@
     /**
      * Small helper to render templates from strings
      * Cloned from SSViewerTest
+     * @param string $templateString
      */
     private function render($templateString, $data = null)
     {

src/Forms/DatetimeField.php

Doc Comments +12 added lines, -3 removed lines

@@ -98,6 +98,9 @@ 
         return $this;
     }
 
+    /**
+     * @param string $name
+     */
     public function setName($name)
     {
         parent::setName($name);
@@ -137,6 +140,9 @@ 
         return $this->html5;
     }
 
+    /**
+     * @param boolean $bool
+     */
     public function setHTML5($bool)
     {
         $this->html5 = $bool;
@@ -146,7 +152,7 @@ 
     /**
      * Assign value posted from form submission
      *
-     * @param mixed $value
+     * @param string $value
      * @param mixed $data
      * @return $this
      */
@@ -335,7 +341,7 @@ 
     /**
      * Assign value from iso8601 string
      *
-     * @param mixed $value
+     * @param string $value
      * @param mixed $data
      * @return $this
      */
@@ -437,7 +443,7 @@ 
     /**
      * Tidy up iso8601-ish date, or approximation
      *
-     * @param string $date Date in ISO 8601 or approximate form
+     * @param string $datetime
      * @return string iso8601 date, or null if not valid
      */
     public function tidyISO8601($datetime)
@@ -532,6 +538,9 @@ 
         return $this;
     }
 
+    /**
+     * @param boolean $bool
+     */
     public function setReadonly($bool)
     {
         parent::setReadonly($bool);