]> source.dussan.org Git - nextcloud-server.git/commitdiff
Scrutinizer Auto-Fixes
authorScrutinizer Auto-Fixer <auto-fixer@scrutinizer-ci.com>
Wed, 19 Feb 2014 08:31:54 +0000 (09:31 +0100)
committerScrutinizer <auto-fixer@scrutinizer-ci.com>
Wed, 19 Feb 2014 08:31:54 +0000 (09:31 +0100)
This patch was automatically generated as part of the following inspection:
https://scrutinizer-ci.com/g/owncloud/core/inspections/cdfecc4e-a37e-4233-8025-f0d7252a8720

Enabled analysis tools:
 - PHP Analyzer
 - JSHint
 - PHP Copy/Paste Detector
 - PHP PDepend

101 files changed:
apps/files_encryption/lib/helper.php
apps/files_external/lib/google.php
apps/files_external/lib/streamwrapper.php
apps/files_external/lib/webdav.php
apps/files_sharing/lib/cache.php
apps/files_sharing/lib/sharedstorage.php
apps/files_versions/lib/versions.php
apps/user_ldap/lib/access.php
apps/user_ldap/lib/connection.php
apps/user_ldap/lib/proxy.php
apps/user_ldap/lib/wizard.php
apps/user_ldap/user_ldap.php
core/command/db/generatechangescript.php
core/command/status.php
core/command/upgrade.php
core/command/user/report.php
lib/private/app.php
lib/private/appconfig.php
lib/private/appframework/app.php
lib/private/appframework/dependencyinjection/dicontainer.php
lib/private/appframework/http/redirectresponse.php
lib/private/appframework/middleware/middlewaredispatcher.php
lib/private/appframework/middleware/security/securitymiddleware.php
lib/private/archive.php
lib/private/archive/tar.php
lib/private/archive/zip.php
lib/private/arrayparser.php
lib/private/backgroundjob.php
lib/private/backgroundjob/joblist.php
lib/private/cache/usercache.php
lib/private/config.php
lib/private/connector/sabre/aborteduploaddetectionplugin.php
lib/private/connector/sabre/quotaplugin.php
lib/private/db.php
lib/private/db/mdb2schemawriter.php
lib/private/filechunking.php
lib/private/fileproxy.php
lib/private/files.php
lib/private/files/cache/updater.php
lib/private/files/fileinfo.php
lib/private/files/filesystem.php
lib/private/files/node/folder.php
lib/private/files/node/node.php
lib/private/files/node/root.php
lib/private/files/storage/loader.php
lib/private/files/storage/mappedlocal.php
lib/private/files/utils/scanner.php
lib/private/geo.php
lib/private/group.php
lib/private/group/dummy.php
lib/private/image.php
lib/private/installer.php
lib/private/l10n.php
lib/private/legacy/appconfig.php
lib/private/legacy/preferences.php
lib/private/migrate.php
lib/private/migration/content.php
lib/private/migration/provider.php
lib/private/ocs.php
lib/private/preferences.php
lib/private/request.php
lib/private/share/searchresultsorter.php
lib/private/template/base.php
lib/private/template/templatefilelocator.php
lib/private/user.php
lib/private/user/database.php
lib/private/util.php
lib/public/appframework/controller.php
lib/public/backgroundjob.php
lib/public/backgroundjob/ijob.php
lib/public/backgroundjob/ijoblist.php
lib/public/contacts/imanager.php
lib/public/files.php
lib/public/iappconfig.php
lib/public/iconfig.php
lib/public/itags.php
lib/public/share.php
tests/lib/api.php
tests/lib/appframework/http/DispatcherTest.php
tests/lib/appframework/http/ResponseTest.php
tests/lib/appframework/middleware/MiddlewareDispatcherTest.php
tests/lib/appframework/middleware/security/SecurityMiddlewareTest.php
tests/lib/appframework/routing/RoutingTest.php
tests/lib/dbschema.php
tests/lib/errorHandler.php
tests/lib/files/cache/homecache.php
tests/lib/files/etagtest.php
tests/lib/files/node/integration.php
tests/lib/files/node/root.php
tests/lib/files/storage/home.php
tests/lib/files/storage/wrapper/quota.php
tests/lib/files/stream/quota.php
tests/lib/group/backend.php
tests/lib/group/database.php
tests/lib/group/group.php
tests/lib/migrate.php
tests/lib/ocs/privatedata.php
tests/lib/share/share.php
tests/lib/template/resourcelocator.php
tests/lib/user.php
tests/testcleanuplistener.php

