summaryrefslogtreecommitdiffstats
path: root/apps/dav/lib/CalDAV
Commit message (Collapse)AuthorAgeFilesLines
...
* Use the new executeQuery and executeUpdate methodsChristoph Wurst2021-04-231-54/+54
| | | | Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* Drop default fetch mode parameterChristoph Wurst2021-04-231-15/+15
| | | | Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* change calendar invite emails to use pngs instead of svgsNick Sweeting2021-03-261-7/+7
| | | | Signed-off-by: Nick Sweeting <git@sweeting.me>
* Merge pull request #22201 from cweiske/20492-move-contacts-with-birthdayJohn Molakvoæ2021-03-261-3/+17
|\ | | | | Delete old birthday calendar object when moving contact to another ad…
| * Delete old birthday calendar object when moving contact to another address bookChristian Weiske2021-03-261-3/+17
| | | | | | | | | | | | | | | | | | | | | | | | | | When an contact is moved to another address book, the contact is copied to the second address book. During copying, the birthday event is created - but it gets the same UID as the contact's birthday event in the first address book. To prevent the "Calendar object with uid already exists" error that followed, we need to delete the old entry before the new one is created. Resolves: https://github.com/nextcloud/server/issues/20492 Signed-off-by: Christian Weiske <cweiske@cweiske.de>
* | Fix l10nValdnet2021-03-251-1/+1
|/ | | Issue: #26286
* Bump nextcloud/coding-standard from 0.3.0 to 0.5.0dependabot-preview[bot]2021-02-186-16/+16
| | | | | | | | | | Bumps [nextcloud/coding-standard](https://github.com/nextcloud/coding-standard) from 0.3.0 to 0.5.0. - [Release notes](https://github.com/nextcloud/coding-standard/releases) - [Changelog](https://github.com/nextcloud/coding-standard/blob/master/CHANGELOG.md) - [Commits](https://github.com/nextcloud/coding-standard/compare/v0.3.0...v0.5.0) Signed-off-by: dependabot-preview[bot] <support@dependabot.com> Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* Fix some RedundantCast in dav codeRoeland Jago Douma2021-02-151-2/+2
| | | | | | For #25641 Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
* Merge pull request #25595 from ↵Roeland Jago Douma2021-02-151-13/+34
|\ | | | | | | | | nextcloud/enh/caldavbackend/getChangesForCalendar_querybuilder Move getChangesForCalendar to QueryBuilder
| * Move getChangesForCalendar to QueryBuilderRoeland Jago Douma2021-02-151-13/+34
| | | | | | | | Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
* | Merge pull request #25625 from nextcloud/enh/psalm/substrChristoph Wurst2021-02-152-3/+6
|\ \ | | | | | | Check substr results
| * | Check substr resultsRoeland Jago Douma2021-02-152-3/+6
| |/ | | | | | | | | | | | | Else this might lead to unexpeted errors. Found by psalm. Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
* | Merge pull request #25636 from nextcloud/fix/psalm/closure_return_typesChristoph Wurst2021-02-151-1/+1
|\ \ | |/ |/| Add some closure return types
| * Add some closure return typesRoeland Jago Douma2021-02-141-1/+1
| | | | | | | | | | | | Found by psalm Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
* | Use correct namespace (casing) in CalendarHomeRoeland Jago Douma2021-02-141-1/+1
|/ | | | Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
* Do not send imip email to invalid recipientsChristoph Wurst2021-02-111-0/+5
| | | | Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* [CalDAV] Validate notified emailsThomas Citharel2021-01-251-0/+9
| | | | Signed-off-by: Thomas Citharel <tcit@tcit.fr>
* Merge pull request #25114 from ↵Roeland Jago Douma2021-01-141-0/+7
|\ | | | | | | | | nextcloud/dismiss-reminder-notifications-from-passed-events Dismiss reminder notifications from passed events
| * Dismiss reminder notifications from passed eventsThomas Citharel2021-01-141-0/+7
| | | | | | | | Signed-off-by: Thomas Citharel <tcit@tcit.fr>
* | Remove unneeded casts that were found by PsalmMorris Jobke2021-01-114-10/+10
|/ | | | | | In preparation of the update of Psalm from 4.2.1 to 4.3.1+ (see https://github.com/nextcloud/server/pull/24521) Signed-off-by: Morris Jobke <hey@morrisjobke.de>
* Bump doctrine/dbal from 2.12.0 to 3.0.0Christoph Wurst2021-01-081-4/+4
| | | | Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* Merge pull request #24911 from ↵Julius Härtl2021-01-071-2/+4
|\ | | | | | | | | nextcloud/fix/tasks-calendar/undefined-index-search Tasks search: make sure we have existing indexes before using them
| * Make sure we have existing indexes before using themJohn Molakvoæ (skjnldsv)2021-01-041-2/+4
| | | | | | | | Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>
* | Merge pull request #24964 from ↵Roeland Jago Douma2021-01-051-1/+6
|\ \ | |/ |/| | | | | nextcloud/bugfix/noid/internal-server-error-when-importing-broken-ics-file Don't throw a 500 when importing a broken ics reminder file
| * Don't throw a 500 when importing a broken ics reminder fileJoas Schilling2021-01-051-1/+6
| | | | | | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* | Update license headersChristoph Wurst2020-12-301-0/+1
|/ | | | Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* Merge pull request #24604 from nextcloud/feat/dav/move-calendar-renameJohn Molakvoæ2020-12-221-1/+3
|\ | | | | Allow to force rename a conflicting calendar
| * Allow to force rename a conflicting calendarJohn Molakvoæ (skjnldsv)2020-12-101-1/+3
| | | | | | | | Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>
* | Merge pull request #24515 from nextcloud/fix/noid/dav-share-groups-special-charsChristoph Wurst2020-12-181-3/+0
|\ \ | | | | | | fixes sharing to group ids with characters that are being url encoded
| * | fixes sharing to group ids with characters that are being url encodedArthur Schiwon2020-12-161-3/+0
| |/ | | | | | | Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
* / Update all license headers for Nextcloud 21Christoph Wurst2020-12-165-2/+6
|/ | | | Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* Use correct year for generated birthdays eventsSven Strickroth2020-11-131-1/+6
| | | | | | | | 'X-APPLE-OMIT-YEAR' is not always present, at least iOS 12.4 uses the hard coded date of 1604 (the start of the gregorian calendar) when the year is unknown. cf. https://forums.bitfire.at/topic/2050/bday-without-year-for-vcard-3-0-if-server-drops-property-x-apple-omit-year and https://gitlab.com/CardBook/CardBook/-/issues/586 Signed-off-by: Sven Strickroth <email@cs-ware.de>
* Merge pull request #23915 from ↵Morris Jobke2020-11-111-4/+4
|\ | | | | | | | | st3iny/fix/noid/png-icons-in-caldav-reminder-emails Use png icons in caldav reminder emails
| * Use png icons in caldav reminder emailsRichard Steinmetz2020-11-071-4/+4
| | | | | | | | Signed-off-by: Richard Steinmetz <richard@steinmetz.cloud>
* | Fix public calendars as they are stored with null on oracleJoas Schilling2020-11-101-12/+34
| | | | | | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* | Merge pull request #23993 from nextcloud/bugfix/noid/close-cursorsMorris Jobke2020-11-101-2/+7
|\ \ | | | | | | Don't leave cursors open
| * | Don't leave cursors open when tests failJoas Schilling2020-11-091-2/+7
| |/ | | | | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* | resolve review commentsleith abdulla2020-10-291-2/+2
| | | | | | | | | | | | | | | | update styling for conditionals and simplifying conditional by only relying on dtstart instead of also counting vevents (since they imply the same condition, assuming valid events) Signed-off-by: leith abdulla <online-nextcloud@eleith.com>
* | handle vcalander documents with recurrence with exemptions or editsleith abdulla2020-10-281-8/+24
|/ | | | | | | | | | | | | | | | | | when importing ics files from third party services (ex: google), long running recurrence vcalander objects can contain numerous vevents relating to past exemptions or edits. these objects are handled incorrectly, leading to incorrect first and/or last occurence dates from being inferred. as a result, a number of imported events are not rendered in the calendar. this PR makes use of EventIterator's constructor that takes in an array of VEVENTS to properly handle these complex cases. Also of note is that other parameter signatures of EventIterator's constructor, will be deprecated, according to comments in sabre dav EventIterator.php Signed-off-by: leith abdulla <online-nextcloud@eleith.com>
* CalDavBackend: check if timerange is array before accessingSimon Spannagel2020-10-161-1/+2
| | | | Signed-off-by: Simon Spannagel <simonspa@kth.se>
* Merge pull request #23028 from ↵Morris Jobke2020-10-051-1/+1
|\ | | | | | | | | nextcloud/bugfix/noid/array-to-string-conversion-on-event-search Fix array to string conversion on event search
| * Fix array to string conversion on event searchJoas Schilling2020-09-241-1/+1
| | | | | | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* | Format code to a single space around binary operatorsChristoph Wurst2020-10-0516-125/+125
| | | | | | | | Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* | broaden exception handling on webcal refresheleith2020-09-291-1/+3
|/ | | | | | | | | | | | | | | | | when iterating through a calendar, recurrance events can throw an exception if no instances of the recurrance are found. this exception is of class `Exception` but the try/catch clause in the webcal refresh loop only catches `BadRequest` exception. this leads to the exception bubbling up and thus other calendar events do not get processed by the event iterator. this PR broadens the exception to handle both BadRequest and NoInstanceFoundException so that the full webcal can be processed, even if minor hiccups are processing on vobject Signed-off-by: leith abdulla <online-nextcloud@eleith.com>
* Remove @suppress SqlInjectionCheckerMorris Jobke2020-09-161-7/+0
| | | | Signed-off-by: Morris Jobke <hey@morrisjobke.de>
* Remove phan config - was replaced by PsalmMorris Jobke2020-09-162-8/+0
| | | | Signed-off-by: Morris Jobke <hey@morrisjobke.de>
* Update license headers for Nextcloud 20 (again)Christoph Wurst2020-09-075-1/+8
| | | | | | | There are still lots of outdated headers, so time for another round of updates. Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* Merge pull request #17456 from ↵John Molakvoæ2020-09-041-54/+132
|\ | | | | | | brad2014/feature/brad2014/12391-improve-imip-mail-message-take-2
| * Minor cleanup: php-cs-fixer, tests, interface consistencyBrad Rubenstein2020-09-021-15/+17
| | | | | | | | | | | | | | | | | | | | | | IMipPlugin.php Removed blank lines to make php-cs-fixer happy. Minor cleanup: bugs found by Psalm static checker IEMailTemplate: The public interface to addBodyListItem also needs to include the new plainIndent parameter. IMipPlugin: Fixes an undefined variable for events that do not have DTEND. Also use explicit string conversion for parameters and properties in several places. The new email template adds an additional blank line before "button" links in plain text, so the tests were fixed to include that additional blank line. Signed-off-by: Brad Rubenstein <brad@wbr.tech>
| * Make icon set for CalDav event presentation, use in iMIP email.brad20142020-08-201-7/+7
| | | | | | | | | | | | | | | | | | | | iMIP e-mails were, as a hack, using filetypes icons for caldav elements (titles, locations). This commit creates a folder of caldav element icons. To start, they are used in iMIP emails, but eventually should be used by any app that wants to have icon labels for caldav elements. Signed-off-by: brad2014 <brad2014@users.noreply.github.com>