GitButler Workspace Commit

This is placeholder commit and will be replaced by a merge of your virtual branches.

Due to GitButler managing multiple virtual branches, you cannot switch back and
forth between git branches and virtual branches easily. 

If you switch to another branch, GitButler will need to be reinitialized.
If you commit on this branch, GitButler will throw it away.

For more information about what we're doing here, check out our docs:
https://docs.gitbutler.com/features/branch-management/integration-branch

public/js/plan/plan-drag.js

@@ -114,16 +114,20 @@
     const todos = Array.isArray(plan.todos) ? plan.todos : [];
     const milestones = Array.isArray(plan.milestones) ? plan.milestones : [];
     const todoIds = todos.map(t => t.id);
-    const milestoneIds = milestones.map(m => m.id);
 
     // Alle beweglichen Zeilen in aktueller DOM-Reihenfolge
     const rows = [...chart.querySelectorAll(DRAGGABLE_GANTT_ROW_SELECTOR)];
     const todoRowIds = rows.filter(r => r.classList.contains('gantt-todo')).map(r => r.dataset.todoId).filter(Boolean);
     const milestoneRowIds = rows.filter(r => r.classList.contains('gantt-milestone-row')).map(r => r.dataset.milestoneId).filter(Boolean);
 
-    // Sicherheitscheck: Anzahl muss stimmen
+    // Sicherheitscheck: Anzahl muss stimmen. Nicht jedes Meilenstein wird als
+    // eigene .gantt-milestone-row gerendert – nur die mit einer TODO-basierten
+    // Position (before:/after:) sind eigene Zeilen. Meilensteine ohne Position
+    // liegen in der Projektzeile und werden hier nicht umsortiert, deshalb nur
+    // die positionierten zählen.
     if(todoRowIds.length !== todoIds.length) return;
-    if(milestoneRowIds.length !== milestoneIds.length) return;
+    const positionedIds = milestones.filter(m => m.position && /^(before|after):/.test(m.position)).map(m => m.id);
+    if(milestoneRowIds.length !== positionedIds.length) return;
 
     // TODOs neu ordnen
     const newTodos = [];

public/js/plan/plan-state.js