index 5ec09629d625530a4011432859f8ca9214668beb..ef3775875f0f5d911e38901d2f803f903c11c31b 100755 (executable)
@@ -425,7 +425,7 @@ class Helper {
        /**
         * @brief glob uses different pattern than regular expressions, escape glob pattern only
         * @param string $path unescaped path
-        * @return escaped path
+        * @return string path
         */
        public static function escapeGlobPattern($path) {
                return preg_replace('/(\*|\?|\[)/', '[$1]', $path);
index 064c1e20a93edc29d8873b690e0fb6848a4987ba..35457f6852841dec9e55621c245d0526e4b6f49d 100644 (file)
@@ -67,7 +67,7 @@ class Google extends \OC\Files\Storage\Common {
        /**
         * Get the Google_DriveFile object for the specified path
         * @param string $path
-        * @return Google_DriveFile|false
+        * @return string
         */
        private function getDriveFile($path) {
                // Remove leading and trailing slashes
index 3e3dc3e3af577091a6cc8326e5a1ee18fd80c0a4..44bd9a0161a4cf14ae8d9179a1587dbe73f40f7d 100644 (file)
@@ -11,6 +11,7 @@ namespace OC\Files\Storage;
 abstract class StreamWrapper extends Common {
 
        /**
+        * @param string $path
         * @return string|null
         */
        abstract public function constructUrl($path);
index c64b2d8b73a09750d9f8292855a1f4aef2b0ddc1..7611316a26a5e2f36d64a30ded3fd10d57eac220 100644 (file)
@@ -336,6 +336,9 @@ class DAV extends \OC\Files\Storage\Common{
                }
        }
 
+       /**
+        * @param string $path
+        */
        public function cleanPath($path) {
                $path = \OC\Files\Filesystem::normalizePath($path);
                // remove leading slash
index 7d63e7a4fe388f085219b920fd8d02e8dd704f9a..aadc54e4a7f2c31c964d2da8ba371c85c0714067 100644 (file)
@@ -396,7 +396,7 @@ class Shared_Cache extends Cache {
         * use the one with the highest id gives the best result with the background scanner, since that is most
         * likely the folder where we stopped scanning previously
         *
-        * @return string|bool the path of the folder or false when no folder matched
+        * @return boolean the path of the folder or false when no folder matched
         */
        public function getIncomplete() {
                return false;
index ebd16f081ba65b905d239cf4b400ad7365d34c98..b922654e5ec2db2e2e8f711eed54d68e843e622d 100644 (file)
@@ -41,6 +41,7 @@ class Shared extends \OC\Files\Storage\Common {
        /**
         * @brief Get the source file path, permissions, and owner for a shared file
         * @param string Shared target file path
+        * @param string $target
         * @return Returns array with the keys path, permissions, and owner or false if not found
         */
        public function getFile($target) {
index fc0bf3bfad0ce8d058404858bdf12fe18d472c5b..2bd9c15bae424988fbbed9b6a16928f1f80ab5cf 100644 (file)
@@ -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
-        * @return result of db delete operation
+        * @return \OC_DB_StatementWrapper of db delete operation
         */
        public static function deleteUser($uid) {
                $query = \OC_DB::prepare('DELETE FROM `*PREFIX*files_versions` WHERE `user`=?');
@@ -420,8 +420,8 @@ class Storage {
 
        /**
         * @brief get list of files we want to expire
-        * @param integer $currentTime timestamp of current time
         * @param array $versions list of versions
+        * @param integer $time
         * @return array containing the list of to deleted versions and the size of them
         */
        protected static function getExpireList($time, $versions) {
index 8a8d8aa5e3a1e3dd8c78fe10eb223c77f532c914..6795aecafeea48cc6a802a7d38e9c40af3331f0b 100644 (file)
@@ -164,6 +164,7 @@ class Access extends LDAPUtility {
 
        /**
         * gives back the database table for the query
+        * @param boolean $isUser
         */
        private function getMapTable($isUser) {
                if($isUser) {
@@ -644,6 +645,8 @@ class Access extends LDAPUtility {
         * @brief executes an LDAP search, optimized for Users
         * @param $filter the LDAP filter for the search
         * @param $attr optional, when a certain attribute shall be filtered out
+        * @param integer $limit
+        * @param integer $offset
         * @returns array with the search result
         *
         * Executes an LDAP search
@@ -661,8 +664,10 @@ class Access extends LDAPUtility {
 
        /**
         * @brief executes an LDAP search, optimized for Groups
-        * @param $filter the LDAP filter for the search
+        * @param string $filter the LDAP filter for the search
         * @param $attr optional, when a certain attribute shall be filtered out
+        * @param integer $limit
+        * @param integer $offset
         * @returns array with the search result
         *
         * Executes an LDAP search
@@ -757,7 +762,7 @@ class Access extends LDAPUtility {
 
        /**
         * @brief executes an LDAP search, but counts the results only
-        * @param $filter the LDAP filter for the search
+        * @param string $filter the LDAP filter for the search
         * @param $base an array containing the LDAP subtree(s) that shall be searched
         * @param $attr optional, array, one or more attributes that shall be
         * retrieved. Results will according to the order in the array.
@@ -1006,6 +1011,9 @@ class Access extends LDAPUtility {
                return $this->combineFilterWithOr($filter);
        }
 
+       /**
+        * @param string $password
+        */
        public function areCredentialsValid($name, $password) {
                $name = $this->DNasBaseParameter($name);
                $testConnection = clone $this->connection;
index 19870550163032091a4d1a495a953c4874e3bc64..b2075748a3bab72226e61c82b8bb5b4563e516c8 100644 (file)
@@ -159,6 +159,9 @@ class Connection extends LDAPUtility {
                return unserialize(base64_decode($this->cache->get($key)));
        }
 
+       /**
+        * @param string $key
+        */
        public function isCached($key) {
                if(!$this->configured) {
                        $this->readConfiguration();
index b68910ff97ff9b1cb717076d4efd8c31db286e19..b27233bcd192d455df1c895a509a1e2c4ee21f6b 100644 (file)
@@ -56,8 +56,13 @@ abstract class Proxy {
 
        /**
         * @param boolean $passOnWhen
+        * @param string $method
         */
        abstract protected function callOnLastSeenOn($id, $method, $parameters, $passOnWhen);
+
+       /**
+        * @param string $method
+        */
        abstract protected function walkBackends($id, $method, $parameters);
 
        /**
@@ -95,6 +100,9 @@ abstract class Proxy {
                return unserialize(base64_decode($this->cache->get($key)));
        }
 
+       /**
+        * @param string $key
+        */
        public function isCached($key) {
                $key = $this->getCacheKey($key);
                return $this->cache->hasKey($key);
index 5079642d9547054b574447723e330080a475caf3..e79090febc14d47015accbbca9553b818110a9ae 100644 (file)
@@ -865,8 +865,8 @@ class Wizard extends LDAPUtility {
        /**
         * @brief does a cumulativeSearch on LDAP to get different values of a
         * specified attribute
-        * @param $filters array, the filters that shall be used in the search
-        * @param $attr the attribute of which a list of values shall be returned
+        * @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
         * be processed if there were already findings, defaults to true
         * @param string $maxF string. if not null, this variable will have the filter that
@@ -933,8 +933,8 @@ class Wizard extends LDAPUtility {
         * @brief determines if and which $attr are available on the LDAP server
         * @param string[] $objectclasses the objectclasses to use as search filter
         * @param string $attr the attribute to look for
-        * @param $dbkey the dbkey of the setting the feature is connected to
-        * @param $confkey the confkey counterpart for the $dbkey as used in the
+        * @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
         * shall be pre-selected via the result
index 1088dafab7154e24a73cd7b1a2cdb8160f8b1d20..4a147cf9884c365b73b7b44bc1d4e25b562c35e3 100644 (file)
@@ -25,7 +25,6 @@
 
 namespace OCA\user_ldap;
 
-use OCA\user_ldap\lib\ILDAPWrapper;
 use OCA\user_ldap\lib\BackendUtility;
 
 class USER_LDAP extends BackendUtility implements \OCP\UserInterface {
@@ -139,7 +138,7 @@ 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
-        * @return image data (provided by LDAP) | false
+        * @return string data (provided by LDAP) | false
         */
        private function getAvatarImage($uid, $dn) {
                $attributes = array('jpegPhoto', 'thumbnailPhoto');
index f971124cfdc16b058a460c4b4957e2da5a7ab777..a4d710aa9742b502f12a0487adabb2267cbe8d36 100644 (file)
@@ -11,7 +11,6 @@ namespace OC\Core\Command\Db;
 use Symfony\Component\Console\Command\Command;
 use Symfony\Component\Console\Input\InputArgument;
 use Symfony\Component\Console\Input\InputInterface;
-use Symfony\Component\Console\Input\InputOption;
 use Symfony\Component\Console\Output\OutputInterface;
 
 class GenerateChangeScript extends Command {
index ea9825b0f619fbf4006335fa68e5867dd348a604..6bc1dba44aab90363a45f97444c2c757fc29f331 100644 (file)
@@ -9,9 +9,7 @@
 namespace OC\Core\Command;
 
 use Symfony\Component\Console\Command\Command;
-use Symfony\Component\Console\Input\InputArgument;
 use Symfony\Component\Console\Input\InputInterface;
-use Symfony\Component\Console\Input\InputOption;
 use Symfony\Component\Console\Output\OutputInterface;
 
 class Status extends Command {
index 1d105b67a0684ad9daeb6c41828214fd1cabb618..128d27aa3dbd5c64f8df5904e8c91c6aa746cd30 100644 (file)
@@ -10,9 +10,7 @@ namespace OC\Core\Command;
 
 use OC\Updater;
 use Symfony\Component\Console\Command\Command;
-use Symfony\Component\Console\Input\InputArgument;
 use Symfony\Component\Console\Input\InputInterface;
-use Symfony\Component\Console\Input\InputOption;
 use Symfony\Component\Console\Output\OutputInterface;
 
 class Upgrade extends Command {
index f95ba251bcca158e2c38a5414b97d0a50c63f793..70c5a8566b722bf432504fd1b4df923766d08cd5 100644 (file)
@@ -9,10 +9,8 @@
 namespace OC\Core\Command\User;
 
 use Symfony\Component\Console\Command\Command;
-use Symfony\Component\Console\Input\InputArgument;
 use Symfony\Component\Console\Input\InputInterface;
 use Symfony\Component\Console\Output\OutputInterface;
-use Symfony\Component\Console\Helper\TableHelper;
 
 class Report extends Command {
        protected function configure() {
index e60a8a4e0b12b1bd5a7757a4536af33df19c92e8..47f983cce35caac6dbe8f4d317335da48c30ddcb 100644 (file)
@@ -640,6 +640,8 @@ class OC_App{
 
        /**
         * register an admin form to be shown
+        * @param string $app
+        * @param string $page
         */
        public static function registerAdmin($app, $page) {
                self::$adminForms[]= $app.'/'.$page.'.php';
index a32c1126c7bcd91f8687c5c2c81b527239c202d4..cdaaebb87e5dbc67d5244b36d7aeb49aa1bc66a5 100644 (file)
@@ -69,6 +69,9 @@ class AppConfig implements \OCP\IAppConfig {
                return $this->cache[$app];
        }
 
+       /**
+        * @param string $app
+        */
        private function getAppValues($app) {
                $appCache = $this->getAppCache($app);
                if (array_search($app, $this->appsLoaded) === false) {
@@ -184,7 +187,7 @@ class AppConfig implements \OCP\IAppConfig {
         * @brief Deletes a key
         * @param string $app app
         * @param string $key key
-        * @return bool
+        * @return boolean|null
         */
        public function deleteKey($app, $key) {
                $where = array(
@@ -200,7 +203,7 @@ class AppConfig implements \OCP\IAppConfig {
        /**
         * @brief Remove app from appconfig
         * @param string $app app
-        * @return bool
+        * @return boolean|null
         *
         * Removes all keys in appconfig belonging to the app.
         */
index b835188661a4ded18272c70476dc690585ce44b9..3b13194862d105dc41af78b451b6fcbd0ec08dd7 100644 (file)
@@ -25,7 +25,6 @@
 namespace OC\AppFramework;
 
 use OC\AppFramework\DependencyInjection\DIContainer;
-use OCP\AppFramework\IAppContainer;
 
 
 /**
index b6ccf60db4b8e9e03d01b7a1e1d50a19651084ba..4821ecaf67b74f918f4619a6ccf874497e925c53 100644 (file)
@@ -34,7 +34,6 @@ use OC\AppFramework\Utility\SimpleContainer;
 use OC\AppFramework\Utility\TimeFactory;
 use OCP\AppFramework\IApi;
 use OCP\AppFramework\IAppContainer;
-use OCP\AppFramework\IMiddleWare;
 use OCP\AppFramework\Middleware;
 use OCP\IServerContainer;
 
index 5c4e1c06dc3102041f427fb9482c7bcbed0a550a..05353349065a18e6baa3c03c3080d813bba3c0b9 100644 (file)
@@ -24,8 +24,8 @@
 
 namespace OC\AppFramework\Http;
 
-use OCP\AppFramework\Http\Response,
-               OCP\AppFramework\Http;
+use OCP\AppFramework\Http\Response;
+use OCP\AppFramework\Http;
 
 
 /**
index 598743e523f97ca290ef5cd7ff242db73e510865..2a715598fc48015b03dd1804c052a674b474b50a 100644 (file)
@@ -26,7 +26,6 @@ namespace OC\AppFramework\Middleware;
 
 use OCP\AppFramework\Controller;
 use OCP\AppFramework\Http\Response;
-use OCP\AppFramework\MiddleWare;
 
 /**
  * This class is used to store and run all the middleware in correct order
index c3143754823972de64196e383cdd9fd02fe195ba..bb02d565fa46392bf6f547358d15c920f7e31241 100644 (file)
@@ -66,7 +66,7 @@ class SecurityMiddleware extends Middleware {
         * This runs all the security checks before a method call. The
         * security checks are determined by inspecting the controller method
         * annotations
-        * @param string/Controller $controller the controllername or string
+        * @param string $controller the controllername or string
         * @param string $methodName the name of the method
         * @throws SecurityException when a security check fails
         */
index a18f8d3ffbd6a629bdc42d1f2409d6466a72e6da..6f51066ddf82d6a71b1785aae42e75cb98e4fe9f 100644 (file)
@@ -88,7 +88,6 @@ abstract class OC_Archive{
        abstract function extractFile($path, $dest);
        /**
         * extract the archive
-        * @param string $path
         * @param string $dest
         * @return bool
         */
index 5538e027ab4e86a7fe749710987592421dbccf37..cbdb565ba35e6440f8c04cfb1379ee7e407ba117 100644 (file)
@@ -22,6 +22,9 @@ class OC_Archive_TAR extends OC_Archive{
        private $tar=null;
        private $path;
 
+       /**
+        * @param string $source
+        */
        function __construct($source) {
                $types=array(null, 'gz', 'bz');
                $this->path=$source;
@@ -122,6 +125,9 @@ class OC_Archive_TAR extends OC_Archive{
                return true;
        }
 
+       /**
+        * @param string $file
+        */
        private function getHeader($file) {
                if ( ! $this->cachedHeaders ) {
                        $this->cachedHeaders = $this->tar->listContent();
index cd5479299a1f1dc6308019d6124f3ffef03337ac..fa5630d7665647525991439055804905b03c304c 100644 (file)
@@ -13,6 +13,9 @@ class OC_Archive_ZIP extends OC_Archive{
        private $zip=null;
        private $path;
 
+       /**
+        * @param string $source
+        */
        function __construct($source) {
                $this->path=$source;
                $this->zip=new ZipArchive();
@@ -125,7 +128,6 @@ class OC_Archive_ZIP extends OC_Archive{
        }
        /**
         * extract the archive
-        * @param string $path
         * @param string $dest
         * @return bool
         */
index c4ad1264fbb6dbd03297cef78d31915acdf47829..d353e486577573572e5f915397c404b9fbf57f7f 100644 (file)
@@ -39,6 +39,9 @@ class ArrayParser {
                return $this->parse($string);
        }
 
+       /**
+        * @param string $string
+        */
        function stripPHPTags($string) {
                $string = trim($string);
                if (substr($string, 0, 5) === '<?php') {
index aa740a89532a84418cba27c6b662df8c18d30bae..afc3c270405e9e4200a30037752e5e09d08667a7 100644 (file)
@@ -38,7 +38,7 @@ class OC_BackgroundJob{
        /**
         * @brief sets the background jobs execution type
         * @param string $type execution type
-        * @return boolean|null
+        * @return false|null
         *
         * This method sets the execution type of the background jobs. Possible types
         * are "none", "ajax", "webcron", "cron"
index 2740d2bd8561629c87c3cf130133176f79196660..26c9026934964c18d48fc8d47b84fa9c59ad2e95 100644 (file)
@@ -31,7 +31,7 @@ class JobList implements IJobList {
        }
 
        /**
-        * @param Job|string $job
+        * @param \Test\BackgroundJob\TestJob $job
         * @param mixed $argument
         */
        public function add($job, $argument = null) {
index baa8820700b9c652e0ed0e39bf4b1f008c61d26b..486e08218a26c9260255e9d62c48e86adbbe6a30 100644 (file)
@@ -35,7 +35,7 @@ class UserCache implements \OCP\ICache {
         * Set a value in the user cache
         *
         * @param string $key
-        * @param mixed $value
+        * @param string $value
         * @param int $ttl Time To Live in seconds. Defaults to 60*60*24
         * @return bool
         */
index 8399d0defbda3ec4c3df0e4fdab738fb78616b61..3649da84973702235a02788401864ab8663a20c5 100644 (file)
@@ -77,7 +77,7 @@ class Config {
        /**
         * @brief Gets a value from config.php
         * @param string $key key
-        * @param mixed $default = null default value
+        * @param string|null $default = null default value
         * @return string the value or $default
         *
         * This function gets the value from config.php. If it does not exist,
@@ -94,7 +94,7 @@ class Config {
        /**
         * @brief Sets a value
         * @param string $key key
-        * @param mixed $value value
+        * @param string $value value
         *
         * This function sets the value and writes the config.php.
         *
index 10cca647e8db5884f7beb3961646e9e46d40edb5..ad759d1d84a7e2142e9066183656836a6d7ce3c8 100644 (file)
@@ -47,7 +47,7 @@ class OC_Connector_Sabre_AbortedUploadDetectionPlugin extends Sabre_DAV_ServerPl
        }
 
        /**
-        * @param $filePath
+        * @param string $filePath
         * @param Sabre_DAV_INode $node
         * @throws Sabre_DAV_Exception_BadRequest
         */
index d62553253439162aa9bbd2bea265df380846f84d..8099794f67010446c001d21970fcd840345a2820 100644 (file)
@@ -46,6 +46,7 @@ class OC_Connector_Sabre_QuotaPlugin extends Sabre_DAV_ServerPlugin {
         * This method is called before any HTTP method and validates there is enough free space to store the file
         *
         * @throws Sabre_DAV_Exception
+        * @param string $uri
         * @return bool
         */
        public function checkQuota($uri, $data = null) {
index 23e9593a5795624976e00830238ea6746bd9c0a6..cfdac766bffaf214457c45b0b393b17f0ca2425f 100644 (file)
@@ -252,7 +252,7 @@ class OC_DB {
         *                                        an array with 'sql' and optionally 'limit' and 'offset' keys
         *                                      .. or a simple sql query string
         * @param array $parameters
-        * @return result
+        * @return OC_DB_StatementWrapper
         * @throws DatabaseException
         */
        static public function executeAudited( $stmt, array $parameters = null) {
@@ -315,6 +315,7 @@ class OC_DB {
         * @brief Insert a row if a matching row doesn't exists.
         * @param string $table. The table to insert into in the form '*PREFIX*tableName'
         * @param array $input. An array of fieldname/value pairs
+        * @param string $table
         * @return boolean number of updated rows
         */
        public static function insertIfNotExist($table, $input) {
index 42d5bbab112c5f85099dd2fbd3c9e355039d01ea..a2a62a8147595ec7a4f235b8fb775da77e4743f6 100644 (file)
@@ -51,6 +51,9 @@ class OC_DB_MDB2SchemaWriter {
                }
        }
 
+       /**
+        * @param SimpleXMLElement $xml
+        */
        private static function saveColumn($column, $xml) {
                $xml->addChild('name', $column->getName());
                switch($column->getType()) {
@@ -114,6 +117,9 @@ class OC_DB_MDB2SchemaWriter {
                }
        }
 
+       /**
+        * @param SimpleXMLElement $xml
+        */
        private static function saveIndex($index, $xml) {
                $xml->addChild('name', $index->getName());
                if ($index->isPrimary()) {
index 09b30af48b28e0395af6a38c317a963d4feee2ad..be7f4e14a113c65e7e4abb3802dbd849d3e735ac 100644 (file)
@@ -41,7 +41,7 @@ class OC_FileChunking {
         * Stores the given $data under the given $key - the number of stored bytes is returned
         *
         * @param string $index
-        * @param $data
+        * @param resource $data
         * @return int
         */
        public function store($index, $data) {
index 62187e1ec74944bf1c2934ca6d830aab9dc30262..88976c1b8e510e46d1f4a1ee96aac85e4b3d2809 100644 (file)
@@ -110,6 +110,7 @@ class OC_FileProxy{
 
        /**
         * @param string $operation
+        * @param string|boolean $path
         *
         * @return string
         */
index 4a6b9d8ca0ec936e201b1ff3cac16ef5bd0fd973..6a6916537d42c6272342cdff21b687aa0b8e577f 100644 (file)
@@ -32,6 +32,9 @@ class OC_Files {
                return \OC\Files\Filesystem::getFileInfo($path, $includeMountPoints);
        }
 
+       /**
+        * @param string $path
+        */
        static public function getDirectoryContent($path){
                return \OC\Files\Filesystem::getDirectoryContent($path);
        }
index e4114e572fed6a60216150467acd66189a174e2f..7a45b9e9e96cb9e1c1dd7036a9e8b1a6a9cd4c95 100644 (file)
@@ -8,8 +8,6 @@
 
 namespace OC\Files\Cache;
 
-use OCP\Util;
-
 /**
  * listen to filesystem hooks and change the cache accordingly
  */
index c77571cd2a7fe0f90feac8ac51923b9195c36e7b..2dbdd80a26bb66da9a915d2da76f3234e22d6b76 100644 (file)
@@ -29,6 +29,10 @@ class FileInfo implements \OCP\Files\FileInfo, \ArrayAccess {
         */
        private $internalPath;
 
+       /**
+        * @param string|boolean $path
+        * @param Storage\Storage $storage
+        */
        public function __construct($path, $storage, $internalPath, $data) {
                $this->path = $path;
                $this->storage = $storage;
index d1ebad3490c7ddc4a87517510e5d05da1bb28b31..0face6aa3bbfcdb29e57989e317380aa74bff12c 100644 (file)
@@ -668,6 +668,9 @@ class Filesystem {
                return self::$defaultInstance->search($query);
        }
 
+       /**
+        * @param string $query
+        */
        static public function searchByMime($query) {
                return self::$defaultInstance->searchByMime($query);
        }
index 923f53821b2b2fea687e56d0c358237072cbce65..d9e0ddc2d610d17c6ab4e9a82c09d88ee01ac78d 100644 (file)
@@ -9,7 +9,6 @@
 namespace OC\Files\Node;
 
 use OC\Files\Cache\Cache;
-use OC\Files\Cache\Scanner;
 use OCP\Files\NotFoundException;
 use OCP\Files\NotPermittedException;
 
index 643a587dfc2f13d19836c611c22c9b0a348d9876..bc07591174976db1bdc8bb496fcb85622e557ee1 100644 (file)
@@ -8,8 +8,6 @@
 
 namespace OC\Files\Node;
 
-use OC\Files\Cache\Cache;
-use OC\Files\Cache\Scanner;
 use OCP\Files\NotFoundException;
 use OCP\Files\NotPermittedException;
 
index a22a2aaf4e9861e68a330922f85b35b122a5d9c8..70135285b0d1b5b56f0abbec6b234d77039907c3 100644 (file)
@@ -9,7 +9,6 @@
 namespace OC\Files\Node;
 
 use OC\Files\Cache\Cache;
-use OC\Files\Cache\Scanner;
 use OC\Files\Mount\Manager;
 use OC\Files\Mount\Mount;
 use OCP\Files\NotFoundException;
index 734131261c817341a1cd9c22697d769c95c9d0d9..966234cb04d981eb5b05fc5fd22c24c8be1fb3d6 100644 (file)
@@ -33,6 +33,9 @@ class Loader {
                return $this->wrap($mountPoint, new $class($arguments));
        }
 
+       /**
+        * @param string|boolean $mountPoint
+        */
        public function wrap($mountPoint, $storage) {
                foreach ($this->storageWrappers as $wrapper) {
                        $storage = $wrapper($mountPoint, $storage);
index c8b22cfb39ffdb15ed3f88950a71d3fbb720201a..1bab3489a28fe87750c1cc923fbc6941676cf723 100644 (file)
@@ -326,12 +326,18 @@ class MappedLocal extends \OC\Files\Storage\Common{
                return $this->filemtime($path)>$time;
        }
 
+       /**
+        * @param string $path
+        */
        private function buildPath($path, $create=true) {
                $path = $this->stripLeading($path);
                $fullPath = $this->datadir.$path;
                return $this->mapper->logicToPhysical($fullPath, $create);
        }
 
+       /**
+        * @param string $path
+        */
        private function cleanMapper($path, $isLogicPath=true, $recursive=true) {
                $fullPath = $path;
                if ($isLogicPath) {
index 558bc60cf0604b62a0dc6bbbd5e4f23c07264cf1..a802a8fcb8bcda9b8212f8aeac0d45d56d04918e 100644 (file)
@@ -69,6 +69,9 @@ class Scanner extends PublicEmitter {
                });
        }
 
+       /**
+        * @param string $dir
+        */
        public function backgroundScan($dir) {
                $mounts = $this->getMounts($dir);
                foreach ($mounts as $mount) {
index ed01ad0b616c6bfe0bf94a786c4b20625b5000db..7094d885af6abfa8e03351c27f51505f2441879d 100644 (file)
@@ -12,6 +12,10 @@ class OC_Geo{
         * @param (string) $longitude - Longitude
         * @return (string) $timezone - closest timezone
         */
+       /**
+        * @param integer $latitude
+        * @param integer $longitude
+        */
        public static function timezone($latitude, $longitude) {
                $alltimezones = DateTimeZone::listIdentifiers();
                $variances = array();
index 444788c97f1fdd1193447a5aeaf2ebe08f9e9ba4..4c187b538aff6d3dcc884109c176023efcbd33f7 100644 (file)
@@ -243,7 +243,7 @@ class OC_Group {
 
        /**
         * @brief get a list of all users in several groups
-        * @param array $gids
+        * @param string[] $gids
         * @param string $search
         * @param int $limit
         * @param int $offset
index 9516fd52ff8b765eaedf732d65a689290ce54040..da26e1b910e633b03437d2cf26a4de4defb7f1f4 100644 (file)
@@ -28,7 +28,7 @@ class OC_Group_Dummy extends OC_Group_Backend {
        private $groups=array();
        /**
         * @brief Try to create a new group
-        * @param $gid The name of the group to create
+        * @param string $gid The name of the group to create
         * @returns true/false
         *
         * Trys to create a new group. If the group name already exists, false will
index 42afa35ea5680791a65a7d7b29561def7b3f22b0..42685ddab5cca91c6964c46826e225f5e2706a2c 100644 (file)
@@ -34,7 +34,7 @@ class OC_Image {
 
        /**
        * @brief Get mime type for an image file.
-       * @param $filepath The path to a local image file.
+       * @param string|null $filepath The path to a local image file.
        * @returns string The mime type if the it could be determined, otherwise an empty string.
        */
        static public function getMimeTypeForFile($filePath) {
@@ -163,6 +163,7 @@ class OC_Image {
        /**
        * @brief Saves the image.
        * @returns bool
+       * @param string $filePath
        */
 
        public function save($filePath=null) {
@@ -704,7 +705,7 @@ class OC_Image {
 
        /**
        * @brief Resizes the image preserving ratio.
-       * @param integer $maxsize The maximum size of either the width or height.
+       * @param integer $maxSize The maximum size of either the width or height.
        * @returns bool
        */
        public function resize($maxSize) {
@@ -852,8 +853,8 @@ class OC_Image {
 
        /**
         * @brief Resizes the image to fit within a boundry while preserving ratio.
-        * @param $maxWidth
-        * @param $maxHeight
+        * @param integer $maxWidth
+        * @param integer $maxHeight
         * @returns bool
         */
        public function fitIn($maxWidth, $maxHeight) {
index 1b974c6e5a500130ddf19d9e5a7ee587496c3099..fe7417990c8af24d2a2b3685ea22f365ff36097b 100644 (file)
@@ -236,30 +236,6 @@ class OC_Installer{
 
        /**
         * @brief Update an application
-        * @param array $data with all information
-        *
-        * This function installs an app. All information needed are passed in the
-        * associative array $data.
-        * The following keys are required:
-        *   - source: string, can be "path" or "http"
-        *
-        * One of the following keys is required:
-        *   - path: path to the file containing the app
-        *   - href: link to the downloadable file containing the app
-        *
-        * The following keys are optional:
-        *   - pretend: boolean, if set true the system won't do anything
-        *   - noupgrade: boolean, if true appinfo/upgrade.php won't be loaded
-        *
-        * This function works as follows
-        *   -# fetching the file
-        *   -# removing the old files
-        *   -# unzipping new file
-        *   -# including appinfo/upgrade.php
-        *   -# setting the installed version
-        *
-        * upgrade.php can determine the current installed version of the app using
-        * "OC_Appconfig::getValue($appid, 'installed_version')"
         */
        public static function updateApp( $app ) {
                $ocsid=OC_Appconfig::getValue( $app, 'ocsid');
@@ -270,7 +246,6 @@ class OC_Installer{
 
        /**
         * @brief Check if an update for the app is available
-        * @param string $name name of the application
         * @return string|false false or the version number of the update
         *
         * The function will check if an update for a version is available
@@ -316,7 +291,7 @@ class OC_Installer{
         * @brief Removes an app
         * @param string $name name of the application to remove
         * @param $options array with options
-        * @return boolean
+        * @return boolean|null
         *
         * This function removes an app. $options is an associative array. The
         * following keys are optional:ja
index 1d8152c58448d2946415d668bccb514f064be54c..1ade18ea4277352d7c341bb94f4fa39dac0b9998 100644 (file)
@@ -99,6 +99,9 @@ class OC_L10N implements \OCP\IL10N {
                $this->lang = $lang;
        }
 
+       /**
+        * @param string $transFile
+        */
        public function load($transFile) {
                $this->app = true;
                include $transFile;
index 46a8068c3b44a01075fa8cda9ee8dc9a50bb218f..b6c3542a673cc9c29972769d0f3a0df08d51c099 100644 (file)
@@ -118,6 +118,8 @@ class OC_Appconfig {
         *
         * @param app
         * @param key
+        * @param string|false $app
+        * @param string|false $key
         * @return array
         */
        public static function getValues($app, $key) {
index a663db7598b5bf2925c32c5a201aae5819484caa..fcde12796ca1cdd7b108117222cf0aa8f2081a5b 100644 (file)
@@ -41,7 +41,7 @@ class OC_Preferences{
        /**
         * @brief Get all apps of a user
         * @param string $user user
-        * @return array with app ids
+        * @return integer[] with app ids
         *
         * This function returns a list of all apps of the user that have at least
         * one entry in the preferences table.
index b930b7192892631b68da817bd53fb199ac0bee4c..3fb3e334ea228b08537c8c1f2f9ce1676dfabae0 100644 (file)
@@ -431,6 +431,7 @@ class OC_Migrate{
        /**
         * @brief connects to migration.db, or creates if not found
         * @param $db optional path to migration.db, defaults to user data dir
+        * @param string $path
         * @return bool whether the operation was successful
         */
        static private function connectDB( $path=null ) {
@@ -460,7 +461,7 @@ class OC_Migrate{
 
        /**
         * @brief creates the tables in migration.db from an apps database.xml
-        * @param $appid string id of the app
+        * @param string $appid string id of the app
         * @return bool whether the operation was successful
         */
        static private function createAppTables( $appid ) {
index e7b6543171aecdb5cf72d9421eb014ccadd93627..43eba89b8d58360b4c85336ebc87e2c9047f3544 100644 (file)
@@ -34,9 +34,9 @@ class OC_Migration_Content{
 
        /**
        * @brief sets up the
-       * @param $zip ZipArchive object
+       * @param ZipArchive $zip ZipArchive object
        * @param $db a database object (required for exporttype user)
-       * @return bool
+       * @return boolean|null
        */
        public function __construct( $zip, $db=null ) {
 
@@ -74,7 +74,7 @@ class OC_Migration_Content{
 
        /**
        * @brief processes the db query
-       * @param $query the query to process
+       * @param string $query the query to process
        * @return string of processed query
        */
        private function processQuery( $query ) {
@@ -130,7 +130,7 @@ class OC_Migration_Content{
 
        /**
        * @brief saves a sql data set into migration.db
-       * @param $data a sql data set returned from self::prepare()->query()
+       * @param OC_DB_StatementWrapper $data a sql data set returned from self::prepare()->query()
        * @param $options array of copyRows options
        * @return void
        */
index 234ab3351f37ad73545109601cd8d10f829dbaf2..2829a97a776f2d4623e4cada016b7e42cb212a9b 100644 (file)
@@ -30,7 +30,7 @@ abstract class OC_Migration_Provider{
 
        /**
        * @brief sets the OC_Migration_Content object to $this->content
-       * @param $content a OC_Migration_Content object
+       * @param OC_Migration_Content $content a OC_Migration_Content object
        */
        public function setData( $uid, $content, $info=null ) {
                $this->content = $content;
index 2305b3eccfed2002c81cccd7b7638a5bb1529268..bbe965ce561b6d986c3fd98f8a089f3e80e25152 100644 (file)
@@ -23,9 +23,6 @@
 *
 */
 
-use Symfony\Component\Routing\Exception\ResourceNotFoundException;
-use Symfony\Component\Routing\Exception\MethodNotAllowedException;
-
 /**
  * Class to handle open collaboration services API requests
  *
@@ -38,7 +35,7 @@ class OC_OCS {
        * @param string $method HTTP method to read the key from
        * @param string $key Parameter to read
        * @param string $type Variable type to format data
-       * @param mixed $default Default value to return if the key is not found
+       * @param string $default Default value to return if the key is not found
        * @return string Data or if the key is not found and no default is set it will exit with a 400 Bad request
        */
        public static function readData($method, $key, $type = 'raw', $default = null) {
index 7ebbf7aa970cd18679adbd77f90abc50dcb82174..d45e6e77089dea6fabfbb71141b54d0fc5d3dcaf 100644 (file)
@@ -111,7 +111,7 @@ class Preferences {
        /**
         * @brief Get all apps of an user
         * @param string $user user
-        * @return array with app ids
+        * @return integer[] with app ids
         *
         * This function returns a list of all apps of the user that have at least
         * one entry in the preferences table.
index a7e7bd0ea1dc307f14914bbf05e9ceb512dd4507..0fd20b3cc1fb7f302cb8a8c71c2c7ec9dc69c616 100755 (executable)
@@ -192,7 +192,7 @@ class OC_Request {
 
        /**
         * @brief Check if the requestor understands gzip
-        * @return boolean true for gzip encoding supported
+        * @return false|string true for gzip encoding supported
         */
        static public function acceptGZip() {
                if (!isset($_SERVER['HTTP_ACCEPT_ENCODING'])) {
index fbf771790970f48d44efe734be9a159b23ec66a4..76abbf308460d076ad9ae5968e72c702f82c502c 100644 (file)
@@ -15,8 +15,8 @@ class SearchResultSorter {
        private $log;
 
        /**
-        * @param $search the search term as was given by the user
-        * @param $key the array key containing the value that should be compared
+        * @param string $search the search term as was given by the user
+        * @param string $key the array key containing the value that should be compared
         * against
         * @param $encoding optional, encoding to use, defaults to UTF-8
         * @param $log optional, an \OC\Log instance
index e698d855a9102a0994478787b378e8a329d48470..232a29939cccafeed9366fbb4bfd8900b93eafc7 100644 (file)
@@ -30,6 +30,8 @@ class Base {
        /**
         * @param string $serverroot
         * @param string|false $app_dir
+        * @param string $theme
+        * @param string $app
         */
        protected function getAppTemplateDirs($theme, $app, $serverroot, $app_dir) {
                // Check if the app is in the app folder or in the root
@@ -47,6 +49,7 @@ class Base {
 
        /**
         * @param string $serverroot
+        * @param string $theme
         */
        protected function getCoreTemplateDirs($theme, $serverroot) {
                return array(
@@ -119,6 +122,7 @@ class Base {
 
        /**
         * @brief doing the actual work
+        * @param string $file
         * @return string content
         *
         * Includes the template file, fetches its output
index 05cd752cd4fe20ed2efa3df5e124aa0ed7c9b7fc..4676fceb37d9a36addd74aeca1bcd83c3cb1b17c 100644 (file)
@@ -15,6 +15,7 @@ class TemplateFileLocator {
 
        /**
         * @param string[] $dirs
+        * @param string $form_factor
         */
        public function __construct( $form_factor, $dirs ) {
                $this->form_factor = $form_factor;
index a8e7967783e4c386b169c5a9675fc1adfd8e18b8..86a01f96258ccc590274aac2024f5c0be078a8a1 100644 (file)
@@ -516,6 +516,8 @@ class OC_User {
         * @returns array with all uids
         *
         * Get a list of all users.
+        * @param integer $limit
+        * @param integer $offset
         */
        public static function getUsers($search = '', $limit = null, $offset = null) {
                $users = self::getManager()->search($search, $limit, $offset);
index 094a8f2e2a4484615f38febaa7b3365caa1970f2..15e6643dfb36f39ebcea8b47b22593b1f712ea4e 100644 (file)
@@ -256,7 +256,7 @@ class OC_User_Database extends OC_User_Backend {
        /**
         * counts the users in the database
         *
-        * @return int | bool
+        * @return false|string | bool
         */
        public function countUsers() {
                $query = OC_DB::prepare('SELECT COUNT(*) FROM `*PREFIX*users`');
index af39d983f6748655d64689e748c1e9880b028303..829eedce044157b13abad6656c22d3db9a6784d7 100755 (executable)
@@ -218,7 +218,7 @@ class OC_Util {
         * @brief add a javascript file
         *
         * @param string $application
-        * @param mixed $file filename
+        * @param string|null $file filename
         * @return void
         */
        public static function addScript( $application, $file = null ) {
@@ -237,7 +237,7 @@ class OC_Util {
         * @brief add a css file
         *
         * @param string $application
-        * @param mixed $file filename
+        * @param string|null $file filename
         * @return void
         */
        public static function addStyle( $application, $file = null ) {
@@ -570,6 +570,7 @@ class OC_Util {
 
        /**
         * @brief Check if the app is enabled, redirects to home if not
+        * @param string $app
         * @return void
         */
        public static function checkAppEnabled($app) {
index dc8da96787168ed41ff290b61fb730e9eb68108d..7c2219bd046ed3346243c49bd71237b5ea90d734 100644 (file)
@@ -68,7 +68,7 @@ abstract class Controller {
         *                     1. URL parameters
         *                     2. POST parameters
         *                     3. GET parameters
-        * @param mixed $default If the key is not found, this value will be returned
+        * @param string $default If the key is not found, this value will be returned
         * @return mixed the content of the array
         */
        public function params($key, $default=null){
@@ -131,7 +131,7 @@ abstract class Controller {
         * @param array $params the template parameters in key => value structure
         * @param string $renderAs user renders a full page, blank only your template
         *                          admin an entry in the admin settings
-        * @param array $headers set additional headers in name/value pairs
+        * @param string[] $headers set additional headers in name/value pairs
         * @return \OCP\AppFramework\Http\TemplateResponse containing the page
         */
        public function render($templateName, array $params=array(),
index bcd27c1d1986de38af92d23db40f75a28c468e96..03b94403b4772995a8b248c865c2b27e4f135b22 100644 (file)
@@ -59,7 +59,7 @@ class BackgroundJob {
         * sets the background jobs execution type
         *
         * @param string $type execution type
-        * @return boolean|null
+        * @return false|null
         *
         * This method sets the execution type of the background jobs. Possible types
         * are "none", "ajax", "webcron", "cron"
@@ -115,7 +115,7 @@ class BackgroundJob {
         * @deprecated
         * Gets one queued task
         * @param int $id ID of the task
-        * @return \OC\BackgroundJob\Job|null array
+        * @return BackgroundJob\IJob array
         */
        public static function findQueuedTask($id) {
                $jobList = \OC::$server->getJobList();
index 5231e9537a95ab34fa4167d47b1122c1cf5956ba..eaf11c4f684368d2becf405e31194f265fb2211b 100644 (file)
@@ -14,6 +14,7 @@ interface IJob {
         *
         * @param \OCP\BackgroundJob\IJobList $jobList The job list that manages the state of this job
         * @param \OC\Log $logger
+        * @return void
         */
        public function execute($jobList, $logger = null);
 
index 72f3fe863da71dc5d1f4f5b4e36cdadb2eba010f..c9b546605b8cd5ce60caf9b6be060997b87cce98 100644 (file)
@@ -14,14 +14,17 @@ interface IJobList {
         *
         * @param \OCP\BackgroundJob\IJob |string $job
         * @param mixed $argument The argument to be passed to $job->run() when the job is exectured
+        * @param string $job
+        * @return void
         */
        public function add($job, $argument = null);
 
        /**
         * Remove a job from the list
         *
-        * @param \OCP\BackgroundJob\IJob|string $job
+        * @param IJob $job
         * @param mixed $argument
+        * @return void
         */
        public function remove($job, $argument = null);
 
@@ -58,6 +61,7 @@ interface IJobList {
         * set the job that was last ran to the current time
         *
         * @param \OCP\BackgroundJob\IJob $job
+        * @return void
         */
        public function setLastJob($job);
 
@@ -72,6 +76,7 @@ interface IJobList {
         * set the lastRun of $job to now
         *
         * @param \OCP\BackgroundJob\IJob $job
+        * @return void
         */
        public function setLastRun($job);
 }
index da9bca7552566f4cf31aa17582b8894ad83f1e2a..5b9d64ecc4125ddb32b9246ea9ed070053eb64e3 100644 (file)
@@ -96,7 +96,7 @@ namespace OCP\Contacts {
                 * This function can be used to delete the contact identified by the given id
                 *
                 * @param object $id the unique identifier to a contact
-                * @param  $address_book_key
+                * @param   $address_book_key
                 * @return bool successful or not
                 */
                function delete($id, $address_book_key);
@@ -106,7 +106,7 @@ namespace OCP\Contacts {
                 * Otherwise the contact will be updated by replacing the entire data set.
                 *
                 * @param array $properties this array if key-value-pairs defines a contact
-                * @param  $address_book_key string to identify the address book in which the contact shall be created or updated
+                * @param   $address_book_key string to identify the address book in which the contact shall be created or updated
                 * @return array representing the contact just created or updated
                 */
                function createOrUpdate($properties, $address_book_key);
index 75c52b88fc78771cfd7a95287bf2340b5e909502..e2d9c81d442ad7f8e6b714c90fc01ee1b4c7f058 100644 (file)
@@ -37,7 +37,6 @@ namespace OCP;
 class Files {
        /**
         * Recusive deletion of folders
-        * @param string $path to the folder
         * @return bool
         */
        static function rmdirr( $dir ) {
index 3b6484c09d56058cd0deba8ecaec1233ce0d3767..1f31898bf2cd5142d6e941a4452dc7b1325a1139 100644 (file)
@@ -57,6 +57,7 @@ interface IAppConfig {
         *
         * @param app
         * @param key
+        * @param string $key
         * @return array
         */
        public function getValues($app, $key);
@@ -68,6 +69,7 @@ interface IAppConfig {
         * @param string $value value
         *
         * Sets a value. If the key did not exist before it will be created.
+        * @return void
         */
        public function setValue($app, $key, $value);
 
index 8944e660780da7586d057662d7d986a96427d983..0ebbd9f5a71f657ff047a151e9baff9e59b4191e 100644 (file)
@@ -47,7 +47,7 @@ interface IConfig {
         * Looks up a system wide defined value
         *
         * @param string $key the key of the value, under which it was saved
-        * @param mixed $default the default value to be returned if the value isn't set
+        * @param string $default the default value to be returned if the value isn't set
         * @return string the saved value
         */
        public function getSystemValue($key, $default = '');
index 7965d4152f9c89b303d9197c5dd9349ec64e04a1..f8ebaa668f1fddaf55ff5e718c287d4be74248dc 100644 (file)
@@ -147,7 +147,7 @@ interface ITags {
        * Creates a tag/object relation.
        *
        * @param int $objid The id of the object
-       * @param int|string $tag The id or name of the tag
+       * @param string $tag The id or name of the tag
        * @return boolean Returns false on database error.
        */
        public function tagAs($objid, $tag);
@@ -156,7 +156,7 @@ interface ITags {
        * Delete single tag/object relation from the db
        *
        * @param int $objid The id of the object
-       * @param int|string $tag The id or name of the tag
+       * @param string $tag The id or name of the tag
        * @return boolean
        */
        public function unTag($objid, $tag);
index f6c44a8322b989ac5b98c362962b639a014dbd5e..ebc555dba5fe5bf784a2b5c703ac4e522d1f8474 100644 (file)
@@ -78,6 +78,9 @@ class Share {
         * @param string Backend class
         * @param string (optional) Depends on item type
         * @param array (optional) List of supported file extensions if this item type depends on files
+        * @param string $itemType
+        * @param string $class
+        * @param string $collectionOf
         * @return boolean true if backend is registered or false if error
         */
        public static function registerBackend($itemType, $class, $collectionOf = null, $supportedFileExtensions = null) {
@@ -404,6 +407,7 @@ class Share {
         * @param mixed Parameters
         * @param int Number of items to return (optional) Returns all by default
         * @param bool include collections
+        * @param string $itemType
         * @return Return depends on format
         */
        public static function getItemsShared($itemType, $format = self::FORMAT_NONE, $parameters = null,
@@ -662,6 +666,8 @@ class Share {
         * Unshare an item from all users, groups, and remove all links
         * @param string Item type
         * @param string Item source
+        * @param string $itemType
+        * @param string $itemSource
         * @return boolean true on success or false on failure
         */
        public static function unshareAll($itemType, $itemSource) {
@@ -694,6 +700,8 @@ class Share {
         * Unshare an item shared with the current user
         * @param string Item type
         * @param string Item target
+        * @param string $itemType
+        * @param string $itemTarget
         * @return boolean true on success or false on failure
         *
         * Unsharing from self is not allowed for items inside collections
@@ -1630,6 +1638,7 @@ class Share {
         * @param string User that is the owner of shared item
         * @param string The suggested target originating from a reshare (optional)
         * @param int The id of the parent group share (optional)
+        * @param integer $shareType
         * @return string Item target
         */
        private static function generateTarget($itemType, $itemSource, $shareType, $shareWith, $uidOwner,
@@ -1937,6 +1946,8 @@ interface Share_Backend {
         * Get the source of the item to be stored in the database
         * @param string Item source
         * @param string Owner of the item
+        * @param string $itemSource
+        * @param string $uidOwner
         * @return boolean Source
         *
         * Return an array if the item is file dependent, the array needs two keys: 'item' and 'file'
@@ -1993,6 +2004,8 @@ interface Share_Backend_File_Dependent extends Share_Backend {
         * Get the file path of the item
         * @param string Item source
         * @param string User that is the owner of shared item
+        * @param string $itemSource
+        * @param string $uidOwner
         * @return boolean
         */
        public function getFilePath($itemSource, $uidOwner);
index 9c4324e63e0b20430955d6957f2c7fac1c6ec6fd..233beebd68a8551964fd1f1d5434ef28a4111c29 100644 (file)
@@ -9,6 +9,10 @@
 class Test_API extends PHPUnit_Framework_TestCase {
 
        // Helps build a response variable
+
+       /**
+        * @param string $message
+        */
        function buildResponse($shipped, $data, $code, $message=null) {
                return array(
                        'shipped' => $shipped,
@@ -18,6 +22,10 @@ class Test_API extends PHPUnit_Framework_TestCase {
        }
 
        // Validate details of the result
+
+       /**
+        * @param OC_OCS_Result $result
+        */
        function checkResult($result, $success) {
                // Check response is of correct type
                $this->assertInstanceOf('OC_OCS_Result', $result);
index 6cf0da879ff181c26f26ca7c18e7d620d7abdae8..9841dcaa1f761300154e1653e76d013eebb1dcf6 100644 (file)
@@ -24,7 +24,6 @@
 
 namespace OC\AppFramework\Http;
 
-use OC\AppFramework\Core\API;
 use OC\AppFramework\Middleware\MiddlewareDispatcher;
 use OCP\AppFramework\Http;
 //require_once(__DIR__ . "/../classloader.php");
@@ -78,6 +77,10 @@ class DispatcherTest extends \PHPUnit_Framework_TestCase {
        }
 
 
+       /**
+        * @param string $out
+        * @param string $httpHeaders
+        */
        private function setMiddlewareExpections($out=null, 
                $httpHeaders=null, $responseHeaders=array(),
                $ex=false, $catchEx=true) {
index 1a38c38c1e7b8e3567db5913d795a1557c04e890..4f21d77a170fa7612873a425344974bb1751292b 100644 (file)
@@ -25,8 +25,8 @@
 namespace OC\AppFramework\Http;
 
 
-use OCP\AppFramework\Http\Response,
-       OCP\AppFramework\Http;
+use OCP\AppFramework\Http\Response;
+use OCP\AppFramework\Http;
 
 
 class ResponseTest extends \PHPUnit_Framework_TestCase {
index 95d42e4eb8ec81315e44fe1b486e4a80fec08016..f16b14150c34a9a18c45255918feba42bbe4e20e 100644 (file)
@@ -50,6 +50,9 @@ class TestMiddleware extends Middleware {
 
        private $beforeControllerThrowsEx;
 
+       /**
+        * @param boolean $beforeControllerThrowsEx
+        */
        public function __construct($beforeControllerThrowsEx) {
                self::$beforeControllerCalled = 0;
                self::$afterControllerCalled = 0;
index dae6135dc54f574e8ca9cebfa5e1b0fb54c857b2..63c48a628291edc8ea47ef95d7713fe20fb4c84c 100644 (file)
@@ -58,6 +58,9 @@ class SecurityMiddlewareTest extends \PHPUnit_Framework_TestCase {
        }
 
 
+       /**
+        * @param string $method
+        */
        private function checkNavEntry($method){
                $api = $this->getAPI();
 
@@ -79,6 +82,10 @@ class SecurityMiddlewareTest extends \PHPUnit_Framework_TestCase {
        }
 
 
+       /**
+        * @param string $method
+        * @param string $test
+        */
        private function ajaxExceptionStatus($method, $test, $status) {
                $api = $this->getAPI();
                $api->expects($this->any())
@@ -183,6 +190,10 @@ class SecurityMiddlewareTest extends \PHPUnit_Framework_TestCase {
        }
 
 
+       /**
+        * @param string $method
+        * @param string $expects
+        */
        private function securityCheck($method, $expects, $shouldFail=false){
                $api = $this->getAPI();
                $api->expects($this->once())
index a7aa922db12becd05a88115ca7608275d466f70f..d0244cf2511784c0635dfe7e8b110499cdccf888 100644 (file)
@@ -78,6 +78,13 @@ class RouteConfigTest extends \PHPUnit_Framework_TestCase
                $this->assertResource($routes, 'admin_accounts', '/admin/accounts', 'AdminAccountsController', 'adminAccountId');
        }
 
+       /**
+        * @param string $name
+        * @param string $verb
+        * @param string $url
+        * @param string $controllerName
+        * @param string $actionName
+        */
        private function assertSimpleRoute($routes, $name, $verb, $url, $controllerName, $actionName)
        {
                // route mocks
@@ -100,6 +107,12 @@ class RouteConfigTest extends \PHPUnit_Framework_TestCase
                $config->register();
        }
 
+       /**
+        * @param string $resourceName
+        * @param string $url
+        * @param string $controllerName
+        * @param string $paramName
+        */
        private function assertResource($yaml, $resourceName, $url, $controllerName, $paramName)
        {
                // router mock
index e8a4353800495927644424c7102f4764e56fc26b..11e9fcdf4fa9a5b4505a7446046420c5abe32b98 100644 (file)
@@ -111,10 +111,16 @@ class Test_DBSchema extends PHPUnit_Framework_TestCase {
                }
        }
 
+       /**
+        * @param string $table
+        */
        public function assertTableExist($table) {
                $this->assertTrue($this->tableExist($table), 'Table ' . $table . ' does not exist');
        }
 
+       /**
+        * @param string $table
+        */
        public function assertTableNotExist($table) {
                $type=OC_Config::getValue( "dbtype", "sqlite" );
                if( $type == 'sqlite' || $type == 'sqlite3' ) {
index 68b87deccb688dfd45ee53bfbb841b41312aaf71..32396eafbeacbded7808049f93ee3a2106e1e4eb 100644 (file)
@@ -56,6 +56,10 @@ class Test_ErrorHandler extends \PHPUnit_Framework_TestCase {
  * @brief dummy class to access protected methods of \OC\Log\ErrorHandler
  */
 class TestableErrorHandler extends \OC\Log\ErrorHandler {
+
+       /**
+        * @param string $msg
+        */
        public static function testRemovePassword($msg) {
                return self::removePassword($msg);
        }
index 87fd0dba4c6466e383731b02d508dd5927b7620f..dbcf6e9caa080a9bd4c977ebdbb0e0e69ad8154a 100644 (file)
@@ -19,6 +19,10 @@ class DummyUser extends \OC\User\User {
         */
        private $uid;
 
+       /**
+        * @param string $uid
+        * @param string $home
+        */
        public function __construct($uid, $home) {
                $this->home = $home;
                $this->uid = $uid;
index 6c41413c4df27a6f9b60bb42f8750c1b157a39b6..ce05adb188a1b6552a7a8f814c6cd52b48b8a1bd 100644 (file)
@@ -68,6 +68,9 @@ class EtagTest extends \PHPUnit_Framework_TestCase {
                $this->assertEquals($originalEtags, $this->getEtags($files));
        }
 
+       /**
+        * @param string[] $files
+        */
        private function getEtags($files) {
                $etags = array();
                foreach ($files as $file) {
index 14e1d05853d9c133d532d8700a14373dcd43a0e2..319f2f9f5f7e60b8720e0e15805ffdd99daa7de6 100644 (file)
@@ -9,10 +9,7 @@
 namespace Test\Files\Node;
 
 use OC\Files\Cache\Cache;
-use OC\Files\Mount\Manager;
 use OC\Files\Node\Root;
-use OCP\Files\NotFoundException;
-use OCP\Files\NotPermittedException;
 use OC\Files\Storage\Temporary;
 use OC\Files\View;
 use OC\User\User;
index 97eaf7f716267fc812fa7267d5a0844938ea2047..27f1a937826082f19e80fc96e4deb6118b00e919 100644 (file)
@@ -8,7 +8,6 @@
 
 namespace Test\Files\Node;
 
-use OC\Files\Cache\Cache;
 use OCP\Files\NotPermittedException;
 use OC\Files\Mount\Manager;
 
index 885291e440403d1f422ff3fcadb4b0b6a3cddd83..51315a2a5561a39e021bd101db855a407decb9ca 100644 (file)
@@ -29,6 +29,10 @@ class DummyUser extends User {
 
        private $uid;
 
+       /**
+        * @param string $uid
+        * @param string $home
+        */
        public function __construct($uid, $home) {
                $this->uid = $uid;
                $this->home = $home;
index 87bafb64d41e9f1c2784ea1d72c79e8a6b4651c4..e1b880255fb5c7b643489a5a1d677015de5aceb0 100644 (file)
@@ -27,6 +27,9 @@ class Quota extends \Test\Files\Storage\Storage {
                \OC_Helper::rmdirr($this->tmpDir);
        }
 
+       /**
+        * @param integer $limit
+        */
        protected function getLimitedStorage($limit) {
                $storage = new \OC\Files\Storage\Local(array('datadir' => $this->tmpDir));
                $storage->getScanner()->scan('');
index b11f0ac74c0b4f3f2295f66585cd87d95a8f639b..d5edace544da0ecfdb88c3f42a43607e2b4a618a 100644 (file)
@@ -13,6 +13,10 @@ class Quota extends \PHPUnit_Framework_TestCase {
                \OC\Files\Stream\Quota::clear();
        }
 
+       /**
+        * @param string $mode
+        * @param integer $limit
+        */
        protected function getStream($mode, $limit) {
                $source = fopen('php://temp', $mode);
                return \OC\Files\Stream\Quota::wrap($source, $limit);
index d308232a78b5ed5a991d7091ddf28169c8357cbd..2c563ae9ac9b1439cbf38bbb214ac00bf75e37b9 100644 (file)
@@ -29,7 +29,7 @@ abstract class Test_Group_Backend extends PHPUnit_Framework_TestCase {
        /**
         * get a new unique group name
         * test cases can override this in order to clean up created groups
-        * @return array
+        * @return string
         */
        public function getGroupName() {
                return uniqid('test_');
@@ -38,7 +38,7 @@ abstract class Test_Group_Backend extends PHPUnit_Framework_TestCase {
        /**
         * get a new unique user name
         * test cases can override this in order to clean up created user
-        * @return array
+        * @return string
         */
        public function getUserName() {
                return uniqid('test_');
index 5278c26f4dfdb7d053297780ac75e0c4c0f0c23f..3e05c656061c9c55953e80da28ba5203483e44d1 100644 (file)
@@ -26,7 +26,7 @@ class Test_Group_Database extends Test_Group_Backend {
        /**
         * get a new unique group name
         * test cases can override this in order to clean up created groups
-        * @return array
+        * @return string
         */
        public function getGroupName() {
                $name=uniqid('test_');
@@ -37,7 +37,7 @@ class Test_Group_Database extends Test_Group_Backend {
        /**
         * get a new unique user name
         * test cases can override this in order to clean up created user
-        * @return array
+        * @return string
         */
        public function getUserName() {
                return uniqid('test_');
index f1fda3b928836d1c0bac790cc404f2de1746f97d..3982c00e45f48be70f767d0a2c3d5302a19261cc 100644 (file)
@@ -13,7 +13,7 @@ use OC\User\User;
 
 class Group extends \PHPUnit_Framework_TestCase {
        /**
-        * @return \PHPUnit_Framework_MockObject_MockObject | \OC\User\Manager
+        * @return \OC\User\Manager | \OC\User\Manager
         */
        protected function getUserManager() {
                $userManager = $this->getMock('\OC\User\Manager');
index 39a9bfc8d5a5fecc726fc13579957f974cf67e95..d438a7a692e8caa42d8fc61da027fa6bcba3141a 100644 (file)
@@ -31,6 +31,7 @@ class Test_Migrate extends PHPUnit_Framework_TestCase {
         * @brief checks for existence of export_info.json and file folder
         * @param string $exportedUser the user that was exported
         * @param string $path the path to the .zip export
+        * @param string $exportedBy
         */
        public function validateUserExport($exportedBy, $exportedUser, $path) {
                $this->assertTrue(file_exists($path));
index ea8413734f17ce916ba906b7e16dafa331af0113..498ab718621acc019a4128005d43f033df3bfccd 100644 (file)
@@ -131,6 +131,7 @@ class Test_OC_OCS_Privatedata extends PHPUnit_Framework_TestCase
 
        /**
         * @param \OC_OCS_Result $result
+        * @param integer $expectedArraySize
         */
        public function assertOcsResult($expectedArraySize, $result) {
                $this->assertEquals(100, $result->getStatusCode());
index d6acee6c92451de4d5a19c800750775f93affe26..a89f100d97a0a19dba7906eb817cc76eedf23695 100644 (file)
@@ -151,6 +151,10 @@ class Test_Share extends PHPUnit_Framework_TestCase {
                );
        }
 
+       /**
+        * @param string $sharer
+        * @param string $receiver
+        */
        protected function shareUserTestFileWithUser($sharer, $receiver) {
                OC_User::setUserId($sharer);
                $this->assertTrue(
@@ -558,6 +562,9 @@ class Test_Share extends PHPUnit_Framework_TestCase {
                );
        }
 
+       /**
+        * @param boolean|string $token
+        */
        protected function getShareByValidToken($token) {
                $row = OCP\Share::getShareByToken($token);
                $this->assertInternalType(
index d80d222e2c96039d0a084858f5fe4a4bbab23244..619560643fea04abdcbe6c9b6b1b3f79e12a80d4 100644 (file)
@@ -7,6 +7,11 @@
  */
 
 class Test_ResourceLocator extends PHPUnit_Framework_TestCase {
+
+       /**
+        * @param string $theme
+        * @param string $form_factor
+        */
        public function getResourceLocator( $theme, $form_factor, $core_map, $party_map, $appsroots ) {
                return $this->getMockForAbstractClass('OC\Template\ResourceLocator',
                        array( $theme, $form_factor, $core_map, $party_map, $appsroots ),
index fdf9e7a08e0ac77e857b26368c773cec186eac42..e2c3282a19f4181e68ff5f92d8ac9969f34891be 100644 (file)
@@ -9,8 +9,6 @@
 
 namespace Test;
 
-use OC\Hooks\PublicEmitter;
-
 class User extends \PHPUnit_Framework_TestCase {
        /**
         * @var \OC_User_Backend | \PHPUnit_Framework_MockObject_MockObject $backend
index a969ece6dd579e3bd8079853459e95865992cb44..299a589ef4ecff89e8f066bea836ce9e4a7aa6e3 100644 (file)
@@ -57,6 +57,9 @@ class TestCleanupListener implements PHPUnit_Framework_TestListener {
                return $this->verbosity === 'detail';
        }
 
+       /**
+        * @param string $dir
+        */
        private function unlinkDir($dir) {
                if ($dh = @opendir($dir)) {
                        while (($file = readdir($dh)) !== false) {