Add warning message.

chamilo · Dec 7, 2017 · cbf3716 · cbf3716
1 parent 975944f
commit cbf3716
Show file tree

Hide file tree

Showing 2 changed files with 22 additions and 12 deletions.
diff --git a/plugin/nosearchindex/lang/english.php b/plugin/nosearchindex/lang/english.php
@@ -5,5 +5,4 @@
 $strings['tool_enable'] = 'Enable plugin';
 
 $strings['CheckTheWritingPermissionsOfRobotsFile'] = 'The plugin needs to create/edit some files: /robots.txt, /robots.dist.txt, /app/home/header_extra_content.txt check your file permissions.';
-
-
+$strings['CheckDirectoryPermissionsInX'] = 'Check write and read permissions in the path %s ';
diff --git a/plugin/nosearchindex/plugin.php b/plugin/nosearchindex/plugin.php
@@ -13,22 +13,33 @@
 $editFile = false;
 
 $file = api_get_path(SYS_PATH).'robots.txt';
+$originalFile = api_get_path(SYS_PATH).'robots.dist.txt';
+$extraContentFile = api_get_home_path().'header_extra_content.txt';
 
 if ($isPlatformAdmin) {
-    $originalFile = api_get_path(SYS_PATH).'robots.dist.txt';
-    $extraContentFile = api_get_home_path().'header_extra_content.txt';
-    if (!file_exists($originalFile)) {
-        copy($file, $originalFile);
-    }
-    if (!file_exists($extraContentFile)) {
-        file_put_contents($extraContentFile, '');
-    }
-
-    $originalContent = file_get_contents($originalFile);
     /** @var FormValidator $form */
     $form = $plugin_info['settings_form'];
 
     if ($form && $form->validate()) {
+        if (is_writable(api_get_path(SYS_PATH))) {
+            if (!file_exists($originalFile)) {
+                copy($file, $originalFile);
+            }
+        } else {
+            Display::addFlash(
+                Display::return_message(
+                    sprintf(
+                        $plugin->get_lang('CheckDirectoryPermissionsInX'),
+                        api_get_path(SYS_PATH)
+                    )
+                )
+            );
+        }
+
+        if (!file_exists($extraContentFile) && is_writable($extraContentFile)) {
+            file_put_contents($extraContentFile, '');
+        }
+
         $values = $form->getSubmitValues();
         $continue = false;
         if (is_readable($file) && is_writable($file) &&