Merge pull request #22149 from owncloud/scrutinizer-patch-1
Scrutinizer Auto-Fixes
This commit is contained in:
commit
a92a480967
|
@ -49,7 +49,6 @@ class MDB2SchemaManager {
|
||||||
/**
|
/**
|
||||||
* saves database scheme to xml file
|
* saves database scheme to xml file
|
||||||
* @param string $file name of file
|
* @param string $file name of file
|
||||||
* @param int|string $mode
|
|
||||||
* @return bool
|
* @return bool
|
||||||
*
|
*
|
||||||
* TODO: write more documentation
|
* TODO: write more documentation
|
||||||
|
@ -123,7 +122,7 @@ class MDB2SchemaManager {
|
||||||
/**
|
/**
|
||||||
* update the database scheme
|
* update the database scheme
|
||||||
* @param string $file file to read structure from
|
* @param string $file file to read structure from
|
||||||
* @return string|boolean
|
* @return boolean
|
||||||
*/
|
*/
|
||||||
public function simulateUpdateDbFromStructure($file) {
|
public function simulateUpdateDbFromStructure($file) {
|
||||||
$toSchema = $this->readSchemaFromFile($file);
|
$toSchema = $this->readSchemaFromFile($file);
|
||||||
|
|
|
@ -53,7 +53,7 @@ class QueryLogger implements IQueryLogger {
|
||||||
}
|
}
|
||||||
|
|
||||||
/**
|
/**
|
||||||
* @return \OCP\Diagnostics\IQuery[]
|
* @return Query[]
|
||||||
*/
|
*/
|
||||||
public function getQueries() {
|
public function getQueries() {
|
||||||
return $this->queries;
|
return $this->queries;
|
||||||
|
|
|
@ -160,6 +160,8 @@ class OC_Files {
|
||||||
/**
|
/**
|
||||||
* @param View $view
|
* @param View $view
|
||||||
* @param string $name
|
* @param string $name
|
||||||
|
* @param string $dir
|
||||||
|
* @param boolean $onlyHeader
|
||||||
*/
|
*/
|
||||||
private static function getSingleFile($view, $dir, $name, $onlyHeader) {
|
private static function getSingleFile($view, $dir, $name, $onlyHeader) {
|
||||||
$filename = $dir . '/' . $name;
|
$filename = $dir . '/' . $name;
|
||||||
|
@ -185,7 +187,7 @@ class OC_Files {
|
||||||
|
|
||||||
/**
|
/**
|
||||||
* @param View $view
|
* @param View $view
|
||||||
* @param $dir
|
* @param string $dir
|
||||||
* @param string[]|string $files
|
* @param string[]|string $files
|
||||||
*/
|
*/
|
||||||
public static function lockFiles($view, $dir, $files) {
|
public static function lockFiles($view, $dir, $files) {
|
||||||
|
@ -290,11 +292,11 @@ class OC_Files {
|
||||||
}
|
}
|
||||||
|
|
||||||
/**
|
/**
|
||||||
* @param $dir
|
* @param string $dir
|
||||||
* @param $files
|
* @param $files
|
||||||
* @param $getType
|
* @param integer $getType
|
||||||
* @param View $view
|
* @param View $view
|
||||||
* @param $filename
|
* @param string $filename
|
||||||
*/
|
*/
|
||||||
private static function unlockAllTheFiles($dir, $files, $getType, $view, $filename) {
|
private static function unlockAllTheFiles($dir, $files, $getType, $view, $filename) {
|
||||||
if ($getType === self::FILE) {
|
if ($getType === self::FILE) {
|
||||||
|
|
|
@ -169,7 +169,7 @@ class Root extends Folder implements IRootFolder {
|
||||||
* @param string $path
|
* @param string $path
|
||||||
* @throws \OCP\Files\NotFoundException
|
* @throws \OCP\Files\NotFoundException
|
||||||
* @throws \OCP\Files\NotPermittedException
|
* @throws \OCP\Files\NotPermittedException
|
||||||
* @return \OCP\Files\Node
|
* @return string
|
||||||
*/
|
*/
|
||||||
public function get($path) {
|
public function get($path) {
|
||||||
$path = $this->normalizePath($path);
|
$path = $this->normalizePath($path);
|
||||||
|
|
|
@ -396,7 +396,7 @@ abstract class Common implements Storage, ILockingStorage {
|
||||||
* get the ETag for a file or folder
|
* get the ETag for a file or folder
|
||||||
*
|
*
|
||||||
* @param string $path
|
* @param string $path
|
||||||
* @return string|false
|
* @return string
|
||||||
*/
|
*/
|
||||||
public function getETag($path) {
|
public function getETag($path) {
|
||||||
return uniqid();
|
return uniqid();
|
||||||
|
|
|
@ -58,6 +58,7 @@ class Dir {
|
||||||
|
|
||||||
/**
|
/**
|
||||||
* @param string $path
|
* @param string $path
|
||||||
|
* @param string[] $content
|
||||||
*/
|
*/
|
||||||
public static function register($path, $content) {
|
public static function register($path, $content) {
|
||||||
self::$dirs[$path] = $content;
|
self::$dirs[$path] = $content;
|
||||||
|
|
|
@ -121,7 +121,7 @@ class Detection implements IMimeTypeDetector {
|
||||||
}
|
}
|
||||||
|
|
||||||
/**
|
/**
|
||||||
* @return array
|
* @return string[]
|
||||||
*/
|
*/
|
||||||
public function getAllAliases() {
|
public function getAllAliases() {
|
||||||
$this->loadAliases();
|
$this->loadAliases();
|
||||||
|
@ -264,7 +264,7 @@ class Detection implements IMimeTypeDetector {
|
||||||
|
|
||||||
/**
|
/**
|
||||||
* Get path to the icon of a file type
|
* Get path to the icon of a file type
|
||||||
* @param string $mimeType the MIME type
|
* @param string $mimetype the MIME type
|
||||||
* @return string the url
|
* @return string the url
|
||||||
*/
|
*/
|
||||||
public function mimeTypeIcon($mimetype) {
|
public function mimeTypeIcon($mimetype) {
|
||||||
|
|
Loading…
Reference in New Issue