fix: improve git editor status handling

Author: Lukas Geiger

.gitattributes

@@ -0,0 +1,11 @@
+* text=auto
+
+*.py text eol=lf
+*.json text eol=lf
+*.md text eol=lf
+*.yml text eol=lf
+*.yaml text eol=lf
+*.bat text eol=crlf
+
+*.png binary
+*.ico binary

CHANGELOG.md

@@ -22,8 +22,11 @@ Format basiert auf [Keep a Changelog](https://keepachangelog.com/de/1.1.0/).
 - Doppelte `run_script`-Definition in `PythonArchitect` beseitigt, damit F5 wieder konsistent über das Debug-Output-Panel läuft.
 - Entfernte Qt6-APIs `fontMetrics().width()` und `setTabStopWidth()` durch aktuelle Alternativen ersetzt.
 - Externe Python-Skripte starten jetzt mit `sys.executable` statt einem hardcodierten `python`/`python3`.
-
 - Die Minimap-Option im Einstellungsdialog nutzt jetzt denselben `show_minimap`-Key wie das Ansicht-Menü und wird auch über den Apply-Button direkt auf die Hauptansicht angewendet.
+- Kombinierte Git-Porcelain-Statuscodes wie `AM` werden in der Statusleiste lesbar zusammengefasst.
+- Git-Diff-Markierungen behandeln ersetzte Zeilen als geändert statt als reine Hinzufügung.
+- `Speichern unter` stellt den bisherigen Dateipfad wieder her, wenn der Dialog abgebrochen wird.
+- Deutsche Übersetzungshinweise und Docstrings nutzen echte Umlaute.
 
 ## [1.0.0] - YYYY-MM-DD
 

PythonBox_v8.py

@@ -1176,6 +1176,38 @@ class GitIntegration:
     def __init__(self):
         self.has_git = shutil.which("git") is not None
 
+    @staticmethod
+    def _describe_status_code(code: str) -> str:
+        """Formatiert einen Git-Porcelain-Status in eine lesbare Beschreibung."""
+        if not code:
+            return ""
+        if code == "??":
+            return "? Untracked"
+        if code == "!!":
+            return "! Ignored"
+
+        status_labels = []
+        label_map = {
+            "M": "● Modified",
+            "A": "+ Added",
+            "D": "− Deleted",
+            "R": "→ Renamed",
+            "C": "⊕ Copied",
+            "U": "⚡ Conflict",
+        }
+
+        for char in code:
+            if char == " ":
+                continue
+            label = label_map.get(char)
+            if label and label not in status_labels:
+                status_labels.append(label)
+
+        if status_labels:
+            return " / ".join(status_labels)
+
+        return code.strip() or code
+
     def get_repo_root(self, file_path: str) -> Optional[str]:
         """Findet das Git-Repository Root"""
         if not self.has_git:
@@ -1208,17 +1240,8 @@ def get_file_status(self, file_path: str) -> str:
             )
             status = result.stdout.strip()
             if status:
-                code = status[:2].strip()
-                status_map = {
-                    'M': '● Modified',
-                    'A': '+ Added',
-                    'D': '− Deleted',
-                    '??': '? Untracked',
-                    'R': '→ Renamed',
-                    'C': '⊕ Copied',
-                    'U': '⚡ Conflict'
-                }
-                return status_map.get(code, code)
+                code = status[:2]
+                return self._describe_status_code(code)
             return "✓ Unchanged"
         except Exception:
             return ""
@@ -1252,19 +1275,27 @@ def get_modified_lines(self, file_path: str) -> Tuple[set, set, set]:
             return added, modified, deleted
 
         current_line = 0
+        pending_deletion = False
         for line in diff.split('\n'):
             if line.startswith('@@'):
                 # Parse @@ -start,count +start,count @@
                 match = re.search(r'\+(\d+)', line)
                 if match:
                     current_line = int(match.group(1)) - 1
+                pending_deletion = False
             elif line.startswith('+') and not line.startswith('+++'):
                 current_line += 1
-                added.add(current_line)
+                if pending_deletion:
+                    modified.add(current_line)
+                else:
+                    added.add(current_line)
+                pending_deletion = False
             elif line.startswith('-') and not line.startswith('---'):
                 deleted.add(current_line)
