recreated json file handling as connector model, fixing #10 and introducing #8 and #7

Reviewed-on: #4

.gitignore

@@ -166,3 +166,7 @@ cython_debug/
 /.idea/modules.xml
 /.idea/inspectionProfiles/profiles_settings.xml
 /.idea/inspectionProfiles/Project_Default.xml
+
+/deb-package/brovski-adressetiketten/usr/local/bin/brovski-adress-etiketten-verwaltung
+/deb-package/brovski-adressetiketten/usr/local/bin/brovski-adressetiketten
+/brovski-adressetiketten.deb

compile.sh

@@ -0,0 +1,20 @@
+#!/usr/bin/env bash
+if [ $VIRTUAL_ENV=="" ]
+then
+  source venv/bin/activate
+fi
+version=$(cat version.txt)
+echo "current version set to: $version"
+new_version=""
+read -r -p "Enter new version or empty to keep the current: " new_version
+if [ "$new_version" != "" ]
+then
+  echo "$new_version" | tee version.txt
+fi
+sed -i "s/VERSION = '[0-9]\.[0-9]\w'/VERSION = '$(cat version.txt)'/g" src/brovski-adress-etiketten-verwaltung.py
+sed -i "s/Version: [0-9]\.[0-9]\w/Version: $(cat version.txt)/g" deb-package/brovski-adressetiketten/DEBIAN/control
+
+pyinstaller --clean --onefile src/brovski-adress-etiketten-verwaltung.py
+cp dist/brovski-adress-etiketten-verwaltung deb-package/brovski-adressetiketten/usr/local/bin/brovski-adressetiketten
+dpkg-deb --build deb-package/brovski-adressetiketten
+mv deb-package/brovski-adressetiketten.deb ./

deb-package/brovski-adressetiketten/DEBIAN/control

@@ -0,0 +1,5 @@
+Package: brovski-adressetiketten
+Version: 0.6a
+Maintainer: Ovski
+Architecture: all
+Description: manage csv files for glables address labels

deb-package/brovski-adressetiketten/usr/share/applications/brovski-adress-etiketten-verwaltung.desktop

@@ -1,8 +1,8 @@
 [Desktop Entry]
 Name=Brovski Adress Etiketten
 GenericName=Brovski Adress Etiketten
-Icon=/home/sroth/app/brovski-adress-etiketten-verwaltung/icon.svg
+Icon=/usr/share/icons/hicolor/512x512/apps/brovski-adressetiketten.svg
-Exec=/home/sroth/app/brovski-adress-etiketten-verwaltung/brovski-adress-etiketten-verwaltung
+Exec=/usr/local/bin/brovski-adressetiketten
 Terminal=false
 Type=Application
 Categories=Utility

src/address.csv

@@ -1,3 +0,0 @@
-alpha muster-firma,otti,tottistrasse,1234 daheim
-c3000,erich roth,zentralstrasse 120,5430 wettingen
-muster-firma,Peter Muster,Peter-Musterstrasse 1,1234 Ort

src/brovski-adress-etiketten-verwaltung.py

@@ -1,271 +1,277 @@
 import csv
 import os
+import sys
 import tkinter as tk
-from configparser import NoOptionError
+from configparser import NoSectionError, NoOptionError
-from configparser import ConfigParser
 from tkinter import messagebox
 from tkinter import ttk
 import json
 
-
+from config import Config
-class Config:
+from connector import JSONConnector
-    parser: ConfigParser
+from windows import SettingsWindow, EditRecord, Window, show_error
-
-    def __init__(self):
-        """
-        Config parser reading config.ini
-
-        Attributes:
-            self.parser: ConfigParser holding list of sections and options
-            self.__filename: Path and name to the config file
-        """
-        self.parser = ConfigParser()
-
-        self.config_file_name = "files/config.ini"
-        self.root_path = os.path.dirname(os.path.abspath(__file__))
-        self.config_file = os.path.join(self.root_path, self.config_file_name)
-
-        self._load()
-
-    def _save(self):
-        with open(self.config_file, 'w') as outfile:
-            self.parser.write(outfile)
-
-    def _load(self):
-        self.parser.read(self.config_file)
-
-    def add_section(self, section):
-        self._load()
-        self.parser.add_section(section)
-        self._save()
-
-    def set(self, section: str, option: str, value: str):
-        self._load()
-        self.parser.set(section, option, value)
-        self._save()
-
-    def get(self, section: str, option: str):
-        self._load()
-        return self.parser.get(section, option)
 
 
 class Application:
     def __init__(self):
