Fix bug in parser

Author: mrcrankhank

README.md

@@ -21,7 +21,7 @@ Will be added once version 0.6 is released
 Take a look at the github releases for detailed information about the features.
 
 ## Bugs in 0.5.6:
-    - [ ] Parse function could overwrite comments
+    - [x] Parse function could overwrite comments
     - [x] Fix phpietadmin login user menu
     - [x] The same object can be added multiple times to a iqn
 

app/models/target/Parser.php

@@ -123,9 +123,20 @@ public function parse_file($filename, callable $callback, array $params, $return
                     // so we just add a index to the key array
                     // multiple keys are only handled in delete functions
                     // there is no function which adds multiple lines
-                    $file_keys[] = max($file_keys) + 1;
+                    if (count($file_keys) === 0) {
+                        if (isset($comments) && is_array($comments)) {
+                            $file_keys[] = count($comments) + 1;
+                        } else {
+                            $file_keys[] =  1;
+                        }
+                    } else {
+                        if (isset($comments) && is_array($comments)) {
+                            $file_keys[] = $callback_data_count + count($comments) + 1;
+                        } else {
+                            $file_keys[] = $callback_data_count + 1;
+                        }
+                    }
                 }
-
                 $file = array_combine($file_keys, $callback_data['file']);
             }