switch from editor_path to data_path, implement RuleManager for readi…

…ng and writing rule files

src/backend/rule_manager.py

@@ -1,8 +1,10 @@
-from typing import List, Dict
+from pathlib import Path
+from typing import List, Dict, Optional
 
 from src.backend.tile_connection import TileConnection
 from src.backend.tile_handler import TileHandler
 from src.backend.tile_status import TileStatus
+from src.config.config_manager import ConfigManager
 from src.globals import NUM_TILES
 from src.widgets.widget_base_tile import BaseTile
 
@@ -17,14 +19,24 @@ def __init__(self):
 
     @staticmethod
     def _loadRuleFile(filename_base) -> RuleConfig:
-        # filename = f"{filename_base}.rules"
-        # TODO check if file exists, if exists, load
-        # StorageFinder.findRule(filename)
-        # if exists
-        # config = ConfigParser()
-        # config.read(filename)
-        # else
-        config = {filename_base: []}
+        filename = f"{filename_base}.rules"
+        data_path = Path(ConfigManager.config()["data_path"])
+        if not data_path:
+            raise ValueError("No editor directory path known")
+
+        automapper_path = data_path.joinpath(Path("editor/automap"))
+        if not automapper_path.exists():
+            automapper_path.mkdir()
+
+        full_file_path = automapper_path.joinpath(Path(filename))
+
+        # load file if it exists
+        if full_file_path.is_file():
+            config = RuleManager._readRuleFile(full_file_path)
+
+        # file doesn't exist
+        else:
+            config = {filename_base: []}
         return config
 
     @staticmethod
@@ -94,8 +106,8 @@ def _createIndexRule(tile_id: int, tile_status: TileStatus):
         str_rotate = " ROTATE" if rotate else ""
         return f"{str_index}{str_x_flip}{str_y_flip}{str_rotate}"
 
-    @classmethod
-    def _writeRuleFile(cls, filename_base, config: RuleConfig):
+    @staticmethod
+    def _writeRuleFile(filename_base, config: RuleConfig):
         filename = f"{filename_base}.rules"
         with open(filename, 'w') as f:
             for key in config.keys():
@@ -104,3 +116,27 @@ def _writeRuleFile(cls, filename_base, config: RuleConfig):
                 for line in lines:
                     f.write(line)
                     f.write('\n')
+
+    @staticmethod
+    def _readRuleFile(full_file_path: Path) -> RuleConfig:
+        config = {}
+
+        with open(str(full_file_path), 'r', encoding='utf-8') as f:
+            section: Optional[str] = None
+
+            # read file line by line
+            while line := f.readline():
+                line = line.rstrip()
+
+                # handle sections
+                if len(line) >= 2 and line[0] == '[' and line[-1] == ']':
+                    section = line[1:-1]
+                    config[section] = []
+
+                # handle normal lines
+                else:
+                    if section:
+                        config[section].append(line)
+                    elif len(line) > 0:  # ignore empty lines
+                        raise ValueError("files may contain rules without section, aborting")
+        return config

src/config/config_manager.py

@@ -25,11 +25,11 @@ def _init(self):
 
     def createConfig(self):
         client_path = StorageFinder.instance().getClientPath()
-        editor_path = StorageFinder.instance().getEditorPath()
+        data_path = StorageFinder.instance().getDataPath()
         data = {
             'version': "1.0.0",
             'client_path': str(client_path),
-            'editor_path': str(editor_path),
+            'data_path': str(data_path),
         }
         self._config = data
 

src/config/storage_finder.py

@@ -31,7 +31,7 @@ def getClientPath(self):
         pass
 
     @abstractmethod
-    def getEditorPath(self):
+    def getDataPath(self):
         pass
 
 
@@ -102,10 +102,10 @@ def getClientPath(self):
                 pass
         return None
 
-    def getEditorPath(self):
+    def getDataPath(self):
         if self.data_path:
             try:
-                p = Path(self.data_path).joinpath("editor")
+                p = Path(self.data_path)
                 if p.is_dir():
                     return p
             except OSError:

src/dialogs/dialog_config_settings.py

@@ -17,7 +17,7 @@ def __init__(self, *args):
 
         configs = [
             ("client_path", "Client path:", QFileDialog.FileMode.ExistingFile),
-            ("editor_path", "Editor directory path:", QFileDialog.FileMode.Directory)
+            ("data_path", "Data directory path:", QFileDialog.FileMode.Directory)
         ]
 
         index = 0

src/widgets/widget_mapper_generator.py

@@ -116,9 +116,9 @@ def startDDNetCheck(self):
         # automap map with debroijn torus
         # open map with ddnet
         map_name = ""
-        editor_path = ConfigManager.instance().config()["client_path"]
-        if not editor_path:
+        client_path = ConfigManager.instance().config()["client_path"]
+        if not client_path:
             self.ddnet_push_button.setDisabled(True)
             return
-        cmd = [editor_path, map_name]
+        cmd = [client_path, map_name]
         subprocess.Popen(cmd, start_new_session=True)