Merge pull request #37943 from nextcloud/fix/fix-getcontent-return-type

Fix file_get_content signatures to make it clear it can return false

3 files changed, 3 insertions, 2 deletions

diff --git a/lib/public/Files/File.php b/lib/public/Files/File.php
index 93d0ab860c1..7c1e36cb3eb 100644
--- a/lib/public/Files/File.php
+++ b/lib/public/Files/File.php
@@ -40,6 +40,7 @@ interface File extends Node {
 	 *
 	 * @return string
 	 * @throws NotPermittedException
+	 * @throws GenericFileException
 	 * @throws LockedException
 	 * @since 6.0.0
 	 */
diff --git a/lib/public/Files/Storage.php b/lib/public/Files/Storage.php
index c165266a1b7..31405a26bfa 100644
--- a/lib/public/Files/Storage.php
+++ b/lib/public/Files/Storage.php
@@ -217,7 +217,7 @@ interface Storage extends IStorage {
 	 * see https://www.php.net/manual/en/function.file_get_contents.php
 	 *
 	 * @param string $path
-	 * @return string|bool
+	 * @return string|false
 	 * @since 6.0.0
 	 */
 	public function file_get_contents($path);
diff --git a/lib/public/Files/Storage/IStorage.php b/lib/public/Files/Storage/IStorage.php
index 2f38165830b..00e98fdfbb6 100644
--- a/lib/public/Files/Storage/IStorage.php
+++ b/lib/public/Files/Storage/IStorage.php
@@ -214,7 +214,7 @@ interface IStorage {
 	 * see https://www.php.net/manual/en/function.file_get_contents.php
 	 *
 	 * @param string $path
-	 * @return string|bool
+	 * @return string|false
 	 * @since 9.0.0
 	 */
 	public function file_get_contents($path);