Skip to content

Commit

Permalink
feat: Ability to invite circles
Browse files Browse the repository at this point in the history
This commit adds support for adding circles as attendees to a calendar event.
The relationship between the imported group and members will be compliant with
the iCal specification.

A circle with the title "testcircle" will be added as an attendee with iCal
attributes "CUTYPE=GROUP` and uri "mailto:circle+CIRCLEID@CIRCLE_INSTANCE".

Members of the circle will be imported as standard attendees. Each member gets
assigned to the circle group entry by assigning them to the group uri using the
iCal member property: "MEMBER='mailto:circle+CIRCLEID@CIRCLE_INSTANCE'".

Searching for circles is only enabled if the circles app is activated.

Circles added to the list of attendees get imported only once and are not
synced yet. While adding a circle, a notice about this is shown to the user.
Only members of local circles which are local users get imported.

Rendering groups in the frontend is done in a separate PR
#5396

Signed-off-by: Jonas Heinrich <[email protected]>
  • Loading branch information
Jonas Heinrich committed Nov 7, 2023
1 parent 27c4402 commit 04eeeaa
Show file tree
Hide file tree
Showing 11 changed files with 347 additions and 17 deletions.
7 changes: 6 additions & 1 deletion appinfo/routes.php
Original file line number Diff line number Diff line change
Expand Up @@ -5,9 +5,12 @@
* Calendar App
*
* @author Georg Ehrke
* @copyright 2018 Georg Ehrke <[email protected]>
* @author Thomas Müller
* @author Jonas Heinrich
*
* @copyright 2018 Georg Ehrke <[email protected]>
* @copyright 2016 Thomas Müller <[email protected]>
* @copyright 2023 Jonas Heinrich <[email protected]>
*
* This library is free software; you can redistribute it and/or
* modify it under the terms of the GNU AFFERO GENERAL PUBLIC LICENSE
Expand Down Expand Up @@ -54,6 +57,8 @@
['name' => 'contact#searchAttendee', 'url' => '/v1/autocompletion/attendee', 'verb' => 'POST'],
['name' => 'contact#searchLocation', 'url' => '/v1/autocompletion/location', 'verb' => 'POST'],
['name' => 'contact#searchPhoto', 'url' => '/v1/autocompletion/photo', 'verb' => 'POST'],
// Circles
['name' => 'contact#getCircleMembers', 'url' => '/v1/circles/getmembers', 'verb' => 'GET'],
// Settings
['name' => 'settings#setConfig', 'url' => '/v1/config/{key}', 'verb' => 'POST'],
// Tools
Expand Down
78 changes: 77 additions & 1 deletion lib/Controller/ContactController.php
Original file line number Diff line number Diff line change
Expand Up @@ -7,10 +7,12 @@
* @author Georg Ehrke
* @author Jakob Röhrl
* @author Christoph Wurst
* @author Jonas Heinrich
*
* @copyright 2019 Georg Ehrke <[email protected]>
* @copyright 2019 Jakob Röhrl <[email protected]>
* @copyright 2019 Christoph Wurst <[email protected]>
* @copyright 2023 Jonas Heinrich <[email protected]>
*
* This library is free software; you can redistribute it and/or
* modify it under the terms of the GNU AFFERO GENERAL PUBLIC LICENSE
Expand All @@ -28,11 +30,15 @@
*/
namespace OCA\Calendar\Controller;

use OCA\Circles\Exceptions\CircleNotFoundException;
use OCP\App\IAppManager;
use OCP\AppFramework\Controller;
use OCP\AppFramework\Http;
use OCP\AppFramework\Http\JSONResponse;
use OCP\AppFramework\QueryException;
use OCP\Contacts\IManager;
use OCP\IRequest;
use OCP\IUserManager;

/**
* Class ContactController
Expand All @@ -43,6 +49,12 @@ class ContactController extends Controller {
/** @var IManager */
private $contactsManager;

/** @var IAppManager */
private $appManager;

/** @var IUserManager */
private $userManager;

