From 49cddf9a0249eb084739662ca1f712cf791f1577 Mon Sep 17 00:00:00 2001 From: Kai Reinhard Date: Wed, 11 Dec 2024 22:26:41 +0100 Subject: [PATCH] WIP: Carddav --- .../kotlin/org/projectforge/carddav/DeleteRequestHandler.kt | 3 +-- .../org/projectforge/carddav/service/AddressService.kt | 6 +++--- 2 files changed, 4 insertions(+), 5 deletions(-) diff --git a/projectforge-carddav/src/main/kotlin/org/projectforge/carddav/DeleteRequestHandler.kt b/projectforge-carddav/src/main/kotlin/org/projectforge/carddav/DeleteRequestHandler.kt index 62ecdbe73b..fe8d03222b 100644 --- a/projectforge-carddav/src/main/kotlin/org/projectforge/carddav/DeleteRequestHandler.kt +++ b/projectforge-carddav/src/main/kotlin/org/projectforge/carddav/DeleteRequestHandler.kt @@ -48,13 +48,12 @@ internal class DeleteRequestHandler { log.debug { "handleDeleteCall: ${requestWrapper.request.method}: '${requestWrapper.requestURI}' body=[${requestWrapper.body}]" } val requestedPath = requestWrapper.requestURI val contactId = CardDavUtils.extractContactId(requestedPath) - val contact = contactList.find { it.id == contactId } if (contactId == null) { log.info { "Contact with id=$contactId not found in personal contact list. Can't delete it." } ResponseUtils.setValues(response, content = "The resource does not exist.", status = HttpStatus.NOT_FOUND) return } - if (addressService.deleteContact(contactId, contact)) { + if (addressService.deleteContact(contactId)) { ResponseUtils.setValues(response, HttpStatus.NO_CONTENT) } else { ResponseUtils.setValues(response, content = "The resource does not exist.", status = HttpStatus.NOT_FOUND) diff --git a/projectforge-carddav/src/main/kotlin/org/projectforge/carddav/service/AddressService.kt b/projectforge-carddav/src/main/kotlin/org/projectforge/carddav/service/AddressService.kt index dc76d3689d..67f245555e 100644 --- a/projectforge-carddav/src/main/kotlin/org/projectforge/carddav/service/AddressService.kt +++ b/projectforge-carddav/src/main/kotlin/org/projectforge/carddav/service/AddressService.kt @@ -84,17 +84,17 @@ class AddressService { return Contact() } */ - fun deleteContact(contactId: Long, contact: Contact?): Boolean { + fun deleteContact(contactId: Long): Boolean { try { val personalAddress = personalAddressDao.getByAddressId(contactId) if (personalAddress?.isFavorite == true) { personalAddress.isFavoriteCard = false personalAddressDao.saveOrUpdate(personalAddress) - log.info("Contact #$contactId '${contact?.displayName}' removed from favorite list.") + log.info("Contact #$contactId removed from favorite list.") } return true } catch (e: Exception) { - log.error(e) { "Exception while deleting contact: id=${contactId}, name=[${contact?.displayName}]" } + log.error(e) { "Exception while deleting contact: id=${contactId}." } return false } }