-        self.config = Config()
-
-        self.address_list = []
-        self.current_record: int | None = None
-
-        # json vars
-        self.json_file_name = "files/address_data.json"
-        self.root_path = os.path.dirname(os.path.abspath(__file__))
-        self.json_file = os.path.join(self.root_path, self.json_file_name)
-
         # tkinter settings
         x_offset = 700
-        y_offset = 400
+        y_offset = 200
-        title = "Brovski Adress-Etiketten Verwaltung"
+        width = 1050
+        height = 700
+        VERSION = '0.6a'
+        title = f"Brovski Adress-Etiketten Verwaltung {VERSION}"
 
         self.root = tk.Tk(className="BrovskiAdressEtiketten")
         self.root.title(title)
-        self.root.geometry(f"+{x_offset}+{y_offset}")
+        self.root.protocol("WM_DELETE_WINDOW", self.on_close)
+        self.root.geometry(f"{width}x{height}+{x_offset}+{y_offset}")
+        self.config = Config()
+        self.config_good = False
 
+        # variables
+        self.address_list = []
+        self.current_record: int | None = None
+        self.sort_order = False
+        self.last_sort_field = "#3"
+        self.filter_active = tk.BooleanVar(value=False)
+
+        # model connector
+        self.model = JSONConnector()
+
+        # init paths to json and csv file
+        self.json_file_name = "brovski-adress-etiketten-verwaltung.json"
+        self.csv_file_name = "brovski-adress-etiketten.csv"
+        self.csv_path = ""
+        self.load_config()
+        self.csv_file = os.path.join(self.csv_path, self.csv_file_name)
+
+        # status bar content
+        self.statusbar = tk.StringVar()
+        self.length_address_list = None
+        self.length_address_list_active = None
+
+        # leave application if settings are bad
+        if not self.config_good:
+            show_error(message_title="Fehler Konfiguration",
+                       message="Die Konfiguration ist fehlerhaft, bitte prüfe deine config.ini",
+                       parent=self.root
+                       )
+            sys.exit()
+
+        # frames
         top_frame = tk.Frame(self.root)
         top_frame.pack(side=tk.TOP, fill=tk.X)
-        button_width = 8
-        tk.Button(top_frame, text="Export CSV", command=self.export_csv, width=button_width).grid(row=0, column=2)
-        tk.Button(top_frame, text="Insert", command=self.insert_record, width=button_width).grid(row=0, column=0)
-        tk.Button(top_frame, text="Delete", command=self.delete_record, width=button_width).grid(row=0, column=1)
-
-        self.aktiv = tk.StringVar()
-        self.firma = tk.StringVar()
-        self.name = tk.StringVar()
-        self.strasse = tk.StringVar()
-        self.plz_ort = tk.StringVar()
-
-        edit_frame = tk.Frame(self.root)
-        edit_frame.pack(side=tk.TOP, fill=tk.X)
-        tk.Label(edit_frame, text="Aktiv").grid(row=0, column=0)
-        tk.Label(edit_frame, text="Firma").grid(row=0, column=1)
-        tk.Label(edit_frame, text="Name").grid(row=0, column=2)
-        tk.Label(edit_frame, text="Strasse").grid(row=0, column=3)
-        tk.Label(edit_frame, text="Plz/Ort").grid(row=0, column=4)
-        edit_aktiv = tk.Checkbutton(edit_frame, variable=self.aktiv, onvalue="x", offvalue="")
-        edit_aktiv.grid(row=1, column=0)
-        edit_firma = tk.Entry(edit_frame, textvariable=self.firma)
-        edit_firma.grid(row=1, column=1)
-        edit_name = tk.Entry(edit_frame, textvariable=self.name)
-        edit_name.grid(row=1, column=2)
-        edit_strasse = tk.Entry(edit_frame, textvariable=self.strasse)
-        edit_strasse.grid(row=1, column=3)
-        edit_plz_ort = tk.Entry(edit_frame, textvariable=self.plz_ort)
-        edit_plz_ort.grid(row=1, column=4)
-        tk.Button(edit_frame, text="Update", command=self.update_record).grid(row=1, column=5)
-
         data_frame = tk.Frame(self.root, bg="teal")
         data_frame.pack(side=tk.TOP, fill=tk.BOTH, expand=True)
