Merge pull request #10518 from owncloud/fix_storage_const_autoload

Fix storage const autoload

apps/files_external/lib/google.php

@@ -286,7 +286,7 @@ class Google extends \OC\Files\Storage\Common {
 				// Check if this is a Google Doc
 				if ($this->getMimeType($path) !== $file->getMimeType()) {
 					// Return unknown file size
-					$stat['size'] = \OC\Files\SPACE_UNKNOWN;
+					$stat['size'] = \OCP\Files\FileInfo::SPACE_UNKNOWN;
 				} else {
 					$stat['size'] = $file->getFileSize();
 				}

apps/files_sharing/lib/sharedstorage.php

@@ -377,7 +377,7 @@ class Shared extends \OC\Files\Storage\Common implements ISharedStorage {
 			list($storage, $internalPath) = \OC\Files\Filesystem::resolvePath($source);
 			return $storage->free_space($internalPath);
 		}
-		return \OC\Files\SPACE_UNKNOWN;
+		return \OCP\Files\FileInfo::SPACE_UNKNOWN;
 	}
 
 	public function getLocalFile($path) {

lib/private/connector/sabre/quotaplugin.php

@@ -72,7 +72,7 @@ class OC_Connector_Sabre_QuotaPlugin extends \Sabre\DAV\ServerPlugin {
 				$length -= $chunkHandler->getCurrentSize();
 			}
 			$freeSpace = $this->getFreeSpace($parentUri);
-			if ($freeSpace !== \OC\Files\SPACE_UNKNOWN && $length > $freeSpace) {
+			if ($freeSpace !== \OCP\Files\FileInfo::SPACE_UNKNOWN && $length > $freeSpace) {
 				if (isset($chunkHandler)) {
 					$chunkHandler->cleanup();
 				}

lib/private/files/filesystem.php

@@ -31,11 +31,9 @@
 namespace OC\Files;
 
 use OC\Files\Storage\Loader;
-const SPACE_NOT_COMPUTED = -1;
-const SPACE_UNKNOWN = -2;
-const SPACE_UNLIMITED = -3;
 
 class Filesystem {
+
 	/**
 	 * @var Mount\Manager $mounts
 	 */

lib/private/files/storage/common.php

@@ -398,7 +398,7 @@ abstract class Common implements \OC\Files\Storage\Storage {
 	 * @return int
 	 */
 	public function free_space($path) {
-		return \OC\Files\SPACE_UNKNOWN;
+		return \OCP\Files\FileInfo::SPACE_UNKNOWN;
 	}
 
 	/**

lib/private/files/storage/dav.php

@@ -243,10 +243,10 @@ class DAV extends \OC\Files\Storage\Common {
 			if (isset($response['{DAV:}quota-available-bytes'])) {
 				return (int)$response['{DAV:}quota-available-bytes'];
 			} else {
-				return \OC\Files\SPACE_UNKNOWN;
+				return \OCP\Files\FileInfo::SPACE_UNKNOWN;
 			}
 		} catch (\Exception $e) {
-			return \OC\Files\SPACE_UNKNOWN;
+			return \OCP\Files\FileInfo::SPACE_UNKNOWN;
 		}
 	}
 

lib/private/files/storage/local.php

@@ -218,7 +218,7 @@ if (\OC_Util::runningOnWindows()) {
 		public function free_space($path) {
 			$space = @disk_free_space($this->datadir . $path);
 			if ($space === false || is_null($space)) {
-				return \OC\Files\SPACE_UNKNOWN;
+				return \OCP\Files\FileInfo::SPACE_UNKNOWN;
 			}
 			return $space;
 		}

lib/private/files/storage/wrapper/quota.php

@@ -50,7 +50,7 @@ class Quota extends Wrapper {
 			}
 			return $data['size'];
 		} else {
-			return \OC\Files\SPACE_NOT_COMPUTED;
+			return \OCP\Files\FileInfo::SPACE_NOT_COMPUTED;
 		}
 	}
 
@@ -66,7 +66,7 @@ class Quota extends Wrapper {
 		} else {
 			$used = $this->getSize($this->sizeRoot);
 			if ($used < 0) {
-				return \OC\Files\SPACE_NOT_COMPUTED;
+				return \OCP\Files\FileInfo::SPACE_NOT_COMPUTED;
 			} else {
 				$free = $this->storage->free_space($path);
 				$quotaFree = max($this->quota - $used, 0);

lib/private/helper.php

@@ -887,7 +887,7 @@ class OC_Helper {
 	 */
 	public static function freeSpace($dir) {
 		$freeSpace = \OC\Files\Filesystem::free_space($dir);
-		if ($freeSpace !== \OC\Files\SPACE_UNKNOWN) {
+		if ($freeSpace !== \OCP\Files\FileInfo::SPACE_UNKNOWN) {
 			$freeSpace = max($freeSpace, 0);
 			return $freeSpace;
 		} else {
@@ -960,7 +960,7 @@ class OC_Helper {
 		}
 		if ($includeExtStorage) {
 			$quota = OC_Util::getUserQuota(\OCP\User::getUser());
-			if ($quota !== \OC\Files\SPACE_UNLIMITED) {
+			if ($quota !== \OCP\Files\FileInfo::SPACE_UNLIMITED) {
 				// always get free space / total space from root + mount points
 				$path = '';
 				return self::getGlobalStorageInfo();

lib/private/util.php

@@ -105,7 +105,7 @@ class OC_Util {
 					if (is_object($storage->getUser())) {
 						$user = $storage->getUser()->getUID();
 						$quota = OC_Util::getUserQuota($user);
-						if ($quota !== \OC\Files\SPACE_UNLIMITED) {
+						if ($quota !== \OCP\Files\FileInfo::SPACE_UNLIMITED) {
 							return new \OC\Files\Storage\Wrapper\Quota(array('storage' => $storage, 'quota' => $quota, 'root' => 'files'));
 						}
 					}
@@ -197,7 +197,7 @@ class OC_Util {
 			$userQuota = $config->getAppValue('files', 'default_quota', 'none');
 		}
 		if($userQuota === 'none') {
-			return \OC\Files\SPACE_UNLIMITED;
+			return \OCP\Files\FileInfo::SPACE_UNLIMITED;
 		}else{
 			return OC_Helper::computerFileSize($userQuota);
 		}
@@ -1244,7 +1244,7 @@ class OC_Util {
 		if (strpos($url, 'http://') !== 0 && strpos($url, 'https://') !== 0) {
 			throw new Exception('$url must start with https:// or http://', 1);
 		}
-		
+
 		if (function_exists('curl_init')) {
 			$curl = curl_init();
 			$max_redirects = 10;

lib/public/files/fileinfo.php

@@ -11,6 +11,19 @@ interface FileInfo {
 	const TYPE_FILE = 'file';
 	const TYPE_FOLDER = 'dir';
 
+	/*
+	 * @const \OCP\Files\FileInfo::SPACE_NOT_COMPUTED Return value for a not computed space value
+	 */
+	const SPACE_NOT_COMPUTED = -1;
+	/*
+	 * @const \OCP\Files\FileInfo::SPACE_UNKNOWN Return value for unknown space value
+	 */
+	const SPACE_UNKNOWN = -2;
+	/*
+	 * @const \OCP\Files\FileInfo::SPACE_UNKNOWN Return value for unlimited space
+	 */
+	const SPACE_UNLIMITED = -3;
+
 	/**
 	 * Get the Etag of the file or folder
 	 *

tests/lib/connector/sabre/quotaplugin.php

@@ -63,7 +63,7 @@ class Test_OC_Connector_Sabre_QuotaPlugin extends PHPUnit_Framework_TestCase {
 			array(1024, array('HTTP_X_EXPECTED_ENTITY_LENGTH' => '1024')),
 			array(1024, array('HTTP_CONTENT_LENGTH' => '512')),
 			array(1024, array('HTTP_OC_TOTAL_LENGTH' => '1024', 'HTTP_CONTENT_LENGTH' => '512')),
-			// OC\Files\FREE_SPACE_UNKNOWN = -2
+			// \OCP\Files\FileInfo::SPACE_UNKNOWN = -2
 			array(-2, array()),
 			array(-2, array('HTTP_X_EXPECTED_ENTITY_LENGTH' => '1024')),
 			array(-2, array('HTTP_CONTENT_LENGTH' => '512')),