From: Bjoern Schiessle <schiessle@owncloud.com>
Date: Tue, 5 Jun 2012 09:50:21 +0000 (+0200)
Subject: write a warning to the log file if .htaccess is not writable instead of failing silently
X-Git-Tag: v4.5.0beta1~74^2~424^2~28
X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=061abb88b568c4cdc35266fb8c32d1bfabcb8c8f;p=nextcloud-server.git

write a warning to the log file if .htaccess is not writable instead of failing silently
---

diff --git a/lib/files.php b/lib/files.php
index 885f6977fbd..705b7a6ca66 100644
--- a/lib/files.php
+++ b/lib/files.php
@@ -373,10 +373,12 @@ class OC_Files {
 			}
 		}
 
-		//supress errors in case we don't have permissions for it
-		if(@file_put_contents(OC::$SERVERROOT.'/.htaccess', $htaccess)) {
-			return OC_Helper::computerFileSize($size);
-		}
+		//check for write permissions
+		if(is_writable(OC::$SERVERROOT.'/.htaccess')) {
+			file_put_contents(OC::$SERVERROOT.'/.htaccess', $htaccess);
+			return OC_Helper::computerFileSize($size);	
+		} else { OC_Log::write('files','Can\'t write upload limit to '.OC::$SERVERROOT.'/.htaccess. Please check the file permissions',OC_Log::WARN); }
+		
 		return false;
 	}