/**
* ContactController constructor.
*
Expand All @@ -52,9 +64,13 @@ class ContactController extends Controller {
*/
public function __construct(string $appName,
IRequest $request,
IManager $contacts) {
IManager $contacts,
IAppManager $appManager,
IUserManager $userManager) {
parent::__construct($appName, $request);
$this->contactsManager = $contacts;
$this->appManager = $appManager;
$this->userManager = $userManager;
}

/**
Expand Down Expand Up @@ -173,6 +189,66 @@ public function searchAttendee(string $search):JSONResponse {
return new JSONResponse($contacts);
}

/**
* Query members of a circle by circleId
*
* @param string $circleId CircleId to query for members
* @return JSONResponse
* @throws Exception
* @throws \OCP\AppFramework\QueryException
*
* @NoAdminRequired
*/
public function getCircleMembers(string $circleId):JSONResponse {
if (!$this->appManager->isEnabledForUser('circles') || !class_exists('\OCA\Circles\Api\v1\Circles')) {
return new JSONResponse();
}
if (!$this->contactsManager->isEnabled()) {
return new JSONResponse();
}

try {
$circle = \OCA\Circles\Api\v1\Circles::detailsCircle($circleId, true);

Check failure on line 211 in lib/Controller/ContactController.php

View workflow job for this annotation

GitHub Actions / Nextcloud dev-stable26

UndefinedClass

lib/Controller/ContactController.php:211:14: UndefinedClass: Class, interface or enum named OCA\Circles\Api\v1\Circles does not exist (see https://psalm.dev/019)

Check failure on line 211 in lib/Controller/ContactController.php

View workflow job for this annotation

GitHub Actions / Nextcloud dev-master

UndefinedClass

lib/Controller/ContactController.php:211:14: UndefinedClass: Class, interface or enum named OCA\Circles\Api\v1\Circles does not exist (see https://psalm.dev/019)

Check failure on line 211 in lib/Controller/ContactController.php

View workflow job for this annotation

GitHub Actions / Nextcloud dev-stable25

UndefinedClass

lib/Controller/ContactController.php:211:14: UndefinedClass: Class, interface or enum named OCA\Circles\Api\v1\Circles does not exist (see https://psalm.dev/019)

Check failure on line 211 in lib/Controller/ContactController.php

View workflow job for this annotation

GitHub Actions / Nextcloud dev-stable27

UndefinedClass

lib/Controller/ContactController.php:211:14: UndefinedClass: Class, interface or enum named OCA\Circles\Api\v1\Circles does not exist (see https://psalm.dev/019)
} catch (QueryException $ex) {
return null;
} catch (CircleNotFoundException $ex) {
return null;
}

if (!$circle) {
return null;
}

$circleMembers = $circle->getInheritedMembers();

foreach ($circleMembers as $circleMember) {
if ($circleMember->isLocal()) {

$circleMemberUserId = $circleMember->getUserId();

$user = $this->userManager->get($circleMemberUserId);

if ($user === null) {
throw new ServiceException('Could not find organizer');
}

$contacts[] = [
'commonName' => $circleMember->getDisplayName(),
'calendarUserType' => 'INDIVIDUAL',
'email' => $user->getEMailAddress(),
'isUser' => true,
'avatar' => $circleMemberUserId,
'hasMultipleEMails' => false,
'dropdownName' => $circleMember->getDisplayName(),
'member' => 'mailto:circle+' . $circleId . '@' . $circleMember->getInstance(),
];
}
}

return new JSONResponse($contacts);
}


/**
* Get a contact's photo based on their email-address
*
Expand Down
14 changes: 14 additions & 0 deletions lib/Controller/ViewController.php
Original file line number Diff line number Diff line change
Expand Up @@ -6,8 +6,10 @@
*
* @author Georg Ehrke
* @author Richard Steinmetz <[email protected]>
* @author Jonas Heinrich <[email protected]>
* @copyright 2019 Georg Ehrke <[email protected]>
* @copyright Copyright (c) 2022 Informatyka Boguslawski sp. z o.o. sp.k., http://www.ib.pl/
* @copyright 2023 Jonas Heinrich <[email protected]>
*
* This library is free software; you can redistribute it and/or
* modify it under the terms of the GNU AFFERO GENERAL PUBLIC LICENSE
Expand All @@ -25,6 +27,7 @@
*/
namespace OCA\Calendar\Controller;

use OC\App\CompareVersion;
use OCA\Calendar\Service\Appointments\AppointmentConfigService;
use OCP\App\IAppManager;
use OCP\AppFramework\Controller;
Expand All @@ -51,6 +54,9 @@ class ViewController extends Controller {
/** @var IAppManager */
private $appManager;

/** @var CompareVersion */
private $compareVersion;

/** @var string */
private $userId;

Expand All @@ -62,13 +68,15 @@ public function __construct(string $appName,
AppointmentConfigService $appointmentConfigService,
IInitialState $initialStateService,
IAppManager $appManager,
CompareVersion $compareVersion,
?string $userId,
IAppData $appData) {
parent::__construct($appName, $request);
$this->config = $config;
$this->appointmentConfigService = $appointmentConfigService;
$this->initialStateService = $initialStateService;
$this->appManager = $appManager;
$this->compareVersion = $compareVersion;
$this->userId = $userId;
$this->appData = $appData;
}
Expand Down Expand Up @@ -117,6 +125,11 @@ public function index():TemplateResponse {
$talkApiVersion = version_compare($this->appManager->getAppVersion('spreed'), '12.0.0', '>=') ? 'v4' : 'v1';
$tasksEnabled = $this->appManager->isEnabledForUser('tasks');

$circleVersion = $this->appManager->getAppVersion('circles');
$isCirclesEnabled = $this->appManager->isEnabledForUser('circles') === true;
// if circles is not installed, we use 0.0.0
$isCircleVersionCompatible = $this->compareVersion->isCompatible($circleVersion ?? '0.0.0', '22');

Check failure on line 131 in lib/Controller/ViewController.php

View workflow job for this annotation

GitHub Actions / Nextcloud dev-stable26

RedundantCondition

lib/Controller/ViewController.php:131:68: RedundantCondition: Type string for $circleVersion is never null (see https://psalm.dev/122)

Check failure on line 131 in lib/Controller/ViewController.php

View workflow job for this annotation

GitHub Actions / Nextcloud dev-stable26

TypeDoesNotContainNull

lib/Controller/ViewController.php:131:86: TypeDoesNotContainNull: Cannot resolve types for $circleVersion - string does not contain null (see https://psalm.dev/090)

Check failure on line 131 in lib/Controller/ViewController.php

View workflow job for this annotation

GitHub Actions / Nextcloud dev-master

RedundantCondition

lib/Controller/ViewController.php:131:68: RedundantCondition: Type string for $circleVersion is never null (see https://psalm.dev/122)

Check failure on line 131 in lib/Controller/ViewController.php

View workflow job for this annotation

GitHub Actions / Nextcloud dev-master

TypeDoesNotContainNull

lib/Controller/ViewController.php:131:86: TypeDoesNotContainNull: Cannot resolve types for $circleVersion - string does not contain null (see https://psalm.dev/090)

Check failure on line 131 in lib/Controller/ViewController.php

View workflow job for this annotation

GitHub Actions / Nextcloud dev-stable25

RedundantCondition

lib/Controller/ViewController.php:131:68: RedundantCondition: Type string for $circleVersion is never null (see https://psalm.dev/122)

Check failure on line 131 in lib/Controller/ViewController.php

View workflow job for this annotation

GitHub Actions / Nextcloud dev-stable25

TypeDoesNotContainNull

lib/Controller/ViewController.php:131:86: TypeDoesNotContainNull: Cannot resolve types for $circleVersion - string does not contain null (see https://psalm.dev/090)

Check failure on line 131 in lib/Controller/ViewController.php

View workflow job for this annotation

GitHub Actions / Nextcloud dev-stable27

RedundantCondition

lib/Controller/ViewController.php:131:68: RedundantCondition: Type string for $circleVersion is never null (see https://psalm.dev/122)

Check failure on line 131 in lib/Controller/ViewController.php

View workflow job for this annotation

GitHub Actions / Nextcloud dev-stable27

TypeDoesNotContainNull

lib/Controller/ViewController.php:131:86: TypeDoesNotContainNull: Cannot resolve types for $circleVersion - string does not contain null (see https://psalm.dev/090)

$this->initialStateService->provideInitialState('app_version', $appVersion);
$this->initialStateService->provideInitialState('event_limit', $eventLimit);
$this->initialStateService->provideInitialState('first_run', $firstRun);
Expand All @@ -138,6 +151,7 @@ public function index():TemplateResponse {
$this->initialStateService->provideInitialState('disable_appointments', $disableAppointments);
$this->initialStateService->provideInitialState('can_subscribe_link', $canSubscribeLink);
$this->initialStateService->provideInitialState('show_resources', $showResources);
$this->initialStateService->provideInitialState('isCirclesEnabled', $isCirclesEnabled && $isCircleVersionCompatible);

return new TemplateResponse($this->appName, 'main');
}
Expand Down
5 changes: 5 additions & 0 deletions psalm.xml
Original file line number Diff line number Diff line change
Expand Up @@ -27,6 +27,8 @@
<referencedClass name="Doctrine\DBAL\Types\Type" />
<referencedClass name="Doctrine\DBAL\Types\Types" />
<referencedClass name="OC\Security\CSP\ContentSecurityPolicyNonceManager" />
<referencedClass name="OC\App\CompareVersion" />
<referencedClass name="OCA\Calendar\Controller\Exception" />
<referencedClass name="Psr\Http\Client\ClientExceptionInterface" />
<referencedClass name="Sabre\VObject\Component\VCalendar" />
<referencedClass name="Sabre\VObject\Component\VTimezone" />
Expand All @@ -48,6 +50,9 @@
<referencedClass name="Doctrine\DBAL\Schema\Table" />
<referencedClass name="OC\Security\CSP\ContentSecurityPolicyNonceManager" />
<referencedClass name="Sabre\VObject\Component\VTimezone" />
<referencedClass name="OC\App\CompareVersion" />
<referencedClass name="OCA\Calendar\Controller\Exception" />
<referencedClass name="OCA\Circles\Api\v1\Circles" />
<referencedClass name="Symfony\Component\Console\Output\OutputInterface" />
</errorLevel>
</UndefinedDocblockClass>
Expand Down
4 changes: 3 additions & 1 deletion src/components/Editor/Invitees/InviteesList.vue
Original file line number Diff line number Diff line change
Expand Up @@ -27,6 +27,7 @@
<div>
<InviteesListSearch v-if="!isReadOnly && !isSharedWithMe && hasUserEmailAddress"
:already-invited-emails="alreadyInvitedEmails"
:organizer="calendarObjectInstance.organizer"
@add-attendee="addAttendee" />
<OrganizerListItem v-if="hasOrganizer"
:is-read-only="isReadOnly || isSharedWithMe"
Expand Down Expand Up @@ -230,7 +231,7 @@ export default {
},
},
methods: {
addAttendee({ commonName, email, calendarUserType, language, timezoneId }) {
addAttendee({ commonName, email, calendarUserType, language, timezoneId, member }) {
this.$store.commit('addAttendee', {
calendarObjectInstance: this.calendarObjectInstance,
commonName,
Expand All @@ -242,6 +243,7 @@ export default {
language,
timezoneId,
organizer: this.$store.getters.getCurrentUserPrincipal,
member,

Check warning on line 246 in src/components/Editor/Invitees/InviteesList.vue

View check run for this annotation

Codecov / codecov/patch

src/components/Editor/Invitees/InviteesList.vue#L246

Added line #L246 was not covered by tests
})
},
removeAttendee(attendee) {
Expand Down
Loading

0 comments on commit 04eeeaa

Please sign in to comment.