+                pending_deletion = True
             else:
                 current_line += 1
+                pending_deletion = False
 
         return added, modified, deleted
 
@@ -3330,9 +3361,11 @@ def save_file(self):
     def save_file_as(self):
         tab = self.tab_editor.current_tab()
         if tab:
+            original_path = tab.file_path
             tab.file_path = None  # Force "Save As" dialog
             index = self.tab_editor.tab_widget.currentIndex()
-            self.tab_editor.save_tab(index)
+            if not self.tab_editor.save_tab(index):
+                tab.file_path = original_path
 
     # --- LIBRARY TREE ---
 

README.md

@@ -24,7 +24,9 @@ PythonBox is a lightweight Python IDE with a dark theme, integrated debugging, c
 - Debug-Toolbar mit Step In, Step Over und Step Out
 - Linter-Integration für Pylint und Flake8
 - Git-Status, Diff und Modified-Markierung
+- Kombinierte Git-Statuscodes werden lesbar angezeigt; ersetzte Diff-Zeilen werden als geändert statt nur als hinzugefügt markiert
 - Qt6-kompatible Editor-Metriken und F5-Ausführung über das Debug-Output-Panel
+- `Speichern unter` behält bei abgebrochenem Dialog den bisherigen Dateipfad
 - Die Minimap-Einstellung bleibt zwischen Ansicht-Menü und Einstellungsdialog synchron, inklusive Fallback für ältere Konfigurationen
 
 ### Windows-Paketierung
@@ -65,7 +67,7 @@ Das Build-Ergebnis liegt anschließend in `dist/`. Build-Artefakte und lokale Re
 
 ## Tests
 