+        bottom_frame = tk.Frame(self.root)
+        bottom_frame.pack(side=tk.BOTTOM, fill=tk.X)
 
-        self.table = ttk.Treeview(data_frame, columns=("0", "1", "2", "3", "4"), show="headings")
+        # top buttons
+        button_width = 8
+        tk.Button(top_frame, text="Insert", command=self.insert_record, width=button_width).pack(side=tk.LEFT)
+        tk.Button(top_frame, text="Delete", command=self.delete_record, width=button_width).pack(side=tk.LEFT)
+        tk.Button(top_frame, text="Export CSV", command=self.export_csv, width=button_width).pack(side=tk.LEFT)
+        tk.Button(top_frame, text="Toggle Aktiv", command=self.toggle_active, width=button_width).pack(side=tk.LEFT)
+        tk.Checkbutton(top_frame, text="Filter aktive", variable=self.filter_active, command=self.populate_table).pack(
+            side=tk.LEFT)
+        tk.Button(top_frame, text="Quit", command=self.on_close, width=button_width).pack(side=tk.RIGHT)
+        tk.Button(top_frame, text="Settings", command=self.show_settings, width=button_width).pack(side=tk.RIGHT)
+
+        # table content
+        scrollbar = ttk.Scrollbar(data_frame, orient=tk.VERTICAL)
+        self.table = ttk.Treeview(data_frame, yscrollcommand=scrollbar.set, columns=("0", "1", "2", "3", "4"),
+                                  show="headings")
+        scrollbar.config(command=self.table.yview)
         self.table.heading('0', text="Aktiv")
+        self.table.column('0', anchor=tk.CENTER, width=0)
         self.table.heading('1', text="Firma")
         self.table.heading('2', text="Name")
         self.table.heading('3', text="Strasse")
         self.table.heading('4', text="Plz/Ort")
-        self.table.pack()
+        self.table.pack(side=tk.LEFT, fill=tk.BOTH, expand=True)
+        scrollbar.pack(side=tk.LEFT, fill=tk.Y)
 
         self.table.bind("<ButtonRelease-1>", self.mouse_click)
+        self.table.bind("<Return>", self.mouse_click_double)
+        self.table.bind("<Double-1>", self.mouse_click_double)
 
-        self._load_file()
+        # bottom status bar
+        tk.Label(bottom_frame, textvariable=self.statusbar).pack(side=tk.LEFT)
+
+        self.first_sort_after_start()
 
         self.root.mainloop()
 
+    def load_config(self):
+        try:
+            self.csv_path = self.config.get("csv", "path")
+            if self.csv_path == "":
+                raise ValueError("Empty JSON or CSV path")
+            self.config_good = True
+        except NoSectionError:
+            self.show_config_error()
+            self.show_settings()
+        except NoOptionError:
+            self.show_config_error()
+            self.show_settings()
+        except ValueError:
+            self.show_config_error()
+            self.show_settings()
+
+    def show_config_error(self):
+        show_error(message_title="Fehlerhafte Konfiguration",
+                   message="Konnte benötigte Parameter in config.ini nicht finden",
+                   parent=self.root)
+
+    def on_close(self):
+        self.root.destroy()
+
+    def show_settings(self):
+        settings = self.open_window_settings()
+        settings.wait_window()
+
     def insert_record(self):
-        if self.current_record is not None:
+        values = {
-            self.clear_entry_fields()
+            "aktiv": "x",
-        children = self.table.get_children()
+            "firma": "Firma",
-        values = [
+            "name": "Name",
-            "x",
+            "strasse": "Strasse",
-            "Firma",
+            "plzort": "Plz/Ort"
-            "Name",
+        }
-            "Strasse",
+        self.model.create_new(values)
-            "Plz/Ort",
+        self.populate_table()
-        ]
+        self.deselect_tree()
-        self.table.insert('', 'end', values=values)
-        self._save_file()
 
     def delete_record(self):
         if self.current_record is None:
             return
 
