summaryrefslogtreecommitdiffstats
path: root/apps/dav/lib
Commit message (Collapse)AuthorAgeFilesLines
* Merge pull request #32876 from nextcloud/fix/fix-string-interpolationblizzz2022-06-151-1/+1
|\ | | | | Use {$var} instead of ${var} for PHP 8.2 compatibility
| * Use {$var} instead of ${var} for PHP 8.2 compatibilityCôme Chilliet2022-06-141-1/+1
| | | | | | | | Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
* | Handle the move operation properly between shared calendarsThomas Citharel2022-06-149-11/+310
|/ | | | | | | | - Introduce a new CalendarObjectMovedEvent typed event dedicated for this operation - Handle the event in the activity backend and add new appropriate activity subjects Signed-off-by: Thomas Citharel <tcit@tcit.fr>
* Merge pull request #30368 from nextcloud/dav-allow-object-propertiesVincent Petry2022-06-103-12/+181
|\ | | | | Allow DAV Object properties
| * Allow DAV Object propertiesThomas Citharel2022-05-163-12/+181
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The current implementation only saves them as string. It seems they can be more complex than that, and that objects were saved directly. You may find such objects saved in some production databases by executing: ```sql SELECT * from oc_properties where propertyvalue = 'Object'; ``` This commit adds a repair job to clean all of these "broken" properties values, adds a new database column to save the type of the property, and handles converting from and to correct values. Implementation is very similar to SabreDAV's own PDO backend: https://github.com/nextcloud/3rdparty/blob/4921806dfb1c5c309eac60195ed34e2749baf3c1/sabre/dav/lib/DAV/PropertyStorage/Backend/PDO.php Signed-off-by: Thomas Citharel <tcit@tcit.fr>
* | Merge pull request #31029 from nextcloud/expose-extra-emails-in-davVincent Petry2022-06-103-0/+20
|\ \ | | | | | | Expose additional emails in {DAV:}alternate-URI-set
| * | Expose additional emails in {DAV:}alternate-URI-setThomas Citharel2022-05-173-0/+20
| | | | | | | | | | | | | | | | | | | | | | | | | | | This allows iMip invitations to be send with an alternative email as "Reply-To" field. Closes #27201 Signed-off-by: Thomas Citharel <tcit@tcit.fr>
* | | Merge pull request #31622 from ↵Vincent Petry2022-06-101-34/+16
|\ \ \ | | | | | | | | | | | | | | | | nextcloud/increase-loglevel-modernize-refresh-webcal Increase loglevel of Webcal parsing errors and modernize code
| * | | Increase loglevel of Webcal parsing errors and modernize codeThomas Citharel2022-05-171-34/+16
| |/ / | | | | | | | | | | | | | | | Closes #31612 Signed-off-by: Thomas Citharel <tcit@tcit.fr>
* | | Merge pull request #32361 from nextcloud/dav-scheduling-default-calendarVincent Petry2022-06-103-7/+48
|\ \ \ | | | | | | | | Put calendar invites into the user's first available calendar
| * | | Put calendar invites into the user's first available calendarThomas Citharel2022-05-123-7/+48
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | If there's no default calendar and we can't find anything with URI 'personal', instead of creating a new one, start by using the first "real personal calendar" available. If not, then we create the default one. Signed-off-by: Thomas Citharel <tcit@tcit.fr>
* | | | Merge pull request #32364 from nextcloud/dav-listenersVincent Petry2022-06-1014-270/+371
|\ \ \ \ | | | | | | | | | | 🧹 Remove all legacy event dispatchers from CalDAV & CardDAV backends
| * | | | Fix a doctype in OCP for IAddressBookThomas Citharel2022-05-171-1/+2
| | | | | | | | | | | | | | | | | | | | Signed-off-by: Thomas Citharel <tcit@tcit.fr>
| * | | | Add some typed propertiesThomas Citharel2022-05-174-79/+30
| | | | | | | | | | | | | | | | | | | | Signed-off-by: Thomas Citharel <tcit@tcit.fr>
| * | | | Remove all legacy event dispatchers from CalDAV & CardDAV backendsThomas Citharel2022-05-1711-190/+339
| | |/ / | |/| | | | | | | | | | | | | | | | | | Move them to proper EventListeners and test them Signed-off-by: Thomas Citharel <tcit@tcit.fr>
* | | | Merge pull request #32800 from ↵Vincent Petry2022-06-103-9/+38
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | nextcloud/bugfix/noid/no-fatal-for-maintenance-mode Don't spam the log with fatals when maintenance mode is enabled or an…
| * | | | Don't spam the log with fatals when maintenance mode is enabled or an ↵Joas Schilling2022-06-103-9/+38
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | upgrade has to be done Signed-off-by: Joas Schilling <coding@schilljs.com>
* | | | | Avoid creating carddav activities for the system userThomas Citharel2022-06-101-2/+16
|/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | Because of the system addressbook synchronisation. Closes #32803 Signed-off-by: Thomas Citharel <tcit@tcit.fr>
* | | | Merge pull request #28703 from nextcloud/enhancement/caldav-reminders-loggingChristoph Wurst2022-06-021-13/+28
|\ \ \ \ | | | | | | | | | | Debug log what happens during reminders processing
| * | | | Debug log what happens during reminders processingChristoph Wurst2022-06-011-13/+28
| | | | | | | | | | | | | | | | | | | | Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* | | | | Use Image class from public APIChristopher Ng2022-06-021-1/+1
| | | | | | | | | | | | | | | | | | | | Signed-off-by: Christopher Ng <chrng8@gmail.com>
* | | | | Merge pull request #32471 from nextcloud/fix-wrong-dav-activitiesChristoph Wurst2022-06-011-2/+3
|\ \ \ \ \ | | | | | | | | | | | | Make sure activities are not created when a deleted calendar object expires
| * | | | | Make sure activities are not created when a deleted calendar object expiresThomas Citharel2022-05-181-2/+3
| | |/ / / | |/| | | | | | | | | | | | | | | | | | | | | | | Closes https://github.com/nextcloud/activity/issues/784 Signed-off-by: Thomas Citharel <tcit@tcit.fr>
* | | | | Make X-HAS-{MD5/SHA256} opt-inCarl Schwan2022-06-011-9/+20
| |/ / / |/| | | | | | | | | | | | | | | | | | | This is not always needed and slow down the upload Signed-off-by: Carl Schwan <carl@carlschwan.eu>
* | | | Update calendar estimationChristopher Ng2022-05-301-6/+17
| | | | | | | | | | | | | | | | Signed-off-by: Christopher Ng <chrng8@gmail.com>
* | | | Update contacts estimationChristopher Ng2022-05-301-7/+14
| | | | | | | | | | | | | | | | Signed-off-by: Christopher Ng <chrng8@gmail.com>
* | | | Update method nameChristopher Ng2022-05-302-2/+2
| | | | | | | | | | | | | | | | Signed-off-by: Christopher Ng <chrng8@gmail.com>
* | | | Move new IMigrator method to a specific interface ISizeEstimationMigratorCôme Chilliet2022-05-302-2/+4
| | | | | | | | | | | | | | | | Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
* | | | Implement getExportEstimatedSize in migratorsCôme Chilliet2022-05-302-0/+30
| | | | | | | | | | | | | | | | Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
* | | | Merge pull request #32580 from ↵Joas Schilling2022-05-251-2/+2
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | nextcloud/bugfix/noid/fix-handling-events-without-name Fix handling events without names in activities
| * | | | Also trim the nameJoas Schilling2022-05-251-1/+1
| | | | | | | | | | | | | | | | | | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
| * | | | Fix handling events without names in activitiesJoas Schilling2022-05-241-2/+2
| | | | | | | | | | | | | | | | | | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* | | | | Merge pull request #32548 from nextcloud/fix/return-typeCarl Schwan2022-05-241-8/+6
|\ \ \ \ \ | |/ / / / |/| | | | Fix return type and make type stricter
| * | | | Fix return type and make type stricterCarl Schwan2022-05-241-8/+6
| | | | | | | | | | | | | | | | | | | | Signed-off-by: Carl Schwan <carl@carlschwan.eu>
* | | | | Rename user-agent used by the Webcal Refresh ServiceThomas Citharel2022-05-201-1/+1
|/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | Some services don't like the "crawler" inside the previous user-agent. Closes https://github.com/nextcloud/calendar/issues/4232 Signed-off-by: Thomas Citharel <tcit@tcit.fr>
* / / / Add email validationAnna Larch2022-05-181-2/+9
|/ / / | | | | | | | | | Signed-off-by: Anna Larch <anna@nextcloud.com>
* | | Merge pull request #31943 from nextcloud/dav-cleanupCarl Schwan2022-05-1733-569/+211
|\ \ \ | |_|/ |/| | Cleanup dav
| * | Fix more psalm issuesCarl Schwan2022-05-1625-186/+89
| | | | | | | | | | | | Signed-off-by: Carl Schwan <carl@carlschwan.eu>
| * | Cleanup davCarl Schwan2022-05-0520-398/+137
| |/ | | | | | | | | | | | | - Remove unused class AppEnabledPlugin - Add more type hinting when possible Signed-off-by: Carl Schwan <carl@carlschwan.eu>
* | Wrap multiple inserts inside a transationCarl Schwan2022-05-121-13/+21
| | | | | | | | Signed-off-by: Carl Schwan <carl@carlschwan.eu>
* | Minor optimizations for saving user personal informationCarl Schwan2022-05-123-19/+24
|/ | | | | | | | | | | | | | | | * Remove double hook: the OC_User::changeUser triggers an OC\AccountManager::userUpdated and the app is already listening to this signal in its Application definition * Make createCard not check if an card exists if we already checked previously. We also don't try to get the card if the user is disabled as we don't use the card in this case We this change we go from 100 DB requests to 80 DB requests when saving an user email address. Signed-off-by: Carl Schwan <carl@carlschwan.eu> (cherry picked from commit c6fd482edf33214a9ad4787e4cac278f871fa7c8)
* Merge pull request #32262 from nextcloud/fix/noid/extra-leading-minusCarl Schwan2022-05-041-1/+1
|\ | | | | Remove leading "minus"
| * remove leading minus which thankfully crashes because this operand does not ↵Julien Veyssier2022-05-041-1/+1
| | | | | | | | | | | | apply to PsrLoggerAdapter Signed-off-by: Julien Veyssier <eneiluj@posteo.net>
* | Merge pull request #31963 from nextcloud/feat/use_setting_in_dav_searchLouis2022-05-041-3/+6
|\ \ | |/ |/| Use share setting in DAV search
| * Use email settings in DAV searchLouis Chemineau2022-04-191-1/+2
| | | | | | | | Signed-off-by: Louis Chemineau <louis@chmn.me>
| * Use share setting in DAV searchLouis Chemineau2022-04-191-2/+4
| | | | | | | | | | | | shareapi_restrict_user_enumeration_full_match_ignore_second_display_name was introduced to ignore second display name during search from the share panel. But this setting was not respected by search from the calendar application. This fix it. Signed-off-by: Louis Chemineau <louis@chmn.me>
* | Fix use of $logger from another scope in apps/dav/lib/Server.phpCôme Chilliet2022-05-021-0/+1
| | | | | | | | Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
* | Add missing use for LoggerInterfaceCôme Chilliet2022-05-021-0/+1
| | | | | | | | Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
* | Migrate last instances of $server::getLogger() from davCôme Chilliet2022-05-022-2/+4
| | | | | | | | Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
* | Migrate dav application from ILogger to LoggerInterfaceCôme Chilliet2022-05-0226-257/+128
| | | | | | | | Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
generated by cgit v1.2.3 (git 2.39.1) at 2025-01-22 05:35:54 +0000