-Die Regressionstests prüfen die Qt6-API-Kompatibilität, die F5-Ausführung über `debug_output.run_normal`, die externe Terminal-Ausführung mit dem aktuellen Python-Interpreter, die Minimap-Einstellungssynchronisation und einen Offscreen-Smoke-Test für das Hauptfenster.
+Die Regressionstests prüfen die Qt6-API-Kompatibilität, die F5-Ausführung über `debug_output.run_normal`, die externe Terminal-Ausführung mit dem aktuellen Python-Interpreter, die Minimap-Einstellungssynchronisation, Git-Status-/Diff-Erkennung, `Speichern unter`-Abbruchverhalten und einen Offscreen-Smoke-Test für das Hauptfenster.
 
 ```bash
 python -m unittest discover -s tests -v

locales/translations.json

@@ -15,4 +15,4 @@
     "de": "⬇️ Einfügen",
     "en": "⬇️ Insert"
   }
-}
+}

tests/test_pythonbox_regressions.py

@@ -1,6 +1,8 @@
 import ast
 import importlib.util
 import os
+import shutil
+import subprocess
 import tempfile
 import unittest
 from pathlib import Path
@@ -105,6 +107,57 @@ def test_non_windows_external_commands_use_current_interpreter(self):
         self.assertEqual(["/opt/current-python/bin/python", "tool.py"], cmd)
 
 
+@unittest.skipUnless(shutil.which("git") is not None, "git required")
+class GitIntegrationRegressionTests(unittest.TestCase):
+    def _create_repo_with_tracked_file(self, initial_text: str):
+        tmp_dir = Path(tempfile.mkdtemp(prefix="pythonbox-git-"))
+        subprocess.run(["git", "init"], cwd=tmp_dir, check=True, capture_output=True)
+
+        file_path = tmp_dir / "demo.py"
+        file_path.write_text(initial_text, encoding="utf-8")
+        subprocess.run(["git", "-C", str(tmp_dir), "add", "demo.py"], check=True, capture_output=True)
+
+        self.addCleanup(shutil.rmtree, tmp_dir, True)
+        return tmp_dir, file_path
+
+    def test_git_status_formats_combined_codes_readably(self):
+        module = load_pythonbox_module()
+
+        _, file_path = self._create_repo_with_tracked_file("a\nb\nc\n")
+        file_path.write_text("a\nX\nc\n", encoding="utf-8")
+
+        git = module.GitIntegration()
+        status = git.get_file_status(str(file_path))
+
+        self.assertEqual("+ Added / ● Modified", status)
+
+    def test_git_modified_lines_classify_replacements_as_modified(self):
+        module = load_pythonbox_module()
+
+        _, file_path = self._create_repo_with_tracked_file("a\nb\nc\n")
+        file_path.write_text("a\nX\nc\n", encoding="utf-8")
+
+        git = module.GitIntegration()
+        added, modified, deleted = git.get_modified_lines(str(file_path))
+
+        self.assertEqual([], sorted(added))
+        self.assertEqual([2], sorted(modified))
+        self.assertEqual([1], sorted(deleted))
+
+    def test_git_modified_lines_classify_insertions_as_added(self):
+        module = load_pythonbox_module()
+
+        _, file_path = self._create_repo_with_tracked_file("a\nc\n")
+        file_path.write_text("a\nb\nc\n", encoding="utf-8")
+
+        git = module.GitIntegration()
+        added, modified, deleted = git.get_modified_lines(str(file_path))
+
+        self.assertEqual([2], sorted(added))
+        self.assertEqual([], sorted(modified))
+        self.assertEqual([], sorted(deleted))
+
+
 class SettingsRegressionTests(unittest.TestCase):
     def _temp_settings(self, module, folder: str):
         settings_path = Path(folder) / "settings.ini"
@@ -158,6 +211,30 @@ def test_main_window_reacts_to_minimap_setting_changes(self):
                 window.deleteLater()
                 app.processEvents()
 
+    def test_save_file_as_cancel_keeps_original_path(self):
+        module = load_pythonbox_module()
+        app = module.QApplication.instance() or module.QApplication([])
+
+        with tempfile.TemporaryDirectory() as temp_dir:
+            original_path = Path(temp_dir) / "demo.py"
+            original_path.write_text("print('hi')\n", encoding="utf-8")
+
+            window = module.PythonArchitect()
+            try:
+                tab = window.tab_editor.current_tab()
+                tab.file_path = str(original_path)
+
+                with mock.patch.object(
+                    module.QFileDialog, "getSaveFileName", return_value=("", "")
+                ):
+                    window.save_file_as()
+
+                self.assertEqual(str(original_path), tab.file_path)
+            finally:
+                window.close()
+                window.deleteLater()
+                app.processEvents()
+
 
 if __name__ == "__main__":
     unittest.main()

translator.py

@@ -1,5 +1,5 @@
 """
-TranslationSystem - Multi-Language Support fuer Anwendungen
+TranslationSystem - Multi-Language Support für Anwendungen
 ============================================================
 Version: 1.0.0 (isoliert aus _LANG)
 Quelle: ARC_EntwicklungsschleifeAdvanced/TranslationSystem.py v2.4
@@ -9,7 +9,7 @@
 from translator import TranslationSystem
 
 translator = TranslationSystem('de')
-label.setText(translator.t('Datei oeffnen'))
+label.setText(translator.t('Datei öffnen'))
 translator.set_language('en')
 """
 
@@ -49,8 +49,8 @@ def __init__(self, default_lang: str = 'de', app_dir: Path = None):
         ]
 
         self.german_hints = [
-            "datei", "bearbeiten", "ansicht", "hilfe", "oeffnen", "speichern",
-            "schliessen", "einstellungen", "abbrechen", "ok", "ja", "nein",
+            "datei", "bearbeiten", "ansicht", "hilfe", "oeffnen", "öffnen", "speichern",
+            "schliessen", "schließen", "einstellungen", "abbrechen", "ok", "ja", "nein",
             "start", "stop", "pause", "fortsetzen", "laden", "aktualisieren",
             "filter", "fehler", "export", "import", "optionen", "anzeigen",
         ]
@@ -75,13 +75,13 @@ def _save_translations(self):
 
     def t(self, key: str) -> str:
         """
-        Uebersetzt einen Key in die aktuelle Sprache.
+        Übersetzt einen Key in die aktuelle Sprache.
 
         Args:
             key: Translation-Key (oft der deutsche Originaltext)
 
         Returns:
-            Uebersetzter Text oder Key als Fallback
+            Übersetzter Text oder Key als Fallback
         """
         if key in self.translations:
             return self.translations[key].get(self.current_lang, key)