+        if len(self.table.selection()) > 1:
+            show_error(message_title="Mehrere Adressen ausgewählt",
+                       message="Es können nur einzelne Adressen gelöscht werden",
+                       parent=self.root)
+            return
+
         if messagebox.askyesno(
                 "Eintrag löschen?",
                 "Willst du diesen Eintrag wirklich löschen?\nDies kann nicht rückgängig gemacht werden"):
-            self.table.delete(self.current_record)
+            print(type(self.current_record))
+            self.model.delete_by_id(self.current_record)
 
-        self.clear_entry_fields()
+        self.deselect_tree()
-        self._save_file()
+        self.populate_table()
 
+    def update_record(self, record: dict):
+        if self.current_record is None:
+            return
+        self.model.update_record(record)
+        self.populate_table()
 
-    def clear_entry_fields(self):
+    def toggle_active(self):
-        entry_var_list = [self.aktiv, self.firma, self.name, self.strasse, self.plz_ort]
+        selection = self.table.selection()
-        for entry in entry_var_list:
+        if len(selection) == 0:
-            entry.set("")
+            return
+
+        item_list = [int(x) for x in selection]
+        for address in self.address_list:
+            record_id = address.get("record_id")
+            if record_id in item_list:
+                address["aktiv"] = "x" if address["aktiv"] == "" else ""
+                self.model.update_record(address)
+                self.table.set(record_id, "0", address["aktiv"])
+
+        self.update_status_bar()
+
+    def deselect_tree(self):
+        while len(self.table.selection()) > 0:
+            self.table.selection_remove(self.table.selection()[0])
         self.current_record = None
 
     def mouse_click(self, event):
+        id_string = self.table.focus()
+        if id_string is not None and id_string != "":
+            self.current_record = int(self.table.focus())
+        else:
+            self.current_record = None
         region = self.table.identify("region", event.x, event.y)
+        match region:
+            case "heading":
+                self.click_on_header(event)
 
-        if region == "heading":
+    def mouse_click_double(self, event):
+        region = self.table.identify("region", event.x, event.y)
+        match region:
+            case "cell":
+                self.click_on_cell()
+
+    def click_on_header(self, event):
         column = self.table.identify_column(event.x)
+        print(f"col: {column}")
+        if self.last_sort_field == column:
+            self.sort_order = False if self.sort_order else True
+        else:
+            self.sort_order = False
+            self.last_sort_field = column
+
         match column:
             case "#1":
-                    self.address_list.sort(key=lambda x: x[int(column[-1]) - 1])
+                field = "aktiv"
-                    self.populate_table()
             case "#2":
-                    self.address_list.sort(key=lambda x: x[int(column[-1]) - 1])
+                field = "firma"
-                    self.populate_table()
             case "#3":
-                    self.address_list.sort(key=lambda x: x[int(column[-1]) - 1])
+                field = "name"
-                    self.populate_table()
             case "#4":
-                    self.address_list.sort(key=lambda x: x[int(column[-1]) - 1])
+                field = "strasse"
-                    self.populate_table()
             case "#5":
-                    self.address_list.sort(key=lambda x: x[int(column[-1]) - 1])
+                field = "plzort"
-                    self.populate_table()
             case _:
-                    print(column)
+                field = "name"
 
-        if region == "cell":
+        self.address_list = self.model.get_all_sorted_by(field, self.sort_order)
-            self.current_record = self.table.focus()
+        self.populate_table(reload=False)
-            values = self.table.item(self.current_record, "values")
-            entry_var_list = [self.aktiv, self.firma, self.name, self.strasse, self.plz_ort]
-            for i in range(len(values)):
-                entry_var_list[i].set(values[i])
 
-    def update_record(self):
+    def click_on_cell(self):
-        if self.current_record is None:
+        self.current_record = int(self.table.focus())
-            return
+        self.open_window_edit_records(self.current_record)
-        values = {
-            0: self.aktiv.get(),
-            1: self.firma.get(),
-            2: self.name.get(),
-            3: self.strasse.get(),
-            4: self.plz_ort.get(),
-        }
-        for key, value in values.items():
-            self.table.set(self.current_record, key, value)
 
