summaryrefslogtreecommitdiffstats
path: root/apps/dav/lib/CardDAV
Commit message (Collapse)AuthorAgeFilesLines
* fix(CardDAV): allow disabling of the system address book via config optionAnna Larch2023-09-051-1/+2
| | | | Signed-off-by: Anna Larch <anna@nextcloud.com>
* fix(CardDAV): set owner-principal, displayname properties for SABAnna Larch2023-08-242-4/+3
| | | | Signed-off-by: Anna Larch <anna@nextcloud.com>
* enh(contacts): show/hide addressbooks for allJohannes Merkel2023-08-161-10/+14
| | | | Signed-off-by: Johannes Merkel <mail@johannesgge.de>
* Merge pull request #38919 from nextcloud/backport/38639/stable27Arthur Schiwon2023-07-121-2/+13
|\ | | | | [stable27] pruneOutdatedSyncTokens deletes all entries
| * fix(dav): close cursor when fetching max idRichard Steinmetz2023-07-111-1/+3
| | | | | | | | Signed-off-by: Richard Steinmetz <richard@steinmetz.cloud>
| * Issue #36644: Fix pruneOutdatedSyncTokens for CalDAVChristof Arnosti2023-06-211-2/+11
| | | | | | | | | | | | | | pruneOutdatedSyncTokens accidentally deletes all entries of the calendarchanges table instead of leaving $limit elements in the table Signed-off-by: Christof Arnosti <charno@charno.ch>
* | Merge pull request #38475 from nextcloud/backport/38454/stable27Arthur Schiwon2023-07-121-1/+1
|\ \ | | | | | | [stable27] fix(carddav): Make system contact phone number RFC compliant
| * | fix(carddav): Make system contact phone number RFC compliantgreta2023-07-101-1/+1
| | | | | | | | | | | | Signed-off-by: greta <gretadoci@gmail.com>
* | | Merge pull request #38792 from nextcloud/backport/38777/stable27Arthur Schiwon2023-07-101-1/+1
|\ \ \ | |/ / |/| | [stable27] fix(carddav): Make SystemAddressBook::__construct $groupManager argument nullable
| * | fix(carddav): Make SystemAddressBook::__construct $groupManager argument ↵Christoph Wurst2023-06-131-1/+1
| |/ | | | | | | | | | | nullable Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* / fix(sab): put location in proper address fieldJohannes Merkel2023-07-071-1/+13
|/ | | | | Signed-off-by: Johannes Merkel <mail@johannesgge.de> Signed-off-by: nextcloud-command <nextcloud-command@users.noreply.github.com>
* Merge pull request #38416 from nextcloud/backport/38308/stable27Joas Schilling2023-05-301-5/+4
|\ | | | | [stable27] fix(carddav): Check if SERVER variables are set before accessing them
| * fix(carddav): Check if SERVER variables are set before accessing themAnna Larch2023-05-231-5/+4
| | | | | | | | Signed-off-by: Anna Larch <anna@nextcloud.com>
* | fix(dav): Fix avatar size in system address bookJoas Schilling2023-05-261-1/+1
| | | | | | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* | fix(carddav): Don't show system address book cards to guestsChristoph Wurst2023-05-251-8/+6
| | | | | | | | Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* | fix(carddav): Check enumeration settings for all SAB methodsChristoph Wurst2023-05-251-1/+69
| | | | | | | | Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* | fix(SAB): Save avatar to SABJohannes Merkel2023-05-231-1/+1
|/ | | | Signed-off-by: Johannes Merkel <mail@johannesgge.de>
* feat(CardDAV): Add Sabre\DAV\IMoveTarget support to OCA\DAV\CardDAV\AddressBookThomas Citharel2023-05-163-2/+153
| | | | | | | | | | This allows to just UPDATE the card row instead of deleting it and reinsert it. It's very similar to https://github.com/nextcloud/server/pull/30120 for calendars. As we need the addressbookid exposed, this introduces OCA\DAV\CardDAV\Card that extends Sabre's. I chose specifically NOT to auto-inject LoggerInterface in Addressbook like in #30120 because the chain of DI is huge just for ONE simple call and it would break an existing dirty call (OCA\Contacts calling OCA\DAV) of ContactsManager in Contacts: https://github.com/nextcloud/contacts/pull/1722 (in SocialApiService), but this is debatable. Signed-off-by: Thomas Citharel <tcit@tcit.fr>
* Merge pull request #37965 from ↵Simon L2023-05-161-26/+41
|\ | | | | | | | | nextcloud/fix/transactional-system-addressbook-sync fix(dav): Run system address book create-if-not-exists in transaction
| * fix(dav): Run system address book create-if-not-exists in transactionChristoph Wurst2023-05-121-26/+41
| | | | | | | | Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* | fix(carddav): Mark system address book as read-onlyChristoph Wurst2023-05-151-0/+10
| | | | | | | | Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* | feat(users): Store and load a user's managerChristoph Wurst2023-05-121-1/+19
| | | | | | | | | | Co-Authored-By: hamza221 <hamzamahjoubi221@gmail.com> Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* | fix(carddav): expose system address bookAnna Larch2023-05-115-22/+146
| | | | | | | | Signed-off-by: Anna Larch <anna@nextcloud.com>
* | feat(dav): store scopes for properties and filter locally scoped properties ↵Anna Larch2023-05-093-54/+235
|/ | | | | | for federated address book sync Signed-off-by: Anna Larch <anna@nextcloud.com>
* feat(carddav): Map user's additional emails into the SAB cardChristoph Wurst2023-05-041-1/+10
| | | | Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* enh(contacts): write profile prop and backend prop to system addressbookJohannes Merkel2023-05-021-0/+6
| | | | Signed-off-by: Johannes Merkel <mail@johannesgge.de>
* refactor(dav): Since we're in a transaction, use QB properly when ↵Thomas Citharel2023-04-171-13/+24
| | | | | | | | incrementing synctoken Now that we're in a transaction, we can reuse the sync token's previous value without trouble, and rewrite the increment UPDATE query without dirty direct SQL. Signed-off-by: Thomas Citharel <tcit@tcit.fr>
* fix(CalDAV/CardDAV): put every method from Cal/CardDAV backends that does ↵Thomas Citharel2023-04-171-316/+330
| | | | | | | | | | | | multiple DB calls in transactions In a lot of methods we're doing read-after-writes (for instance calling updateProperties after touching calendar objects). There's also a lot of deleting methods that do stuff sequentially which could cause trouble. This should avoid this kind of issues. Signed-off-by: Thomas Citharel <tcit@tcit.fr>
* Fix DAV typesjld31032023-04-051-1/+2
| | | | Signed-off-by: jld3103 <jld3103yt@gmail.com>
* Use implementations instead of interfaces for accessing private methodsjld31032023-03-301-1/+1
| | | | Signed-off-by: jld3103 <jld3103yt@gmail.com>
* Do not reuse query builder objects in DAV account deletionJoas Schilling2023-02-011-0/+3
| | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* Revert "Fix sync errors for duplicate cards with the same UID for different ↵Christopher Ng2023-01-132-4/+2
| | | | | | | | backends" This reverts commit ddcee3dbab01b4f4fc163c68a54055facfa8af16. Signed-off-by: Christopher Ng <chrng8@gmail.com>
* Fix sync errors for duplicate cards with the same UID for different backendsAnna Larch2023-01-052-2/+4
| | | | Signed-off-by: Anna Larch <anna@nextcloud.com>
* Merge pull request #34807 from nextcloud/fix/compute-notification-parsed-subjectCôme Chilliet2022-12-191-15/+1
|\ | | | | Compute notification parsed subject from rich subject when possible
| * Get rid of all useless calls to setParsedSubject now that setRichSubject is ↵Côme Chilliet2022-11-291-15/+1
| | | | | | | | | | | | able to do it Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
* | Cleanup contactsinteractionCarl Schwan2022-11-251-9/+2
|/ | | | | | - Add more typing and use PHP 7.4 typed properties Signed-off-by: Carl Schwan <carl@carlschwan.eu>
* Switch to DisplayNameCache for CardDAVAnna Larch2022-11-031-15/+1
| | | | Signed-off-by: Anna Larch <anna@nextcloud.com>
* Add a background job to prune outdated sync tokensThomas Citharel2022-10-271-0/+14
| | | | | | | | We remove all outdated sync tokens, based on their auto-incremented ID. By default we only keep the last 10 000, but this can be configurable. Signed-off-by: Thomas Citharel <tcit@tcit.fr>
* Wrap address book creation write-read in a transactionChristoph Wurst2022-10-031-13/+23
| | | | Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* Merge pull request #33104 from nextcloud/enhancement/add-logging-federationblizzz2022-10-011-1/+3
|\ | | | | Federation - add logging, update the server status if sync fails
| * Add logging to federationAnna Larch2022-07-271-1/+3
| | | | | | | | Signed-off-by: Anna Larch <anna@nextcloud.com>
* | Merge pull request #33450 from sazanof/bug/base64-card-dav-searchblizzz2022-10-011-1/+1
|\ \ | | | | | | Fix photo url generation (add URI type to VCard2Array)
| * | Add URI type to VCardMikhail Sazanov2022-08-031-1/+1
| |/ | | | | | | Signed-off-by: Mikhail Sazanov <m@sazanof.ru>
* | Check address book URI length before creationJoas Schilling2022-08-261-0/+4
| | | | | | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* | Merge pull request #33615 from ↵Joas Schilling2022-08-191-21/+1
|\ \ | | | | | | | | | | | | nextcloud/perf/noid/user-displayname-cache-for-activity-providers Use user name cache in activity providers
| * | Use user name cache in activity providersJoas Schilling2022-08-191-21/+1
| | | | | | | | | | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* | | Fix carddav activitiesJoas Schilling2022-08-193-5/+5
|/ / | | | | | | | | | | | | | | The settings where combined last minute but at the same time the activities where not adjusted to map an existing setting so the filter was not possible to even limit it to the types that the activities had. Signed-off-by: Joas Schilling <coding@schilljs.com>
* / performance optimisation for carddav backend blob data readAnna Larch2022-08-151-1/+6
|/ | | | Signed-off-by: Anna Larch <anna@nextcloud.com>
* Fix typos in apps/dav subdirectoryluz paz2022-07-261-1/+1
| | | | | | Found via `codespell -q 3 -S l10n -L jus ./apps/dav` Signed-off-by: luz paz <luzpaz@github.com>
* Merge pull request #32869 from nextcloud/dav-no-deleted-user-activitiesblizzz2022-07-231-1/+12
|\ | | | | Avoid calendar & addressbook activities being created on user being deleted