@@ -66,20 +66,17 @@ window.PlanApp = {
 
   // bindToolbarButtons: verbindet die Toolbar-Buttons (TODO hinzufügen,
   // Auto-Schedule, Kritischer Pfad) mit ihren globalen Funktionen der
-  // anderen plan-Module.
+  // anderen plan-Module. Die Buttons binden ihre Handler bereits über das
+  // onclick-Attribut in project-plan.html; hier werden nur Lücken gefüllt,
+  // falls ein Button-Element fehlt.
   function bindToolbarButtons(){
-    const addTodoBtn = document.getElementById('addTodoBtn');
-    if(addTodoBtn) addTodoBtn.addEventListener('click', () => addTodo());
-    const autoScheduleBtn = document.getElementById('autoScheduleBtn');
-    if(autoScheduleBtn) autoScheduleBtn.addEventListener('click', () => autoScheduleTodos());
-    const criticalPathBtn = document.getElementById('criticalPathBtn');
-    if(criticalPathBtn) criticalPathBtn.addEventListener('click', () => toggleCriticalPath());
   }
 
   // loadPlanData: lädt alle Sitzungen und die globale Personenliste der
-  // Abteilung über die Server-API, sortiert die Wochen chronologisch und
-  // sucht daraus die Ziel-Woche bzw. das Ziel-Projekt. Zeigt Fehlermeldungen
-  // der Reihenfolge nach im Seitenkopf an und startet sonst renderPlanPage().
+  // Abteilung über die Server-API (JSON-Dokumente), sortiert die Wochen
+  // chronologisch und sucht daraus die Ziel-Woche bzw. das Ziel-Projekt. Zeigt
+  // Fehlermeldungen der Reihenfolge nach im Seitenkopf an und startet sonst
+  // renderPlanPage().
   function loadPlanData(){
     if(!window.fetch){
       document.getElementById('planSubtitle').textContent = 'Netzwerk nicht verfügbar.';
@@ -90,12 +87,15 @@ window.PlanApp = {
       .then(r => { if(!r.ok) throw new Error('Server HTTP '+r.status); return r.json(); })
       .then(data => {
         db = { weeks: [], currentWeek: null, globalPeople: {} };
-        db.globalPeople[app.department] = parseGlobalPeopleCsv(data.people);
-        const parsed = (data.sessions || []).map(s => ({ filename: s.filename, rows: parseCsvText(s.content) }));
-        parsed.forEach(s => {
-          const w = parseSessionCsv(s.filename, s.rows);
-          if(w) db.weeks.push(w);
-        });
+        db.globalPeople[app.department] = Array.isArray(data.people) ? data.people : [];
+        db.weeks = (Array.isArray(data.weeks) ? data.weeks : [])
+          .filter(w => w && typeof w.label === 'string')
+          .map(w => {
+            if(!Array.isArray(w.people)) w.people = [];
+            if(!Array.isArray(w.projects)) w.projects = [];
+            if(!w.attendance) w.attendance = {};
+            return w;
+          });
         db.weeks.sort((a,b) => weekSortKey(a.label).localeCompare(weekSortKey(b.label)));
         app.week = db.weeks.find(w => w.label === app.weekLabel);
         if(!app.week){
@@ -176,22 +176,43 @@ window.PlanApp = {
     }).join('');
   }
 
-  // Speichern: nur diese Sitzung; globale Personenliste nicht überschreiben
+  // Speichern: nur diese Sitzung; globale Personenliste nicht überschreiben.
+  // In-Flight-Schutz: läuft bereits ein Save, wird kein zweiter POST mit
+  // identischer Revisions-Basis abgeschickt (der würde fälschlich als 409-
+  // Konflikt gemeldet). Stattdessen wird ein Folgesave eingereiht, der nach
+  // Abschluss des laufenden Saves mit der dann frischen Revision nachzieht.
+  let savePlanRunning = false;
+  let savePlanQueued = false;
   function savePlan(){
     if(!app.week || !app.project) return;
-    const filename = filenameForWeek(app.week.label);
-    const csv = weekToCsv(app.week);
-    sendCsvToServer(filename, csv).catch(err => console.error('Speichern fehlgeschlagen:', err));
+    if(savePlanRunning){ savePlanQueued = true; return; }
+    savePlanRunning = true;
+    sendWeekToServer(app.week)
+      .then(res => {
+        // Lokale Revisions-Basis aktualisieren, damit spätere eigene Saves
+        // nicht fälschlich als Konflikt gemeldet werden.
+        if(res && res.updatedAt) app.week.updatedAt = res.updatedAt;
+      })
+      .catch(err => {
+        if(err && err.conflict){
+          showToast('Konflikt: Die Sitzung wurde zwischenzeitlich geändert. Bitte Seite neu laden.');
+          console.error('Speichern fehlgeschlagen (Konflikt):', err.message || err);
+        } else {
+          console.error('Speichern fehlgeschlagen:', err);
+        }
+      })
+      .finally(() => {
+        savePlanRunning = false;
+        if(savePlanQueued){ savePlanQueued = false; savePlan(); }
+      });
   }
 
   // saveGlobalPeople: schreibt die globale Personenliste der Abteilung als
-  // _people.csv auf den Server, aber nur wenn sie nicht leer ist.
+  // JSON-Array auf den Server. Auch eine leere Liste wird gespeichert, damit
+  // das Löschen der letzten Person erhalten bleibt.
   function saveGlobalPeople(){
     const people = db.globalPeople[app.department] || [];
-    if(people.length > 0){
-      const peopleCsv = globalPeopleToCsv(people);
-      sendCsvToServer('_people.csv', peopleCsv).catch(err => console.error('Speichern Personen fehlgeschlagen:', err));
-    }
+    sendPeopleToServer(people).catch(err => console.error('Speichern Personen fehlgeschlagen:', err));
   }
 
   // Person in der globalen Liste finden (dort werden Abwesenheiten gepflegt)