-        self.clear_entry_fields()
-        self._save_file()
-
-    def _load_file(self):
-        try:
-            with open(self.json_file, "r", encoding="utf-8") as f:
-                self.address_list = json.load(f)
-                self.address_list.sort(key=lambda x: (x[0], x[1]))
-        except FileNotFoundError:
-            self.show_error(
-                message_title="File not found",
-                message=f"{self.json_file_name} not found, creating empty file at {self.root_path}"
-            )
-            self.address_list = [["firma", "name", "adresse", "plz/ort"]]
-            with open(self.json_file, "w", encoding="utf-8") as f:
-                json.dump(self.address_list, f)
-        self.populate_table()
-
-    def _save_file(self):
-        self.export_table_to_address_list()
-        try:
-            with open(self.json_file, "w", encoding="utf-8") as f:
-                json.dump(self.address_list, f, indent=4, sort_keys=True)
-        except FileNotFoundError:
-            self.show_error(
-                message_title="Unexpected Error: File not found?!", message=f"{self.json_file_name} not found"
-            )
 
     def export_csv(self):
         try:
-            file_name = self.config.get("csv", "file_name")
+            with open(self.csv_file, "w", encoding="utf-8") as f:
-        except NoOptionError:
-            self.show_error("Error: Option missing", "Option file_name is missing [csv]")
-            return
-
-        try:
-            path = self.config.get("csv", "path")
-        except NoOptionError:
-            self.show_error("Error: Option missing", "Option path is missing [csv]")
-            return
-
-        if file_name == "":
-            self.show_error("Error: Bad config file", "var file_name for section [csv] not set")
-            return
-
-        if path == "":
-            self.show_error(
-                "Alert: CSV path not set",
-                f"var path for section [csv] not set, file will be saved in {self.root_path}"
-            )
-            path = self.root_path
-
-        csv_file = os.path.join(path, file_name)
-        try:
-            with open(csv_file, "w", encoding="utf-8") as f:
                 writer = csv.writer(f, delimiter=",")
                 for address in self.address_list:
-                    if address[0] != "x":
+                    if address["aktiv"] != "x":
                         continue
-                    del address[0]
+                    line = []
-                    if address[0] == "":
+                    if address["firma"] != "":
-                        del address[0]
+                        line.append(address["firma"])
-                    writer.writerow(address)
+                    line.append(address["name"])
+                    line.append(address["strasse"])
+                    line.append(address["plzort"])
+                    writer.writerow(line)
         except FileNotFoundError:
-            self.show_error("Unexpected error", f"Could not write file {csv_file}")
+            show_error(message_title="Unexpected error",
+                       message=f"Could not write file {self.csv_file}",
+                       parent=self.root
+                       )
 
-    def populate_table(self):
+    def populate_table(self, reload=True):
+        if reload:
+            self.address_list = self.model.get_all()
+        if len(self.address_list) == 0:
+            return
         self.delete_all_table_items()
-        for index, item in enumerate(self.address_list):
+        for address in self.address_list:
-            self.table.insert('', 'end', iid=index, values=item)
+            # skip inactive records if filter is true
+            if self.filter_active.get() and address["aktiv"] != "x":
+                continue
+            print(address)
+            self.table.insert('', 'end', iid=address["record_id"],
+                              values=(address["aktiv"], address["firma"], address["name"], address["strasse"],
+                                      address["plzort"])
+                              )
+        self.update_status_bar()
 
     def export_table_to_address_list(self):
         self.address_list.clear()
@@ -273,14 +279,40 @@ class Application:
             self.address_list.append([])
             for value in self.table.item(child)['values']:
                 self.address_list[-1].append(value)
+        self.update_status_bar()
 
     def delete_all_table_items(self):
         for item in self.table.get_children():
             self.table.delete(item)
 
