From f0d39f2eee2ef96d5c7e55a3ad1d8a67f648c2d3 Mon Sep 17 00:00:00 2001
From: Claude <claude@webklar.local>
Date: Fri, 16 Oct 2026 13:36:34 +0000
Subject: [PATCH] =?UTF-8?q?Arbeitsbl=C3=A4tter:=20Abo-Vorgabe=20bei=20Einr?=
 =?UTF-8?q?ichtungs-Tickets?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Bei Tickets mit einrichtungAbrechnung 'abo' steht "Vom Abo-Stundenkonto abziehen" im Dialog von Anfang an auf an (auch wenn der Abo-Status erst nach dem Öffnen ankommt), beim Nachtragen der Zeit eines Git-Blatts wird countsToPlan beim ersten Eintragen gesetzt. 'projekt', Kommentare und Kunden ohne Wartungs-Abo bekommen keine Vorgabe; ein von Hand abgehakter Haken bleibt aus.

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
---
 src/components/CreateWorksheetModal.jsx | 14 ++++++++--
 src/hooks/useWorksheets.js              |  7 +++--
 src/lib/planVorgabe.js                  | 32 +++++++++++++++++++++
 src/pages/TicketPage.jsx                |  3 +-
 tests/unit/planVorgabe.test.mjs         | 37 +++++++++++++++++++++++++
 5 files changed, 86 insertions(+), 7 deletions(-)
 create mode 100644 src/lib/planVorgabe.js
 create mode 100644 tests/unit/planVorgabe.test.mjs

diff --git a/src/components/CreateWorksheetModal.jsx b/src/components/CreateWorksheetModal.jsx
index d6828b9..62633d6 100644
--- a/src/components/CreateWorksheetModal.jsx
+++ b/src/components/CreateWorksheetModal.jsx
@@ -7,6 +7,7 @@ import { Button, DateTimeInput, Dialog, DurationSelect, Field } from './ui'
 import { roundUpToStep, isValidStep } from '../lib/timeSteps'
 import { validateSupporters } from '../lib/support'
 import SupportersEditor from './SupportersEditor'
+import { countsToPlanWert } from '../lib/planVorgabe'
 
 // Akquise-Tickets: Ergebnis des Kundenkontakts (statt technischer Status)
 const ACQUISITION_STATUS = ['In Kontakt', 'Zugesagt', 'Abgesagt']
@@ -31,7 +32,7 @@ const initialForm = (workorder, newStatus) => ({
   endTime: '',
   details: '',
   isComment: false,
-  countsToPlan: false,
+  countsToPlan: null, // null = unberührt: dann gilt die Abo-Vorgabe des Tickets (planVorgabe.js)
   supporters: []
 })
 
