silverstripe / silverstripe-translatable

code/model/Translatable.php

Doc Comments +3 added lines, -6 removed lines

@@ -287,7 +287,7 @@ 
      * any database records (like pages), as this locale will be attached
      * to all new records.
      * 
-     * @param $locale String
+     * @param string $locale String
      */
     public static function set_default_locale($locale)
     {
@@ -323,7 +323,6 @@ 
      * to "auto-filter" all SELECT queries by this language.
      * See {@link disable_locale_filter()} on how to override this behaviour temporarily.
      * 
-     * @param string $lang New reading language.
      */
     public static function set_current_locale($locale)
     {
@@ -370,8 +369,6 @@ 
      * @param string $join A single join clause.  This can be used for filtering, only 1 
      *               instance of each DataObject will be returned.
      * @param string $limit A limit expression to be inserted into the LIMIT clause.
-     * @param string $containerClass The container class to return the results in.
-     * @param string $having A filter to be inserted into the HAVING clause.
      * @return mixed The objects matching the conditions.
      */
     public static function get_by_locale($class, $locale, $filter = '', $sort = '', $join = "", $limit = "")
@@ -638,7 +635,7 @@ 
      * Check if a given SQLSelect filters on the Locale field
      *
      * @param SQLSelect $query
-     * @return boolean
+     * @return boolean|null
      */
     protected function filtersOnLocale($query)
     {
@@ -1677,7 +1674,7 @@ 
     }
     
     /**
-     * @return boolean
+     * @return null|false
      */
     public function canEdit($member)
     {

tests/unit/TranslatableTest.php

Doc Comments +7 added lines

@@ -45,6 +45,9 @@ 
         parent::tearDown();
     }
 
+    /**
+     * @param string $message
+     */
     public function assertArrayEqualsAfterSort($expected, $actual, $message = null)
     {
         sort($expected);
@@ -233,6 +236,10 @@ 
         );
     }
 
+    /**
+     * @param string $class
+     * @param DataObject|null $node
+     */
     public function assertClass($class, $node)
     {
         $this->assertNotNull($node);