BurningFlipside / CommonCode

Auth/class.LDAPAuthenticator.php

Doc Comments +1 added lines, -1 removed lines

@@ -325,7 +325,7 @@
 
     
     /**
-     * @param boolean|\Data\Filter $filter The filter to user when reading users
+     * @param \Data\Filter $filter The filter to user when reading users
      * @param boolean|array   $select The fields to return
      * @param boolean|integer $top The number of records to return
      * @param boolean|integer $skip The number of records to skip

class.AuthProvider.php

Doc Comments +4 added lines, -4 removed lines

@@ -253,10 +253,10 @@
      * Get an array of Auth\PendingUser from a filtered set
      *
      * @param Data\Filter|boolean $filter The filter conditions or false to retreive all
-     * @param array|boolean $select The user fields to obtain or false to obtain all
-     * @param integer|boolean $top The number of users to obtain or false to obtain all
-     * @param integer|boolean $skip The number of users to skip or false to skip none
-     * @param array|boolean $orderby The field to sort by and the method to sort or false to not sort
+     * @param boolean $select The user fields to obtain or false to obtain all
+     * @param boolean $top The number of users to obtain or false to obtain all
+     * @param boolean $skip The number of users to skip or false to skip none
+     * @param boolean $orderby The field to sort by and the method to sort or false to not sort
      * @param string|boolean $methodName The AuthMethod if information is desired only from a particular Auth\Authenticator
      *
      * @return array|boolean An array of Auth\PendingUser objects or false if no pending users were found

Data/class.FilterClause.php

Doc Comments +9 added lines

@@ -28,12 +28,18 @@ 
         return substr($haystack, 0, strlen($needle)) === $needle;
     }
 
+    /**
+     * @param string $string
+     */
     protected function filterIsFunction($string)
     {
         return (self::str_startswith($string, 'substringof') || self::str_startswith($string, 'contains') ||
                 self::str_startswith($string, 'indexof'));
     }
 
+    /**
+     * @param string $op
+     */
     protected function odataOpToStd($op)
     {
         switch($op)
@@ -146,6 +152,9 @@ 
         }
     }
 
