fixed some coding standards issues as detected by code sniffer

diff --git a/engine/classes/ElggPriorityList.php b/engine/classes/ElggPriorityList.php
index b5f8fe16397c301145f311d6cf92f0ea84eff217..416df885c59c09b904ec4838d5e4377124ac64d0 100644 (file)
--- a/engine/classes/ElggPriorityList.php
+++ b/engine/classes/ElggPriorityList.php
@@ -165,9 +165,9 @@ class ElggPriorityList
        /**
         * Move an existing element to a new priority.
         *
-        * @param mixed  $element      The element to move
-        * @param int    $new_priority The new priority for the element
-        * @param bool   $strict       Whether to check the type of the element match
+        * @param mixed $element      The element to move
+        * @param int   $new_priority The new priority for the element
+        * @param bool  $strict       Whether to check the type of the element match
         * @return bool
         */
        public function move($element, $new_priority, $strict = false) {
@@ -354,7 +354,12 @@ class ElggPriorityList
                return ($key !== NULL && $key !== FALSE);
        }
 
-       // Countable
+       /**
+        * Countable interface
+        *
+        * @see Countable::count()
+        * @return int
+        */
        public function count() {
                return count($this->elements);
        }

diff --git a/engine/classes/ElggTranslit.php b/engine/classes/ElggTranslit.php
index 676c59fc889597f21b2537f8ef5e24521a4f7436..79116fc018eb42c726aa821b234e1227f463a578 100644 (file)
--- a/engine/classes/ElggTranslit.php
+++ b/engine/classes/ElggTranslit.php
@@ -20,11 +20,10 @@
  * and is licensed under the LGPL. For more information, see
  * <http://www.doctrine-project.org>.
  *
- * @author      Konsta Vesterinen <kvesteri@cc.hut.fi>
- * @author      Jonathan H. Wage <jonwage@gmail.com>
- *
- * @author      Steve Clay <steve@mrclay.org>
- * @package     Elgg.Core
+ * @package Elgg.Core
+ * @author  Konsta Vesterinen <kvesteri@cc.hut.fi>
+ * @author  Jonathan H. Wage <jonwage@gmail.com>
+ * @author  Steve Clay <steve@mrclay.org>
  *
  * @access private Plugin authors should not use this directly
  */
@@ -32,8 +31,9 @@ class ElggTranslit {
 
        /**
         * Create a version of a string for embedding in a URL
-        * @param string $string a UTF-8 string
-        * @param string $separator
+        *
+        * @param string $string    A UTF-8 string
+        * @param string $separator The character to separate words with
         * @return string
         */
        static public function urlize($string, $separator = '-') {
@@ -98,6 +98,7 @@ class ElggTranslit {
 
        /**
         * Transliterate Western multibyte chars to ASCII
+        *
         * @param string $utf8 a UTF-8 string
         * @return string
         */
@@ -247,6 +248,7 @@ class ElggTranslit {
 
        /**
         * Tests that "normalizer_normalize" exists and works
+        *
         * @return bool
         */
        static public function hasNormalizerSupport() {
@@ -255,7 +257,7 @@ class ElggTranslit {
                        $form_c = "\xC3\x85"; // 'LATIN CAPITAL LETTER A WITH RING ABOVE' (U+00C5)
                        $form_d = "A\xCC\x8A"; // A followed by 'COMBINING RING ABOVE' (U+030A)
                        $ret = (function_exists('normalizer_normalize')
-                                   && $form_c === normalizer_normalize($form_d));
+                               && $form_c === normalizer_normalize($form_d));
                }
                return $ret;
        }