Check if .htaccess is writable on install/update

symphonycms · Mar 24, 2016 · bfea95b · bfea95b
1 parent 52f3030
commit bfea95b
Showing 1 changed file with 7 additions and 0 deletions.
diff --git a/extension.driver.php b/extension.driver.php
@@ -62,6 +62,9 @@ public function install()
         $htaccess = new HTAccess();
         try {
             if ($htaccess->exists()) {
+                if (!$htaccess->is_writable()) {
+                    throw new Exception(__('.htaccess exists but is not writable.'));
+                }
                 $htaccess->enableExtension();
             }
             // Create workspace directory
@@ -93,6 +96,10 @@ public function update($previousVersion = false)
         require_once 'lib/class.htaccess.php';
         $htaccess = new HTAccess();
 
+        if ($htaccess->exists() && !$htaccess->is_writable()) {
+            throw new Exception(__('.htaccess exists but is not writable.'));
+        }
+
         if (version_compare($previousVersion, '1.15', '<')) {
             // Move /manifest/jit-trusted-sites into /workspace/jit-image-manipulation
             if (General::realiseDirectory(WORKSPACE . '/jit-image-manipulation', Symphony::Configuration()->get('write_mode', 'directory')) && file_exists(MANIFEST . '/jit-trusted-sites')) {