]> source.dussan.org Git - nextcloud-server.git/commitdiff
Fix PHPDoc in /apps
authorRobin McCorkell <rmccorkell@karoshi.org.uk>
Tue, 13 May 2014 11:29:25 +0000 (12:29 +0100)
committerRobin McCorkell <rmccorkell@karoshi.org.uk>
Fri, 16 May 2014 21:21:57 +0000 (22:21 +0100)
31 files changed:
apps/files/lib/helper.php
apps/files_encryption/hooks/hooks.php
apps/files_encryption/lib/crypt.php
apps/files_encryption/lib/helper.php
apps/files_encryption/lib/keymanager.php
apps/files_encryption/lib/proxy.php
apps/files_encryption/lib/session.php
apps/files_encryption/lib/stream.php
apps/files_encryption/lib/util.php
apps/files_encryption/tests/crypt.php
apps/files_encryption/tests/util.php
apps/files_external/lib/config.php
apps/files_external/lib/dropbox.php
apps/files_sharing/lib/api.php
apps/files_sharing/tests/base.php
apps/files_trashbin/lib/trashbin.php
apps/files_versions/lib/versions.php
apps/user_ldap/group_ldap.php
apps/user_ldap/group_proxy.php
apps/user_ldap/lib/access.php
apps/user_ldap/lib/configuration.php
apps/user_ldap/lib/connection.php
apps/user_ldap/lib/ildapwrapper.php
apps/user_ldap/lib/jobs.php
apps/user_ldap/lib/ldap.php
apps/user_ldap/lib/proxy.php
apps/user_ldap/lib/wizard.php
apps/user_ldap/lib/wizardresult.php
apps/user_ldap/tests/user_ldap.php
apps/user_ldap/user_ldap.php
apps/user_ldap/user_proxy.php

