[Feature] Add calendar public flag option (#200)

Author: 1Luc1

.env

@@ -69,6 +69,9 @@ CALDAV_ENABLED=true
 CARDDAV_ENABLED=true
 WEBDAV_ENABLED=false
 
+# Can calendar be public available ?
+PUBLIC_CALENDAR_ENABLED=true
+
 # What mail is used as the sender for invites ?
 INVITE_FROM_ADDRESS=[email protected]
 

config/services.yaml

@@ -7,6 +7,8 @@ parameters:
     default_database_driver: "mysql"
     default_admin_auth_bypass: "false"
     timezone: '%env(APP_TIMEZONE)%'
+    public_calendar_enabled: '%env(default:default_public_calendar_enabled:bool:PUBLIC_CALENDAR_ENABLED)%'
+    default_public_calendar_enabled: "true"
 
 services:
     # default configuration for services in *this* file
@@ -49,6 +51,7 @@ services:
             $calDAVEnabled: "%env(bool:CALDAV_ENABLED)%"
             $cardDAVEnabled: "%env(bool:CARDDAV_ENABLED)%"
             $webDAVEnabled: "%env(bool:WEBDAV_ENABLED)%"
+            $publicCalendarEnabled: "%env(default:default_public_calendar_enabled:bool:PUBLIC_CALENDAR_ENABLED)%"
             $inviteAddress: "%env(INVITE_FROM_ADDRESS)%"
             $authMethod: "%env(AUTH_METHOD)%"
             $authRealm: "%env(AUTH_REALM)%"

src/Controller/Admin/CalendarController.php

@@ -80,17 +80,22 @@ public function calendarEdit(ManagerRegistry $doctrine, Request $request, string
             $calendarInstance->setCalendar($calendar);
         }
 
+        $isPublicCalendarEnabled = $this->getParameter('public_calendar_enabled');
+
         $form = $this->createForm(CalendarInstanceType::class, $calendarInstance, [
             'new' => !$id,
             'shared' => $calendarInstance->isShared(),
+            'public_calendar_enabled' => $isPublicCalendarEnabled
         ]);
 
         $components = explode(',', $calendarInstance->getCalendar()->getComponents());
 
         $form->get('events')->setData(in_array(Calendar::COMPONENT_EVENTS, $components));
         $form->get('todos')->setData(in_array(Calendar::COMPONENT_TODOS, $components));
         $form->get('notes')->setData(in_array(Calendar::COMPONENT_NOTES, $components));
-        $form->get('public')->setData($calendarInstance->isPublic());
+        if ($isPublicCalendarEnabled) {
+            $form->get('public')->setData($calendarInstance->isPublic());
+        }
         $form->get('principalUri')->setData(Principal::PREFIX.$username);
 
         $form->handleRequest($request);
@@ -110,7 +115,7 @@ public function calendarEdit(ManagerRegistry $doctrine, Request $request, string
                 if ($form->get('notes')->getData()) {
                     $components[] = Calendar::COMPONENT_NOTES;
                 }
-                if (true === $form->get('public')->getData()) {
+                if ($isPublicCalendarEnabled && true === $form->get('public')->getData()) {
                     $calendarInstance->setAccess(CalendarInstance::ACCESS_PUBLIC);
                 } else {
                     $calendarInstance->setAccess(CalendarInstance::ACCESS_SHAREDOWNER);
@@ -120,7 +125,7 @@ public function calendarEdit(ManagerRegistry $doctrine, Request $request, string
             }
 
             // We want to remove all shares if a calendar goes public
-            if (true === $form->get('public')->getData() && $id) {
+            if ($isPublicCalendarEnabled && true === $form->get('public')->getData() && $id) {
                 $calendarId = $calendarInstance->getCalendar()->getId();
                 $instances = $doctrine->getRepository(CalendarInstance::class)->findSharedInstancesOfInstance($calendarId, false);
                 foreach ($instances as $instance) {

src/Controller/DAVController.php

@@ -47,6 +47,13 @@ class DAVController extends AbstractController
      */
     protected $webDAVEnabled;
 
+    /**
+     * is Public Calendar enabled?
+     *
+     * @var bool
+     */
+    protected $publicCalendarEnabled;
+
     /**
      * Mail address to send mails from.
      *
@@ -135,13 +142,14 @@ class DAVController extends AbstractController
      */
     protected $server;
 
-    public function __construct(MailerInterface $mailer, BasicAuth $basicAuthBackend, IMAPAuth $IMAPAuthBackend, LDAPAuth $LDAPAuthBackend, UrlGeneratorInterface $router, EntityManagerInterface $entityManager, LoggerInterface $logger, string $publicDir, bool $calDAVEnabled = true, bool $cardDAVEnabled = true, bool $webDAVEnabled = false, ?string $inviteAddress = null, ?string $authMethod = null, ?string $authRealm = null, ?string $webdavPublicDir = null, ?string $webdavHomesDir = null, ?string $webdavTmpDir = null)
+    public function __construct(MailerInterface $mailer, BasicAuth $basicAuthBackend, IMAPAuth $IMAPAuthBackend, LDAPAuth $LDAPAuthBackend, UrlGeneratorInterface $router, EntityManagerInterface $entityManager, LoggerInterface $logger, string $publicDir, bool $calDAVEnabled = true, bool $cardDAVEnabled = true, bool $webDAVEnabled = false, bool $publicCalendarEnabled, ?string $inviteAddress = null, ?string $authMethod = null, ?string $authRealm = null, ?string $webdavPublicDir = null, ?string $webdavHomesDir = null, ?string $webdavTmpDir = null)
     {
         $this->publicDir = $publicDir;
 
         $this->calDAVEnabled = $calDAVEnabled;
         $this->cardDAVEnabled = $cardDAVEnabled;
         $this->webDAVEnabled = $webDAVEnabled;
+        $this->publicCalendarEnabled = $publicCalendarEnabled;
         $this->inviteAddress = $inviteAddress ?? null;
 
         $this->webdavPublicDir = $webdavPublicDir;
@@ -230,7 +238,7 @@ private function initServer(string $authMethod, string $authRealm = User::DEFAUL
         $this->server->addPlugin(new \Sabre\DAV\Browser\Plugin(false)); // We disable the file creation / upload / sharing in the browser
         $this->server->addPlugin(new \Sabre\DAV\Sync\Plugin());
 
-        $aclPlugin = new PublicAwareDAVACLPlugin($this->em);
+        $aclPlugin = new PublicAwareDAVACLPlugin($this->em, $this->publicCalendarEnabled);
         $aclPlugin->hideNodesFromListings = true;
 
         // Fetch admins, if any

src/Form/CalendarInstanceType.php

@@ -73,6 +73,10 @@ public function buildForm(FormBuilderInterface $builder, array $options): void
             ->add('save', SubmitType::class, [
                 'label' => 'save',
             ]);
+
+        if (!$options['public_calendar_enabled']) {
+            $builder->remove('public');
+        }
     }
 
     public function configureOptions(OptionsResolver $resolver): void
@@ -81,6 +85,7 @@ public function configureOptions(OptionsResolver $resolver): void
             'new' => false,
             'shared' => false,
             'data_class' => CalendarInstance::class,
+            'public_calendar_enabled' => true
         ]);
     }
 }

src/Plugins/PublicAwareDAVACLPlugin.php

@@ -14,9 +14,15 @@ class PublicAwareDAVACLPlugin extends \Sabre\DAVACL\Plugin
      */
     protected $em;
 
-    public function __construct(EntityManagerInterface $entityManager)
+    /**
+     * @var bool
+     */
+    protected $public_calendar_enabled;
+
+    public function __construct(EntityManagerInterface $entityManager, bool $public_enabled)
     {
         $this->em = $entityManager;
+        $this->public_calendar_enabled = $public_enabled;
     }
 
     /**
@@ -55,7 +61,7 @@ public function getAcl($node): array
 
             $calendar = $this->em->getRepository(CalendarInstance::class)->findOneById($calendarInstanceId);
 
-            if ($calendar && $calendar->isPublic()) {
+            if ($calendar && $calendar->isPublic() && $this->public_calendar_enabled) {
                 // We must add the ACL on the object itself
                 $acl[] = [
                     'principal' => '{DAV:}unauthenticated',