-    @staticmethod
+    def open_window_settings(self):
-    def show_error(message_title: str, message: str):
+        window = SettingsWindow(self, self.root)
-        messagebox.showwarning(title=message_title, message=message)
+        window.wm_transient(self.root)
+        window.wait_visibility()
+        window.grab_set()
+        return window
+
+    def open_window_edit_records(self, record_id: int):
+        window = EditRecord(self, self.root, record_id)
+        window.wm_transient(self.root)
+        window.wait_visibility()
+        window.grab_set()
+
+    def update_status_bar(self):
+        self._count_address_records()
+        self.statusbar.set(f"Adressen: {self.length_address_list} | Aktive Adressen: {self.length_address_list_active}")
+
+    def _count_address_records(self):
+        self.length_address_list = len(self.address_list)
+        count = 0
+        for address in self.address_list:
+            if address["aktiv"] == "x":
+                count += 1
+        self.length_address_list_active = count
+
+    def first_sort_after_start(self):
+        self.address_list.sort(key=lambda x: (x["firma"], x["name"]))
+        self.populate_table()
 
 
 if __name__ == '__main__':

src/compile.sh

@@ -1,2 +0,0 @@
-#!/usr/bin/env bash
-pyinstaller --clean --add-data=files/config.ini:files --add-data=files/address_data.json:files brovski-adress-etiketten-verwaltung.py

src/config.py

@@ -0,0 +1,45 @@
+import os
+from configparser import ConfigParser
+
+
+class Config:
+    parser: ConfigParser
+
+    def __init__(self):
+        """
+        Config parser reading config.ini
+
+        Attributes:
+            self.parser: ConfigParser holding list of sections and options
+            self.__filename: Path and name to the config file
+        """
+        self.parser = ConfigParser()
+
+        home_path = os.environ["HOME"]
+        full_path = os.path.join(home_path, ".config", "brovski-adress-etiketten" )
+        if not os.path.exists(full_path):
+            os.makedirs(full_path)
+        self.config_file = os.path.join(full_path, "config.ini")
+
+        self._load()
+
+    def _save(self):
+        with open(self.config_file, 'w') as outfile:
+            self.parser.write(outfile)
+
+    def _load(self):
+        self.parser.read(self.config_file)
+
+    def add_section(self, section):
+        self._load()
+        self.parser.add_section(section)
+        self._save()
+
+    def set(self, section: str, option: str, value: str):
+        self._load()
+        self.parser.set(section, option, value)
+        self._save()
+
+    def get(self, section: str, option: str):
+        self._load()
+        return self.parser.get(section, option)

src/connector.py

@@ -0,0 +1,82 @@
+import json
+import os
+from abc import ABC, abstractmethod
+
+import config
+
+
+class Connector(ABC):
+    def __init__(self):
+        pass
+
+    @abstractmethod
+    def get_all(self) -> list:
+        pass
+
+    @abstractmethod
+    def get_by_id(self, record_id: int) -> dict:
+        pass
+
+    @abstractmethod
+    def delete_by_id(self, record_id: int):
+        pass
+
+    @abstractmethod
+    def update_record(self, updated_record: dict):
+        pass
+
+    @abstractmethod
+    def create_new(self, values: dict) -> int:
+        pass
+
+
+class JSONConnector(Connector):
+    def __init__(self):
+        super().__init__()
+        self.config = config.Config()
+        self.json_path = self.config.get("json", "path")
+        self.json_file = os.path.join(self.json_path, "brovski-adress-etiketten-verwaltung.json")
+
+    def get_all(self) -> list:
+        with open(self.json_file, "r") as f:
+            return json.load(f)
+
+    def get_all_sorted_by(self, field: str, reverse=False) -> list:
+        with open(self.json_file, "r") as f:
+            data = json.load(f)
+        return sorted(data, key=lambda k: k[field], reverse=reverse)
+
+    def get_by_id(self, record_id: int) -> dict:
+        data = self.get_all()
+        for record in data:
+            if record["record_id"] == record_id:
+                return record
+
+    def delete_by_id(self, record_id: int):
+        data = self.get_all()
+        for idx, record in enumerate(data):
+            if record["record_id"] == record_id:
+                del data[idx]
+        self._write_to_file(data)
+
+    def update_record(self, new_record: dict):
+        data = self.get_all()
+        for idx, record in enumerate(data):
+            if record["record_id"] == new_record["record_id"]:
+                data[idx] = new_record
+        self._write_to_file(data)
+
+    def create_new(self, record: dict) -> int:
+        data = self.get_all()
+        if len(data) == 0:
+            next_id = 0
+        else:
+            next_id = max(data, key=lambda x: x["record_id"])["record_id"] + 1
+        record["record_id"] = next_id
+        data.append(record)
+        self._write_to_file(data)
+        return next_id
+
+    def _write_to_file(self, data):
+        with open(self.json_file, "w") as f:
+            json.dump(data, f, indent=4)