index 7e0f47bf84fc85e01f63d6670ac58077f9d9a159..58d0886a636309ebebe383d5991fdc6afc359f64 100644 (file)
@@ -167,7 +167,7 @@ class Helper
        /**
         * Sort the given file info array
         *
-        * @param \OCP\Files\FileInfo[] files to sort
+        * @param \OCP\Files\FileInfo[] $files files to sort
         * @param string $sortAttribute attribute to sort on
         * @param bool $sortDescending true for descending sort, false otherwise
         * @return \OCP\Files\FileInfo[] sorted files
index c9ffeb454a1b10896b1eaf85e727659793794513..fdc1126f448c75f2242ecd65cbbe76ad39f4cb8b 100644 (file)
@@ -262,7 +262,7 @@ class Hooks {
         * @brief check if files can be encrypted to every user.\r
         */\r
        /**\r
-        * @param $params\r
+        * @param array $params\r
         */\r
        public static function preShared($params) {\r
 \r
index 7d86eb72ed1e11b335e9b81c9fbfe771b2778c39..5cdc92e85f7d4c826d361209c99482314a7edcba 100755 (executable)
@@ -125,7 +125,7 @@ class Crypt {
 \r
        /**\r
         * @brief Check if a file's contents contains an IV and is symmetrically encrypted\r
-        * @param $content\r
+        * @param string $content\r
         * @return boolean\r
         * @note see also OCA\Encryption\Util->isEncryptedPath()\r
         */\r
@@ -179,10 +179,9 @@ class Crypt {
 \r
        /**\r
         * @brief Check if a file is encrypted via legacy system\r
-        * @param $data\r
+        * @param boolean $isCatFileContent\r
         * @param string $relPath The path of the file, relative to user/data;\r
         *        e.g. filename or /Docs/filename, NOT admin/files/filename\r
-        * @param boolean $isCatFileContent\r
         * @return boolean\r
         */\r
        public static function isLegacyEncryptedContent($isCatFileContent, $relPath) {\r
@@ -323,7 +322,7 @@ class Crypt {
 \r
        /**\r
         * @brief Symmetrically decrypts keyfile content\r
-        * @param $keyfileContent\r
+        * @param string $keyfileContent\r
         * @param string $passphrase\r
         * @throws \Exception\r
         * @return string|false\r
@@ -362,7 +361,7 @@ class Crypt {
         * @brief Decrypt private key and check if the result is a valid keyfile\r
         * @param string $encryptedKey encrypted keyfile\r
         * @param string $passphrase to decrypt keyfile\r
-        * @return encrypted private key or false\r
+        * @return string|false encrypted private key or false\r
         *\r
         * This function decrypts a file\r
         */\r
@@ -435,9 +434,9 @@ class Crypt {
 \r
        /**\r
         * @brief Asymmetrically encrypt a file using multiple public keys\r
-        * @param $encryptedContent\r
-        * @param $shareKey\r
-        * @param $privateKey\r
+        * @param string $encryptedContent\r
+        * @param string $shareKey\r
+        * @param mixed $privateKey\r
         * @return false|string\r
         * @internal param string $plainContent content to be encrypted\r
         * @return string $plainContent decrypted string\r
@@ -498,7 +497,7 @@ class Crypt {
 \r
        /**\r
         * @brief Generate a pseudo random 256-bit ASCII key, used as file key\r
-        * @return $key Generated key\r
+        * @return string|false Generated key\r
         */\r
        public static function generateKey() {\r
 \r
@@ -524,7 +523,7 @@ class Crypt {
 \r
        /**\r
         * @brief Get the blowfish encryption handler for a key\r
-        * @param $key string (optional)\r
+        * @param string $key (optional)\r
         * @return \Crypt_Blowfish blowfish object\r
         *\r
         * if the key is left out, the default handler will be used\r
@@ -561,7 +560,7 @@ class Crypt {
        }\r
 \r
        /**\r
-        * @param $data\r
+        * @param string $data\r
         * @param string $key\r
         * @param int $maxLength\r
         * @return string\r
index c3a31ff9fbc9b756449b8c45647141309de88dd2..2455c605dfed16c732d95ba32fcfc8582e70759e 100755 (executable)
@@ -102,7 +102,7 @@ class Helper {
        /**
         * @brief enable recovery
         *
-        * @param $recoveryKeyId
+        * @param string $recoveryKeyId
         * @param string $recoveryPassword
         * @internal param \OCA\Encryption\Util $util
         * @internal param string $password
index 5aa8a6ab06e270de42ebb46fd7b5ca102dba1a82..0732153f313c944587e9f471767f8a287a714d82 100755 (executable)
@@ -57,7 +57,7 @@ class Keymanager {
        /**
         * @brief retrieve public key for a specified user
         * @param \OC\Files\View $view
-        * @param $userId
+        * @param string $userId
         * @return string public key or false
         */
        public static function getPublicKey(\OC\Files\View $view, $userId) {
@@ -76,7 +76,7 @@ class Keymanager {
        /**
         * @brief Retrieve a user's public and private key
         * @param \OC\Files\View $view
-        * @param $userId
+        * @param string $userId
         * @return array keys: privateKey, publicKey
         */
        public static function getUserKeys(\OC\Files\View $view, $userId) {
@@ -297,7 +297,7 @@ class Keymanager {
         *
         * @param \OC\Files\View $view
         * @param string $path where the share key is stored
-        * @param $shareKey
+        * @param string $shareKey
         * @return bool true/false
         * @note The keyfile is not encrypted here. Client code must
         * asymmetrically encrypt the keyfile before passing it to this method
index 919853107a8c4031b6ceae6b1447faf467a23018..b5bb03cb1c1741ac1b937b243902f5f7d41238f9 100644 (file)
@@ -79,8 +79,8 @@ class Proxy extends \OC_FileProxy {
        }
 
        /**
-        * @param $path
-        * @param $data
+        * @param string $path
+        * @param string $data
         * @return bool
         */
        public function preFile_put_contents($path, &$data) {
@@ -225,7 +225,7 @@ class Proxy extends \OC_FileProxy {
 
 
        /**
-        * @param $path
+        * @param string $path
         * @param $result
         * @return resource
         */
@@ -261,8 +261,8 @@ class Proxy extends \OC_FileProxy {
        }
 
        /**
-        * @param $path
-        * @param $data
+        * @param string $path
+        * @param array $data
         * @return array
         */
        public function postGetFileInfo($path, $data) {
@@ -285,9 +285,9 @@ class Proxy extends \OC_FileProxy {
        }
 
        /**
-        * @param $path
-        * @param $size
-        * @return bool
+        * @param string $path
+        * @param int $size
+        * @return int|bool
         */
        public function postFileSize($path, $size) {
 
index dea27d7a18645253c963b7712127803fdd3b2ad5..a3870014b3c8a77ff0b74a5ebddcc3043b2c92a4 100644 (file)
@@ -121,7 +121,7 @@ class Session {
 
        /**
         * @brief Sets status of encryption app
-        * @param string $init  INIT_SUCCESSFUL, INIT_EXECUTED, NOT_INOITIALIZED
+        * @param string $init  INIT_SUCCESSFUL, INIT_EXECUTED, NOT_INITIALIZED
         * @return bool
         *
         * @note this doesn not indicate of the init was successful, we just remeber the try!
@@ -145,7 +145,7 @@ class Session {
 
        /**
         * @brief Gets status if we already tried to initialize the encryption app
-        * @return init status INIT_SUCCESSFUL, INIT_EXECUTED, NOT_INOITIALIZED
+        * @return string init status INIT_SUCCESSFUL, INIT_EXECUTED, NOT_INITIALIZED
         *
         * @note this doesn not indicate of the init was successful, we just remeber the try!
         */
index 8b575174e2edc2e0ea8f1d6b6f867861fccf5b0d..9a204e088a310a956bdcb9f7fc479aba4224e5e2 100644 (file)
@@ -79,10 +79,10 @@ class Stream {
        private $privateKey;
 
        /**
-        * @param $path raw path relative to data/
-        * @param $mode
-        * @param $options
-        * @param $opened_path
+        * @param string $path raw path relative to data/
+        * @param string $mode
+        * @param int $options
+        * @param string $opened_path
         * @return bool
         */
        public function stream_open($path, $mode, $options, &$opened_path) {
@@ -187,7 +187,7 @@ class Stream {
        }
 
        /**
-        * @param $offset
+        * @param int $offset
         * @param int $whence
         * @return bool true if fseek was successful, otherwise false
         */
@@ -202,7 +202,7 @@ class Stream {
        }
 
        /**
-        * @param $count
+        * @param int $count
         * @return bool|string
         * @throws \Exception
         */
@@ -426,9 +426,9 @@ class Stream {
 
 
        /**
-        * @param $option
-        * @param $arg1
-        * @param $arg2
+        * @param int $option
+        * @param int $arg1
+        * @param int|null $arg2
         */
        public function stream_set_option($option, $arg1, $arg2) {
                $return = false;
@@ -454,7 +454,7 @@ class Stream {
        }
 
        /**
-        * @param $mode
+        * @param int $mode
         */
        public function stream_lock($mode) {
                return flock($this->handle, $mode);
index b96e4590507c51719c09993ea2130d304fca48fe..84dbec6fdfcaf8a989d54ecd5840d1201378a309 100644 (file)
@@ -54,7 +54,7 @@ class Util {
 
        /**
         * @param \OC\Files\View $view
-        * @param $userId
+        * @param string $userId
         * @param bool $client
         */
        public function __construct($view, $userId, $client = false) {
@@ -1238,7 +1238,7 @@ class Util {
 
        /**
         * @brief check if files are already migrated to the encryption system
-        * @return migration status, false = in case of no record
+        * @return int|false migration status, false = in case of no record
         * @note If records are not being returned, check for a hidden space
         *       at the start of the uid in db
         */
@@ -1398,8 +1398,7 @@ class Util {
 
        /**
         * @brief get owner of the shared files.
-        * @param $id
-        * @internal param int $Id of a share
+        * @param int $id ID of a share
         * @return string owner
         */
        public function getOwnerFromSharedFile($id) {
@@ -1479,7 +1478,7 @@ class Util {
        }
 
        /**
-        * @param $password
+        * @param string $password
         * @return bool
         */
        public function checkRecoveryPassword($password) {
@@ -1639,7 +1638,7 @@ class Util {
 
        /**
         * @brief check if the file is stored on a system wide mount point
-        * @param $path relative to /data/user with leading '/'
+        * @param string $path relative to /data/user with leading '/'
         * @return boolean
         */
        public function isSystemWideMountPoint($path) {
index 83c2ab30b8ebf8543dd524a937244abc33d16c0f..b98798d72651d89ee32799ae4f09e46c08dda963 100755 (executable)
@@ -662,7 +662,7 @@ class Test_Encryption_Crypt extends \PHPUnit_Framework_TestCase {
        /**
         * @brief encryption using legacy blowfish method
         * @param string $data data to encrypt
-        * @param $passwd string password
+        * @param string $passwd password
         * @return string
         */
        function legacyEncrypt($data, $passwd) {
index 11961f2767508c0107c716687753882d70d51df1..395bf152cd9cd9f7097deeb1107ea056e9947f18 100755 (executable)
@@ -554,7 +554,7 @@ class Test_Encryption_Util extends \PHPUnit_Framework_TestCase {
        }
 
        /**
-        * @param $user
+        * @param string $user
         * @param bool $create
         * @param bool $password
         */
@@ -595,7 +595,7 @@ class Test_Encryption_Util extends \PHPUnit_Framework_TestCase {
         * to be able to test the migration path
         *
         * @param integer $status needed migration status for test
-        * @param $user for which user the status should be set
+        * @param string $user for which user the status should be set
         * @return boolean
         */
        private function setMigrationStatus($status, $user) {
index 9906b7670a8219e85f997fc88358f2547f3f8b93..e8618398c59315c9f9e2c7162621ec8cfac50b4e 100755 (executable)
@@ -610,9 +610,9 @@ class OC_Mount_Config {
 
        /**
         * Returns a dependency missing message
-        * @param $l OC_L10N
-        * @param $module string
-        * @param $backend string
+        * @param OC_L10N $l
+        * @param string $module
+        * @param string $backend
         * @return string
         */
        private static function getSingleDependencyMessage($l, $module, $backend) {
@@ -658,7 +658,7 @@ class OC_Mount_Config {
        /**
         * Encrypt a single password
         * @param string $password plain text password
-        * @return encrypted password
+        * @return string encrypted password
         */
        private static function encryptPassword($password) {
                $cipher = self::getCipher();
@@ -670,7 +670,7 @@ class OC_Mount_Config {
        /**
         * Decrypts a single password
         * @param string $encryptedPassword encrypted password
-        * @return plain text password
+        * @return string plain text password
         */
        private static function decryptPassword($encryptedPassword) {
                $cipher = self::getCipher();
index 38de3360f2b40c348427e5c15a04774ebd67a33f..9b0e6befee334863bb9fc449787e6164d82f65ef 100755 (executable)
@@ -65,8 +65,8 @@ class Dropbox extends \OC\Files\Storage\Common {
        /**
         * @brief Returns the path's metadata
         * @param string $path path for which to return the metadata
-        * @param $list if true, also return the directory's contents
-        * @return directory contents if $list is true, file metadata if $list is
+        * @param bool $list if true, also return the directory's contents
+        * @return mixed directory contents if $list is true, file metadata if $list is
         * false, null if the file doesn't exist or "false" if the operation failed
         */
        private function getMetaData($path, $list = false) {
index fd3c4a7756aa3fde9f139840c4f7ca9ad65ea080..5749836bed153a629d9c3de635d72ed7ac0e4fdd 100644 (file)
@@ -386,7 +386,7 @@ class Api {
        /**
         * @brief update password for public link share
         * @param array $share information about the share
-        * @param type $params 'password'
+        * @param array $params 'password'
         * @return \OC_OCS_Result
         */
        private static function updatePassword($share, $params) {
index 11bbcb133f5367169b1b9eb21e527e09ad3e73d6..27f9419810cc5d5d2093c3de26fc48c5bbd505bd 100644 (file)
@@ -97,7 +97,7 @@ abstract class Test_Files_Sharing_Base extends \PHPUnit_Framework_TestCase {
        }
 
        /**
-        * @param $user
+        * @param string $user
         * @param bool $create
         * @param bool $password
         */
index e2f220051c7758bd733d8b19721bc0e8c782ef68..5baa307299c90c126dfcaedee3ee8df316d1d647 100644 (file)
@@ -96,7 +96,7 @@ class Trashbin {
        /**
         * move file to the trash bin
         *
-        * @param $file_path path to the deleted file/directory relative to the files root directory
+        * @param string $file_path path to the deleted file/directory relative to the files root directory
         */
        public static function move2trash($file_path) {
                $user = \OCP\User::getUser();
@@ -155,11 +155,11 @@ class Trashbin {
        /**
         * Move file versions to trash so that they can be restored later
         *
-        * @param $file_path path to original file
-        * @param $filename of deleted file
+        * @param string $file_path path to original file
+        * @param string $filename of deleted file
         * @param integer $timestamp when the file was deleted
         *
-        * @return size of stored versions
+        * @return int size of stored versions
         */
        private static function retainVersions($file_path, $filename, $timestamp) {
                $size = 0;
@@ -200,11 +200,11 @@ class Trashbin {
        /**
         * Move encryption keys to trash so that they can be restored later
         *
-        * @param $file_path path to original file
-        * @param $filename of deleted file
+        * @param string $file_path path to original file
+        * @param string $filename of deleted file
         * @param integer $timestamp when the file was deleted
         *
-        * @return size of encryption keys
+        * @return int size of encryption keys
         */
        private static function retainEncryptionKeys($file_path, $filename, $timestamp) {
                $size = 0;
@@ -299,9 +299,9 @@ class Trashbin {
        /**
         * restore files from trash bin
         *
-        * @param $file path to the deleted file
-        * @param $filename name of the file
-        * @param $timestamp time when the file was deleted
+        * @param string $file path to the deleted file
+        * @param string $filename name of the file
+        * @param int $timestamp time when the file was deleted
         *
         * @return bool
         */
@@ -376,11 +376,11 @@ class Trashbin {
         * @brief restore versions from trash bin
         *
         * @param \OC\Files\View $view file view
-        * @param $file complete path to file
-        * @param $filename name of file once it was deleted
+        * @param string $file complete path to file
+        * @param string $filename name of file once it was deleted
         * @param string $uniqueFilename new file name to restore the file without overwriting existing files
-        * @param $location location if file
-        * @param $timestamp deleteion time
+        * @param string $location location if file
+        * @param int $timestamp deleteion time
         *
         */
        private static function restoreVersions($view, $file, $filename, $uniqueFilename, $location, $timestamp) {
@@ -424,11 +424,11 @@ class Trashbin {
         * @brief restore encryption keys from trash bin
         *
         * @param \OC\Files\View $view
-        * @param $file complete path to file
-        * @param $filename name of file
+        * @param string $file complete path to file
+        * @param string $filename name of file
         * @param string $uniqueFilename new file name to restore the file without overwriting existing files
-        * @param $location location of file
-        * @param $timestamp deleteion time
+        * @param string $location location of file
+        * @param int $timestamp deleteion time
         *
         */
        private static function restoreEncryptionKeys($view, $file, $filename, $uniqueFilename, $location, $timestamp) {
@@ -539,10 +539,10 @@ class Trashbin {
        /**
         * @brief delete file from trash bin permanently
         *
-        * @param $filename path to the file
-        * @param $timestamp of deletion time
+        * @param string $filename path to the file
+        * @param int $timestamp of deletion time
         *
-        * @return size of deleted files
+        * @return int size of deleted files
         */
        public static function delete($filename, $timestamp = null) {
                $user = \OCP\User::getUser();
@@ -634,9 +634,9 @@ class Trashbin {
        /**
         * check to see whether a file exists in trashbin
         *
-        * @param $filename path to the file
-        * @param $timestamp of deletion time
-        * @return true if file exists, otherwise false
+        * @param string $filename path to the file
+        * @param int $timestamp of deletion time
+        * @return bool true if file exists, otherwise false
         */
        public static function file_exists($filename, $timestamp = null) {
                $user = \OCP\User::getUser();
@@ -656,7 +656,7 @@ class Trashbin {
         * @brief deletes used space for trash bin in db if user was deleted
         *
         * @param type $uid id of deleted user
-        * @return result of db delete operation
+        * @return bool result of db delete operation
         */
        public static function deleteUser($uid) {
                $query = \OC_DB::prepare('DELETE FROM `*PREFIX*files_trash` WHERE `user`=?');
@@ -672,7 +672,7 @@ class Trashbin {
         * calculate remaining free space for trash bin
         *
         * @param integer $trashbinSize current size of the trash bin
-        * @return available free space for trash bin
+        * @return int available free space for trash bin
         */
        private static function calculateFreeSpace($trashbinSize) {
                $softQuota = true;
@@ -808,8 +808,8 @@ class Trashbin {
        /**
         * find all versions which belong to the file we want to restore
         *
-        * @param $filename name of the file which should be restored
-        * @param $timestamp timestamp when the file was deleted
+        * @param string $filename name of the file which should be restored
+        * @param int $timestamp timestamp when the file was deleted
         */
        private static function getVersionsFromTrash($filename, $timestamp) {
                $view = new \OC\Files\View('/' . \OCP\User::getUser() . '/files_trashbin/versions');
@@ -841,8 +841,8 @@ class Trashbin {
        /**
         * find unique extension for restored file if a file with the same name already exists
         *
-        * @param $location where the file should be restored
-        * @param $filename name of the file
+        * @param string $location where the file should be restored
+        * @param string $filename name of the file
         * @param \OC\Files\View $view filesystem view relative to users root directory
         * @return string with unique extension
         */
@@ -903,7 +903,7 @@ class Trashbin {
        /**
         * get current size of trash bin from a given user
         *
-        * @param $user user who owns the trash bin
+        * @param string $user user who owns the trash bin
         * @return mixed trash bin size or false if no trash bin size is stored
         */
        private static function getTrashbinSize($user) {
index e97510dead80b2c596d008db29d1bcae9af12b70..3bbbb26791bfb354d92d9c082571891507900ee0 100644 (file)
@@ -53,7 +53,7 @@ class Storage {
        /**
         * get current size of all versions from a given user
         *
-        * @param $user user who owns the versions
+        * @param string $user user who owns the versions
         * @return mixed versions size or false if no versions size is stored
         */
        private static function getVersionsSize($user) {
@@ -69,8 +69,8 @@ class Storage {
        /**
         * write to the database how much space is in use for versions
         *
-        * @param $user owner of the versions
-        * @param $size size of the versions
+        * @param string $user owner of the versions
+        * @param int $size size of the versions
         */
        private static function setVersionsSize($user, $size) {
                if ( self::getVersionsSize($user) === false) {
@@ -336,7 +336,7 @@ class Storage {
        /**
         * @brief deletes used space for files versions in db if user was deleted
         *
-        * @param type $uid id of deleted user
+        * @param string $uid id of deleted user
         * @return \OC_DB_StatementWrapper of db delete operation
         */
        public static function deleteUser($uid) {
@@ -346,8 +346,8 @@ class Storage {
 
        /**
         * @brief get the size of all stored versions from a given user
-        * @param $uid id from the user
-        * @return size of vesions
+        * @param string $uid id from the user
+        * @return int size of versions
         */
        private static function calculateSize($uid) {
                if (\OCP\Config::getSystemValue('files_versions', Storage::DEFAULTENABLED) == 'true') {
index e5bec602f6218857fa6934ecfc38110bf595ccc2..c868e17f7b19ce7da7d4a08edcb8dded4ca128d1 100644 (file)
@@ -40,9 +40,9 @@ class GROUP_LDAP extends BackendUtility implements \OCP\GroupInterface {
 
        /**
         * @brief is user in group?
-        * @param $uid uid of the user
-        * @param $gid gid of the group
-        * @returns true/false
+        * @param string $uid uid of the user
+        * @param string $gid gid of the group
+        * @return bool
         *
         * Checks whether the user is member of a group or not.
         */
@@ -127,8 +127,8 @@ class GROUP_LDAP extends BackendUtility implements \OCP\GroupInterface {
 
        /**
         * @brief Get all groups a user belongs to
-        * @param $uid Name of the user
-        * @returns array with group names
+        * @param string $uid Name of the user
+        * @return array with group names
         *
         * This function fetches all groups a user belongs to. It does not check
         * if the user exists at all.
@@ -205,7 +205,7 @@ class GROUP_LDAP extends BackendUtility implements \OCP\GroupInterface {
 
        /**
         * @brief get a list of all users in a group
-        * @returns array with user ids
+        * @return array with user ids
         */
        public function usersInGroup($gid, $search = '', $limit = -1, $offset = 0) {
                if(!$this->enabled) {
@@ -288,7 +288,7 @@ class GROUP_LDAP extends BackendUtility implements \OCP\GroupInterface {
         * @brief returns the number of users in a group, who match the search term
         * @param string the internal group name
         * @param string optional, a search string
-        * @returns int | bool
+        * @return int | bool
         */
        public function countUsersInGroup($gid, $search = '') {
                $cachekey = 'countUsersInGroup-'.$gid.'-'.$search;
@@ -364,7 +364,7 @@ class GROUP_LDAP extends BackendUtility implements \OCP\GroupInterface {
 
        /**
         * @brief get a list of all groups
-        * @returns array with group names
+        * @return array with group names
         *
         * Returns a list with all groups (used by getGroups)
         */
@@ -403,7 +403,7 @@ class GROUP_LDAP extends BackendUtility implements \OCP\GroupInterface {
 
        /**
         * @brief get a list of all groups using a paged search
-        * @returns array with group names
+        * @return array with group names
         *
         * Returns a list with all groups
         * Uses a paged search if available to override a
@@ -481,8 +481,8 @@ class GROUP_LDAP extends BackendUtility implements \OCP\GroupInterface {
 
        /**
        * @brief Check if backend implements actions
-       * @param $actions bitwise-or'ed actions
-       * @returns boolean
+       * @param int $actions bitwise-or'ed actions
+       * @return boolean
        *
        * Returns the supported actions as int to be
        * compared with OC_USER_BACKEND_CREATE_USER etc.
index ea94990ffe40d8e8a7cba821cc7da69ab30a86e6..555587b1bfaf6cb1a20aa30a4fe61426fef9c377 100644 (file)
@@ -31,7 +31,7 @@ class Group_Proxy extends lib\Proxy implements \OCP\GroupInterface {
 
        /**
         * @brief Constructor
-        * @param $serverConfigPrefixes array containing the config Prefixes
+        * @param array $serverConfigPrefixes array containing the config Prefixes
         */
        public function __construct($serverConfigPrefixes, ILDAPWrapper $ldap) {
                parent::__construct($ldap);
@@ -46,9 +46,9 @@ class Group_Proxy extends lib\Proxy implements \OCP\GroupInterface {
 
        /**
         * @brief Tries the backends one after the other until a positive result is returned from the specified method
-        * @param $gid string, the gid connected to the request
-        * @param $method string, the method of the group backend that shall be called
-        * @param $parameters an array of parameters to be passed
+        * @param string $gid the gid connected to the request
+        * @param string $method the method of the group backend that shall be called
+        * @param array $parameters an array of parameters to be passed
         * @return mixed, the result of the method or false
         */
        protected function walkBackends($gid, $method, $parameters) {
@@ -64,10 +64,10 @@ class Group_Proxy extends lib\Proxy implements \OCP\GroupInterface {
 
        /**
         * @brief Asks the backend connected to the server that supposely takes care of the gid from the request.
-        * @param $gid string, the gid connected to the request
-        * @param $method string, the method of the group backend that shall be called
-        * @param $parameters an array of parameters to be passed
-        * @param $passOnWhen the result matches this variable
+        * @param string $gid the gid connected to the request
+        * @param string $method the method of the group backend that shall be called
+        * @param array $parameters an array of parameters to be passed
+        * @param mixed $passOnWhen the result matches this variable
         * @return mixed, the result of the method or false
         */
        protected function callOnLastSeenOn($gid, $method, $parameters, $passOnWhen) {
@@ -96,9 +96,9 @@ class Group_Proxy extends lib\Proxy implements \OCP\GroupInterface {
 
        /**
         * @brief is user in group?
-        * @param $uid uid of the user
-        * @param $gid gid of the group
-        * @returns true/false
+        * @param string $uid uid of the user
+        * @param string $gid gid of the group
+        * @return bool
         *
         * Checks whether the user is member of a group or not.
         */
@@ -108,8 +108,8 @@ class Group_Proxy extends lib\Proxy implements \OCP\GroupInterface {
 
        /**
         * @brief Get all groups a user belongs to
-        * @param $uid Name of the user
-        * @returns array with group names
+        * @param string $uid Name of the user
+        * @return array with group names
         *
         * This function fetches all groups a user belongs to. It does not check
         * if the user exists at all.
@@ -129,7 +129,7 @@ class Group_Proxy extends lib\Proxy implements \OCP\GroupInterface {
 
        /**
         * @brief get a list of all users in a group
-        * @returns array with user ids
+        * @return array with user ids
         */
        public function usersInGroup($gid, $search = '', $limit = -1, $offset = 0) {
                $users = array();
@@ -148,7 +148,7 @@ class Group_Proxy extends lib\Proxy implements \OCP\GroupInterface {
         * @brief returns the number of users in a group, who match the search term
         * @param string the internal group name
         * @param string optional, a search string
-        * @returns int | bool
+        * @return int | bool
         */
        public function countUsersInGroup($gid, $search = '') {
                return $this->handleRequest(
@@ -157,7 +157,7 @@ class Group_Proxy extends lib\Proxy implements \OCP\GroupInterface {
 
        /**
         * @brief get a list of all groups
-        * @returns array with group names
+        * @return array with group names
         *
         * Returns a list with all groups
         */
@@ -185,8 +185,8 @@ class Group_Proxy extends lib\Proxy implements \OCP\GroupInterface {
 
        /**
         * @brief Check if backend implements actions
-        * @param $actions bitwise-or'ed actions
-        * @returns boolean
+        * @param int $actions bitwise-or'ed actions
+        * @return boolean
         *
         * Returns the supported actions as int to be
         * compared with OC_USER_BACKEND_CREATE_USER etc.
index c721703bce5417e0a4c50593a2160ef6099d6b01..0097c94ba2b397cb0d8bd035ca520dff7d00793e 100644 (file)
@@ -383,7 +383,7 @@ class Access extends LDAPUtility {
        }
 
        /**
-        * @param $dn
+        * @param string $dn
         * @return bool|string
         */
        private function findMappedUser($dn) {
@@ -403,7 +403,7 @@ class Access extends LDAPUtility {
        }
 
        /**
-        * @param $dn
+        * @param string $dn
         * @return bool|string
         */
        private function findMappedGroup($dn) {
@@ -630,8 +630,8 @@ class Access extends LDAPUtility {
        }
 
        /**
-        * @param $filter
-        * @param $attr
+        * @param string $filter
+        * @param string $attr
         * @param int $limit
         * @param int $offset
         * @return array
@@ -641,8 +641,8 @@ class Access extends LDAPUtility {
        }
 
        /**
-        * @param $filter
-        * @param $attr
+        * @param string $filter
+        * @param string $attr
         * @param int $limit
         * @param int $offset
         * @return array
@@ -652,7 +652,7 @@ class Access extends LDAPUtility {
        }
 
        /**
-        * @param $list
+        * @param array $list
         * @param bool $manyAttributes
         * @return array
         */
@@ -840,9 +840,9 @@ class Access extends LDAPUtility {
        }
 
        /**
-        * @param $searchResults
-        * @param $limit
-        * @param $hasHitLimit
+        * @param array $searchResults
+        * @param int $limit
+        * @param bool $hasHitLimit
         * @return int
         */
        private function countEntriesInSearchResults($searchResults, $limit, &$hasHitLimit) {
@@ -969,7 +969,7 @@ class Access extends LDAPUtility {
        }
 
        /**
-        * @param $name
+        * @param string $name
         * @return bool|mixed|string
         */
        public function sanitizeUsername($name) {
@@ -1003,7 +1003,7 @@ class Access extends LDAPUtility {
 
        /**
         * @brief combines the input filters with AND
-        * @param $filters array, the filters to connect
+        * @param array $filters the filters to connect
         * @return string the combined filter
         */
        public function combineFilterWithAnd($filters) {
@@ -1012,7 +1012,7 @@ class Access extends LDAPUtility {
 
        /**
         * @brief combines the input filters with AND
-        * @param $filters array, the filters to connect
+        * @param array $filters the filters to connect
         * @return string the combined filter
         */
        public function combineFilterWithOr($filters) {
@@ -1021,7 +1021,7 @@ class Access extends LDAPUtility {
 
        /**
         * @brief combines the input filters with given operator
-        * @param $filters array, the filters to connect
+        * @param array $filters the filters to connect
         * @param string $operator either & or |
         * @return string the combined filter
         */
@@ -1062,7 +1062,7 @@ class Access extends LDAPUtility {
        /**
         * @brief creates a filter part for searches
         * @param string $search the search term
-        * @param $searchAttributes
+        * @param array|null $searchAttributes
         * @param string $fallbackAttribute a fallback attribute in case the user
         * did not define search attributes. Typically the display name attribute.
         * @return string the final filter part to use in LDAP searches
@@ -1152,7 +1152,7 @@ class Access extends LDAPUtility {
        }
 
        /**
-        * @param $dn
+        * @param string $dn
         * @param bool $isUser
         * @return array|bool|false
         */
@@ -1220,7 +1220,7 @@ class Access extends LDAPUtility {
        /**
         * @brief checks if the given DN is part of the given base DN(s)
         * @param string $dn the DN
-        * @param $bases array containing the allowed base DN or DNs
+        * @param array $bases array containing the allowed base DN or DNs
         * @return bool
         */
        private function isDNPartOfBase($dn, $bases) {
index a9d65963ad5a336688986c5bef16db09aedc2ce8..c6c3c6b89bf19c0505ce19240a1cd5ecf5c8e99d 100644 (file)
@@ -81,7 +81,7 @@ class Configuration {
        );
 
        /**
-        * @param $configPrefix
+        * @param string $configPrefix
         * @param bool $autoRead
         */
        public function __construct($configPrefix, $autoRead = true) {
@@ -92,7 +92,7 @@ class Configuration {
        }
 
        /**
-        * @param $name
+        * @param string $name
         * @return mixed|void
         */
        public function __get($name) {
@@ -102,8 +102,8 @@ class Configuration {
        }
 
        /**
-        * @param $name
-        * @param $value
+        * @param string $name
+        * @param mixed $value
         */
        public function __set($name, $value) {
                $this->setConfiguration(array($name => $value));
@@ -120,7 +120,7 @@ class Configuration {
         * @brief set LDAP configuration with values delivered by an array, not read
         * from configuration. It does not save the configuration! To do so, you
         * must call saveConfiguration afterwards.
-        * @param $config array that holds the config parameters in an associated
+        * @param array $config array that holds the config parameters in an associated
         * array
         * @param array &$applied optional; array where the set fields will be given to
         * @return false|null
@@ -250,7 +250,7 @@ class Configuration {
        }
 
        /**
-        * @param $varName
+        * @param string $varName
         * @return array|string
         */
        protected function getMultiLine($varName) {
@@ -265,8 +265,8 @@ class Configuration {
        }
 
        /**
-        * @param $varName
-        * @param $value
+        * @param string $varName
+        * @param array|string $value
         */
        protected function setMultiLine($varName, $value) {
                if(empty($value)) {
@@ -282,7 +282,7 @@ class Configuration {
        }
 
        /**
-        * @param $varName
+        * @param string $varName
         * @return string
         */
        protected function getPwd($varName) {
@@ -290,7 +290,7 @@ class Configuration {
        }
 
        /**
-        * @param $varName
+        * @param string $varName
         * @return string
         */
        protected function getLcValue($varName) {
@@ -298,7 +298,7 @@ class Configuration {
        }
 
        /**
-        * @param $varName
+        * @param string $varName
         * @return string
         */
        protected function getSystemValue($varName) {
@@ -307,7 +307,7 @@ class Configuration {
        }
 
        /**
-        * @param $varName
+        * @param string $varName
         * @return string
         */
        protected function getValue($varName) {
@@ -321,16 +321,16 @@ class Configuration {
        }
 
        /**
-        * @param $varName
-        * @param $value
+        * @param string $varName
+        * @param mixed $value
         */
        protected function setValue($varName, $value) {
                $this->config[$varName] = $value;
        }
 
        /**
-        * @param $varName
-        * @param $value
+        * @param string $varName
+        * @param string $value
         * @return bool
         */
        protected function saveValue($varName, $value) {
@@ -448,4 +448,4 @@ class Configuration {
                return $array;
        }
 
-}
\ No newline at end of file
+}
index 847a984bb4d2665df230bbb4960dcd044cd0a682..7e2f8d1d957341f2afd97f1022733db3de46e8f6 100644 (file)
@@ -84,7 +84,7 @@ class Connection extends LDAPUtility {
        }
 
        /**
-        * @param $name
+        * @param string $name
         * @return bool|mixed|void
         */
        public function __get($name) {
@@ -100,8 +100,8 @@ class Connection extends LDAPUtility {
        }
 
        /**
-        * @param $name
-        * @param $value
+        * @param string $name
+        * @param mixed $value
         */
        public function __set($name, $value) {
                $this->doNotValidate = false;
@@ -142,7 +142,7 @@ class Connection extends LDAPUtility {
        }
 
        /**
-        * @param $key
+        * @param string|null $key
         * @return string
         */
        private function getCacheKey($key) {
@@ -154,7 +154,7 @@ class Connection extends LDAPUtility {
        }
 
        /**
-        * @param $key
+        * @param string $key
         * @return mixed|null
         */
        public function getFromCache($key) {
@@ -174,7 +174,7 @@ class Connection extends LDAPUtility {
        }
 
        /**
-        * @param $key
+        * @param string $key
         * @return bool
         */
        public function isCached($key) {
@@ -189,8 +189,8 @@ class Connection extends LDAPUtility {
        }
 
        /**
-        * @param $key
-        * @param $value
+        * @param string $key
+        * @param mixed $value
         */
        public function writeToCache($key, $value) {
                if(!$this->configured) {
@@ -224,7 +224,7 @@ class Connection extends LDAPUtility {
 
        /**
         * @brief set LDAP configuration with values delivered by an array, not read from configuration
-        * @param $config array that holds the config parameters in an associated array
+        * @param array $config array that holds the config parameters in an associated array
         * @param array &$setParameters optional; array where the set fields will be given to
         * @return boolean true if config validates, false otherwise. Check with $setParameters for detailed success on single parameters
         */
@@ -523,8 +523,8 @@ class Connection extends LDAPUtility {
        }
 
        /**
-        * @param $host
-        * @param $port
+        * @param string $host
+        * @param string $port
         * @return false|void
         */
        private function doConnect($host, $port) {
index 146ada125bb2a60a1c6fb1b2365af81adc17ad62..e26fdfd2c36274ea89c8d7e5a2c72a914e64a8a9 100644 (file)
@@ -52,15 +52,15 @@ interface ILDAPWrapper {
         * @param int $pageSize number of results per page
         * @param bool $isCritical Indicates whether the pagination is critical of not.
         * @param array $cookie structure sent by LDAP server
-        * @return true on success, false otherwise
+        * @return bool true on success, false otherwise
         */
        public function controlPagedResult($link, $pageSize, $isCritical, $cookie);
 
        /**
         * @brief Retrieve the LDAP pagination cookie
-        * @param $link LDAP link resource
-        * @param $result LDAP result resource
-        * @param array $cookie structure sent by LDAP server
+        * @param resource $link LDAP link resource
+        * @param resource $result LDAP result resource
+        * @param string $cookie structure sent by LDAP server
         * @return bool true on success, false otherwise
         *
         * Corresponds to ldap_control_paged_result_response
@@ -169,8 +169,8 @@ interface ILDAPWrapper {
 
        /**
         * @brief Sort the result of a LDAP search
-        * @param $link LDAP link resource
-        * @param $result LDAP result resource
+        * @param resource $link LDAP link resource
+        * @param resource $result LDAP result resource
         * @param string $sortFilter attribute to use a key in sort
         */
        public function sort($link, $result, $sortFilter);
index 49c777197215139e7b3d18ea1f880f8a75611570..dd9d1eef7fbbdf8d8b49287187f9c623cf7fd860 100644 (file)
@@ -34,7 +34,7 @@ class Jobs extends \OC\BackgroundJob\TimedJob {
        }
 
        /**
-        * @param $argument
+        * @param mixed $argument
         */
        public function run($argument){
                Jobs::updateGroups();
@@ -69,7 +69,7 @@ class Jobs extends \OC\BackgroundJob\TimedJob {
        }
 
        /**
-        * @param $groups
+        * @param array $groups
         */
        static private function handleKnownGroups($groups) {
                \OCP\Util::writeLog('user_ldap', 'bgJ "updateGroups" â€“ Dealing with known Groups.', \OCP\Util::DEBUG);
@@ -107,7 +107,7 @@ class Jobs extends \OC\BackgroundJob\TimedJob {
        }
 
        /**
-        * @param $createdGroups
+        * @param array $createdGroups
         */
        static private function handleCreatedGroups($createdGroups) {
                \OCP\Util::writeLog('user_ldap', 'bgJ "updateGroups" â€“ dealing with created Groups.', \OCP\Util::DEBUG);
@@ -129,7 +129,7 @@ class Jobs extends \OC\BackgroundJob\TimedJob {
        }
 
        /**
-        * @param $removedGroups
+        * @param array $removedGroups
         */
        static private function handleRemovedGroups($removedGroups) {
                \OCP\Util::writeLog('user_ldap', 'bgJ "updateGroups" â€“ dealing with removed groups.', \OCP\Util::DEBUG);
index 9962d1267b8ad1d1549fa45aa0de666281efe262..212eea0d8e97c472b410bcc4bf4be7807375f664 100644 (file)
@@ -49,7 +49,7 @@ class LDAP implements ILDAPWrapper {
        /**
         * @param LDAP $link
         * @param LDAP $result
-        * @param $cookie
+        * @param string $cookie
         * @return bool|LDAP
         */
        public function controlPagedResultResponse($link, $result, &$cookie) {
@@ -65,7 +65,7 @@ class LDAP implements ILDAPWrapper {
         * @param LDAP $link
         * @param int $pageSize
         * @param bool $isCritical
-        * @param array $cookie
+        * @param string $cookie
         * @return mixed|true
         */
        public function controlPagedResult($link, $pageSize, $isCritical, $cookie) {
@@ -248,7 +248,7 @@ class LDAP implements ILDAPWrapper {
 
        /**
         * @param string $functionName
-        * @param $args
+        * @param array $args
         */
        private function preFunctionCall($functionName, $args) {
                $this->curFunc = $functionName;
index 50e8071a584ef23f17bfbf9b0186ab4ecbced6d9..f7ba9082e0999ea22e3b587246e06debc6764a9d 100644 (file)
@@ -38,7 +38,7 @@ abstract class Proxy {
        }
 
        /**
-        * @param $configPrefix
+        * @param string $configPrefix
         */
        private function addAccess($configPrefix) {
                $connector = new Connection($this->ldap, $configPrefix);
@@ -46,7 +46,7 @@ abstract class Proxy {
        }
 
        /**
-        * @param $configPrefix
+        * @param string $configPrefix
         * @return mixed
         */
        protected function getAccess($configPrefix) {
@@ -57,7 +57,7 @@ abstract class Proxy {
        }
 
        /**
-        * @param $uid
+        * @param string $uid
         * @return string
         */
        protected function getUserCacheKey($uid) {
@@ -65,7 +65,7 @@ abstract class Proxy {
        }
 
        /**
-        * @param $gid
+        * @param string $gid
         * @return string
         */
        protected function getGroupCacheKey($gid) {
@@ -73,25 +73,25 @@ abstract class Proxy {
        }
 
        /**
-        * @param $id
-        * @param $method
-        * @param $parameters
+        * @param string $id
+        * @param string $method
+        * @param array $parameters
         * @param bool $passOnWhen
         * @return mixed
         */
        abstract protected function callOnLastSeenOn($id, $method, $parameters, $passOnWhen);
 
        /**
-        * @param $id
-        * @param $method
-        * @param $parameters
+        * @param string $id
+        * @param string $method
+        * @param array $parameters
         * @return mixed
         */
        abstract protected function walkBackends($id, $method, $parameters);
 
        /**
         * @brief Takes care of the request to the User backend
-        * @param $id
+        * @param string $id
         * @param string $method string, the method of the user backend that shall be called
         * @param array $parameters an array of parameters to be passed
         * @param bool $passOnWhen
@@ -106,7 +106,7 @@ abstract class Proxy {
        }
 
        /**
-        * @param $key
+        * @param string|null $key
         * @return string
         */
        private function getCacheKey($key) {
@@ -118,7 +118,7 @@ abstract class Proxy {
        }
 
        /**
-        * @param $key
+        * @param string $key
         * @return mixed|null
         */
        public function getFromCache($key) {
@@ -131,7 +131,7 @@ abstract class Proxy {
        }
 
        /**
-        * @param $key
+        * @param string $key
         * @return bool
         */
        public function isCached($key) {
@@ -140,8 +140,8 @@ abstract class Proxy {
        }
 
        /**
-        * @param $key
-        * @param $value
+        * @param string $key
+        * @param mixed $value
         */
        public function writeToCache($key, $value) {
                $key   = $this->getCacheKey($key);
index 8d944c9a9573f51a6d5838e19d716ac7a760a46b..62dd138d7e3faf56a274ee0eb3faf62c6126d466 100644 (file)
@@ -909,7 +909,7 @@ class Wizard extends LDAPUtility {
         * specified attribute
         * @param string[] $filters array, the filters that shall be used in the search
         * @param string $attr the attribute of which a list of values shall be returned
-        * @param $lfw bool, whether the last filter is a wildcard which shall not
+        * @param bool $lfw whether the last filter is a wildcard which shall not
         * be processed if there were already findings, defaults to true
         * @param int $dnReadLimit the amount of how many DNs should be analyzed.
         * The lower, the faster
@@ -987,7 +987,7 @@ class Wizard extends LDAPUtility {
         * @param string $dbkey the dbkey of the setting the feature is connected to
         * @param string $confkey the confkey counterpart for the $dbkey as used in the
         * Configuration class
-        * @param $po boolean, whether the objectClass with most result entries
+        * @param bool $po whether the objectClass with most result entries
         * shall be pre-selected via the result
         * @return array, list of found items.
         * @throws \Exception
@@ -1041,9 +1041,9 @@ class Wizard extends LDAPUtility {
 
        /**
         * @brief appends a list of values fr
-        * @param $result resource, the return value from ldap_get_attributes
+        * @param resource $result the return value from ldap_get_attributes
         * @param string $attribute the attribute values to look for
-        * @param &$known array, new values will be appended here
+        * @param array &$known new values will be appended here
         * @return int, state on of the class constants LRESULT_PROCESSED_OK,
         * LRESULT_PROCESSED_INVALID or LRESULT_PROCESSED_SKIP
         */
index 5ed16732ca00103a676b67a9254d3c3f7b48aa61..42b0bc046549a7dfb7380af8decd5003632c291e 100644 (file)
@@ -29,8 +29,8 @@ class WizardResult {
        protected $markedChange = false;
 
        /**
-        * @param $key
-        * @param $value
+        * @param string $key
+        * @param mixed $value
         */
        public function addChange($key, $value) {
                $this->changes[$key] = $value;
@@ -72,4 +72,4 @@ class WizardResult {
                }
                return $result;
        }
-}
\ No newline at end of file
+}
index cb4c3b316dc7566c8415d9aaa7d52e848082e29a..ca8aeb12f8743965be1f14b0921162a339d005a0 100644 (file)
@@ -79,7 +79,7 @@ class Test_User_Ldap_Direct extends \PHPUnit_Framework_TestCase {
 
        /**
         * @brief Prepares the Access mock for checkPassword tests
-        * @param $access mock of \OCA\user_ldap\lib\Access
+        * @param \OCA\user_ldap\lib\Access $access mock
         * @return void
         */
        private function prepareAccessForCheckPassword(&$access) {
@@ -187,7 +187,7 @@ class Test_User_Ldap_Direct extends \PHPUnit_Framework_TestCase {
 
        /**
         * @brief Prepares the Access mock for getUsers tests
-        * @param $access mock of \OCA\user_ldap\lib\Access
+        * @param \OCA\user_ldap\lib\Access $access mock
         * @return void
         */
        private function prepareAccessForGetUsers(&$access) {
@@ -557,4 +557,4 @@ class Test_User_Ldap_Direct extends \PHPUnit_Framework_TestCase {
                $result = $backend->countUsers();
                $this->assertFalse($result);
        }
-}
\ No newline at end of file
+}
index 757de6b60f4cedbd57162294f6ace9961c4b8d8c..2d44072410df5677444285c29b41203be7dd7e19 100644 (file)
@@ -70,8 +70,8 @@ class USER_LDAP extends BackendUtility implements \OCP\UserInterface {
 
        /**
         * @brief reads jpegPhoto and set is as avatar if available
-        * @param $uid string ownCloud user name
-        * @param $dn string the user's LDAP DN
+        * @param string $uid ownCloud user name
+        * @param string $dn the user's LDAP DN
         * @return void
         */
        private function updateAvatar($uid, $dn) {
@@ -118,7 +118,7 @@ class USER_LDAP extends BackendUtility implements \OCP\UserInterface {
 
        /**
         * @brief checks whether the user is allowed to change his avatar in ownCloud
-        * @param $uid string the ownCloud user name
+        * @param string $uid the ownCloud user name
         * @return boolean either the user can or cannot
         */
        public function canChangeAvatar($uid) {
@@ -136,8 +136,8 @@ class USER_LDAP extends BackendUtility implements \OCP\UserInterface {
 
        /**
         * @brief reads the image from LDAP that shall be used as Avatar
-        * @param $uid string, the ownCloud user name
-        * @param $dn string, the user DN
+        * @param string $uid the ownCloud user name
+        * @param string $dn the user DN
         * @return string data (provided by LDAP) | false
         */
        private function getAvatarImage($uid, $dn) {
@@ -200,7 +200,7 @@ class USER_LDAP extends BackendUtility implements \OCP\UserInterface {
 
        /**
         * @brief Get a list of all users
-        * @returns array with all uids
+        * @return array with all uids
         *
         * Get a list of all users.
         */
@@ -272,8 +272,8 @@ class USER_LDAP extends BackendUtility implements \OCP\UserInterface {
 
        /**
        * @brief delete a user
-       * @param $uid The username of the user to delete
-       * @returns true/false
+       * @param string $uid The username of the user to delete
+       * @return bool
        *
        * Deletes a user
        */
@@ -326,8 +326,8 @@ class USER_LDAP extends BackendUtility implements \OCP\UserInterface {
 
        /**
         * @brief get display name of the user
-        * @param $uid user ID of the user
-        * @return display name
+        * @param string $uid user ID of the user
+        * @return string display name
         */
        public function getDisplayName($uid) {
                if(!$this->userExists($uid)) {
@@ -353,7 +353,7 @@ class USER_LDAP extends BackendUtility implements \OCP\UserInterface {
 
        /**
         * @brief Get a list of all display names
-        * @returns array with  all displayNames (value) and the correspondig uids (key)
+        * @return array with  all displayNames (value) and the correspondig uids (key)
         *
         * Get a list of all display names and user ids.
         */
@@ -374,8 +374,8 @@ class USER_LDAP extends BackendUtility implements \OCP\UserInterface {
 
                /**
        * @brief Check if backend implements actions
-       * @param $actions bitwise-or'ed actions
-       * @returns boolean
+       * @param int $actions bitwise-or'ed actions
+       * @return boolean
        *
        * Returns the supported actions as int to be
        * compared with OC_USER_BACKEND_CREATE_USER etc.
index 2cb3dfb2c60f60b5154406adb02f593cca32d1d1..412d6a05adf571b91c093512e66f4f5298cde086 100644 (file)
@@ -31,7 +31,7 @@ class User_Proxy extends lib\Proxy implements \OCP\UserInterface {
 
        /**
         * @brief Constructor
-        * @param $serverConfigPrefixes array containing the config Prefixes
+        * @param array $serverConfigPrefixes array containing the config Prefixes
         */
        public function __construct($serverConfigPrefixes, ILDAPWrapper $ldap) {
                parent::__construct($ldap);
@@ -46,10 +46,10 @@ class User_Proxy extends lib\Proxy implements \OCP\UserInterface {
 
        /**
         * @brief Tries the backends one after the other until a positive result is returned from the specified method
-        * @param $uid string, the uid connected to the request
-        * @param $method string, the method of the user backend that shall be called
-        * @param $parameters an array of parameters to be passed
-        * @return mixed, the result of the method or false
+        * @param string $uid the uid connected to the request
+        * @param string $method the method of the user backend that shall be called
+        * @param array $parameters an array of parameters to be passed
+        * @return mixed the result of the method or false
         */
        protected  function walkBackends($uid, $method, $parameters) {
                $cacheKey = $this->getUserCacheKey($uid);
@@ -69,11 +69,11 @@ class User_Proxy extends lib\Proxy implements \OCP\UserInterface {
 
        /**
         * @brief Asks the backend connected to the server that supposely takes care of the uid from the request.
-        * @param $uid string, the uid connected to the request
-        * @param $method string, the method of the user backend that shall be called
-        * @param $parameters an array of parameters to be passed
-        * @param $passOnWhen the result matches this variable
-        * @return mixed, the result of the method or false
+        * @param string $uid the uid connected to the request
+        * @param string $method the method of the user backend that shall be called
+        * @param array $parameters an array of parameters to be passed
+        * @param mixed $passOnWhen the result matches this variable
+        * @return mixed the result of the method or false
         */
        protected  function callOnLastSeenOn($uid, $method, $parameters, $passOnWhen) {
                $cacheKey = $this->getUserCacheKey($uid);
@@ -106,8 +106,8 @@ class User_Proxy extends lib\Proxy implements \OCP\UserInterface {
 
        /**
         * @brief Check if backend implements actions
-        * @param $actions bitwise-or'ed actions
-        * @returns boolean
+        * @param int $actions bitwise-or'ed actions
+        * @return boolean
         *
         * Returns the supported actions as int to be
         * compared with OC_USER_BACKEND_CREATE_USER etc.
@@ -119,7 +119,7 @@ class User_Proxy extends lib\Proxy implements \OCP\UserInterface {
 
        /**
         * @brief Get a list of all users
-        * @returns array with all uids
+        * @return array with all uids
         *
         * Get a list of all users.
         */
@@ -146,9 +146,9 @@ class User_Proxy extends lib\Proxy implements \OCP\UserInterface {
 
        /**
         * @brief Check if the password is correct
-        * @param $uid The username
-        * @param $password The password
-        * @returns true/false
+        * @param string $uid The username
+        * @param string $password The password
+        * @return bool
         *
         * Check if the password is correct without logging in the user
         */
@@ -167,8 +167,8 @@ class User_Proxy extends lib\Proxy implements \OCP\UserInterface {
 
        /**
         * @brief get display name of the user
-        * @param $uid user ID of the user
-        * @return display name
+        * @param string $uid user ID of the user
+        * @return string display name
         */
        public function getDisplayName($uid) {
                return $this->handleRequest($uid, 'getDisplayName', array($uid));
@@ -176,7 +176,7 @@ class User_Proxy extends lib\Proxy implements \OCP\UserInterface {
 
        /**
         * @brief checks whether the user is allowed to change his avatar in ownCloud
-        * @param $uid string the ownCloud user name
+        * @param string $uid the ownCloud user name
         * @return boolean either the user can or cannot
         */
        public function canChangeAvatar($uid) {
@@ -185,7 +185,7 @@ class User_Proxy extends lib\Proxy implements \OCP\UserInterface {
 
        /**
         * @brief Get a list of all display names
-        * @returns array with  all displayNames (value) and the corresponding uids (key)
+        * @return array with  all displayNames (value) and the corresponding uids (key)
         *
         * Get a list of all display names and user ids.
         */
@@ -203,8 +203,8 @@ class User_Proxy extends lib\Proxy implements \OCP\UserInterface {
 
        /**
         * @brief delete a user
-        * @param $uid The username of the user to delete
-        * @returns true/false
+        * @param string $uid The username of the user to delete
+        * @return bool
         *
         * Deletes a user
         */
@@ -221,7 +221,7 @@ class User_Proxy extends lib\Proxy implements \OCP\UserInterface {
 
        /**
         * @brief Count the number of users
-        * @returns int | bool
+        * @return int|bool
         */
        public function countUsers() {
                $users = false;