refactor(contacts): decryptContactData aus shared-Helper nutzen #293

Merged

niboer merged 1 commit from refactor/decrypt-contact-data into main

2026-07-11 13:05:16 +02:00

niboer commented

2026-07-11 12:32:51 +02:00

Owner

Allgemein

closes #283

Altes Verhalten

decryptFields() ist lokal in contacts.ts definiert und entschluesselt manuell jedes Adressfeld (phone, email, addressStreet, addressZip, addressCity)
Doppelte Logik zur zentralen decryptContactData() in backend/src/lib/contact-decrypt.ts

Neues Verhalten

contacts.ts importiert decryptContactData aus ../lib/contact-decrypt.js
Die lokale decryptFields()-Funktion wurde entfernt
contacts.ts importiert decrypt nicht mehr direkt (entfernt)

Begruendung

Vermeidung von duplizierter Logik. Die extrahierte Hilfsfunktion ist bereits vorhanden und typisicher.

Test-Hinweise

npm test in backend/ besteht (622 Tests pass)
Betroffen: GET /api/v1/persons/:person_id/contacts und PUT /api/v1/persons/:person_id/contacts (beide rufen decryptContactData auf)

Checkliste

Keine neuen UI-Elemente (Backend-only Refactoring)
Keine neuen Routen (Backend-only Refactoring)

### Allgemein - closes #283 ### Altes Verhalten - `decryptFields()` ist lokal in `contacts.ts` definiert und entschluesselt manuell jedes Adressfeld (phone, email, addressStreet, addressZip, addressCity) - Doppelte Logik zur zentralen `decryptContactData()` in `backend/src/lib/contact-decrypt.ts` ### Neues Verhalten - `contacts.ts` importiert `decryptContactData` aus `../lib/contact-decrypt.js` - Die lokale `decryptFields()`-Funktion wurde entfernt - `contacts.ts` importiert `decrypt` nicht mehr direkt (entfernt) ### Begruendung Vermeidung von duplizierter Logik. Die extrahierte Hilfsfunktion ist bereits vorhanden und typisicher. ### Test-Hinweise - `npm test` in backend/ besteht (622 Tests pass) - Betroffen: `GET /api/v1/persons/:person_id/contacts` und `PUT /api/v1/persons/:person_id/contacts` (beide rufen `decryptContactData` auf) ### Checkliste - [ ] Keine neuen UI-Elemente (Backend-only Refactoring) - [ ] Keine neuen Routen (Backend-only Refactoring)

niboer added 1 commit

2026-07-11 12:32:51 +02:00

refactor(contacts): decryptContactData aus shared-Helper nutzen

CI Backend / lint-backend (pull_request) Successful in 1m26s

Details

CI Frontend / lint-frontend (pull_request) Successful in 1m52s

Details

CI Meta / conventional-commit (pull_request) Successful in 2s

Details

E2E / e2e (pull_request) Successful in 3m39s

Details

CI Backend / test-backend (pull_request) Successful in 3m56s

Details

CI Frontend / test-frontend (pull_request) Successful in 2m0s

Details

1398568051

refs #283

Aktuelles: decryptFields() ist lokal in contacts.ts definiert (duplizierte Logik)
Neu: contacts.ts importiert decryptContactData aus contact-decrypt.ts; decryptFields() entfernt

niboer scheduled this pull request to auto merge when all checks succeed

2026-07-11 12:42:22 +02:00

niboer merged commit 300cb8bfd1 into main

2026-07-11 13:05:16 +02:00

niboer deleted branch refactor/decrypt-contact-data

2026-07-11 13:05:16 +02:00

niboer referenced this pull request from a commit

2026-07-11 13:05:17 +02:00

refactor(contacts): decryptContactData aus shared-Helper nutzen (#293)