@@ -50,6 +51,13 @@ export default function CreateWorksheetModal({ isOpen, onClose, workorder, onCre
     : (workorder?.status || 'Open')
 
   const [formData, setFormData] = useState(() => initialForm(workorder, defaultStatus))
+  // Der Abo-Status kommt oft erst nach dem Öffnen: der Wert wird bei jedem Rendern aus Handwahl und Vorgabe berechnet
+  const countsToPlan = countsToPlanWert({
+    gewaehlt: formData.countsToPlan,
+    ticket: workorder,
+    aboHoursAllowed,
+    isComment: formData.isComment
+  })
 
   const [loading, setLoading] = useState(false)
   const [error, setError] = useState('')
@@ -130,7 +138,7 @@ export default function CreateWorksheetModal({ isOpen, onClose, workorder, onCre
         endTime: timeToWoms(formData.endTime),
         details: formData.details,
         isComment: formData.isComment,
-        countsToPlan: aboHoursAllowed && !formData.isComment && formData.countsToPlan,
+        countsToPlan,
         employeeShort: user?.prefs?.shortCode || '', // Aus User-Preferences
         supporters
       }
@@ -222,7 +230,7 @@ export default function CreateWorksheetModal({ isOpen, onClose, workorder, onCre
         <label className="check" title="Zeit wird vom Stundenguthaben des Wartungs-Abos abgezogen und im Kundenportal angezeigt">
           <input
             type="checkbox"
-            checked={formData.countsToPlan && !formData.isComment}
+            checked={countsToPlan}
             disabled={formData.isComment}
             onChange={(e) => handleChange('countsToPlan', e.target.checked)}
           />
diff --git a/src/hooks/useWorksheets.js b/src/hooks/useWorksheets.js
index ea266e0..f60b507 100644
--- a/src/hooks/useWorksheets.js
+++ b/src/hooks/useWorksheets.js
@@ -228,15 +228,16 @@ export function useWorksheets(woid = null) {
 
   /**
    * Aendert nachtraeglich Beginn, Ende und Arbeitszeit - nur fuer den Ersteller
-   * (employee = eigener Eintrag aus "employees", fuer Git-Pushes)
+   * (employee = eigener Eintrag aus "employees", fuer Git-Pushes).
+   * extra: weitere Felder fuer dasselbe Update (z. B. countsToPlan aus der Abo-Vorgabe)
    */
-  const updateWorksheetTimes = async (worksheet, times, currentUser, employee) => {
+  const updateWorksheetTimes = async (worksheet, times, currentUser, employee, extra = {}) => {
     if (!canEditWorksheetTimes(worksheet, currentUser, employee)) {
       return { success: false, error: 'Nur wer das Arbeitsblatt erstellt hat, kann die Zeiten ändern.' }
     }
     const { startDate, startTime, endDate, endTime, totalTime } = times
     if (!isValidStep(totalTime)) return { success: false, error: STEP_ERROR }
-    const data = { startDate, startTime, endDate, endTime, totalTime }
+    const data = { startDate, startTime, endDate, endTime, totalTime, ...extra }
     // Unterstuetzung nur schreiben, wenn sie sich geaendert hat. Hat inzwischen jemand anderes
     // (z. B. ueber "Ich habe unterstuetzt") etwas geaendert, nicht still ueberschreiben.
     if (times.supporters) {
diff --git a/src/lib/planVorgabe.js b/src/lib/planVorgabe.js
new file mode 100644
index 0000000..e649455
--- /dev/null
+++ b/src/lib/planVorgabe.js
@@ -0,0 +1,32 @@
+// Abo-Vorgabe für Arbeitsblätter an Einrichtungs-Tickets (V9): Hängt das Ticket an einer Funktion, die aus dem Abo
+// eingerichtet wird (einrichtungAbrechnung 'abo'), steht "Vom Abo-Stundenkonto abziehen" von Anfang an auf an.
+// 'projekt' (im Projekt enthalten) und alle anderen Tickets bekommen keine Vorgabe. Nur Vergleiche, keine Zeitlogik.
+import { isGitWorksheet, needsTimeEntry } from './ticketForm'
+
+export const EINRICHTUNG_ABRECHNUNGEN = ['abo', 'projekt']
+
+// 'abo' | 'projekt' | '' – nur genau diese Werte, alles andere (Großschreibung, Zahl, fehlendes Ticket) gilt als nicht gesetzt
+export function einrichtungAbrechnung(ticket) {
+  const wert = ticket?.einrichtungAbrechnung
+  return EINRICHTUNG_ABRECHNUNGEN.includes(wert) ? wert : ''
+}
+
+// Vorgabe für countsToPlan: nur bei Abo-Einrichtung und wenn das Wartungs-Abo Stunden erlaubt
+export function countsToPlanVorgabe(ticket, aboHoursAllowed) {
+  return einrichtungAbrechnung(ticket) === 'abo' && aboHoursAllowed === true
+}
+
+// Wert des Hakens im Dialog. gewaehlt: null = unberührt (dann gilt die Vorgabe, auch wenn der Abo-Status erst nach dem
+// Öffnen ankommt), true/false = von Hand gesetzt. Ohne erlaubte Abo-Stunden und bei Kommentar ist der Wert immer false.
+export function countsToPlanWert({ gewaehlt, ticket, aboHoursAllowed, isComment }) {
+  if (isComment || aboHoursAllowed !== true) return false
+  if (gewaehlt === null || gewaehlt === undefined) return countsToPlanVorgabe(ticket, aboHoursAllowed)
+  return Boolean(gewaehlt)
+}
+
+// Felder, die beim Nachtragen der Zeit eines Git-Blatts mitgeschrieben werden: nur beim ersten Eintragen der Zeit
+// (das Blatt hat noch keine) und nur, wenn die Vorgabe greift. Eine bestehende Entscheidung bleibt unberührt.
+export function nachtragPlanFelder({ worksheet, ticket, aboHoursAllowed }) {
+  if (!isGitWorksheet(worksheet) || !needsTimeEntry(worksheet) || worksheet.countsToPlan) return {}
+  return countsToPlanVorgabe(ticket, aboHoursAllowed) ? { countsToPlan: true } : {}
+}
diff --git a/src/pages/TicketPage.jsx b/src/pages/TicketPage.jsx
index 21b3c93..26fe539 100644
--- a/src/pages/TicketPage.jsx
+++ b/src/pages/TicketPage.jsx
@@ -18,6 +18,7 @@ import { databases, DATABASE_ID, COLLECTIONS } from '../lib/appwrite'
 import { elapsedLabel } from '../lib/elapsed'
 import { canAddOwnSupport } from '../lib/support'
 import { canEditWorksheetTimes, serviceTypeLabel } from '../lib/ticketForm'
+import { nachtragPlanFelder } from '../lib/planVorgabe'
 import { listBackPath, resolveTicketTab, ticketPath, ticketRef } from '../lib/ticketRoutes'
 import { statusSelectOptions, ticketTypeLabel } from '../lib/ticketStatus'
 import { appointmentText, assigneeOptions, dueText, ticketNumber } from '../lib/ticketView'
@@ -307,7 +308,7 @@ function TicketView({ ticket, updateWorkorder, tab }) {
           worksheet={editTimesWorksheet}
           currentUserId={user?.$id}
           onClose={() => setEditTimesWorksheet(null)}
-          onSave={(ws, times) => updateWorksheetTimes(ws, times, user, employee)}
+          onSave={(ws, times) => updateWorksheetTimes(ws, times, user, employee, nachtragPlanFelder({ worksheet: ws, ticket, aboHoursAllowed }))}
           employeesState={employeesState}
         />
       )}
diff --git a/tests/unit/planVorgabe.test.mjs b/tests/unit/planVorgabe.test.mjs
new file mode 100644
index 0000000..104bbe0
--- /dev/null
+++ b/tests/unit/planVorgabe.test.mjs
@@ -0,0 +1,37 @@
+import test from 'node:test'
+import assert from 'node:assert/strict'
+import { loadModule } from './load.mjs'
+const { EINRICHTUNG_ABRECHNUNGEN, einrichtungAbrechnung, countsToPlanVorgabe, countsToPlanWert, nachtragPlanFelder } = await loadModule('src/lib/planVorgabe.js')
+const abo = { einrichtungAbrechnung: 'abo' }, projekt = { einrichtungAbrechnung: 'projekt' }
+
+test('einrichtungAbrechnung: nur exakt abo oder projekt', () => {
+  assert.deepEqual(EINRICHTUNG_ABRECHNUNGEN, ['abo', 'projekt'])
+  assert.deepEqual([abo, projekt, {}, null, undefined, { einrichtungAbrechnung: 'Abo' }, { einrichtungAbrechnung: 5 }].map(einrichtungAbrechnung), ['abo', 'projekt', '', '', '', '', ''])
+})
+test('countsToPlanVorgabe: nur Abo-Einrichtung mit erlaubten Stunden', () => {
+  assert.equal(countsToPlanVorgabe(abo, true), true)
+  for (const [t, ok] of [[abo, false], [abo, undefined], [projekt, true], [{}, true], [null, true]]) assert.equal(countsToPlanVorgabe(t, ok), false)
+})
+test('countsToPlanWert: Vorgabe, späte Antwort des Abo-Status, Handwahl, Kommentar', () => {
+  const w = (o) => countsToPlanWert({ gewaehlt: null, ticket: abo, aboHoursAllowed: true, isComment: false, ...o })
+  assert.equal(w({}), true)                                         // unberührt: Vorgabe
+  assert.equal(w({ aboHoursAllowed: false }), false)                // Abo-Status lädt noch
+  assert.equal(w({ aboHoursAllowed: true }), true)                  // …und kommt danach: die Vorgabe greift trotzdem
+  assert.equal(w({ gewaehlt: false }), false)                       // von Hand abgehakt bleibt aus
+  assert.equal(w({ gewaehlt: true, ticket: projekt }), true)        // von Hand anhaken ist immer möglich
+  assert.equal(w({ gewaehlt: true, aboHoursAllowed: false }), false)
+  assert.equal(w({ ticket: projekt }), false)
+  assert.equal(w({ ticket: {} }), false)
+  for (const gewaehlt of [null, true, false]) assert.equal(w({ gewaehlt, isComment: true }), false)
+})
+test('nachtragPlanFelder: nur das erste Nachtragen einer Git-Zeit auf einer Abo-Einrichtung', () => {
+  const git = { serviceType: 'GIT', isComment: true, startTime: '', totalTime: 0, countsToPlan: false }
+  const f = (o) => nachtragPlanFelder({ worksheet: git, ticket: abo, aboHoursAllowed: true, ...o })
+  assert.deepEqual(f({}), { countsToPlan: true })
+  assert.deepEqual(f({ ticket: projekt }), {})
+  assert.deepEqual(f({ ticket: {} }), {})
+  assert.deepEqual(f({ aboHoursAllowed: false }), {})
+  assert.deepEqual(f({ worksheet: { ...git, startTime: '0900', totalTime: 45 } }), {})    // Zeit ist schon eingetragen: Entscheidung bleibt
+  assert.deepEqual(f({ worksheet: { ...git, countsToPlan: true } }), {})
+  assert.deepEqual(f({ worksheet: { serviceType: 'Remote', totalTime: 0, startTime: '' } }), {})
+})