Create backup before editing a config file

README.md

@@ -67,6 +67,7 @@ In version 0.6:
     - [x] Log also successful messages
     - [x] User Model: Delete user, develop better way to check if user is in use
     - [x] Support for live resizing of targets (with workaround, since iet doesn't support)
+    - [x] Create backup before editing a config file
     - [ ] Rework controller/models
         - [x] Targets
         - [x] Users
@@ -82,7 +83,6 @@ In version 0.6:
         - [ ] Database model
         - [ ] phpietadmin user
     - [ ] Database error log
-    - [ ] Create backup before editing a config file
     - [ ] Create first user via gui
 
 * Misc

app/models/Std.php

@@ -283,8 +283,8 @@ public function tail($filename, $lines = 10, $buffer = 4096) {
 	 * @return bool
 	 */
 	public function backupFile($path, $type = 'file') {
-		$backupDir = $this->database->get_config('backupDir');
-		$maxBackups = $this->database->get_config('maxBackups');
+		$backupDir = $this->database->get_config('backupDir')['value'];
+		$maxBackups = $this->database->get_config('maxBackups')['value'];
 		$backupDirFiles = $backupDir . '/files';
 		$backupDirDb = $backupDir . '/db';
 

app/models/target/Parser.php

@@ -153,12 +153,16 @@ public function parse_file($filename, callable $callback, array $params, $return
             $data .= "\n";
 
             if ($test === false) {
-                // Write back
-                if (file_put_contents($filename, $data, LOCK_EX) !== false) {
-                    return 0;
-                } else {
-                    return 6;
-                }
+				if($this->std->backupFile($filename, 'file') === true) {
+					// Write back
+					if (file_put_contents($filename, $data, LOCK_EX) !== false) {
+						return 0;
+					} else {
+						return 6;
+					}
+				} else {
+					return 12;
+				}
             } else {
                 echo htmlspecialchars($data);
                 return 0;

app/notes.txt

@@ -11,6 +11,7 @@
     9 wrong usage
     10 passwords do not match
     11 function not available
+    12 backup unsuccessful
 
 
     http://manpages.ubuntu.com/manpages/trusty/man5/ietd.conf.5.html