src/files/address_data.json

@@ -1,45 +0,0 @@
-[
-        [
-                "x",
-                "Matthias Freilinger",
-                "Gibelstrasse 8",
-                "6231 Schlierbach"
-        ],
-        [
-                "x",
-                "Andreas Fischer",
-                "Dorfrain 3A",
-                "5702 Niederlenz"
-        ],
-        [
-                "x",
-                "Chrege Braun",
-                "Emausstrasse 21",
-                "5621 Zufikon"
-        ],
-        [
-                "x",
-                "Sarah Märki",
-                "Hubgasse 6",
-                "8570 Weinfelden"
-        ],
-        [
-                "x",
-                "Richard Hanselmann",
-                "Schemel 90",
-                "5077 Elfingen"
-        ],
-        [
-                "x",
-                "Simon Lüthi",
-                "Delphinweg 1",
-                "5616 Meisterschwanden"
-
-        ],
-        [
-                "x",
-                "Jacqueline Felder",
-                "Rappenmööslistr. 20",
-                "8840 Einsiedeln"
-        ]
-]

src/files/config.ini

@@ -1,3 +0,0 @@
-[csv]
-file_name=address.csv
-path=

src/windows.py

@@ -0,0 +1,158 @@
+import tkinter as tk
+from configparser import NoSectionError, NoOptionError
+from tkinter import font, filedialog, messagebox
+
+from config import Config
+from connector import JSONConnector
+
+
+def show_error(message_title: str, message: str, parent: tk.Tk | tk.Toplevel):
+    messagebox.showwarning(
+        title=message_title,
+        message=message,
+        parent=parent
+    )
+
+
+class Window(tk.Toplevel):
+    def __init__(self, parent, root: tk.Tk):
+        super().__init__(root)
+        self.parent = parent
+        self.root = root
+        self.protocol("WM_DELETE_WINDOW", self.close_window)
+        self.bind("<Escape>", self.close_window)
+
+    def close_window(self, event = None):
+        self._destroy_window()
+
+    def _destroy_window(self):
+        self.parent.populate_table()
+        self.root.update()
+        self.destroy()
+
+
+class EditRecord(Window):
+    def __init__(self, parent, root: tk.Tk, record_id: int):
+        super().__init__(parent, root)
+        self.bind("<Return>", self._update)
+
+        self.model = JSONConnector()
+
+        record = self.model.get_by_id(record_id)
+
+        button_width = 8
+        self.title("Adresse bearbeiten")
+        self.record_id = tk.IntVar(value=record_id)
+        self.aktiv = tk.StringVar(value=record.get("aktiv"))
+        self.firma = tk.StringVar(value=record.get("firma"))
+        self.name = tk.StringVar(value=record.get("name"))
+        self.strasse = tk.StringVar(value=record.get("strasse"))
+        self.plz_ort = tk.StringVar(value=record.get("plzort"))
+
+        edit_frame = tk.Frame(self)
+        edit_frame.pack(side=tk.TOP, fill=tk.X, pady=20, padx=20)
+        tk.Label(edit_frame, text="Aktiv", anchor=tk.W).grid(row=0, column=0)
+        tk.Label(edit_frame, text="Firma").grid(row=0, column=1)
+        tk.Label(edit_frame, text="Name").grid(row=0, column=2)
+        tk.Label(edit_frame, text="Strasse").grid(row=0, column=3)
+        tk.Label(edit_frame, text="Plz/Ort").grid(row=0, column=4)
+        edit_aktiv = tk.Checkbutton(edit_frame, variable=self.aktiv, onvalue="x", offvalue="")
+        edit_aktiv.grid(row=1, column=0)
+        edit_firma = tk.Entry(edit_frame, textvariable=self.firma)
+        edit_firma.grid(row=1, column=1)
+        edit_name = tk.Entry(edit_frame, textvariable=self.name)
+        edit_name.grid(row=1, column=2)
+        edit_strasse = tk.Entry(edit_frame, textvariable=self.strasse)
+        edit_strasse.grid(row=1, column=3)
+        edit_plz_ort = tk.Entry(edit_frame, textvariable=self.plz_ort)
+        edit_plz_ort.grid(row=1, column=4)
+
+        button_frame = tk.Frame(self)
+        button_frame.pack(side=tk.TOP, pady=10)
+        tk.Button(button_frame, text="Save", command=self._update, width=button_width).pack(side=tk.LEFT)
+        tk.Button(button_frame, text="Abbrechen", command=self.close_window, width=button_width).pack(side=tk.LEFT)
+
+    def _update(self, event = None):
+        new_record = {
+            "record_id": self.record_id.get(),
+            "aktiv": self.aktiv.get(),
+            "firma": self.firma.get(),
+            "name": self.name.get(),
+            "strasse": self.strasse.get(),
+            "plzort": self.plz_ort.get(),
+        }
+        self.model.update_record(new_record)
+        self.close_window()
+
+
+class SettingsWindow(Window):
+    def __init__(self, parent, root: tk.Tk):
+        super().__init__(parent, root)
+        self.geometry(f"500x330+{self.root.winfo_x() + 20}+{self.root.winfo_y() + 20}")
+        self.config = Config()
+        self.json_file = tk.StringVar()
+        self.csv_file = tk.StringVar()
+
+        title = font.Font(family='Ubuntu Mono', size=20, weight=font.BOLD)
+        tk.Label(self, text="Einstellungen", font=title).pack(pady=20)
+
+        path_frame = tk.Frame(self)
+        path_frame.pack(pady=(10, 40))
+
+        tk.Label(path_frame, text="Datenpfad JSON Datei").grid(row=0, column=0)
+        tk.Entry(path_frame, textvariable=self.json_file, width=50).grid(row=1, column=0)
+        tk.Button(path_frame, text="Pfad", command=lambda: self.set_json_path(self.json_file.get())).grid(row=1,
+                                                                                                          column=1)
+
+        tk.Label(path_frame, text="Datenpfad CSV Export Datei").grid(row=2, column=0)
+        tk.Entry(path_frame, textvariable=self.csv_file, width=50).grid(row=3, column=0)
+        tk.Button(path_frame, text="Pfad", command=lambda: self.set_csv_path(self.csv_file.get())).grid(row=3, column=1)
+
+        bottom_frame = tk.Frame(self)
+        bottom_frame.pack()
+        tk.Button(bottom_frame, text="Ok", command=self.ok).pack(side=tk.LEFT)
+        tk.Button(bottom_frame, text="Abbrechen", command=self.cancel).pack(side=tk.LEFT)
+
+        self.load_config()
+
+    def set_json_path(self, initial_dir: str = ""):
+        initial_dir = "~/" if initial_dir == "" else initial_dir
+        new_path = filedialog.askdirectory(initialdir=initial_dir, title="Datenpfad JSON Datei")
+        new_path = initial_dir if len(new_path) == 0 else new_path
+        self.json_file.set(new_path)
+
+    def set_csv_path(self, initial_dir: str = ""):
+        initial_dir = "~/" if initial_dir == "" else initial_dir
+        new_path = filedialog.askdirectory(initialdir=initial_dir, title="Datenpfad CSV Datei")
+        new_path = initial_dir if len(new_path) == 0 else new_path
+        self.csv_file.set(new_path)
+
+    def load_config(self):
+        try:
+            self.json_file.set(self.config.get("json", "path"))
+        except NoSectionError:
+            self.config.add_section("json")
+            self.config.set("json", "path", "")
+        except NoOptionError:
+            self.config.set("json", "path", "")
+
+        try:
+            self.csv_file.set(self.config.get("csv", "path"))
+        except NoSectionError:
+            self.config.add_section("csv")
+            self.config.set("csv", "path", "")
+        except NoOptionError:
+            self.config.set("csv", "path", "")
+
+    def ok(self):
+        if self.json_file.get() == "" or self.csv_file.get() == "":
+            show_error(message_title="Fehlerhafte Konfiguration",
+                       message="Pfad für JSON oder CSV Datei fehlt",
+                       parent=self)
+            return
+        self.config.set("json", "path", self.json_file.get())
+        self.config.set("csv", "path", self.csv_file.get())
+        self.close_window()
+
+    def cancel(self):
+        self.close_window()

version.txt

@@ -0,0 +1 @@
+0.6a