+    /**
+     * @param string $op
+     */
     private function getMongoClauseArray($op, $var2)
     {
         return array($this->opToMongo($op)=>$var2);

LDAP/class.LDAPServer.php

Doc Comments -1 removed lines

@@ -6,7 +6,6 @@
  * @author Chris Wright
  * @version 2.0
  * @param string $subject The subject string
- * @param bool $dn Treat subject as a DN if TRUE
  * @param string|array $ignore Set of characters to leave untouched
  * @return string The escaped string
  */

Auth/class.Authenticator.php

Doc Comments +9 added lines, -9 removed lines

@@ -92,13 +92,13 @@ 
     /**
      * Get the groups that match the filters
      *
-     * @param mixed $filter The filter to use to find a set of groups
+     * @param boolean $filter The filter to use to find a set of groups
      * @param mixed $select The list of fields to select
      * @param mixed $top The number of groups to select
      * @param mixed $skip The number of groups to skip
      * @param mixed $orderby The fields to sort by
      *
-     * @return array All groups that fit the filters
+     * @return boolean All groups that fit the filters
      *
      * @SuppressWarnings("UnusedFormalParameter")
      */
@@ -110,13 +110,13 @@ 
     /**
      * Get the users that match the filters
      *
-     * @param mixed $filter The filter to use to find a set of users
+     * @param boolean $filter The filter to use to find a set of users
      * @param mixed $select The list of fields to select
      * @param mixed $top The number of users to select
      * @param mixed $skip The number of users to skip
      * @param mixed $orderby The fields to sort by
      *
-     * @return array All users that fit the filters
+     * @return boolean All users that fit the filters
      *
      * @SuppressWarnings("UnusedFormalParameter")
      */
@@ -128,13 +128,13 @@ 
     /**
      * Get the pending users that match the filters
      *
-     * @param mixed $filter The filter to use to find a set of users
+     * @param boolean $filter The filter to use to find a set of users
      * @param mixed $select The list of fields to select
      * @param mixed $top The number of users to select
      * @param mixed $skip The number of users to skip
      * @param mixed $orderby The fields to sort by
      *
-     * @return array All users that fit the filters
+     * @return boolean All users that fit the filters
      *
      * @SuppressWarnings("UnusedFormalParameter")
      */
@@ -231,7 +231,7 @@ 
      *
      * @param string $hash The hash to search by
      *
-     * @return Auth\User The user whoes hash was specified
+     * @return boolean The user whoes hash was specified
      *
      * @SuppressWarnings("UnusedFormalParameter")
      */
@@ -245,7 +245,7 @@ 
      *
      * @param string $hash The hash to search by
      *
-     * @return Auth\PendingUser The user whoes hash was specified
+     * @return boolean The user whoes hash was specified
      *
      * @SuppressWarnings("UnusedFormalParameter")
      */
@@ -257,7 +257,7 @@ 
     /**
      * Get the host name this authenticator uses
      *
-     * @return string The host name this authenticator uses
+     * @return boolean The host name this authenticator uses
      */
     public function getHostName()
     {

Auth/class.SQLAuthenticator.php

Doc Comments +11 added lines, -1 removed lines

@@ -140,7 +140,7 @@ 
     /**
      * Get the data table for Pending Users
      *
-     * @return boolean|\Data\DataTable The Pending User Data Table
+     * @return boolean The Pending User Data Table
      */
     private function getPendingUserDataTable()
     {
@@ -189,6 +189,10 @@ 
         return new SQLUser($data, $this);
     }
 
+    /**
+     * @param string $tableName
+     * @param string $className
+     */
     private function getEntityByFilter($tableName, $filterStr, $className)
     {
         $dataTable = $this->getDataTable($tableName);
@@ -211,6 +215,9 @@ 
         return $this->getEntityByFilter('user', "uid eq '$name'", '\Auth\SQLUser');
     }
 
+    /**
+     * @param string $dataTableName
+     */
     private function getDataByFilter($dataTableName, $filter, $select, $top, $skip, $orderby)
     {
         $dataTable = $this->getDataTable($dataTableName);
@@ -276,6 +283,9 @@ 
         return $dataTable->count();
     }
 
+    /**
+     * @param \Data\Filter $filter
+     */
     private function searchPendingUsers($filter, $select, $top, $skip, $orderby)
     {
         $userDataTable = $this->getPendingUserDataTable();

Data/class.SQLDataSet.php

Doc Comments +14 added lines, -1 removed lines

@@ -116,6 +116,10 @@ 
         return $sql;
     }
 
+    /**
+     * @param boolean|string $count
+     * @param boolean|string $skip
+     */
     private function getLimitClause($count, $skip)
     {
         if($count === false)
@@ -139,7 +143,7 @@ 
      * @param string $select The colums to read
      * @param boolean|string $count The number of rows to read
      * @param boolean|string $skip The number of rows to skip over
-     * @param boolean|array $sort The array to sort by or false to not sort
+     * @param boolean $sort The array to sort by or false to not sort
      *
      * @return false|array An array of all the returned records
      */
@@ -177,6 +181,9 @@ 
         return $ret;
     }
 
+    /**
+     * @param string $tablename
+     */
     public function update($tablename, $where, $data)
     {
         $set = array();
@@ -199,6 +206,9 @@ 
         return true;
     }
 
+    /**
+     * @param string $tablename
+     */
     public function create($tablename, $data)
     {
         $set = array();
@@ -222,6 +232,9 @@ 
         return true;
     }
 
+    /**
+     * @param string $tablename
+     */
     public function delete($tablename, $where)
     {
         $sql = "DELETE FROM $tablename WHERE $where";

Email/class.SMTPServer.php

Spacing +1 added lines, -1 removed lines

@@ -32,7 +32,7 @@
     {
     }
 
-    public function connect($hostName, $port=25, $useTLS=true)
+    public function connect($hostName, $port = 25, $useTLS = true)
     {
         $ret = $this->smtp->connect($hostName, $port);
         if($ret === false)

Email/class.EmailRouter.php

Braces +4 added lines, -1 removed lines

@@ -41,7 +41,10 @@
         {
             $tmp = $this->routeSingle($dest, $rawMessage);
             $ret[] = $tmp;
-            if($tmp === false) $error = true;
+            if($tmp === false)
+            {
+                $error = true;
+            }
         }
         if($error === true)
         {