| @@ -72,7 +72,7 @@ discard block | ||
| 72 | 72 | /** | 
| 73 | 73 | * Who is this email going to | 
| 74 | 74 | * | 
| 75 | - * @return array The recipients of the email | |
| 75 | + * @return string The recipients of the email | |
| 76 | 76 | */ | 
| 77 | 77 | public function getToAddresses() | 
| 78 | 78 |      { | 
| @@ -82,7 +82,7 @@ discard block | ||
| 82 | 82 | /** | 
| 83 | 83 | * Who is this email going to (CC) | 
| 84 | 84 | * | 
| 85 | - * @return array The recipients of the email | |
| 85 | + * @return string The recipients of the email | |
| 86 | 86 | */ | 
| 87 | 87 | public function getCCAddresses() | 
| 88 | 88 |      { | 
| @@ -92,7 +92,7 @@ discard block | ||
| 92 | 92 | /** | 
| 93 | 93 | * Who is this email going to (BCC) | 
| 94 | 94 | * | 
| 95 | - * @return array The recipients of the email | |
| 95 | + * @return string The recipients of the email | |
| 96 | 96 | */ | 
| 97 | 97 | public function getBCCAddresses() | 
| 98 | 98 |      { | 
| @@ -90,7 +90,7 @@ | ||
| 90 | 90 | * We need the ability to obtain the user's unhashed plain text password to allow for it to be sent | 
| 91 | 91 | * to the correct backend which will hash it | 
| 92 | 92 | * | 
| 93 | - * @return boolean|string The current password | |
| 93 | + * @return boolean The current password | |
| 94 | 94 | */ | 
| 95 | 95 | public function getPassword() | 
| 96 | 96 |      { | 
| @@ -140,7 +140,7 @@ discard block | ||
| 140 | 140 | /** | 
| 141 | 141 | * Get the data table for Pending Users | 
| 142 | 142 | * | 
| 143 | - * @return boolean|\Data\DataTable The Pending User Data Table | |
| 143 | + * @return boolean The Pending User Data Table | |
| 144 | 144 | */ | 
| 145 | 145 | private function getPendingUserDataTable() | 
| 146 | 146 |      { | 
| @@ -213,6 +213,9 @@ discard block | ||
| 213 | 213 | return new SQLUser($users[0], $this); | 
| 214 | 214 | } | 
| 215 | 215 | |
| 216 | + /** | |
| 217 | + * @param string $dataTableName | |
| 218 | + */ | |
| 216 | 219 | private function getDataByFilter($dataTableName, $filter, $select, $top, $skip, $orderby) | 
| 217 | 220 |      { | 
| 218 | 221 | $dataTable = $this->getDataTable($dataTableName); | 
| @@ -278,6 +281,9 @@ discard block | ||
| 278 | 281 | return $dataTable->count(); | 
| 279 | 282 | } | 
| 280 | 283 | |
| 284 | + /** | |
| 285 | + * @param \Data\Filter $filter | |
| 286 | + */ | |
| 281 | 287 | private function searchPendingUsers($filter, $select, $top, $skip, $orderby) | 
| 282 | 288 |      { | 
| 283 | 289 | $userDataTable = $this->getPendingUserDataTable(); | 
| @@ -113,6 +113,9 @@ discard block | ||
| 113 | 113 |          throw new \Exception('No such table '.$name); | 
| 114 | 114 | } | 
| 115 | 115 | |
| 116 | + /** | |
| 117 | + * @param string $tablename | |
| 118 | + */ | |
| 116 | 119 | function read($tablename, $where = false, $select = '*', $count = false, $skip = false, $sort = false) | 
| 117 | 120 |      { | 
| 118 | 121 | if($select === false) | 
| @@ -158,6 +161,9 @@ discard block | ||
| 158 | 161 | return $ret; | 
| 159 | 162 | } | 
| 160 | 163 | |
| 164 | + /** | |
| 165 | + * @param string $tablename | |
| 166 | + */ | |
| 161 | 167 | function update($tablename, $where, $data) | 
| 162 | 168 |      { | 
| 163 | 169 | $set = array(); | 
| @@ -180,6 +186,9 @@ discard block | ||
| 180 | 186 | return true; | 
| 181 | 187 | } | 
| 182 | 188 | |
| 189 | + /** | |
| 190 | + * @param string $tablename | |
| 191 | + */ | |
| 183 | 192 | function create($tablename, $data) | 
| 184 | 193 |      { | 
| 185 | 194 | $set = array(); | 
| @@ -203,6 +212,9 @@ discard block | ||
| 203 | 212 | return true; | 
| 204 | 213 | } | 
| 205 | 214 | |
| 215 | + /** | |
| 216 | + * @param string $tablename | |
| 217 | + */ | |
| 206 | 218 | function delete($tablename, $where) | 
| 207 | 219 |      { | 
| 208 | 220 | $sql = "DELETE FROM $tablename WHERE $where"; | 
| @@ -210,7 +210,7 @@ | ||
| 210 | 210 | /** | 
| 211 | 211 | * Get the LDAP filter represented by the passed object | 
| 212 | 212 | * | 
| 213 | - * @param boolean|string|\Data\Filter $filter The fiter to use | |
| 213 | + * @param boolean $filter The fiter to use | |
| 214 | 214 | * | 
| 215 | 215 | * @return string The filter in LDAP format | 
| 216 | 216 | */ | 
| @@ -325,7 +325,7 @@ | ||
| 325 | 325 | |
| 326 | 326 | |
| 327 | 327 | /** | 
| 328 | - * @param boolean|\Data\Filter $filter The filter to user when reading users | |
| 328 | + * @param \Data\Filter $filter The filter to user when reading users | |
| 329 | 329 | * @param boolean|array $select The fields to return | 
| 330 | 330 | * @param boolean|integer $top The number of records to return | 
| 331 | 331 | * @param boolean|integer $skip The number of records to skip | 
| @@ -43,7 +43,7 @@ | ||
| 43 | 43 | * | 
| 44 | 44 | * @param string $fieldName The name of the field to retrieve | 
| 45 | 45 | * | 
| 46 | - * @return mixed string the value of the field | |
| 46 | + * @return string string the value of the field | |
| 47 | 47 | */ | 
| 48 | 48 | protected function getFieldSingleValue($fieldName) | 
| 49 | 49 |      { | 
| @@ -253,10 +253,10 @@ | ||
| 253 | 253 | * Get an array of Auth\PendingUser from a filtered set | 
| 254 | 254 | * | 
| 255 | 255 | * @param Data\Filter|boolean $filter The filter conditions or false to retreive all | 
| 256 | - * @param array|boolean $select The user fields to obtain or false to obtain all | |
| 257 | - * @param integer|boolean $top The number of users to obtain or false to obtain all | |
| 258 | - * @param integer|boolean $skip The number of users to skip or false to skip none | |
| 259 | - * @param array|boolean $orderby The field to sort by and the method to sort or false to not sort | |
| 256 | + * @param boolean $select The user fields to obtain or false to obtain all | |
| 257 | + * @param boolean $top The number of users to obtain or false to obtain all | |
| 258 | + * @param boolean $skip The number of users to skip or false to skip none | |
| 259 | + * @param boolean $orderby The field to sort by and the method to sort or false to not sort | |
| 260 | 260 | * @param string|boolean $methodName The AuthMethod if information is desired only from a particular Auth\Authenticator | 
| 261 | 261 | * | 
| 262 | 262 | * @return array|boolean An array of Auth\PendingUser objects or false if no pending users were found | 
| @@ -93,7 +93,7 @@ discard block | ||
| 93 | 93 | /** | 
| 94 | 94 | * The groups the user is a part of | 
| 95 | 95 | * | 
| 96 | - * @return boolean|array The user's Auth\Group structures | |
| 96 | + * @return boolean The user's Auth\Group structures | |
| 97 | 97 | */ | 
| 98 | 98 | public function getGroups() | 
| 99 | 99 |      { | 
| @@ -576,7 +576,7 @@ discard block | ||
| 576 | 576 | * | 
| 577 | 577 | * @param stdClass $data The user's new data | 
| 578 | 578 | * | 
| 579 | - * @return boolean true if the user's data was changed, false otherwise | |
| 579 | + * @return boolean|null true if the user's data was changed, false otherwise | |
| 580 | 580 | */ | 
| 581 | 581 | public function editUser($data) | 
| 582 | 582 |      { | 
| @@ -603,7 +603,7 @@ discard block | ||
| 603 | 603 | /** | 
| 604 | 604 | * Obtain the user's password reset hash | 
| 605 | 605 | * | 
| 606 | - * @return string|false A hash if available, false otherwise | |
| 606 | + * @return boolean A hash if available, false otherwise | |
| 607 | 607 | */ | 
| 608 | 608 | public function getPasswordResetHash() | 
| 609 | 609 |      { |